[PATCH v3 17/17] be2iscsi: Update the driver version

Driver version: 11.0.0.0 Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> --- drivers/scsi/be2iscsi/be_main.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/be2iscsi/be_main.h b/drivers/scsi/be2iscsi/be_main.h index 41c708c..16a6fd0

[PATCH v3 13/17] scsi_transport_iscsi: Add 25G and 40G speed definition

iscsi_port_speed and iscsi_port_speed_names have new entries for 25Gbps and 40Gbps link speeds. Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> --- drivers/scsi/scsi_transport_iscsi.c | 2 ++ include/scsi/iscsi_if.h | 2 ++ 2 files changed, 4 insertions(+)

[PATCH v3 15/17] be2iscsi: Fix async link event processing

-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> --- drivers/scsi/be2iscsi/be_cmds.c | 48 +++- drivers/scsi/be2iscsi/be_cmds.h | 32 +-- drivers/scsi/be2iscsi/be_iscsi.c | 2 +- 3 files changed, 34 insertions(+), 48 del

[PATCH v3 16/17] be2iscsi: Fix WRB leak in login/logout path

for posting the login-negotiated parameters back in pool. Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> Reviewed-by: Hannes Reinecke <h...@suse.com> --- drivers/scsi/be2iscsi/be_main.c | 72 - 1 file changed, 49 insertions(+),

[PATCH v3 01/17] be2iscsi: Fix soft lockup in mgmt_get_all_if_id path using bmbx

-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> Reviewed-by: Hannes Reinecke <h...@suse.com> --- drivers/scsi/be2iscsi/be_cmds.c | 60 - drivers/scsi/be2iscsi/be_mgmt.c | 32 +++--- 2 files changed, 51 insertions(+), 41 deleti

[PATCH v3 00/17] be2iscsi: driver update 11.0.0.0

This patch is generated against for-next branch. Jitendra Bhivare (17): be2iscsi: Fix soft lockup in mgmt_get_all_if_id path using bmbx be2iscsi: Fix mbox synchronization replacing spinlock with mutex be2iscsi: Fix to use atomic bit operations for tag_state be2iscsi: Fix to synchronize

[PATCH v3 04/17] be2iscsi: Fix to synchronize tag allocation using spin_lock

alloc_mcc_tag/free_mcc_tag is now done under mcc_lock spin_lock Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> --- drivers/scsi/be2iscsi/be_cmds.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/be2iscsi/be_cmds.c b/drivers/scsi/be

[PATCH v3 05/17] be2iscsi: Set mbox timeout to 30s

FW recommended timeout for all mbox command is 30s. Use msleep instead mdelay to relinquish CPU when polling for mbox completion. Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> Reviewed-by: Hannes Reinecke <h...@suse.com> --- drivers/scsi/be2iscsi/be_cmds.c | 5 +

RE: [PATCH v2 17/17] be2iscsi: Update the driver version

Please ignore patch v2 set sending v3 shortly. -Original Message- From: Jitendra Bhivare [mailto:jitendra.bhiv...@avagotech.com] Sent: Tuesday, December 22, 2015 10:46 PM To: linux-scsi@vger.kernel.org; micha...@cs.wisc.edu Cc: Jitendra Bhivare Subject: [PATCH v2 17/17] be2iscsi: Update

[PATCH v2 11/17] be2iscsi: Fix return value for MCC completion

Change return value of completed MCC EBUSY to EINVAL. Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> Reviewed-by: Hannes Reinecke <h...@suse.com> --- drivers/scsi/be2iscsi/be_cmds.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/s

[PATCH v2 10/17] be2iscsi: Add FW config validation

System crash in I+T card personality Fix to add validation for ULP in initiator mode, physical port number, and supported queue, icd, cid counts. Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> --- drivers/scsi/be2iscsi/be_main.c | 2 +- drivers/scsi/be2iscsi/be_main.h

[PATCH v2 04/17] be2iscsi: Fix to synchronize tag allocation using spin_lock

alloc_mcc_tag/free_mcc_tag is now done under mcc_lock spin_lock Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> Reviewed-by: Hannes Reinecke <h...@suse.com> --- drivers/scsi/be2iscsi/be_cmds.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/

[PATCH v2 09/17] be2iscsi: Fix to handle misconfigured optics events

Log messages for misconfigured transceivers reported by FW. Register async events that driver handles using MCC_CREATE_EXT ioctl. Errors messages for faulted/uncertified/unqualified optics are logged. Added IOCTL to get port_name to be displayed in error message. Signed-off-by: Jitendra Bhivare

[PATCH v2 14/17] be2iscsi: Fix to process 25G link speed info from FW

Async link event provides port_speed info. Cache the port_speed info and use the same to report in ISCSI_HOST_PARAM_PORT_SPEED query. Removed link status query IOCTL used to do the same. Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> --- drivers/scsi/be2iscsi/be_cmds.c

[PATCH v2 17/17] be2iscsi: Update the driver version

Driver version: 11.0.0.0 Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> --- drivers/scsi/be2iscsi/be_main.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/be2iscsi/be_main.h b/drivers/scsi/be2iscsi/be_main.h index 41c708c..16a6fd0

[PATCH v2 13/17] scsi_transport_iscsi: Add 25G and 40G speed definition

iscsi_port_speed and iscsi_port_speed_names have new entries for 25Gbps and 40Gbps link speeds. Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> --- drivers/scsi/scsi_transport_iscsi.c | 2 ++ include/scsi/iscsi_if.h | 2 ++ 2 files changed, 4 insertions(+)

[PATCH v2 08/17] be2iscsi: Fix VLAN support for IPv6 network

Configuring VLAN parameters through IPv6 interface was not supported in driver. Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> Reviewed-by: Hannes Reinecke <h...@suse.com> --- drivers/scsi/be2iscsi/be_iscsi.c | 4 1 file changed, 4 insertions(+) diff --git a/

[PATCH v2 12/17] be2iscsi: Fix IOPOLL implementation

is done only after iopoll completes. Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> Reviewed-by: Hannes Reinecke <h...@suse.com> --- drivers/scsi/be2iscsi/be_cmds.c | 2 +- drivers/scsi/be2iscsi/be_iscsi.c | 2 +- drivers/scsi/be2iscsi/be_

[PATCH v2 07/17] be2iscsi: Fix to remove shutdown entry point

Null pointer dereference in shutdown path after taking dump. Shutdown path is not needed as FW comes up clean every time during probe after issuing FUNCTION reset MBOX command. Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> Reviewed-by: Hannes Reinecke <h...

[PATCH v2 02/17] be2iscsi: Fix mbox synchronization replacing spinlock with mutex

This is second part of actual fix for soft lockup. All mbox cmds issued using BMBX and MCC are synchronized using mutex mbox_lock instead of spin_lock. Used mutex_lock_interruptible where ever possible. Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> --- drivers/scsi/be

[PATCH v2 00/17] be2iscsi: driver update 11.0.0.0

This patch is generated against for-next branch. Jitendra Bhivare (17): be2iscsi: Fix soft lockup in mgmt_get_all_if_id path using bmbx be2iscsi: Fix mbox synchronization replacing spinlock with mutex be2iscsi: Fix to synchronize tag allocation using spin_lock be2iscsi: Set mbox timeout

[PATCH v2 01/17] be2iscsi: Fix soft lockup in mgmt_get_all_if_id path using bmbx

-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> Reviewed-by: Hannes Reinecke <h...@suse.com> --- drivers/scsi/be2iscsi/be_cmds.c | 60 - drivers/scsi/be2iscsi/be_mgmt.c | 32 +++--- 2 files changed, 51 insertions(+), 41 deleti

[PATCH v2 06/17] be2iscsi: Added return value check for mgmt_get_all_if_id

