Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available

From: Dave Chinner
Date: Thu May 05 2016 - 21:40:11 EST

On Thu, May 05, 2016 at 09:04:09PM +0100, David Howells wrote:
> Jeff Layton <jlayton@xxxxxxxxxxxxxxx> wrote:
> > I don't see a real attack vector here either, but OTOH is there a
> > potential user of this at the moment?
> I'm not sure. BSD stat has an st_gen, so it's possible something out there
> will use it if it exists.

Oh, I know of several userspace applications that use the inode
generation number for some purpose. However, all of them are so
tightly tied to the XFS internal structure, implementation and the
XFS specific bulkstat interface that they cannot be considered
generic applications.

> > An earlier chunk of the patch description says:
> >
> > (7) Inode generation number: Useful for FUSE and userspace NFS servers
> >      [Bernd Schubert].  This was asked for but later deemed unnecessary
> >      with the open-by-handle capability available
> >
> > ...the last bit seems to indicate that we don't really need this
> > anyway, as most userland servers now work with filehandles from the
> > kernel.
> >
> > Maybe leave it out for now? It can always be added later.
> Yeah... probably a good idea.

Fine by me.


Dave Chinner