Re: [PATCH 02/19] fsinfo: Add syscalls to other arches [ver #16]

[Geert Uytterhoeven]

On Fri, Feb 21, 2020 at 3:52 PM Christian Brauner
<christian.brauner@xxxxxxxxxx> wrote:
> On Tue, Feb 18, 2020 at 05:05:13PM +0000, David Howells wrote:
> > Add the fsinfo syscall to the other arches.
>
> Over the last couple of kernel releases we have established the
> convention that we wire-up a syscall for all arches at the same time and
> not e.g. x86 and then the other separately.

Indeed.

> > Signed-off-by: David Howells <dhowells@xxxxxxxxxx>
> > ---
> >
> >  arch/alpha/kernel/syscalls/syscall.tbl      |    1 +
> >  arch/arm/tools/syscall.tbl                  |    1 +
> >  arch/arm64/include/asm/unistd.h             |    2 +-
> >  arch/arm64/include/asm/unistd32.h           |    2 +-
> >  arch/ia64/kernel/syscalls/syscall.tbl       |    1 +
> >  arch/m68k/kernel/syscalls/syscall.tbl       |    2 ++

Suspicious diffstat...

> >  arch/microblaze/kernel/syscalls/syscall.tbl |    1 +
> >  arch/mips/kernel/syscalls/syscall_n32.tbl   |    1 +
> >  arch/mips/kernel/syscalls/syscall_n64.tbl   |    1 +
> >  arch/mips/kernel/syscalls/syscall_o32.tbl   |    1 +
> >  arch/parisc/kernel/syscalls/syscall.tbl     |    1 +
> >  arch/powerpc/kernel/syscalls/syscall.tbl    |    1 +
> >  arch/s390/kernel/syscalls/syscall.tbl       |    1 +
> >  arch/sh/kernel/syscalls/syscall.tbl         |    1 +
> >  arch/sparc/kernel/syscalls/syscall.tbl      |    1 +
> >  arch/xtensa/kernel/syscalls/syscall.tbl     |    1 +
> >  16 files changed, 17 insertions(+), 2 deletions(-)

> > --- a/arch/m68k/kernel/syscalls/syscall.tbl
> > +++ b/arch/m68k/kernel/syscalls/syscall.tbl
> > @@ -437,3 +437,5 @@
> >  435  common  clone3                          __sys_clone3
> >  437  common  openat2                         sys_openat2
> >  438  common  pidfd_getfd                     sys_pidfd_getfd
> > +# 435 reserved for clone3

Indeed, bogus line above (bad rebase?).

> > +439  common  fsinfo                          sys_fsinfo

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

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