From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 435DF31F985; Sat, 26 Sep 2026 00:50:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790383852; cv=none; b=gNiYLyY1yvwsK1XLgRoJ/e//dqms3FhtCfpKvh9q5mIZbxqhKxgsYxmdNq0cZjztTu9vhjN41GGVlFI1X6tCSlyk6rvLef9jGc6Li3RDohbJ/NbRUdYPex9FU6EYwpRXBX9wbyg8RwaoVHT5rPASRGy/52Cm9YNexKKnj8MGxEI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790383852; c=relaxed/simple; bh=vDEkazdRLF4novShMyP5tuhmgMvX29p9/UO9KqWcgPw=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=JO3F7qH6+U2ht999eh6+Y3JcMB28tJSLJAjc+Ue6aWvvHtarwppYDYaQY563C0hseQ8lPfb/+Y1JhOGwSeKP+IcGlLfw7uS0DQl2nlTybzGPQJ+G7UqiVSJ/eiTvBi7BU+goNzE1szhcOkXR+xWnOC2c0hNlg/l9ShrxI9pnPgQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=oLVH+G3k; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="oLVH+G3k" Received: by smtp.kernel.org (Postfix) with ESMTPS id 24009C2BD05; Sat, 26 Sep 2026 00:50:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1790383852; bh=vDEkazdRLF4novShMyP5tuhmgMvX29p9/UO9KqWcgPw=; h=From:Date:Subject:References:In-Reply-To:To:Cc:Reply-To:From; b=oLVH+G3kOdTztlkvAcQ11NjWqYDFfKVrZ1QupyfZfNo3deil4Ey0uOqy9cpb2y/vF Xv4R1Ms+zaRGy/VYvAzoD3ySTt7RGa6X8ZGdwUdhUsYiRYjziZcOXR14t40sCLkmbJ WJJPYRuwyc0fkpAQpKPJs34awU0X19xg0aI8Liu6+zuEILNHnGtQq42Zw5OTCIJAaL UFeLMAooIa8BuYYGK/EYPkD31rV5TfKI+Ia5JrrKYI643Rz7D7wgPXAqRmQpY3tMRX SIR3x7hRvKtGHCzT288kI7gDeRVW9uWG0dPUpB3PM6+ZpMD542Q/UEHiZOiwQVD5Dp YhQmvVytPl7cQ== Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0C515C9832A; Sat, 26 Sep 2026 00:50:52 +0000 (UTC) From: Ackerley Tng via B4 Relay Date: Fri, 25 Sep 2026 17:50:59 -0700 Subject: [PATCH RFC 12/17] KVM: selftests: Test guest_memfd with mounted tmpfs root directory 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: 7bit Message-Id: <20260925-gmem-tmpfs-backend-v1-12-d36159822d18@google.com> References: <20260925-gmem-tmpfs-backend-v1-0-d36159822d18@google.com> In-Reply-To: <20260925-gmem-tmpfs-backend-v1-0-d36159822d18@google.com> To: Hugh Dickins , Baolin Wang , Andrew Morton , Sean Christopherson , Paolo Bonzini , David Hildenbrand , Jonathan Corbet , Shuah Khan , Randy Dunlap , Shuah Khan , vannapurve@google.com, erdemaktas@google.com, jxgao@google.com, rientjes@google.com, fvdl@google.com, jthoughton@google.com, tarunsahu@google.com, pratyush@kernel.org, fuad.tabba@linux.dev, Gregory Price , David Woodhouse , yan.y.zhao@intel.com, michael.roth@amd.com, suzuki.poulose@arm.com, Christian Brauner , Jason Gunthorpe , Nicolin Chen , Xu Yilun , aik@amd.com, aneesh.kumar@kernel.org, Vlastimil Babka Cc: kernel-team@android.com, kernel-team@meta.com, linux-kernel@vger.kernel.org, linux-mm@kvack.org, kvm@vger.kernel.org, linux-doc@vger.kernel.org, linux-kselftest@vger.kernel.org, Ackerley Tng X-Mailer: b4 0.17-dev X-Developer-Signature: v=1; a=ed25519-sha256; t=1790383850; l=7758; i=ackerleytng@google.com; s=20260225; h=from:subject:message-id; bh=I2qZYT+hdR9asssZySkypzr+BY173budj+AtlVNc+WY=; b=4/VFnb+9s+yMlUcYP3RlcOKI9CoI44IGLvMZ9TOrzcZL5l3AvKLpTfTxSoQJqrZvDZH53/7xc 1NdnFokQecUCT4JzQmqjmtlO9cMXSiBhIGbWZLyXfeo1J5GdVqZAuGI X-Developer-Key: i=ackerleytng@google.com; a=ed25519; pk=sAZDYXdm6Iz8FHitpHeFlCMXwabodTm7p8/3/8xUxuU= X-Endpoint-Received: by B4 Relay for ackerleytng@google.com/20260225 with auth_id=649 X-Original-From: Ackerley Tng Reply-To: ackerleytng@google.com From: Ackerley Tng Add test coverage verifying that opening the root directory of an existing mounted tmpfs filesystem and passing that file descriptor as the resource pool succeeds across all guest_memfd subtests and flag combinations. Signed-off-by: Ackerley Tng --- tools/testing/selftests/kvm/guest_memfd_test.c | 137 +++++++++++++++++++------ 1 file changed, 103 insertions(+), 34 deletions(-) diff --git a/tools/testing/selftests/kvm/guest_memfd_test.c b/tools/testing/selftests/kvm/guest_memfd_test.c index 73212d5307049..94ef64474f2a0 100644 --- a/tools/testing/selftests/kvm/guest_memfd_test.c +++ b/tools/testing/selftests/kvm/guest_memfd_test.c @@ -10,11 +10,13 @@ #include #include #include +#include #include #include #include #include +#include #include #include @@ -541,66 +543,122 @@ static void test_resource_fd_tmpfs_huge(struct kvm_vm *vm) close(pool_fd); } -#define ____gmem_test(__test, __vm, __flags, __gmem_size, args...) \ +enum gmem_pool_type { + GMEM_POOL_NONE, + GMEM_POOL_FSMOUNT, + GMEM_POOL_MOUNTED_DIR, +}; + +struct gmem_pool { + int fd; + char path[PATH_MAX]; + bool is_mounted; +}; + +static struct gmem_pool create_gmem_pool(size_t size, enum gmem_pool_type type) +{ + struct gmem_pool pool = { .fd = -1, .is_mounted = false }; + int mnt_fd; + + if (type == GMEM_POOL_NONE) + return pool; + + mnt_fd = create_tmpfs_pool_fd("never", true, size); + TEST_REQUIRE(mnt_fd >= 0); + + if (type == GMEM_POOL_FSMOUNT) { + pool.fd = mnt_fd; + return pool; + } + + strcpy(pool.path, "/tmp/gmem_test_dir_XXXXXX"); + TEST_ASSERT(mkdtemp(pool.path), "mkdtemp failed"); + + if (syscall(__NR_move_mount, mnt_fd, "", AT_FDCWD, pool.path, + MOVE_MOUNT_F_EMPTY_PATH)) { + close(mnt_fd); + rmdir(pool.path); + TEST_REQUIRE(false); + } + close(mnt_fd); + + pool.fd = open(pool.path, O_RDONLY | O_DIRECTORY); + TEST_ASSERT(pool.fd >= 0, "open mounted tmpfs root failed"); + pool.is_mounted = true; + + return pool; +} + +static void destroy_gmem_pool(struct gmem_pool *pool) +{ + if (pool->fd >= 0) + close(pool->fd); + if (pool->is_mounted) { + TEST_ASSERT(!umount(pool->path), "umount failed"); + TEST_ASSERT(!rmdir(pool->path), "rmdir failed"); + } +} + +#define ____gmem_test(__test, __vm, __flags, __pool_type, __gmem_size, args...) \ do { \ - int pool_fd = -1; \ + struct gmem_pool pool = { .fd = -1 }; \ int fd; \ \ if ((__flags) & GUEST_MEMFD_FLAG_USE_RESOURCE) { \ - pool_fd = create_tmpfs_pool_fd("never", true, __gmem_size); \ + pool = create_gmem_pool(__gmem_size, __pool_type); \ fd = vm_create_guest_memfd_resource(__vm, __gmem_size, \ - __flags, pool_fd); \ + __flags, pool.fd); \ } else { \ fd = vm_create_guest_memfd(__vm, __gmem_size, __flags); \ } \ \ test_##__test(args); \ close(fd); \ - if (pool_fd >= 0) \ - close(pool_fd); \ + destroy_gmem_pool(&pool); \ } while (0) -#define __gmem_test(__test, __vm, __flags, __gmem_size) \ - ____gmem_test(__test, __vm, __flags, __gmem_size, fd, __gmem_size) +#define __gmem_test(__test, __vm, __flags, __pool_type, __gmem_size) \ + ____gmem_test(__test, __vm, __flags, __pool_type, __gmem_size, fd, __gmem_size) -#define gmem_test(__test, __vm, __flags) \ - __gmem_test(__test, __vm, __flags, page_size * 4) +#define gmem_test(__test, __vm, __flags, __pool_type) \ + __gmem_test(__test, __vm, __flags, __pool_type, page_size * 4) -#define __gmem_test_vm(__test, __vm, __flags, __gmem_size) \ - ____gmem_test(__test, __vm, __flags, __gmem_size, __vm, fd, __gmem_size) +#define __gmem_test_vm(__test, __vm, __flags, __pool_type, __gmem_size) \ + ____gmem_test(__test, __vm, __flags, __pool_type, __gmem_size, __vm, fd, __gmem_size) -#define gmem_test_vm(__test, __vm, __flags) \ - __gmem_test_vm(__test, __vm, __flags, page_size * 4) +#define gmem_test_vm(__test, __vm, __flags, __pool_type) \ + __gmem_test_vm(__test, __vm, __flags, __pool_type, page_size * 4) -static void __test_guest_memfd(struct kvm_vm *vm, u64 flags) +static void __test_guest_memfd(struct kvm_vm *vm, u64 flags, + enum gmem_pool_type pool_type) { test_create_guest_memfd_multiple(vm); test_create_guest_memfd_invalid_sizes(vm, flags); - gmem_test(file_read_write, vm, flags); + gmem_test(file_read_write, vm, flags, pool_type); if (flags & GUEST_MEMFD_FLAG_MMAP) { if (flags & GUEST_MEMFD_FLAG_INIT_SHARED) { size_t pmd_size = get_trans_hugepagesz(); - gmem_test(mmap_supported, vm, flags); - gmem_test(fault_overflow, vm, flags); - gmem_test(numa_allocation, vm, flags); - __gmem_test(collapse, vm, flags, pmd_size); + gmem_test(mmap_supported, vm, flags, pool_type); + gmem_test(fault_overflow, vm, flags, pool_type); + gmem_test(numa_allocation, vm, flags, pool_type); + __gmem_test(collapse, vm, flags, pool_type, pmd_size); } else { - gmem_test(fault_private, vm, flags); + gmem_test(fault_private, vm, flags, pool_type); } - gmem_test(mmap_cow, vm, flags); - gmem_test(mbind, vm, flags); + gmem_test(mmap_cow, vm, flags, pool_type); + gmem_test(mbind, vm, flags, pool_type); } else { - gmem_test(mmap_not_supported, vm, flags); + gmem_test(mmap_not_supported, vm, flags, pool_type); } - gmem_test(file_size, vm, flags); - gmem_test(fallocate, vm, flags); - gmem_test(invalid_punch_hole, vm, flags); - gmem_test_vm(invalid_binding, vm, flags); + gmem_test(file_size, vm, flags, pool_type); + gmem_test(fallocate, vm, flags, pool_type); + gmem_test(invalid_punch_hole, vm, flags, pool_type); + gmem_test_vm(invalid_binding, vm, flags, pool_type); } static void test_guest_memfd(unsigned long vm_type) @@ -616,23 +674,34 @@ static void test_guest_memfd(unsigned long vm_type) test_resource_fd_tmpfs_swap(vm); test_resource_fd_tmpfs_huge(vm); - __test_guest_memfd(vm, 0); - __test_guest_memfd(vm, GUEST_MEMFD_FLAG_USE_RESOURCE); + __test_guest_memfd(vm, 0, GMEM_POOL_NONE); + __test_guest_memfd(vm, GUEST_MEMFD_FLAG_USE_RESOURCE, GMEM_POOL_FSMOUNT); + __test_guest_memfd(vm, GUEST_MEMFD_FLAG_USE_RESOURCE, GMEM_POOL_MOUNTED_DIR); flags = vm_check_cap(vm, KVM_CAP_GUEST_MEMFD_FLAGS); if (flags & GUEST_MEMFD_FLAG_MMAP) { - __test_guest_memfd(vm, GUEST_MEMFD_FLAG_MMAP); + __test_guest_memfd(vm, GUEST_MEMFD_FLAG_MMAP, GMEM_POOL_NONE); + __test_guest_memfd(vm, GUEST_MEMFD_FLAG_MMAP | + GUEST_MEMFD_FLAG_USE_RESOURCE, + GMEM_POOL_FSMOUNT); __test_guest_memfd(vm, GUEST_MEMFD_FLAG_MMAP | - GUEST_MEMFD_FLAG_USE_RESOURCE); + GUEST_MEMFD_FLAG_USE_RESOURCE, + GMEM_POOL_MOUNTED_DIR); } /* MMAP should always be supported if INIT_SHARED is supported. */ if (flags & GUEST_MEMFD_FLAG_INIT_SHARED) { __test_guest_memfd(vm, GUEST_MEMFD_FLAG_MMAP | - GUEST_MEMFD_FLAG_INIT_SHARED); + GUEST_MEMFD_FLAG_INIT_SHARED, + GMEM_POOL_NONE); + __test_guest_memfd(vm, GUEST_MEMFD_FLAG_MMAP | + GUEST_MEMFD_FLAG_INIT_SHARED | + GUEST_MEMFD_FLAG_USE_RESOURCE, + GMEM_POOL_FSMOUNT); __test_guest_memfd(vm, GUEST_MEMFD_FLAG_MMAP | GUEST_MEMFD_FLAG_INIT_SHARED | - GUEST_MEMFD_FLAG_USE_RESOURCE); + GUEST_MEMFD_FLAG_USE_RESOURCE, + GMEM_POOL_MOUNTED_DIR); } kvm_vm_free(vm); -- 2.56.0.rc1.315.gc6ed9934b7-goog