On Fri, Jan 17, 2020 at 03:55:09PM +0000, David Howells wrote: > commit b9423c912b770e5b9e4228d90da92b6a69693d8e > Author: David Howells <dhowells@xxxxxxxxxx> > Date: Fri Jan 17 15:37:46 2020 +0000 > > nfs: Return EINVAL rather than ERANGE for mount parse errors > > Return EINVAL rather than ERANGE for mount parse errors as the userspace > mount command doesn't necessarily understand what to do with anything other > than EINVAL. > > The old code returned -ERANGE as an intermediate error that then get > converted to -EINVAL, whereas the new code returns -ERANGE. > > This was induced by passing minorversion=1 to a v4 mount where > CONFIG_NFS_V4_1 was disabled in the kernel build. > > Fixes: 68f65ef40e1e ("NFS: Convert mount option parsing to use functionality from fs_parser.h") > Reported-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx> > Signed-off-by: David Howells <dhowells@xxxxxxxxxx> > > diff --git a/fs/nfs/fs_context.c b/fs/nfs/fs_context.c > index 429315c011ae..74508ed9aeec 100644 > --- a/fs/nfs/fs_context.c > +++ b/fs/nfs/fs_context.c > @@ -769,8 +769,7 @@ static int nfs_fs_context_parse_param(struct fs_context *fc, > out_invalid_address: > return nfs_invalf(fc, "NFS: Bad IP address specified"); > out_of_bounds: > - nfs_invalf(fc, "NFS: Value for '%s' out of range", param->key); > - return -ERANGE; > + return nfs_invalf(fc, "NFS: Value for '%s' out of range", param->key); > } > > /* Yes, the boards boots up, thanks! Tested-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx> I did not run extensive tests but few boots show also 2-3 seconds faster mount of NFS root (faster switch from initramfs to proper user-space from NFS). Best regards, Krzysztof