[pnfs] [PATCH 2/2] nfs41: Release backchannel resources associated with session

Ricardo Labiaga ricardo.labiaga at netapp.com
Mon Feb 4 13:59:38 EST 2008


Benny, 

xprt_destroy_backchannel() lives in
net/sunrpc/xprt_destroy_backchannel.c which has not yet been pushed to
the nfs41 branch.  You may want to apply this patch to the
rlabiaga-bc-response-v1 branch instead.

- ricardo


On Mon, 2008-02-04 at 19:44 +0200, Benny Halevy wrote:
> Applied onto the nfs41 branch
> 
> Benny
> 
> On Jan. 31, 2008, 13:58 +0200, Ricardo Labiaga <ricardo.labiaga at netapp.com> wrote:
> > Frees the preallocated backchannel resources that are associated with
> > this session when the session is destroyed.
> > 
> > Signed-off-by: Ricardo Labiaga <ricardo.labiaga at netapp.com>
> > ---
> >  fs/nfs/super.c |    9 +++++++++
> >  1 files changed, 9 insertions(+), 0 deletions(-)
> > 
> > diff --git a/fs/nfs/super.c b/fs/nfs/super.c
> > index e3379fb..436f82f 100644
> > --- a/fs/nfs/super.c
> > +++ b/fs/nfs/super.c
> > @@ -1732,6 +1732,11 @@ static int nfs4_get_sb(struct file_system_type *fs_type,
> >  		switch (server->nfs_client->cl_minorversion) {
> >  		case 1:
> >  			if (server->session) {
> > +				dprintk("%s Destroy backchannel for xprt%p\n",
> > +					__FUNCTION__, server->client->cl_xprt);
> > +				xprt_destroy_backchannel(
> > +					server->client->cl_xprt,
> > +					NFS41_BC_MIN_CALLBACKS);
> >  				dprintk("%s Destroy session %p/%p\n",
> >  					__FUNCTION__, server->session, server);
> >  				nfs4_proc_destroy_session(server);
> > @@ -1790,6 +1795,10 @@ static void nfs4_kill_super(struct super_block *sb)
> >  	switch (server->nfs_client->cl_minorversion) {
> >  	case 1:
> >  		if (server->session) {
> > +			dprintk("%s Destroy backchannel for xprt%p\n",
> > +				__FUNCTION__, server->client->cl_xprt);
> > +			xprt_destroy_backchannel(server->client->cl_xprt,
> > +				NFS41_BC_MIN_CALLBACKS);
> >  			dprintk("%s Destroy session %p/%p\n",
> >  				__FUNCTION__, server->session, server);
> >  			nfs4_proc_destroy_session(server);
> 


More information about the pNFS mailing list