On Wed, Feb 05, 2020 at 10:44:22PM +0100, mwilck@xxxxxxxx wrote: > From: Martin Wilck <mwilck@xxxxxxxx> > > The purpose of the UNLOADING flag is to avoid port login procedures > to continue when a controller is in the process of shutting down. > It makes sense to set this flag before starting session teardown. > The only operations that must be able to continue are LOGO, PRLO, > and the like. > > Furthermore, use atomic test_and_set_bit() to avoid the shutdown > being run multiple times in parallel. In qla2x00_disable_board_on_pci_error(), > the test for UNLOADING is postponed until after the check for an already > disabled PCI board. > > Signed-off-by: Martin Wilck <mwilck@xxxxxxxx> > --- > drivers/scsi/qla2xxx/qla_os.c | 25 +++++++++++-------------- > 1 file changed, 11 insertions(+), 14 deletions(-) > Hi Martin, Reviewed-by: Roman Bolshakov <r.bolshakov@xxxxxxxxx> Thanks, Roman