* [PATCH 0/5] more test on current stateid processing
@ 2012-01-15 19:05 Tigran Mkrtchyan
2012-01-15 19:05 ` [PATCH 1/5] client: move current stateid test into a dedicated module Tigran Mkrtchyan
` (7 more replies)
0 siblings, 8 replies; 11+ messages in thread
From: Tigran Mkrtchyan @ 2012-01-15 19:05 UTC (permalink / raw)
To: linux-nfs, bfields; +Cc: Tigran Mkrtchyan
This is the result of testing my changes in linux:
$ ./testserver.py --maketree 192.168.178.20:/tmp currentstateid
.....
**************************************************
CSID1 st_current_stateid.testOpenAndClose : PASS
CSID2 st_current_stateid.testLockLockU : PASS
CSID3 st_current_stateid.testOpenWriteClose : PASS
CSID4 st_current_stateid.testLockWriteLocku : PASS
CSID5 st_current_stateid.testOpenPutrootfhClose : PASS
CSID6 st_current_stateid.testCloseNoStateid : PASS
CSID7 st_current_stateid.testOpenLayoutGet : FAILURE
OP_LAYOUTGET should return NFS4_OK, instead got
NFS4ERR_NOTSUPP
CSID8 st_current_stateid.testOpenSetattr : PASS
CSID9 st_current_stateid.testOpenFreestateidClose : PA
**************************************************
$
Tigran Mkrtchyan (5):
client: move current stateid test into a dedicated module
client: test current stateid processing on PUTROOTFH
client: test current stateid processing with OPEN+LAYOUTGET in a
single compound
client: test current stateid processing with OPEN and SETATTR
client: test OPEN+FREE_STETEID+CLOSE in a single compound
nfs4.1/server41tests/__init__.py | 1 +
nfs4.1/server41tests/st_current_stateid.py | 161 ++++++++++++++++++++++++++++
nfs4.1/server41tests/st_open.py | 76 -------------
3 files changed, 162 insertions(+), 76 deletions(-)
create mode 100644 nfs4.1/server41tests/st_current_stateid.py
--
1.7.7.5
^ permalink raw reply [flat|nested] 11+ messages in thread* [PATCH 1/5] client: move current stateid test into a dedicated module 2012-01-15 19:05 [PATCH 0/5] more test on current stateid processing Tigran Mkrtchyan @ 2012-01-15 19:05 ` Tigran Mkrtchyan 2012-01-15 19:05 ` [PATCH 2/5] client: test current stateid processing on PUTROOTFH Tigran Mkrtchyan ` (6 subsequent siblings) 7 siblings, 0 replies; 11+ messages in thread From: Tigran Mkrtchyan @ 2012-01-15 19:05 UTC (permalink / raw) To: linux-nfs, bfields; +Cc: Tigran Mkrtchyan Signed-off-by: Tigran Mkrtchyan <tigran.mkrtchyan@desy.de> --- nfs4.1/server41tests/__init__.py | 1 + nfs4.1/server41tests/st_current_stateid.py | 86 ++++++++++++++++++++++++++++ nfs4.1/server41tests/st_open.py | 76 ------------------------ 3 files changed, 87 insertions(+), 76 deletions(-) create mode 100644 nfs4.1/server41tests/st_current_stateid.py diff --git a/nfs4.1/server41tests/__init__.py b/nfs4.1/server41tests/__init__.py index eb92538..2bedb57 100644 --- a/nfs4.1/server41tests/__init__.py +++ b/nfs4.1/server41tests/__init__.py @@ -20,4 +20,5 @@ __all__ = ["st_exchange_id.py", # draft 21 "st_block.py", ## "st_debug.py", ## "st_loop", + "st_current_stateid.py", ] diff --git a/nfs4.1/server41tests/st_current_stateid.py b/nfs4.1/server41tests/st_current_stateid.py new file mode 100644 index 0000000..1f24ec9 --- /dev/null +++ b/nfs4.1/server41tests/st_current_stateid.py @@ -0,0 +1,86 @@ +from st_create_session import create_session +from nfs4_const import * + +from environment import check, checklist, fail, create_file, open_file, close_file +from environment import open_create_file_op +from nfs4_type import open_owner4, openflag4, createhow4, open_claim4 +from nfs4_type import creatverfattr, fattr4, stateid4, locker4, lock_owner4 +from nfs4_type import open_to_lock_owner4 +import nfs4_ops as op +import threading + + +current_stateid = stateid4(1, '\0' * 12) + +def testOpenAndClose(t, env): + """test current state id processing by having OPEN and CLOSE + in a single compound + + FLAGS: currentstateid all + CODE: CSID1 + """ + sess1 = env.c1.new_client_session(env.testname(t)) + + open_op = open_create_file_op(sess1, env.testname(t), open_create=OPEN4_CREATE) + res = sess1.compound(open_op + [op.close(0, current_stateid)]) + check(res, NFS4_OK) + +def testLockLockU(t, env): + """test current state id processing by having LOCK and LOCKU + in a single compound + + FLAGS: currentstateid all + CODE: CSID2 + """ + sess1 = env.c1.new_client_session(env.testname(t)) + + res = create_file(sess1, env.testname(t)) + check(res) + fh = res.resarray[-1].object + stateid = res.resarray[-2].stateid + + open_to_lock_owner = open_to_lock_owner4( 0, stateid, 0, lock_owner4(0, "lock1")) + lock_owner = locker4(open_owner=open_to_lock_owner, new_lock_owner=True) + lock_ops = [ op.lock(WRITE_LT, False, 0, NFS4_UINT64_MAX, lock_owner), + op.locku(WRITE_LT, 0, current_stateid, 0, NFS4_UINT64_MAX) ] + res = sess1.compound([op.putfh(fh)] + lock_ops) + check(res, NFS4_OK) + +def testOpenWriteClose(t, env): + """test current state id processing by having OPEN, WRITE and CLOSE + in a single compound + + FLAGS: currentstateid all + CODE: CSID3 + """ + sess1 = env.c1.new_client_session(env.testname(t)) + + data = "write test data" + open_op = open_create_file_op(sess1, env.testname(t), open_create=OPEN4_CREATE) + res = sess1.compound(open_op + [op.write(current_stateid, 5, FILE_SYNC4, data), + op.close(0, current_stateid)]) + check(res, NFS4_OK) + +def testLockWriteLocku(t, env): + """test current state id processing by having LOCK, WRITE and LOCKU + in a single compound + + FLAGS: currentstateid all + CODE: CSID4 + """ + sess1 = env.c1.new_client_session(env.testname(t)) + + res = create_file(sess1, env.testname(t)) + check(res) + fh = res.resarray[-1].object + stateid = res.resarray[-2].stateid + + data = "write test data" + open_to_lock_owner = open_to_lock_owner4( 0, stateid, 0, lock_owner4(0, "lock1")) + lock_owner = locker4(open_owner=open_to_lock_owner, new_lock_owner=True) + lock_ops = [ op.lock(WRITE_LT, False, 0, NFS4_UINT64_MAX, lock_owner), + op.write(current_stateid, 5, FILE_SYNC4, data), + op.locku(WRITE_LT, 0, current_stateid, 0, NFS4_UINT64_MAX), + op.close(0, stateid)] + res = sess1.compound([op.putfh(fh)] + lock_ops) + check(res, NFS4_OK) diff --git a/nfs4.1/server41tests/st_open.py b/nfs4.1/server41tests/st_open.py index 1c51f71..895916a 100644 --- a/nfs4.1/server41tests/st_open.py +++ b/nfs4.1/server41tests/st_open.py @@ -240,79 +240,3 @@ def testOPENClaimFH(t, env): if res.resarray[-1].data != desired: fail("Expected %r, got %r" % (desired, res.resarray[-1].data)) -def testOpenAndClose(t, env): - """test current state id processing by having OPEN and CLOSE - in a single compound - - FLAGS: currentstateid open all - CODE: OPEN31 - """ - current_stateid = stateid4(1, '\0' * 12) - sess1 = env.c1.new_client_session(env.testname(t)) - - open_op = open_create_file_op(sess1, env.testname(t), open_create=OPEN4_CREATE) - res = sess1.compound(open_op + [op.close(0, current_stateid)]) - check(res, NFS4_OK) - -def testLockLockU(t, env): - """test current state id processing by having LOCK and LOCKU - in a single compound - - FLAGS: currentstateid open all - CODE: OPEN32 - """ - current_stateid = stateid4(1, '\0' * 12) - sess1 = env.c1.new_client_session(env.testname(t)) - - res = create_file(sess1, env.testname(t)) - check(res) - fh = res.resarray[-1].object - stateid = res.resarray[-2].stateid - - open_to_lock_owner = open_to_lock_owner4( 0, stateid, 0, lock_owner4(0, "lock1")) - lock_owner = locker4(open_owner=open_to_lock_owner, new_lock_owner=True) - lock_ops = [ op.lock(WRITE_LT, False, 0, NFS4_UINT64_MAX, lock_owner), - op.locku(WRITE_LT, 0, current_stateid, 0, NFS4_UINT64_MAX) ] - res = sess1.compound([op.putfh(fh)] + lock_ops) - check(res, NFS4_OK) - -def testOpenWriteClose(t, env): - """test current state id processing by having OPEN, WRITE and CLOSE - in a single compound - - FLAGS: currentstateid open all - CODE: OPEN33 - """ - current_stateid = stateid4(1, '\0' * 12) - sess1 = env.c1.new_client_session(env.testname(t)) - - data = "write test data" - open_op = open_create_file_op(sess1, env.testname(t), open_create=OPEN4_CREATE) - res = sess1.compound(open_op + [op.write(current_stateid, 5, FILE_SYNC4, data), - op.close(0, current_stateid)]) - check(res, NFS4_OK) - -def testLockWriteLocku(t, env): - """test current state id processing by having LOCK, WRITE and LOCKU - in a single compound - - FLAGS: currentstateid open all - CODE: OPEN34 - """ - current_stateid = stateid4(1, '\0' * 12) - sess1 = env.c1.new_client_session(env.testname(t)) - - res = create_file(sess1, env.testname(t)) - check(res) - fh = res.resarray[-1].object - stateid = res.resarray[-2].stateid - - data = "write test data" - open_to_lock_owner = open_to_lock_owner4( 0, stateid, 0, lock_owner4(0, "lock1")) - lock_owner = locker4(open_owner=open_to_lock_owner, new_lock_owner=True) - lock_ops = [ op.lock(WRITE_LT, False, 0, NFS4_UINT64_MAX, lock_owner), - op.write(current_stateid, 5, FILE_SYNC4, data), - op.locku(WRITE_LT, 0, current_stateid, 0, NFS4_UINT64_MAX), - op.close(0, stateid)] - res = sess1.compound([op.putfh(fh)] + lock_ops) - check(res, NFS4_OK) -- 1.7.7.5 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 2/5] client: test current stateid processing on PUTROOTFH 2012-01-15 19:05 [PATCH 0/5] more test on current stateid processing Tigran Mkrtchyan 2012-01-15 19:05 ` [PATCH 1/5] client: move current stateid test into a dedicated module Tigran Mkrtchyan @ 2012-01-15 19:05 ` Tigran Mkrtchyan 2012-01-15 19:05 ` [PATCH 3/5] client: test current stateid processing with OPEN+LAYOUTGET in a single compound Tigran Mkrtchyan ` (5 subsequent siblings) 7 siblings, 0 replies; 11+ messages in thread From: Tigran Mkrtchyan @ 2012-01-15 19:05 UTC (permalink / raw) To: linux-nfs, bfields; +Cc: Tigran Mkrtchyan Signed-off-by: Tigran Mkrtchyan <tigran.mkrtchyan@desy.de> --- nfs4.1/server41tests/st_current_stateid.py | 31 ++++++++++++++++++++++++++++ 1 files changed, 31 insertions(+), 0 deletions(-) diff --git a/nfs4.1/server41tests/st_current_stateid.py b/nfs4.1/server41tests/st_current_stateid.py index 1f24ec9..e2da451 100644 --- a/nfs4.1/server41tests/st_current_stateid.py +++ b/nfs4.1/server41tests/st_current_stateid.py @@ -84,3 +84,34 @@ def testLockWriteLocku(t, env): op.close(0, stateid)] res = sess1.compound([op.putfh(fh)] + lock_ops) check(res, NFS4_OK) + +def testOpenPutrootfhClose(t, env): + """test current state id processing by having OPEN, PUTROOTFH and CLOSE + in a single compound + + FLAGS: currentstateid all + CODE: CSID5 + """ + sess1 = env.c1.new_client_session(env.testname(t)) + + open_op = open_create_file_op(sess1, env.testname(t), open_create=OPEN4_CREATE) + res = sess1.compound(open_op + [op.putrootfh(), op.close(0, current_stateid)]) + checklist(res, [NFS4ERR_STALE_STATEID, NFS4ERR_BAD_STATEID]) + +def testCloseNoStateid(t, env): + """test current state id processing by having CLOSE + without operation which provides stateid + + FLAGS: currentstateid all + CODE: CSID6 + """ + sess1 = env.c1.new_client_session(env.testname(t)) + + res = create_file(sess1, env.testname(t)) + check(res) + fh = res.resarray[-1].object + stateid = res.resarray[-2].stateid + + res = sess1.compound([op.putfh(fh), op.close(0, current_stateid)]) + checklist(res, [NFS4ERR_STALE_STATEID, NFS4ERR_BAD_STATEID]) + -- 1.7.7.5 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 3/5] client: test current stateid processing with OPEN+LAYOUTGET in a single compound 2012-01-15 19:05 [PATCH 0/5] more test on current stateid processing Tigran Mkrtchyan 2012-01-15 19:05 ` [PATCH 1/5] client: move current stateid test into a dedicated module Tigran Mkrtchyan 2012-01-15 19:05 ` [PATCH 2/5] client: test current stateid processing on PUTROOTFH Tigran Mkrtchyan @ 2012-01-15 19:05 ` Tigran Mkrtchyan 2012-01-15 19:05 ` [PATCH 4/5] client: test current stateid processing with OPEN and SETATTR Tigran Mkrtchyan ` (4 subsequent siblings) 7 siblings, 0 replies; 11+ messages in thread From: Tigran Mkrtchyan @ 2012-01-15 19:05 UTC (permalink / raw) To: linux-nfs, bfields; +Cc: Tigran Mkrtchyan Signed-off-by: Tigran Mkrtchyan <tigran.mkrtchyan@desy.de> --- nfs4.1/server41tests/st_current_stateid.py | 16 ++++++++++++++++ 1 files changed, 16 insertions(+), 0 deletions(-) diff --git a/nfs4.1/server41tests/st_current_stateid.py b/nfs4.1/server41tests/st_current_stateid.py index e2da451..788c095 100644 --- a/nfs4.1/server41tests/st_current_stateid.py +++ b/nfs4.1/server41tests/st_current_stateid.py @@ -115,3 +115,19 @@ def testCloseNoStateid(t, env): res = sess1.compound([op.putfh(fh), op.close(0, current_stateid)]) checklist(res, [NFS4ERR_STALE_STATEID, NFS4ERR_BAD_STATEID]) +def testOpenLayoutGet(t, env): + """test current state id processing by having OPEN and LAYOUTGET + in a single compound + + FLAGS: currentstateid all + CODE: CSID7 + """ + sess = env.c1.new_client_session(env.testname(t), + flags=EXCHGID4_FLAG_USE_PNFS_MDS) + + open_op = open_create_file_op(sess, env.testname(t), open_create=OPEN4_CREATE) + res = sess.compound( open_op + + [op.layoutget(False, LAYOUT4_NFSV4_1_FILES, LAYOUTIOMODE4_RW, + 0, 8192, 8192, current_stateid, 0xffff)]) + check(res, NFS4_OK) + -- 1.7.7.5 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 4/5] client: test current stateid processing with OPEN and SETATTR 2012-01-15 19:05 [PATCH 0/5] more test on current stateid processing Tigran Mkrtchyan ` (2 preceding siblings ...) 2012-01-15 19:05 ` [PATCH 3/5] client: test current stateid processing with OPEN+LAYOUTGET in a single compound Tigran Mkrtchyan @ 2012-01-15 19:05 ` Tigran Mkrtchyan 2012-01-15 19:05 ` [PATCH 5/5] client: test OPEN+FREE_STETEID+CLOSE in a single compound Tigran Mkrtchyan ` (3 subsequent siblings) 7 siblings, 0 replies; 11+ messages in thread From: Tigran Mkrtchyan @ 2012-01-15 19:05 UTC (permalink / raw) To: linux-nfs, bfields; +Cc: Tigran Mkrtchyan Signed-off-by: Tigran Mkrtchyan <tigran.mkrtchyan@desy.de> --- nfs4.1/server41tests/st_current_stateid.py | 15 +++++++++++++++ 1 files changed, 15 insertions(+), 0 deletions(-) diff --git a/nfs4.1/server41tests/st_current_stateid.py b/nfs4.1/server41tests/st_current_stateid.py index 788c095..3732907 100644 --- a/nfs4.1/server41tests/st_current_stateid.py +++ b/nfs4.1/server41tests/st_current_stateid.py @@ -131,3 +131,18 @@ def testOpenLayoutGet(t, env): 0, 8192, 8192, current_stateid, 0xffff)]) check(res, NFS4_OK) +def testOpenSetattr(t, env): + """test current state id processing by having OPEN and SETATTR + in a single compound + + FLAGS: currentstateid all + CODE: CSID8 + """ + size = 8 + sess = env.c1.new_client_session(env.testname(t), + flags=EXCHGID4_FLAG_USE_PNFS_MDS) + + open_op = open_create_file_op(sess, env.testname(t), open_create=OPEN4_CREATE) + res = sess.compound( open_op + + [ op.setattr(current_stateid, {FATTR4_SIZE: size})]) + check(res, NFS4_OK) -- 1.7.7.5 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 5/5] client: test OPEN+FREE_STETEID+CLOSE in a single compound 2012-01-15 19:05 [PATCH 0/5] more test on current stateid processing Tigran Mkrtchyan ` (3 preceding siblings ...) 2012-01-15 19:05 ` [PATCH 4/5] client: test current stateid processing with OPEN and SETATTR Tigran Mkrtchyan @ 2012-01-15 19:05 ` Tigran Mkrtchyan 2012-01-15 20:54 ` [PATCH 0/5] more test on current stateid processing J. Bruce Fields ` (2 subsequent siblings) 7 siblings, 0 replies; 11+ messages in thread From: Tigran Mkrtchyan @ 2012-01-15 19:05 UTC (permalink / raw) To: linux-nfs, bfields; +Cc: Tigran Mkrtchyan Signed-off-by: Tigran Mkrtchyan <tigran.mkrtchyan@desy.de> --- nfs4.1/server41tests/st_current_stateid.py | 13 +++++++++++++ 1 files changed, 13 insertions(+), 0 deletions(-) diff --git a/nfs4.1/server41tests/st_current_stateid.py b/nfs4.1/server41tests/st_current_stateid.py index 3732907..fb0c360 100644 --- a/nfs4.1/server41tests/st_current_stateid.py +++ b/nfs4.1/server41tests/st_current_stateid.py @@ -146,3 +146,16 @@ def testOpenSetattr(t, env): res = sess.compound( open_op + [ op.setattr(current_stateid, {FATTR4_SIZE: size})]) check(res, NFS4_OK) + +def testOpenFreestateidClose(t, env): + """test current state id processing by having OPEN, FREE_STATEID and CLOSE + in a single compound + + FLAGS: currentstateid all + CODE: CSID9 + """ + sess1 = env.c1.new_client_session(env.testname(t)) + + open_op = open_create_file_op(sess1, env.testname(t), open_create=OPEN4_CREATE) + res = sess1.compound(open_op + [op.free_stateid(current_stateid), op.close(0, current_stateid)]) + checklist(res, [NFS4ERR_STALE_STATEID, NFS4ERR_BAD_STATEID]) -- 1.7.7.5 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 0/5] more test on current stateid processing 2012-01-15 19:05 [PATCH 0/5] more test on current stateid processing Tigran Mkrtchyan ` (4 preceding siblings ...) 2012-01-15 19:05 ` [PATCH 5/5] client: test OPEN+FREE_STETEID+CLOSE in a single compound Tigran Mkrtchyan @ 2012-01-15 20:54 ` J. Bruce Fields 2012-01-15 22:31 ` Tiramisu Mokka 2012-01-17 16:42 ` J. Bruce Fields 2012-01-24 11:53 ` Benny Halevy 7 siblings, 1 reply; 11+ messages in thread From: J. Bruce Fields @ 2012-01-15 20:54 UTC (permalink / raw) To: Tigran Mkrtchyan; +Cc: linux-nfs On Sun, Jan 15, 2012 at 08:05:23PM +0100, Tigran Mkrtchyan wrote: > > This is the result of testing my changes in linux: > > $ ./testserver.py --maketree 192.168.178.20:/tmp currentstateid > ..... > > ************************************************** > CSID1 st_current_stateid.testOpenAndClose : PASS > CSID2 st_current_stateid.testLockLockU : PASS > CSID3 st_current_stateid.testOpenWriteClose : PASS > CSID4 st_current_stateid.testLockWriteLocku : PASS > CSID5 st_current_stateid.testOpenPutrootfhClose : PASS > CSID6 st_current_stateid.testCloseNoStateid : PASS > CSID7 st_current_stateid.testOpenLayoutGet : FAILURE > OP_LAYOUTGET should return NFS4_OK, instead got > NFS4ERR_NOTSUPP We should fix pynfs41 so no pnfs-dependent tests are run without "pnfs" being explicitly asked for on the commandline.... --b. > CSID8 st_current_stateid.testOpenSetattr : PASS > CSID9 st_current_stateid.testOpenFreestateidClose : PA > > ************************************************** > $ > > Tigran Mkrtchyan (5): > client: move current stateid test into a dedicated module > client: test current stateid processing on PUTROOTFH > client: test current stateid processing with OPEN+LAYOUTGET in a > single compound > client: test current stateid processing with OPEN and SETATTR > client: test OPEN+FREE_STETEID+CLOSE in a single compound > > nfs4.1/server41tests/__init__.py | 1 + > nfs4.1/server41tests/st_current_stateid.py | 161 ++++++++++++++++++++++++++++ > nfs4.1/server41tests/st_open.py | 76 ------------- > 3 files changed, 162 insertions(+), 76 deletions(-) > create mode 100644 nfs4.1/server41tests/st_current_stateid.py > > -- > 1.7.7.5 > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 0/5] more test on current stateid processing 2012-01-15 20:54 ` [PATCH 0/5] more test on current stateid processing J. Bruce Fields @ 2012-01-15 22:31 ` Tiramisu Mokka 0 siblings, 0 replies; 11+ messages in thread From: Tiramisu Mokka @ 2012-01-15 22:31 UTC (permalink / raw) To: J. Bruce Fields; +Cc: Tigran Mkrtchyan, linux-nfs -- kofemann /** caffeinated mutations of the core personality */ On Sun, Jan 15, 2012 at 21:54, J. Bruce Fields <bfields@fieldses.org> wrote: > On Sun, Jan 15, 2012 at 08:05:23PM +0100, Tigran Mkrtchyan wrote: >> >> This is the result of testing my changes in linux: >> >> $ ./testserver.py --maketree 192.168.178.20:/tmp currentstateid >> ..... >> >> ************************************************** >> CSID1 st_current_stateid.testOpenAndClose : PASS >> CSID2 st_current_stateid.testLockLockU : PASS >> CSID3 st_current_stateid.testOpenWriteClose : PASS >> CSID4 st_current_stateid.testLockWriteLocku : PASS >> CSID5 st_current_stateid.testOpenPutrootfhClose : PASS >> CSID6 st_current_stateid.testCloseNoStateid : PASS >> CSID7 st_current_stateid.testOpenLayoutGet : FAILURE >> OP_LAYOUTGET should return NFS4_OK, instead got >> NFS4ERR_NOTSUPP > > We should fix pynfs41 so no pnfs-dependent tests are run without "pnfs" > being explicitly asked for on the commandline.... Well, echange_id has this information. pynfs have to respect it. Tigran. > > --b. > >> CSID8 st_current_stateid.testOpenSetattr : PASS >> CSID9 st_current_stateid.testOpenFreestateidClose : PA >> >> ************************************************** >> $ >> >> Tigran Mkrtchyan (5): >> client: move current stateid test into a dedicated module >> client: test current stateid processing on PUTROOTFH >> client: test current stateid processing with OPEN+LAYOUTGET in a >> single compound >> client: test current stateid processing with OPEN and SETATTR >> client: test OPEN+FREE_STETEID+CLOSE in a single compound >> >> nfs4.1/server41tests/__init__.py | 1 + >> nfs4.1/server41tests/st_current_stateid.py | 161 ++++++++++++++++++++++++++++ >> nfs4.1/server41tests/st_open.py | 76 ------------- >> 3 files changed, 162 insertions(+), 76 deletions(-) >> create mode 100644 nfs4.1/server41tests/st_current_stateid.py >> >> -- >> 1.7.7.5 >> > -- > To unsubscribe from this list: send the line "unsubscribe linux-nfs" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 0/5] more test on current stateid processing 2012-01-15 19:05 [PATCH 0/5] more test on current stateid processing Tigran Mkrtchyan ` (5 preceding siblings ...) 2012-01-15 20:54 ` [PATCH 0/5] more test on current stateid processing J. Bruce Fields @ 2012-01-17 16:42 ` J. Bruce Fields 2012-01-24 11:53 ` Benny Halevy 7 siblings, 0 replies; 11+ messages in thread From: J. Bruce Fields @ 2012-01-17 16:42 UTC (permalink / raw) To: Tigran Mkrtchyan; +Cc: linux-nfs Applied the pynfs changes, thanks! --b. On Sun, Jan 15, 2012 at 08:05:23PM +0100, Tigran Mkrtchyan wrote: > > This is the result of testing my changes in linux: > > $ ./testserver.py --maketree 192.168.178.20:/tmp currentstateid > ..... > > ************************************************** > CSID1 st_current_stateid.testOpenAndClose : PASS > CSID2 st_current_stateid.testLockLockU : PASS > CSID3 st_current_stateid.testOpenWriteClose : PASS > CSID4 st_current_stateid.testLockWriteLocku : PASS > CSID5 st_current_stateid.testOpenPutrootfhClose : PASS > CSID6 st_current_stateid.testCloseNoStateid : PASS > CSID7 st_current_stateid.testOpenLayoutGet : FAILURE > OP_LAYOUTGET should return NFS4_OK, instead got > NFS4ERR_NOTSUPP > CSID8 st_current_stateid.testOpenSetattr : PASS > CSID9 st_current_stateid.testOpenFreestateidClose : PA > > ************************************************** > $ > > Tigran Mkrtchyan (5): > client: move current stateid test into a dedicated module > client: test current stateid processing on PUTROOTFH > client: test current stateid processing with OPEN+LAYOUTGET in a > single compound > client: test current stateid processing with OPEN and SETATTR > client: test OPEN+FREE_STETEID+CLOSE in a single compound > > nfs4.1/server41tests/__init__.py | 1 + > nfs4.1/server41tests/st_current_stateid.py | 161 ++++++++++++++++++++++++++++ > nfs4.1/server41tests/st_open.py | 76 ------------- > 3 files changed, 162 insertions(+), 76 deletions(-) > create mode 100644 nfs4.1/server41tests/st_current_stateid.py > > -- > 1.7.7.5 > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 0/5] more test on current stateid processing 2012-01-15 19:05 [PATCH 0/5] more test on current stateid processing Tigran Mkrtchyan ` (6 preceding siblings ...) 2012-01-17 16:42 ` J. Bruce Fields @ 2012-01-24 11:53 ` Benny Halevy 2012-01-24 12:18 ` Tigran Mkrtchyan 7 siblings, 1 reply; 11+ messages in thread From: Benny Halevy @ 2012-01-24 11:53 UTC (permalink / raw) To: Tigran Mkrtchyan; +Cc: linux-nfs, bfields On 2012-01-15 21:05, Tigran Mkrtchyan wrote: > This is the result of testing my changes in linux: > > $ ./testserver.py --maketree 192.168.178.20:/tmp currentstateid > ..... > > ************************************************** > CSID1 st_current_stateid.testOpenAndClose : PASS > CSID2 st_current_stateid.testLockLockU : PASS > CSID3 st_current_stateid.testOpenWriteClose : PASS > CSID4 st_current_stateid.testLockWriteLocku : PASS > CSID5 st_current_stateid.testOpenPutrootfhClose : PASS > CSID6 st_current_stateid.testCloseNoStateid : PASS > CSID7 st_current_stateid.testOpenLayoutGet : FAILURE > OP_LAYOUTGET should return NFS4_OK, instead got > NFS4ERR_NOTSUPP I guess this is not against my kernel but rather against Bruce's / mainline? Benny > CSID8 st_current_stateid.testOpenSetattr : PASS > CSID9 st_current_stateid.testOpenFreestateidClose : PA > > ************************************************** > $ > > Tigran Mkrtchyan (5): > client: move current stateid test into a dedicated module > client: test current stateid processing on PUTROOTFH > client: test current stateid processing with OPEN+LAYOUTGET in a > single compound > client: test current stateid processing with OPEN and SETATTR > client: test OPEN+FREE_STETEID+CLOSE in a single compound > > nfs4.1/server41tests/__init__.py | 1 + > nfs4.1/server41tests/st_current_stateid.py | 161 ++++++++++++++++++++++++++++ > nfs4.1/server41tests/st_open.py | 76 ------------- > 3 files changed, 162 insertions(+), 76 deletions(-) > create mode 100644 nfs4.1/server41tests/st_current_stateid.py > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 0/5] more test on current stateid processing 2012-01-24 11:53 ` Benny Halevy @ 2012-01-24 12:18 ` Tigran Mkrtchyan 0 siblings, 0 replies; 11+ messages in thread From: Tigran Mkrtchyan @ 2012-01-24 12:18 UTC (permalink / raw) To: Benny Halevy; +Cc: linux-nfs, bfields On Tue, Jan 24, 2012 at 12:53 PM, Benny Halevy <bhalevy@tonian.com> wrote: > On 2012-01-15 21:05, Tigran Mkrtchyan wrote: >> This is the result of testing my changes in linux: >> >> $ ./testserver.py --maketree 192.168.178.20:/tmp currentstateid >> ..... >> >> ************************************************** >> CSID1 st_current_stateid.testOpenAndClose : PASS >> CSID2 st_current_stateid.testLockLockU : PASS >> CSID3 st_current_stateid.testOpenWriteClose : PASS >> CSID4 st_current_stateid.testLockWriteLocku : PASS >> CSID5 st_current_stateid.testOpenPutrootfhClose : PASS >> CSID6 st_current_stateid.testCloseNoStateid : PASS >> CSID7 st_current_stateid.testOpenLayoutGet : FAILURE >> OP_LAYOUTGET should return NFS4_OK, instead got >> NFS4ERR_NOTSUPP > > I guess this is not against my kernel but rather against Bruce's / mainline? Yes, This is against Bruce's tree. With your I get LAYOUT_TRYLATER :-D Tigran. > > Benny > >> CSID8 st_current_stateid.testOpenSetattr : PASS >> CSID9 st_current_stateid.testOpenFreestateidClose : PA >> >> ************************************************** >> $ >> >> Tigran Mkrtchyan (5): >> client: move current stateid test into a dedicated module >> client: test current stateid processing on PUTROOTFH >> client: test current stateid processing with OPEN+LAYOUTGET in a >> single compound >> client: test current stateid processing with OPEN and SETATTR >> client: test OPEN+FREE_STETEID+CLOSE in a single compound >> >> nfs4.1/server41tests/__init__.py | 1 + >> nfs4.1/server41tests/st_current_stateid.py | 161 ++++++++++++++++++++++++++++ >> nfs4.1/server41tests/st_open.py | 76 ------------- >> 3 files changed, 162 insertions(+), 76 deletions(-) >> create mode 100644 nfs4.1/server41tests/st_current_stateid.py >> ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2012-01-24 12:18 UTC | newest] Thread overview: 11+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-01-15 19:05 [PATCH 0/5] more test on current stateid processing Tigran Mkrtchyan 2012-01-15 19:05 ` [PATCH 1/5] client: move current stateid test into a dedicated module Tigran Mkrtchyan 2012-01-15 19:05 ` [PATCH 2/5] client: test current stateid processing on PUTROOTFH Tigran Mkrtchyan 2012-01-15 19:05 ` [PATCH 3/5] client: test current stateid processing with OPEN+LAYOUTGET in a single compound Tigran Mkrtchyan 2012-01-15 19:05 ` [PATCH 4/5] client: test current stateid processing with OPEN and SETATTR Tigran Mkrtchyan 2012-01-15 19:05 ` [PATCH 5/5] client: test OPEN+FREE_STETEID+CLOSE in a single compound Tigran Mkrtchyan 2012-01-15 20:54 ` [PATCH 0/5] more test on current stateid processing J. Bruce Fields 2012-01-15 22:31 ` Tiramisu Mokka 2012-01-17 16:42 ` J. Bruce Fields 2012-01-24 11:53 ` Benny Halevy 2012-01-24 12:18 ` Tigran Mkrtchyan
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).