diff options
author | James Bottomley <James.Bottomley@steeleye.com> | 2005-12-09 17:45:22 -0500 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.(none)> | 2005-12-14 00:29:27 -0500 |
commit | c9526497cf03ee775c3a6f8ba62335735f98de7a (patch) | |
tree | 46619a6e57ca84b51fed852722114c2fd25a9289 /drivers/scsi/scsi_lib.c | |
parent | 322e079f1b606e46b79bb8b8e6cf6110b5f2aa3f (diff) |
[SCSI] Consolidate REQ_BLOCK_PC handling path (fix ipod panic)
This follows on from Jens' patch and consolidates all of the ULD
separate handlers for REQ_BLOCK_PC into a single call which has his
fix for our direction bug.
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/scsi/scsi_lib.c')
-rw-r--r-- | drivers/scsi/scsi_lib.c | 33 |
1 files changed, 21 insertions, 12 deletions
diff --git a/drivers/scsi/scsi_lib.c b/drivers/scsi/scsi_lib.c index 097888721ec4..9be5769d44aa 100644 --- a/drivers/scsi/scsi_lib.c +++ b/drivers/scsi/scsi_lib.c | |||
@@ -1078,6 +1078,26 @@ static void scsi_generic_done(struct scsi_cmnd *cmd) | |||
1078 | scsi_io_completion(cmd, cmd->result == 0 ? cmd->bufflen : 0, 0); | 1078 | scsi_io_completion(cmd, cmd->result == 0 ? cmd->bufflen : 0, 0); |
1079 | } | 1079 | } |
1080 | 1080 | ||
1081 | void scsi_setup_blk_pc_cmnd(struct scsi_cmnd *cmd, int retries) | ||
1082 | { | ||
1083 | struct request *req = cmd->request; | ||
1084 | |||
1085 | BUG_ON(sizeof(req->cmd) > sizeof(cmd->cmnd)); | ||
1086 | memcpy(cmd->cmnd, req->cmd, sizeof(cmd->cmnd)); | ||
1087 | cmd->cmd_len = req->cmd_len; | ||
1088 | if (!req->data_len) | ||
1089 | cmd->sc_data_direction = DMA_NONE; | ||
1090 | else if (rq_data_dir(req) == WRITE) | ||
1091 | cmd->sc_data_direction = DMA_TO_DEVICE; | ||
1092 | else | ||
1093 | cmd->sc_data_direction = DMA_FROM_DEVICE; | ||
1094 | |||
1095 | cmd->transfersize = req->data_len; | ||
1096 | cmd->allowed = retries; | ||
1097 | cmd->timeout_per_command = req->timeout; | ||
1098 | } | ||
1099 | EXPORT_SYMBOL_GPL(scsi_setup_blk_pc_cmnd); | ||
1100 | |||
1081 | static int scsi_prep_fn(struct request_queue *q, struct request *req) | 1101 | static int scsi_prep_fn(struct request_queue *q, struct request *req) |
1082 | { | 1102 | { |
1083 | struct scsi_device *sdev = q->queuedata; | 1103 | struct scsi_device *sdev = q->queuedata; |
@@ -1213,18 +1233,7 @@ static int scsi_prep_fn(struct request_queue *q, struct request *req) | |||
1213 | goto kill; | 1233 | goto kill; |
1214 | } | 1234 | } |
1215 | } else { | 1235 | } else { |
1216 | memcpy(cmd->cmnd, req->cmd, sizeof(cmd->cmnd)); | 1236 | scsi_setup_blk_pc_cmnd(cmd, 3); |
1217 | cmd->cmd_len = req->cmd_len; | ||
1218 | if (!req->data_len) | ||
1219 | cmd->sc_data_direction = DMA_NONE; | ||
1220 | else if (rq_data_dir(req) == WRITE) | ||
1221 | cmd->sc_data_direction = DMA_TO_DEVICE; | ||
1222 | else | ||
1223 | cmd->sc_data_direction = DMA_FROM_DEVICE; | ||
1224 | |||
1225 | cmd->transfersize = req->data_len; | ||
1226 | cmd->allowed = 3; | ||
1227 | cmd->timeout_per_command = req->timeout; | ||
1228 | cmd->done = scsi_generic_done; | 1237 | cmd->done = scsi_generic_done; |
1229 | } | 1238 | } |
1230 | } | 1239 | } |