Received-SPF: pass (google.com: domain of linux-kernel+bounces-82572-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
From: Dongli Zhang <dongli.zhang@oracle.com>
To: kvm@vger.kernel.org, linux-kselftest@vger.kernel.org
Cc: linux-kernel@vger.kernel.org, pbonzini@redhat.com, shuah@kernel.org,
        seanjc@google.com
Subject: [PATCH 1/1] KVM: selftests: close the memfd in some gmem tests
Date: Mon, 26 Feb 2024 17:57:16 -0800
Message-Id: <20240227015716.27284-1-dongli.zhang@oracle.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: 
	=?us-ascii?Q?L0EJoQLhXoohwRRN4OQMdikC6x7ovYMqvO/r07ObHWrTmhGqVf4J3W8W2cqi?=
 =?us-ascii?Q?U73LahfeY6yaVCBoXEVnBiLzM0FXA7CcejlX7rYUyW2UETJ7bhkfeukFSR1/?=
 =?us-ascii?Q?Gkpwe8meL90NmH0xpv66pOyfkB5AqY4TlkA8dAcoCCPyh82aXsZ6II2ZaW1D?=
 =?us-ascii?Q?dISh570CxhqAraDUns7DVdbelDZRX1wVPQnUFZe4upoYGY/Zsz/qZ+qD4RCU?=
 =?us-ascii?Q?R4gJBcvQl4ZterT11F98kfSBeDm1Pxp0QzLja6L8igAvcTWEHegQOEyBjbhw?=
 =?us-ascii?Q?+81cLH3/YKCb3pm3X8nFOmvOGU9mNCMEpc2IpO0/KQpuhvBUBtux2zzVG8Nk?=
 =?us-ascii?Q?qe3xKqf8TnPgYpUbJ6/awRAgfYFjkCbY2Cia5GJKV5skK2gL5KVVOTjBplvz?=
 =?us-ascii?Q?YUWOdwM+XfwjHKWTfL5zUrzWf3CS743w3rRpzDNtTKQ8EDKFGwEEQwxYNYfu?=
 =?us-ascii?Q?8xTCEiDxE+EzJvMbYuJLaie5n0ixHKKEoasiXD7VUk6tijxYtVl3qLIt9mMH?=
 =?us-ascii?Q?rYTawjOUf+oGex8p2F408UUUHIMVtot4LVd/C5ZC3ozxVMiHNFGn9BjNBFc8?=
 =?us-ascii?Q?GNgyFQfLbOICGQOcsn1mMctmfFSPFVqTOwCrtnaDiqdSlrdksMJQLpj+/65T?=
 =?us-ascii?Q?z2/vPJjmB9NFnc28c0b38ivQKGljzQpmoccM2PF0+5tJSRXmDR/GfLO86spR?=
 =?us-ascii?Q?y2J3JfzHBoBtiWJAEGOvaP+ZSLfs65y66s8181zkapfxa2It2jsCZfIri6ao?=
 =?us-ascii?Q?zOaHbMR+pPDZ/SM4NOq7Uxx3lhKheHgpkEQHVrlsmpcjCKDTVgnBcstHo/6/?=
 =?us-ascii?Q?KPjvus96kNOrbGdKKsml8/0EiJQxQPKVDfGuV3AVvgvxMppN9t4Er1ApH1Cd?=
 =?us-ascii?Q?gh/W+dWk8v0TqAD0CawCbVX+zybbKspuu6VwbwOpvAdt2L9sYYRZxCF1sQ3v?=
 =?us-ascii?Q?dpbRoQ/8XQ1GB6dc/6sIitgFQCF1Wg6Hg7pGSueZ+LPVzXajpC2ZEthKyhnM?=
 =?us-ascii?Q?Dz5uJJAc3mqGyC8PLnum44z0hUrfR0kt/plkLmwwQOG9vx8ZxWPrQlYaifOL?=
 =?us-ascii?Q?Efv5/je1Oq1SKcHnuDosP8LvGpXHibp5E1LmVzAycKr90+eJrnNt9NHIXzQ2?=
 =?us-ascii?Q?8kPsmDuctuKHzMK1CHb3AurR6gA12Fi7YH6mdlD6MDEdezPIwYrFdKL2OgXB?=
 =?us-ascii?Q?d6G51bLo+s4251rjziDOYcVB0ZzC00jDYXxbmIG8sNE3Y7OXpFDL7HBv4RJp?=
 =?us-ascii?Q?Le1LV1ku8VrmaPxa023YEfsgrAWjgu8HRCpkHxe1n2rHp1yL0LJ7bYkY7kJ2?=
 =?us-ascii?Q?RvKVrRdBEeD4IyLjxKuM9wuBNL1zFGf9Z740z3huebLCsykH1CxWpF/kxmvd?=
 =?us-ascii?Q?j0fmGm0FMFhkDTW/r6I0QXQ5BVh038jdsG49sDIS5inR8rRPsvgvYzL2mChT?=
 =?us-ascii?Q?VIiL459EdNbyDoyIBNhiQA2X2UwBB82vx39+ZRsJeN09vAa2UEvmvZj5YBSI?=
 =?us-ascii?Q?ZslfkELZyQmbqRKDXKvlSNEtJdIs5huIVuRop4XF8NPVIwei3qah9HXSxep/?=
 =?us-ascii?Q?bENBnhu+hmGi3k7hAz0Azyuh4U4phKuKBmV5iOwi?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: 
	6dd9DBxq8CngAK5O5H/tIKOKXyAAIYGnBAbqGzuqKsr5rmdiB7zrkeFO8MP5cgv9y/VXTfeqiTBiOMf0+k6XnMv4p7PuNOUL6yk8+jZ09gb5fojQ/gDm2c+FZvtJOQjml3qPmJ/i0w7bZ1MebIVyBkYFbULYz/55lN5V8Q0BbLwSgZdWs2z+9Md/7CZWz6NGkqQ9kKgaIVEwaEmsnBqkdNNbFfr/WesQ8IJUA4z4QU6OZ27slHJIrXp7EvFHKMZpngthF06RV/Xya5gqe97ecizRZ+tqw//3HKllxMhLKkGso2EZhpNTt76trlTAYxV89VHUvno5X7HYxDva4Q4u0OF0IxlyVHwS3sHK81Cy5l7NHT6yPp3+Wrgmqwj5Fyfr3iQw29YpdfRNnzjKjbFF9ORGRmpl6t3r/ck9i45fv82x+8Ty7bWUqQFVD2FIhuXNqRVxBg25xr6vXT8qn4pp1JPKsL+rFJk+JdvwCdh6YOFzo7g/S3pYyeE9t2SnIv62m3Bytpez9iandKLfAskNLclefRl52Pq9HVg2eJwhIL6nUzKkDwHJSGjJJIafw4th7xdABa3lKOqbyhlXu8AaxISVHrnb4cDxsiPm/2JWzfU=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3f33f336-4301-4cff-4181-08dc373795f5
X-MS-Exchange-CrossTenant-AuthSource: BYAPR10MB2663.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Feb 2024 01:58:26.0002
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: RrARC2uGB58g6RspEmffP2AVMxFerCS40EH7dduJ+jSrgXFpmLXtXUO+JaCZjOuqLnv538v+7HZqhYP8fasMGg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR10MB5340
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.272,Aquarius:18.0.1011,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2024-02-26_11,2024-02-26_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 bulkscore=0 suspectscore=0 phishscore=0
 spamscore=0 mlxlogscore=999 malwarescore=0 adultscore=0 mlxscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2311290000
 definitions=main-2402270015
X-Proofpoint-GUID: cJsExeayTeOEGst9XTOyIQwWV7YCfHt1
X-Proofpoint-ORIG-GUID: cJsExeayTeOEGst9XTOyIQwWV7YCfHt1

Add the missing close(fd) in guest_memfd_test.c and
private_mem_conversions_test.c.

Signed-off-by: Dongli Zhang <dongli.zhang@oracle.com>
---
 tools/testing/selftests/kvm/guest_memfd_test.c                 | 3 +++
 .../selftests/kvm/x86_64/private_mem_conversions_test.c        | 2 ++
 2 files changed, 5 insertions(+)

diff --git a/tools/testing/selftests/kvm/guest_memfd_test.c b/tools/testing/selftests/kvm/guest_memfd_test.c
index c78a98c1a915..92eae206baa6 100644
--- a/tools/testing/selftests/kvm/guest_memfd_test.c
+++ b/tools/testing/selftests/kvm/guest_memfd_test.c
@@ -167,6 +167,9 @@ static void test_create_guest_memfd_multiple(struct kvm_vm *vm)
 	TEST_ASSERT(ret != -1, "memfd fstat should succeed");
 	TEST_ASSERT(st1.st_size == 4096, "first memfd st_size should still match requested size");
 	TEST_ASSERT(st1.st_ino != st2.st_ino, "different memfd should have different inode numbers");
+
+	close(fd2);
+	close(fd1);
 }
 
 int main(int argc, char *argv[])
diff --git a/tools/testing/selftests/kvm/x86_64/private_mem_conversions_test.c b/tools/testing/selftests/kvm/x86_64/private_mem_conversions_test.c
index 65ad38b6be1f..e0f642d2a3c4 100644
--- a/tools/testing/selftests/kvm/x86_64/private_mem_conversions_test.c
+++ b/tools/testing/selftests/kvm/x86_64/private_mem_conversions_test.c
@@ -434,6 +434,8 @@ static void test_mem_conversions(enum vm_mem_backing_src_type src_type, uint32_t
 
 	r = fallocate(memfd, FALLOC_FL_KEEP_SIZE, 0, memfd_size);
 	TEST_ASSERT(!r, __KVM_SYSCALL_ERROR("fallocate()", r));
+
+	close(memfd);
 }
 
 static void usage(const char *cmd)

base-commit: b82879c81b4f09d040e365f9eb7ca036d86dcdfd
-- 
2.34.1