Re: [PATCH v4 16/21] mm/mm_init: simplify deferred_free_pages() migratetype init

Andrew Morton <[email protected]> Mon, 27 Jul 2026 15:53:56 -0700
Newsgroups org.infradead.lists.kexec,org.kernel.vger.linux-kernel,org.kvack.linux-mm
Message-ID <[email protected]>
On Mon, 27 Jul 2026 13:08:18 +0300 Mike Rapoport <[email protected]> wrote:

> On Mon, Jul 27, 2026 at 11:43:31AM +0800, Muchun Song wrote:
> > 
> > 
> > > On Jul 27, 2026, at 11:25, Andrew Morton <[email protected]> wrote:
> > > 
> > > On Sun, 26 Jul 2026 13:16:54 +0300 Mike Rapoport <[email protected]> wrote:
> > > 
> > >> On Sat, Jul 25, 2026 at 07:21:16PM +0200, Pratyush Yadav wrote:
> > >>> From: Muchun Song <[email protected]>
> > >>> 
> > >>> deferred_free_pages() open-codes two loops to initialize the pageblock
> > >>> migratetype for a range of pages.
> > >>> 
> > >>> Replace them with pageblock_migratetype_init_range() to remove the
> > >>> duplication and make the code clearer (Note that deferred_free_pages() may
> > >>> be called from atomic context).
> > >>> 
> > >>> Link: https://lore.kernel.org/[email protected]
> > >>> Signed-off-by: Muchun Song <[email protected]>
> > >>> Acked-by: Mike Rapoport (Microsoft) <[email protected]>
> > >>> Acked-by: Oscar Salvador <[email protected]>
> > >>> Signed-off-by: Pratyush Yadav (Google) <[email protected]>
> > >>> ---
> > >>> 
> > >>> Notes:
> > >>>    This patch is taken from Muchun's series [0]. The patch as of today is
> > >>>    in mm-unstable.
> > >> 
> > >> @Andrew,
> > >> 
> > >> It looks like it'll take some time for this patch to get into mm-stable so
> > >> we can't base this work on a branch in the mm tree :(
> > > 
> > > I didn't know anyone was using mm-stable :(
> 
> Maybe nobody uses it because it's not really stable? ;-)

Well it's supposed to be.  Rebases are rare.

The larger problem is that it doesn't contain anything until -rc5ish
because we haven't stabilized anything yet!

> > I'd like to confirm that this is a completely independent cleanup patch,
> > so it can be merged on its own.
> 
> That's great, thanks for confirming!
> 
> Then I think the easiest would be to take "mm/mm_init: simplify
> deferred_free_pages() migratetype init" via liveupdate tree.

No probs, I'll drop it when Mark reports the duplicate.