From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8A8FEC3DA6E for ; Thu, 4 Jan 2024 00:09:39 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id F38386B02DF; Wed, 3 Jan 2024 19:09:38 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id EC2116B02E1; Wed, 3 Jan 2024 19:09:38 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id D136D6B02D9; Wed, 3 Jan 2024 19:09:38 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id ADBE36B02CD for ; Wed, 3 Jan 2024 19:09:38 -0500 (EST) Received: from smtpin25.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id 7DE37402F4 for ; Thu, 4 Jan 2024 00:09:38 +0000 (UTC) X-FDA: 81639694836.25.7215525 Received: from out-186.mta1.migadu.com (out-186.mta1.migadu.com [95.215.58.186]) by imf29.hostedemail.com (Postfix) with ESMTP id 55180120002 for ; Thu, 4 Jan 2024 00:09:36 +0000 (UTC) Authentication-Results: imf29.hostedemail.com; dkim=pass header.d=linux.dev header.s=key1 header.b=s6nyM+eB; dmarc=pass (policy=none) header.from=linux.dev; spf=pass (imf29.hostedemail.com: domain of itaru.kitayama@linux.dev designates 95.215.58.186 as permitted sender) smtp.mailfrom=itaru.kitayama@linux.dev ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1704326976; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=8frufT5RlWWzipG2q0mi58H0Swj/zq/wij+Dt0ONUuI=; b=63ULw6uUXxvPMyc2tEELC0yrtfAcbC/5FMvEs5wl+QUJLP9C1mAyiI0+W1Y7IZCwWec19b 7wusv8oJTP+8F5Niol2zdrDhr2Ue1H9FQupqhpStsoyAyJxrWnSEOKBh5UDAgbZZkxQh5T 4M9gov7U6ZhiNELgrVoXlGh47IO6dh0= ARC-Authentication-Results: i=1; imf29.hostedemail.com; dkim=pass header.d=linux.dev header.s=key1 header.b=s6nyM+eB; dmarc=pass (policy=none) header.from=linux.dev; spf=pass (imf29.hostedemail.com: domain of itaru.kitayama@linux.dev designates 95.215.58.186 as permitted sender) smtp.mailfrom=itaru.kitayama@linux.dev ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1704326976; a=rsa-sha256; cv=none; b=sc9Q8QUC1C3ZNcMWts7bSFUA0un0AGhtah3b0lxPaee0fAGbrrGN8KDnWS70LR6l4Rj319 43B8i0N6oOYcGZkVORFqZR8JQypuBOHVSv2B4P5weKM0FZydjSKMtJyXPQpVhZki6LT/LW eEnyJBiWjkruNPb8QgL6OfP2dXwX5Ec= Date: Thu, 4 Jan 2024 09:09:19 +0900 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1704326973; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=8frufT5RlWWzipG2q0mi58H0Swj/zq/wij+Dt0ONUuI=; b=s6nyM+eBQL+MMaQjupAHuVd5/ftHBHJXFro7qAhxPZqeIX+BLZ7WcEv26gW7HWhaNU1/vn XwTUKpD6l7C8Z8S0tRqBGqQBQQA45jREYc2LBmc95Bb7weaQ1VSs+OwjhitwWxje3q68sr xgUXsNh8C4JNGh7AU01JFPSiTKUUftM= X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: Itaru Kitayama To: Ryan Roberts Cc: Andrew Morton , Matthew Wilcox , Yin Fengwei , David Hildenbrand , Yu Zhao , Catalin Marinas , Anshuman Khandual , Yang Shi , "Huang, Ying" , Zi Yan , Luis Chamberlain , Itaru Kitayama , "Kirill A. Shutemov" , John Hubbard , David Rientjes , Vlastimil Babka , Hugh Dickins , Kefeng Wang , Barry Song <21cnbao@gmail.com>, Alistair Popple , linux-mm@kvack.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v9 09/10] selftests/mm/cow: Generalize do_run_with_thp() helper Message-ID: References: <20231207161211.2374093-1-ryan.roberts@arm.com> <20231207161211.2374093-10-ryan.roberts@arm.com> <7d07caae-ae22-4cda-a3d0-4f542f52817a@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <7d07caae-ae22-4cda-a3d0-4f542f52817a@arm.com> X-Migadu-Flow: FLOW_OUT X-Rspamd-Server: rspam09 X-Rspamd-Queue-Id: 55180120002 X-Stat-Signature: ercto4juekzbrg9fi4fm9oxgrkj98s9i X-Rspam-User: X-HE-Tag: 1704326976-383126 X-HE-Meta: U2FsdGVkX19KjymDqb64MgtjF+zx9vA7KkM6++TE9Ha350ctolMs/oIMJy2g8QofMgT0Sa6Ltaq6TprkUKV+AIGTLO1NXKBZOsIMHmj7OzAxmpIw5JlSE5p1TwUI9jfaHfHQ68U0wRhZGHY+B2jYytzg36/FNp8V8FKcEXmmzlw8j1icoYdHXm3sNlDoctwvsei9BNnYUrBcW/Fq4z1mRJFEzTof7DbSbzzYbkSlLBzXolFykQqPSq4AKKaT1VOtfPlVmMVjv2KVgAnwMrmobLLvbO5LWBik6XgJ76nPG7SyIlOFho2Nl/uPz3U2eey1oULzQQnHHvLdkvbSZ/Cq5IKQNJngttKnRK4DG77Cuw60aMUUABdnFcs9vPZSj4yOSpEMr7zOC2AqWg2rF2hO/fQ6f61aFtlWU5fWWAr5Psqc2jsMKKQdvwxWX1X2BdnLQvWH2oGIlZLzUByrhMMAA7iuEmmR5It6gHslnQLpL/z62fFzXaIOpJR+aKMx2cydFSqTp5fNJm2e+MVGj0yMuD+VHBwICne6AuDgUzTxaHRVfGSvHc+02uhFtEtLcP+AVhUxrq4VEgjxeDEa0Fw91XNs/1K5LFvpNRnxJZ+vN+fVHUiKuOydbnMji3KyVbg4bFUngcvFwTYQnxkgq/f4+dsTp248vptZ8VRRbXpAtqyD0fhIr5LjrUcBG0yQZoENXHsq0gfcPACsk88CY3vzpv/GjFvvB8zzMcTejT2cj7krKn3uqpwxyuJMKvmNztdwIidr2CLPgqDMRZpJwHmdlEQJyqcDVXV/ZBBwLTFxjXDElgqV9ZA0TbIP5rKr2raXJCFOsutyJF/KoE1KBsUGPuSUpMu5DGMCrVVL5d0GZZUUuYxa/laAgvg3PxMF0mBy52AfJXSPIpixXXdjeNFviRfHYzBT/w+q8BNFFnOcnseOHUqtZd0KMmb79FDnjC2cr9mEUgj/n820+bd/fvU 0N4IqwEV XThipM171HaG7XO6SpXBvcef296yc91hlyzk22mihBh7MdssbB+KtB93ZQimzYxHgyyOdolpG3GC584eW7JowFyOxJWAk4FRJ30CWvXR8/IFx3+89o2BU4RnmchT082Ijxv1sWEqIemcDFjVUMCqCl/5Z8M0PIy7t65YLOkoqs3lyHB/PiqNU5yoIguzA+IQEyUHigTqDoNVq/o0Y7aa9v7rZ/rMwXGcLJySqMmGXQeFcM3q8A6M6Oae/4xyRcjEP0bJhizZtKAiFVeLB6owmqaXbxFJ0qFt9Jvvk4dKhKah/7tlRsV/DYE6FdkKyL9zB8tEk1oYh+gS/6Kw3b4vXTWiJYw== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On Wed, Jan 03, 2024 at 08:33:24AM +0000, Ryan Roberts wrote: > On 03/01/2024 06:21, Itaru Kitayama wrote: > > On Thu, Dec 07, 2023 at 04:12:10PM +0000, Ryan Roberts wrote: > >> do_run_with_thp() prepares (PMD-sized) THP memory into different states > >> before running tests. With the introduction of multi-size THP, we would > >> like to reuse this logic to also test those smaller THP sizes. So let's > >> add a thpsize parameter which tells the function what size THP it should > >> operate on. > >> > >> A separate commit will utilize this change to add new tests for > >> multi-size THP, where available. > >> > >> Reviewed-by: David Hildenbrand > >> Tested-by: Kefeng Wang > >> Tested-by: John Hubbard > >> Signed-off-by: Ryan Roberts > > > > Tested-by: Itaru Kitayama > > Thanks for testing! > > > > > I am replying to all this time; Ryan, do you think it's okay to run > > 700 of selftests/mm/cow tests? Even on FVP, they did not take longer > > though. > > What exactly is your concern, the amount of time it takes to run the tests? I've > found (at least on real HW) that the time it takes to run a test is dominated by > accessing the folio's memory. So adding all of the new tests that test sizes > between order-2 and PMD_ORDER-1 is ~equivalent to running the existing PMD_ORDER > tests twice. And the runtime of those is barely noticable compared to the > PUD_ORDER HugeTLB tests. So I don't think we are impacting runtime by much. > Sounds like your experience says that's also true for FVP? My primary concern was the time amount of time, but going back from mm-unstable/mm-stable, which contains your multi THP changes to Linus' master, I see what you were saying - the total number of tests for the "cow" program is the same. And I am convinced that the total time won't be changed much. On FVP EVP RevC, if those kselftests tests are not focusing on stress testing, tests are processed reasonably "fast". Lastly, as you tried to come up a series, the way mm kselftests executed is not so intuitive to me, as I am starting tests from the run_kselftest.sh script, I don't think mm tests are run using the -t (specific test) or -c (entire mm collection) options. Thanks, Itaru. > > > > >> --- > >> tools/testing/selftests/mm/cow.c | 121 +++++++++++++++++-------------- > >> 1 file changed, 67 insertions(+), 54 deletions(-) > >> > >> diff --git a/tools/testing/selftests/mm/cow.c b/tools/testing/selftests/mm/cow.c > >> index 7324ce5363c0..4d0b5a125d3c 100644 > >> --- a/tools/testing/selftests/mm/cow.c > >> +++ b/tools/testing/selftests/mm/cow.c > >> @@ -32,7 +32,7 @@ > >> > >> static size_t pagesize; > >> static int pagemap_fd; > >> -static size_t thpsize; > >> +static size_t pmdsize; > >> static int nr_hugetlbsizes; > >> static size_t hugetlbsizes[10]; > >> static int gup_fd; > >> @@ -734,7 +734,7 @@ enum thp_run { > >> THP_RUN_PARTIAL_SHARED, > >> }; > >> > >> -static void do_run_with_thp(test_fn fn, enum thp_run thp_run) > >> +static void do_run_with_thp(test_fn fn, enum thp_run thp_run, size_t thpsize) > >> { > >> char *mem, *mmap_mem, *tmp, *mremap_mem = MAP_FAILED; > >> size_t size, mmap_size, mremap_size; > >> @@ -759,11 +759,11 @@ static void do_run_with_thp(test_fn fn, enum thp_run thp_run) > >> } > >> > >> /* > >> - * Try to populate a THP. Touch the first sub-page and test if we get > >> - * another sub-page populated automatically. > >> + * Try to populate a THP. Touch the first sub-page and test if > >> + * we get the last sub-page populated automatically. > >> */ > >> mem[0] = 0; > >> - if (!pagemap_is_populated(pagemap_fd, mem + pagesize)) { > >> + if (!pagemap_is_populated(pagemap_fd, mem + thpsize - pagesize)) { > >> ksft_test_result_skip("Did not get a THP populated\n"); > >> goto munmap; > >> } > >> @@ -773,12 +773,14 @@ static void do_run_with_thp(test_fn fn, enum thp_run thp_run) > >> switch (thp_run) { > >> case THP_RUN_PMD: > >> case THP_RUN_PMD_SWAPOUT: > >> + assert(thpsize == pmdsize); > >> break; > >> case THP_RUN_PTE: > >> case THP_RUN_PTE_SWAPOUT: > >> /* > >> * Trigger PTE-mapping the THP by temporarily mapping a single > >> - * subpage R/O. > >> + * subpage R/O. This is a noop if the THP is not pmdsize (and > >> + * therefore already PTE-mapped). > >> */ > >> ret = mprotect(mem + pagesize, pagesize, PROT_READ); > >> if (ret) { > >> @@ -875,52 +877,60 @@ static void do_run_with_thp(test_fn fn, enum thp_run thp_run) > >> munmap(mremap_mem, mremap_size); > >> } > >> > >> -static void run_with_thp(test_fn fn, const char *desc) > >> +static void run_with_thp(test_fn fn, const char *desc, size_t size) > >> { > >> - ksft_print_msg("[RUN] %s ... with THP\n", desc); > >> - do_run_with_thp(fn, THP_RUN_PMD); > >> + ksft_print_msg("[RUN] %s ... with THP (%zu kB)\n", > >> + desc, size / 1024); > >> + do_run_with_thp(fn, THP_RUN_PMD, size); > >> } > >> > >> -static void run_with_thp_swap(test_fn fn, const char *desc) > >> +static void run_with_thp_swap(test_fn fn, const char *desc, size_t size) > >> { > >> - ksft_print_msg("[RUN] %s ... with swapped-out THP\n", desc); > >> - do_run_with_thp(fn, THP_RUN_PMD_SWAPOUT); > >> + ksft_print_msg("[RUN] %s ... with swapped-out THP (%zu kB)\n", > >> + desc, size / 1024); > >> + do_run_with_thp(fn, THP_RUN_PMD_SWAPOUT, size); > >> } > >> > >> -static void run_with_pte_mapped_thp(test_fn fn, const char *desc) > >> +static void run_with_pte_mapped_thp(test_fn fn, const char *desc, size_t size) > >> { > >> - ksft_print_msg("[RUN] %s ... with PTE-mapped THP\n", desc); > >> - do_run_with_thp(fn, THP_RUN_PTE); > >> + ksft_print_msg("[RUN] %s ... with PTE-mapped THP (%zu kB)\n", > >> + desc, size / 1024); > >> + do_run_with_thp(fn, THP_RUN_PTE, size); > >> } > >> > >> -static void run_with_pte_mapped_thp_swap(test_fn fn, const char *desc) > >> +static void run_with_pte_mapped_thp_swap(test_fn fn, const char *desc, size_t size) > >> { > >> - ksft_print_msg("[RUN] %s ... with swapped-out, PTE-mapped THP\n", desc); > >> - do_run_with_thp(fn, THP_RUN_PTE_SWAPOUT); > >> + ksft_print_msg("[RUN] %s ... with swapped-out, PTE-mapped THP (%zu kB)\n", > >> + desc, size / 1024); > >> + do_run_with_thp(fn, THP_RUN_PTE_SWAPOUT, size); > >> } > >> > >> -static void run_with_single_pte_of_thp(test_fn fn, const char *desc) > >> +static void run_with_single_pte_of_thp(test_fn fn, const char *desc, size_t size) > >> { > >> - ksft_print_msg("[RUN] %s ... with single PTE of THP\n", desc); > >> - do_run_with_thp(fn, THP_RUN_SINGLE_PTE); > >> + ksft_print_msg("[RUN] %s ... with single PTE of THP (%zu kB)\n", > >> + desc, size / 1024); > >> + do_run_with_thp(fn, THP_RUN_SINGLE_PTE, size); > >> } > >> > >> -static void run_with_single_pte_of_thp_swap(test_fn fn, const char *desc) > >> +static void run_with_single_pte_of_thp_swap(test_fn fn, const char *desc, size_t size) > >> { > >> - ksft_print_msg("[RUN] %s ... with single PTE of swapped-out THP\n", desc); > >> - do_run_with_thp(fn, THP_RUN_SINGLE_PTE_SWAPOUT); > >> + ksft_print_msg("[RUN] %s ... with single PTE of swapped-out THP (%zu kB)\n", > >> + desc, size / 1024); > >> + do_run_with_thp(fn, THP_RUN_SINGLE_PTE_SWAPOUT, size); > >> } > >> > >> -static void run_with_partial_mremap_thp(test_fn fn, const char *desc) > >> +static void run_with_partial_mremap_thp(test_fn fn, const char *desc, size_t size) > >> { > >> - ksft_print_msg("[RUN] %s ... with partially mremap()'ed THP\n", desc); > >> - do_run_with_thp(fn, THP_RUN_PARTIAL_MREMAP); > >> + ksft_print_msg("[RUN] %s ... with partially mremap()'ed THP (%zu kB)\n", > >> + desc, size / 1024); > >> + do_run_with_thp(fn, THP_RUN_PARTIAL_MREMAP, size); > >> } > >> > >> -static void run_with_partial_shared_thp(test_fn fn, const char *desc) > >> +static void run_with_partial_shared_thp(test_fn fn, const char *desc, size_t size) > >> { > >> - ksft_print_msg("[RUN] %s ... with partially shared THP\n", desc); > >> - do_run_with_thp(fn, THP_RUN_PARTIAL_SHARED); > >> + ksft_print_msg("[RUN] %s ... with partially shared THP (%zu kB)\n", > >> + desc, size / 1024); > >> + do_run_with_thp(fn, THP_RUN_PARTIAL_SHARED, size); > >> } > >> > >> static void run_with_hugetlb(test_fn fn, const char *desc, size_t hugetlbsize) > >> @@ -1091,15 +1101,15 @@ static void run_anon_test_case(struct test_case const *test_case) > >> > >> run_with_base_page(test_case->fn, test_case->desc); > >> run_with_base_page_swap(test_case->fn, test_case->desc); > >> - if (thpsize) { > >> - run_with_thp(test_case->fn, test_case->desc); > >> - run_with_thp_swap(test_case->fn, test_case->desc); > >> - run_with_pte_mapped_thp(test_case->fn, test_case->desc); > >> - run_with_pte_mapped_thp_swap(test_case->fn, test_case->desc); > >> - run_with_single_pte_of_thp(test_case->fn, test_case->desc); > >> - run_with_single_pte_of_thp_swap(test_case->fn, test_case->desc); > >> - run_with_partial_mremap_thp(test_case->fn, test_case->desc); > >> - run_with_partial_shared_thp(test_case->fn, test_case->desc); > >> + if (pmdsize) { > >> + run_with_thp(test_case->fn, test_case->desc, pmdsize); > >> + run_with_thp_swap(test_case->fn, test_case->desc, pmdsize); > >> + run_with_pte_mapped_thp(test_case->fn, test_case->desc, pmdsize); > >> + run_with_pte_mapped_thp_swap(test_case->fn, test_case->desc, pmdsize); > >> + run_with_single_pte_of_thp(test_case->fn, test_case->desc, pmdsize); > >> + run_with_single_pte_of_thp_swap(test_case->fn, test_case->desc, pmdsize); > >> + run_with_partial_mremap_thp(test_case->fn, test_case->desc, pmdsize); > >> + run_with_partial_shared_thp(test_case->fn, test_case->desc, pmdsize); > >> } > >> for (i = 0; i < nr_hugetlbsizes; i++) > >> run_with_hugetlb(test_case->fn, test_case->desc, > >> @@ -1120,7 +1130,7 @@ static int tests_per_anon_test_case(void) > >> { > >> int tests = 2 + nr_hugetlbsizes; > >> > >> - if (thpsize) > >> + if (pmdsize) > >> tests += 8; > >> return tests; > >> } > >> @@ -1329,7 +1339,7 @@ static void run_anon_thp_test_cases(void) > >> { > >> int i; > >> > >> - if (!thpsize) > >> + if (!pmdsize) > >> return; > >> > >> ksft_print_msg("[INFO] Anonymous THP tests\n"); > >> @@ -1338,13 +1348,13 @@ static void run_anon_thp_test_cases(void) > >> struct test_case const *test_case = &anon_thp_test_cases[i]; > >> > >> ksft_print_msg("[RUN] %s\n", test_case->desc); > >> - do_run_with_thp(test_case->fn, THP_RUN_PMD); > >> + do_run_with_thp(test_case->fn, THP_RUN_PMD, pmdsize); > >> } > >> } > >> > >> static int tests_per_anon_thp_test_case(void) > >> { > >> - return thpsize ? 1 : 0; > >> + return pmdsize ? 1 : 0; > >> } > >> > >> typedef void (*non_anon_test_fn)(char *mem, const char *smem, size_t size); > >> @@ -1419,7 +1429,7 @@ static void run_with_huge_zeropage(non_anon_test_fn fn, const char *desc) > >> } > >> > >> /* For alignment purposes, we need twice the thp size. */ > >> - mmap_size = 2 * thpsize; > >> + mmap_size = 2 * pmdsize; > >> mmap_mem = mmap(NULL, mmap_size, PROT_READ | PROT_WRITE, > >> MAP_PRIVATE | MAP_ANONYMOUS, -1, 0); > >> if (mmap_mem == MAP_FAILED) { > >> @@ -1434,11 +1444,11 @@ static void run_with_huge_zeropage(non_anon_test_fn fn, const char *desc) > >> } > >> > >> /* We need a THP-aligned memory area. */ > >> - mem = (char *)(((uintptr_t)mmap_mem + thpsize) & ~(thpsize - 1)); > >> - smem = (char *)(((uintptr_t)mmap_smem + thpsize) & ~(thpsize - 1)); > >> + mem = (char *)(((uintptr_t)mmap_mem + pmdsize) & ~(pmdsize - 1)); > >> + smem = (char *)(((uintptr_t)mmap_smem + pmdsize) & ~(pmdsize - 1)); > >> > >> - ret = madvise(mem, thpsize, MADV_HUGEPAGE); > >> - ret |= madvise(smem, thpsize, MADV_HUGEPAGE); > >> + ret = madvise(mem, pmdsize, MADV_HUGEPAGE); > >> + ret |= madvise(smem, pmdsize, MADV_HUGEPAGE); > >> if (ret) { > >> ksft_test_result_fail("MADV_HUGEPAGE failed\n"); > >> goto munmap; > >> @@ -1457,7 +1467,7 @@ static void run_with_huge_zeropage(non_anon_test_fn fn, const char *desc) > >> goto munmap; > >> } > >> > >> - fn(mem, smem, thpsize); > >> + fn(mem, smem, pmdsize); > >> munmap: > >> munmap(mmap_mem, mmap_size); > >> if (mmap_smem != MAP_FAILED) > >> @@ -1650,7 +1660,7 @@ static void run_non_anon_test_case(struct non_anon_test_case const *test_case) > >> run_with_zeropage(test_case->fn, test_case->desc); > >> run_with_memfd(test_case->fn, test_case->desc); > >> run_with_tmpfile(test_case->fn, test_case->desc); > >> - if (thpsize) > >> + if (pmdsize) > >> run_with_huge_zeropage(test_case->fn, test_case->desc); > >> for (i = 0; i < nr_hugetlbsizes; i++) > >> run_with_memfd_hugetlb(test_case->fn, test_case->desc, > >> @@ -1671,7 +1681,7 @@ static int tests_per_non_anon_test_case(void) > >> { > >> int tests = 3 + nr_hugetlbsizes; > >> > >> - if (thpsize) > >> + if (pmdsize) > >> tests += 1; > >> return tests; > >> } > >> @@ -1681,10 +1691,13 @@ int main(int argc, char **argv) > >> int err; > >> > >> pagesize = getpagesize(); > >> - thpsize = read_pmd_pagesize(); > >> - if (thpsize) > >> + pmdsize = read_pmd_pagesize(); > >> + if (pmdsize) { > >> + ksft_print_msg("[INFO] detected PMD size: %zu KiB\n", > >> + pmdsize / 1024); > >> ksft_print_msg("[INFO] detected THP size: %zu KiB\n", > >> - thpsize / 1024); > >> + pmdsize / 1024); > >> + } > >> nr_hugetlbsizes = detect_hugetlb_page_sizes(hugetlbsizes, > >> ARRAY_SIZE(hugetlbsizes)); > >> detect_huge_zeropage(); > >> -- > >> 2.25.1 > >> >