* Re: [PATCH -next] mm/memcontrol.c: make cgroup_memory_noswap static
2022-04-21 12:47 [PATCH -next] mm/memcontrol.c: make cgroup_memory_noswap static Lu Jialin
@ 2022-04-21 14:22 ` Johannes Weiner
2022-04-21 15:25 ` Shakeel Butt
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Johannes Weiner @ 2022-04-21 14:22 UTC (permalink / raw)
To: Lu Jialin
Cc: Andrew Morton, Michal Hocko, Roman Gushchin, Shakeel Butt,
Muchun Song, Matthew Wilcox, Yang Shi, Suren Baghdasaryan,
Vlastimil Babka, Wang Weiyang, cgroups, linux-kernel, linux-mm
On Thu, Apr 21, 2022 at 08:47:36PM +0800, Lu Jialin wrote:
> cgroup_memory_noswap is only used in mm/memcontrol.c, therefore just make
> it static, and remove export in include/linux/memcontrol.h
>
> Signed-off-by: Lu Jialin <lujialin4@huawei.com>
Nice. Unused since 2d1c498072de ("mm: memcontrol: make swap tracking
an integral part of memory control"), which removed the two instances
from mm/swap_cgroup.c.
Acked-by: Johannes Weiner <hannes@cmpxchg.org>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH -next] mm/memcontrol.c: make cgroup_memory_noswap static
2022-04-21 12:47 [PATCH -next] mm/memcontrol.c: make cgroup_memory_noswap static Lu Jialin
2022-04-21 14:22 ` Johannes Weiner
@ 2022-04-21 15:25 ` Shakeel Butt
2022-04-21 18:23 ` Roman Gushchin
2022-04-22 2:23 ` Muchun Song
3 siblings, 0 replies; 5+ messages in thread
From: Shakeel Butt @ 2022-04-21 15:25 UTC (permalink / raw)
To: Lu Jialin
Cc: Johannes Weiner, Andrew Morton, Michal Hocko, Roman Gushchin,
Muchun Song, Matthew Wilcox, Yang Shi, Suren Baghdasaryan,
Vlastimil Babka, Wang Weiyang, Cgroups, LKML, Linux MM
On Thu, Apr 21, 2022 at 5:48 AM Lu Jialin <lujialin4@huawei.com> wrote:
>
> cgroup_memory_noswap is only used in mm/memcontrol.c, therefore just make
> it static, and remove export in include/linux/memcontrol.h
>
> Signed-off-by: Lu Jialin <lujialin4@huawei.com>
Acked-by: Shakeel Butt <shakeelb@google.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH -next] mm/memcontrol.c: make cgroup_memory_noswap static
2022-04-21 12:47 [PATCH -next] mm/memcontrol.c: make cgroup_memory_noswap static Lu Jialin
2022-04-21 14:22 ` Johannes Weiner
2022-04-21 15:25 ` Shakeel Butt
@ 2022-04-21 18:23 ` Roman Gushchin
2022-04-22 2:23 ` Muchun Song
3 siblings, 0 replies; 5+ messages in thread
From: Roman Gushchin @ 2022-04-21 18:23 UTC (permalink / raw)
To: Lu Jialin
Cc: Johannes Weiner, Andrew Morton, Michal Hocko, Shakeel Butt,
Muchun Song, Matthew Wilcox, Yang Shi, Suren Baghdasaryan,
Vlastimil Babka, Wang Weiyang, cgroups, linux-kernel, linux-mm
On Thu, Apr 21, 2022 at 08:47:36PM +0800, Lu Jialin wrote:
> cgroup_memory_noswap is only used in mm/memcontrol.c, therefore just make
> it static, and remove export in include/linux/memcontrol.h
>
> Signed-off-by: Lu Jialin <lujialin4@huawei.com>
Acked-by: Roman Gushchin <roman.gushchin@linux.dev>
Thanks!
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH -next] mm/memcontrol.c: make cgroup_memory_noswap static
2022-04-21 12:47 [PATCH -next] mm/memcontrol.c: make cgroup_memory_noswap static Lu Jialin
` (2 preceding siblings ...)
2022-04-21 18:23 ` Roman Gushchin
@ 2022-04-22 2:23 ` Muchun Song
3 siblings, 0 replies; 5+ messages in thread
From: Muchun Song @ 2022-04-22 2:23 UTC (permalink / raw)
To: Lu Jialin
Cc: Johannes Weiner, Andrew Morton, Michal Hocko, Roman Gushchin,
Shakeel Butt, Matthew Wilcox, Yang Shi, Suren Baghdasaryan,
Vlastimil Babka, Wang Weiyang, cgroups, linux-kernel, linux-mm
On Thu, Apr 21, 2022 at 08:47:36PM +0800, Lu Jialin wrote:
> cgroup_memory_noswap is only used in mm/memcontrol.c, therefore just make
> it static, and remove export in include/linux/memcontrol.h
>
> Signed-off-by: Lu Jialin <lujialin4@huawei.com>
Reviewed-by: Muchun Song <songmuchun@bytedance.com>
Thanks.
^ permalink raw reply [flat|nested] 5+ messages in thread