Commit 2601d8a0 authored Jun 05, 2010 by Matt Carlson Committed by David S. Miller Jun 06, 2010

tg3: Off-by-one error in RSS setup



The driver was incorrectly programming the indirection table such that
rx traffic intended for the second ring went to the first ring, rx
traffic intended for the third ring went to the second ring, etc.  This
patch changes the code so that rx traffic is diverted to the proper
ring.

Signed-off-by: Matt Carlson <mcarlson@broadcom.com>
Signed-off-by: Michael Chan <mchan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

parent b28f6428

drivers/net/tg3.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -8228,7 +8228,7 @@ static int tg3_reset_hw(struct tg3 *tp, int reset_phy)
		for (i = 0; i < TG3_RSS_INDIR_TBL_SIZE; i++) {
		int idx = i % sizeof(val);

		ent[idx] = i % (tp->irq_cnt - 1);
		ent[idx] = (i % (tp->irq_cnt - 1)) + 1;
		if (idx == sizeof(val) - 1) {
		tw32(reg, val);
		reg += 4;

Admin message