diff options
| author | Nick Piggin <npiggin@suse.de> | 2009-09-15 21:32:58 +0200 |
|---|---|---|
| committer | Jens Axboe <jens.axboe@oracle.com> | 2009-09-16 15:18:52 +0200 |
| commit | deed62edffe600bc5b379c872d3004116e001b66 (patch) | |
| tree | 2c1837366be7dac9986a250d8b5cebf6f54fdea0 | |
| parent | 49db041430e8a856dbc3af15430bf068f1c74655 (diff) | |
| download | linux-3.10-deed62edffe600bc5b379c872d3004116e001b66.tar.gz linux-3.10-deed62edffe600bc5b379c872d3004116e001b66.tar.bz2 linux-3.10-deed62edffe600bc5b379c872d3004116e001b66.zip | |
writeback: remove smp_mb(), it's not needed with list_add_tail_rcu()
list_add_tail_rcu contains required barriers.
Signed-off-by: Nick Piggin <npiggin@suse.de>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
| -rw-r--r-- | fs/fs-writeback.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/fs-writeback.c b/fs/fs-writeback.c index 34757758511..59c99e72918 100644 --- a/fs/fs-writeback.c +++ b/fs/fs-writeback.c @@ -151,10 +151,10 @@ static void bdi_queue_work(struct backing_dev_info *bdi, struct bdi_work *work) BUG_ON(!bdi->wb_cnt); /* - * Make sure stores are seen before it appears on the list + * list_add_tail_rcu() contains the necessary barriers to + * make sure the above stores are seen before the item is + * noticed on the list */ - smp_mb(); - spin_lock(&bdi->wb_lock); list_add_tail_rcu(&work->list, &bdi->work_list); spin_unlock(&bdi->wb_lock); |