From nobody Fri Jun 17 21:52:31 2022 X-Original-To: dev-commits-src-main@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 224C183AA29; Fri, 17 Jun 2022 21:52:32 +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 4LPt7N0RsFz3GQl; Fri, 17 Jun 2022 21:52:32 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1655502752; 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=0/9SQCJc/R+yfJ+uuqyoZMTFcl9/MmlbLKaLrxNHj04=; b=qbHQ2c9HBH1Mub2G2eXR6uz8PUudrWF+xr3zS5CWQ9GVplXkWd742pdf8M60Np1ZnVeJ9c exRHAEJGgqt9fWDPAhEWB/dQr7O01lLbhVw4muOq+5SS/L2keFO+IKbwoab3Nx8NeAZ01t poppZzr12+uwVeEMuULWQLqIXZXtxlIBN6VS473JCf8bixAkLlYXnT5ESldZn9BlbSDA8b DIAD7E/kFuUtDlh3b8plTvKLVxzIWPz4qHZqX4os26iq4sWoA+IUPq6BzjatOFFKjhSKAx eg7h0ycMbzbZiY3dwGxb+zqBRThr0Lv1dNn4WnN2jRVP9bHqkVWh7M+MNAb7/g== 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 E397127ADC; Fri, 17 Jun 2022 21:52:31 +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 25HLqVja009636; Fri, 17 Jun 2022 21:52:31 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 25HLqVM1009635; Fri, 17 Jun 2022 21:52:31 GMT (envelope-from git) Date: Fri, 17 Jun 2022 21:52:31 GMT Message-Id: <202206172152.25HLqVM1009635@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: 1e70163c5084 - main - nfscl: Clean up the code by removing unused arguments List-Id: Commit messages for the main branch of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-main@freebsd.org X-BeenThere: dev-commits-src-main@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: 1e70163c50847f0b48659ab006b70b33e2fcd926 Auto-Submitted: auto-generated ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1655502752; 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=0/9SQCJc/R+yfJ+uuqyoZMTFcl9/MmlbLKaLrxNHj04=; b=A/7PNt3DQtGcMGsr66ZyjlCEgbLJSZQnNy+Hqov4HFs3rtZYzitQtiJ8RASryzWgR8mhtS HFr5FG9Ny2yPTL+qb/UBgDMUhfhx+PLjIm5nK+bRrDieucIbZqzF7IBjBmyqK6EB3cw7h3 XYkz0avEAmOF5Qhp1Pw7CnKEIACyQ3QoHPHPVdHpz6JvDRY+xb/FSD9NGOPtztwy4eUmwh Lv2ggja7fV++rzSC3d99s6uu0saAB40TiGjJKlX+FN0ozLmpBw+Stxp13mktAH8WfD75wa wB+yjj79X/pkiyGmZhjAt3uDDMi/wbh0HgGD739VwcLBK2Ol/9lundJXeoCUgg== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1655502752; a=rsa-sha256; cv=none; b=usxekAx/6eqSxLuj67VKHTeP+43Ih0w91xlylH1xINVH4Cu+Gf6LIRaAO6z858B73P4tKn VoV89WxFk/YgZCurcsuSpHudiE+ahM26sTdlGj7EK146xCeE5ejk6e2/TCg5v25Ts02vwY jFQRLiWBcdAz5X3+S5DoNyHA+iwY154va1UT005fiHh0tW01tTw2DUr7zUW+K+76bbcYFr 6r7+jwpuFigFA2eQThuEUcZNtiTc7DOQM3Jn0fWMMXTrfM/Rd1hrxbeUtrk+wWKW6sAFY8 +YMeg6oepFOTSX0WUT+/ZjHYc6KDViDDF0YQrNECuhR4oJcMtCqRbC/tBCJCjw== 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=1e70163c50847f0b48659ab006b70b33e2fcd926 commit 1e70163c50847f0b48659ab006b70b33e2fcd926 Author: Rick Macklem AuthorDate: 2022-06-17 21:51:11 +0000 Commit: Rick Macklem CommitDate: 2022-06-17 21:51:11 +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 defined in nfs_clrpcops.c and called in nfs_clvnops.c and nfs_clvfsops.c. Future commits will do the same for other functions. This commit should not result in a semantics change. --- sys/fs/nfs/nfs_var.h | 8 ++++---- sys/fs/nfsclient/nfs_clrpcops.c | 11 ++++------- sys/fs/nfsclient/nfs_clvfsops.c | 4 ++-- sys/fs/nfsclient/nfs_clvnops.c | 8 +++----- 4 files changed, 13 insertions(+), 18 deletions(-) diff --git a/sys/fs/nfs/nfs_var.h b/sys/fs/nfs/nfs_var.h index 990a017ce372..07848a963763 100644 --- a/sys/fs/nfs/nfs_var.h +++ b/sys/fs/nfs/nfs_var.h @@ -510,9 +510,9 @@ int nfsrpc_lock(struct nfsrv_descript *, struct nfsmount *, vnode_t, int nfsrpc_statfs(vnode_t, struct nfsstatfs *, struct nfsfsinfo *, struct ucred *, NFSPROC_T *, struct nfsvattr *, int *); int nfsrpc_fsinfo(vnode_t, struct nfsfsinfo *, struct ucred *, - NFSPROC_T *, struct nfsvattr *, int *, void *); + NFSPROC_T *, struct nfsvattr *, int *); int nfsrpc_pathconf(vnode_t, struct nfsv3_pathconf *, - struct ucred *, NFSPROC_T *, struct nfsvattr *, int *, void *); + struct ucred *, NFSPROC_T *, struct nfsvattr *, int *); int nfsrpc_renew(struct nfsclclient *, struct nfsclds *, struct ucred *, NFSPROC_T *); int nfsrpc_rellockown(struct nfsmount *, struct nfscllockowner *, uint8_t *, @@ -521,8 +521,8 @@ int nfsrpc_getdirpath(struct nfsmount *, u_char *, struct ucred *, NFSPROC_T *); int nfsrpc_delegreturn(struct nfscldeleg *, struct ucred *, struct nfsmount *, NFSPROC_T *, int); -int nfsrpc_getacl(vnode_t, struct ucred *, NFSPROC_T *, NFSACL_T *, void *); -int nfsrpc_setacl(vnode_t, struct ucred *, NFSPROC_T *, NFSACL_T *, void *); +int nfsrpc_getacl(vnode_t, struct ucred *, NFSPROC_T *, NFSACL_T *); +int nfsrpc_setacl(vnode_t, struct ucred *, NFSPROC_T *, NFSACL_T *); int nfsrpc_exchangeid(struct nfsmount *, struct nfsclclient *, struct nfssockreq *, int, uint32_t, struct nfsclds **, struct ucred *, NFSPROC_T *); diff --git a/sys/fs/nfsclient/nfs_clrpcops.c b/sys/fs/nfsclient/nfs_clrpcops.c index 347d81667e19..0f1e64d019bb 100644 --- a/sys/fs/nfsclient/nfs_clrpcops.c +++ b/sys/fs/nfsclient/nfs_clrpcops.c @@ -4698,8 +4698,7 @@ nfsmout: */ int nfsrpc_pathconf(vnode_t vp, struct nfsv3_pathconf *pc, - struct ucred *cred, NFSPROC_T *p, struct nfsvattr *nap, int *attrflagp, - void *stuff) + struct ucred *cred, NFSPROC_T *p, struct nfsvattr *nap, int *attrflagp) { struct nfsrv_descript nfsd, *nd = &nfsd; struct nfsmount *nmp; @@ -4759,7 +4758,7 @@ nfsmout: */ int nfsrpc_fsinfo(vnode_t vp, struct nfsfsinfo *fsp, struct ucred *cred, - NFSPROC_T *p, struct nfsvattr *nap, int *attrflagp, void *stuff) + NFSPROC_T *p, struct nfsvattr *nap, int *attrflagp) { u_int32_t *tl; struct nfsrv_descript nfsd, *nd = &nfsd; @@ -4992,8 +4991,7 @@ nfsrpc_delegreturn(struct nfscldeleg *dp, struct ucred *cred, * nfs getacl call. */ int -nfsrpc_getacl(vnode_t vp, struct ucred *cred, NFSPROC_T *p, - struct acl *aclp, void *stuff) +nfsrpc_getacl(vnode_t vp, struct ucred *cred, NFSPROC_T *p, struct acl *aclp) { struct nfsrv_descript nfsd, *nd = &nfsd; int error; @@ -5022,8 +5020,7 @@ nfsrpc_getacl(vnode_t vp, struct ucred *cred, NFSPROC_T *p, * nfs setacl call. */ int -nfsrpc_setacl(vnode_t vp, struct ucred *cred, NFSPROC_T *p, - struct acl *aclp, void *stuff) +nfsrpc_setacl(vnode_t vp, struct ucred *cred, NFSPROC_T *p, struct acl *aclp) { int error; struct nfsmount *nmp = VFSTONFS(vp->v_mount); diff --git a/sys/fs/nfsclient/nfs_clvfsops.c b/sys/fs/nfsclient/nfs_clvfsops.c index 343272db31fa..8dc7036242e1 100644 --- a/sys/fs/nfsclient/nfs_clvfsops.c +++ b/sys/fs/nfsclient/nfs_clvfsops.c @@ -309,7 +309,7 @@ nfs_statfs(struct mount *mp, struct statfs *sbp) if (NFSHASNFSV3(nmp) && !NFSHASGOTFSINFO(nmp)) { mtx_unlock(&nmp->nm_mtx); error = nfsrpc_fsinfo(vp, &fs, td->td_ucred, td, &nfsva, - &attrflag, NULL); + &attrflag); if (!error) gotfsinfo = 1; } else @@ -370,7 +370,7 @@ ncl_fsinfo(struct nfsmount *nmp, struct vnode *vp, struct ucred *cred, struct nfsvattr nfsva; int error, attrflag; - error = nfsrpc_fsinfo(vp, &fs, cred, td, &nfsva, &attrflag, NULL); + error = nfsrpc_fsinfo(vp, &fs, cred, td, &nfsva, &attrflag); if (!error) { if (attrflag) (void) nfscl_loadattrcache(&vp, &nfsva, NULL, 0, 1); diff --git a/sys/fs/nfsclient/nfs_clvnops.c b/sys/fs/nfsclient/nfs_clvnops.c index 2e3cab0799ec..36037c021e4e 100644 --- a/sys/fs/nfsclient/nfs_clvnops.c +++ b/sys/fs/nfsclient/nfs_clvnops.c @@ -3655,8 +3655,7 @@ nfs_getacl(struct vop_getacl_args *ap) if (ap->a_type != ACL_TYPE_NFS4) return (EOPNOTSUPP); - error = nfsrpc_getacl(ap->a_vp, ap->a_cred, ap->a_td, ap->a_aclp, - NULL); + error = nfsrpc_getacl(ap->a_vp, ap->a_cred, ap->a_td, ap->a_aclp); if (error > NFSERR_STALE) { (void) nfscl_maperr(ap->a_td, error, (uid_t)0, (gid_t)0); error = EPERM; @@ -3671,8 +3670,7 @@ nfs_setacl(struct vop_setacl_args *ap) if (ap->a_type != ACL_TYPE_NFS4) return (EOPNOTSUPP); - error = nfsrpc_setacl(ap->a_vp, ap->a_cred, ap->a_td, ap->a_aclp, - NULL); + error = nfsrpc_setacl(ap->a_vp, ap->a_cred, ap->a_td, ap->a_aclp); if (error > NFSERR_STALE) { (void) nfscl_maperr(ap->a_td, error, (uid_t)0, (gid_t)0); error = EPERM; @@ -4427,7 +4425,7 @@ nfs_pathconf(struct vop_pathconf_args *ap) * be used for _PC_NFS4_ACL as well. */ error = nfsrpc_pathconf(vp, &pc, td->td_ucred, td, &nfsva, - &attrflag, NULL); + &attrflag); if (attrflag != 0) (void) nfscl_loadattrcache(&vp, &nfsva, NULL, 0, 1); if (error != 0)