diff options
author | Kamezawa Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com> | 2012-07-31 19:41:40 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-07-31 21:42:39 -0400 |
commit | 41326c17fc1b45289a82567553249f60d3638a23 (patch) | |
tree | c871764fb31bd7a4a57c2d55f87b19332f4c69ef | |
parent | bff6bb83f38105b39b0cc3a9ad81103edbb56f7a (diff) |
memcg: rename MEM_CGROUP_CHARGE_TYPE_MAPPED as MEM_CGROUP_CHARGE_TYPE_ANON
Now, in memcg, 2 "MAPPED" enum/macro are found
MEM_CGROUP_CHARGE_TYPE_MAPPED
MEM_CGROUP_STAT_FILE_MAPPED
Thier names looks similar to each other but the former is used for
accounting anonymous memory. rename it as TYPE_ANON.
Signed-off-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Acked-by: Michal Hocko <mhocko@suse.cz>
Acked-by: Johannes Weiner <hannes@cmpxchg.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | mm/memcontrol.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/mm/memcontrol.c b/mm/memcontrol.c index 91ce7b2ee1e4..1b3dd27ead6d 100644 --- a/mm/memcontrol.c +++ b/mm/memcontrol.c | |||
@@ -378,7 +378,7 @@ static bool move_file(void) | |||
378 | 378 | ||
379 | enum charge_type { | 379 | enum charge_type { |
380 | MEM_CGROUP_CHARGE_TYPE_CACHE = 0, | 380 | MEM_CGROUP_CHARGE_TYPE_CACHE = 0, |
381 | MEM_CGROUP_CHARGE_TYPE_MAPPED, | 381 | MEM_CGROUP_CHARGE_TYPE_ANON, |
382 | MEM_CGROUP_CHARGE_TYPE_SHMEM, /* used by page migration of shmem */ | 382 | MEM_CGROUP_CHARGE_TYPE_SHMEM, /* used by page migration of shmem */ |
383 | MEM_CGROUP_CHARGE_TYPE_FORCE, /* used by force_empty */ | 383 | MEM_CGROUP_CHARGE_TYPE_FORCE, /* used by force_empty */ |
384 | MEM_CGROUP_CHARGE_TYPE_SWAPOUT, /* for accounting swapcache */ | 384 | MEM_CGROUP_CHARGE_TYPE_SWAPOUT, /* for accounting swapcache */ |
@@ -2519,7 +2519,7 @@ static void __mem_cgroup_commit_charge(struct mem_cgroup *memcg, | |||
2519 | spin_unlock_irq(&zone->lru_lock); | 2519 | spin_unlock_irq(&zone->lru_lock); |
2520 | } | 2520 | } |
2521 | 2521 | ||
2522 | if (ctype == MEM_CGROUP_CHARGE_TYPE_MAPPED) | 2522 | if (ctype == MEM_CGROUP_CHARGE_TYPE_ANON) |
2523 | anon = true; | 2523 | anon = true; |
2524 | else | 2524 | else |
2525 | anon = false; | 2525 | anon = false; |
@@ -2728,7 +2728,7 @@ int mem_cgroup_newpage_charge(struct page *page, | |||
2728 | VM_BUG_ON(page->mapping && !PageAnon(page)); | 2728 | VM_BUG_ON(page->mapping && !PageAnon(page)); |
2729 | VM_BUG_ON(!mm); | 2729 | VM_BUG_ON(!mm); |
2730 | return mem_cgroup_charge_common(page, mm, gfp_mask, | 2730 | return mem_cgroup_charge_common(page, mm, gfp_mask, |
2731 | MEM_CGROUP_CHARGE_TYPE_MAPPED); | 2731 | MEM_CGROUP_CHARGE_TYPE_ANON); |
2732 | } | 2732 | } |
2733 | 2733 | ||
2734 | static void | 2734 | static void |
@@ -2842,7 +2842,7 @@ void mem_cgroup_commit_charge_swapin(struct page *page, | |||
2842 | struct mem_cgroup *memcg) | 2842 | struct mem_cgroup *memcg) |
2843 | { | 2843 | { |
2844 | __mem_cgroup_commit_charge_swapin(page, memcg, | 2844 | __mem_cgroup_commit_charge_swapin(page, memcg, |
2845 | MEM_CGROUP_CHARGE_TYPE_MAPPED); | 2845 | MEM_CGROUP_CHARGE_TYPE_ANON); |
2846 | } | 2846 | } |
2847 | 2847 | ||
2848 | void mem_cgroup_cancel_charge_swapin(struct mem_cgroup *memcg) | 2848 | void mem_cgroup_cancel_charge_swapin(struct mem_cgroup *memcg) |
@@ -2945,7 +2945,7 @@ __mem_cgroup_uncharge_common(struct page *page, enum charge_type ctype) | |||
2945 | anon = PageAnon(page); | 2945 | anon = PageAnon(page); |
2946 | 2946 | ||
2947 | switch (ctype) { | 2947 | switch (ctype) { |
2948 | case MEM_CGROUP_CHARGE_TYPE_MAPPED: | 2948 | case MEM_CGROUP_CHARGE_TYPE_ANON: |
2949 | /* | 2949 | /* |
2950 | * Generally PageAnon tells if it's the anon statistics to be | 2950 | * Generally PageAnon tells if it's the anon statistics to be |
2951 | * updated; but sometimes e.g. mem_cgroup_uncharge_page() is | 2951 | * updated; but sometimes e.g. mem_cgroup_uncharge_page() is |
@@ -3005,7 +3005,7 @@ void mem_cgroup_uncharge_page(struct page *page) | |||
3005 | if (page_mapped(page)) | 3005 | if (page_mapped(page)) |
3006 | return; | 3006 | return; |
3007 | VM_BUG_ON(page->mapping && !PageAnon(page)); | 3007 | VM_BUG_ON(page->mapping && !PageAnon(page)); |
3008 | __mem_cgroup_uncharge_common(page, MEM_CGROUP_CHARGE_TYPE_MAPPED); | 3008 | __mem_cgroup_uncharge_common(page, MEM_CGROUP_CHARGE_TYPE_ANON); |
3009 | } | 3009 | } |
3010 | 3010 | ||
3011 | void mem_cgroup_uncharge_cache_page(struct page *page) | 3011 | void mem_cgroup_uncharge_cache_page(struct page *page) |
@@ -3248,7 +3248,7 @@ int mem_cgroup_prepare_migration(struct page *page, | |||
3248 | * mapcount will be finally 0 and we call uncharge in end_migration(). | 3248 | * mapcount will be finally 0 and we call uncharge in end_migration(). |
3249 | */ | 3249 | */ |
3250 | if (PageAnon(page)) | 3250 | if (PageAnon(page)) |
3251 | ctype = MEM_CGROUP_CHARGE_TYPE_MAPPED; | 3251 | ctype = MEM_CGROUP_CHARGE_TYPE_ANON; |
3252 | else if (page_is_file_cache(page)) | 3252 | else if (page_is_file_cache(page)) |
3253 | ctype = MEM_CGROUP_CHARGE_TYPE_CACHE; | 3253 | ctype = MEM_CGROUP_CHARGE_TYPE_CACHE; |
3254 | else | 3254 | else |
@@ -3287,7 +3287,7 @@ void mem_cgroup_end_migration(struct mem_cgroup *memcg, | |||
3287 | unlock_page_cgroup(pc); | 3287 | unlock_page_cgroup(pc); |
3288 | anon = PageAnon(used); | 3288 | anon = PageAnon(used); |
3289 | __mem_cgroup_uncharge_common(unused, | 3289 | __mem_cgroup_uncharge_common(unused, |
3290 | anon ? MEM_CGROUP_CHARGE_TYPE_MAPPED | 3290 | anon ? MEM_CGROUP_CHARGE_TYPE_ANON |
3291 | : MEM_CGROUP_CHARGE_TYPE_CACHE); | 3291 | : MEM_CGROUP_CHARGE_TYPE_CACHE); |
3292 | 3292 | ||
3293 | /* | 3293 | /* |