Re: [GIT PULL] pidfd patches for v5.1-rc1

From: Christian Brauner
Date: Sat Mar 09 2019 - 14:26:59 EST


On Wed, Mar 06, 2019 at 10:07:13AM +1100, Stephen Rothwell wrote:
> Hi all,
>
> On Tue, 5 Mar 2019 18:13:01 +0100 Christian Brauner <christian@xxxxxxxxxx> wrote:
> >
> > This is the first direct pr I had to send so I hope I didn't make any
> > horrible mistakes.
>
> One thing: you probably should have mentioned that these changes
> conflict with changes in the block and tip trees:

Ah, apparently I didn't make this clear enough in the PR message: Arnd,
Jens, and I aligned the three PRs wrt to the syscall numbers to minimize
merge conflicts. 424 for this one and 425-427 for Jens' io_uring
syscalls. There will still be a minor conflict resulting from this that
hopefully should be easy to fix. I hope that's ok.

Christian

>
> > On Tue, 22 Jan 2019 14:10:27 +1100 Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> wrote:
> > >
> > > Today's linux-next merge of the pidfd tree got conflicts in:
> > >
> > > arch/x86/entry/syscalls/syscall_32.tbl
> > > arch/x86/entry/syscalls/syscall_64.tbl
> > > include/uapi/asm-generic/unistd.h
> > >
> > > between commits:
> > >
> > > 63a96220ad45 ("arch: add split IPC system calls where needed")
> > > 0bd4bb9c5612 ("y2038: add 64-bit time_t syscalls to all 32-bit architectures")
> > >
> > > from the y2038 tree and commit:
> > >
> > > 3d2991bc7a67 ("signal: add pidfd_send_signal() syscall")
> > >
> > > from the pidfd tree.
> >
> > This is now a conflict between the block, tip and pidfd trees. The
> > resolution now looks like below.
> >
> > diff --cc arch/x86/entry/syscalls/syscall_32.tbl
> > index 8da78595d69d,234d91df8ca6..000000000000
> > --- a/arch/x86/entry/syscalls/syscall_32.tbl
> > +++ b/arch/x86/entry/syscalls/syscall_32.tbl
> > @@@ -396,39 -396,6 +396,40 @@@
> > 382 i386 pkey_free sys_pkey_free __ia32_sys_pkey_free
> > 383 i386 statx sys_statx __ia32_sys_statx
> > 384 i386 arch_prctl sys_arch_prctl __ia32_compat_sys_arch_prctl
> > -385 i386 io_pgetevents sys_io_pgetevents __ia32_compat_sys_io_pgetevents
> > +385 i386 io_pgetevents sys_io_pgetevents_time32 __ia32_compat_sys_io_pgetevents
> > 386 i386 rseq sys_rseq __ia32_sys_rseq
> > +# don't use numbers 387 through 392, add new calls at the end
> > +393 i386 semget sys_semget __ia32_sys_semget
> > +394 i386 semctl sys_semctl __ia32_compat_sys_semctl
> > +395 i386 shmget sys_shmget __ia32_sys_shmget
> > +396 i386 shmctl sys_shmctl __ia32_compat_sys_shmctl
> > +397 i386 shmat sys_shmat __ia32_compat_sys_shmat
> > +398 i386 shmdt sys_shmdt __ia32_sys_shmdt
> > +399 i386 msgget sys_msgget __ia32_sys_msgget
> > +400 i386 msgsnd sys_msgsnd __ia32_compat_sys_msgsnd
> > +401 i386 msgrcv sys_msgrcv __ia32_compat_sys_msgrcv
> > +402 i386 msgctl sys_msgctl __ia32_compat_sys_msgctl
> > +403 i386 clock_gettime64 sys_clock_gettime __ia32_sys_clock_gettime
> > +404 i386 clock_settime64 sys_clock_settime __ia32_sys_clock_settime
> > +405 i386 clock_adjtime64 sys_clock_adjtime __ia32_sys_clock_adjtime
> > +406 i386 clock_getres_time64 sys_clock_getres __ia32_sys_clock_getres
> > +407 i386 clock_nanosleep_time64 sys_clock_nanosleep __ia32_sys_clock_nanosleep
> > +408 i386 timer_gettime64 sys_timer_gettime __ia32_sys_timer_gettime
> > +409 i386 timer_settime64 sys_timer_settime __ia32_sys_timer_settime
> > +410 i386 timerfd_gettime64 sys_timerfd_gettime __ia32_sys_timerfd_gettime
> > +411 i386 timerfd_settime64 sys_timerfd_settime __ia32_sys_timerfd_settime
> > +412 i386 utimensat_time64 sys_utimensat __ia32_sys_utimensat
> > +413 i386 pselect6_time64 sys_pselect6 __ia32_compat_sys_pselect6_time64
> > +414 i386 ppoll_time64 sys_ppoll __ia32_compat_sys_ppoll_time64
> > +416 i386 io_pgetevents_time64 sys_io_pgetevents __ia32_sys_io_pgetevents
> > +417 i386 recvmmsg_time64 sys_recvmmsg __ia32_compat_sys_recvmmsg_time64
> > +418 i386 mq_timedsend_time64 sys_mq_timedsend __ia32_sys_mq_timedsend
> > +419 i386 mq_timedreceive_time64 sys_mq_timedreceive __ia32_sys_mq_timedreceive
> > +420 i386 semtimedop_time64 sys_semtimedop __ia32_sys_semtimedop
> > +421 i386 rt_sigtimedwait_time64 sys_rt_sigtimedwait __ia32_compat_sys_rt_sigtimedwait_time64
> > +422 i386 futex_time64 sys_futex __ia32_sys_futex
> > +423 i386 sched_rr_get_interval_time64 sys_sched_rr_get_interval __ia32_sys_sched_rr_get_interval
> > + 424 i386 pidfd_send_signal sys_pidfd_send_signal __ia32_sys_pidfd_send_signal
> > +425 i386 io_uring_setup sys_io_uring_setup __ia32_sys_io_uring_setup
> > +426 i386 io_uring_enter sys_io_uring_enter __ia32_sys_io_uring_enter
> > +427 i386 io_uring_register sys_io_uring_register __ia32_sys_io_uring_register
> > diff --cc arch/x86/entry/syscalls/syscall_64.tbl
> > index c768447f97ec,58f4b3ad4fe0..000000000000
> > --- a/arch/x86/entry/syscalls/syscall_64.tbl
> > +++ b/arch/x86/entry/syscalls/syscall_64.tbl
> > @@@ -343,11 -343,7 +343,12 @@@
> > 332 common statx __x64_sys_statx
> > 333 common io_pgetevents __x64_sys_io_pgetevents
> > 334 common rseq __x64_sys_rseq
> > +# don't use numbers 387 through 423, add new calls after the last
> > +# 'common' entry
> > + 424 common pidfd_send_signal __x64_sys_pidfd_send_signal
> > +425 common io_uring_setup __x64_sys_io_uring_setup
> > +426 common io_uring_enter __x64_sys_io_uring_enter
> > +427 common io_uring_register __x64_sys_io_uring_register
> >
> > #
> > # x32-specific system call numbers start at 512 to avoid cache impact
> > diff --cc include/uapi/asm-generic/unistd.h
> > index 94fde9a14830,c861e7d1053b..000000000000
> > --- a/include/uapi/asm-generic/unistd.h
> > +++ b/include/uapi/asm-generic/unistd.h
> > @@@ -740,58 -740,11 +740,60 @@@ __SC_COMP_3264(__NR_io_pgetevents, sys_
> > __SYSCALL(__NR_rseq, sys_rseq)
> > #define __NR_kexec_file_load 294
> > __SYSCALL(__NR_kexec_file_load, sys_kexec_file_load)
> > +/* 295 through 402 are unassigned to sync up with generic numbers, don't use */
> > +#if __BITS_PER_LONG == 32
> > +#define __NR_clock_gettime64 403
> > +__SYSCALL(__NR_clock_gettime64, sys_clock_gettime)
> > +#define __NR_clock_settime64 404
> > +__SYSCALL(__NR_clock_settime64, sys_clock_settime)
> > +#define __NR_clock_adjtime64 405
> > +__SYSCALL(__NR_clock_adjtime64, sys_clock_adjtime)
> > +#define __NR_clock_getres_time64 406
> > +__SYSCALL(__NR_clock_getres_time64, sys_clock_getres)
> > +#define __NR_clock_nanosleep_time64 407
> > +__SYSCALL(__NR_clock_nanosleep_time64, sys_clock_nanosleep)
> > +#define __NR_timer_gettime64 408
> > +__SYSCALL(__NR_timer_gettime64, sys_timer_gettime)
> > +#define __NR_timer_settime64 409
> > +__SYSCALL(__NR_timer_settime64, sys_timer_settime)
> > +#define __NR_timerfd_gettime64 410
> > +__SYSCALL(__NR_timerfd_gettime64, sys_timerfd_gettime)
> > +#define __NR_timerfd_settime64 411
> > +__SYSCALL(__NR_timerfd_settime64, sys_timerfd_settime)
> > +#define __NR_utimensat_time64 412
> > +__SYSCALL(__NR_utimensat_time64, sys_utimensat)
> > +#define __NR_pselect6_time64 413
> > +__SC_COMP(__NR_pselect6_time64, sys_pselect6, compat_sys_pselect6_time64)
> > +#define __NR_ppoll_time64 414
> > +__SC_COMP(__NR_ppoll_time64, sys_ppoll, compat_sys_ppoll_time64)
> > +#define __NR_io_pgetevents_time64 416
> > +__SYSCALL(__NR_io_pgetevents_time64, sys_io_pgetevents)
> > +#define __NR_recvmmsg_time64 417
> > +__SC_COMP(__NR_recvmmsg_time64, sys_recvmmsg, compat_sys_recvmmsg_time64)
> > +#define __NR_mq_timedsend_time64 418
> > +__SYSCALL(__NR_mq_timedsend_time64, sys_mq_timedsend)
> > +#define __NR_mq_timedreceive_time64 419
> > +__SYSCALL(__NR_mq_timedreceive_time64, sys_mq_timedreceive)
> > +#define __NR_semtimedop_time64 420
> > +__SYSCALL(__NR_semtimedop_time64, sys_semtimedop)
> > +#define __NR_rt_sigtimedwait_time64 421
> > +__SC_COMP(__NR_rt_sigtimedwait_time64, sys_rt_sigtimedwait, compat_sys_rt_sigtimedwait_time64)
> > +#define __NR_futex_time64 422
> > +__SYSCALL(__NR_futex_time64, sys_futex)
> > +#define __NR_sched_rr_get_interval_time64 423
> > +__SYSCALL(__NR_sched_rr_get_interval_time64, sys_sched_rr_get_interval)
> > +#endif
> > + #define __NR_pidfd_send_signal 424
> > + __SYSCALL(__NR_pidfd_send_signal, sys_pidfd_send_signal)
> > +#define __NR_io_uring_setup 425
> > +__SYSCALL(__NR_io_uring_setup, sys_io_uring_setup)
> > +#define __NR_io_uring_enter 426
> > +__SYSCALL(__NR_io_uring_enter, sys_io_uring_enter)
> > +#define __NR_io_uring_register 427
> > +__SYSCALL(__NR_io_uring_register, sys_io_uring_register)
> >
> > #undef __NR_syscalls
> > -#define __NR_syscalls 425
> > +#define __NR_syscalls 428
> >
> > /*
> > * 32 bit systems traditionally used different
>
> --
> Cheers,
> Stephen Rothwell