scsi: Remove scsi_print_command when calling abort
authorHannes Reinecke <hare@suse.de>
Fri, 24 Oct 2014 12:26:58 +0000 (14:26 +0200)
committerChristoph Hellwig <hch@lst.de>
Wed, 12 Nov 2014 10:16:04 +0000 (11:16 +0100)
Calling scsi_print_command should not be necessary during abort;
if the information is required one should enable scsi logging.

Signed-off-by: Hannes Reinecke <hare@suse.de>
Reviewed-by: Robert Elliott <elliott@hp.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
drivers/scsi/53c700.c
drivers/scsi/NCR5380.c
drivers/scsi/arm/fas216.c
drivers/scsi/atari_NCR5380.c
drivers/scsi/ps3rom.c
drivers/scsi/stex.c
drivers/scsi/sun3_NCR5380.c

index 179a24ec75613704da0267d0bae698b16904637f..474cc6dc98e2a1a29a55aadaa28212de11d3e79c 100644 (file)
@@ -1906,9 +1906,7 @@ NCR_700_abort(struct scsi_cmnd * SCp)
 {
        struct NCR_700_command_slot *slot;
 
-       scmd_printk(KERN_INFO, SCp,
-               "New error handler wants to abort command\n\t");
-       scsi_print_command(SCp);
+       scmd_printk(KERN_INFO, SCp, "abort command\n");
 
        slot = (struct NCR_700_command_slot *)SCp->host_scribble;
 
index 45da3c8233226f78f49e7a2609707f9eba43d3bd..50873bb6b7ee791787de361e6e142b60c9f8b31d 100644 (file)
@@ -2666,9 +2666,8 @@ static int NCR5380_abort(Scsi_Cmnd * cmd) {
        struct Scsi_Host *instance = cmd->device->host;
        struct NCR5380_hostdata *hostdata = (struct NCR5380_hostdata *) instance->hostdata;
        Scsi_Cmnd *tmp, **prev;
-       
-       printk(KERN_WARNING "scsi%d : aborting command\n", instance->host_no);
-       scsi_print_command(cmd);
+
+       scmd_printk(KERN_WARNING, cmd, "aborting command\n");
 
        NCR5380_print_status(instance);
 
index d2581cb41ec8689fe45973ff3b24364101a1e936..e64c3af7c1a0166d47ffe18dba3b5c2754385c74 100644 (file)
@@ -2423,14 +2423,11 @@ int fas216_eh_abort(struct scsi_cmnd *SCpnt)
 
        info->stats.aborts += 1;
 
-       printk(KERN_WARNING "scsi%d: abort command ", info->host->host_no);
-       __scsi_print_command(SCpnt->cmnd, SCpnt->cmd_len);
+       scmd_printk(KERN_WARNING, SCpnt, "abort command\n");
 
        print_debug_list();
        fas216_dumpstate(info);
 
-       printk(KERN_WARNING "scsi%d: abort %p ", info->host->host_no, SCpnt);
-
        switch (fas216_find_command(info, SCpnt)) {
        /*
         * We found the command, and cleared it out.  Either
@@ -2438,7 +2435,7 @@ int fas216_eh_abort(struct scsi_cmnd *SCpnt)
         * target, or the busylun bit is not set.
         */
        case res_success:
-               printk("success\n");
+               scmd_printk(KERN_WARNING, SCpnt, "abort %p success\n", SCpnt);
                result = SUCCESS;
                break;
 
@@ -2448,14 +2445,13 @@ int fas216_eh_abort(struct scsi_cmnd *SCpnt)
         * if the bus is free.
         */
        case res_hw_abort:
-               
 
        /*
         * We are unable to abort the command for some reason.
         */
        default:
        case res_failed:
-               printk("failed\n");
+               scmd_printk(KERN_WARNING, SCpnt, "abort %p failed\n", SCpnt);
                break;
        }
 
index 79e6f045c2a911348a99dfd02a87ab2603b7168c..229c61bfbc6b44fac393c34633c963c7b6abbdbf 100644 (file)
@@ -2623,8 +2623,7 @@ int NCR5380_abort(Scsi_Cmnd *cmd)
        Scsi_Cmnd *tmp, **prev;
        unsigned long flags;
 
-       printk(KERN_NOTICE "scsi%d: aborting command\n", HOSTNO);
-       scsi_print_command(cmd);
+       scmd_printk(KERN_NOTICE, cmd, "aborting command\n");
 
        NCR5380_print_status(instance);
 
index ef23fabe3924ea138db50d14c354d0fb6a55c6ae..b3b48b5a984c54021e45aa5056efdf58500e2617 100644 (file)
@@ -220,10 +220,6 @@ static int ps3rom_queuecommand_lck(struct scsi_cmnd *cmd,
        unsigned char opcode;
        int res;
 
-#ifdef DEBUG
-       scsi_print_command(cmd);
-#endif
-
        priv->curr_cmd = cmd;
        cmd->scsi_done = done;
 
index 1aa4befcfbd0120b11df14a4b0f918b00f5be72d..713af13b858e04cadcaf3d5da8da78a148a26cfc 100644 (file)
@@ -1162,9 +1162,7 @@ static int stex_abort(struct scsi_cmnd *cmd)
        int result = SUCCESS;
        unsigned long flags;
 
-       printk(KERN_INFO DRV_NAME
-               "(%s): aborting command\n", pci_name(hba->pdev));
-       scsi_print_command(cmd);
+       scmd_printk(KERN_INFO, cmd, "aborting command\n");
 
        base = hba->mmio_base;
        spin_lock_irqsave(host->host_lock, flags);
@@ -1352,9 +1350,8 @@ static int stex_reset(struct scsi_cmnd *cmd)
 
        hba = (struct st_hba *) &cmd->device->host->hostdata[0];
 
-       printk(KERN_INFO DRV_NAME
-               "(%s): resetting host\n", pci_name(hba->pdev));
-       scsi_print_command(cmd);
+       shost_printk(KERN_INFO, cmd->device->host,
+                    "resetting host\n");
 
        return stex_do_reset(hba) ? FAILED : SUCCESS;
 }
index 1a2367a1b1f290dbff1e5065d947516502ce79ba..3abd796b98930905a6a8ba089327f71b42bfe26e 100644 (file)
@@ -2608,8 +2608,7 @@ static int NCR5380_abort(struct scsi_cmnd *cmd)
     struct scsi_cmnd *tmp, **prev;
     unsigned long flags;
 
-    printk(KERN_NOTICE "scsi%d: aborting command\n", HOSTNO);
-    scsi_print_command(cmd);
+    scmd_printk(KERN_NOTICE, cmd, "aborting command\n");
 
     NCR5380_print_status (instance);