NFSv4: Record the OPEN create mode used in the nfs4_opendata structure

If we're doing NFSv4.1 against a server that has persistent sessions,
then we should not need to call SETATTR in order to reset the file
attributes immediately after doing an exclusive create.

Note that since the create mode depends on the type of session that
has been negotiated with the server, we should not choose the
mode until after we've got a session slot.

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>

1 files changed, 16 insertions, 21 deletions

diff --git a/fs/nfs/nfs4xdr.c b/fs/nfs/nfs4xdr.c
index 0b744895b9e1..fef71cbec501 100644
--- a/fs/nfs/nfs4xdr.c
+++ b/fs/nfs/nfs4xdr.c
@@ -1366,33 +1366,28 @@ static inline void encode_openhdr(struct xdr_stream *xdr, const struct nfs_opena
 
 static inline void encode_createmode(struct xdr_stream *xdr, const struct nfs_openargs *arg)
 {
+        struct iattr dummy;
         __be32 *p;
-        struct nfs_client *clp;
 
         p = reserve_space(xdr, 4);
-        switch(arg->open_flags & O_EXCL) {
-        case 0:
+        switch(arg->createmode) {
+        case NFS4_CREATE_UNCHECKED:
                 *p = cpu_to_be32(NFS4_CREATE_UNCHECKED);
                 encode_attrs(xdr, arg->u.attrs, arg->server);
                 break;
-        default:
-                clp = arg->server->nfs_client;
-                if (clp->cl_mvops->minor_version > 0) {
-                        if (nfs4_has_persistent_session(clp)) {
-                                *p = cpu_to_be32(NFS4_CREATE_GUARDED);
-                                encode_attrs(xdr, arg->u.attrs, arg->server);
-                        } else {
-                                struct iattr dummy;
-
-                                *p = cpu_to_be32(NFS4_CREATE_EXCLUSIVE4_1);
-                                encode_nfs4_verifier(xdr, &arg->u.verifier);
-                                dummy.ia_valid = 0;
-                                encode_attrs(xdr, &dummy, arg->server);
-                        }
-                } else {
-                        *p = cpu_to_be32(NFS4_CREATE_EXCLUSIVE);
-                        encode_nfs4_verifier(xdr, &arg->u.verifier);
-                }
+        case NFS4_CREATE_GUARDED:
+                *p = cpu_to_be32(NFS4_CREATE_GUARDED);
+                encode_attrs(xdr, arg->u.attrs, arg->server);
+                break;
+        case NFS4_CREATE_EXCLUSIVE:
+                *p = cpu_to_be32(NFS4_CREATE_EXCLUSIVE);
+                encode_nfs4_verifier(xdr, &arg->u.verifier);
+                break;
+        case NFS4_CREATE_EXCLUSIVE4_1:
+                *p = cpu_to_be32(NFS4_CREATE_EXCLUSIVE4_1);
+                encode_nfs4_verifier(xdr, &arg->u.verifier);
+                dummy.ia_valid = 0;
+                encode_attrs(xdr, &dummy, arg->server);
         }
 }