Commit a8c730e8 authored by Jens Axboe's avatar Jens Axboe Committed by James Bottomley

[SCSI] fix panic when ejecting ieee1394 ipod

The scsi_library routines don't correctly set DMA_NONE when
req->data_len is zero (instead they check the command type first, so
if it's write, we end up with req->data_len == 0 and direction as
DMA_TO_DEVICE which confuses some drivers)
Signed-off-by: default avatarJames Bottomley <James.Bottomley@SteelEye.com>
parent 85631672
...@@ -1215,12 +1215,12 @@ static int scsi_prep_fn(struct request_queue *q, struct request *req) ...@@ -1215,12 +1215,12 @@ static int scsi_prep_fn(struct request_queue *q, struct request *req)
} else { } else {
memcpy(cmd->cmnd, req->cmd, sizeof(cmd->cmnd)); memcpy(cmd->cmnd, req->cmd, sizeof(cmd->cmnd));
cmd->cmd_len = req->cmd_len; cmd->cmd_len = req->cmd_len;
if (rq_data_dir(req) == WRITE) if (!req->data_len)
cmd->sc_data_direction = DMA_NONE;
else if (rq_data_dir(req) == WRITE)
cmd->sc_data_direction = DMA_TO_DEVICE; cmd->sc_data_direction = DMA_TO_DEVICE;
else if (req->data_len)
cmd->sc_data_direction = DMA_FROM_DEVICE;
else else
cmd->sc_data_direction = DMA_NONE; cmd->sc_data_direction = DMA_FROM_DEVICE;
cmd->transfersize = req->data_len; cmd->transfersize = req->data_len;
cmd->allowed = 3; cmd->allowed = 3;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment