Received-SPF: pass (google.com: domain of linux-nfs+bounces-1825-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Precedence: bulk
MIME-Version: 1.0
References: <20240207182912.30981-1-olga.kornievskaia@gmail.com> <be83ac24a9d17fc78590d4b182afd3f4e6c1e8a7.camel@hammerspace.com>
In-Reply-To: <be83ac24a9d17fc78590d4b182afd3f4e6c1e8a7.camel@hammerspace.com>
From: Olga Kornievskaia <olga.kornievskaia@gmail.com>
Date: Wed, 7 Feb 2024 14:51:05 -0500
Message-ID: <CAN-5tyFTx4bUvuF627E_2x9Kw2h9tccqU-7KfCtJHyFazTTLYg@mail.gmail.com>
Subject: Re: [PATCH 1/1] NFSv4.1/pnfs: error gracefully on partial pnfs layout
To: Trond Myklebust <trondmy@hammerspace.com>
Cc: "anna.schumaker@netapp.com" <anna.schumaker@netapp.com>, 
	"linux-nfs@vger.kernel.org" <linux-nfs@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Wed, Feb 7, 2024 at 2:12=E2=80=AFPM Trond Myklebust <trondmy@hammerspace=
.com> wrote:
>
> On Wed, 2024-02-07 at 13:29 -0500, Olga Kornievskaia wrote:
> > From: Olga Kornievskaia <kolga@netapp.com>
> >
> > Currently, if the server returns a partial layout, the client gets
> > stuck asking for a layout indefinitely. Until we add support for
> > partial layouts, treat partial layout as layout unavailable error.
> >
> > Signed-off-by: Olga Kornievskaia <kolga@netapp.com>
> > ---
> >  fs/nfs/nfs4proc.c | 6 ++++++
> >  1 file changed, 6 insertions(+)
> >
> > diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c
> > index dae4c1b6cc1c..108bc7f3e8c2 100644
> > --- a/fs/nfs/nfs4proc.c
> > +++ b/fs/nfs/nfs4proc.c
> > @@ -9790,6 +9790,12 @@ nfs4_proc_layoutget(struct nfs4_layoutget
> > *lgp,
> >       if (status !=3D 0)
> >               goto out;
> >
> > +     /* Since client does not support partial pnfs layout, then
> > treat
> > +      * getting a partial layout as LAYOUTUNAVAILABLE error
> > +      */
> > +     if (lgp->args.range.length !=3D lgp->res.range.length)
> > +             task->tk_status =3D -NFS4ERR_LAYOUTUNAVAILABLE;
>
>
> I think this case is better handled by allowing the caller to set lgp-
> >args.minlength to an appropriate minimum value.

I do not understand what this suggestion means. What I can think of is
that the caller would set an appropriate minimum value and the code
here would check that the result is at least as large?

If so, can you explain why that's more desirable? Seems to me it'd be
more lines for something that would be removed later?

>
> > +
> >       if (task->tk_status < 0) {
> >               exception->retry =3D 1;
> >               status =3D nfs4_layoutget_handle_exception(task, lgp,
> > exception);
>
> --
> Trond Myklebust
> Linux NFS client maintainer, Hammerspace
> trond.myklebust@hammerspace.com
>
>