On Fri, 10 Jul 2026 17:24:27 +0800, Muchun Song <[email protected]> wrote: > The hugetlb vmemmap selftest was added to check the old HVO > layout where tail vmemmap pages reused the head page. That > assumption no longer matches the current HVO mapping layout. > > HVO now keeps a private backing page for the head vmemmap page > and remaps redundant tail vmemmap pages to a shared read-only > backing page. The old page flag check is therefore testing an > obsolete implementation detail rather than the current ABI or > behavior. > > Remove the stale test and its build, run, and ignore entries. > > Signed-off-by: Muchun Song <[email protected]>
LGTM, always happy to see red in the diffstat! And we definitely do not want a test testing something that isn't real (...!). > > > diff --git a/tools/testing/selftests/mm/.gitignore > b/tools/testing/selftests/mm/.gitignore > index 9ccd9e1447e6..227476d8ff1f 100644 > --- a/tools/testing/selftests/mm/.gitignore > +++ b/tools/testing/selftests/mm/.gitignore > @@ -3,11 +3,9 @@ cow > hugepage-mmap > hugepage-mremap > hugepage-shm > -hugepage-vmemmap > hugetlb-mmap > hugetlb-mremap > hugetlb-shm > -hugetlb-vmemmap It was here twice :) wow. Reviewed-by: Lorenzo Stoakes <[email protected]> -- Lorenzo Stoakes <[email protected]>
