scsi: fnic: Correcting rport check location in fnic_queuecommand_lck
authorSatish Kharat <satishkh@cisco.com>
Wed, 16 Nov 2016 08:54:01 +0000 (00:54 -0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 17 Nov 2016 01:41:49 +0000 (20:41 -0500)
Signed-off-by: Satish Kharat <satishkh@cisco.com>
Signed-off-by: Sesidhar Baddela <sebaddel@cisco.com>
Reviewed-by: Tomas Henzl <thenzl@redhat.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/fnic/fnic_scsi.c

index bfaba069937f46ceb242e1bfb94c5997f08fcaa8..2544a37ece0afdc19742d01954487f87dd4c3c5c 100644 (file)
@@ -441,30 +441,38 @@ static int fnic_queuecommand_lck(struct scsi_cmnd *sc, void (*done)(struct scsi_
        unsigned long ptr;
        spinlock_t *io_lock = NULL;
        int io_lock_acquired = 0;
+       struct fc_rport_libfc_priv *rp;
 
        if (unlikely(fnic_chk_state_flags_locked(fnic, FNIC_FLAGS_IO_BLOCKED)))
                return SCSI_MLQUEUE_HOST_BUSY;
 
        rport = starget_to_rport(scsi_target(sc->device));
+       if (!rport) {
+               FNIC_SCSI_DBG(KERN_DEBUG, fnic->lport->host,
+                               "returning DID_NO_CONNECT for IO as rport is NULL\n");
+               sc->result = DID_NO_CONNECT << 16;
+               done(sc);
+               return 0;
+       }
+
        ret = fc_remote_port_chkready(rport);
        if (ret) {
+               FNIC_SCSI_DBG(KERN_DEBUG, fnic->lport->host,
+                               "rport is not ready\n");
                atomic64_inc(&fnic_stats->misc_stats.rport_not_ready);
                sc->result = ret;
                done(sc);
                return 0;
        }
 
-       if (rport) {
-               struct fc_rport_libfc_priv *rp = rport->dd_data;
-
-               if (!rp || rp->rp_state != RPORT_ST_READY) {
-                       FNIC_SCSI_DBG(KERN_DEBUG, fnic->lport->host,
+       rp = rport->dd_data;
+       if (!rp || rp->rp_state != RPORT_ST_READY) {
+               FNIC_SCSI_DBG(KERN_DEBUG, fnic->lport->host,
                                "returning DID_NO_CONNECT for IO as rport is removed\n");
-                       atomic64_inc(&fnic_stats->misc_stats.rport_not_ready);
-                       sc->result = DID_NO_CONNECT<<16;
-                       done(sc);
-                       return 0;
-               }
+               atomic64_inc(&fnic_stats->misc_stats.rport_not_ready);
+               sc->result = DID_NO_CONNECT<<16;
+               done(sc);
+               return 0;
        }
 
        if (lp->state != LPORT_ST_READY || !(lp->link_up))