mbox series

[v2,0/3] scsi: qla2xxx: fixes for driver unloading

Message ID 20200205214422.3657-1-mwilck@suse.com (mailing list archive)
Headers show
Series scsi: qla2xxx: fixes for driver unloading | expand

Message

Martin Wilck Feb. 5, 2020, 9:44 p.m. UTC
From: Martin Wilck <mwilck@suse.com>

Hello Martin, Himanshu, all,

here is v2 of the little series I first submitted on Nov 29, 2019.

Changes wrt v2:
 - Added patch 3 to set the UNLOADING flag before waiting for sessions
   to end (Roman)
 - Use test_and_set_bit() for UNLOADING (Hannes)

Martin Wilck (3):
  scsi: qla2xxx: avoid sending mailbox commands if firmware is stopped
  scsi: qla2xxx: don't shut down firmware before closing sessions
  scsi: qla2xxx: set UNLOADING before waiting for session deletion

 drivers/scsi/qla2xxx/qla_mbx.c |  3 +++
 drivers/scsi/qla2xxx/qla_os.c  | 39 +++++++++++++++++-----------------
 2 files changed, 22 insertions(+), 20 deletions(-)

Comments

Martin Wilck March 9, 2020, 4:44 p.m. UTC | #1
Hello Martin, Himanshu,

On Wed, 2020-02-05 at 22:44 +0100, mwilck@suse.com wrote:
> From: Martin Wilck <mwilck@suse.com>
> 
> Hello Martin, Himanshu, all,
> 
> here is v2 of the little series I first submitted on Nov 29, 2019.
> 
> Changes wrt v2:
>  - Added patch 3 to set the UNLOADING flag before waiting for
> sessions
>    to end (Roman)
>  - Use test_and_set_bit() for UNLOADING (Hannes)
> 
> Martin Wilck (3):
>   scsi: qla2xxx: avoid sending mailbox commands if firmware is
> stopped
>   scsi: qla2xxx: don't shut down firmware before closing sessions
>   scsi: qla2xxx: set UNLOADING before waiting for session deletion
> 
>  drivers/scsi/qla2xxx/qla_mbx.c |  3 +++
>  drivers/scsi/qla2xxx/qla_os.c  | 39 +++++++++++++++++---------------
> --
>  2 files changed, 22 insertions(+), 20 deletions(-)
> 

what about this series? Will it be merged for 5.7? It got positive
reviews from Daniel, and no negative ones so far, and it still applies
cleanly to 5.7/scsi-queue.

Thanks,
Martin W.
Martin K. Petersen March 11, 2020, 2:44 a.m. UTC | #2
Martin,

> what about this series? Will it be merged for 5.7? It got positive
> reviews from Daniel, and no negative ones so far, and it still applies
> cleanly to 5.7/scsi-queue.

Still waiting for feedback from Marvell on these changes (and your other
qla2xxx patch).