aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/scsi/arm
diff options
context:
space:
mode:
authorHannes Reinecke <hare@suse.de>2014-10-24 08:26:58 -0400
committerChristoph Hellwig <hch@lst.de>2014-11-12 05:16:04 -0500
commit1fa6b5fbba8c7d4d0cbc428efc4838b813046420 (patch)
tree9a5b12b11f97e27137fba560cf83540a70a9938b /drivers/scsi/arm
parenta9a47bf58ac1d5525ae99922e055d8de87eeae78 (diff)
scsi: Remove scsi_print_command when calling abort
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>
Diffstat (limited to 'drivers/scsi/arm')
-rw-r--r--drivers/scsi/arm/fas216.c10
1 files changed, 3 insertions, 7 deletions
diff --git a/drivers/scsi/arm/fas216.c b/drivers/scsi/arm/fas216.c
index d2581cb41ec8..e64c3af7c1a0 100644
--- a/drivers/scsi/arm/fas216.c
+++ b/drivers/scsi/arm/fas216.c
@@ -2423,14 +2423,11 @@ int fas216_eh_abort(struct scsi_cmnd *SCpnt)
2423 2423
2424 info->stats.aborts += 1; 2424 info->stats.aborts += 1;
2425 2425
2426 printk(KERN_WARNING "scsi%d: abort command ", info->host->host_no); 2426 scmd_printk(KERN_WARNING, SCpnt, "abort command\n");
2427 __scsi_print_command(SCpnt->cmnd, SCpnt->cmd_len);
2428 2427
2429 print_debug_list(); 2428 print_debug_list();
2430 fas216_dumpstate(info); 2429 fas216_dumpstate(info);
2431 2430
2432 printk(KERN_WARNING "scsi%d: abort %p ", info->host->host_no, SCpnt);
2433
2434 switch (fas216_find_command(info, SCpnt)) { 2431 switch (fas216_find_command(info, SCpnt)) {
2435 /* 2432 /*
2436 * We found the command, and cleared it out. Either 2433 * We found the command, and cleared it out. Either
@@ -2438,7 +2435,7 @@ int fas216_eh_abort(struct scsi_cmnd *SCpnt)
2438 * target, or the busylun bit is not set. 2435 * target, or the busylun bit is not set.
2439 */ 2436 */
2440 case res_success: 2437 case res_success:
2441 printk("success\n"); 2438 scmd_printk(KERN_WARNING, SCpnt, "abort %p success\n", SCpnt);
2442 result = SUCCESS; 2439 result = SUCCESS;
2443 break; 2440 break;
2444 2441
@@ -2448,14 +2445,13 @@ int fas216_eh_abort(struct scsi_cmnd *SCpnt)
2448 * if the bus is free. 2445 * if the bus is free.
2449 */ 2446 */
2450 case res_hw_abort: 2447 case res_hw_abort:
2451
2452 2448
2453 /* 2449 /*
2454 * We are unable to abort the command for some reason. 2450 * We are unable to abort the command for some reason.
2455 */ 2451 */
2456 default: 2452 default:
2457 case res_failed: 2453 case res_failed:
2458 printk("failed\n"); 2454 scmd_printk(KERN_WARNING, SCpnt, "abort %p failed\n", SCpnt);
2459 break; 2455 break;
2460 } 2456 }
2461 2457