Reduce repetitive codes for open_file and create_file
Signed-off-by: Mi Jinlong <mijinlong@xxxxxxxxxxxxxx>
---
nfs4.1/server41tests/environment.py | 86 ++++++++++++++++++++++-------------
1 files changed, 54 insertions(+), 32 deletions(-)
diff --git a/nfs4.1/server41tests/environment.py b/nfs4.1/server41tests/environment.py
index 43cb12b..6e68e83 100644
--- a/nfs4.1/server41tests/environment.py
+++ b/nfs4.1/server41tests/environment.py
@@ -444,12 +444,15 @@ def create_obj(sess, path, kind=NF4DIR, attrs={FATTR4_MODE:0755}):
ops = use_obj(path[:-1]) + [op.create(kind, path[-1], attrs)]
return sess.compound(ops)
-def create_file(sess, owner, path=None, attrs={FATTR4_MODE: 0644},
- access=OPEN4_SHARE_ACCESS_BOTH,
- deny=OPEN4_SHARE_DENY_NONE,
- mode=GUARDED4, verifier=None, want_deleg=False,
- # Setting the following should induce server errors
- seqid=0, clientid=0):
+def open_create_file(sess, owner, path=None, attrs={FATTR4_MODE: 0644},
+ access=OPEN4_SHARE_ACCESS_BOTH,
+ deny=OPEN4_SHARE_DENY_NONE,
+ mode=GUARDED4, verifier=None,
+ claim_type=CLAIM_NULL,
+ want_deleg=False,
+ deleg_type=None,
+ open_create=OPEN4_NOCREATE,
+ seqid=0, clientid=0):
# Set defaults
if path is None:
dir = sess.c.homedir
@@ -457,16 +460,47 @@ def create_file(sess, owner, path=None, attrs={FATTR4_MODE: 0644},
else:
dir = path[:-1]
name = path[-1]
+
if ((mode==EXCLUSIVE4) or (mode==EXCLUSIVE4_1)) and (verifier==None):
verifier = sess.c.verifier
+
if not want_deleg and access & OPEN4_SHARE_ACCESS_WANT_DELEG_MASK == 0:
access |= OPEN4_SHARE_ACCESS_WANT_NO_DELEG
- # Create the file
+
+ # Open the file
+ if claim_type==CLAIM_NULL:
+ fh_op = use_obj(dir)
+ elif claim_type==CLAIM_PREVIOUS:
+ fh_op = [op.putfh(path)]
+ name = None
+
+ if open_create==OPEN4_CREATE:
+ openflag=openflag4(OPEN4_CREATE, createhow4(mode, attrs, verifier,
+ creatverfattr(verifier, attrs)))
+ openclaim=open_claim4(CLAIM_NULL, name)
+ else:
+ openflag=openflag4(OPEN4_NOCREATE)
+ openclaim=open_claim4(claim_type, name, deleg_type)
+
open_op = op.open(seqid, access, deny, open_owner4(clientid, owner),
- openflag4(OPEN4_CREATE, createhow4(mode, attrs, verifier,
- creatverfattr(verifier, attrs))),
- open_claim4(CLAIM_NULL, name))
- return sess.compound(use_obj(dir) + [open_op, op.getfh()])
+ openflag, openclaim)
+
+ return sess.compound(fh_op + [open_op, op.getfh()])
+
+def create_file(sess, owner, path=None, attrs={FATTR4_MODE: 0644},
+ access=OPEN4_SHARE_ACCESS_BOTH,
+ deny=OPEN4_SHARE_DENY_NONE,
+ mode=GUARDED4, verifier=None, want_deleg=False,
+ # Setting the following should induce server errors
+ seqid=0, clientid=0):
+
+ claim_type=CLAIM_NULL
+ deleg_type=None
+ open_create=OPEN4_CREATE
+
+ return open_create_file(sess, owner, path, attrs, access, deny, mode,
+ verifier, claim_type, want_deleg, deleg_type,
+ open_create, seqid, clientid)
def open_file(sess, owner, path=None,
access=OPEN4_SHARE_ACCESS_READ,
@@ -476,27 +510,15 @@ def open_file(sess, owner, path=None,
deleg_type=None,
# Setting the following should induce server errors
seqid=0, clientid=0):
- # Set defaults
- if path is None:
- dir = sess.c.homedir
- name = owner
- else:
- dir = path[:-1]
- name = path[-1]
- if not want_deleg and access & OPEN4_SHARE_ACCESS_WANT_DELEG_MASK == 0:
- access |= OPEN4_SHARE_ACCESS_WANT_NO_DELEG
- # Open the file
- if claim_type==CLAIM_NULL:
- fh_op = use_obj(dir)
- elif claim_type==CLAIM_PREVIOUS:
- fh_op = [op.putfh(path)]
- name = None
- if not want_deleg and access & OPEN4_SHARE_ACCESS_WANT_DELEG_MASK == 0:
- access |= OPEN4_SHARE_ACCESS_WANT_NO_DELEG
- open_op = op.open(seqid, access, deny, open_owner4(clientid, owner),
- openflag4(OPEN4_NOCREATE),
- open_claim4(claim_type, name, deleg_type))
- return sess.compound(fh_op + [open_op, op.getfh()])
+
+ attrs=None
+ mode=None
+ verifier=None
+ open_create=OPEN4_NOCREATE
+
+ return open_create_file(sess, owner, path, attrs, access, deny, mode,
+ verifier, claim_type, want_deleg, deleg_type,
+ open_create, seqid, clientid)
def create_confirm(sess, owner, path=None, attrs={FATTR4_MODE: 0644},
access=OPEN4_SHARE_ACCESS_BOTH,
--
1.7.5.1
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
