Antw: [PATCH 11/16] BNX2I: Added return code check for chip kwqe submission request

2010-11-10 Thread Ulrich Windl
>>> "Eddie Wai" schrieb am 11.11.2010 um 00:04 in Nachricht <1289430297-30221-12-git-send-email-eddie@broadcom.com>: [...] > diff --git a/drivers/scsi/bnx2i/bnx2i.h b/drivers/scsi/bnx2i/bnx2i.h > index d153607..72feb6c 100644 > --- a/drivers/scsi/bnx2i/bnx2i.h > +++ b/drivers/scsi/bnx2i/bn

[PATCH 05/16] BNX2I: Modified the bnx2i stop path to compensate for in progress ops

2010-11-10 Thread Eddie Wai
The stop path has been augmented to wait a max of 10s for all in progress offload and destroy activities to complete before proceeding to terminate all active connections (via iscsid or forcefully). Note that any new offload and destroy requests are now blocked and return to the caller immediately

[PATCH 14/16] BNX2I: Allow to abort the connection if connect request times out

2010-11-10 Thread Eddie Wai
In the situation where the connect completion response arrives after the connect request has already timed out, the connection was not being aborted but only the resource was being freed. This creates a problem for 5771X (10g) as the chip flags this with an assertion. This change will properly ab

[PATCH 16/16] BNX2I: Updated version to 2.6.2.2

2010-11-10 Thread Eddie Wai
Signed-off-by: Eddie Wai --- drivers/scsi/bnx2i/bnx2i_init.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/bnx2i/bnx2i_init.c b/drivers/scsi/bnx2i/bnx2i_init.c index 131ef80..99ca605 100644 --- a/drivers/scsi/bnx2i/bnx2i_init.c +++ b/drivers/scsi/bnx2i/b

[PATCH 02/16] BNX2I: Added fix for NOP-Out response panic from unsolicited NOP-In

2010-11-10 Thread Eddie Wai
The patch fixes the following situations where NOP-Out pkt is called for: - local unsolicited NOP-Out requests (requesting no NOP-In response) - local NOP-Out responses to unsolicited NOP-In requests kernel panic is observed due to double session spin_lock requests; one in the bnx2i_process_nopin_

[PATCH 15/16] BNX2I: Updated copyright and maintainer info

2010-11-10 Thread Eddie Wai
Signed-off-by: Eddie Wai --- drivers/scsi/bnx2i/57xx_iscsi_constants.h |3 ++- drivers/scsi/bnx2i/57xx_iscsi_hsi.h |3 ++- drivers/scsi/bnx2i/bnx2i.h|3 ++- drivers/scsi/bnx2i/bnx2i_hwi.c|3 ++- drivers/scsi/bnx2i/bnx2i_init.c |3 ++-

[PATCH 11/16] BNX2I: Added return code check for chip kwqe submission request

2010-11-10 Thread Eddie Wai
Added the handling for cases when a chip request is made to the CNIC module but the hardware is not ready to accept. This would lead to many unnecessary wait timeouts. This code adds check in the connect establishment and destruction path. Signed-off-by: Eddie Wai --- drivers/scsi/bnx2i/bnx2i.h

[PATCH 08/16] BNX2I: Added mutex lock protection to conn_get_param

2010-11-10 Thread Eddie Wai
Added net_dev mutex lock protection before accessing the csk parameters. Signed-off-by: Eddie Wai Reviewed-by: Benjamin Li --- drivers/scsi/bnx2i/bnx2i_iscsi.c | 17 + 1 files changed, 13 insertions(+), 4 deletions(-) diff --git a/drivers/scsi/bnx2i/bnx2i_iscsi.c b/drivers/sc

[PATCH 12/16] BNX2I: Added feature to silently drop NOPOUT request

2010-11-10 Thread Eddie Wai
In the case the chip is undergoing different invasive operation which requires a chip reset, all NOPOUT request during this period must be silently dropped from queuing to the hardware. This patch will respond to libiscsi immediately with sent success. Since the request was not actually sent, the

