diff options
author | Christof Schmitt <christof.schmitt@de.ibm.com> | 2008-12-19 10:57:02 -0500 |
---|---|---|
committer | James Bottomley <James.Bottomley@HansenPartnership.com> | 2008-12-29 12:38:29 -0500 |
commit | b632ade282895562924d18b8eedd11a825f4b08c (patch) | |
tree | 3236e33c981f27fd8718cc5d432d35d621749141 /drivers/s390 | |
parent | 39eb7e9aca2a582330ddb6f1167272268e6b3965 (diff) |
[SCSI] zfcp: Remove unnecessary warning message
Remove a message that was emitted for a port that could not initially
be opened. This is a rare case when the port discovery hits an
initiator port and only confuses the user with an initator port logged
in the message. Remove the whole special case: The failed "open port"
request triggers required follow-up actions anyway.
Signed-off-by: Christof Schmitt <christof.schmitt@de.ibm.com>
Acked-by: Felix Beck <felix@linux.vnet.ibm.com>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Diffstat (limited to 'drivers/s390')
-rw-r--r-- | drivers/s390/scsi/zfcp_dbf.c | 2 | ||||
-rw-r--r-- | drivers/s390/scsi/zfcp_fsf.c | 6 |
2 files changed, 1 insertions, 7 deletions
diff --git a/drivers/s390/scsi/zfcp_dbf.c b/drivers/s390/scsi/zfcp_dbf.c index 150bd5390b1d..cb6df609953e 100644 --- a/drivers/s390/scsi/zfcp_dbf.c +++ b/drivers/s390/scsi/zfcp_dbf.c | |||
@@ -522,7 +522,7 @@ static const char *zfcp_rec_dbf_ids[] = { | |||
522 | [29] = "link down", | 522 | [29] = "link down", |
523 | [30] = "link up status read", | 523 | [30] = "link up status read", |
524 | [31] = "open port failed", | 524 | [31] = "open port failed", |
525 | [32] = "open port failed", | 525 | [32] = "", |
526 | [33] = "close port", | 526 | [33] = "close port", |
527 | [34] = "open unit failed", | 527 | [34] = "open unit failed", |
528 | [35] = "exclusive open unit failed", | 528 | [35] = "exclusive open unit failed", |
diff --git a/drivers/s390/scsi/zfcp_fsf.c b/drivers/s390/scsi/zfcp_fsf.c index 9bba56b16831..e6416f8541b0 100644 --- a/drivers/s390/scsi/zfcp_fsf.c +++ b/drivers/s390/scsi/zfcp_fsf.c | |||
@@ -1423,13 +1423,7 @@ static void zfcp_fsf_open_port_handler(struct zfcp_fsf_req *req) | |||
1423 | switch (header->fsf_status_qual.word[0]) { | 1423 | switch (header->fsf_status_qual.word[0]) { |
1424 | case FSF_SQ_INVOKE_LINK_TEST_PROCEDURE: | 1424 | case FSF_SQ_INVOKE_LINK_TEST_PROCEDURE: |
1425 | case FSF_SQ_ULP_DEPENDENT_ERP_REQUIRED: | 1425 | case FSF_SQ_ULP_DEPENDENT_ERP_REQUIRED: |
1426 | req->status |= ZFCP_STATUS_FSFREQ_ERROR; | ||
1427 | break; | ||
1428 | case FSF_SQ_NO_RETRY_POSSIBLE: | 1426 | case FSF_SQ_NO_RETRY_POSSIBLE: |
1429 | dev_warn(&req->adapter->ccw_device->dev, | ||
1430 | "Remote port 0x%016Lx could not be opened\n", | ||
1431 | (unsigned long long)port->wwpn); | ||
1432 | zfcp_erp_port_failed(port, 32, req); | ||
1433 | req->status |= ZFCP_STATUS_FSFREQ_ERROR; | 1427 | req->status |= ZFCP_STATUS_FSFREQ_ERROR; |
1434 | break; | 1428 | break; |
1435 | } | 1429 | } |