Use of mutex_lock_interruptible can return -EINTR, handle and log the error. Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> Reviewed-by: Hannes Reinecke <h...@suse.com> --- drivers/scsi/be2iscsi/be_iscsi.c | 7 --- drivers/scsi/be2iscsi/be_mgmt.c | 10 ++-

[PATCH v2 15/17] be2iscsi: Fix async link event processing

-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> --- drivers/scsi/be2iscsi/be_cmds.c | 48 +++- drivers/scsi/be2iscsi/be_cmds.h | 32 +-- drivers/scsi/be2iscsi/be_iscsi.c | 2 +- 3 files changed, 34 insertions(+), 48 del

[PATCH v2 05/17] be2iscsi: Set mbox timeout to 30s

FW recommended timeout for all mbox command is 30s. Use msleep instead mdelay to relinquish CPU when polling for mbox completion. Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> Reviewed-by: Hannes Reinecke <h...@suse.com> --- drivers/scsi/be2iscsi/be_cmds.c | 5 +

[PATCH v2 16/17] be2iscsi: Fix WRB leak in login/logout path

for posting the login-negotiated parameters back in pool. Signed-off-by: Jitendra Bhivare <jitendra.bhiv...@avagotech.com> Reviewed-by: Hannes Reinecke <h...@suse.com> --- drivers/scsi/be2iscsi/be_main.c | 72 - 1 file changed, 49 insertions(+),

RE: [PATCH 10/15] be2iscsi: Add FW config validation

It stands for Dual ULP Aware. Will add a comment explaining that. Thanks, JB -Original Message- From: Hannes Reinecke [mailto:h...@suse.de] Sent: Friday, December 18, 2015 2:34 PM To: Jitendra Bhivare; linux-scsi@vger.kernel.org; micha...@cs.wisc.edu Subject: Re: [PATCH 10/15] be2iscsi

RE: [PATCH 10/15] be2iscsi: Add FW config validation

Yes, the faults are not being used, it just adds to the confusion, we just intend to get the link state change - UP or DOWN. -Original Message- From: Hannes Reinecke [mailto:h...@suse.de] Sent: Friday, December 18, 2015 2:34 PM To: Jitendra Bhivare; linux-scsi@vger.kernel.org; micha

RE: [PATCH 03/15] be2iscsi: Fix to use atomic operations for tag_state

:44 PM To: Jitendra Bhivare; linux-scsi@vger.kernel.org Subject: Re: [PATCH 03/15] be2iscsi: Fix to use atomic operations for tag_state On 12/20/15 3:01 AM, Mike Christie wrote: > On 12/20/2015 01:44 AM, Mike Christie wrote: > >>> diff --git a/drivers/scsi/be2iscsi/be_cmds.c >

[PATCH 12/15] be2iscsi: Fix IOPOLL implementation

From: Jitendra OS not responding when running 2 port traffic on 72 CPUs system. be2iscsi IRQs gets affined to CPU0 when irqbalancer is disabled. be_iopoll processing completions in BLOCK_IOPOLL_SOFTIRQ hogged CPU0. 1. Use budget to exit the polling loop.

[PATCH 11/15] be2iscsi: Fix return value for MCC completion

From: Jitendra Change return value of completed MCC EBUSY to EINVAL. Signed-off-by: Jitendra --- drivers/scsi/be2iscsi/be_cmds.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

[PATCH 10/15] be2iscsi: Add FW config validation

From: Jitendra System crash in I+T card personality Fix to add validation for ULP in initiator mode, physical port number, and supported queue, icd, cid counts. Signed-off-by: Jitendra --- drivers/scsi/be2iscsi/be_main.c |2

[PATCH 08/15] be2iscsi: Fix VLAN support for IPv6 network

From: Jitendra Configuring VLAN parameters through IPv6 interface was not supported in driver. Signed-off-by: Jitendra --- drivers/scsi/be2iscsi/be_iscsi.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git

[PATCH 09/15] be2iscsi: Fix to handle misconfigured optics events

