[LTP] [PATCH v6 2/2] syscalls/fsmount01: Add test for new mount API v5.2

[Jan Stancek <jstancek@redhat.com>]

----- Original Message -----
> Hi Petr,
> 
> On Sun, Feb 16, 2020 at 9:17 PM Petr Vorel <pvorel@suse.cz> wrote:
> 
> > ...
> > > >  include/lapi/syscalls/powerpc64.in            |  4 +
> > > Is there any reason why only add syscall num for ppc64?
> > The other numbers are already there:
> > 01e4dc222 lapi/syscalls: Add MIPS support
> > c2f27f6e9 Add syscall numbers for new file-system related syscalls
> >
> 
> Good to know this.
> 
> Not sure if we should add a note in the commit message to prevent confusion
> > later (probably not needed).
> >
> 
> Or just mentionion that commit(c2f27f6e9 Add syscall numbers ...) message.
> 
> > BTW, I like the way Viresh Kumar gives in his fsmount.h, it looks more
> > tidy
> > > and clean.
> > > http://lists.linux.it/pipermail/ltp/2020-February/015413.html
> > Hm, competing implementations.
> > Both tries to handle preventing redefinitions (e.g. FSOPEN_CLOEXEC) once
> > the API hits libc headers (at least in musl it might be soon).
> > Zorro tries to bind them to function check (e.g. #ifndef HAVE_FSMOUNT,
> > #ifndef
> > HAVE_MOVE_MOUNT), Viresh just use single check #ifndef OPEN_TREE_CLONE.
> > I slightly prefer Viresh way (it's unlikely that libc headers would
> > include just
> >
> +1 Viresh way.
> 
> 
> > part of the new mount API definitions, although obviously the most safest
> > way
> > would be to either guard with #ifndef each definition or just give up on
> > testing
> > header and copy whole include/uapi/linux/mount.h (+ avoid using
> > sys/mount.h;
> > that's the way used for include/lapi/bpf.h).
> >
> 
> @Cyril, @Jan, any else suggestion?

I'd go with additions to lapi, and avoid copying entire linux/mount.h. And use
#ifndef for each definition. v7 is currently not doing that, but it's easy
to add if we run into problems later, when/if there are additions to mount API.