Re: Failure to fallback to nfsd-v3 (?)

J. Bruce Fields
2010-02-08T18:30:48Z

On Sun, Feb 07, 2010 at 03:31:03AM -0500, Christoph Hellwig wrote:
> On Sun, Feb 07, 2010 at 05:23:49PM +0900, OGAWA Hirofumi wrote:
> > "J. Bruce Fields" <bfields@fieldses.org> writes:
> >
> > >> And the following commit seems to change the behavior.
> > >>
> > >> [nfsd4: fix error return when pseudoroot missing]
> > >> f39bde24b275ddc45df1ed835725b609e178c7a0
> > >>
> > >> Well, anyway, is this a expected behavior change, or something bug?
> > >
> > > It's expected. I'd recommend turning off nfsv4 on the server (add "-N4"
> > > to the rpc.nfsd commandline) for now.
> >
> > This looks like the silent user visible change. So, it would be better
> > to add more comment at least in changelog.
>
> Or rather it should be fixed. We should not silently break existing
> and probably rather common setups.

It's really a bug that we're returning an error at all from this point
in the code, so the new client mount code includes a workaround for this
case, added with the code that attempts v4 first by default.

The error we're returning was also wrong. Unfortunately, my patch to
fix the error went in at around the same time the new mount code went
into nfs-utils, and we communicated poorly about the change.

So:
- I'm reverting the server patch. I'd like to reinstate it
eventually, depending on how widespread the new mount code
was.
- We'll also make the mount workaround more robust.
- Most importantly, we need to stamp out the cases where the
server hits this error.

--b.