[LTP] [PATCH v4] Correctly check if PIDFD_INFO_EXIT is available
Andrea Cervesato
andrea.cervesato@suse.de
Wed Jul 30 14:20:14 CEST 2025
From: Andrea Cervesato <andrea.cervesato@suse.com>
When systems are not having the PIDFD_INFO_EXIT implementation,
ioctl_pidfd testing suite might fail with:
ioctl_pidfd.h:32: TBROK: ioctl(...) failed: ENOTTY (25)
Fix the ioctl_pidfd_info_exit_supported() behavior, considering ENOTTY,
EINVAL and ESRCH errors like a sign for not having PIDFD_INFO_EXIT
implemented in the system.
Reviewed-by: Cyril Hrubis <chrubis@suse.cz>
Reviewed-by: Avinesh Kumar <akumar@suse.de>
Tested-by: Avinesh Kumar <akumar@suse.de>
Signed-off-by: Andrea Cervesato <andrea.cervesato@suse.com>
---
Changes in v4:
- use the right AND logic...
- Link to v3: https://lore.kernel.org/r/20250730-ioctl_pidfd_supported-v3-1-f40c047b29de@suse.com
Changes in v3:
- take in consideration also EINVAL and ESRCH errors
- Link to v2: https://lore.kernel.org/r/20250729-ioctl_pidfd_supported-v2-1-f9ef90134138@suse.com
Changes in v2:
- close pidfd when PIDFD_INFO_EXIT is supported
- Link to v1: https://lore.kernel.org/r/20250729-ioctl_pidfd_supported-v1-1-985626a0c46b@suse.com
---
testcases/kernel/syscalls/ioctl/ioctl_pidfd.h | 24 ++++++++++++++++++------
1 file changed, 18 insertions(+), 6 deletions(-)
diff --git a/testcases/kernel/syscalls/ioctl/ioctl_pidfd.h b/testcases/kernel/syscalls/ioctl/ioctl_pidfd.h
index 8249ac753cf7fb8a3b749d55c7f0c3b30482c114..811f969cdee8147dfa43bc2ab1124dfd9fd21e41 100644
--- a/testcases/kernel/syscalls/ioctl/ioctl_pidfd.h
+++ b/testcases/kernel/syscalls/ioctl/ioctl_pidfd.h
@@ -11,9 +11,10 @@
static inline int ioctl_pidfd_info_exit_supported(void)
{
- int ret = 0;
+ int ret;
pid_t pid;
int pidfd;
+ int supported = 0;
struct pidfd_info info;
if (tst_kvercmp(6, 15, 0) >= 0)
@@ -29,13 +30,24 @@ static inline int ioctl_pidfd_info_exit_supported(void)
pidfd = SAFE_PIDFD_OPEN(pid, 0);
SAFE_WAITPID(pid, NULL, 0);
- SAFE_IOCTL(pidfd, PIDFD_GET_INFO, &info);
- SAFE_CLOSE(pidfd);
+ ret = ioctl(pidfd, PIDFD_GET_INFO, &info);
+ if (ret == -1) {
+ /* - ENOTTY: old kernels not implementing fs/pidfs.c:pidfd_ioctl
+ * - EINVAL: until v6.13 kernel
+ * - ESRCH: all kernels between v6.13 and v6.15
+ */
+ if (errno != ENOTTY &&
+ errno != EINVAL &&
+ errno != ESRCH)
+ tst_brk(TBROK | TERRNO, "ioctl error");
+ } else {
+ if (info.mask & PIDFD_INFO_EXIT)
+ supported = 1;
+ }
- if (info.mask & PIDFD_INFO_EXIT)
- ret = 1;
+ SAFE_CLOSE(pidfd);
- return ret;
+ return supported;
}
#endif
---
base-commit: 890d6d64a235548cec559ffaf2e9208b80e6902b
change-id: 20250729-ioctl_pidfd_supported-437d25a4c382
Best regards,
--
Andrea Cervesato <andrea.cervesato@suse.com>
More information about the ltp
mailing list