[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <c4ddc913-fcf5-43c4-b95e-7c71748dba03@vivo.com>
Date: Fri, 25 Apr 2025 09:11:32 +0800
From: Huan Yang <link@...o.com>
To: Shakeel Butt <shakeel.butt@...ux.dev>
Cc: Johannes Weiner <hannes@...xchg.org>, Michal Hocko <mhocko@...nel.org>,
Roman Gushchin <roman.gushchin@...ux.dev>,
Muchun Song <muchun.song@...ux.dev>,
Andrew Morton <akpm@...ux-foundation.org>, Petr Mladek <pmladek@...e.com>,
Sebastian Andrzej Siewior <bigeasy@...utronix.de>,
Francesco Valla <francesco@...la.it>,
Huang Shijie <shijie@...amperecomputing.com>, KP Singh <kpsingh@...nel.org>,
"Paul E. McKenney" <paulmck@...nel.org>,
Rasmus Villemoes <linux@...musvillemoes.dk>,
"Uladzislau Rezki (Sony)" <urezki@...il.com>,
Guo Weikang <guoweikang.kernel@...il.com>,
Raul E Rangel <rrangel@...omium.org>, cgroups@...r.kernel.org,
linux-mm@...ck.org, linux-kernel@...r.kernel.org,
Boqun Feng <boqun.feng@...il.com>, Geert Uytterhoeven
<geert@...ux-m68k.org>, Paul Moore <paul@...l-moore.com>,
"Mike Rapoport (Microsoft)" <rppt@...nel.org>, opensource.kernel@...o.com
Subject: Re: [PATCH v2 3/3] mm/memcg: introduce mem_cgroup_early_init
在 2025/4/25 00:00, Shakeel Butt 写道:
> On Thu, Apr 24, 2025 at 08:09:29PM +0800, Huan Yang wrote:
>> When cgroup_init() creates root_mem_cgroup through css_online callback,
>> some critical resources might not be fully initialized, forcing later
>> operations to perform conditional checks for resource availability.
>>
>> This patch introduces mem_cgroup_early_init() to address the init order,
>> it invoke before cgroup_init, so, compare mem_cgroup_init which invoked
>> by initcall, mem_cgroup_early_init can use to prepare some key resources
>> before root_mem_cgroup alloc.
>>
>> Signed-off-by: Huan Yang <link@...o.com>
>> Suggested-by: Shakeel Butt <shakeel.butt@...ux.dev>
> Please move this patch as the first patch of the series and also remove
> the "early" from the function name as it has a different meaning in the
OK,
Thanks.
> context of cgroup init. Something like either memcg_init() or
> memcg_kmem_caches_init().
Powered by blists - more mailing lists