* [PATCH] nfs4.1: check whether old session is discard for EID5f
@ 2013-12-24 2:58 Kinglong Mee
2014-01-06 20:42 ` J. Bruce Fields
0 siblings, 1 reply; 2+ messages in thread
From: Kinglong Mee @ 2013-12-24 2:58 UTC (permalink / raw)
To: J. Bruce Fields; +Cc: linux-nfs
Signed-off-by: Kinglong Mee <kinglongmee@gmail.com>
---
nfs4.1/server41tests/st_exchange_id.py | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/nfs4.1/server41tests/st_exchange_id.py b/nfs4.1/server41tests/st_exchange_id.py
index c31af95..d2c0c97 100644
--- a/nfs4.1/server41tests/st_exchange_id.py
+++ b/nfs4.1/server41tests/st_exchange_id.py
@@ -244,7 +244,12 @@ def testNoUpdate101(t, env):
# Old session state should not be discarded until confirm:
res = sess1.compound([])
check(res)
- # FIXME - more checks here
+
+ # Old session state should be discarded after confirm:
+ sess2 = c2.create_session()
+ check(res)
+ res = sess1.compound([])
+ check(res, NFS4ERR_BADSESSION)
def testNoUpdate101b(t, env):
"""
--
1.8.4.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] nfs4.1: check whether old session is discard for EID5f
2013-12-24 2:58 [PATCH] nfs4.1: check whether old session is discard for EID5f Kinglong Mee
@ 2014-01-06 20:42 ` J. Bruce Fields
0 siblings, 0 replies; 2+ messages in thread
From: J. Bruce Fields @ 2014-01-06 20:42 UTC (permalink / raw)
To: Kinglong Mee; +Cc: linux-nfs
On Tue, Dec 24, 2013 at 10:58:50AM +0800, Kinglong Mee wrote:
>
> Signed-off-by: Kinglong Mee <kinglongmee@gmail.com>
Applied and pushed out with the previous pynfs patch; let me know if you
see anything missing in my pynfs repo.
--b.
> ---
> nfs4.1/server41tests/st_exchange_id.py | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/nfs4.1/server41tests/st_exchange_id.py b/nfs4.1/server41tests/st_exchange_id.py
> index c31af95..d2c0c97 100644
> --- a/nfs4.1/server41tests/st_exchange_id.py
> +++ b/nfs4.1/server41tests/st_exchange_id.py
> @@ -244,7 +244,12 @@ def testNoUpdate101(t, env):
> # Old session state should not be discarded until confirm:
> res = sess1.compound([])
> check(res)
> - # FIXME - more checks here
> +
> + # Old session state should be discarded after confirm:
> + sess2 = c2.create_session()
> + check(res)
> + res = sess1.compound([])
> + check(res, NFS4ERR_BADSESSION)
>
> def testNoUpdate101b(t, env):
> """
> --
> 1.8.4.2
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-01-06 20:42 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-24 2:58 [PATCH] nfs4.1: check whether old session is discard for EID5f Kinglong Mee
2014-01-06 20:42 ` J. Bruce Fields
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).