Re: [PATCH v4] mm: memcontrol: Add the missing numa_stat interface for cgroup v2

2020-09-15 Thread Shakeel Butt
On Mon, Sep 14, 2020 at 10:59 PM Muchun Song wrote: > > In the cgroup v1, we have a numa_stat interface. This is useful for > providing visibility into the numa locality information within an > memcg since the pages are allowed to be allocated from any physical > node. One of the use cases is eval

Re: [PATCH v4] mm: memcontrol: Add the missing numa_stat interface for cgroup v2

2020-09-15 Thread Randy Dunlap
Hi, On 9/14/20 10:58 PM, Muchun Song wrote: > In the cgroup v1, we have a numa_stat interface. This is useful for > providing visibility into the numa locality information within an > memcg since the pages are allowed to be allocated from any physical > node. One of the use cases is evaluating app

Re: [External] Re: [PATCH v4] mm: memcontrol: Add the missing numa_stat interface for cgroup v2

2020-09-15 Thread Muchun Song
On Tue, Sep 15, 2020 at 11:45 PM Randy Dunlap wrote: > > Hi, > > On 9/14/20 10:58 PM, Muchun Song wrote: > > In the cgroup v1, we have a numa_stat interface. This is useful for > > providing visibility into the numa locality information within an > > memcg since the pages are allowed to be allocat

Re: [External] Re: [PATCH v4] mm: memcontrol: Add the missing numa_stat interface for cgroup v2

2020-09-15 Thread Randy Dunlap
On 9/15/20 9:01 AM, Muchun Song wrote: > On Tue, Sep 15, 2020 at 11:45 PM Randy Dunlap wrote: >> >>> +static int __init numa_stats_init(void) >>> +{ >>> + int i; >>> + >>> + for (i = 0; i < ARRAY_SIZE(numa_stats); i++) { >>> +#ifdef CONFIG_TRANSPARENT_HUGEPAGE >>> + if (numa_s

[PATCH v4] mm: memcontrol: Add the missing numa_stat interface for cgroup v2

2020-09-14 Thread Muchun Song
In the cgroup v1, we have a numa_stat interface. This is useful for providing visibility into the numa locality information within an memcg since the pages are allowed to be allocated from any physical node. One of the use cases is evaluating application performance by combining this information wi