From nobody Sat Apr 16 01:55:02 2022 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id E5B505D2303; Sat, 16 Apr 2022 01:55:02 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4KgGVG67Tzz4hkK; Sat, 16 Apr 2022 01:55:02 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1650074102; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=PSc7Z92yqqrvWSQlL3Wx0rVfPT9owMgx4SqI8StxDNs=; b=BVXY3t8xn0rFeIRXOgNwHcWvpCFD9Vhj3qyvtbn60i/nKxP5Zn+gRUGy8J49aPa1eGa5nt P5Gwy8Fx+WQJ1ivOMZjDjaAE3mOF2aQDQLwC4EkQYh+fRjL179tE0oo02KbIV/qZIwmVxB 17oE6qXw/wnPmCtBYEYngW54H97VfehyO4BRoW64jffh4iKRyryrAqu3Ls82sU2A8qTuX4 hACsSUjjs1sao0o/bh0TDeD+T5DUkEk3wKv2P5kJm7M9frdbJZCmQ8si/Jd76vaJRelObn RarQLaXp7DEUaaHgpg6oPsjh8IfMpkjKXZuyne3sSWeYXF4Nh8Wk1TrJ79iIFA== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id B2F0B206FE; Sat, 16 Apr 2022 01:55:02 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.16.1/8.16.1) with ESMTP id 23G1t2kU097082; Sat, 16 Apr 2022 01:55:02 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 23G1t2K4097081; Sat, 16 Apr 2022 01:55:02 GMT (envelope-from git) Date: Sat, 16 Apr 2022 01:55:02 GMT Message-Id: <202204160155.23G1t2K4097081@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org From: Rick Macklem Subject: git: 32c3e0f04921 - main - nfscl: Clean up the code by removing unused arguments List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: rmacklem X-Git-Repository: src X-Git-Refname: refs/heads/main X-Git-Reftype: branch X-Git-Commit: 32c3e0f04921f1641c431433cf00ab2a1d9c100e Auto-Submitted: auto-generated ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1650074102; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=PSc7Z92yqqrvWSQlL3Wx0rVfPT9owMgx4SqI8StxDNs=; b=jODePfpeDL8ULDvAKughd774nmgFNrTl85GkepVjUyxdJ7rOniPCniYZBA0ORGmRAlhjTZ OqxOTn7LNs+vyH5IhReXo0q4Wk9wf4qvF+nvUtnXDfa5nrjzpyQdrm2UlFmvVwIQIYMWhr lXa96F9YIcPsTgm2ujJ5Vngyg0pHoXEb5tR1vJRonr0FHvVUiGIz4oVTc5fXV/20n4TvOs BT3Ca3b5rjljP79CItqaiumyy9kR4T8NGN/pcZSZHe9kBVjfb7ErwdP6XVuLEL3TepfFS+ kh4lx0XKjQ+i+LEo8T2HBkQ+uRZSJE8yFSH24xXy9+IcvAJigEZ2Z71IV0Lq3w== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1650074102; a=rsa-sha256; cv=none; b=kOHn4w/ujXSR2p5aPW2tWoQ/sv37626uVztPzr2SQBBX7uSkSYm+0DPTVLSUQ6aUBTq8f/ bFeCNJfFWwcKQQNmvbzkry1668cfQ1PVEhqxK1T8Ry+jVHM9GknrMWwutyc0PmR2cvJC4+ aiUNwE3f/67TBuEUhqk33nAIlSJ8+/eNj5U3iPENVG/+HDdbC7T6kXyH9VrX6P2xB/w05L 7FuxjtkEjDJjdDYvgj9w8QM2wIHYHTtyMVBiJPekPzdxJZeEWk4J9ZOMoeiUnjCBK8s4+1 wfW32+0BL5QUbFfPx6gfr9vBQMH2Per7n4HAEV5ebtkrbDqnEvHHiIgAPAny0A== ARC-Authentication-Results: i=1; mx1.freebsd.org; none X-ThisMailContainsUnwantedMimeParts: N The branch main has been updated by rmacklem: URL: https://cgit.FreeBSD.org/src/commit/?id=32c3e0f04921f1641c431433cf00ab2a1d9c100e commit 32c3e0f04921f1641c431433cf00ab2a1d9c100e Author: Rick Macklem AuthorDate: 2022-04-16 01:51:01 +0000 Commit: Rick Macklem CommitDate: 2022-04-16 01:51:01 +0000 nfscl: Clean up the code by removing unused arguments The "void *stuff" (also called fstuff and dstuff) argument was used by the Mac OSX port. For FreeBSD, this argument is always NULL, so remove it to clean up the code. This commit gets rid of "stuff" for assorted functions local to nfs_clrpcops.c. Future commits will do the same for other functions. --- sys/fs/nfsclient/nfs_clrpcops.c | 57 +++++++++++++++++++---------------------- 1 file changed, 27 insertions(+), 30 deletions(-) diff --git a/sys/fs/nfsclient/nfs_clrpcops.c b/sys/fs/nfsclient/nfs_clrpcops.c index ae6c5ca77775..48cd6467c78f 100644 --- a/sys/fs/nfsclient/nfs_clrpcops.c +++ b/sys/fs/nfsclient/nfs_clrpcops.c @@ -129,26 +129,26 @@ struct nfsclwritedsdorpc { }; static int nfsrpc_setattrrpc(vnode_t , struct vattr *, nfsv4stateid_t *, - struct ucred *, NFSPROC_T *, struct nfsvattr *, int *, void *); + struct ucred *, NFSPROC_T *, struct nfsvattr *, int *); static int nfsrpc_readrpc(vnode_t , struct uio *, struct ucred *, - nfsv4stateid_t *, NFSPROC_T *, struct nfsvattr *, int *, void *); + nfsv4stateid_t *, NFSPROC_T *, struct nfsvattr *, int *); static int nfsrpc_writerpc(vnode_t , struct uio *, int *, int *, struct ucred *, nfsv4stateid_t *, NFSPROC_T *, struct nfsvattr *, int *, void *); static int nfsrpc_deallocaterpc(vnode_t, off_t, off_t, nfsv4stateid_t *, - struct nfsvattr *, int *, struct ucred *, NFSPROC_T *, void *); + struct nfsvattr *, int *, struct ucred *, NFSPROC_T *); static int nfsrpc_createv23(vnode_t , char *, int, struct vattr *, nfsquad_t, int, struct ucred *, NFSPROC_T *, struct nfsvattr *, - struct nfsvattr *, struct nfsfh **, int *, int *, void *); + struct nfsvattr *, struct nfsfh **, int *, int *); static int nfsrpc_createv4(vnode_t , char *, int, struct vattr *, nfsquad_t, int, struct nfsclowner *, struct nfscldeleg **, struct ucred *, NFSPROC_T *, struct nfsvattr *, struct nfsvattr *, struct nfsfh **, int *, - int *, void *, int *); + int *, int *); static int nfsrpc_locku(struct nfsrv_descript *, struct nfsmount *, struct nfscllockowner *, u_int64_t, u_int64_t, u_int32_t, struct ucred *, NFSPROC_T *, int); static int nfsrpc_setaclrpc(vnode_t, struct ucred *, NFSPROC_T *, - struct acl *, nfsv4stateid_t *, void *); + struct acl *, nfsv4stateid_t *); static int nfsrpc_layouterror(struct nfsmount *, uint8_t *, int, uint64_t, uint64_t, nfsv4stateid_t *, struct ucred *, NFSPROC_T *, uint32_t, uint32_t, char *); @@ -194,7 +194,7 @@ static int nfsrpc_adviseds(vnode_t, uint64_t, int, int, struct nfsclds *, struct nfsfh *, int, int, struct ucred *, NFSPROC_T *); #endif static int nfsrpc_allocaterpc(vnode_t, off_t, off_t, nfsv4stateid_t *, - struct nfsvattr *, int *, struct ucred *, NFSPROC_T *, void *); + struct nfsvattr *, int *, struct ucred *, NFSPROC_T *); static void nfsrv_setuplayoutget(struct nfsrv_descript *, int, uint64_t, uint64_t, uint64_t, nfsv4stateid_t *, int, int, int); static int nfsrv_parseug(struct nfsrv_descript *, int, uid_t *, gid_t *, @@ -207,7 +207,7 @@ static int nfsrpc_getopenlayout(struct nfsmount *, vnode_t, u_int8_t *, static int nfsrpc_getcreatelayout(vnode_t, char *, int, struct vattr *, nfsquad_t, int, struct nfsclowner *, struct nfscldeleg **, struct ucred *, NFSPROC_T *, struct nfsvattr *, struct nfsvattr *, - struct nfsfh **, int *, int *, void *, int *); + struct nfsfh **, int *, int *, int *); static int nfsrpc_openlayoutrpc(struct nfsmount *, vnode_t, u_int8_t *, int, uint8_t *, int, uint32_t, struct nfsclopen *, uint8_t *, int, struct nfscldeleg **, nfsv4stateid_t *, int, int, int, int *, @@ -215,7 +215,7 @@ static int nfsrpc_openlayoutrpc(struct nfsmount *, vnode_t, u_int8_t *, static int nfsrpc_createlayout(vnode_t, char *, int, struct vattr *, nfsquad_t, int, struct nfsclowner *, struct nfscldeleg **, struct ucred *, NFSPROC_T *, struct nfsvattr *, struct nfsvattr *, - struct nfsfh **, int *, int *, void *, int *, nfsv4stateid_t *, + struct nfsfh **, int *, int *, int *, nfsv4stateid_t *, int, int, int, int *, struct nfsclflayouthead *, int *); static int nfsrpc_layoutget(struct nfsmount *, uint8_t *, int, int, uint64_t, uint64_t, uint64_t, int, int, nfsv4stateid_t *, int *, @@ -1312,10 +1312,9 @@ nfsrpc_setattr(vnode_t vp, struct vattr *vap, NFSACL_T *aclp, } if (vap != NULL) error = nfsrpc_setattrrpc(vp, vap, &stateid, cred, p, - rnap, attrflagp, stuff); + rnap, attrflagp); else - error = nfsrpc_setaclrpc(vp, cred, p, aclp, &stateid, - stuff); + error = nfsrpc_setaclrpc(vp, cred, p, aclp, &stateid); if (error == NFSERR_OPENMODE && mode == NFSV4OPEN_ACCESSREAD) { NFSLOCKMNT(nmp); nmp->nm_state |= NFSSTA_OPENMODE; @@ -1352,7 +1351,7 @@ nfsrpc_setattr(vnode_t vp, struct vattr *vap, NFSACL_T *aclp, static int nfsrpc_setattrrpc(vnode_t vp, struct vattr *vap, nfsv4stateid_t *stateidp, struct ucred *cred, NFSPROC_T *p, - struct nfsvattr *rnap, int *attrflagp, void *stuff) + struct nfsvattr *rnap, int *attrflagp) { u_int32_t *tl; struct nfsrv_descript nfsd, *nd = &nfsd; @@ -1717,7 +1716,7 @@ nfsrpc_read(vnode_t vp, struct uio *uiop, struct ucred *cred, NFSV4OPEN_ACCESSREAD, 0, newcred, p, &stateid, &lckp); error = nfsrpc_readrpc(vp, uiop, newcred, &stateid, p, nap, - attrflagp, stuff); + attrflagp); if (error == NFSERR_OPENMODE) { NFSLOCKMNT(nmp); nmp->nm_state |= NFSSTA_OPENMODE; @@ -1756,7 +1755,7 @@ nfsrpc_read(vnode_t vp, struct uio *uiop, struct ucred *cred, static int nfsrpc_readrpc(vnode_t vp, struct uio *uiop, struct ucred *cred, nfsv4stateid_t *stateidp, NFSPROC_T *p, struct nfsvattr *nap, - int *attrflagp, void *stuff) + int *attrflagp) { u_int32_t *tl; int error = 0, len, retlen, tsiz, eof = 0; @@ -2142,7 +2141,7 @@ nfsrpc_deallocate(vnode_t vp, off_t offs, off_t len, struct nfsvattr *nap, cred, p, &stateid, &lckp); } error = nfsrpc_deallocaterpc(vp, offs, len, &stateid, nap, - attrflagp, cred, p, stuff); + attrflagp, cred, p); if (error == NFSERR_STALESTATEID) nfscl_initiate_recovery(nmp->nm_clp); if (lckp != NULL) @@ -2175,7 +2174,7 @@ nfsrpc_deallocate(vnode_t vp, off_t offs, off_t len, struct nfsvattr *nap, static int nfsrpc_deallocaterpc(vnode_t vp, off_t offs, off_t len, nfsv4stateid_t *stateidp, struct nfsvattr *nap, int *attrflagp, - struct ucred *cred, NFSPROC_T *p, void *stuff) + struct ucred *cred, NFSPROC_T *p) { uint32_t *tl; struct nfsnode *np = VTONFS(vp); @@ -2331,11 +2330,11 @@ nfsrpc_create(vnode_t dvp, char *name, int namelen, struct vattr *vap, nfs_numnfscbd == 0 || retrycnt > 0) error = nfsrpc_createv4(dvp, name, namelen, vap, cverf, fmode, owp, &dp, cred, p, dnap, nnap, nfhpp, - attrflagp, dattrflagp, dstuff, &unlocked); + attrflagp, dattrflagp, &unlocked); else error = nfsrpc_getcreatelayout(dvp, name, namelen, vap, cverf, fmode, owp, &dp, cred, p, dnap, nnap, nfhpp, - attrflagp, dattrflagp, dstuff, &unlocked); + attrflagp, dattrflagp, &unlocked); /* * There is no need to invalidate cached attributes here, * since new post-delegation issue attributes are always @@ -2364,8 +2363,7 @@ nfsrpc_create(vnode_t dvp, char *name, int namelen, struct vattr *vap, error = EIO; } else { error = nfsrpc_createv23(dvp, name, namelen, vap, cverf, - fmode, cred, p, dnap, nnap, nfhpp, attrflagp, dattrflagp, - dstuff); + fmode, cred, p, dnap, nnap, nfhpp, attrflagp, dattrflagp); } return (error); } @@ -2377,7 +2375,7 @@ static int nfsrpc_createv23(vnode_t dvp, char *name, int namelen, struct vattr *vap, nfsquad_t cverf, int fmode, struct ucred *cred, NFSPROC_T *p, struct nfsvattr *dnap, struct nfsvattr *nnap, struct nfsfh **nfhpp, - int *attrflagp, int *dattrflagp, void *dstuff) + int *attrflagp, int *dattrflagp) { u_int32_t *tl; int error = 0; @@ -2426,7 +2424,7 @@ nfsrpc_createv4(vnode_t dvp, char *name, int namelen, struct vattr *vap, nfsquad_t cverf, int fmode, struct nfsclowner *owp, struct nfscldeleg **dpp, struct ucred *cred, NFSPROC_T *p, struct nfsvattr *dnap, struct nfsvattr *nnap, struct nfsfh **nfhpp, int *attrflagp, - int *dattrflagp, void *dstuff, int *unlockedp) + int *dattrflagp, int *unlockedp) { u_int32_t *tl; int error = 0, deleg, newone, ret, acesize, limitby; @@ -5000,7 +4998,7 @@ nfsrpc_setacl(vnode_t vp, struct ucred *cred, NFSPROC_T *p, */ static int nfsrpc_setaclrpc(vnode_t vp, struct ucred *cred, NFSPROC_T *p, - struct acl *aclp, nfsv4stateid_t *stateidp, void *stuff) + struct acl *aclp, nfsv4stateid_t *stateidp) { struct nfsrv_descript nfsd, *nd = &nfsd; int error; @@ -7381,7 +7379,7 @@ nfsrpc_allocate(vnode_t vp, off_t off, off_t len, struct nfsvattr *nap, error = EIO; else error = nfsrpc_allocaterpc(vp, off, len, &stateid, - nap, attrflagp, cred, p, stuff); + nap, attrflagp, cred, p); if (error == NFSERR_STALESTATEID) nfscl_initiate_recovery(nmp->nm_clp); if (lckp != NULL) @@ -7411,8 +7409,7 @@ nfsrpc_allocate(vnode_t vp, off_t off, off_t len, struct nfsvattr *nap, */ static int nfsrpc_allocaterpc(vnode_t vp, off_t off, off_t len, nfsv4stateid_t *stateidp, - struct nfsvattr *nap, int *attrflagp, struct ucred *cred, NFSPROC_T *p, - void *stuff) + struct nfsvattr *nap, int *attrflagp, struct ucred *cred, NFSPROC_T *p) { uint32_t *tl; int error; @@ -8068,7 +8065,7 @@ nfsrpc_createlayout(vnode_t dvp, char *name, int namelen, struct vattr *vap, nfsquad_t cverf, int fmode, struct nfsclowner *owp, struct nfscldeleg **dpp, struct ucred *cred, NFSPROC_T *p, struct nfsvattr *dnap, struct nfsvattr *nnap, struct nfsfh **nfhpp, int *attrflagp, - int *dattrflagp, void *dstuff, int *unlockedp, nfsv4stateid_t *stateidp, + int *dattrflagp, int *unlockedp, nfsv4stateid_t *stateidp, int usecurstateid, int layouttype, int layoutlen, int *retonclosep, struct nfsclflayouthead *flhp, int *laystatp) { @@ -8326,7 +8323,7 @@ nfsrpc_getcreatelayout(vnode_t dvp, char *name, int namelen, struct vattr *vap, nfsquad_t cverf, int fmode, struct nfsclowner *owp, struct nfscldeleg **dpp, struct ucred *cred, NFSPROC_T *p, struct nfsvattr *dnap, struct nfsvattr *nnap, struct nfsfh **nfhpp, int *attrflagp, - int *dattrflagp, void *dstuff, int *unlockedp) + int *dattrflagp, int *unlockedp) { struct nfscllayout *lyp; struct nfsclflayouthead flh; @@ -8347,7 +8344,7 @@ nfsrpc_getcreatelayout(vnode_t dvp, char *name, int namelen, struct vattr *vap, layoutlen = tsep->nfsess_maxcache - (NFSX_STATEID + 3 * NFSX_UNSIGNED); error = nfsrpc_createlayout(dvp, name, namelen, vap, cverf, fmode, owp, dpp, cred, p, dnap, nnap, nfhpp, attrflagp, dattrflagp, - dstuff, unlockedp, &stateid, 1, layouttype, layoutlen, &retonclose, + unlockedp, &stateid, 1, layouttype, layoutlen, &retonclose, &flh, &laystat); NFSCL_DEBUG(4, "aft nfsrpc_createlayoutrpc laystat=%d err=%d\n", laystat, error);