aboutsummaryrefslogtreecommitdiffstats
path: root/fs/nfs
diff options
context:
space:
mode:
authorTrond Myklebust <Trond.Myklebust@netapp.com>2012-12-15 15:21:52 -0500
committerTrond Myklebust <Trond.Myklebust@netapp.com>2012-12-15 15:21:52 -0500
commit8e63b6a8adabb0551124c3b78f7f5f36912c3728 (patch)
tree47043a752823e4aba91f906f95d3c5006f759202 /fs/nfs
parente8794440849d1d15fa11251ef1622e6160614874 (diff)
NFSv4.1: Move the RPC timestamp out of the slot.
Shave a few bytes off the slot table size by moving the RPC timestamp into the sequence results. Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/nfs')
-rw-r--r--fs/nfs/nfs4proc.c14
-rw-r--r--fs/nfs/nfs4session.c3
-rw-r--r--fs/nfs/nfs4session.h1
3 files changed, 8 insertions, 10 deletions
diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c
index 9003b8f6b77..afb428e63b5 100644
--- a/fs/nfs/nfs4proc.c
+++ b/fs/nfs/nfs4proc.c
@@ -419,7 +419,6 @@ static int nfs41_sequence_done(struct rpc_task *task, struct nfs4_sequence_res *
{
struct nfs4_session *session;
struct nfs4_slot *slot;
- unsigned long timestamp;
struct nfs_client *clp;
int ret = 1;
@@ -444,9 +443,8 @@ static int nfs41_sequence_done(struct rpc_task *task, struct nfs4_sequence_res *
case 0:
/* Update the slot's sequence and clientid lease timer */
++slot->seq_nr;
- timestamp = slot->renewal_time;
clp = session->clp;
- do_renew_lease(clp, timestamp);
+ do_renew_lease(clp, res->sr_timestamp);
/* Check sequence flags */
if (res->sr_status_flags != 0)
nfs4_schedule_lease_recovery(clp);
@@ -473,10 +471,11 @@ static int nfs41_sequence_done(struct rpc_task *task, struct nfs4_sequence_res *
* Could this slot have been previously retired?
* If so, then the server may be expecting seq_nr = 1!
*/
- if (slot->seq_nr == 1)
- break;
- slot->seq_nr = 1;
- goto retry_nowait;
+ if (slot->seq_nr != 1) {
+ slot->seq_nr = 1;
+ goto retry_nowait;
+ }
+ break;
case -NFS4ERR_SEQ_FALSE_RETRY:
++slot->seq_nr;
goto retry_nowait;
@@ -567,6 +566,7 @@ int nfs41_setup_sequence(struct nfs4_session *session,
slot->slot_nr, slot->seq_nr);
res->sr_slot = slot;
+ res->sr_timestamp = jiffies;
res->sr_status_flags = 0;
/*
* sr_status is only set in decode_sequence, and so will remain
diff --git a/fs/nfs/nfs4session.c b/fs/nfs/nfs4session.c
index 1e6c87c443a..0e1cc1f4e51 100644
--- a/fs/nfs/nfs4session.c
+++ b/fs/nfs/nfs4session.c
@@ -143,7 +143,6 @@ struct nfs4_slot *nfs4_alloc_slot(struct nfs4_slot_table *tbl)
if (slotid > tbl->highest_used_slotid ||
tbl->highest_used_slotid == NFS4_NO_SLOT)
tbl->highest_used_slotid = slotid;
- ret->renewal_time = jiffies;
ret->generation = tbl->generation;
out:
@@ -228,9 +227,9 @@ static bool nfs41_assign_slot(struct rpc_task *task, void *pslot)
if (nfs4_session_draining(tbl->session) && !args->sa_privileged)
return false;
- slot->renewal_time = jiffies;
slot->generation = tbl->generation;
args->sa_slot = slot;
+ res->sr_timestamp = jiffies;
res->sr_slot = slot;
res->sr_status_flags = 0;
res->sr_status = 1;
diff --git a/fs/nfs/nfs4session.h b/fs/nfs/nfs4session.h
index 04f834cab16..d17b08091d4 100644
--- a/fs/nfs/nfs4session.h
+++ b/fs/nfs/nfs4session.h
@@ -19,7 +19,6 @@ struct nfs4_slot {
struct nfs4_slot_table *table;
struct nfs4_slot *next;
unsigned long generation;
- unsigned long renewal_time;
u32 slot_nr;
u32 seq_nr;
};