On Wed, Jan 12, 2022 at 12:56 AM Michal Hocko <mhocko@xxxxxxxx> wrote: > > On Wed 12-01-22 00:46:34, Wei Yang wrote: > > On Tue, Jan 11, 2022 at 09:40:20AM +0100, Michal Hocko wrote: > > >On Tue 11-01-22 01:02:59, Wei Yang wrote: > > >> Instead of use "-1", let's use NUMA_NO_NODE for consistency. > > >> > > >> Signed-off-by: Wei Yang <richard.weiyang@xxxxxxxxx> > > > > > >I am not really sure this is worth it. After the merge window I plan to > > >post http://lkml.kernel.org/r/20211214100732.26335-1-mhocko@xxxxxxxxxx. > > > > Give me some time to understand it :-) > > Just for the record, here is what I have put on top of that series: > --- > From b7195eba02fe6308a6927450f4630057c05e808e Mon Sep 17 00:00:00 2001 > From: Wei Yang <richard.weiyang@xxxxxxxxx> > Date: Tue, 11 Jan 2022 09:45:25 +0100 > Subject: [PATCH] memcg: do not tweak node in alloc_mem_cgroup_per_node_info > > alloc_mem_cgroup_per_node_info is allocated for each possible node and > this used to be a problem because not !node_online nodes didn't have > appropriate data structure allocated. This has changed by "mm: handle > uninitialized numa nodes gracefully" so we can drop the special casing > here. > > Signed-off-by: Wei Yang <richard.weiyang@xxxxxxxxx> > Signed-off-by: Michal Hocko <mhocko@xxxxxxxx> Reviewed-by: Shakeel Butt <shakeelb@xxxxxxxxxx>