Re: [PATCH v4 0/4] m68k: system call table generation support
Hi Geert, On Sat, 27 Oct 2018 at 20:17, Geert Uytterhoeven wrote: > > Hi Firoz, > > On Fri, Oct 26, 2018 at 7:06 AM Firoz Khan wrote: > > The purpose of this patch series is, we can easily add/modify/delete > > system call table support by changing entry in syscall.tbl file > > instead of manually changing many files. The other goal is to unify > > the system call table generation support implementation across all > > the architectures. > > > > The system call tables are in different format in all architecture. > > It will be difficult to manually add, modify or delete the system > > calls in the respective files manually. To make it easy by keeping > > a script and which'll generate uapi header file and syscall table > > file. > > > > syscall.tbl contains the list of available system calls along with > > system call number and corresponding entry point. Add a new system > > call in this architecture will be possible by adding new entry in > > the syscall.tbl file. > > > > Adding a new table entry consisting of: > > - System call number. > > - ABI. > > - System call name. > > - Entry point name. > > > > ARM, s390 and x86 architecuture does exist the similar support. I > > leverage their implementation to come up with a generic solution. > > > > I have done the same support for work for alpha, ia64, microblaze, > > mips, parisc, powerpc, sh, sparc, and xtensa. Below mentioned git > > repository contains more details. > > Git repo:- https://github.com/frzkhn/system_call_table_generator/ > > > > Finally, this is the ground work to solve the Y2038 issue. We need > > to add two dozen of system calls to solve Y2038 issue. So this patch > > series will help to add new system calls easily by adding new entry > > in the syscall.tbl. > > Thanks for the update! > > Can you please tell the audience what has been changed in v4? Sure. - I added the comments in syscall.tbl and solved mixed (space and tab) indentation. - Modified the scripts (syscalltbl.sh and syscallhdr.sh) as per the feedback provided by Arnd, Finn Thain, Eugene Syromiatnikov and Rolf Eike Beer. Eg:- - echo options are not portable; changed to printf - "let" is a bash extension; changed posix-conformant expression. - Finally I optimized the script and solved mixed indentation. > > When posting a new version of a patch or patch series, it is a good > idea to include a changelog in the cover letter and/or patches. Sure, will do! Thanks Firoz > > Thanks! > > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- > ge...@linux-m68k.org > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like > that. > -- Linus Torvalds
Re: [PATCH v4 0/4] m68k: system call table generation support
Hi Firoz, On Fri, Oct 26, 2018 at 7:06 AM Firoz Khan wrote: > The purpose of this patch series is, we can easily add/modify/delete > system call table support by changing entry in syscall.tbl file > instead of manually changing many files. The other goal is to unify > the system call table generation support implementation across all > the architectures. > > The system call tables are in different format in all architecture. > It will be difficult to manually add, modify or delete the system > calls in the respective files manually. To make it easy by keeping > a script and which'll generate uapi header file and syscall table > file. > > syscall.tbl contains the list of available system calls along with > system call number and corresponding entry point. Add a new system > call in this architecture will be possible by adding new entry in > the syscall.tbl file. > > Adding a new table entry consisting of: > - System call number. > - ABI. > - System call name. > - Entry point name. > > ARM, s390 and x86 architecuture does exist the similar support. I > leverage their implementation to come up with a generic solution. > > I have done the same support for work for alpha, ia64, microblaze, > mips, parisc, powerpc, sh, sparc, and xtensa. Below mentioned git > repository contains more details. > Git repo:- https://github.com/frzkhn/system_call_table_generator/ > > Finally, this is the ground work to solve the Y2038 issue. We need > to add two dozen of system calls to solve Y2038 issue. So this patch > series will help to add new system calls easily by adding new entry > in the syscall.tbl. Thanks for the update! Can you please tell the audience what has been changed in v4? When posting a new version of a patch or patch series, it is a good idea to include a changelog in the cover letter and/or patches. Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
[PATCH v4 0/4] m68k: system call table generation support
The purpose of this patch series is, we can easily add/modify/delete system call table support by changing entry in syscall.tbl file instead of manually changing many files. The other goal is to unify the system call table generation support implementation across all the architectures. The system call tables are in different format in all architecture. It will be difficult to manually add, modify or delete the system calls in the respective files manually. To make it easy by keeping a script and which'll generate uapi header file and syscall table file. syscall.tbl contains the list of available system calls along with system call number and corresponding entry point. Add a new system call in this architecture will be possible by adding new entry in the syscall.tbl file. Adding a new table entry consisting of: - System call number. - ABI. - System call name. - Entry point name. ARM, s390 and x86 architecuture does exist the similar support. I leverage their implementation to come up with a generic solution. I have done the same support for work for alpha, ia64, microblaze, mips, parisc, powerpc, sh, sparc, and xtensa. Below mentioned git repository contains more details. Git repo:- https://github.com/frzkhn/system_call_table_generator/ Finally, this is the ground work to solve the Y2038 issue. We need to add two dozen of system calls to solve Y2038 issue. So this patch series will help to add new system calls easily by adding new entry in the syscall.tbl. Firoz Khan (4): m68k: rename system call table file name m68k: add __NR_syscalls along with NR_syscalls m68k: add system call table generation support m68k: generate uapi header and syscall table header files arch/m68k/Makefile | 3 + arch/m68k/include/asm/Kbuild| 1 + arch/m68k/include/asm/unistd.h | 3 +- arch/m68k/include/uapi/asm/Kbuild | 1 + arch/m68k/include/uapi/asm/unistd.h | 385 +- arch/m68k/kernel/Makefile | 2 +- arch/m68k/kernel/syscall_table.S| 26 +++ arch/m68k/kernel/syscalls/Makefile | 38 +++ arch/m68k/kernel/syscalls/syscall.tbl | 389 ++ arch/m68k/kernel/syscalls/syscallhdr.sh | 36 +++ arch/m68k/kernel/syscalls/syscalltbl.sh | 32 +++ arch/m68k/kernel/syscalltable.S | 403 12 files changed, 529 insertions(+), 790 deletions(-) create mode 100644 arch/m68k/kernel/syscall_table.S create mode 100644 arch/m68k/kernel/syscalls/Makefile create mode 100644 arch/m68k/kernel/syscalls/syscall.tbl create mode 100644 arch/m68k/kernel/syscalls/syscallhdr.sh create mode 100644 arch/m68k/kernel/syscalls/syscalltbl.sh delete mode 100644 arch/m68k/kernel/syscalltable.S -- 1.9.1