diff options
| author | NeilBrown <neilb@suse.de> | 2019-11-28 03:56:43 +0100 |
|---|---|---|
| committer | J. Bruce Fields <bfields@redhat.com> | 2019-11-30 20:59:52 +0100 |
| commit | 466e16f0920f3ffdfa49713212fa334fb3dc08f1 (patch) | |
| tree | fce5a8414513e8b4c7587d7f8f711e278b8b89ee /fs/nfsd/vfs.c | |
| parent | nfsd: Ensure CLONE persists data and metadata changes to the target file (diff) | |
| download | linux-466e16f0920f3ffdfa49713212fa334fb3dc08f1.tar.xz linux-466e16f0920f3ffdfa49713212fa334fb3dc08f1.zip | |
nfsd: check for EBUSY from vfs_rmdir/vfs_unink.
vfs_rmdir and vfs_unlink can return -EBUSY if the
target is a mountpoint. This currently gets passed to
nfserrno() by nfsd_unlink(), and that results in a WARNing,
which is not user-friendly.
Possibly the best NFSv4 error is NFS4ERR_FILE_OPEN, because
there is a sense in which the object is currently in use
by some other task. The Linux NFSv4 client will map this
back to EBUSY, which is an added benefit.
For NFSv3, the best we can do is probably NFS3ERR_ACCES, which isn't
true, but is not less true than the other options.
Signed-off-by: NeilBrown <neilb@suse.de>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs/nfsd/vfs.c')
| -rw-r--r-- | fs/nfsd/vfs.c | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c index cf423fea0c6f..c0dc491537a6 100644 --- a/fs/nfsd/vfs.c +++ b/fs/nfsd/vfs.c @@ -1815,7 +1815,17 @@ nfsd_unlink(struct svc_rqst *rqstp, struct svc_fh *fhp, int type, out_drop_write: fh_drop_write(fhp); out_nfserr: - err = nfserrno(host_err); + if (host_err == -EBUSY) { + /* name is mounted-on. There is no perfect + * error status. + */ + if (nfsd_v4client(rqstp)) + err = nfserr_file_open; + else + err = nfserr_acces; + } else { + err = nfserrno(host_err); + } out: return err; } |