diff options
author | Jens Axboe <jaxboe@fusionio.com> | 2011-03-07 07:59:06 (GMT) |
---|---|---|
committer | Jens Axboe <jaxboe@fusionio.com> | 2011-03-07 07:59:06 (GMT) |
commit | 93803e0140c6216b68fe926ccc611297120da273 (patch) | |
tree | c70e354f3029a00f83293b0d6bedf6738437ebab /block/cfq-iosched.c | |
parent | da527770007fce8e4541947d47918248286da875 (diff) | |
download | linux-93803e0140c6216b68fe926ccc611297120da273.tar.xz |
cfq-iosched: fix race in cfq_set_request()
We need to hold the queue lock over the reference increment,
it's not atomic anymore.
Signed-off-by: Jens Axboe <jaxboe@fusionio.com>
Diffstat (limited to 'block/cfq-iosched.c')
-rw-r--r-- | block/cfq-iosched.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/block/cfq-iosched.c b/block/cfq-iosched.c index 3202c7e..fb2141e 100644 --- a/block/cfq-iosched.c +++ b/block/cfq-iosched.c @@ -3675,12 +3675,11 @@ new_queue: cfqq->allocated[rw]++; - spin_unlock_irqrestore(q->queue_lock, flags); - cfqq->ref++; rq->elevator_private[0] = cic; rq->elevator_private[1] = cfqq; rq->elevator_private[2] = cfq_ref_get_cfqg(cfqq->cfqg); + spin_unlock_irqrestore(q->queue_lock, flags); return 0; queue_fail: |