Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37;
MIME-Version: 1.0
References: <20230917232646.30810-1-trondmy@kernel.org>
In-Reply-To: <20230917232646.30810-1-trondmy@kernel.org>
From:   Olga Kornievskaia <aglo@umich.edu>
Date:   Sun, 17 Sep 2023 22:55:16 -0400
Message-ID: <CAN-5tyFFNY30PzjAD58dweDYfLKS04NgkOryF93Pu22adbLr_w@mail.gmail.com>
Subject: Re: [PATCH] Revert "SUNRPC dont update timeout value on connection reset"
To:     trondmy@kernel.org
Cc:     Anna Schumaker <Anna.Schumaker@netapp.com>,
        linux-nfs@vger.kernel.org, Olga Kornievskaia <kolga@netapp.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Sun, Sep 17, 2023 at 7:38=E2=80=AFPM <trondmy@kernel.org> wrote:
>
> From: Trond Myklebust <trond.myklebust@hammerspace.com>
>
> This reverts commit 88428cc4ae7abcc879295fbb19373dd76aad2bdd.
>
> The problem this commit is intended to fix was comprehensively fixed
> in commit 7de62bc09fe6 ("SUNRPC dont update timeout value on connection
> reset").
> Since then, this commit has been preventing the correct timeout of soft
> mounted requests.

And if we revert this commit then we get back the problem that when
the server RSTs the connection between the timeouts then the client
waits double the time (instead of the correct time).

> Cc: stable@vger.kernel.org # 5.9.x: 09252177d5f9: SUNRPC: Handle major ti=
meout in xprt_adjust_timeout()
> Cc: stable@vger.kernel.org # 5.9.x: 7de62bc09fe6: SUNRPC dont update time=
out value on connection
> reset
> Cc: stable@vger.kernel.org # 5.9.x
> Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
> ---
>  net/sunrpc/clnt.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/net/sunrpc/clnt.c b/net/sunrpc/clnt.c
> index 5a7de7e55548..7f533c1041a4 100644
> --- a/net/sunrpc/clnt.c
> +++ b/net/sunrpc/clnt.c
> @@ -2476,8 +2476,7 @@ call_status(struct rpc_task *task)
>                 goto out_exit;
>         }
>         task->tk_action =3D call_encode;
> -       if (status !=3D -ECONNRESET && status !=3D -ECONNABORTED)
> -               rpc_check_timeout(task);
> +       rpc_check_timeout(task);
>         return;
>  out_exit:
>         rpc_call_rpcerror(task, status);
> --
> 2.41.0
>