On Wed, Mar 05, 2025 at 01:06:35PM +0100, Oleg Nesterov wrote: > On 03/05, Christian Brauner wrote: > > > > Christian Brauner (16): > > pidfs: switch to copy_struct_to_user() > > pidfd: rely on automatic cleanup in __pidfd_prepare() > > pidfs: move setting flags into pidfs_alloc_file() > > pidfs: use private inode slab cache > > pidfs: record exit code and cgroupid at exit > > pidfs: allow to retrieve exit information > > selftests/pidfd: fix header inclusion > > pidfs/selftests: ensure correct headers for ioctl handling > > selftests/pidfd: expand common pidfd header > > selftests/pidfd: add first PIDFD_INFO_EXIT selftest > > selftests/pidfd: add second PIDFD_INFO_EXIT selftest > > selftests/pidfd: add third PIDFD_INFO_EXIT selftest > > selftests/pidfd: add fourth PIDFD_INFO_EXIT selftest > > selftests/pidfd: add fifth PIDFD_INFO_EXIT selftest > > selftests/pidfd: add sixth PIDFD_INFO_EXIT selftest > > selftests/pidfd: add seventh PIDFD_INFO_EXIT selftest > > I see nothing wrong in V3. For 1-6 Fwiw, I'm quite happy with the tests since we now also have test for multi-threaded exec behavior with pidfd polling. > > Reviewed-by: Oleg Nesterov <oleg@xxxxxxxxxx> Thanks for all the help and the Ack! There'll be more patches this or next cycle though. ;)