diff options
author | Christof Schmitt <christof.schmitt@de.ibm.com> | 2010-07-08 03:53:08 -0400 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2010-07-28 10:46:39 -0400 |
commit | 835dc29887073eec7817559a07558f955383d099 (patch) | |
tree | 721268f31d4ad6d55a3733adcddce57efa793c62 /drivers/s390/scsi/zfcp_scsi.c | |
parent | 5a7de559b4e0169ff4cfca654b4e4f0014996e57 (diff) |
[SCSI] zfcp: Use forced_reopen in terminate_rport_io callback
When running in non-NPIV mode, the port_reopen in terminate_rport_io
might succeed even though the remote port is not available. If the
same port connection is held open from another operating system, the
reopen is only a virtual operation and might not hit the SAN. Fix this
by changing the call to forced_reopen that forces a logout/login
operation in the SAN.
Reviewed-by: Swen Schillig <swen@vnet.ibm.com>
Signed-off-by: Christof Schmitt <christof.schmitt@de.ibm.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/s390/scsi/zfcp_scsi.c')
-rw-r--r-- | drivers/s390/scsi/zfcp_scsi.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/drivers/s390/scsi/zfcp_scsi.c b/drivers/s390/scsi/zfcp_scsi.c index 153f69b26e78..4cab33a2f7be 100644 --- a/drivers/s390/scsi/zfcp_scsi.c +++ b/drivers/s390/scsi/zfcp_scsi.c | |||
@@ -506,8 +506,10 @@ static void zfcp_set_rport_dev_loss_tmo(struct fc_rport *rport, u32 timeout) | |||
506 | * @rport: The FC rport where to teminate I/O | 506 | * @rport: The FC rport where to teminate I/O |
507 | * | 507 | * |
508 | * Abort all pending SCSI commands for a port by closing the | 508 | * Abort all pending SCSI commands for a port by closing the |
509 | * port. Using a reopen avoiding a conflict with a shutdown | 509 | * port. Using a reopen avoids a conflict with a shutdown |
510 | * overwriting a reopen. | 510 | * overwriting a reopen. The "forced" ensures that a disappeared port |
511 | * is not opened again as valid due to the cached plogi data in | ||
512 | * non-NPIV mode. | ||
511 | */ | 513 | */ |
512 | static void zfcp_scsi_terminate_rport_io(struct fc_rport *rport) | 514 | static void zfcp_scsi_terminate_rport_io(struct fc_rport *rport) |
513 | { | 515 | { |
@@ -519,7 +521,7 @@ static void zfcp_scsi_terminate_rport_io(struct fc_rport *rport) | |||
519 | port = zfcp_get_port_by_wwpn(adapter, rport->port_name); | 521 | port = zfcp_get_port_by_wwpn(adapter, rport->port_name); |
520 | 522 | ||
521 | if (port) { | 523 | if (port) { |
522 | zfcp_erp_port_reopen(port, 0, "sctrpi1", NULL); | 524 | zfcp_erp_port_forced_reopen(port, 0, "sctrpi1", NULL); |
523 | put_device(&port->dev); | 525 | put_device(&port->dev); |
524 | } | 526 | } |
525 | } | 527 | } |