Re: [RFC PATCH 03/57] mm/collapse: add collapse.h for the shared collapse state

[Lorenzo Stoakes (ARM)]

On Mon, Aug 24, 2026 at 01:36:03PM +0100, Kiryl Shutsemau wrote:
> On Mon, Aug 24, 2026 at 02:28:39PM +0200, David Hildenbrand (Arm) wrote:
> > On 8/24/26 14:10, Kiryl Shutsemau wrote:
> > > On Mon, Aug 24, 2026 at 01:47:54PM +0200, David Hildenbrand (Arm) wrote:
> > >> On 8/20/26 13:11, Kiryl Shutsemau wrote:
> > >>>
> > >>> I went for a clean collapse.c on purpose: the engine is new code, and a
> > >>> new file keeps it readable while both mechanisms are in the tree, with 28
> > >>> as a plain delete from khugepaged.c.
> > >>>
> > >>> Your shape is less churn, though, and it keeps blame: 29-35 move about
> > >>> 1700 lines as add-and-delete today, and most of that would stay put, with
> > >>> only the daemon -- mm_slot, the scan loop, sysfs -- extracted into a new
> > >>> khugepaged.c.
> > >>>
> > >>> If you prefer it that way, I can give it a try for v2.

Please don't respin this as a 57 patch series.

You need to break this into a smaller initial series that lays some
foundations.

Cf. what I've done with scalable CoW series as an example.

> > >>>
> > >>> The rename has to open the series, so it will collide with anything in
> > >>> flight that touches khugepaged.c.
> > >>>
> > >>
> > >> Kiryl, I have to ask, are you using AI to write your replies?
> > >
> > > Yes. I use it check my claims and fix my grammar.
> > >
> > > Why?
> > >
> >
> > It doesn't read the way you would usually write. In a bad way.
> >
> > E.g., "Your shape", "has to open the series"
> >
> > Consider telling the LLM to not change your writing style and focus only on
> > fixing grammar.
>
> Right. Asking for rewrite with proper grammar was a bad move on my part.

Even this reads as AI-generated... Consider writing the replies yourself
please.

You've generated a ton of work here and the AI-style giant replies just
create more.

>
> --
>   Kiryl Shutsemau / Kirill A. Shutemov

--
Cheers, Lorenzo