diff options
author | Andreas Herrmann <aherrman@de.ibm.com> | 2005-06-13 07:13:45 -0400 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.(none)> | 2005-06-13 22:23:37 -0400 |
commit | 66c8684abf80a9999392d639627afea80ac62e06 (patch) | |
tree | 7c6aec32754b0fba3986289263a8bd85a6ea1b77 /drivers/s390 | |
parent | d6cbbad7296538b6a38c0fe36e6ecf67f1e600a7 (diff) |
[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/s390')
-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) | |||
1753 | ct_iu_req = zfcp_sg_to_address(ct->req); | 1753 | ct_iu_req = zfcp_sg_to_address(ct->req); |
1754 | ct_iu_resp = zfcp_sg_to_address(ct->resp); | 1754 | ct_iu_resp = zfcp_sg_to_address(ct->resp); |
1755 | 1755 | ||
1756 | if ((ct->status != 0) || zfcp_check_ct_response(&ct_iu_resp->header)) { | 1756 | if (ct->status != 0) |
1757 | goto failed; | ||
1758 | |||
1759 | if (zfcp_check_ct_response(&ct_iu_resp->header)) { | ||
1757 | /* FIXME: do we need some specific erp entry points */ | 1760 | /* FIXME: do we need some specific erp entry points */ |
1758 | atomic_set_mask(ZFCP_STATUS_PORT_INVALID_WWPN, &port->status); | 1761 | atomic_set_mask(ZFCP_STATUS_PORT_INVALID_WWPN, &port->status); |
1759 | goto failed; | 1762 | goto failed; |