On Mon, Sep 7, 2015 at 5:18 AM, Mkrtchyan, Tigran <tigran.mkrtchyan@xxxxxxx> wrote: > > > ----- Original Message ----- >> From: "Trond Myklebust" <trond.myklebust@xxxxxxxxxxxxxxx> >> To: linux-nfs@xxxxxxxxxxxxxxx >> Sent: Sunday, September 6, 2015 1:06:57 AM >> Subject: [PATCH 1/2] NFSv4: Express delegation limit in units of pages > >> Since we're tracking modifications to the page cache on a per-page >> basis, it makes sense to express the limit to how much we may cache >> in units of pages. >> >> Signed-off-by: Trond Myklebust <trond.myklebust@xxxxxxxxxxxxxxx> >> --- >> fs/nfs/delegation.c | 5 +++-- >> fs/nfs/delegation.h | 2 +- >> fs/nfs/nfs4xdr.c | 16 ++++++++++------ >> include/linux/nfs_xdr.h | 2 +- >> 4 files changed, 15 insertions(+), 10 deletions(-) >> >> diff --git a/fs/nfs/delegation.c b/fs/nfs/delegation.c >> index 029d688a969f..4817f66c8d47 100644 >> --- a/fs/nfs/delegation.c >> +++ b/fs/nfs/delegation.c >> @@ -175,7 +175,8 @@ void nfs_inode_reclaim_delegation(struct inode *inode, >> struct rpc_cred *cred, >> if (delegation->inode != NULL) { >> nfs4_stateid_copy(&delegation->stateid, &res->delegation); >> delegation->type = res->delegation_type; >> - delegation->maxsize = res->maxsize; >> + delegation->pagemod_limit = res->pagemod_limit; >> + ; > > is this extra line with semicolon intended or a typo? > Oops. Definitely a typo. Thanks for spotting... Cheers Trond -- 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