Re: [PATCH 1/3] nfs41: Mark stateids in need of reclaim if state manager gets stale clientid

[Trond Myklebust <Trond.Myklebust@netapp.com>]

On Sat, 2009-12-05 at 12:11 -0800, Ricardo Labiaga wrote: 
> The state manager was not marking the stateids as needing to be reclaimed
> after reestablishing the clientid.
> 
> Signed-off-by: Ricardo Labiaga <Ricardo.Labiaga@netapp.com>
> ---
>  fs/nfs/nfs4state.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/fs/nfs/nfs4state.c b/fs/nfs/nfs4state.c
> index 630199d..ae90df8 100644
> --- a/fs/nfs/nfs4state.c
> +++ b/fs/nfs/nfs4state.c
> @@ -1157,6 +1157,7 @@ static void nfs4_session_recovery_handle_error(struct nfs_client *clp, int err)
>  	case -NFS4ERR_STALE_CLIENTID:
>  		set_bit(NFS4CLNT_LEASE_EXPIRED, &clp->cl_state);
>  		set_bit(NFS4CLNT_SESSION_SETUP, &clp->cl_state);
> +		nfs4_state_start_reclaim_reboot(clp);
>  	}
>  }
>  

So, why do we need a special nfs4_session_recovery_handle_error() that
just mirrors the existing nfs4_recovery_handle_error().

Please just get rid of it...

Trond