All of lore.kernel.org
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <krzk@kernel.org>
To: David Howells <dhowells@redhat.com>
Cc: Trond Myklebust <trond.myklebust@hammerspace.com>,
	Anna Schumaker <anna.schumaker@netapp.com>,
	linux-nfs@vger.kernel.org,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Al Viro <viro@zeniv.linux.org.uk>,
	Scott Mayhew <smayhew@redhat.com>, Arnd Bergmann <arnd@arndb.de>
Subject: Re: [BISECT BUG] NFS v4 root not working after 6d972518b821 ("NFS: Add fs_context support.")
Date: Fri, 17 Jan 2020 15:40:55 +0100	[thread overview]
Message-ID: <20200117144055.GB3215@pi3> (raw)
In-Reply-To: <433863.1579270803@warthog.procyon.org.uk>

On Fri, Jan 17, 2020 at 02:20:03PM +0000, David Howells wrote:
> You seem to be running afoul of the check here:
> 
> 	case Opt_minorversion:
> 		if (result.uint_32 > NFS4_MAX_MINOR_VERSION)
> 			goto out_of_bounds;
> 		ctx->minorversion = result.uint_32;
> 		break;
> 
> which would seem to indicate that the mount process is supplying
> minorversion=X as an option.  Can you modify your kernel to print param->key
> and param->string at the top of nfs_fs_context_parse_param()?  Adding
> something like:
> 
> 	pr_notice("NFSOP '%s=%s'\n", param->key, param->string);
> 
> will likely suffice unless you're directly driving the new mount API - in
> which case param->string might be things other than a string, but that's
> unlikely.  It might also be NULL, but printk should handle that.

The output:

NFS-Mount: 192.168.1.10:/srv/nfs/odroidhc1
Waiting 10 seconds for device /dev/nfs ...
[   14.652366] random: crng init done
Mount cmd:
mount.nfs4 -o vers=4,nolock 192.168.1.10:/srv/nfs/odroidhc1 /new_root
[   22.938314] NFSOP 'source=192.168.1.10:/srv/nfs/odroidhc1'
[   22.942638] NFSOP 'nolock=(null)'
[   22.945772] NFSOP 'vers=4.2'
[   22.948660] NFSOP 'addr=192.168.1.10'
[   22.952350] NFSOP 'clientaddr=192.168.1.12'
[   22.956831] NFS4: Couldn't follow remote path
[   22.971001] NFSOP 'source=192.168.1.10:/srv/nfs/odroidhc1'
[   22.975217] NFSOP 'nolock=(null)'
[   22.978444] NFSOP 'vers=4'
[   22.981265] NFSOP 'minorversion=1'
[   22.984513] NFS: Value for 'minorversion' out of range
mount.nfs4: Numerical result out of range
:: running cleanup hook [udev]
ERROR: Failed to mount the real root device.
Bailing out, you are on your own. Good luck.

sh: can't access tty; job control turned off

Best regards,
Krzysztof


  reply	other threads:[~2020-01-17 14:41 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-01-16 11:33 [BISECT BUG] NFS v4 root not working after 6d972518b821 ("NFS: Add fs_context support.") Krzysztof Kozlowski
2020-01-17  0:49 ` Scott Mayhew
2020-01-17 11:53   ` Krzysztof Kozlowski
2020-01-17 12:54 ` David Howells
2020-01-17 13:16   ` Krzysztof Kozlowski
2020-01-17 14:08   ` David Howells
2020-01-17 14:36     ` Krzysztof Kozlowski
2020-01-17 14:20 ` David Howells
2020-01-17 14:40   ` Krzysztof Kozlowski [this message]
2020-01-17 15:12   ` David Howells
2020-01-17 15:17     ` Trond Myklebust
2020-01-17 15:32       ` Scott Mayhew
2020-01-17 15:44     ` David Howells
2020-01-17 15:48   ` [PATCH] nfs: Return EINVAL rather than ERANGE for mount parse errors David Howells
2020-01-17 15:55   ` [PATCH v2] " David Howells
2020-01-17 16:51     ` Krzysztof Kozlowski
2020-01-17 17:18     ` David Howells
2020-01-17 20:21       ` Schumaker, Anna
2020-01-17 21:12       ` David Howells

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200117144055.GB3215@pi3 \
    --to=krzk@kernel.org \
    --cc=anna.schumaker@netapp.com \
    --cc=arnd@arndb.de \
    --cc=dhowells@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-nfs@vger.kernel.org \
    --cc=smayhew@redhat.com \
    --cc=trond.myklebust@hammerspace.com \
    --cc=viro@zeniv.linux.org.uk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.