Re: file metadata via fs API (was: [GIT PULL] Filesystem Information)

From: Al Viro
Date: Tue Aug 11 2020 - 10:08:47 EST


On Tue, Aug 11, 2020 at 03:54:19PM +0200, Miklos Szeredi wrote:
> On Wed, Aug 05, 2020 at 10:24:23AM +0200, Miklos Szeredi wrote:
> > On Tue, Aug 4, 2020 at 4:36 PM Miklos Szeredi <miklos@xxxxxxxxxx> wrote:
> >
> > > I think we already lost that with the xattr API, that should have been
> > > done in a way that fits this philosophy. But given that we have "/"
> > > as the only special purpose char in filenames, and even repetitions
> > > are allowed, it's hard to think of a good way to do that. Pity.
> >
> > One way this could be solved is to allow opting into an alternative
> > path resolution mode.
> >
> > E.g.
> > openat(AT_FDCWD, "foo/bar//mnt/info", O_RDONLY | O_ALT);
>
> Proof of concept patch and test program below.
>
> Opted for triple slash in the hope that just maybe we could add a global
> /proc/sys/fs/resolve_alt knob to optionally turn on alternative (non-POSIX) path
> resolution without breaking too many things. Will try that later...
>
> Comments?

Hell, NO. This is unspeakably tasteless. And full of lovely corner cases wrt
symlink bodies, etc.

Consider that one NAKed. I'm seriously unhappy with the entire fsinfo thing
in general, but this one is really over the top.