scsi: fnic: Kill 'exclude_id' argument to fnic_cleanup_io()
authorHannes Reinecke <hare@suse.de>
Thu, 29 Apr 2021 12:25:15 +0000 (14:25 +0200)
committerMartin K. Petersen <martin.petersen@oracle.com>
Mon, 3 May 2021 02:47:42 +0000 (22:47 -0400)
'exclude_id' is always SCSI_NO_TAG which will never be reached when
traversing the list of tags.

Link: https://lore.kernel.org/r/20210429122517.39659-2-hare@suse.de
Reviewed-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/fnic/fnic_scsi.c

index e619a82..f41d1b1 100644 (file)
@@ -102,7 +102,7 @@ static const char *fnic_fcpio_status_to_str(unsigned int status)
        return fcpio_status_str[status];
 }
 
-static void fnic_cleanup_io(struct fnic *fnic, int exclude_id);
+static void fnic_cleanup_io(struct fnic *fnic);
 
 static inline spinlock_t *fnic_io_lock_hash(struct fnic *fnic,
                                            struct scsi_cmnd *sc)
@@ -638,7 +638,7 @@ static int fnic_fcpio_fw_reset_cmpl_handler(struct fnic *fnic,
        atomic64_inc(&reset_stats->fw_reset_completions);
 
        /* Clean up all outstanding io requests */
-       fnic_cleanup_io(fnic, SCSI_NO_TAG);
+       fnic_cleanup_io(fnic);
 
        atomic64_set(&fnic->fnic_stats.fw_stats.active_fw_reqs, 0);
        atomic64_set(&fnic->fnic_stats.io_stats.active_ios, 0);
@@ -1361,7 +1361,7 @@ int fnic_wq_copy_cmpl_handler(struct fnic *fnic, int copy_work_to_do)
        return wq_work_done;
 }
 
-static void fnic_cleanup_io(struct fnic *fnic, int exclude_id)
+static void fnic_cleanup_io(struct fnic *fnic)
 {
        int i;
        struct fnic_io_req *io_req;
@@ -1372,9 +1372,6 @@ static void fnic_cleanup_io(struct fnic *fnic, int exclude_id)
        struct fnic_stats *fnic_stats = &fnic->fnic_stats;
 
        for (i = 0; i < fnic->fnic_max_tag_id; i++) {
-               if (i == exclude_id)
-                       continue;
-
                io_lock = fnic_io_lock_tag(fnic, i);
                spin_lock_irqsave(io_lock, flags);
                sc = scsi_host_find_tag(fnic->lport->host, i);