diff options
| author | Alan Stern <stern@rowland.harvard.edu> | 2005-09-15 13:52:51 -0400 |
|---|---|---|
| committer | James Bottomley <jejb@mulgrave.(none)> | 2005-09-17 15:24:53 -0500 |
| commit | b95be99d52ce4f9db9ff0bd5f10e9e2066da6d2e (patch) | |
| tree | 6b01de07c638fd596cc254ca6a095aa4f5ef9ba2 | |
| parent | 1832a5862f2e1b4e5835611ee14bc30a8ed3cad5 (diff) | |
| download | blackbird-op-linux-b95be99d52ce4f9db9ff0bd5f10e9e2066da6d2e.tar.gz blackbird-op-linux-b95be99d52ce4f9db9ff0bd5f10e9e2066da6d2e.zip | |
[SCSI] fix oops in scsi_release_buffers()
I found one other thing that needs to be fixed. The call to
scsi_release_buffers in scsi_unprep_request causes an oops, because the
sgtable has already been freed in scsi_io_completion. The following patch
is needed.
Signed-off-by: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
| -rw-r--r-- | drivers/scsi/scsi_lib.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/scsi/scsi_lib.c b/drivers/scsi/scsi_lib.c index 0c6924eecf59..f065cbc1a6e5 100644 --- a/drivers/scsi/scsi_lib.c +++ b/drivers/scsi/scsi_lib.c @@ -118,7 +118,6 @@ static void scsi_unprep_request(struct request *req) req->flags &= ~REQ_DONTPREP; req->special = (req->flags & REQ_SPECIAL) ? cmd->sc_request : NULL; - scsi_release_buffers(cmd); scsi_put_command(cmd); } @@ -1512,7 +1511,6 @@ static void scsi_request_fn(struct request_queue *q) * cases (host limits or settings) should run the queue at some * later time. */ - scsi_unprep_request(req); spin_lock_irq(q->queue_lock); blk_requeue_request(q, req); sdev->device_busy--; |
