Re: [PATCH v2 0/4] nfsd: don't let nfs4_file pin down the inode when it has no open state

["J. Bruce Fields" <bfields@fieldses.org>]

On Wed, Jul 23, 2014 at 04:17:37PM -0400, Jeff Layton wrote:
> This is a port of the patches that Trond sent the other day onto the
> current tip of Bruce's for-3.17 branch. It basically changes how
> nfs4_file objects are hashed. Instead of using the inode pointer (and
> pinning down an inode in the process), it uses the filehandle. This
> allows us to avoid taking an inode reference directly for the nfs4_file.
> With this, they're only taken by virtue of the files in the fi_fds
> array.

Looks OK to me.  I'll give it a day in case Christoph or someone spots a
problem.

--b.

> 
> Jeff Layton (1):
>   nfsd: Do not let nfs4_file pin the struct inode
> 
> Trond Myklebust (3):
>   nfsd: Store the filehandle with the struct nfs4_file
>   nfsd: Use the filehandle to look up the struct nfs4_file instead of
>     inode
>   nfsd: nfs4_check_fh - make it actually check the filehandle
> 
>  fs/nfsd/nfs4state.c | 86 +++++++++++++++++++++++++++++++++--------------------
>  fs/nfsd/state.h     |  2 +-
>  2 files changed, 54 insertions(+), 34 deletions(-)
> 
> -- 
> 1.9.3
>