[PATCH 09/16] BNX2I: Removed the dynamic registration of CNIC

2010-11-10 Thread Eddie Wai
The code no longer needs to dynamically register and unregister the CNIC device. The CNIC device will be kept registered until module unload. Signed-off-by: Eddie Wai Reviewed-by: Michael Chan --- drivers/scsi/bnx2i/bnx2i.h |3 - drivers/scsi/bnx2i/bnx2i_init.c | 82 --

[PATCH 03/16] BNX2I: Fixed the endian bug in the TMF LUN cmd send

2010-11-10 Thread Eddie Wai
Added a be32_to_cpu call for the TMF LUN wqe. Signed-off-by: Eddie Wai --- drivers/scsi/bnx2i/bnx2i_hwi.c |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/bnx2i/bnx2i_hwi.c b/drivers/scsi/bnx2i/bnx2i_hwi.c index 32cf930..c9a3c0f 100644 --- a/drivers/scsi

[PATCH 07/16] BNX2I: Fixed the remote TCP RST handling for the 570X (1g)

2010-11-10 Thread Eddie Wai
Modified the handling of the remote TCP RST code so the chip can now flush the tx pipe accordingly upon a remote TCP RST reception. Signed-off-by: Eddie Wai --- drivers/scsi/bnx2i/bnx2i_hwi.c | 11 --- drivers/scsi/bnx2i/bnx2i_iscsi.c |8 +++- 2 files changed, 11 insertions(+

[PATCH 06/16] BNX2I: Added code to handle the binding of an existing connection

2010-11-10 Thread Eddie Wai
This is the case when iscsid gets re-launched due to features like iSCSI boot which requires the daemon to re-launch due to pivot root. If the code detected the connection had an existing endpoint, the old endpoint must get cleaned up. Signed-off-by: Eddie Wai Acked-by: Anil Veerabhadrappa ---

[PATCH 13/16] BNX2I: Cleaned up various error conditions in ep_connect/disconnect

2010-11-10 Thread Eddie Wai
Various error conditions inside ep_connect and ep_disconnect were either not being handled or not being handled correctly. This patch fixes all those issues. Signed-off-by: Eddie Wai Acked-by: Anil Veerabhadrappa --- drivers/scsi/bnx2i/bnx2i_iscsi.c | 67 ++---

[PATCH 04/16] BNX2I: Updated the handling of NETEVENTs to alleviate recovery

2010-11-10 Thread Eddie Wai
The NETDEV_GOING_DOWN handling will no longer request to put all active sessions into recovery mode until the NETDEV_DOWN event. This will eliminate the unnecessary recovery request to iscsid in the NETDEV_GOING_DOWN->NETDEV_UNREGISTER path which will remove the host anyway. Signed-off-by: Eddie W

[PATCH 10/16] BNX2I: Fixed a cid leak issue for 5771X (10g)

2010-11-10 Thread Eddie Wai
A cid leak issue was found when the connect destroy request exceeded the driver's disconnection timeout. This will lead to a cid resource leak issue. The fix is to allow the cid cleanup even when this happens. Signed-off-by: Eddie Wai Acked-by: Anil Veerabhadrappa Reviewed-by: Michael Chan ---

[PATCH 00/16] BNX2I: Patch set to fix various bug fixes

2010-11-10 Thread Eddie Wai
Hello, This patch set contains various bug fixes as categorized in the following: - scsi command handling (nop-ins, tmfs) - various connection clean up bug fixes Your comments are grealy appreciated. Please review, thanks. Eddie Eddie Wai (16): BNX2I: Fixed bugs in the handling of unsolicite

[PATCH 01/16] BNX2I: Fixed bugs in the handling of unsolicited NOP-Ins

2010-11-10 Thread Eddie Wai
Unsolicited NOP-Ins are placed in the receive queue of the hardware which requires to be read out regardless if the receive pipe is suspended or not. This patch adds the disposal of this RQ element under this condition. Also fixed the bug in the unsolicited NOP-In handling routine which checks for