From patchwork Thu Mar 3 09:07:09 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mi Jinlong X-Patchwork-Id: 605551 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter1.kernel.org (8.14.4/8.14.3) with ESMTP id p2396Q6J004080 for ; Thu, 3 Mar 2011 09:06:27 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751612Ab1CCJGY (ORCPT ); Thu, 3 Mar 2011 04:06:24 -0500 Received: from cn.fujitsu.com ([222.73.24.84]:51477 "EHLO song.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1750783Ab1CCJGU (ORCPT ); Thu, 3 Mar 2011 04:06:20 -0500 Received: from tang.cn.fujitsu.com (tang.cn.fujitsu.com [10.167.250.3]) by song.cn.fujitsu.com (Postfix) with ESMTP id E2BAD17011F; Thu, 3 Mar 2011 17:06:17 +0800 (CST) Received: from mailserver.fnst.cn.fujitus.com (tang.cn.fujitsu.com [127.0.0.1]) by tang.cn.fujitsu.com (8.14.3/8.13.1) with ESMTP id p2390Q2k020619; Thu, 3 Mar 2011 17:00:26 +0800 Received: from [127.0.0.1] ([10.167.225.24]) by mailserver.fnst.cn.fujitus.com (Lotus Domino Release 8.5.1FP4) with ESMTP id 2011030317050849-273124 ; Thu, 3 Mar 2011 17:05:08 +0800 Message-ID: <4D6F5A3D.6050000@cn.fujitsu.com> Date: Thu, 03 Mar 2011 17:07:09 +0800 From: Mi Jinlong User-Agent: Thunderbird 2.0.0.9 (Windows/20071031) MIME-Version: 1.0 To: "J. Bruce Fields" CC: NFS Subject: Re: [PATCH 02/17] CLNT: test lease period of unconfirmed record References: <4D6DE26C.3010103@cn.fujitsu.com> <4D6DE34A.2020404@cn.fujitsu.com> <20110303060225.GD7724@fieldses.org> In-Reply-To: <20110303060225.GD7724@fieldses.org> X-MIMETrack: Itemize by SMTP Server on mailserver/fnst(Release 8.5.1FP4|July 25, 2010) at 2011-03-03 17:05:08, Serialize by Router on mailserver/fnst(Release 8.5.1FP4|July 25, 2010) at 2011-03-03 17:05:09, Serialize complete at 2011-03-03 17:05:09 Sender: linux-nfs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-nfs@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter1.kernel.org [140.211.167.41]); Thu, 03 Mar 2011 09:06:27 +0000 (UTC) ================================= From 3512a8141710190c92a4b836854f1df755b8e802 Mon Sep 17 00:00:00 2001 From: Mi Jinlong Date: Thu, 3 Mar 2011 19:58:22 +0800 Subject: [PATCH] CLNT: test lease period of unconfirmed record Signed-off-by: Mi Jinlong --- nfs4.1/server41tests/st_exchange_id.py | 36 ++++++++++++++++++++++++++++++++ 1 files changed, 36 insertions(+), 0 deletions(-) diff --git a/nfs4.1/server41tests/st_exchange_id.py b/nfs4.1/server41tests/st_exchange_id.py index b5e34e6..3986759 100644 --- a/nfs4.1/server41tests/st_exchange_id.py +++ b/nfs4.1/server41tests/st_exchange_id.py @@ -1,10 +1,15 @@ from nfs4_const import * import nfs4_ops as op +import time from environment import check, fail from nfs4_type import * from rpc import RPCAcceptError, GARBAGE_ARGS, RPCTimeout from nfs4lib import NFS4Error, hash_oids, encrypt_oids +def _getleasetime(sess): + res = sess.compound([op.putrootfh(), op.getattr(1 << FATTR4_LEASE_TIME)]) + return res.resarray[-1].obj_attributes[FATTR4_LEASE_TIME] + def testSupported(t, env): """Do a simple EXCHANGE_ID - no flags @@ -436,3 +441,34 @@ def testNotOnlyOp(t, env): res = c.compound([op.exchange_id(owner, 0, protect, [c.impl_id]), op.putrootfh()]) # per draft 21 18.35.3, server MUST return NFS4ERR_NOT_ONLY_OP check(res, NFS4ERR_NOT_ONLY_OP) + +def testLeasePeriod(t, env): + """Any unconfirmed record that is not confirmed within + a lease period SHOULD be removed. + + FLAGS: exchange_id all + CODE: EID9 + """ + c1 = env.c1.new_client("%s_1" % env.testname(t)) + c2 = env.c1.new_client("%s_2" % env.testname(t)) + + # Get server's lease time + c3 = env.c1.new_client("%s_3" % env.testname(t)) + sess = c3.create_session() + lease = _getleasetime(sess) + + # CREATE_SESSION + chan_attrs = channel_attrs4(0,8192,8192,8192,128,8,[]) + time.sleep(lease - 10) + # Inside lease period, create_session will success. + res1 = c1.c.compound([op.create_session(c1.clientid, c1.seqid, 0, + chan_attrs, chan_attrs, + 123, [])], None) + check(res1) + + time.sleep(lease + 10) + # After lease period, create_session will get error NFS4ERR_STALE_CLIENTID + res2 = c2.c.compound([op.create_session(c2.clientid, c2.seqid, 0, + chan_attrs, chan_attrs, + 123, [])], None) + check(res2, NFS4ERR_STALE_CLIENTID)