From: Jitendra Log messages for misconfigured transceivers reported by FW. Register async events that driver handles using MCC_CREATE_EXT ioctl. Errors messages for faulted/uncertified/unqualified optics are logged. Added IOCTL to get port_name to be displayed in

[PATCH 05/15] be2iscsi: Set mbox timeout to 30s

From: Jitendra FW recommended timeout for all mbox command is 30s. Use msleep instead mdelay to relinquish CPU when polling for mbox completion. Signed-off-by: Jitendra --- drivers/scsi/be2iscsi/be_cmds.c |5 +++-- 1 files

[PATCH 04/15] be2iscsi: Fix to synchronize tag allocation using spin_lock

From: Jitendra alloc_mcc_tag/free_mcc_tag is now done under mcc_lock spin_lock Signed-off-by: Jitendra --- drivers/scsi/be2iscsi/be_cmds.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git

[PATCH 01/15] be2iscsi: Fix soft lockup in mgmt_get_all_if_id path using bmbx

From: Jitendra We are taking mbox_lock spinlock which disables pre-emption before we poll for mbox completion. Waiting there with spinlock held in excess of 20s will cause soft lockup. Actual fix is to change mbox_lock to mutex. The changes are done in phases.

[PATCH 06/15] be2iscsi: Added return value check for mgmt_get_all_if_id

From: Jitendra Use of mutex_lock_interruptible can return -EINTR, handle and log the error. Signed-off-by: Jitendra --- drivers/scsi/be2iscsi/be_iscsi.c |7 --- drivers/scsi/be2iscsi/be_mgmt.c | 10 ++ 2 files

[PATCH 02/15] be2iscsi: Fix mbox synchronization replacing spinlock with mutex

From: Jitendra This is second part of actual fix for soft lockup. All mbox cmds issued using BMBX and MCC are synchronized using mutex mbox_lock instead of spin_lock. Used mutex_lock_interruptible where ever possible. Signed-off-by: Jitendra

[PATCH 03/15] be2iscsi: Fix to use atomic operations for tag_state

From: Jitendra Replace lock based tag_state manipulations with atomic operations. Signed-off-by: Jitendra --- drivers/scsi/be2iscsi/be.h |2 +- drivers/scsi/be2iscsi/be_cmds.c | 26 -- 2 files

[PATCH 07/15] be2iscsi: Fix to remove shutdown entry point

From: Jitendra Null pointer dereference in shutdown path after taking dump. Shutdown path is not needed as FW comes up clean every time during probe after issuing FUNCTION reset MBOX command. Signed-off-by: Jitendra ---

[PATCH 00/15] be2iscsi: driver update 11.0.0.0

From: Jitendra This patch is generated against for-next branch. Jitendra (15): be2iscsi: Fix soft lockup in mgmt_get_all_if_id path using bmbx be2iscsi: Fix mbox synchronization replacing spinlock with mutex be2iscsi: Fix to use atomic operations for

[PATCH 13/15] be2iscsi: Fix to process 25G link speed info from FW

From: Jitendra Async link event provides port_speed info. Use the same to report in ISCSI_HOST_PARAM_PORT_SPEED query. Removed link status query IOCTL used to do the same. 25G and 40G are defined in kernel enum iscsi_port_speed. Fixed get_nic_conf structure

[PATCH 14/15] be2iscsi: Fix WRB leak in login/logout path

From: Jitendra Login/Logout loop was hanging after few hours. /var/log/message showed that alloc_wrb_handle() function was not able to allocate any new WRB. Sep 11 11:25:22 Jhelum10 kernel: connection32513:0: Could not send nopout Sep 11 11:25:22 Jhelum10 kernel:

RE: [PATCH 5/9] be2iscsi: Fix to handle misconfigured optics events

so sure what you are suggesting. port_name IOCTL is needed to display the error message. Will split FW config validation. Thanks, JB -Original Message- From: Hannes Reinecke [mailto:h...@suse.de] Sent: Monday, December 14, 2015 8:49 PM To: Jitendra Bhivare; linux-scsi@vger.kernel.

