Commit 250dccc0 authored by Nick Piggin's avatar Nick Piggin Committed by Linus Torvalds

[PATCH] blk: no memory barrier

This memory barrier is not needed because the waitqueue will only get waiters
on it in the following situations:

rq->count has exceeded the threshold - however all manipulations of ->count
are performed under the runqueue lock, and so we will correctly pick up any
waiter.

Memory allocation for the request fails.  In this case, there is no additional
help provided by the memory barrier.  We are guaranteed to eventually wake up
waiters because the request allocation mempool guarantees that if the mem
allocation for a request fails, there must be some requests in flight.  They
will wake up waiters when they are retired.
Signed-off-by: default avatarNick Piggin <nickpiggin@yahoo.com.au>
Cc: Jens Axboe <axboe@suse.de>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 040c928c
...@@ -1842,7 +1842,6 @@ static void __freed_request(request_queue_t *q, int rw) ...@@ -1842,7 +1842,6 @@ static void __freed_request(request_queue_t *q, int rw)
clear_queue_congested(q, rw); clear_queue_congested(q, rw);
if (rl->count[rw] + 1 <= q->nr_requests) { if (rl->count[rw] + 1 <= q->nr_requests) {
smp_mb();
if (waitqueue_active(&rl->wait[rw])) if (waitqueue_active(&rl->wait[rw]))
wake_up(&rl->wait[rw]); wake_up(&rl->wait[rw]);
......
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