diff options
| author |   Andreas Herrmann <aherrman@de.ibm.com> | 2005-06-13 13:13:45 +0200 |
|---|---|---|
| committer |   James Bottomley <jejb@mulgrave.(none)> | 2005-06-13 21:23:37 -0500 |
| commit | 66c8684abf80a9999392d639627afea80ac62e06 (patch) | |
| tree | 7c6aec32754b0fba3986289263a8bd85a6ea1b77 /drivers | |
| parent | [SCSI] aic7xxx: clean up eisa support (diff) | |
| download | linux-dev-66c8684abf80a9999392d639627afea80ac62e06.tar.xz linux-dev-66c8684abf80a9999392d639627afea80ac62e06.zip | |
[SCSI] zfcp: fix wrong handling of failed requests for GID_PN command
Fixes the handling of failed requests for GID_PN nameserver command:
Set ZFCP_STATUS_PORT_INVALID_WWPN only if indicated by response
payload for GID_PN nameserver command and not if fsf request fails.
Signed-off-by: Andreas Herrmann <aherrman@de.ibm.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/s390/scsi/zfcp_aux.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/s390/scsi/zfcp_aux.c b/drivers/s390/scsi/zfcp_aux.c index 52fc1d789066..6bb4d332b474 100644 --- a/drivers/s390/scsi/zfcp_aux.c +++ b/drivers/s390/scsi/zfcp_aux.c @@ -1753,7 +1753,10 @@ static void zfcp_ns_gid_pn_handler(unsigned long data) ct_iu_req = zfcp_sg_to_address(ct->req); ct_iu_resp = zfcp_sg_to_address(ct->resp); - if ((ct->status != 0) || zfcp_check_ct_response(&ct_iu_resp->header)) { + if (ct->status != 0) + goto failed; + + if (zfcp_check_ct_response(&ct_iu_resp->header)) { /* FIXME: do we need some specific erp entry points */ atomic_set_mask(ZFCP_STATUS_PORT_INVALID_WWPN, &port->status); goto failed; |