Commit 617757de authored by Maurizio Lombardi's avatar Maurizio Lombardi Committed by Martin K. Petersen

bnx2fc: bnx2fc_eh_abort(): fix wrong return code.

If the link is not ready, the bnx2fc_eh_abort() function should return
FAILED.
Signed-off-by: default avatarMaurizio Lombardi <mlombard@redhat.com>
Reviewed-by: default avatarEwan Milne <emilne@redhat.com>
Acked-by: default avatarChad Dupuis <chad.dupuis@qlogic.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent d4faee08
...@@ -1104,8 +1104,7 @@ int bnx2fc_eh_abort(struct scsi_cmnd *sc_cmd) ...@@ -1104,8 +1104,7 @@ int bnx2fc_eh_abort(struct scsi_cmnd *sc_cmd)
struct bnx2fc_cmd *io_req; struct bnx2fc_cmd *io_req;
struct fc_lport *lport; struct fc_lport *lport;
struct bnx2fc_rport *tgt; struct bnx2fc_rport *tgt;
int rc = FAILED; int rc;
rc = fc_block_scsi_eh(sc_cmd); rc = fc_block_scsi_eh(sc_cmd);
if (rc) if (rc)
...@@ -1114,7 +1113,7 @@ int bnx2fc_eh_abort(struct scsi_cmnd *sc_cmd) ...@@ -1114,7 +1113,7 @@ int bnx2fc_eh_abort(struct scsi_cmnd *sc_cmd)
lport = shost_priv(sc_cmd->device->host); lport = shost_priv(sc_cmd->device->host);
if ((lport->state != LPORT_ST_READY) || !(lport->link_up)) { if ((lport->state != LPORT_ST_READY) || !(lport->link_up)) {
printk(KERN_ERR PFX "eh_abort: link not ready\n"); printk(KERN_ERR PFX "eh_abort: link not ready\n");
return rc; return FAILED;
} }
tgt = (struct bnx2fc_rport *)&rp[1]; tgt = (struct bnx2fc_rport *)&rp[1];
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment