nfsd: switch purge_old() to use start_removing_noperm()
Rather than explicit locking, use the start_removing_noperm() and end_removing() wrappers. This was not done with other start_removing changes due to conflicting in-flight patches. Reviewed-by: Jeff Layton <jlayton@kernel.org> Signed-off-by: NeilBrown <neil@brown.name> Link: https://patch.msgid.link/20260224222542.3458677-8-neilb@ownmail.net Signed-off-by: Christian Brauner <brauner@kernel.org>master
parent
c4573e18e2
commit
6cb3411962
|
|
@ -352,16 +352,14 @@ purge_old(struct dentry *parent, char *cname, struct nfsd_net *nn)
|
|||
if (nfs4_has_reclaimed_state(name, nn))
|
||||
goto out_free;
|
||||
|
||||
inode_lock_nested(d_inode(parent), I_MUTEX_PARENT);
|
||||
child = lookup_one(&nop_mnt_idmap, &QSTR(cname), parent);
|
||||
child = start_removing_noperm(parent, &QSTR(cname));
|
||||
if (!IS_ERR(child)) {
|
||||
status = vfs_rmdir(&nop_mnt_idmap, d_inode(parent), child, NULL);
|
||||
if (status)
|
||||
printk("failed to remove client recovery directory %pd\n",
|
||||
child);
|
||||
dput(child);
|
||||
}
|
||||
inode_unlock(d_inode(parent));
|
||||
end_removing(child);
|
||||
|
||||
out_free:
|
||||
kfree(name.data);
|
||||
|
|
|
|||
Loading…
Reference in New Issue