Re: [PATCH 1/2] NFSv4: Express delegation limit in units of pages

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux