linux-kernel - Re: [PATCH] nfs: Don't return referenced delegations

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <4FA345DA4F4AE44899BD2B03EEEC2FA91192E4A7@SACEXCMBX04-PRD.hq.netapp.com>
Date:	Fri, 14 Dec 2012 17:45:45 +0000
From:	"Myklebust, Trond" <Trond.Myklebust@...app.com>
To:	Yanchuan Nian <ycnian@...il.com>
CC:	"linux-nfs@...r.kernel.org" <linux-nfs@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] nfs: Don't return referenced delegations

On Fri, 2012-12-14 at 20:53 +0800, Yanchuan Nian wrote:
> 
> 
> 2012/12/13 Myklebust, Trond <Trond.Myklebust@...app.com>
>         On Thu, 2012-12-13 at 14:37 +0800, ycnian@...il.com wrote:
>         > From: Yanchuan Nian <ycnian@...il.com>
>         >
>         > The client returns unreferenced delegations in state
>         management. It scans all
>         > delegations and tests the NFS_DELEGATION_REFERENCED flag. if
>         this flag is not
>         > set, NFS_DELEGATION_RETURN will be set and the delegation
>         will be returned.
>         > But unfortunately NFS_DELEGATION_REFERENCED is cleared after
>         the testing, so
>         > delegations which are still being used will be returned in
>         the next state
>         > management.
>         >
>         > Signed-off-by: Yanchuan Nian <ycnian@...il.com>
>         > ---
>         >  fs/nfs/delegation.c |    2 +-
>         >  1 files changed, 1 insertions(+), 1 deletions(-)
>         >
>         > diff --git a/fs/nfs/delegation.c b/fs/nfs/delegation.c
>         > index 81c5eec..485e8c0 100644
>         > --- a/fs/nfs/delegation.c
>         > +++ b/fs/nfs/delegation.c
>         > @@ -506,7 +506,7 @@ static void
>         nfs_mark_return_unreferenced_delegations(struct nfs_server
>         *server)
>         >       struct nfs_delegation *delegation;
>         >
>         >       list_for_each_entry_rcu(delegation,
>         &server->delegations, super_list) {
>         > -             if
>         (test_and_clear_bit(NFS_DELEGATION_REFERENCED,
>         &delegation->flags))
>         > +             if (test_bit(NFS_DELEGATION_REFERENCED,
>         &delegation->flags))
>         >                       continue;
>         >               nfs_mark_return_delegation(server,
>         delegation);
>         >       }
>         
>         
>         The clearing of the bit here is 100% intentional. This is a
>         variant on
>         basic mark-and-sweep garbage collection where if the bit
>         hasn't been set
>         again the next time we scan, then we assume the delegation
>         isn't being
>         used.
>  
> Oh, now I know. If the client gets an open delegation from the server
> without
> any operations in the following few minutes, the delegation will be
> returned.
> After that, if the client wants to read data from the server, it has
> to
> revalidate local cache first. In this situation, the client can't get
> benefit 
> from delegation. RFC 3530 says The server will recall a delegation
> if there are conflicting operations from another client. So why
> doesn't the
> client hold the delegation until receiving the recall request? Is
> there some 
> special reason? I am new to NFS and sorry to trouble you.

It's about resource management. Both the client and the server need to
keep state in memory for each delegation that is held. What we've seen
is servers running out of state if the client is too greedy about
holding onto delegations that aren't being used. In theory the server
could start recalling those delegations, but how does it know which ones
are still important to the client?

For that reason, we try to be good NFS citizens by returning delegations
that are not being used by processes. The scan is done once every lease
period (so typically once a minute). If a delegation hasn't been used in
that lease period, then it is returned.

-- 
Trond Myklebust
Linux NFS client maintainer

NetApp
Trond.Myklebust@...app.com
www.netapp.com