Hi Helge, On Mon, 8 Oct 2018 at 11:36, Helge Deller <deller@xxxxxx> wrote: > > On 08.10.2018 07:52, Firoz Khan wrote: > > Hi Helge, > > > > On Mon, 8 Oct 2018 at 11:11, Helge Deller <deller@xxxxxx> wrote: > >> > >> On 08.10.2018 07:16, Firoz Khan wrote: > >>> Signed-off-by: Firoz Khan <firoz.khan@xxxxxxxxxx> > >>> --- > >>> arch/parisc/kernel/syscalls/syscall.tbl | 3 ++- > >>> 1 file changed, 2 insertions(+), 1 deletion(-) > >>> > >>> diff --git a/arch/parisc/kernel/syscalls/syscall.tbl b/arch/parisc/kernel/syscalls/syscall.tbl > >>> index 4e85293..4334bb7 100644 > >>> --- a/arch/parisc/kernel/syscalls/syscall.tbl > >>> +++ b/arch/parisc/kernel/syscalls/syscall.tbl > >>> @@ -349,4 +349,5 @@ > >>> 347 common preadv2 sys_preadv2 compat_sys_preadv2 > >>> 348 common pwritev2 sys_pwritev2 compat_sys_pwritev2 > >>> 349 common statx sys_statx > >>> -350 common io_pgetevents sys_io_pgetevents compat_sys_io_pgetevents > >>> \ No newline at end of file > >>> +350 common io_pgetevents sys_io_pgetevents compat_sys_io_pgetevents > >>> +351 common rseq sys_rseq compat_sys_rseq > >> > >> You can't add the rseq syscall for parisc yet. > >> It needs additional code in the kernel for parisc which hasn't been tested yet. > >> See my initial untested patch at https://patchwork.kernel.org/patch/10495209/ > > > > Thanks for your update! > > > > When I compiled the kernel I got below warnings. > > > > <stdin>:696:2: warning: #warning syscall nfsservctl not implemented [-Wcpp] > > <stdin>:1335:2: warning: #warning syscall rseq not implemented [-Wcpp] > > > > I added an IGNORE entry nfsservctl in script/checksyscalls.sh because this > > syscall is gone. But we definitely have to keep rseq entry on parisc > > architecture. > > I prefer to keep the warning for rseq for now. I'm fine with this. > It reminds me that we still may want the rseq syscall. > If the warning is a problem, you may simply add the __IGNORE_rseq define. But I still feel to keep an IGNORE entry, so once you test your patch; we can remove IGNORE entry and update the syscall.tbl. I would like you to take the call here :) Thanks Firoz > > > I can ignore this patch currently as your patch not yet tested. > > > > FYI, I have merged the system call table files based on our previous > > discussions. > > Please comment on that. > > I'll do after testing. > > Thanks! > Helge > > > I would appreciate if you can perform a boot test without this patch > > on the actual > > platform. > > > > Firoz > > > >> > >> Helge >
