[PATCH v2 2/5] linux: Use compile_c_snippet to check linux/pidfd.h availability

Florian Weimer fweimer@redhat.com
Wed Aug 10 17:38:37 GMT 2022


* Adhemerval Zanella:

> Instead of tying to a specific kernel version.
>
> Checked on x86_64-linux-gnu.
> ---
>  sysdeps/unix/sysv/linux/tst-pidfd-consts.py | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/sysdeps/unix/sysv/linux/tst-pidfd-consts.py b/sysdeps/unix/sysv/linux/tst-pidfd-consts.py
> index e207b55eef..d732173abd 100644
> --- a/sysdeps/unix/sysv/linux/tst-pidfd-consts.py
> +++ b/sysdeps/unix/sysv/linux/tst-pidfd-consts.py
> @@ -33,10 +33,12 @@ def main():
>                          help='C compiler (including options) to use')
>      args = parser.parse_args()
>  
> -    linux_version_headers = glibcsyscalls.linux_kernel_version(args.cc)
> -    # Linux started to provide pidfd.h with 5.10.
> -    if linux_version_headers < (5, 10):
> +    if glibcextract.compile_c_snippet(
> +            '#include <linux/pidfd.h>',
> +            args.cc).returncode != 0:
>          sys.exit (77)
> +
> +    linux_version_headers = glibcsyscalls.linux_kernel_version(args.cc)
>      linux_version_glibc = (5, 19)
>      sys.exit(glibcextract.compare_macro_consts(
>                  '#include <sys/pidfd.h>\n',

Looks fine once compile_c_snippet is there.

Reviewed-by: Florian Weimer <fweimer@redhat.com>

Thanks,
Florian



More information about the Libc-alpha mailing list