From: Christof Schmitt <christof.schmitt@xxxxxxxxxx> 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@xxxxxxxxxx> Acked-by: Felix Beck <felix@xxxxxxxxxxxxxxxxxx> --- drivers/s390/scsi/zfcp_dbf.c | 2 +- drivers/s390/scsi/zfcp_fsf.c | 6 ------ 2 files changed, 1 insertion(+), 7 deletions(-) --- a/drivers/s390/scsi/zfcp_dbf.c 2008-12-19 14:33:56.000000000 +0100 +++ b/drivers/s390/scsi/zfcp_dbf.c 2008-12-19 14:34:09.000000000 +0100 @@ -519,7 +519,7 @@ static const char *zfcp_rec_dbf_ids[] = [29] = "link down", [30] = "link up status read", [31] = "open port failed", - [32] = "open port failed", + [32] = "", [33] = "close port", [34] = "open unit failed", [35] = "exclusive open unit failed", --- a/drivers/s390/scsi/zfcp_fsf.c 2008-12-19 14:33:56.000000000 +0100 +++ b/drivers/s390/scsi/zfcp_fsf.c 2008-12-19 14:34:09.000000000 +0100 @@ -1420,13 +1420,7 @@ static void zfcp_fsf_open_port_handler(s switch (header->fsf_status_qual.word[0]) { case FSF_SQ_INVOKE_LINK_TEST_PROCEDURE: case FSF_SQ_ULP_DEPENDENT_ERP_REQUIRED: - req->status |= ZFCP_STATUS_FSFREQ_ERROR; - break; case FSF_SQ_NO_RETRY_POSSIBLE: - dev_warn(&req->adapter->ccw_device->dev, - "Remote port 0x%016Lx could not be opened\n", - (unsigned long long)port->wwpn); - zfcp_erp_port_failed(port, 32, req); req->status |= ZFCP_STATUS_FSFREQ_ERROR; break; } -- -- To unsubscribe from this list: send the line "unsubscribe linux-s390" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html