[BNX2]: Fix netdev watchdog on 5708. (7947b20e) · Commits · 戴 / test

There's a bug in the driver that only initializes half of the context memory on the 5708. Surprisingly, this works most of the time except for some occasional netdev watchdogs when sending a lot of 64-byte packets. The fix is to add the missing code to initialize the 2nd halves of all context memory. Signed-off-by:

Michael Chan <mchan@broadcom.com> Acked-by:

Jeff Garzik <jeff@garzik.org> Signed-off-by:

David S. Miller <davem@davemloft.net>

drivers/net/bnx2.c

+13 −8

Original line number	Diff line number	Diff line
		@@ -1811,6 +1811,7 @@ bnx2_init_context(struct bnx2 *bp)
		vcid = 96;
		while (vcid) {
		u32 vcid_addr, pcid_addr, offset;
		int i;

		vcid--;

		@@ -1831,18 +1832,22 @@ bnx2_init_context(struct bnx2 *bp)
		pcid_addr = vcid_addr;
		}

		for (i = 0; i < (CTX_SIZE / PHY_CTX_SIZE); i++) {
		vcid_addr += (i << PHY_CTX_SHIFT);
		pcid_addr += (i << PHY_CTX_SHIFT);

		REG_WR(bp, BNX2_CTX_VIRT_ADDR, 0x00);
		REG_WR(bp, BNX2_CTX_PAGE_TBL, pcid_addr);

		/* Zero out the context. */
		for (offset = 0; offset < PHY_CTX_SIZE; offset += 4) {
		for (offset = 0; offset < PHY_CTX_SIZE; offset += 4)
		CTX_WR(bp, 0x00, offset, 0);
		}

		REG_WR(bp, BNX2_CTX_VIRT_ADDR, vcid_addr);
		REG_WR(bp, BNX2_CTX_PAGE_TBL, pcid_addr);
		}
		}
		}

		static int
		bnx2_alloc_bad_rbuf(struct bnx2 *bp)

Admin message