NFSD: Add nfsd4_encode_fattr4_supported_attrs()
authorChuck Lever <chuck.lever@oracle.com>
Mon, 18 Sep 2023 13:57:32 +0000 (09:57 -0400)
committerChuck Lever <chuck.lever@oracle.com>
Mon, 16 Oct 2023 16:44:13 +0000 (12:44 -0400)
Refactor the encoder for FATTR4_SUPPORTED_ATTRS into a helper. In a
subsequent patch, this helper will be called from a bitmask loop.

Reviewed-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
fs/nfsd/nfs4xdr.c

index 3160628ae0e4312fd156385e8222c2eae8cdfadf..c8fbfc175dab0d9e12b9a535784583e1e1170036 100644 (file)
@@ -2939,7 +2939,9 @@ out_resource:
 }
 
 struct nfsd4_fattr_args {
+       struct svc_rqst         *rqstp;
        struct svc_fh           *fhp;
+       struct dentry           *dentry;
        struct kstat            stat;
        struct kstatfs          statfs;
        struct nfs4_acl         *acl;
@@ -2960,6 +2962,22 @@ static __be32 nfsd4_encode_fattr4__false(struct xdr_stream *xdr,
        return nfsd4_encode_bool(xdr, false);
 }
 
+static __be32 nfsd4_encode_fattr4_supported_attrs(struct xdr_stream *xdr,
+                                                 const struct nfsd4_fattr_args *args)
+{
+       struct nfsd4_compoundres *resp = args->rqstp->rq_resp;
+       u32 minorversion = resp->cstate.minorversion;
+       u32 supp[3];
+
+       memcpy(supp, nfsd_suppattrs[minorversion], sizeof(supp));
+       if (!IS_POSIXACL(d_inode(args->dentry)))
+               supp[0] &= ~FATTR4_WORD0_ACL;
+       if (!args->contextsupport)
+               supp[2] &= ~FATTR4_WORD2_SECURITY_LABEL;
+
+       return nfsd4_encode_bitmap4(xdr, supp[0], supp[1], supp[2]);
+}
+
 /*
  * Note: @fhp can be NULL; in this case, we might have to compose the filehandle
  * ourselves.
@@ -2999,6 +3017,9 @@ nfsd4_encode_fattr(struct xdr_stream *xdr, struct svc_fh *fhp,
        BUG_ON(bmval1 & NFSD_WRITEONLY_ATTRS_WORD1);
        BUG_ON(!nfsd_attrs_supported(minorversion, bmval));
 
+       args.rqstp = rqstp;
+       args.dentry = dentry;
+
        args.rdattr_err = 0;
        if (exp->ex_fslocs.migrated) {
                status = fattr_handle_absent_fs(&bmval0, &bmval1, &bmval2,
@@ -3087,30 +3108,9 @@ nfsd4_encode_fattr(struct xdr_stream *xdr, struct svc_fh *fhp,
                goto out_resource;
 
        if (bmval0 & FATTR4_WORD0_SUPPORTED_ATTRS) {
-               u32 supp[3];
-
-               memcpy(supp, nfsd_suppattrs[minorversion], sizeof(supp));
-
-               if (!IS_POSIXACL(dentry->d_inode))
-                       supp[0] &= ~FATTR4_WORD0_ACL;
-               if (!args.contextsupport)
-                       supp[2] &= ~FATTR4_WORD2_SECURITY_LABEL;
-               if (!supp[2]) {
-                       p = xdr_reserve_space(xdr, 12);
-                       if (!p)
-                               goto out_resource;
-                       *p++ = cpu_to_be32(2);
-                       *p++ = cpu_to_be32(supp[0]);
-                       *p++ = cpu_to_be32(supp[1]);
-               } else {
-                       p = xdr_reserve_space(xdr, 16);
-                       if (!p)
-                               goto out_resource;
-                       *p++ = cpu_to_be32(3);
-                       *p++ = cpu_to_be32(supp[0]);
-                       *p++ = cpu_to_be32(supp[1]);
-                       *p++ = cpu_to_be32(supp[2]);
-               }
+               status = nfsd4_encode_fattr4_supported_attrs(xdr, &args);
+               if (status != nfs_ok)
+                       goto out;
        }
        if (bmval0 & FATTR4_WORD0_TYPE) {
                p = xdr_reserve_space(xdr, 4);