[PATCH 22/26] scsi: fixup logging messages in scsi_error.c

2014-10-07 Thread Hannes Reinecke
Use the matching scope for logging messages to allow for
better command tracing.

Suggested-by: Robert Elliott 
Reviewed-by: Robert Elliott 
Reviewed-by: Christoph Hellwig 
Signed-off-by: Hannes Reinecke 
---
 drivers/scsi/hosts.c  |  4 +--
 drivers/scsi/scsi_error.c | 77 +++
 2 files changed, 39 insertions(+), 42 deletions(-)

diff --git a/drivers/scsi/hosts.c b/drivers/scsi/hosts.c
index 6de80e3..06030e1 100644
--- a/drivers/scsi/hosts.c
+++ b/drivers/scsi/hosts.c
@@ -485,8 +485,8 @@ struct Scsi_Host *scsi_host_alloc(struct scsi_host_template 
*sht, int privsize)
WQ_UNBOUND | WQ_MEM_RECLAIM,
   1, shost->host_no);
if (!shost->tmf_work_q) {
-   printk(KERN_WARNING "scsi%d: failed to create tmf workq\n",
-  shost->host_no);
+   shost_printk(KERN_WARNING, shost,
+"failed to create tmf workq\n");
goto fail_kthread;
}
scsi_proc_hostdir_add(shost->hostt);
diff --git a/drivers/scsi/scsi_error.c b/drivers/scsi/scsi_error.c
index a113e99..087ca02 100644
--- a/drivers/scsi/scsi_error.c
+++ b/drivers/scsi/scsi_error.c
@@ -1156,9 +1156,9 @@ int scsi_eh_get_sense(struct list_head *work_q,
shost = scmd->device->host;
if (scsi_host_eh_past_deadline(shost)) {
SCSI_LOG_ERROR_RECOVERY(3,
-   shost_printk(KERN_INFO, shost,
-   "skip %s, past eh deadline\n",
-__func__));
+   scmd_printk(KERN_INFO, scmd,
+   "%s: skip request sense, past eh 
deadline\n",
+current->comm));
break;
}
if (status_byte(scmd->result) != CHECK_CONDITION)
@@ -1265,9 +1265,9 @@ static int scsi_eh_test_devices(struct list_head 
*cmd_list,
/* Push items back onto work_q */
list_splice_init(cmd_list, work_q);
SCSI_LOG_ERROR_RECOVERY(3,
-   shost_printk(KERN_INFO, sdev->host,
-"skip %s, past eh 
deadline",
-__func__));
+   sdev_printk(KERN_INFO, sdev,
+   "%s: skip test device, past 
eh deadline",
+   current->comm));
break;
}
}
@@ -1318,21 +1318,20 @@ static int scsi_eh_abort_cmds(struct list_head *work_q,
if (scsi_host_eh_past_deadline(shost)) {
list_splice_init(&check_list, work_q);
SCSI_LOG_ERROR_RECOVERY(3,
-   shost_printk(KERN_INFO, shost,
-   "skip %s, past eh deadline\n",
-__func__));
+   scmd_printk(KERN_INFO, scmd,
+   "%s: skip aborting cmd, past eh 
deadline\n",
+   current->comm));
return list_empty(work_q);
}
SCSI_LOG_ERROR_RECOVERY(3,
-   shost_printk(KERN_INFO, shost,
-"%s: aborting cmd: 0x%p\n",
-current->comm, scmd));
+   scmd_printk(KERN_INFO, scmd,
+"%s: aborting cmd\n", current->comm));
rtn = scsi_try_to_abort_cmd(shost->hostt, scmd);
if (rtn == FAILED) {
SCSI_LOG_ERROR_RECOVERY(3,
-   shost_printk(KERN_INFO, shost,
-"%s: aborting cmd failed: 0x%p\n",
-current->comm, scmd));
+   scmd_printk(KERN_INFO, scmd,
+   "%s: aborting cmd failed\n",
+current->comm));
list_splice_init(&check_list, work_q);
return list_empty(work_q);
}
@@ -1390,9 +1389,9 @@ static int scsi_eh_stu(struct Scsi_Host *shost,
shost_for_each_device(sdev, shost) {
if (scsi_host_eh_past_deadline(shost)) {
SCSI_LOG_ERROR_RECOVERY(3,
-   shost_printk(KERN_INFO, shost,
-   "skip %s, past eh deadline\n",
-   

[PATCH 22/26] scsi: fixup logging messages in scsi_error.c

2014-10-06 Thread Hannes Reinecke
Use the matching scope for logging messages to allow for
better command tracing.

Suggested-by: Robert Elliott 
Reviewed-by: Robert Elliott 
Reviewed-by: Christoph Hellwig 
Signed-off-by: Hannes Reinecke 
---
 drivers/scsi/hosts.c  |  4 +--
 drivers/scsi/scsi_error.c | 77 +++
 2 files changed, 39 insertions(+), 42 deletions(-)

diff --git a/drivers/scsi/hosts.c b/drivers/scsi/hosts.c
index 6de80e3..06030e1 100644
--- a/drivers/scsi/hosts.c
+++ b/drivers/scsi/hosts.c
@@ -485,8 +485,8 @@ struct Scsi_Host *scsi_host_alloc(struct scsi_host_template 
*sht, int privsize)
WQ_UNBOUND | WQ_MEM_RECLAIM,
   1, shost->host_no);
if (!shost->tmf_work_q) {
-   printk(KERN_WARNING "scsi%d: failed to create tmf workq\n",
-  shost->host_no);
+   shost_printk(KERN_WARNING, shost,
+"failed to create tmf workq\n");
goto fail_kthread;
}
scsi_proc_hostdir_add(shost->hostt);
diff --git a/drivers/scsi/scsi_error.c b/drivers/scsi/scsi_error.c
index a113e99..087ca02 100644
--- a/drivers/scsi/scsi_error.c
+++ b/drivers/scsi/scsi_error.c
@@ -1156,9 +1156,9 @@ int scsi_eh_get_sense(struct list_head *work_q,
shost = scmd->device->host;
if (scsi_host_eh_past_deadline(shost)) {
SCSI_LOG_ERROR_RECOVERY(3,
-   shost_printk(KERN_INFO, shost,
-   "skip %s, past eh deadline\n",
-__func__));
+   scmd_printk(KERN_INFO, scmd,
+   "%s: skip request sense, past eh 
deadline\n",
+current->comm));
break;
}
if (status_byte(scmd->result) != CHECK_CONDITION)
@@ -1265,9 +1265,9 @@ static int scsi_eh_test_devices(struct list_head 
*cmd_list,
/* Push items back onto work_q */
list_splice_init(cmd_list, work_q);
SCSI_LOG_ERROR_RECOVERY(3,
-   shost_printk(KERN_INFO, sdev->host,
-"skip %s, past eh 
deadline",
-__func__));
+   sdev_printk(KERN_INFO, sdev,
+   "%s: skip test device, past 
eh deadline",
+   current->comm));
break;
}
}
@@ -1318,21 +1318,20 @@ static int scsi_eh_abort_cmds(struct list_head *work_q,
if (scsi_host_eh_past_deadline(shost)) {
list_splice_init(&check_list, work_q);
SCSI_LOG_ERROR_RECOVERY(3,
-   shost_printk(KERN_INFO, shost,
-   "skip %s, past eh deadline\n",
-__func__));
+   scmd_printk(KERN_INFO, scmd,
+   "%s: skip aborting cmd, past eh 
deadline\n",
+   current->comm));
return list_empty(work_q);
}
SCSI_LOG_ERROR_RECOVERY(3,
-   shost_printk(KERN_INFO, shost,
-"%s: aborting cmd: 0x%p\n",
-current->comm, scmd));
+   scmd_printk(KERN_INFO, scmd,
+"%s: aborting cmd\n", current->comm));
rtn = scsi_try_to_abort_cmd(shost->hostt, scmd);
if (rtn == FAILED) {
SCSI_LOG_ERROR_RECOVERY(3,
-   shost_printk(KERN_INFO, shost,
-"%s: aborting cmd failed: 0x%p\n",
-current->comm, scmd));
+   scmd_printk(KERN_INFO, scmd,
+   "%s: aborting cmd failed\n",
+current->comm));
list_splice_init(&check_list, work_q);
return list_empty(work_q);
}
@@ -1390,9 +1389,9 @@ static int scsi_eh_stu(struct Scsi_Host *shost,
shost_for_each_device(sdev, shost) {
if (scsi_host_eh_past_deadline(shost)) {
SCSI_LOG_ERROR_RECOVERY(3,
-   shost_printk(KERN_INFO, shost,
-   "skip %s, past eh deadline\n",
-