Re: Mount version older than kernel: 4 vs. 6

[Olaf Kirch <okir@suse.de>]

[-- Attachment #1: Type: text/plain, Size: 864 bytes --]

On Wed, Aug 04, 2004 at 02:50:09PM +0200, Frank Steiner wrote:
> Just a stupid question: why does the nfsmount.h from the kernel source
> define NFS_MOUNT_VERSION 6? Shouldn't this number match the latest nfs
> version, i.e., 4? I've never heard of nfs v6 yet :-)
> Or is the meaning of NFS_MOUNT_VERSION different?

That's the version of the NFS kernel mount struct, and is
absolutely unrelated to the NFS version (except that both
grow monotonically over time).

> Would it be safe to patch nfsmount.h and set the number to 4 for my
> own kernel to get rid of this warning, or could that break sth. else?
> It just fills my logs and I want to get rid of it...

Our kernel has a patch to silence that warning for versions < 3.
But I wouldn't mind if it was nixed entirely.

Olaf
-- 
Olaf Kirch     |  The Hardware Gods hate me.
okir@suse.de   |
---------------+ 

[-- Attachment #2: nfs-mount-warning --]
[-- Type: text/plain, Size: 1143 bytes --]

Bug #36841: Disable ``mount version is older than kernel'' warning:
The kernel for SL 9.1 will support a higher mount version than the
mount binary.

Bug #37689: Some people still saw kernel messages complaining about
the mount version. It seems this happens when the first mount attempt
using kernel mount version 4 fails for some reason (for instance if
the server doesn't support NFSv3) mount falls back to mount version 3.
So we should really warn about versions < 3 only.    --okir

Index: linux-2.6.4/fs/nfs/inode.c
===================================================================
--- linux-2.6.4.orig/fs/nfs/inode.c
+++ linux-2.6.4/fs/nfs/inode.c
@@ -1389,8 +1389,9 @@ static struct super_block *nfs_get_sb(st
 		memset(root->data+root->size, 0, sizeof(root->data)-root->size);
 
 	if (data->version != NFS_MOUNT_VERSION) {
-		printk("nfs warning: mount version %s than kernel\n",
-			data->version < NFS_MOUNT_VERSION ? "older" : "newer");
+		if (data->version < 3)
+			printk("nfs warning: mount version %d is older "
+			       "than 3\n", data->version);
 		if (data->version < 2)
 			data->namlen = 0;
 		if (data->version < 3)