From nobody Sat Sep 27 20:25:44 2025 Received: from mail-qv1-f48.google.com (mail-qv1-f48.google.com [209.85.219.48]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 95BA3305E3C; Fri, 15 Aug 2025 13:56:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.48 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1755266166; cv=none; b=ue3Dtgr1YdwaHl2F1TczM9u8EC4YND+NQgnrvCyoTYSmr2DGDKtTW6dm/jp1f8sAZ2Y5Y1NXy/jK0hs9IpHHk853TaDBJglmRX/zH53ndJzL3xzwXBvEHoDFW8TNeG8XFauf66OeaCfDxIhYC7BSfkOHPUxrVtOgK/PFSnOnmgo= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1755266166; c=relaxed/simple; bh=9Nh5dCrF7vmkiPO6+SZKBUdCvfvTaXBCeQl+556ey5A=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=hWGBQnFvqG8eCvDmDS++LX6A9dwlqJ3vZd9hDxE0tcO49XdkAf8to5WeenQnNj8oqj6/KroDW1oafS+SnRYAAISEUp6q7dtUHY3va6wDUl/f2/n2ymXlcODtljmSWJZ7XVg81ahNtwlobqrxLrCompMc9SryzdiTnvfphP6KKcA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=kA4lBaui; arc=none smtp.client-ip=209.85.219.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="kA4lBaui" Received: by mail-qv1-f48.google.com with SMTP id 6a1803df08f44-70a88ddb1a2so18633756d6.0; Fri, 15 Aug 2025 06:56:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1755266163; x=1755870963; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=ThKI922dULUmYFiaW3lL2ZDEHLRMIx7W7/OmILgNixg=; b=kA4lBauixBhyLcAKgZeDSW5i89lXD8ocvkZw6eNWZ20J3bawHCgwSi6WVGUV5fTSr1 Ok2yAcqJ1swHLDUhQN+oW4xwvgyegD1EE6NQePRC1csZQHQ58gPwKaRPBhKCzx2FW+Jw P0BOaVjj69WdxwI9OdLRT3GioQW9jF9BwyKYIXLS4cvWWknA/JgsuU0wubH/TR2NMfn/ Ins/AlfOq9BSN45NVh53YsfMWUn9aMX0rB+LT8vh3v7OoS62c0igL6vQd+BQ98qBkPhA vSjmkqFfGs7qd/D+98UcmVTNPKwUYbOh5VL7BO7aLGOvJZ7bnUXJZjyIeHPkm7Fq76wZ OaPA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1755266163; x=1755870963; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=ThKI922dULUmYFiaW3lL2ZDEHLRMIx7W7/OmILgNixg=; b=UCG6eJOUlaQUj1t8TyaEX6hdL81tGXFCdongk00Gt6ElBUTSrhF2Kqi0ljuMGSEtvm uxb0wmXZKSZbVWtT0qSuigjS/3n4DBIPwLJrKpcTf0Kugjsh3jJlFEro7G5R+2A/3DZR STmAyfZb3l9M18JVR+p4p2hQDXxY6CTzCB37rZbKD/9/ORFiEdOgWv+4fTJlgvvkR+uo TdupIix/mu45u3EXhhp4WE4CinY4dpsKs9uhGNc5irJVa7lmEPYB2oMsL5cwi0+XZQUE XAIF3yAqJb3Y0FlgAc7HImgaDglec55i+CN/KoTPU9IxC5tJqjBL+Uc/Vl98VFI6i6fh lRDA== X-Forwarded-Encrypted: i=1; AJvYcCWxF/v+B6PD8Bz7cPwVgdxrK4chGvdJF12ml41hTuVbyIkir7o96HXpo0NCFpON3PPRrXmh0KIPwnTUQzdD@vger.kernel.org, AJvYcCXhcZngeMJZUGrLRzpEXwBHhMOJ2K/I5XiYhr9ejMGLI2mghlEOpWuAp+trWoObo5i3MXFr4HTbTQE=@vger.kernel.org X-Gm-Message-State: AOJu0YxOywqR+0BLoezu1IgAogNHI1DUnoHfIX6vFKGvi5/b78q0RbQR HnLqjUrXX7odUggSoonjw3K9vNkTFgTHVpSpmqrNfyr7LnlJyJ+3cUd1 X-Gm-Gg: ASbGncuog93Q0VtcZR5DKtTVrBl17eu1kzJ30ZZZ55e+COkqKWA89HZBOTsOSZU/ujw pE+v66PiR5+HPb2GKJCjy+yTud88VNjjs/gLHQzcqoUlfyGr4txbHzlHfgRVvrI0R4wn/bGJch5 QUG54Um+YCxANlIlyGR5yoGGoFOG4vLg8Hg302svGQ+Jw3ebL1jpLbFG2VAMM+g9eul1FnwUjd4 qRJQ9xkGNAE6ZMG+t91Ek+yIeA7DLO2D5cr5elimukptwr/wre7l0WMJKUGtqKOe0X+BE96ZPZd l5agOrvSjXdwU+C1BxoGLz4XYiFleYHmPHQN1qJpxxrRRrUznq7asfDYXgIr33Mg0putnXxIVld dS22TXCkakaXW++mLhY6L X-Google-Smtp-Source: AGHT+IG+TCWxSTubxOqPY0sQAGowCS1LkSUfiep5FDN5JBRpx5/efo7iB20uzg87MScqVq8pUB5pxg== X-Received: by 2002:a05:6214:246c:b0:709:7345:9aa3 with SMTP id 6a1803df08f44-70ba6524e9fmr26519706d6.14.1755266163499; Fri, 15 Aug 2025 06:56:03 -0700 (PDT) Received: from localhost ([2a03:2880:20ff:72::]) by smtp.gmail.com with ESMTPSA id 6a1803df08f44-70ba95d3449sm7954516d6.77.2025.08.15.06.56.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 15 Aug 2025 06:56:02 -0700 (PDT) From: Usama Arif To: Andrew Morton , david@redhat.com, linux-mm@kvack.org Cc: linux-fsdevel@vger.kernel.org, corbet@lwn.net, rppt@kernel.org, surenb@google.com, mhocko@suse.com, hannes@cmpxchg.org, baohua@kernel.org, shakeel.butt@linux.dev, riel@surriel.com, ziy@nvidia.com, laoar.shao@gmail.com, dev.jain@arm.com, baolin.wang@linux.alibaba.com, npache@redhat.com, lorenzo.stoakes@oracle.com, Liam.Howlett@oracle.com, ryan.roberts@arm.com, vbabka@suse.cz, jannh@google.com, Arnd Bergmann , sj@kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, kernel-team@meta.com, Usama Arif Subject: [PATCH v5 6/7] selftests: prctl: introduce tests for disabling THPs completely Date: Fri, 15 Aug 2025 14:54:58 +0100 Message-ID: <20250815135549.130506-7-usamaarif642@gmail.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20250815135549.130506-1-usamaarif642@gmail.com> References: <20250815135549.130506-1-usamaarif642@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" The test will set the global system THP setting to never, madvise or always depending on the fixture variant and the 2M setting to inherit before it starts (and reset to original at teardown). The fixture setup will also test if PR_SET_THP_DISABLE prctl call can be made to disable all THPs and skip if it fails. This tests if the process can: - successfully get the policy to disable THPs completely. - never get a hugepage when the THPs are completely disabled with the prctl, including with MADV_HUGE and MADV_COLLAPSE. - successfully reset the policy of the process. - after reset, only get hugepages with: - MADV_COLLAPSE when policy is set to never. - MADV_HUGE and MADV_COLLAPSE when policy is set to madvise. - always when policy is set to "always". - never get a THP with MADV_NOHUGEPAGE. - repeat the above tests in a forked process to make sure the policy is carried across forks. Signed-off-by: Usama Arif Acked-by: David Hildenbrand Reviewed-by: Lorenzo Stoakes --- tools/testing/selftests/mm/.gitignore | 1 + tools/testing/selftests/mm/Makefile | 1 + .../testing/selftests/mm/prctl_thp_disable.c | 175 ++++++++++++++++++ tools/testing/selftests/mm/thp_settings.c | 9 +- tools/testing/selftests/mm/thp_settings.h | 1 + 5 files changed, 186 insertions(+), 1 deletion(-) create mode 100644 tools/testing/selftests/mm/prctl_thp_disable.c diff --git a/tools/testing/selftests/mm/.gitignore b/tools/testing/selftest= s/mm/.gitignore index e7b23a8a05fe2..eb023ea857b31 100644 --- a/tools/testing/selftests/mm/.gitignore +++ b/tools/testing/selftests/mm/.gitignore @@ -58,3 +58,4 @@ pkey_sighandler_tests_32 pkey_sighandler_tests_64 guard-regions merge +prctl_thp_disable diff --git a/tools/testing/selftests/mm/Makefile b/tools/testing/selftests/= mm/Makefile index d75f1effcb791..bd5d17beafa64 100644 --- a/tools/testing/selftests/mm/Makefile +++ b/tools/testing/selftests/mm/Makefile @@ -87,6 +87,7 @@ TEST_GEN_FILES +=3D on-fault-limit TEST_GEN_FILES +=3D pagemap_ioctl TEST_GEN_FILES +=3D pfnmap TEST_GEN_FILES +=3D process_madv +TEST_GEN_FILES +=3D prctl_thp_disable TEST_GEN_FILES +=3D thuge-gen TEST_GEN_FILES +=3D transhuge-stress TEST_GEN_FILES +=3D uffd-stress diff --git a/tools/testing/selftests/mm/prctl_thp_disable.c b/tools/testing= /selftests/mm/prctl_thp_disable.c new file mode 100644 index 0000000000000..e9e519c85224c --- /dev/null +++ b/tools/testing/selftests/mm/prctl_thp_disable.c @@ -0,0 +1,175 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Basic tests for PR_GET/SET_THP_DISABLE prctl calls + * + * Author(s): Usama Arif + */ +#include +#include +#include +#include +#include +#include +#include + +#include "../kselftest_harness.h" +#include "thp_settings.h" +#include "vm_util.h" + +enum thp_collapse_type { + THP_COLLAPSE_NONE, + THP_COLLAPSE_MADV_NOHUGEPAGE, + THP_COLLAPSE_MADV_HUGEPAGE, /* MADV_HUGEPAGE before access */ + THP_COLLAPSE_MADV_COLLAPSE, /* MADV_COLLAPSE after access */ +}; + +/* + * Function to mmap a buffer, fault it in, madvise it appropriately (before + * page fault for MADV_HUGE, and after for MADV_COLLAPSE), and check if the + * mmap region is huge. + * Returns: + * 0 if test doesn't give hugepage + * 1 if test gives a hugepage + * -errno if mmap fails + */ +static int test_mmap_thp(enum thp_collapse_type madvise_buf, size_t pmdsiz= e) +{ + char *mem, *mmap_mem; + size_t mmap_size; + int ret; + + /* For alignment purposes, we need twice the THP size. */ + mmap_size =3D 2 * pmdsize; + mmap_mem =3D (char *)mmap(NULL, mmap_size, PROT_READ | PROT_WRITE, + MAP_PRIVATE | MAP_ANONYMOUS, -1, 0); + if (mmap_mem =3D=3D MAP_FAILED) + return -errno; + + /* We need a THP-aligned memory area. */ + mem =3D (char *)(((uintptr_t)mmap_mem + pmdsize) & ~(pmdsize - 1)); + + if (madvise_buf =3D=3D THP_COLLAPSE_MADV_HUGEPAGE) + madvise(mem, pmdsize, MADV_HUGEPAGE); + else if (madvise_buf =3D=3D THP_COLLAPSE_MADV_NOHUGEPAGE) + madvise(mem, pmdsize, MADV_NOHUGEPAGE); + + /* Ensure memory is allocated */ + memset(mem, 1, pmdsize); + + if (madvise_buf =3D=3D THP_COLLAPSE_MADV_COLLAPSE) + madvise(mem, pmdsize, MADV_COLLAPSE); + + /* HACK: make sure we have a separate VMA that we can check reliably. */ + mprotect(mem, pmdsize, PROT_READ); + + ret =3D check_huge_anon(mem, 1, pmdsize); + munmap(mmap_mem, mmap_size); + return ret; +} + +static void prctl_thp_disable_completely_test(struct __test_metadata *cons= t _metadata, + size_t pmdsize, + enum thp_enabled thp_policy) +{ + ASSERT_EQ(prctl(PR_GET_THP_DISABLE, NULL, NULL, NULL, NULL), 1); + + /* tests after prctl overrides global policy */ + ASSERT_EQ(test_mmap_thp(THP_COLLAPSE_NONE, pmdsize), 0); + + ASSERT_EQ(test_mmap_thp(THP_COLLAPSE_MADV_NOHUGEPAGE, pmdsize), 0); + + ASSERT_EQ(test_mmap_thp(THP_COLLAPSE_MADV_HUGEPAGE, pmdsize), 0); + + ASSERT_EQ(test_mmap_thp(THP_COLLAPSE_MADV_COLLAPSE, pmdsize), 0); + + /* Reset to global policy */ + ASSERT_EQ(prctl(PR_SET_THP_DISABLE, 0, NULL, NULL, NULL), 0); + + /* tests after prctl is cleared, and only global policy is effective */ + ASSERT_EQ(test_mmap_thp(THP_COLLAPSE_NONE, pmdsize), + thp_policy =3D=3D THP_ALWAYS ? 1 : 0); + + ASSERT_EQ(test_mmap_thp(THP_COLLAPSE_MADV_NOHUGEPAGE, pmdsize), 0); + + ASSERT_EQ(test_mmap_thp(THP_COLLAPSE_MADV_HUGEPAGE, pmdsize), + thp_policy =3D=3D THP_NEVER ? 0 : 1); + + ASSERT_EQ(test_mmap_thp(THP_COLLAPSE_MADV_COLLAPSE, pmdsize), 1); +} + +FIXTURE(prctl_thp_disable_completely) +{ + struct thp_settings settings; + size_t pmdsize; +}; + +FIXTURE_VARIANT(prctl_thp_disable_completely) +{ + enum thp_enabled thp_policy; +}; + +FIXTURE_VARIANT_ADD(prctl_thp_disable_completely, never) +{ + .thp_policy =3D THP_NEVER, +}; + +FIXTURE_VARIANT_ADD(prctl_thp_disable_completely, madvise) +{ + .thp_policy =3D THP_MADVISE, +}; + +FIXTURE_VARIANT_ADD(prctl_thp_disable_completely, always) +{ + .thp_policy =3D THP_ALWAYS, +}; + +FIXTURE_SETUP(prctl_thp_disable_completely) +{ + if (!thp_available()) + SKIP(return, "Transparent Hugepages not available\n"); + + self->pmdsize =3D read_pmd_pagesize(); + if (!self->pmdsize) + SKIP(return, "Unable to read PMD size\n"); + + if (prctl(PR_SET_THP_DISABLE, 1, NULL, NULL, NULL)) + SKIP(return, "Unable to disable THPs completely for the process\n"); + + thp_save_settings(); + thp_read_settings(&self->settings); + self->settings.thp_enabled =3D variant->thp_policy; + self->settings.hugepages[sz2ord(self->pmdsize, getpagesize())].enabled = =3D THP_INHERIT; + thp_write_settings(&self->settings); +} + +FIXTURE_TEARDOWN(prctl_thp_disable_completely) +{ + thp_restore_settings(); +} + +TEST_F(prctl_thp_disable_completely, nofork) +{ + prctl_thp_disable_completely_test(_metadata, self->pmdsize, variant->thp_= policy); +} + +TEST_F(prctl_thp_disable_completely, fork) +{ + int ret =3D 0; + pid_t pid; + + /* Make sure prctl changes are carried across fork */ + pid =3D fork(); + ASSERT_GE(pid, 0); + + if (!pid) + prctl_thp_disable_completely_test(_metadata, self->pmdsize, variant->thp= _policy); + + wait(&ret); + if (WIFEXITED(ret)) + ret =3D WEXITSTATUS(ret); + else + ret =3D -EINVAL; + ASSERT_EQ(ret, 0); +} + +TEST_HARNESS_MAIN diff --git a/tools/testing/selftests/mm/thp_settings.c b/tools/testing/self= tests/mm/thp_settings.c index bad60ac52874a..574bd0f8ae480 100644 --- a/tools/testing/selftests/mm/thp_settings.c +++ b/tools/testing/selftests/mm/thp_settings.c @@ -382,10 +382,17 @@ unsigned long thp_shmem_supported_orders(void) return __thp_supported_orders(true); } =20 -bool thp_is_enabled(void) +bool thp_available(void) { if (access(THP_SYSFS, F_OK) !=3D 0) return false; + return true; +} + +bool thp_is_enabled(void) +{ + if (!thp_available()) + return false; =20 int mode =3D thp_read_string("enabled", thp_enabled_strings); =20 diff --git a/tools/testing/selftests/mm/thp_settings.h b/tools/testing/self= tests/mm/thp_settings.h index 6c07f70beee97..76eeb712e5f10 100644 --- a/tools/testing/selftests/mm/thp_settings.h +++ b/tools/testing/selftests/mm/thp_settings.h @@ -84,6 +84,7 @@ void thp_set_read_ahead_path(char *path); unsigned long thp_supported_orders(void); unsigned long thp_shmem_supported_orders(void); =20 +bool thp_available(void); bool thp_is_enabled(void); =20 #endif /* __THP_SETTINGS_H__ */ --=20 2.47.3