diff options
author | Hannes Reinecke <hare@suse.de> | 2014-10-24 08:26:58 -0400 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2014-11-12 05:16:04 -0500 |
commit | 1fa6b5fbba8c7d4d0cbc428efc4838b813046420 (patch) | |
tree | 9a5b12b11f97e27137fba560cf83540a70a9938b /drivers/scsi/arm | |
parent | a9a47bf58ac1d5525ae99922e055d8de87eeae78 (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.c | 10 |
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 | ||