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

[Benny Halevy]

On Feb. 04, 2008, 20:59 +0200, Ricardo Labiaga <ricardo.labiaga at netapp.com> wrote:
> 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.

Done.
(Also moved 636db99 nfs41: Add missing call to destroy session in nfs4_get_sb.
as it depends on linux/sunrpc/bc_xprt.h)

Benny

> 
> - 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);
> _______________________________________________
> pNFS mailing list
> pNFS at linux-nfs.org
> http://linux-nfs.org/cgi-bin/mailman/listinfo/pnfs