diff options
author | Konstantin Khlebnikov <khlebnikov@openvz.org> | 2011-07-25 20:12:31 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-07-25 23:57:11 -0400 |
commit | 72c4783210f77fd743f0a316858d33f27db51e7c (patch) | |
tree | 4efc95eb0aaade090bac42e72c5973ada6d2cdb1 /mm/page-writeback.c | |
parent | 76d3fbf8fbf6cc78ceb63549e0e0c5bc8a88f838 (diff) |
mm: remove useless rcu lock-unlock from mapping_tagged()
radix_tree_tagged() is lockless - it reads from a member of the raid-tree
root node. It does not require any protection.
Signed-off-by: Konstantin Khlebnikov <khlebnikov@openvz.org>
Cc: Hugh Dickins <hughd@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/page-writeback.c')
-rw-r--r-- | mm/page-writeback.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/mm/page-writeback.c b/mm/page-writeback.c index 31f698862420..919b45eb57ad 100644 --- a/mm/page-writeback.c +++ b/mm/page-writeback.c | |||
@@ -1405,10 +1405,6 @@ EXPORT_SYMBOL(test_set_page_writeback); | |||
1405 | */ | 1405 | */ |
1406 | int mapping_tagged(struct address_space *mapping, int tag) | 1406 | int mapping_tagged(struct address_space *mapping, int tag) |
1407 | { | 1407 | { |
1408 | int ret; | 1408 | return radix_tree_tagged(&mapping->page_tree, tag); |
1409 | rcu_read_lock(); | ||
1410 | ret = radix_tree_tagged(&mapping->page_tree, tag); | ||
1411 | rcu_read_unlock(); | ||
1412 | return ret; | ||
1413 | } | 1409 | } |
1414 | EXPORT_SYMBOL(mapping_tagged); | 1410 | EXPORT_SYMBOL(mapping_tagged); |