Re: [PATCH 05/14] tools headers UAPI: Sync linux/sched.h with the kernel

From: Christian Brauner
Date: Tue May 28 2019 - 13:57:13 EST


On May 28, 2019 7:50:11 PM GMT+02:00, Arnaldo Carvalho de Melo <acme@xxxxxxxxxx> wrote:
>From: Arnaldo Carvalho de Melo <acme@xxxxxxxxxx>
>
>To pick up the change in:
>
> b3e583825266 ("clone: add CLONE_PIDFD")
>
>This requires changes in the 'perf trace' beautification routines for
>the 'clone' syscall args, which is done in a followup patch.
>
>This silences the following perf build warning:
>
>Warning: Kernel ABI header at 'tools/include/uapi/linux/sched.h'
>differs from latest version at 'include/uapi/linux/sched.h'
> diff -u tools/include/uapi/linux/sched.h include/uapi/linux/sched.h
>
>Cc: Adrian Hunter <adrian.hunter@xxxxxxxxx>
>Cc: Brendan Gregg <brendan.d.gregg@xxxxxxxxx>
>Cc: Christian Brauner <christian@xxxxxxxxxx>
>Cc: Jiri Olsa <jolsa@xxxxxxxxxx>
>Cc: Luis ClÃudio GonÃalves <lclaudio@xxxxxxxxxx>
>Cc: Namhyung Kim <namhyung@xxxxxxxxxx>
>Link:
>https://lkml.kernel.org/n/tip-lenja6gmy26dkt0ybk747qgq@xxxxxxxxxxxxxx
>Signed-off-by: Arnaldo Carvalho de Melo <acme@xxxxxxxxxx>
>---
> tools/include/uapi/linux/sched.h | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/tools/include/uapi/linux/sched.h
>b/tools/include/uapi/linux/sched.h
>index 22627f80063e..ed4ee170bee2 100644
>--- a/tools/include/uapi/linux/sched.h
>+++ b/tools/include/uapi/linux/sched.h
>@@ -10,6 +10,7 @@
>#define CLONE_FS 0x00000200 /* set if fs info shared between processes
>*/
>#define CLONE_FILES 0x00000400 /* set if open files shared between
>processes */
>#define CLONE_SIGHAND 0x00000800 /* set if signal handlers and blocked
>signals shared */
>+#define CLONE_PIDFD 0x00001000 /* set if a pidfd should be placed in
>parent */
>#define CLONE_PTRACE 0x00002000 /* set if we want to let tracing
>continue on the child too */
>#define CLONE_VFORK 0x00004000 /* set if the parent wants the child to
>wake it up on mm_release */
>#define CLONE_PARENT 0x00008000 /* set if we want to have the same
>parent as the cloner */

Thanks for doing that!

Reviewed-by: Christian Brauner <christian@xxxxxxxxxx>