From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 84C0B4B7172; Thu, 24 Sep 2026 19:11:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790277093; cv=none; b=ms1kk0qjFtF3qvv/GjsicAdV0D3Qn+amRhb8lTkclHXRwAbrfPpTZO3uIZhXnOlrKQOoCzkWpe1GDVI2qCAX84rBbwg9mJNzw7AqdTUBeXWq2XMr/BPWlDh82caCsLgHxv1uaw8Ff3atKlr18MR7wqlTIAzDdauWz26ukRANCbE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790277093; c=relaxed/simple; bh=MuRaI8x6GXzrvwzPsbKH9WYnwNTgr9phZb/ghgk7nkI=; h=From:Subject:Date:Message-Id:MIME-Version:Content-Type:To:Cc; b=UCQzqkjCr6BB4Z2M4bvvKca/pJX3s52kJXQqZtHtWV3ddtMZwaMhNlVPguEks2PLO/X+XyAEM1TSh2FOS5AAi4XNRLHpNyk/6sxOVikdF6rJUjVnLgSI9wcKl2BK1yXGhwemSCKUbnhvfIUfUfLMaLQL305u9/A/f9vPwAofwWY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=UC/myava; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="UC/myava" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 0DCDF1E5E; Thu, 24 Sep 2026 12:11:26 -0700 (PDT) Received: from e129823.arm.com (e129823.arm.com [10.2.213.3]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 377863F86F; Thu, 24 Sep 2026 12:11:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1790277089; bh=MuRaI8x6GXzrvwzPsbKH9WYnwNTgr9phZb/ghgk7nkI=; h=From:Subject:Date:To:Cc:From; b=UC/myavav/0wDURi/e5uj2wHJrDw+VEwwctECMjrr5anj2mKS+Cp+xTXTVJB2nCIX 3B0zs4MYI65fHMkSpfEnoIhHBpmXNcWjB+5mC30jeDF+PJVWlfd3xmoX3RzB+757Ma DD7B0O4vG2wzemfy3Pr9Etn//5k5NFWuqiWEQvBc= From: Yeoreum Yun Subject: [PATCH v8 0/4] kselftest: mm: fix some failure of split_huge_page_test Date: Thu, 24 Sep 2026 20:11:15 +0100 Message-Id: <20260924-fix_split-v8-0-cba7359d882a@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit X-B4-Tracking: v=1; b=H4sIANN1tWoC/23QTWrDMBAF4KsErasyGo3+suo9SimWLDeCJg5SM C3Bd6+cTcelyzea7zHoLlquJTdxPNxFzUtpZb704J8OIp2Gy0eWZexZIKAFjyCn8vXerp/lJie ioENORNGLvn+tuT8+ul7fej6Vdpvr96N6Udv0v5ZFSZBDJA3J+OTS9DLU83Oaz2LrWDR3njvdn XYEiMbbgeLe0a8LgNxRd95MoAOYyUW1d4Y7x53ZHGL0Csj4mPbOMqeIO9udsnl0CrUH+HOnYw4 Vd667EU1OIShlAvuXdV1/ALSH+g+4AQAA To: Andrew Morton , David Hildenbrand , Lorenzo Stoakes , Zi Yan , Baolin Wang , "Liam R. Howlett" , Nico Pache , Ryan Roberts , Dev Jain , Barry Song , Lance Yang , Usama Arif , Vlastimil Babka , Mike Rapoport , Suren Baghdasaryan , Michal Hocko , Shuah Khan , Kevin Brodsky Cc: linux-mm@kvack.org, linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org, Yeoreum Yun X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=4292; i=yeoreum.yun@arm.com; h=from:subject:message-id; bh=MuRaI8x6GXzrvwzPsbKH9WYnwNTgr9phZb/ghgk7nkI=; b=owEB7QES/pANAwAKAW3Vw9FaxTEzAcsmYgBqtXXaIiT9BwKaFmYL+IRymkn1pS2cgMwCN8AJR jX5E9q/5yKJAbMEAAEKAB0WIQQtg+CS3QUzuFh1pJ1t1cPRWsUxMwUCarV12gAKCRBt1cPRWsUx M9CCC/4uKyIQhnU8tD0YMdkYRstxqpebMFuqAgDJCdJplKeWGziBfSV2btEkq3l6IaJ7C3bVHnb 3pDEFPHgL2O0ZT7QVQfq1qgfFJ+mf6/TiHidHnFWNwr+S77h7ltwAi3V8qzxk/x1mD2shDGGbsv anYESYv4f82duEtnyksxxPfYr3596oLcHCJXnSt6tUJsOcxpuOx3Vz+4HVbw03hNErKtMFZaAJu SQyqC338rRMCsFE1Evz/UI2geKUtW+V0UeednqsNO32cH0B7O0a6Q0Dw6tayJoMDRkrDKwUaHk7 2TcYBZqJ4Cr5jDyWUpfhgtU2vRigVuZtxfjomXTgTsMBogNAz4AQsDj385YN7FJOBjODTovX3AE WnAZ5UlSSdVRQuI8RuIDkrJeYD2xTmpmNehRhJNaFI+fFa6/7y+nyCM9BhcvlpBWrzqwlBmwLnd i12uy3RllTdMYX57SEYZAR1OyuxUu39IpgcEiiqjSJtqf4BOzdFYMkKQa/RRb2kmiU1Sw= X-Developer-Key: i=yeoreum.yun@arm.com; a=openpgp; fpr=2D83E092DD0533B85875A49D6DD5C3D15AC53133 split_huge_page_test can fail for the following reasons: 1. During the test, khugepaged may collapse previously split pages again, causing intermittent failures. 2. Since glibc commit 321e1fc73f (“malloc: Enable 2MB THP by default on AArch64”), glibc may call madvise(MADV_HUGEPAGE) for sufficiently large allocations made by memalign(). The underlying VMA may start at a different address from the aligned address returned by memalign(). Moreover, a subsequent madvise(MADV_HUGEPAGE) call does not split the VMA because it already has the same advice. This causes the test to fail because the check_huge_xxx() helpers incorrectly require the address returned by memalign() to match the VMA start address reported in /proc/self/smaps. Address these issues by applying MADV_NOHUGEPAGE after faulting in the huge page, preventing khugepaged from collapsing it again, and instead of relying on /proc/self/smaps, use /proc/self/pagemap and /proc/kpageflags to detect huge-page mappings and large folios: 1. If hpage_size == pmd_pagesize, check PAGE_IS_HUGE instead of using check_large_folios(), since only the mapping type matters. This identifies PMD-mapped huge pages. 2. Otherwise, use check_large_folios() to detect large folios. This covers mTHP cases. 3. Check the folio flags according to the type of huge page. Since check_huge_shmem() was required to distinguish shmem huge pages because /proc/self/smaps reports them using a dedicated “ShmemPmdMapped” entry, as opposed to “FilePmdMapped” for file-backed huge pages. Now that /proc/self/smaps is no longer used to detect huge pages and /proc/kpageflags is used instead, it is sufficient to distinguish between file-backed and anonymous pages since the ShmemPmdMapped is also kind of file-backed. Therefore, remove check_huge_shmem() and use check_huge_file() instead. This patch based on mm-unstable --- Changes in v8: - add intermediate patch for __check_pmd_huge(). - add r-b, a-b tags. - Link to v7: https://lore.kernel.org/r/20260921-fix_split-v7-0-d25ec991159f@arm.com Changes in v7: - change khugepaged's check_huge() callback. - Link to v6: https://lore.kernel.org/r/20260914-fix_split-v6-0-16ed7123800b@arm.com Changes in v6: - drop alloc_isolated_mem() patch. - remove check_huge_shmem(). - Link to v5: https://lore.kernel.org/r/20260907-fix_split-v5-0-822b810458bc@arm.com Changes in v5: - rename madv_nohuge() to disable_khugepaged(). - Link to v4: https://lore.kernel.org/r/20260902-fix_split-v4-0-85f03905f7b1@arm.com Changes in v4: - reparse commit message. - introduce alloc_isolated_mem(). - make madv_nohuge() helper according to suggestion. - Link to v3: https://lore.kernel.org/r/20260828-fix_split-v3-0-374022586a4b@arm.com Changes in v3: - change message in case of failure of madvise() with MADV_NOHUGEPAGE. - fix collapse_single_mthp (mthp_khugepaged:anon) test case failure. - Link to v2: https://lore.kernel.org/all/20260826-fix_split-v2-0-71153c7f579a@arm.com/ Changes in v2: - rebase to mm-unstable. - add message in case of failure of madvise() with MADV_NOHUGEPAGE. - fix wrong setup expected_huge in check_huge_shmem(). - Link to v1: https://lore.kernel.org/r/20260820-fix_split-v1-0-ab430c58c7cf@arm.com --- Yeoreum Yun (4): kselftest: mm: prevent random failure of huge page split for khugepaged kselftest: mm: replace usage of /proc/self/smaps for __check_pmd_huge() kselftest: mm: integrate huge page checks kselftest: mm: remove check_huge_shmem() tools/testing/selftests/mm/folio_split_race_test.c | 2 +- tools/testing/selftests/mm/khugepaged.c | 38 +----- tools/testing/selftests/mm/split_huge_page_test.c | 19 +++ tools/testing/selftests/mm/uffd-common.c | 4 +- tools/testing/selftests/mm/vm_util.c | 127 ++++++++++++--------- tools/testing/selftests/mm/vm_util.h | 1 - 6 files changed, 102 insertions(+), 89 deletions(-) --- base-commit: fff9c87241bb46aa09ed3a91d6002cba966d7776 change-id: 20260820-fix_split-f44939ec44b8 Best regards, -- Sincerely, Yeoreum Yun