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

[Christian Brauner <christian@brauner.io>]

On May 28, 2019 7:50:11 PM GMT+02:00, Arnaldo Carvalho de Melo <acme@kernel.org> wrote:
>From: Arnaldo Carvalho de Melo <acme@redhat.com>
>
>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@intel.com>
>Cc: Brendan Gregg <brendan.d.gregg@gmail.com>
>Cc: Christian Brauner <christian@brauner.io>
>Cc: Jiri Olsa <jolsa@kernel.org>
>Cc: Luis Cláudio Gonçalves <lclaudio@redhat.com>
>Cc: Namhyung Kim <namhyung@kernel.org>
>Link:
>https://lkml.kernel.org/n/tip-lenja6gmy26dkt0ybk747qgq@git.kernel.org
>Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
>---
> 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@brauner.io>