From: Roman Gushchin <roman.gushchin@linux.dev>
To: Tao Cui <cui.tao@linux.dev>
Cc: Johannes Weiner <hannes@cmpxchg.org>,
Michal Hocko <mhocko@suse.com>,
Shakeel Butt <shakeel.butt@linux.dev>,
muchun.song@linux.dev, cgroups@vger.kernel.org,
linux-mm@kvack.org, linux-kernel@vger.kernel.org,
Tao Cui <cuitao@kylinos.cn>
Subject: Re: [RFC PATCH 0/3] mm, memcg: isolate deprecated v1 state from struct mem_cgroup
Date: Mon, 02 Nov 2026 16:52:44 +0000 [thread overview]
Message-ID: <7ia41p9341lv.fsf@castle.c.googlers.com> (raw)
In-Reply-To: <32e80536-64e4-4ae3-a7da-4a37ed415f0a@linux.dev> (Tao Cui's message of "Mon, 2 Nov 2026 21:03:05 +0800")
Tao Cui <cui.tao@linux.dev> writes:
> Hi Roman,
>
> 在 2026/10/30 14:05, Roman Gushchin 写道:
>> I believe we want to delete this code sooner than later, why would we rewrite it before that?
>>
>
> My assumption was that v1 would still be maintained for some time
> before its eventual removal, and the goal was to reduce the coupling
> between the remaining v1 code and the shared core structures during
> that period, rather than to rewrite the implementation for its own
> sake.
I understand. But personally I don't see big issues with the current
code caused by the v1 stuff. Maybe I miss something...
The proposed changes on the other hand create difficulties for those
who use v1 and maintain some local changes or simple backport upstream
fixes. Of course, it's not a reason to not change the upstream code if
it's justified, but in this case I'm hardly convinced.
> If the expectation is that the remaining v1 implementation will be
> removed in the near future, then I agree this work becomes much harder
> to justify.
>
> Am I being too optimistic about how long the remaining v1 code is
> expected to stay around?
Well, Idk, it was considered deprecated for quite some time already,
so maybe announcing that it will be deleted in a year is reasonable?
Shakeel, Johannes, Michal, wdyt?
next prev parent reply other threads:[~2026-11-02 16:52 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-16 12:57 [RFC PATCH 0/3] mm, memcg: isolate deprecated v1 state from struct mem_cgroup Tao Cui
2026-09-16 12:57 ` [PATCH 1/3] mm, memcg: introduce struct mem_cgroup_v1 Tao Cui
2026-09-16 12:57 ` [PATCH 2/3] mm, memcg: move v1-only members into mem_cgroup_v1 Tao Cui
2026-09-16 12:57 ` [PATCH 3/3] docs: cgroup-v1: note the v1 memory controller implementation boundary Tao Cui
2026-09-17 20:26 ` [RFC PATCH 0/3] mm, memcg: isolate deprecated v1 state from struct mem_cgroup Shakeel Butt
2026-09-18 9:41 ` Michal Hocko
2026-09-18 10:22 ` Tao Cui
2026-09-18 16:24 ` Johannes Weiner
2026-10-30 5:40 ` Tao Cui
2026-10-30 6:05 ` Roman Gushchin
2026-11-02 13:03 ` Tao Cui
2026-11-02 16:52 ` Roman Gushchin [this message]
2026-11-02 17:29 ` Michal Hocko
2026-11-02 19:15 ` Johannes Weiner
2026-11-02 20:48 ` Michal Hocko
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7ia41p9341lv.fsf@castle.c.googlers.com \
--to=roman.gushchin@linux.dev \
--cc=cgroups@vger.kernel.org \
--cc=cui.tao@linux.dev \
--cc=cuitao@kylinos.cn \
--cc=hannes@cmpxchg.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=mhocko@suse.com \
--cc=muchun.song@linux.dev \
--cc=shakeel.butt@linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox