[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aYIqa6jxn0TnpuKN@kernel.org>
Date: Tue, 3 Feb 2026 19:03:39 +0200
From: Mike Rapoport <rppt@...nel.org>
To: "Vishal Moola (Oracle)" <vishal.moola@...il.com>
Cc: linux-kernel@...r.kernel.org, linux-mm@...ck.org, x86@...nel.org,
akpm@...ux-foundation.org,
"Matthew Wilcox (Oracle)" <willy@...radead.org>,
Dave Hansen <dave.hansen@...ux.intel.com>,
Andy Lutomirski <luto@...nel.org>,
Peter Zijlstra <peterz@...radead.org>
Subject: Re: [PATCH v3 0/3] Convert 64-bit x86/mm/pat to ptdescs
On Mon, Feb 02, 2026 at 09:20:02AM -0800, Vishal Moola (Oracle) wrote:
> x86/mm/pat should be using ptdescs. One line has already been
> converted to pagetable_free(), while the allocation sites use
> get_free_pages(). This causes issues separately allocating ptdescs
> from struct page.
>
> These patches convert the allocation/free sites to use ptdescs. In
> the short term, this helps enable Matthew's work to allocate frozen
> pagetables[1]. And in the long term, this will help us cleanly split
> ptdesc allocations from struct page.
>
> The pgd_list should also be using ptdescs (for 32bit in this file). This
> can be done in a different patchset since there's other users of pgd_list
> that still need to be converted.
>
> [1] https://lore.kernel.org/linux-mm/20251113140448.1814860-1-willy@infradead.org/
> [2] https://lore.kernel.org/linux-mm/20251020001652.2116669-1-willy@infradead.org/
>
> ------
>
> I've also tested this on a tree that separately allocates ptdescs. That
> didn't find any lingering alloc/free issues.
>
> Based on current mm-new.
>
> v3:
> - Move comment regarding 32-bit conversions into the cover letter
> - Correct the handling for the pagetable_alloc() error path
>
> Vishal Moola (Oracle) (3):
> x86/mm/pat: Convert pte code to use ptdescs
> x86/mm/pat: Convert pmd code to use ptdescs
> x86/mm/pat: Convert split_large_page() to use ptdescs
Acked-by: Mike Rapoport (Microsoft) <rppt@...nel.org>
With some minor comments in patch 1/3
> arch/x86/mm/pat/set_memory.c | 56 +++++++++++++++++++++---------------
> 1 file changed, 33 insertions(+), 23 deletions(-)
>
> --
> 2.52.0
>
--
Sincerely yours,
Mike.
Powered by blists - more mailing lists