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 picard.linux.it (picard.linux.it [213.254.12.146]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id EF886C624DB for ; Sat, 5 Sep 2026 12:45:52 +0000 (UTC) Received: from picard.linux.it (localhost [IPv6:::1]) by picard.linux.it (Postfix) with ESMTP id 5D7D63E2013 for ; Sat, 5 Sep 2026 14:45:51 +0200 (CEST) Received: from in-6.smtp.seeweb.it (in-6.smtp.seeweb.it [IPv6:2001:4b78:1:20::6]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (secp384r1)) (No client certificate requested) by picard.linux.it (Postfix) with ESMTPS id 3AAF13E1EB5 for ; Sat, 5 Sep 2026 14:45:33 +0200 (CEST) Received: from mx0b-001b2d01.pphosted.com (mx0b-001b2d01.pphosted.com [148.163.158.5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by in-6.smtp.seeweb.it (Postfix) with ESMTPS id 1ABF71400108 for ; Sat, 5 Sep 2026 14:45:32 +0200 (CEST) Received: from pps.filterd (m0356516.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 685BWdA6159622 for ; Sat, 5 Sep 2026 12:45:30 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ibm.com; h=cc :content-transfer-encoding:date:from:message-id:mime-version :subject:to; s=pp1; bh=I6IuCHv/SYv3GbfNUB1pbIAUDE8tV9LvJX2ziF22i Uo=; b=ix/1+M7GmmwDlkfGMD35g+RJ6VsXIsWSDS6+QJoPVNjaZbFdJclhdSDYl pj761z0q5YKoIn9VWNxCbiNiLpGjhanD2/3h1GK9xuGr4X14sJOzqDKkivNzC9Xq x/KfUVjUCPp0qd/ow3jBV96UJE/XYmRuJU+GGOpdvT+49qeX8HdReWGGV4Blfmqs ZHOCYtbALmtSohC8sGKiNzYrluJ167cURTYgTrf3OL17K/68OF4UnXcQudW2yt3i LUvqp7d6zB0XhT6ZRgAZcPgOvStgTTcytngR5c/ntofcgJIeRQONfqiT0M+b74MM 2M+asOWAM7awXYPDxuIVNPjy6BxqA== Received: from ppma11.dal12v.mail.ibm.com (db.9e.1632.ip4.static.sl-reverse.com [50.22.158.219]) by mx0a-001b2d01.pphosted.com (PPS) with ESMTPS id 4ggbj7s3x0-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT) for ; Sat, 05 Sep 2026 12:45:30 +0000 (GMT) Received: from pps.filterd (ppma11.dal12v.mail.ibm.com [127.0.0.1]) by ppma11.dal12v.mail.ibm.com (8.18.1.7/8.18.1.7) with ESMTP id 685CfKRw003168 for ; Sat, 5 Sep 2026 12:45:29 GMT Received: from smtprelay05.fra02v.mail.ibm.com ([9.218.2.225]) by ppma11.dal12v.mail.ibm.com (PPS) with ESMTPS id 4ggbjts419-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT) for ; Sat, 05 Sep 2026 12:45:29 +0000 (GMT) Received: from smtpav03.fra02v.mail.ibm.com (smtpav03.fra02v.mail.ibm.com [10.20.54.102]) by smtprelay05.fra02v.mail.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 685CjPQe50725170 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Sat, 5 Sep 2026 12:45:25 GMT Received: from smtpav03.fra02v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id B098420043; Sat, 5 Sep 2026 12:45:25 +0000 (GMT) Received: from smtpav03.fra02v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 01D7C20040; Sat, 5 Sep 2026 12:45:25 +0000 (GMT) Received: from li-49acbccc-2994-11b2-a85c-d51b89e344f1.ibm.com.com (unknown [9.43.69.4]) by smtpav03.fra02v.mail.ibm.com (Postfix) with ESMTP; Sat, 5 Sep 2026 12:45:24 +0000 (GMT) From: Pavithra To: ltp@lists.linux.it Date: Sat, 5 Sep 2026 18:15:22 +0530 Message-ID: <20260905124522.961300-1-pavrampu@linux.ibm.com> X-Mailer: git-send-email 2.55.0 MIME-Version: 1.0 X-TM-AS-GCONF: 00 X-Proofpoint-GUID: lZx1_JnQnyRLT-xwP0hwx-vMRIBXvJ6U X-Proofpoint-ORIG-GUID: lZx1_JnQnyRLT-xwP0hwx-vMRIBXvJ6U X-Authority-Analysis: v=2.4 cv=RNCD2Yi+ c=1 sm=1 tr=0 ts=6a9c0eea cx=c_pps a=aDMHemPKRhS1OARIsFnwRA==:117 a=aDMHemPKRhS1OARIsFnwRA==:17 a=VdqzKS8jKosA:10 a=VkNPw1HP01LnGYTKEx00:22 a=RnoormkPH1_aCDwRdu11:22 a=Y2IxJ9c9Rs8Kov3niI8_:22 a=VwQbUJbxAAAA:8 a=VnNF1IyMAAAA:8 a=FPk0Lnvqx-u2F3UbQ_8A:9 X-Proofpoint-Spam-Info: AW1haW4tMjYwOTA1MDEzNyBTYWx0ZWRfX8hH29vMgXyv6 GtHpihjVdmhD0hoCrmEn5/TRVEEIImV4S7e8F45lZ4c5O1+yEoO10s0H/cTpvG9HPhiR5JUjK84 KqpeGP9hMIGvZEYID+P99BpsWvxQYzQ= X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwOTA1MDEzNyBTYWx0ZWRfXzasuJLJPrFCr 1Iv8bhyGaA4d/iazA5RcQcq45619RQpLQM0LvwHHklksTunrJiJec+t2DpGRrVmjh0BXQt0Kt46 xQqF5RFqiL1y8JdU03I3ZA+Y/EFlCjR+MbExptbbQUwRzoaHCOo0n+mQ/PDi11r06vJEIzB1YJP NrkROOKLK+HsCb5Myi3OFD1X5kIojEuDNkniYlAlu0KbOn3CyOSC/jCuX+PgfHj23QaG3XlA8D2 E/BetnbNKGje8Tryx8YpCI8qCAlCd0vf3wJwxhOh/ETuNFyMVXO9mzis1ProA8wuhC7H4xWKTGY GCyygVRpjwKxIzXwUgZAxhAYkMrCDw+F+BmEUH/aJeSx/eYyfHOHDlIw1RdZu2sZ73l/dHgqH+6 5Pz277Czuha4/dJr6mvwdiBngUFq1CEfne7PFaxHXkcuF2azIsqCj9S0Xu0O3q/V6cL2fosLPUu OBOgNr9fk8RKPuxc02A== X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1176,Hydra:6.1.134,FMLib:17.12.100.49 definitions=2026-09-05_04,2026-09-03_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501 lowpriorityscore=0 bulkscore=0 clxscore=1015 spamscore=0 impostorscore=0 adultscore=0 phishscore=0 suspectscore=0 malwarescore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2606150000 definitions=main-2609050137 X-Virus-Scanned: clamav-milter 1.0.9 at in-6.smtp.seeweb.it X-Virus-Status: Clean Subject: [LTP] [PATCH v7] hugetlb/hugemmap: add hugemmap33 to test hugetlbfs quota accounting X-BeenThere: ltp@lists.linux.it X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux Test Project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: pavrampu@linux.ibm.com Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ltp-bounces+ltp=archiver.kernel.org@lists.linux.it Sender: "ltp" Test hugetlbfs quota accounting with filesystem size limits to check for regressions in quota handling for MAP_PRIVATE and MAP_SHARED pages. Signed-off-by: Pavithra --- v6 -> v7: - Name function pointer parameters 'size' and 'mmap_flags' to fix make check warnings. Link to v6: https://lore.kernel.org/ltp/20260902075156.240295-1-pavrampu@linux.ibm.com/ --- runtest/hugetlb | 1 + testcases/kernel/mem/.gitignore | 1 + .../kernel/mem/hugetlb/hugemmap/hugemmap33.c | 260 ++++++++++++++++++ 3 files changed, 262 insertions(+) create mode 100644 testcases/kernel/mem/hugetlb/hugemmap/hugemmap33.c diff --git a/runtest/hugetlb b/runtest/hugetlb index 621c9718a..e3ee42c2f 100644 --- a/runtest/hugetlb +++ b/runtest/hugetlb @@ -35,6 +35,7 @@ hugemmap29 hugemmap29 hugemmap30 hugemmap30 hugemmap31 hugemmap31 hugemmap32 hugemmap32 +hugemmap33 hugemmap33 hugemmap34 hugemmap34 hugemmap35 hugemmap35 hugemmap36 hugemmap36 diff --git a/testcases/kernel/mem/.gitignore b/testcases/kernel/mem/.gitignore index 9e706e1c8..763f134dd 100644 --- a/testcases/kernel/mem/.gitignore +++ b/testcases/kernel/mem/.gitignore @@ -35,6 +35,7 @@ /hugetlb/hugemmap/hugemmap30 /hugetlb/hugemmap/hugemmap31 /hugetlb/hugemmap/hugemmap32 +/hugetlb/hugemmap/hugemmap33 /hugetlb/hugemmap/hugemmap34 /hugetlb/hugemmap/hugemmap35 /hugetlb/hugemmap/hugemmap36 diff --git a/testcases/kernel/mem/hugetlb/hugemmap/hugemmap33.c b/testcases/kernel/mem/hugetlb/hugemmap/hugemmap33.c new file mode 100644 index 000000000..910b699f2 --- /dev/null +++ b/testcases/kernel/mem/hugetlb/hugemmap/hugemmap33.c @@ -0,0 +1,260 @@ +// SPDX-License-Identifier: LGPL-2.1-or-later +/* + * Copyright (C) 2005-2007 David Gibson & Adam Litke, IBM Corporation. + * Copyright (c) Linux Test Project, 2024 + * Copyright (C) 2025-2026 Naveed & Pavithra, IBM Corporation. + * Assisted with AI tools + */ + +/*\ + * Test hugetlbfs quota accounting with filesystem size limits. + * + * The number of global huge pages available to a mounted hugetlbfs filesystem + * can be limited using a quota mechanism by setting the size attribute at + * mount time. Older kernels did not properly handle quota accounting for + * MAP_PRIVATE pages and MAP_SHARED reservations. + * + * Commit a1e78772d72b introduced MAP_PRIVATE reservations at mmap() time, + * so quota is checked upfront and mmap() fails with ENOMEM when over quota. + * Before this, quota was only checked at fault time resulting in SIGBUS. + * + * Root is required to mount the quota-limited hugetlbfs instance. + */ + +#define _GNU_SOURCE +#include +#include +#include +#include +#include "hugetlb.h" + +#define MNTPOINT "hugetlbfs/" + +static long hpage_size; +static int quota_mounted; + +static void verify_quota_stat(long tot, long free, long avail) +{ + struct statfs s; + + SAFE_STATFS(MNTPOINT, &s); + + if ((long)s.f_blocks != tot || (long)s.f_bfree != free || (long)s.f_bavail != avail) { + tst_res(TFAIL, "Bad quota counters: total=%li free=%li avail=%li, expected %li %li %li", + (long)s.f_blocks, (long)s.f_bfree, (long)s.f_bavail, + tot, free, avail); + return; + } + tst_res(TPASS, "Quota counters are correct: total=%li free=%li avail=%li", + tot, free, avail); +} + +static void verify_quota_map_unmap(unsigned long size, int mmap_flags) +{ + int fd; + char path[PATH_MAX]; + + snprintf(path, sizeof(path), "%s/test_file_%d", MNTPOINT, getpid()); + fd = SAFE_OPEN(path, O_CREAT | O_RDWR, 0600); + SAFE_UNLINK(path); + + TESTPTR(mmap(NULL, size, PROT_READ | PROT_WRITE, mmap_flags, fd, 0)); + if (TST_RET_PTR == MAP_FAILED) { + tst_res(TFAIL | TERRNO, "mmap failed unexpectedly"); + SAFE_CLOSE(fd); + return; + } + + tst_res(TPASS, "Untouched mapping quota test passed as expected"); + SAFE_CLOSE(fd); +} + +static void verify_quota_ok(unsigned long size, int mmap_flags) +{ + int fd; + char *a, *b; + char path[PATH_MAX]; + + snprintf(path, sizeof(path), "%s/test_file_%d", MNTPOINT, getpid()); + fd = SAFE_OPEN(path, O_CREAT | O_RDWR, 0600); + SAFE_UNLINK(path); + + TESTPTR(mmap(NULL, size, PROT_READ | PROT_WRITE, mmap_flags, fd, 0)); + a = TST_RET_PTR; + if (a == MAP_FAILED) { + tst_res(TFAIL | TERRNO, "mmap failed unexpectedly"); + SAFE_CLOSE(fd); + return; + } + + for (b = a; b < a + size; b += hpage_size) + *b = 1; + + tst_res(TPASS, "Quota test passed as expected"); + SAFE_MUNMAP(a, size); + SAFE_CLOSE(fd); +} + +static void verify_quota_fail(unsigned long size, int mmap_flags) +{ + int fd; + char *a; + char path[PATH_MAX]; + + snprintf(path, sizeof(path), "%s/test_file_%d", MNTPOINT, getpid()); + fd = SAFE_OPEN(path, O_CREAT | O_RDWR, 0600); + SAFE_UNLINK(path); + + TESTPTR(mmap(NULL, size, PROT_READ | PROT_WRITE, mmap_flags, fd, 0)); + a = TST_RET_PTR; + if (a == MAP_FAILED) { + if (TST_ERR == ENOMEM) { + tst_res(TPASS | TERRNO, "mmap failed as expected due to quota"); + SAFE_CLOSE(fd); + return; + } + tst_res(TFAIL | TERRNO, "mmap failed unexpectedly"); + SAFE_CLOSE(fd); + return; + } + + tst_res(TFAIL, "mmap succeeded but quota exhaustion was expected"); + SAFE_MUNMAP(a, size); + SAFE_CLOSE(fd); +} + +static void verify_quota_private_resv_fail(unsigned long size, int mmap_flags) +{ + int fd; + char *a, *b, *c; + char path[PATH_MAX]; + + snprintf(path, sizeof(path), "%s/test_file_%d", MNTPOINT, getpid()); + fd = SAFE_OPEN(path, O_CREAT | O_RDWR, 0600); + SAFE_UNLINK(path); + + TESTPTR(mmap(NULL, size, PROT_READ | PROT_WRITE, mmap_flags, fd, 0)); + a = TST_RET_PTR; + if (a == MAP_FAILED) { + tst_res(TFAIL | TERRNO, "mmap failed unexpectedly"); + SAFE_CLOSE(fd); + return; + } + + for (b = a; b < a + size; b += hpage_size) + *b = 1; + + TESTPTR(mmap(NULL, size, PROT_READ | PROT_WRITE, MAP_PRIVATE, fd, 0)); + c = TST_RET_PTR; + if (c == MAP_FAILED) { + if (TST_ERR == ENOMEM) { + tst_res(TPASS | TERRNO, + "MAP_PRIVATE mmap failed as expected due to quota"); + SAFE_MUNMAP(a, size); + SAFE_CLOSE(fd); + return; + } + tst_res(TFAIL | TERRNO, "MAP_PRIVATE mmap failed unexpectedly"); + SAFE_MUNMAP(a, size); + SAFE_CLOSE(fd); + return; + } + + tst_res(TFAIL, "MAP_PRIVATE mmap succeeded but quota exhaustion was expected"); + SAFE_MUNMAP(c, size); + SAFE_MUNMAP(a, size); + SAFE_CLOSE(fd); +} + +static struct tcase { + void (*fn)(unsigned long size, int mmap_flags); + unsigned long size_mult; /* multiplied by hpage_size at runtime */ + int mmap_flags; + const char *desc; + int check_stat; /* call verify_quota_stat(1,1,1) after */ +} tcases[] = { + { verify_quota_map_unmap, 1, MAP_PRIVATE, + "Unused quota cleanup for untouched MAP_PRIVATE mapping", 1 }, + { verify_quota_map_unmap, 1, MAP_SHARED, + "Unused quota cleanup for untouched MAP_SHARED mapping", 1 }, + { verify_quota_ok, 1, MAP_PRIVATE, + "Page instantiation within quota (MAP_PRIVATE)", 0 }, + { verify_quota_ok, 1, MAP_SHARED, + "Page instantiation within quota (MAP_SHARED)", 0 }, + { verify_quota_fail, 2, MAP_SHARED, + "Page instantiation over quota (MAP_SHARED)", 0 }, + { verify_quota_fail, 2, MAP_PRIVATE, + "Private mapping quota check (MAP_PRIVATE)", 0 }, + { verify_quota_private_resv_fail, 1, MAP_SHARED, + "Second MAP_PRIVATE reservation over quota (MAP_SHARED base)", 0 }, + { verify_quota_private_resv_fail, 1, MAP_PRIVATE, + "Second MAP_PRIVATE reservation over quota (MAP_PRIVATE base)", 0 }, + { verify_quota_ok, 1, MAP_SHARED, + "Operation within quota after failures (MAP_SHARED)", 0 }, + { verify_quota_ok, 1, MAP_PRIVATE, + "Operation within quota after failures (MAP_PRIVATE)", 0 }, +}; + +static void run_test(unsigned int i) +{ + const struct tcase *tc = &tcases[i]; + pid_t pid; + int status; + + tst_res(TINFO, "%s", tc->desc); + + pid = SAFE_FORK(); + if (pid == 0) { + tc->fn(tc->size_mult * hpage_size, tc->mmap_flags); + exit(0); + } + + SAFE_WAITPID(pid, &status, 0); + + if (WIFSIGNALED(status)) + tst_res(TFAIL, "Child killed by signal %d", WTERMSIG(status)); + + if (tc->check_stat) + verify_quota_stat(1, 1, 1); +} + +static void setup(void) +{ + char mount_opts[BUFSIZ]; + + hpage_size = tst_get_hugepage_size(); + + snprintf(mount_opts, sizeof(mount_opts), "size=%luK", + hpage_size / 1024); + + if (mount("none", MNTPOINT, "hugetlbfs", 0, mount_opts) == -1) { + if (errno == ENODEV) + tst_brk(TCONF, "hugetlbfs not supported"); + tst_brk(TBROK | TERRNO, "mount() failed"); + } + quota_mounted = 1; + + tst_res(TINFO, "Mounted hugetlbfs with quota at %s (size=%luK)", + MNTPOINT, hpage_size / 1024); +} + +static void cleanup(void) +{ + if (quota_mounted) + SAFE_UMOUNT(MNTPOINT); +} + +static struct tst_test test = { + .tags = (struct tst_tag[]) { + {"linux-git", "a1e78772d72b"}, + {} + }, + .needs_root = 1, + .mntpoint = MNTPOINT, + .forks_child = 1, + .setup = setup, + .cleanup = cleanup, + .test = run_test, + .tcnt = ARRAY_SIZE(tcases), + .hugepages = {2, TST_NEEDS}, +}; -- 2.55.0 -- Mailing list info: https://lists.linux.it/listinfo/ltp