diff options
author | Hugh Dickins <hugh@veritas.com> | 2005-10-29 21:16:05 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-10-30 00:40:38 -0400 |
commit | 4294621f41a85497019fae64341aa5351a1921b7 (patch) | |
tree | fdeb7eb44384a99d0679ffa6de5019bab0ea2166 /mm/nommu.c | |
parent | 404351e67a9facb475abf1492245374a28d13e90 (diff) |
[PATCH] mm: rss = file_rss + anon_rss
I was lazy when we added anon_rss, and chose to change as few places as
possible. So currently each anonymous page has to be counted twice, in rss
and in anon_rss. Which won't be so good if those are atomic counts in some
configurations.
Change that around: keep file_rss and anon_rss separately, and add them
together (with get_mm_rss macro) when the total is needed - reading two
atomics is much cheaper than updating two atomics. And update anon_rss
upfront, typically in memory.c, not tucked away in page_add_anon_rmap.
Signed-off-by: Hugh Dickins <hugh@veritas.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'mm/nommu.c')
-rw-r--r-- | mm/nommu.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/nommu.c b/mm/nommu.c index 0ef241ae3763..599924886eb5 100644 --- a/mm/nommu.c +++ b/mm/nommu.c | |||
@@ -1083,7 +1083,7 @@ void update_mem_hiwater(struct task_struct *tsk) | |||
1083 | unsigned long rss; | 1083 | unsigned long rss; |
1084 | 1084 | ||
1085 | if (likely(tsk->mm)) { | 1085 | if (likely(tsk->mm)) { |
1086 | rss = get_mm_counter(tsk->mm, rss); | 1086 | rss = get_mm_rss(tsk->mm); |
1087 | if (tsk->mm->hiwater_rss < rss) | 1087 | if (tsk->mm->hiwater_rss < rss) |
1088 | tsk->mm->hiwater_rss = rss; | 1088 | tsk->mm->hiwater_rss = rss; |
1089 | if (tsk->mm->hiwater_vm < tsk->mm->total_vm) | 1089 | if (tsk->mm->hiwater_vm < tsk->mm->total_vm) |