RE: [PATCH 6/9] be2iscsi: Fix IOPOLL implementation

: Hannes Reinecke [mailto:h...@suse.de] Sent: Monday, December 14, 2015 8:54 PM To: Jitendra Bhivare; linux-scsi@vger.kernel.org; micha...@cs.wisc.edu Subject: Re: [PATCH 6/9] be2iscsi: Fix IOPOLL implementation On 12/14/2015 07:11 AM, Jitendra Bhivare wrote: > From: Jitendra <jitendr

RE: [PATCH 1/9] be2iscsi: Fix soft lockup in mgmt_get_all_if_id path using bmbx

Message- From: Hannes Reinecke [mailto:h...@suse.de] Sent: Monday, December 14, 2015 8:34 PM To: Jitendra Bhivare; linux-scsi@vger.kernel.org; micha...@cs.wisc.edu Subject: Re: [PATCH 1/9] be2iscsi: Fix soft lockup in mgmt_get_all_if_id path using bmbx On 12/14/2015 07:10 AM, Jitendra Bhivare

[PATCH 5/9] be2iscsi: Fix to handle misconfigured optics events

From: Jitendra Log messages for misconfigured transceivers reported by FW. Register async events that driver handles using MCC_CREATE_EXT ioctl. Errors messages for faulted/uncertified/unqualified optics are logged. Added FW config validation. Signed-off-by:

[PATCH 3/9] be2iscsi: Fix to remove shutdown entry point

From: Jitendra Null pointer dereference in shutdown path after taking dump. Shutdown path is not needed as FW comes up clean every time during probe after issuing FUNCTION reset MBOX command. Signed-off-by: Jitendra ---

[PATCH 7/9] be2iscsi: Fix to process 25G link speed info from FW

From: Jitendra Async link event provides port_speed info. Use the same to report in ISCSI_HOST_PARAM_PORT_SPEED query. Removed link status query IOCTL used to do the same. 25G and 40G are defined in kernel enum iscsi_port_speed. Fixed get_nic_conf structure

[PATCH 2/9] be2iscsi: Fix mbox synchronization replacing spinlock with mutex

From: Jitendra This is second part of actual fix for soft lockup. All mbox cmds issued using BMBX and MCC are synchronized using mbox_lock. alloc_mcc_tag/free_mcc_tag is done under mcc_lock and tag_state is accessed using atomic operations. Mailbox command time

[PATCH 0/9] be2iscsi driver update to 11.0.0.0

From: Jitendra This patch is generated against for-next branch. Jitendra (9): be2iscsi: Fix soft lockup in mgmt_get_all_if_id path using bmbx be2iscsi: Fix mbox synchronization replacing spinlock with mutex be2iscsi: Fix to remove shutdown entry point

[PATCH 4/9] be2iscsi: Fix VLAN support for IPv6 network

From: Jitendra Added VLAN operations in set IPv6 address for interface. Signed-off-by: Jitendra --- drivers/scsi/be2iscsi/be_iscsi.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git

[PATCH 8/9] be2iscsi: Fix WRB leak in login/logout path

From: Jitendra Login/Logout loop was hanging after few hours. /var/log/message showed that alloc_wrb_handle() function was not able to allocate any new WRB. Sep 11 11:25:22 Jhelum10 kernel: connection32513:0: Could not send nopout Sep 11 11:25:22 Jhelum10 kernel:

[PATCH 6/9] be2iscsi: Fix IOPOLL implementation

From: Jitendra OS not responding when running 2 port traffic on 72 CPUs system. be2iscsi IRQs gets affined to CPU0 when irqbalancer is disabled. be_iopoll processing completions in BLOCK_IOPOLL_SOFTIRQ hogged CPU0. 1. Use budget to exit the polling loop in

[PATCH 9/9] be2iscsi: Update the driver version

From: Jitendra Driver version: 11.0.0.0 Signed-off-by: Jitendra --- drivers/scsi/be2iscsi/be_main.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/be2iscsi/be_main.h

<    1   2   3