aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/scsi/scsi_lib.c
diff options
context:
space:
mode:
authorJames Bottomley <James.Bottomley@steeleye.com>2005-10-02 12:45:08 -0400
committerJames Bottomley <jejb@mulgrave.(none)>2005-10-28 15:23:02 -0400
commit9ccfc756a70d454dfa82f48897e2883560c01a0e (patch)
tree9a6d3b10b1ec0e5fe7a63252a21598a03e93ad4e /drivers/scsi/scsi_lib.c
parent9a41a62b74388827998253d62c58707e63cc5874 (diff)
[SCSI] move the mid-layer printk's over to shost/starget/sdev_printk
This should eliminate (at least in the mid layer) to make numeric assumptions about any of the enumeration variables. As a side effect, it will also make all the messages consistent and line us up nicely for the error logging strategy (if it ever shows itself again). Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/scsi/scsi_lib.c')
-rw-r--r--drivers/scsi/scsi_lib.c46
1 files changed, 23 insertions, 23 deletions
diff --git a/drivers/scsi/scsi_lib.c b/drivers/scsi/scsi_lib.c
index 3ff53880978..19ac888001e 100644
--- a/drivers/scsi/scsi_lib.c
+++ b/drivers/scsi/scsi_lib.c
@@ -951,16 +951,16 @@ void scsi_io_completion(struct scsi_cmnd *cmd, unsigned int good_bytes,
951 return; 951 return;
952 } 952 }
953 if (!(req->flags & REQ_QUIET)) 953 if (!(req->flags & REQ_QUIET))
954 dev_printk(KERN_INFO, 954 sdev_printk(KERN_INFO,
955 &cmd->device->sdev_gendev, 955 cmd->device,
956 "Device not ready.\n"); 956 "Device not ready.\n");
957 scsi_end_request(cmd, 0, this_count, 1); 957 scsi_end_request(cmd, 0, this_count, 1);
958 return; 958 return;
959 case VOLUME_OVERFLOW: 959 case VOLUME_OVERFLOW:
960 if (!(req->flags & REQ_QUIET)) { 960 if (!(req->flags & REQ_QUIET)) {
961 dev_printk(KERN_INFO, 961 sdev_printk(KERN_INFO,
962 &cmd->device->sdev_gendev, 962 cmd->device,
963 "Volume overflow, CDB: "); 963 "Volume overflow, CDB: ");
964 __scsi_print_command(cmd->data_cmnd); 964 __scsi_print_command(cmd->data_cmnd);
965 scsi_print_sense("", cmd); 965 scsi_print_sense("", cmd);
966 } 966 }
@@ -981,8 +981,9 @@ void scsi_io_completion(struct scsi_cmnd *cmd, unsigned int good_bytes,
981 } 981 }
982 if (result) { 982 if (result) {
983 if (!(req->flags & REQ_QUIET)) { 983 if (!(req->flags & REQ_QUIET)) {
984 dev_printk(KERN_INFO, &cmd->device->sdev_gendev, 984 sdev_printk(KERN_INFO, cmd->device,
985 "SCSI error: return code = 0x%x\n", result); 985 "SCSI error: return code = 0x%x\n",
986 result);
986 987
987 if (driver_byte(result) & DRIVER_SENSE) 988 if (driver_byte(result) & DRIVER_SENSE)
988 scsi_print_sense("", cmd); 989 scsi_print_sense("", cmd);
@@ -1141,8 +1142,8 @@ static int scsi_prep_fn(struct request_queue *q, struct request *req)
1141 * online before trying any recovery commands 1142 * online before trying any recovery commands
1142 */ 1143 */
1143 if (unlikely(!scsi_device_online(sdev))) { 1144 if (unlikely(!scsi_device_online(sdev))) {
1144 printk(KERN_ERR "scsi%d (%d:%d): rejecting I/O to offline device\n", 1145 sdev_printk(KERN_ERR, sdev,
1145 sdev->host->host_no, sdev->id, sdev->lun); 1146 "rejecting I/O to offline device\n");
1146 goto kill; 1147 goto kill;
1147 } 1148 }
1148 if (unlikely(sdev->sdev_state != SDEV_RUNNING)) { 1149 if (unlikely(sdev->sdev_state != SDEV_RUNNING)) {
@@ -1151,8 +1152,8 @@ static int scsi_prep_fn(struct request_queue *q, struct request *req)
1151 if (sdev->sdev_state == SDEV_DEL) { 1152 if (sdev->sdev_state == SDEV_DEL) {
1152 /* Device is fully deleted, no commands 1153 /* Device is fully deleted, no commands
1153 * at all allowed down */ 1154 * at all allowed down */
1154 printk(KERN_ERR "scsi%d (%d:%d): rejecting I/O to dead device\n", 1155 sdev_printk(KERN_ERR, sdev,
1155 sdev->host->host_no, sdev->id, sdev->lun); 1156 "rejecting I/O to dead device\n");
1156 goto kill; 1157 goto kill;
1157 } 1158 }
1158 /* OK, we only allow special commands (i.e. not 1159 /* OK, we only allow special commands (i.e. not
@@ -1187,8 +1188,8 @@ static int scsi_prep_fn(struct request_queue *q, struct request *req)
1187 specials_only == SDEV_BLOCK) 1188 specials_only == SDEV_BLOCK)
1188 goto defer; 1189 goto defer;
1189 1190
1190 printk(KERN_ERR "scsi%d (%d:%d): rejecting I/O to device being removed\n", 1191 sdev_printk(KERN_ERR, sdev,
1191 sdev->host->host_no, sdev->id, sdev->lun); 1192 "rejecting I/O to device being removed\n");
1192 goto kill; 1193 goto kill;
1193 } 1194 }
1194 1195
@@ -1315,9 +1316,8 @@ static inline int scsi_dev_queue_ready(struct request_queue *q,
1315 */ 1316 */
1316 if (--sdev->device_blocked == 0) { 1317 if (--sdev->device_blocked == 0) {
1317 SCSI_LOG_MLQUEUE(3, 1318 SCSI_LOG_MLQUEUE(3,
1318 printk("scsi%d (%d:%d) unblocking device at" 1319 sdev_printk(KERN_INFO, sdev,
1319 " zero depth\n", sdev->host->host_no, 1320 "unblocking device at zero depth\n"));
1320 sdev->id, sdev->lun));
1321 } else { 1321 } else {
1322 blk_plug_device(q); 1322 blk_plug_device(q);
1323 return 0; 1323 return 0;
@@ -1436,8 +1436,8 @@ static void scsi_request_fn(struct request_queue *q)
1436 break; 1436 break;
1437 1437
1438 if (unlikely(!scsi_device_online(sdev))) { 1438 if (unlikely(!scsi_device_online(sdev))) {
1439 printk(KERN_ERR "scsi%d (%d:%d): rejecting I/O to offline device\n", 1439 sdev_printk(KERN_ERR, sdev,
1440 sdev->host->host_no, sdev->id, sdev->lun); 1440 "rejecting I/O to offline device\n");
1441 scsi_kill_request(req, q); 1441 scsi_kill_request(req, q);
1442 continue; 1442 continue;
1443 } 1443 }
@@ -1893,10 +1893,10 @@ scsi_device_set_state(struct scsi_device *sdev, enum scsi_device_state state)
1893 1893
1894 illegal: 1894 illegal:
1895 SCSI_LOG_ERROR_RECOVERY(1, 1895 SCSI_LOG_ERROR_RECOVERY(1,
1896 dev_printk(KERN_ERR, &sdev->sdev_gendev, 1896 sdev_printk(KERN_ERR, sdev,
1897 "Illegal state transition %s->%s\n", 1897 "Illegal state transition %s->%s\n",
1898 scsi_device_state_name(oldstate), 1898 scsi_device_state_name(oldstate),
1899 scsi_device_state_name(state)) 1899 scsi_device_state_name(state))
1900 ); 1900 );
1901 return -EINVAL; 1901 return -EINVAL;
1902} 1902}