NFSD: introduce struct nfsd_attrs

[ Upstream commit 7fe2a71dda349a1afa75781f0cc7975be9784d15 ]

The attributes that nfsd might want to set on a file include 'struct
iattr' as well as an ACL and security label.
The latter two are passed around quite separately from the first, in
part because they are only needed for NFSv4.  This leads to some
clumsiness in the code, such as the attributes NOT being set in
nfsd_create_setattr().

We need to keep the directory locked until all attributes are set to
ensure the file is never visibile without all its attributes.  This need
combined with the inconsistent handling of attributes leads to more
clumsiness.

As a first step towards tidying this up, introduce 'struct nfsd_attrs'.
This is passed (by reference) to vfs.c functions that work with
attributes, and is assembled by the various nfs*proc functions which
call them.  As yet only iattr is included, but future patches will
expand this.

Signed-off-by: NeilBrown <neilb@suse.de>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>

diff --git a/fs/nfsd/nfs3proc.c b/fs/nfsd/nfs3proc.c
index 57854ca022d185a129bb77e818ad31900dee9f7c..113567b3a98a517f926ba9eb7a97b6072a2fa938 100644 (file)
--- a/fs/nfsd/nfs3proc.c
+++ b/fs/nfsd/nfs3proc.c
@@ -67,12 +67,15 @@ nfsd3_proc_setattr(struct svc_rqst *rqstp)
 {
        struct nfsd3_sattrargs *argp = rqstp->rq_argp;
        struct nfsd3_attrstat *resp = rqstp->rq_resp;
+       struct nfsd_attrs attrs = {
+               .na_iattr       = &argp->attrs,
+       };
 
        dprintk("nfsd: SETATTR(3)  %s\n",
                                SVCFH_fmt(&argp->fh));
 
        fh_copy(&resp->fh, &argp->fh);
-       resp->status = nfsd_setattr(rqstp, &resp->fh, &argp->attrs,
+       resp->status = nfsd_setattr(rqstp, &resp->fh, &attrs,
                                    argp->check_guard, argp->guardtime);
        return rpc_success;
 }
@@ -233,6 +236,9 @@ nfsd3_create_file(struct svc_rqst *rqstp, struct svc_fh *fhp,
 {
        struct iattr *iap = &argp->attrs;
        struct dentry *parent, *child;
+       struct nfsd_attrs attrs = {
+               .na_iattr       = iap,
+       };
        __u32 v_mtime, v_atime;
        struct inode *inode;
        __be32 status;
@@ -331,7 +337,7 @@ nfsd3_create_file(struct svc_rqst *rqstp, struct svc_fh *fhp,
        }
 
 set_attr:
-       status = nfsd_create_setattr(rqstp, fhp, resfhp, iap);
+       status = nfsd_create_setattr(rqstp, fhp, resfhp, &attrs);
 
 out:
        fh_unlock(fhp);
@@ -368,6 +374,9 @@ nfsd3_proc_mkdir(struct svc_rqst *rqstp)
 {
        struct nfsd3_createargs *argp = rqstp->rq_argp;
        struct nfsd3_diropres *resp = rqstp->rq_resp;
+       struct nfsd_attrs attrs = {
+               .na_iattr       = &argp->attrs,
+       };
 
        dprintk("nfsd: MKDIR(3)    %s %.*s\n",
                                SVCFH_fmt(&argp->fh),
@@ -378,7 +387,7 @@ nfsd3_proc_mkdir(struct svc_rqst *rqstp)
        fh_copy(&resp->dirfh, &argp->fh);
        fh_init(&resp->fh, NFS3_FHSIZE);
        resp->status = nfsd_create(rqstp, &resp->dirfh, argp->name, argp->len,
-                                  &argp->attrs, S_IFDIR, 0, &resp->fh);
+                                  &attrs, S_IFDIR, 0, &resp->fh);
        fh_unlock(&resp->dirfh);
        return rpc_success;
 }
@@ -428,6 +437,9 @@ nfsd3_proc_mknod(struct svc_rqst *rqstp)
 {
        struct nfsd3_mknodargs *argp = rqstp->rq_argp;
        struct nfsd3_diropres  *resp = rqstp->rq_resp;
+       struct nfsd_attrs attrs = {
+               .na_iattr       = &argp->attrs,
+       };
        int type;
        dev_t   rdev = 0;
 
@@ -453,7 +465,7 @@ nfsd3_proc_mknod(struct svc_rqst *rqstp)
 
        type = nfs3_ftypes[argp->ftype];
        resp->status = nfsd_create(rqstp, &resp->dirfh, argp->name, argp->len,
-                                  &argp->attrs, type, rdev, &resp->fh);
+                                  &attrs, type, rdev, &resp->fh);
        fh_unlock(&resp->dirfh);
 out:
        return rpc_success;

diff --git a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c
index ae0948271da9ccc51130d7111be792d622d62660..9b04611a318d7e41c90e7fefe0fa980df6736ced 100644 (file)
--- a/fs/nfsd/nfs4proc.c
+++ b/fs/nfsd/nfs4proc.c
@@ -286,6 +286,9 @@ nfsd4_create_file(struct svc_rqst *rqstp, struct svc_fh *fhp,
                  struct svc_fh *resfhp, struct nfsd4_open *open)
 {
        struct iattr *iap = &open->op_iattr;
+       struct nfsd_attrs attrs = {
+               .na_iattr       = iap,
+       };
        struct dentry *parent, *child;
        __u32 v_mtime, v_atime;
        struct inode *inode;
@@ -404,7 +407,7 @@ nfsd4_create_file(struct svc_rqst *rqstp, struct svc_fh *fhp,
        }
 
 set_attr:
-       status = nfsd_create_setattr(rqstp, fhp, resfhp, iap);
+       status = nfsd_create_setattr(rqstp, fhp, resfhp, &attrs);
 
 out:
        fh_unlock(fhp);
@@ -787,6 +790,9 @@ nfsd4_create(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
             union nfsd4_op_u *u)
 {
        struct nfsd4_create *create = &u->create;
+       struct nfsd_attrs attrs = {
+               .na_iattr       = &create->cr_iattr,
+       };
        struct svc_fh resfh;
        __be32 status;
        dev_t rdev;
@@ -818,7 +824,7 @@ nfsd4_create(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
                        goto out_umask;
                status = nfsd_create(rqstp, &cstate->current_fh,
                                     create->cr_name, create->cr_namelen,
-                                    &create->cr_iattr, S_IFBLK, rdev, &resfh);
+                                    &attrs, S_IFBLK, rdev, &resfh);
                break;
 
        case NF4CHR:
@@ -829,26 +835,26 @@ nfsd4_create(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
                        goto out_umask;
                status = nfsd_create(rqstp, &cstate->current_fh,
                                     create->cr_name, create->cr_namelen,
-                                    &create->cr_iattr, S_IFCHR, rdev, &resfh);
+                                    &attrs, S_IFCHR, rdev, &resfh);
                break;
 
        case NF4SOCK:
                status = nfsd_create(rqstp, &cstate->current_fh,
                                     create->cr_name, create->cr_namelen,
-                                    &create->cr_iattr, S_IFSOCK, 0, &resfh);
+                                    &attrs, S_IFSOCK, 0, &resfh);
                break;
 
        case NF4FIFO:
                status = nfsd_create(rqstp, &cstate->current_fh,
                                     create->cr_name, create->cr_namelen,
-                                    &create->cr_iattr, S_IFIFO, 0, &resfh);
+                                    &attrs, S_IFIFO, 0, &resfh);
                break;
 
        case NF4DIR:
                create->cr_iattr.ia_valid &= ~ATTR_SIZE;
                status = nfsd_create(rqstp, &cstate->current_fh,
                                     create->cr_name, create->cr_namelen,
-                                    &create->cr_iattr, S_IFDIR, 0, &resfh);
+                                    &attrs, S_IFDIR, 0, &resfh);
                break;
 
        default:
@@ -1142,6 +1148,9 @@ nfsd4_setattr(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
              union nfsd4_op_u *u)
 {
        struct nfsd4_setattr *setattr = &u->setattr;
+       struct nfsd_attrs attrs = {
+               .na_iattr       = &setattr->sa_iattr,
+       };
        __be32 status = nfs_ok;
        int err;
 
@@ -1174,7 +1183,7 @@ nfsd4_setattr(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
                                &setattr->sa_label);
        if (status)
                goto out;
-       status = nfsd_setattr(rqstp, &cstate->current_fh, &setattr->sa_iattr,
+       status = nfsd_setattr(rqstp, &cstate->current_fh, &attrs,
                                0, (time64_t)0);
 out:
        fh_drop_write(&cstate->current_fh);

diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c
index 7122ebc50a0359823bf5163999841da6ff4e2588..a299aeaa0de070b17d79e7bbbab83ad789c3f988 100644 (file)
--- a/fs/nfsd/nfs4state.c
+++ b/fs/nfsd/nfs4state.c
@@ -5077,11 +5077,14 @@ nfsd4_truncate(struct svc_rqst *rqstp, struct svc_fh *fh,
                .ia_valid = ATTR_SIZE,
                .ia_size = 0,
        };
+       struct nfsd_attrs attrs = {
+               .na_iattr       = &iattr,
+       };
        if (!open->op_truncate)
                return 0;
        if (!(open->op_share_access & NFS4_SHARE_ACCESS_WRITE))
                return nfserr_inval;
-       return nfsd_setattr(rqstp, fh, &iattr, 0, (time64_t)0);
+       return nfsd_setattr(rqstp, fh, &attrs, 0, (time64_t)0);
 }
 
 static __be32 nfs4_get_vfs_file(struct svc_rqst *rqstp, struct nfs4_file *fp,

diff --git a/fs/nfsd/nfsproc.c b/fs/nfsd/nfsproc.c
index f65eba938a57da72dea00e9176f6aa701fa3285b..c75d83bc3f21b50a0fc9a601b301dea463f96593 100644 (file)
--- a/fs/nfsd/nfsproc.c
+++ b/fs/nfsd/nfsproc.c
@@ -51,6 +51,9 @@ nfsd_proc_setattr(struct svc_rqst *rqstp)
        struct nfsd_sattrargs *argp = rqstp->rq_argp;
        struct nfsd_attrstat *resp = rqstp->rq_resp;
        struct iattr *iap = &argp->attrs;
+       struct nfsd_attrs attrs = {
+               .na_iattr       = iap,
+       };
        struct svc_fh *fhp;
 
        dprintk("nfsd: SETATTR  %s, valid=%x, size=%ld\n",
@@ -100,7 +103,7 @@ nfsd_proc_setattr(struct svc_rqst *rqstp)
                }
        }
 
-       resp->status = nfsd_setattr(rqstp, fhp, iap, 0, (time64_t)0);
+       resp->status = nfsd_setattr(rqstp, fhp, &attrs, 0, (time64_t)0);
        if (resp->status != nfs_ok)
                goto out;
 
@@ -261,6 +264,9 @@ nfsd_proc_create(struct svc_rqst *rqstp)
        svc_fh          *dirfhp = &argp->fh;
        svc_fh          *newfhp = &resp->fh;
        struct iattr    *attr = &argp->attrs;
+       struct nfsd_attrs attrs = {
+               .na_iattr       = attr,
+       };
        struct inode    *inode;
        struct dentry   *dchild;
        int             type, mode;
@@ -386,7 +392,7 @@ nfsd_proc_create(struct svc_rqst *rqstp)
        if (!inode) {
                /* File doesn't exist. Create it and set attrs */
                resp->status = nfsd_create_locked(rqstp, dirfhp, argp->name,
-                                                 argp->len, attr, type, rdev,
+                                                 argp->len, &attrs, type, rdev,
                                                  newfhp);
        } else if (type == S_IFREG) {
                dprintk("nfsd:   existing %s, valid=%x, size=%ld\n",
@@ -397,7 +403,7 @@ nfsd_proc_create(struct svc_rqst *rqstp)
                 */
                attr->ia_valid &= ATTR_SIZE;
                if (attr->ia_valid)
-                       resp->status = nfsd_setattr(rqstp, newfhp, attr, 0,
+                       resp->status = nfsd_setattr(rqstp, newfhp, &attrs, 0,
                                                    (time64_t)0);
        }
 
@@ -512,6 +518,9 @@ nfsd_proc_mkdir(struct svc_rqst *rqstp)
 {
        struct nfsd_createargs *argp = rqstp->rq_argp;
        struct nfsd_diropres *resp = rqstp->rq_resp;
+       struct nfsd_attrs attrs = {
+               .na_iattr       = &argp->attrs,
+       };
 
        dprintk("nfsd: MKDIR    %s %.*s\n", SVCFH_fmt(&argp->fh), argp->len, argp->name);
 
@@ -523,7 +532,7 @@ nfsd_proc_mkdir(struct svc_rqst *rqstp)
        argp->attrs.ia_valid &= ~ATTR_SIZE;
        fh_init(&resp->fh, NFS_FHSIZE);
        resp->status = nfsd_create(rqstp, &argp->fh, argp->name, argp->len,
-                                  &argp->attrs, S_IFDIR, 0, &resp->fh);
+                                  &attrs, S_IFDIR, 0, &resp->fh);
        fh_put(&argp->fh);
        if (resp->status != nfs_ok)
                goto out;

diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c
index 6689ad5bb790d8b79aef325ce20a9685f16bd8e9..489225de05a2a91e717bafecdaed12b28ee761ff 100644 (file)
--- a/fs/nfsd/vfs.c
+++ b/fs/nfsd/vfs.c
@@ -350,11 +350,13 @@ nfsd_get_write_access(struct svc_rqst *rqstp, struct svc_fh *fhp,
  * Set various file attributes.  After this call fhp needs an fh_put.
  */
 __be32
-nfsd_setattr(struct svc_rqst *rqstp, struct svc_fh *fhp, struct iattr *iap,
+nfsd_setattr(struct svc_rqst *rqstp, struct svc_fh *fhp,
+            struct nfsd_attrs *attr,
             int check_guard, time64_t guardtime)
 {
        struct dentry   *dentry;
        struct inode    *inode;
+       struct iattr    *iap = attr->na_iattr;
        int             accmode = NFSD_MAY_SATTR;
        umode_t         ftype = 0;
        __be32          err;
@@ -1203,14 +1205,15 @@ out:
  * @rqstp: RPC transaction being executed
  * @fhp: NFS filehandle of parent directory
  * @resfhp: NFS filehandle of new object
- * @iap: requested attributes of new object
+ * @attrs: requested attributes of new object
  *
  * Returns nfs_ok on success, or an nfsstat in network byte order.
  */
 __be32
 nfsd_create_setattr(struct svc_rqst *rqstp, struct svc_fh *fhp,
-                   struct svc_fh *resfhp, struct iattr *iap)
+                   struct svc_fh *resfhp, struct nfsd_attrs *attrs)
 {
+       struct iattr *iap = attrs->na_iattr;
        __be32 status;
 
        /*
@@ -1231,7 +1234,7 @@ nfsd_create_setattr(struct svc_rqst *rqstp, struct svc_fh *fhp,
         * if the attributes have not changed.
         */
        if (iap->ia_valid)
-               status = nfsd_setattr(rqstp, resfhp, iap, 0, (time64_t)0);
+               status = nfsd_setattr(rqstp, resfhp, attrs, 0, (time64_t)0);
        else
                status = nfserrno(commit_metadata(resfhp));
 
@@ -1270,11 +1273,12 @@ nfsd_check_ignore_resizing(struct iattr *iap)
 /* The parent directory should already be locked: */
 __be32
 nfsd_create_locked(struct svc_rqst *rqstp, struct svc_fh *fhp,
-               char *fname, int flen, struct iattr *iap,
-               int type, dev_t rdev, struct svc_fh *resfhp)
+                  char *fname, int flen, struct nfsd_attrs *attrs,
+                  int type, dev_t rdev, struct svc_fh *resfhp)
 {
        struct dentry   *dentry, *dchild;
        struct inode    *dirp;
+       struct iattr    *iap = attrs->na_iattr;
        __be32          err;
        int             host_err;
 
@@ -1348,7 +1352,7 @@ nfsd_create_locked(struct svc_rqst *rqstp, struct svc_fh *fhp,
        if (host_err < 0)
                goto out_nfserr;
 
-       err = nfsd_create_setattr(rqstp, fhp, resfhp, iap);
+       err = nfsd_create_setattr(rqstp, fhp, resfhp, attrs);
 
 out:
        dput(dchild);
@@ -1367,8 +1371,8 @@ out_nfserr:
  */
 __be32
 nfsd_create(struct svc_rqst *rqstp, struct svc_fh *fhp,
-               char *fname, int flen, struct iattr *iap,
-               int type, dev_t rdev, struct svc_fh *resfhp)
+           char *fname, int flen, struct nfsd_attrs *attrs,
+           int type, dev_t rdev, struct svc_fh *resfhp)
 {
        struct dentry   *dentry, *dchild = NULL;
        __be32          err;
@@ -1400,7 +1404,7 @@ nfsd_create(struct svc_rqst *rqstp, struct svc_fh *fhp,
        dput(dchild);
        if (err)
                return err;
-       return nfsd_create_locked(rqstp, fhp, fname, flen, iap, type,
+       return nfsd_create_locked(rqstp, fhp, fname, flen, attrs, type,
                                        rdev, resfhp);
 }
 

diff --git a/fs/nfsd/vfs.h b/fs/nfsd/vfs.h
index 26347d76f44a0d52879000775ecebe2f1253a57f..d8b1a36fca956e68d48cb79759a2a5343d701e4a 100644 (file)
--- a/fs/nfsd/vfs.h
+++ b/fs/nfsd/vfs.h
@@ -42,6 +42,10 @@ struct nfsd_file;
 typedef int (*nfsd_filldir_t)(void *, const char *, int, loff_t, u64, unsigned);
 
 /* nfsd/vfs.c */
+struct nfsd_attrs {
+       struct iattr            *na_iattr;      /* input */
+};
+
 int            nfsd_cross_mnt(struct svc_rqst *rqstp, struct dentry **dpp,
                                struct svc_export **expp);
 __be32         nfsd_lookup(struct svc_rqst *, struct svc_fh *,
@@ -50,7 +54,7 @@ __be32                 nfsd_lookup_dentry(struct svc_rqst *, struct svc_fh *,
                                const char *, unsigned int,
                                struct svc_export **, struct dentry **);
 __be32         nfsd_setattr(struct svc_rqst *, struct svc_fh *,
-                               struct iattr *, int, time64_t);
+                               struct nfsd_attrs *, int, time64_t);
 int nfsd_mountpoint(struct dentry *, struct svc_export *);
 #ifdef CONFIG_NFSD_V4
 __be32          nfsd4_set_nfs4_label(struct svc_rqst *, struct svc_fh *,
@@ -63,14 +67,14 @@ __be32              nfsd4_clone_file_range(struct svc_rqst *rqstp,
                                       u64 count, bool sync);
 #endif /* CONFIG_NFSD_V4 */
 __be32         nfsd_create_locked(struct svc_rqst *, struct svc_fh *,
-                               char *name, int len, struct iattr *attrs,
+                               char *name, int len, struct nfsd_attrs *attrs,
                                int type, dev_t rdev, struct svc_fh *res);
 __be32         nfsd_create(struct svc_rqst *, struct svc_fh *,
-                               char *name, int len, struct iattr *attrs,
+                               char *name, int len, struct nfsd_attrs *attrs,
                                int type, dev_t rdev, struct svc_fh *res);
 __be32         nfsd_access(struct svc_rqst *, struct svc_fh *, u32 *, u32 *);
 __be32         nfsd_create_setattr(struct svc_rqst *rqstp, struct svc_fh *fhp,
-                               struct svc_fh *resfhp, struct iattr *iap);
+                               struct svc_fh *resfhp, struct nfsd_attrs *iap);
 __be32         nfsd_commit(struct svc_rqst *rqst, struct svc_fh *fhp,
                                u64 offset, u32 count, __be32 *verf);
 #ifdef CONFIG_NFSD_V4