Re: [PATCH] scsi_transport_fc: Also check for NOTPRESENT in fc_remote_port_add()

2017-09-25 Thread Martin K. Petersen
Hannes, > During failover there is a small race window between > fc_remote_port_add() and fc_timeout_deleted_rport(); the latter drops > the lock after setting the port to NOTPRESENT, so if > fc_remote_port_add() is called right at that time it will fail to > detect the existing rport and

Re: [PATCH] scsi_transport_fc: Also check for NOTPRESENT in fc_remote_port_add()

2017-09-25 Thread Johannes Thumshirn
Looks good, Reviewed-by: Johannes Thumshirn -- Johannes Thumshirn Storage jthumsh...@suse.de+49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: Felix Imendörffer, Jane Smithard, Graham

[PATCH] scsi_transport_fc: Also check for NOTPRESENT in fc_remote_port_add()

2017-09-25 Thread Hannes Reinecke
During failover there is a small race window between fc_remote_port_add() and fc_timeout_deleted_rport(); the latter drops the lock after setting the port to NOTPRESENT, so if fc_remote_port_add() is called right at that time it will fail to detect the existing rport and happily adding a new