Re: [RESEND RFC PATCH v2 0/5] mm/damon: add mTHP collapse and split actions

[wang lian <lianux.mm@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 4305 bytes --]

Hi,Lorenzo

> On Jul 2, 2026, at 18:23, Lorenzo Stoakes <ljs@kernel.org> wrote:
> 
> +cc all those you missed.
> 
> I really need to write a bot to do this, because I'm getting a little tired of
> pointing this out :))
> 
> On Thu, Jul 02, 2026 at 05:46:28PM +0800, Lian Wang wrote:
>> include/linux/damon.h      |  10 +++
>> include/linux/khugepaged.h |   9 +++
>> mm/damon/core.c            |   2 +
>> mm/damon/sysfs-schemes.c   |  77 ++++++++++++++++++++++
>> mm/damon/vaddr.c           | 128 +++++++++++++++++++++++++++++++++++++
>> mm/khugepaged.c            |  46 +++++++++++++
>> 6 files changed, 272 insertions(+)
> 
> You are doing damon changes, and that belongs to SJ, sure.
> 
> But you're also changing core THP code? Please ensure you cc- THP people because
> without our approval this cannot be merged:
> 
> $ scripts/get_maintainer.pl 20260702094633.75658-1-lianux.mm@gmail.com.mbx <mailto:20260702094633.75658-1-lianux.mm@gmail.com.mbx>
> SJ Park <sj@kernel.org <mailto:sj@kernel.org>> (maintainer:DAMON)
> Andrew Morton <akpm@linux-foundation.org <mailto:akpm@linux-foundation.org>> (maintainer:MEMORY MANAGEMENT - THP (TRANSPARENT HUGE PAGE))
> David Hildenbrand <david@kernel.org <mailto:david@kernel.org>> (maintainer:MEMORY MANAGEMENT - THP (TRANSPARENT HUGE PAGE))
> Lorenzo Stoakes <ljs@kernel.org <mailto:ljs@kernel.org>> (maintainer:MEMORY MANAGEMENT - THP (TRANSPARENT HUGE PAGE))
> Zi Yan <ziy@nvidia.com <mailto:ziy@nvidia.com>> (reviewer:MEMORY MANAGEMENT - THP (TRANSPARENT HUGE PAGE))
> Baolin Wang <baolin.wang@linux.alibaba.com <mailto:baolin.wang@linux.alibaba.com>> (reviewer:MEMORY MANAGEMENT - THP (TRANSPARENT HUGE PAGE))
> "Liam R. Howlett" <liam@infradead.org <mailto:liam@infradead.org>> (reviewer:MEMORY MANAGEMENT - THP (TRANSPARENT HUGE PAGE))
> Nico Pache <npache@redhat.com <mailto:npache@redhat.com>> (reviewer:MEMORY MANAGEMENT - THP (TRANSPARENT HUGE PAGE))
> Ryan Roberts <ryan.roberts@arm.com <mailto:ryan.roberts@arm.com>> (reviewer:MEMORY MANAGEMENT - THP (TRANSPARENT HUGE PAGE))
> Dev Jain <dev.jain@arm.com <mailto:dev.jain@arm.com>> (reviewer:MEMORY MANAGEMENT - THP (TRANSPARENT HUGE PAGE))
> Barry Song <baohua@kernel.org <mailto:baohua@kernel.org>> (reviewer:MEMORY MANAGEMENT - THP (TRANSPARENT HUGE PAGE))
> Lance Yang <lance.yang@linux.dev <mailto:lance.yang@linux.dev>> (reviewer:MEMORY MANAGEMENT - THP (TRANSPARENT HUGE PAGE))
> damon@lists.linux.dev <mailto:damon@lists.linux.dev> (open list:DAMON)
> linux-mm@kvack.org <mailto:linux-mm@kvack.org> (open list:DAMON)
> linux-kernel@vger.kernel.org <mailto:linux-kernel@vger.kernel.org> (open list)
> 
>> 
>> --
>> 2.50.1 (Apple Git-155)
>> 
> 

Thank you for pointing this out again, and I deeply appreciate all the tedious 
work you do as a maintainer to keep the mailing lists aligned. 

As a newcomer trying to transition into complex feature development, I must 
admit my initial design is far from perfect. I leveraged AI tools to assist 
with certain implementation paths, and although I extensively self-reviewed, 
tested, and reasoned through every single line of code, your and SJ's 
feedback clearly shows there are still major architectural flaws. Just like my 
very first kernel patch—which was strictly reviewed and guided by you and David H. 
—I know that your sharp feedback is exactly what will help me grow as a kernel engineer.

I will completely step back and radically rethink the design of this series, 
especially regarding the cross-subsystem encapsulation and locking hazards. 

As for that bot you mentioned getting tired of running manually—let me write 
it for you. An imperfect newcomer who just made these exact mistakes knows 
precisely where the traps are. I can craft a pre-flight patch check script or 
a CI bot that automatically validates the `get_maintainer.pl` output against 
the patch file diffstat before any email is dispatched, throwing a loud warning 
if core MM files are touched but their respective maintainers are missing from 
the CC line. 

What do you think? I would love to build this tool to save your and other 
maintainers' valuable time in the future.

Thanks again for steering me in the right direction!

Best regards,
Wang Lian
> Thanks, Lorenzo


[-- Attachment #2: Type: text/html, Size: 37873 bytes --]