From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael Chan" Subject: Re: [PATCH net-next] bnx2x: Disable LRO on FCoE or iSCSI boot device Date: Fri, 14 Oct 2011 13:59:58 -0700 Message-ID: <1318625998.9304.8.camel@nseg_linux_HP1.broadcom.com> References: <1318563481-19631-1-git-send-email-mchan@broadcom.com> <4E9855EC.1020509@hp.com> <4E985DE8.3090308@hp.com> <4E9898F6.6000302@intel.com> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Cc: "'Rick Jones'" , "'davem@davemloft.net'" , "'netdev@vger.kernel.org'" , "Dmitry Kravkov" , "Eilon Greenstein" , eddie.wai@broadcom.com To: "John Fastabend" Return-path: Received: from mms2.broadcom.com ([216.31.210.18]:2830 "EHLO mms2.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754057Ab1JNVJs (ORCPT ); Fri, 14 Oct 2011 17:09:48 -0400 In-Reply-To: <4E9898F6.6000302@intel.com> Sender: netdev-owner@vger.kernel.org List-ID: On Fri, 2011-10-14 at 13:17 -0700, John Fastabend wrote: > On 10/14/2011 9:15 AM, Michael Chan wrote: > > Rick Jones wrote: > > > >> On 10/14/2011 08:53 AM, Michael Chan wrote: > >>> Rick Jones wrote: > >>> > >>>> Is this perhaps saying that a bnx2x-driven device being used for > >>>> FCoE or iSCSI boot must not permit *any* run-time configuration > >>>> change which leads to a NIC reset? > >>>> > >>> > >>> That is right. Unless you have a multipath configuration with > >> multiple > >>> ports, then you can reset one port at a time. > >> > >> So, should there also be a "cnic_boot_device" check in many of the > >> "capital letter" ethtool paths? > >> > > > > If the user is doing ethtool configuration changes or device shutdown, > > it is more obvious what the consequence will be. The user may also be > > careful to do it on a multipath setup. > > > > The reset caused by the auto turn-off of LRO when you enable > > ip_forward or bridging will not be obvious to the user. In addition, > > all devices with LRO turned on will be reset at the same time so even > > multipath will not survive. > > > > But after the reset the device should login and SCSI layer should > handle retries. So I don't see why this is a problem. Why do we > need to handle this any different from any other link events? > During a link down event, the iSCSI state does not get reset. When link comes back up quickly enough, there should be just some retransmissions and everything should recover. The root file system won't tolerate a chip reset that will reset the iSCSI state.