diff options
author | Herbert Xu <herbert@gondor.apana.org.au> | 2010-03-31 20:13:39 +0000 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-04-01 01:31:14 -0700 |
commit | 1af185034662b75279f90e1c7cb958271d4121e2 (patch) | |
tree | d6159d863ecb7854d56f740cc0bec2b37b2dee8a | |
parent | 6072f7491f5ef391a575e18a1165e72a3eef1601 (diff) | |
download | lwn-1af185034662b75279f90e1c7cb958271d4121e2.tar.gz lwn-1af185034662b75279f90e1c7cb958271d4121e2.zip |
ide: Must hold queue lock when requeueing
ide-atapi requeues requests without holding the queue lock.
This patch fixes it by using ide_requeue_and_plug.
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/ide/ide-atapi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/ide/ide-atapi.c b/drivers/ide/ide-atapi.c index eb2181a6a11c..9aedb9aa544f 100644 --- a/drivers/ide/ide-atapi.c +++ b/drivers/ide/ide-atapi.c @@ -263,8 +263,8 @@ void ide_retry_pc(ide_drive_t *drive) * of it. The failed command will be retried after sense data * is acquired. */ - blk_requeue_request(failed_rq->q, failed_rq); drive->hwif->rq = NULL; + ide_requeue_and_plug(drive, failed_rq); if (ide_queue_sense_rq(drive, pc)) { blk_start_request(failed_rq); ide_complete_rq(drive, -EIO, blk_rq_bytes(failed_rq)); |