Author: rmacklem
Date: Wed May  7 22:33:36 2014
New Revision: 265621
URL: http://svnweb.freebsd.org/changeset/base/265621

Log:
  MFC: r264845
  Remove an unnecessary level of indirection for an argument.
  This simplifies the code and should avoid the clang sparc
  port from generating an abort() call.

Modified:
  stable/10/sys/fs/nfsserver/nfs_nfsdstate.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/fs/nfsserver/nfs_nfsdstate.c
==============================================================================
--- stable/10/sys/fs/nfsserver/nfs_nfsdstate.c  Wed May  7 22:27:03 2014        
(r265620)
+++ stable/10/sys/fs/nfsserver/nfs_nfsdstate.c  Wed May  7 22:33:36 2014        
(r265621)
@@ -79,7 +79,7 @@ static int nfsrv_getstate(struct nfsclie
 static void nfsrv_getowner(struct nfsstatehead *hp, struct nfsstate *new_stp,
     struct nfsstate **stpp);
 static int nfsrv_getlockfh(vnode_t vp, u_short flags,
-    struct nfslockfile **new_lfpp, fhandle_t *nfhp, NFSPROC_T *p);
+    struct nfslockfile *new_lfp, fhandle_t *nfhp, NFSPROC_T *p);
 static int nfsrv_getlockfile(u_short flags, struct nfslockfile **new_lfpp,
     struct nfslockfile **lfpp, fhandle_t *nfhp, int lockit);
 static void nfsrv_insertlock(struct nfslock *new_lop,
@@ -1985,7 +1985,7 @@ tryagain:
        MALLOC(new_lfp, struct nfslockfile *, sizeof (struct nfslockfile),
            M_NFSDLOCKFILE, M_WAITOK);
        if (vp)
-               getfhret = nfsrv_getlockfh(vp, new_stp->ls_flags, &new_lfp,
+               getfhret = nfsrv_getlockfh(vp, new_stp->ls_flags, new_lfp,
                    NULL, p);
        NFSLOCKSTATE();
        /*
@@ -2235,7 +2235,7 @@ tryagain:
            M_NFSDSTATE, M_WAITOK);
        MALLOC(new_deleg, struct nfsstate *, sizeof (struct nfsstate),
            M_NFSDSTATE, M_WAITOK);
-       getfhret = nfsrv_getlockfh(vp, new_stp->ls_flags, &new_lfp,
+       getfhret = nfsrv_getlockfh(vp, new_stp->ls_flags, new_lfp,
            NULL, p);
        NFSLOCKSTATE();
        /*
@@ -3142,11 +3142,10 @@ out:
  * Get the file handle for a lock structure.
  */
 static int
-nfsrv_getlockfh(vnode_t vp, u_short flags,
-    struct nfslockfile **new_lfpp, fhandle_t *nfhp, NFSPROC_T *p)
+nfsrv_getlockfh(vnode_t vp, u_short flags, struct nfslockfile *new_lfp,
+    fhandle_t *nfhp, NFSPROC_T *p)
 {
        fhandle_t *fhp = NULL;
-       struct nfslockfile *new_lfp;
        int error;
 
        /*
@@ -3154,7 +3153,7 @@ nfsrv_getlockfh(vnode_t vp, u_short flag
         * a fhandle_t on the stack.
         */
        if (flags & NFSLCK_OPEN) {
-               new_lfp = *new_lfpp;
+               KASSERT(new_lfp != NULL, ("nfsrv_getlockfh: new_lfp NULL"));
                fhp = &new_lfp->lf_fh;
        } else if (nfhp) {
                fhp = nfhp;
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to