git: 51a9b978e750 - main - nfs server: improve use of the VFS KPI

[Mark de Groot]

unsubscribe
________________________________
Van: owner-dev-commits-src-all at freebsd.org <owner-dev-commits-src-all at freebsd.org> namens Konstantin Belousov <kib at FreeBSD.org>
Verzonden: zaterdag 2 januari 2021 19:18
Aan: src-committers at FreeBSD.org <src-committers at FreeBSD.org>; dev-commits-src-all at FreeBSD.org <dev-commits-src-all at FreeBSD.org>; dev-commits-src-main at FreeBSD.org <dev-commits-src-main at FreeBSD.org>
Onderwerp: git: 51a9b978e750 - main - nfs server: improve use of the VFS KPI

The branch main has been updated by kib:

URL: https://cgit.FreeBSD.org/src/commit/?id=51a9b978e75021415fdced616b4e4bc373a20a8a

commit 51a9b978e75021415fdced616b4e4bc373a20a8a
Author:     Konstantin Belousov <kib at FreeBSD.org>
AuthorDate: 2021-01-01 15:35:44 +0000
Commit:     Konstantin Belousov <kib at FreeBSD.org>
CommitDate: 2021-01-02 18:17:12 +0000

    nfs server: improve use of the VFS KPI

    In particular, do not assume that vn_start_write() returns the same mp
    as it was passed in, or never returns error.

    Also be more accurate to return NULL vp and mp when error occured, to
    catch wrong control flow easier.

    Stop checking for NULL mp before calling vn_finished_write(), NULL mp
    is handled transparently by the function.

    Reviewed by:    rmacklem
    Tested by:      pho
    MFC after:      1 week
    Sponsored by:   The FreeBSD Foundation
    Differential Revision:  https://reviews.freebsd.org/D27881
---
 sys/fs/nfsserver/nfs_nfsdport.c   | 33 ++++++++++++++++++++-------------
 sys/fs/nfsserver/nfs_nfsdsocket.c |  6 ++----
 2 files changed, 22 insertions(+), 17 deletions(-)

diff --git a/sys/fs/nfsserver/nfs_nfsdport.c b/sys/fs/nfsserver/nfs_nfsdport.c
index e9a9443dc08c..727a83005fa0 100644
--- a/sys/fs/nfsserver/nfs_nfsdport.c
+++ b/sys/fs/nfsserver/nfs_nfsdport.c
@@ -3243,28 +3243,35 @@ nfsd_fhtovp(struct nfsrv_descript *nd, struct nfsrvfh *nfp, int lktype,
     struct vnode **vpp, struct nfsexstuff *exp,
     struct mount **mpp, int startwrite)
 {
-       struct mount *mp;
+       struct mount *mp, *mpw;
         struct ucred *credanon;
         fhandle_t *fhp;
+       int error;

+       if (mpp != NULL)
+               *mpp = NULL;
+       *vpp = NULL;
         fhp = (fhandle_t *)nfp->nfsrvfh_data;
-       /*
-        * Check for the special case of the nfsv4root_fh.
-        */
         mp = vfs_busyfs(&fhp->fh_fsid);
-       if (mpp != NULL)
-               *mpp = mp;
         if (mp == NULL) {
-               *vpp = NULL;
                 nd->nd_repstat = ESTALE;
                 goto out;
         }

         if (startwrite) {
-               vn_start_write(NULL, mpp, V_WAIT);
+               mpw = mp;
+               error = vn_start_write(NULL, &mpw, V_WAIT);
+               if (error != 0) {
+                       mpw = NULL;
+                       vfs_unbusy(mp);
+                       nd->nd_repstat = ESTALE;
+                       goto out;
+               }
                 if (lktype == LK_SHARED && !(MNT_SHARED_WRITES(mp)))
                         lktype = LK_EXCLUSIVE;
-       }
+       } else
+               mpw = NULL;
+
         nd->nd_repstat = nfsvno_fhtovp(mp, fhp, nd->nd_nam, lktype, vpp, exp,
             &credanon);
         vfs_unbusy(mp);
@@ -3276,6 +3283,7 @@ nfsd_fhtovp(struct nfsrv_descript *nd, struct nfsrvfh *nfp, int lktype,
         if (!nd->nd_repstat && exp->nes_exflag == 0 &&
             !(nd->nd_flag & ND_NFSV4)) {
                 vput(*vpp);
+               *vpp = NULL;
                 nd->nd_repstat = EACCES;
         }

@@ -3336,11 +3344,10 @@ nfsd_fhtovp(struct nfsrv_descript *nd, struct nfsrvfh *nfp, int lktype,
         if (credanon != NULL)
                 crfree(credanon);
         if (nd->nd_repstat) {
-               if (startwrite)
-                       vn_finished_write(mp);
+               vn_finished_write(mpw);
                 *vpp = NULL;
-               if (mpp != NULL)
-                       *mpp = NULL;
+       } else if (mpp != NULL) {
+               *mpp = mpw;
         }

 out:
diff --git a/sys/fs/nfsserver/nfs_nfsdsocket.c b/sys/fs/nfsserver/nfs_nfsdsocket.c
index 530ebb8a8cc8..e9602c352420 100644
--- a/sys/fs/nfsserver/nfs_nfsdsocket.c
+++ b/sys/fs/nfsserver/nfs_nfsdsocket.c
@@ -612,8 +612,7 @@ tryagain:
                 nfsrvd_statstart(nfsv3to4op[nd->nd_procnum], /*now*/ NULL);
                 nfsrvd_statend(nfsv3to4op[nd->nd_procnum], /*bytes*/ 0,
                    /*now*/ NULL, /*then*/ NULL);
-               if (mp != NULL && nfsrv_writerpc[nd->nd_procnum] != 0)
-                       vn_finished_write(mp);
+               vn_finished_write(mp);
                 goto out;
         }

@@ -643,8 +642,7 @@ tryagain:
                         error = (*(nfsrv3_procs0[nd->nd_procnum]))(nd, isdgram,
                             vp, &nes);
                 }
-               if (mp != NULL && nfsrv_writerpc[nd->nd_procnum] != 0)
-                       vn_finished_write(mp);
+               vn_finished_write(mp);

                 if (error == 0 && nd->nd_repstat == ERELOOKUP) {
                         /*
_______________________________________________
dev-commits-src-all at freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/dev-commits-src-all
To unsubscribe, send any mail to "dev-commits-src-all-unsubscribe at freebsd.org"