| Message ID | 20190110162435.309262-1-arnd@arndb.de (mailing list archive) |
|---|---|
| Headers | show |
| Series | arch: synchronize syscall tables in preparation for y2038 | expand |
Hi Arnd, On Thu, Jan 10, 2019 at 5:26 PM Arnd Bergmann <arnd@arndb.de> wrote: > The system call tables have diverged a bit over the years, and a number > of the recent additions never made it into all architectures, for one > reason or another. > > This is an attempt to clean it up as far as we can without breaking > compatibility, doing a number of steps: Thanks a lot! > - Add system calls that have not yet been integrated into all > architectures but that we definitely want there. It looks like you missed wiring up io_pgetevents() on m68k. Is that intentional? Gr{oetje,eeting}s, Geert
On Thu, Jan 10, 2019 at 5:59 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > Hi Arnd, > > On Thu, Jan 10, 2019 at 5:26 PM Arnd Bergmann <arnd@arndb.de> wrote: > > The system call tables have diverged a bit over the years, and a number > > of the recent additions never made it into all architectures, for one > > reason or another. > > > > This is an attempt to clean it up as far as we can without breaking > > compatibility, doing a number of steps: > > Thanks a lot! > > > - Add system calls that have not yet been integrated into all > > architectures but that we definitely want there. > > It looks like you missed wiring up io_pgetevents() on m68k. > Is that intentional? Yes, I thought I had described that somewhere but maybe I forgot: semtimedop() and io_pgetevents() get replaced with time64 versions in the follow-up, so I only added them in 64-bit architectures. If you think we should have both io_pgetevents() and io_pgetevents_time32() on all 32-bit architectures, I can add that as well. Arnd
On Thu, 10 Jan 2019, Arnd Bergmann wrote: > - Add system calls that have not yet been integrated into all > architectures but that we definitely want there. glibc has a note that alpha lacks statfs64, any plans for that?
Hi Arnd, On Thu, Jan 10, 2019 at 6:06 PM Arnd Bergmann <arnd@arndb.de> wrote: > On Thu, Jan 10, 2019 at 5:59 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > On Thu, Jan 10, 2019 at 5:26 PM Arnd Bergmann <arnd@arndb.de> wrote: > > > The system call tables have diverged a bit over the years, and a number > > > of the recent additions never made it into all architectures, for one > > > reason or another. > > > > > > This is an attempt to clean it up as far as we can without breaking > > > compatibility, doing a number of steps: > > > > Thanks a lot! > > > > > - Add system calls that have not yet been integrated into all > > > architectures but that we definitely want there. > > > > It looks like you missed wiring up io_pgetevents() on m68k. > > Is that intentional? > > Yes, I thought I had described that somewhere but maybe I > forgot: semtimedop() and io_pgetevents() get replaced with > time64 versions in the follow-up, so I only added them in > 64-bit architectures. If you think we should have both > io_pgetevents() and io_pgetevents_time32() on all 32-bit > architectures, I can add that as well. Thanks, sounds fine to me. Gr{oetje,eeting}s, Geert
On Thu, Jan 10, 2019 at 7:10 PM Joseph Myers <joseph@codesourcery.com> wrote: > > On Thu, 10 Jan 2019, Arnd Bergmann wrote: > > > - Add system calls that have not yet been integrated into all > > architectures but that we definitely want there. > > glibc has a note that alpha lacks statfs64, any plans for that? Good catch, I missed that because all other 64-bit architectures have a statfs() call with 64-bit fields. I see that it also has an osf_statfs64 structure and system call with lots of padding and some oddly sized fields: f_type, f_flags and f_namemax are only 16 bits wide, the rest is all 64-bit. Adding the regular statfs64() should be easy enough, we just need to decide which layout to use: a) use the currently unused 'struct statfs64' as provided by the alpha uapi headers, which has a 32-bit __statfs_word but 64-bit f_blocks, f_bfree, f_bavail, f_files, and f_ffree. b) copy asm-generic/statfs.h to the alpha asm/statfs.h and change statfs64 to have the regular layout that we use on all other 64-bit architectures, using all 64-bit fields. The other open question for alpha (as mentioned in one of the patches I sent) would be whether to add get{eg,eu,g,p,pp,u}id() with the regular calling conventions. Arnd
On Thu, Jan 10, 2019 at 7:11 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > On Thu, Jan 10, 2019 at 6:06 PM Arnd Bergmann <arnd@arndb.de> wrote: > > On Thu, Jan 10, 2019 at 5:59 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > > On Thu, Jan 10, 2019 at 5:26 PM Arnd Bergmann <arnd@arndb.de> wrote: > > > > The system call tables have diverged a bit over the years, and a number > > > > of the recent additions never made it into all architectures, for one > > > > reason or another. > > > > > > > > This is an attempt to clean it up as far as we can without breaking > > > > compatibility, doing a number of steps: > > > > > > Thanks a lot! > > > > > > > - Add system calls that have not yet been integrated into all > > > > architectures but that we definitely want there. > > > > > > It looks like you missed wiring up io_pgetevents() on m68k. > > > Is that intentional? > > > > Yes, I thought I had described that somewhere but maybe I > > forgot: semtimedop() and io_pgetevents() get replaced with > > time64 versions in the follow-up, so I only added them in > > 64-bit architectures. If you think we should have both > > io_pgetevents() and io_pgetevents_time32() on all 32-bit > > architectures, I can add that as well. > > Thanks, sounds fine to me. Just to be sure, you mean it's fine to not add it, not that we should add it? Arnd
On Thu, Jan 10, 2019 at 11:42:32PM +0100, Arnd Bergmann wrote: > On Thu, Jan 10, 2019 at 7:10 PM Joseph Myers <joseph@codesourcery.com> wrote: > > > > On Thu, 10 Jan 2019, Arnd Bergmann wrote: > > > > > - Add system calls that have not yet been integrated into all > > > architectures but that we definitely want there. > > > > glibc has a note that alpha lacks statfs64, any plans for that? > > Good catch, I missed that because all other 64-bit architectures > have a statfs() call with 64-bit fields. I see that it also has an > osf_statfs64 structure and system call with lots of padding and some > oddly sized fields: f_type, f_flags and f_namemax are only 16 bits > wide, the rest is all 64-bit. > > Adding the regular statfs64() should be easy enough, we just need to > decide which layout to use: > > a) use the currently unused 'struct statfs64' as provided by the > alpha uapi headers, which has a 32-bit __statfs_word but > 64-bit f_blocks, f_bfree, f_bavail, f_files, and f_ffree. > > b) copy asm-generic/statfs.h to the alpha asm/statfs.h and > change statfs64 to have the regular layout that we use > on all other 64-bit architectures, using all 64-bit fields. > > The other open question for alpha (as mentioned in one of the > patches I sent) would be whether to add get{eg,eu,g,p,pp,u}id() > with the regular calling conventions. I would be interested in seeing those provided on Alpha. There are a handful of projects choosing to sidestep glibc for these syscalls and they break on Alpha. Such projects are never keen to include special assembler code (because the extant syscalls on Alpha are not C ABI compliant) to support a legacy arch. Cheers Michael.
Hi Arnd, On Thu, Jan 10, 2019 at 11:43 PM Arnd Bergmann <arnd@arndb.de> wrote: > On Thu, Jan 10, 2019 at 7:11 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > On Thu, Jan 10, 2019 at 6:06 PM Arnd Bergmann <arnd@arndb.de> wrote: > > > On Thu, Jan 10, 2019 at 5:59 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > > > On Thu, Jan 10, 2019 at 5:26 PM Arnd Bergmann <arnd@arndb.de> wrote: > > > > > The system call tables have diverged a bit over the years, and a number > > > > > of the recent additions never made it into all architectures, for one > > > > > reason or another. > > > > > > > > > > This is an attempt to clean it up as far as we can without breaking > > > > > compatibility, doing a number of steps: > > > > > > > > Thanks a lot! > > > > > > > > > - Add system calls that have not yet been integrated into all > > > > > architectures but that we definitely want there. > > > > > > > > It looks like you missed wiring up io_pgetevents() on m68k. > > > > Is that intentional? > > > > > > Yes, I thought I had described that somewhere but maybe I > > > forgot: semtimedop() and io_pgetevents() get replaced with > > > time64 versions in the follow-up, so I only added them in > > > 64-bit architectures. If you think we should have both > > > io_pgetevents() and io_pgetevents_time32() on all 32-bit > > > architectures, I can add that as well. > > > > Thanks, sounds fine to me. > > Just to be sure, you mean it's fine to not add it, not that we should > add it? I'm fine with not having the legacy 32-bit ones. Sorry for being unclear. Gr{oetje,eeting}s, Geert
The system call tables have diverged a bit over the years, and a number of the recent additions never made it into all architectures, for one reason or another. This is an attempt to clean it up as far as we can without breaking compatibility, doing a number of steps: - Add system calls that have not yet been integrated into all architectures but that we definitely want there. - Add the separate ipc syscalls on all architectures that traditionally only had sys_ipc(). This version is done without support for IPC_OLD that is we have in sys_ipc. The new semtimedop_time64 syscall will only be added here, not in sys_ipc - Add syscall numbers for a couple of syscalls that we probably don't need everywhere, in particular pkey_* and rseq, for the purpose of symmetry: if it's in asm-generic/unistd.h, it makes sense to have it everywhere. - Prepare for having the same system call numbers for any future calls. In combination with the generated tables, this hopefully makes it easier to add new calls across all architectures together. Most of the contents of this series are unrelated to the actual y2038 work, but for the moment, that second series is based on this one. If there are any concerns about changes here, I can drop or rewrite any individual patch in this series. My plan is to merge any patches in this series that are found to be good together with the y2038 patches for linux-5.1, so please review and provide Acks for merging through my tree, or pick them up for 5.0 if they seem urgent enough. Arnd Arnd Bergmann (15): ia64: add __NR_umount2 definition ia64: add statx and io_pgetevents syscalls ia64: assign syscall numbers for perf and seccomp alpha: wire up io_pgetevents system call alpha: update syscall macro definitions ARM: add migrate_pages() system call ARM: add kexec_file_load system call number m68k: assign syscall number for seccomp sh: remove duplicate unistd_32.h file sh: add statx system call mips: fix n32 compat_ipc_parse_version sparc64: fix sparc_ipc type conversion ipc: rename old-style shmctl/semctl/msgctl syscalls arch: add split IPC system calls where needed arch: add pkey and rseq syscall numbers everywhere arch/alpha/include/asm/unistd.h | 10 - arch/alpha/include/uapi/asm/unistd.h | 5 + arch/alpha/kernel/syscalls/syscall.tbl | 15 +- arch/arm/include/asm/unistd.h | 1 - arch/arm/tools/syscall.tbl | 8 +- arch/arm64/include/asm/unistd.h | 2 +- arch/arm64/include/asm/unistd32.h | 10 +- arch/ia64/include/asm/unistd.h | 14 - arch/ia64/include/uapi/asm/unistd.h | 2 + arch/ia64/kernel/syscalls/syscall.tbl | 10 +- arch/m68k/kernel/syscalls/syscall.tbl | 16 + arch/microblaze/kernel/syscalls/syscall.tbl | 6 +- arch/mips/Kconfig | 1 + arch/mips/kernel/syscalls/syscall_n32.tbl | 6 +- arch/mips/kernel/syscalls/syscall_n64.tbl | 6 +- arch/mips/kernel/syscalls/syscall_o32.tbl | 11 + arch/parisc/include/asm/unistd.h | 3 - arch/parisc/kernel/syscalls/syscall.tbl | 4 + arch/powerpc/kernel/syscalls/syscall.tbl | 12 + arch/s390/include/asm/unistd.h | 3 - arch/s390/kernel/syscalls/syscall.tbl | 15 + arch/sh/include/uapi/asm/unistd_32.h | 403 -------------------- arch/sh/kernel/syscalls/syscall.tbl | 16 + arch/sparc/include/asm/unistd.h | 5 - arch/sparc/kernel/sys_sparc_64.c | 2 +- arch/sparc/kernel/syscalls/syscall.tbl | 16 + arch/x86/entry/syscalls/syscall_32.tbl | 11 + arch/xtensa/kernel/syscalls/syscall.tbl | 7 +- include/linux/syscalls.h | 3 + ipc/msg.c | 39 +- ipc/sem.c | 39 +- ipc/shm.c | 40 +- ipc/syscall.c | 12 +- ipc/util.h | 21 +- kernel/sys_ni.c | 3 + 35 files changed, 271 insertions(+), 506 deletions(-) delete mode 100644 arch/sh/include/uapi/asm/unistd_32.h