Re: [PATCH v3] vfs: support statx(..., NULL, AT_EMPTY_PATH, ...)

From: Xi Ruoyao
Date: Thu Sep 26 2024 - 09:02:51 EST


On Wed, 2024-09-25 at 17:56 +0200, Mateusz Guzik wrote:
> On Wed, Sep 25, 2024 at 4:30 PM Xi Ruoyao <xry111@xxxxxxxxxxx> wrote:
> > There's a special case, AT_FDCWD + NULL + AT_EMPTY_PATH, still resulting
> > EFAULT, while AT_FDCWD + "" + AT_EMPTY_PATH is OK (returning the stat of
> > current directory).
> >
> > I know allowing NULL with AT_FDCWD won't produce any performance gain,
> > but it seems the difference would make the document of the API more
> > nasty.
> >
> > So is it acceptable to make the kernel "hide" this difference, i.e.
> > accept AT_FDCWD + NULL + AT_EMPTY_PATH as-is AT_FDCWD + "" +
> > AT_EMPTY_PATH?
> >
>
> huh, that indeed makes sense to add. kind of weird this was not sorted
> out at the time, but i'm not going to pointer a finger at myself :) so
> ACK from me as far as the idea goes
>
> I presume you can do the honors? :)

Should I use a Fixes: tag in the commit then? (I.e. should it be
backported to 6.11?)


--
Xi Ruoyao <xry111@xxxxxxxxxxx>
School of Aerospace Science and Technology, Xidian University