[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20250305-bannen-zugriff-cd7508258062@brauner>
Date: Wed, 5 Mar 2025 11:53:37 +0100
From: Christian Brauner <brauner@...nel.org>
To: NeilBrown <neilb@...e.de>
Cc: Miklos Szeredi <miklos@...redi.hu>,
Alexander Viro <viro@...iv.linux.org.uk>, Jan Kara <jack@...e.cz>, Chuck Lever <chuck.lever@...cle.com>,
Jeff Layton <jlayton@...nel.org>, Trond Myklebust <trondmy@...nel.org>,
Anna Schumaker <anna@...nel.org>, linux-nfs@...r.kernel.org, Ilya Dryomov <idryomov@...il.com>,
Xiubo Li <xiubli@...hat.com>, ceph-devel@...r.kernel.org, linux-fsdevel@...r.kernel.org,
Richard Weinberger <richard@....at>, Anton Ivanov <anton.ivanov@...bridgegreys.com>,
Johannes Berg <johannes@...solutions.net>, linux-um@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 4/6 - REVISED] fuse: return correct dentry for ->mkdir
On Wed, Mar 05, 2025 at 08:48:20AM +1100, NeilBrown wrote:
>
> Subject: [PATCH] fuse: return correct dentry for ->mkdir
>
> fuse already uses d_splice_alias() to ensure an appropriate dentry is
> found for a newly created dentry. Now that ->mkdir can return that
> dentry we do so.
>
> This requires changing create_new_entry() to return a dentry and
> handling that change in all callers.
>
> Note that when create_new_entry() is asked to create anything other than
> a directory we can be sure it will NOT return an alternate dentry as
> d_splice_alias() only returns an alternate dentry for directories.
> So we don't need to check for that case when passing one the result.
>
> Reviewed-by: Jeff Layton <jlayton@...nel.org>
> Signed-off-by: NeilBrown <neilb@...e.de>
> ---
> fs/fuse/dir.c | 48 +++++++++++++++++++++++++++++++-----------------
> 1 file changed, 31 insertions(+), 17 deletions(-)
>
> Thanks for the suggestion Miklos - this looks much better.
>
> Christian: could you please replace the fuse patch in your tree
> with this version? Thanks.
Sure, done so now. Thanks!
Powered by blists - more mailing lists