Thread (13 messages) 13 messages, 4 authors, 2011-10-27

Re: [PATCH net-next] bnx2x: Disable LRO on FCoE or iSCSI boot device

From: David Miller <davem@davemloft.net>
Date: 2011-10-19 20:47:08

From: "Michael Chan" <mchan@broadcom.com>
Date: Wed, 19 Oct 2011 13:12:52 -0700
On Wed, 2011-10-19 at 13:06 -0700, David Miller wrote:
quoted
From: "Michael Chan" <mchan@broadcom.com>
Date: Thu, 13 Oct 2011 20:38:01 -0700
quoted
From: Dmitry Kravkov <redacted>

For an FCoE or iSCSI boot device, the networking side must stay "up" all
the time.  Otherwise, the FCoE/iSCSI interface driven by bnx2i/bnx2fc
will be reset and we'll lose the root file system.

If LRO is enabled, scripts that enable IP forwarding or bridging will
disable LRO and cause the device to be reset.  Disabling LRO on these
boot devices will prevent the reset.

Signed-off-by: Dmitry Kravkov <redacted>
Signed-off-by: Michael Chan <mchan@broadcom.com>
You're still going to have bugs after this.

What if you get a FIFO overflow or other error condition which requires
a chip reset?  You'll lose the root filesystem.
That would be no different than a scsi driver experiencing fatal errors,
wouldn't it?
It's not fatal if you can bring the chip back up after the reset
because this is networking.

These things are protocols, built on top of networking technology,
with retransmits, handshakes, and all sorts of features designed
to provide reliability.

Things like a LRO change ought to be completely transparent.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help