diff options
author | Hugh Dickins <hugh.dickins@tiscali.co.uk> | 2009-12-15 02:59:16 +0100 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-12-15 17:53:18 +0100 |
commit | 93d17715a5b960d34220f2edba3e6cee9b5b1c58 (patch) | |
tree | 9ab08ff4169464bf5b8d86c8082296fd6e3d069b /crypto/sha1_generic.c | |
parent | vmscan: make consistent of reclaim bale out between do_try_to_free_page and s... (diff) | |
download | linux-93d17715a5b960d34220f2edba3e6cee9b5b1c58.tar.xz linux-93d17715a5b960d34220f2edba3e6cee9b5b1c58.zip |
ksm: three remove_rmap_item_from_tree cleanups
1. remove_rmap_item_from_tree() is called as a precaution from
various places: don't dirty the rmap_item cacheline unnecessarily,
just mask the flags out of the address when they have been set.
2. First get_next_rmap_item() removes an unstable rmap_item from its tree,
then shortly afterwards cmp_and_merge_page() removes a stable rmap_item
from its tree: it's easier just to do both at once (but definitely keep
the BUG_ON(age > 1) which guards against a future omission).
3. When cmp_and_merge_page() moves an rmap_item from unstable to stable
tree, it does its own rb_erase() and accounting: that's better
expressed by remove_rmap_item_from_tree().
Signed-off-by: Hugh Dickins <hugh.dickins@tiscali.co.uk>
Cc: Izik Eidus <ieidus@redhat.com>
Cc: Andrea Arcangeli <aarcange@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'crypto/sha1_generic.c')
0 files changed, 0 insertions, 0 deletions