On 25 Mar 2025, at 18:35, trondmy@xxxxxxxxxx wrote:
> From: Trond Myklebust <trond.myklebust@xxxxxxxxxxxxxxx>
>
> If someone calls nfs_mark_client_ready(clp, status) with a negative
> value for status, then that should signal that the nfs_client is no
> longer valid.
>
> Signed-off-by: Trond Myklebust <trond.myklebust@xxxxxxxxxxxxxxx>
> ---
> fs/nfs/nfs4state.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/nfs/nfs4state.c b/fs/nfs/nfs4state.c
> index f1f7eaa97973..272d2ebdae0f 100644
> --- a/fs/nfs/nfs4state.c
> +++ b/fs/nfs/nfs4state.c
> @@ -1403,7 +1403,7 @@ int nfs4_schedule_stateid_recovery(const struct nfs_server *server, struct nfs4_
> dprintk("%s: scheduling stateid recovery for server %s\n", __func__,
> clp->cl_hostname);
> nfs4_schedule_state_manager(clp);
> - return 0;
> + return clp->cl_cons_state < 0 ? clp->cl_cons_state : 0;
> }
> EXPORT_SYMBOL_GPL(nfs4_schedule_stateid_recovery);
>
> --
> 2.49.0
Reviewed-by: Benjamin Coddington <bcodding@xxxxxxxxxx>
Ben
