Re: [PATCH 2/2] nfsd: drop fh_update() from S_IFDIR branch of nfsd_create_locked()

[Jeff Layton <jlayton@kernel.org>]

On Wed, 2025-02-26 at 17:18 +1100, NeilBrown wrote:
> nfsd_create_locked() doesn't need to explicitly call fh_update().
> On success (which is the only time that fh_update() matters at all),
> nfsd_create_setattr() will be called and it will call fh_update().
> 
> This extra call is not harmful, but is not necessary.
> 
> Signed-off-by: NeilBrown <neilb@suse.de>
> ---
>  fs/nfsd/vfs.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c
> index 29cb7b812d71..1035010f1198 100644
> --- a/fs/nfsd/vfs.c
> +++ b/fs/nfsd/vfs.c
> @@ -1505,11 +1505,8 @@ nfsd_create_locked(struct svc_rqst *rqstp, struct svc_fh *fhp,
>  			}
>  			dput(resfhp->fh_dentry);
>  			resfhp->fh_dentry = dget(d);
> -			err = fh_update(resfhp);
>  			dput(dchild);
>  			dchild = d;
> -			if (err)
> -				goto out;
>  		}
>  		break;
>  	case S_IFCHR:

Not really related to 1/2 but ok.

Reviewed-by: Jeff Layton <jlayton@kernel.org>