On Thu 11-05-17 20:16:23, Roman Gushchin wrote: > Track the following reclaim counters for every memory cgroup: > PGREFILL, PGSCAN, PGSTEAL, PGACTIVATE, PGDEACTIVATE, PGLAZYFREE and > PGLAZYFREED.
yes, those are definitely useful. I have an old patch to add them as well but never managed to clean it up and post... > These values are exposed using the memory.stats interface of cgroup v2. Is there any reason to not add them to v1? This should be rather trivial after recent changes from Johannes. > The meaning of each value is the same as for global counters, > available using /proc/vmstat. > > Also, for consistency, rename mem_cgroup_count_vm_event() to > count_memcg_event_mm(). > > Signed-off-by: Roman Gushchin <[email protected]> > Suggested-by: Johannes Weiner <[email protected]> > Cc: Johannes Weiner <[email protected]> > Cc: Tejun Heo <[email protected]> > Cc: Li Zefan <[email protected]> > Cc: Michal Hocko <[email protected]> > Cc: Vladimir Davydov <[email protected]> > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] the patch itself looks good to me. I will have to double check it after I am done with what I am doing currently and then will add my Acked-by -- Michal Hocko SUSE Labs
