2011-02-17 23:44:19

by Jan Engelhardt

[permalink] [raw]
Subject: umount.nfs4 is disturbed by /proc/mounts


Recent distributions start to have /etc/mtab being a symlink to
/proc/mounts with util-linux 2.19. Subsequently, this annoys umount.nfs4
from nfs-client-1.2.3 and emits a warning:

jng-0:/ # grep nfs4 /proc/mounts
10.10.1.1:/home/ /home nfs4 rw,relatime,vers=4,rsize=1048576,wsize=1048576,namlen=255,hard,proto=tcp,port=0,timeo=600,retrans=2,sec=sys,clientaddr=10.10.1.212,minorversion=0,local_lock=none,addr=10.10.1.1 0 0
jng-0:/ # umount /home
umount.nfs4: invalid value for 'port=' option




2011-02-18 21:48:51

by Chuck Lever

[permalink] [raw]
Subject: Re: umount.nfs4 is disturbed by /proc/mounts


On Feb 18, 2011, at 11:56 AM, Karel Zak wrote:

> On Fri, Feb 18, 2011 at 02:28:28PM -0500, Jeff Layton wrote:
>> On Fri, 18 Feb 2011 09:11:54 -0800
>> Chuck Lever <[email protected]> wrote:
>>
>>>
>>> On Feb 17, 2011, at 3:44 PM, Jan Engelhardt wrote:
>>>
>>>>
>>>> Recent distributions start to have /etc/mtab being a symlink to
>>>> /proc/mounts with util-linux 2.19. Subsequently, this annoys umount.nfs4
>>>> from nfs-client-1.2.3 and emits a warning:
>>>>
>>>> jng-0:/ # grep nfs4 /proc/mounts
>>>> 10.10.1.1:/home/ /home nfs4 rw,relatime,vers=4,rsize=1048576,wsize=1048576,namlen=255,hard,proto=tcp,port=0,timeo=600,retrans=2,sec=sys,clientaddr=10.10.1.212,minorversion=0,local_lock=none,addr=10.10.1.1 0 0
>>>> jng-0:/ # umount /home
>>>> umount.nfs4: invalid value for 'port=' option
>>>
>>> I can take a look at this.
>>>
>>
>> FWIW, I just fixed mount.cifs recently based on recommendations from
>> Karel:
>>
>> http://git.samba.org/?p=cifs-utils.git;a=commitdiff;h=fba28cfe2f13dd8bdae3cec76178f42b001a40ca
>>
>> ...this is the same approach that /bin/mount uses, so you're probably
>> safe to use a similar one for mount/umount.nfs.
>
> This is not the same problem. The NFS umount/remount depends on
> (userspace) values from /etc/mtab.

Right, this problem is not dependent on the existence of mtab. There is a real bug here, I suggest. port=0 is a valid specification, and the mount option parser in mount.nfs and umount.nfs should accept it. That's not dependent on whether /etc/mtab is a link or whether we are using libmount to do the work.

I have a very simple patch to fix this now, just needs testing.

> The solution is to link mount.nfs with libmount. I'm working on this
> change... http://www.spinics.net/lists/linux-nfs/msg19165.html
>
>
> I have already talked about mtab issues at fedora-devel list, see
> http://lists.fedoraproject.org/pipermail/devel/2011-January/147995.html
>
> The mount.nfs and pam_mount are on my TODO list for the next weeks.
> If the changes in mount.nfs will be successful then I'd like to link
> mount.cifs with libmount too.
>
> Note that mount(8) in Fedora still supports regular mtab too, and
> the regular mtab is still util-linux upstream default.
>
> Karel
>
> --
> Karel Zak <[email protected]>
> http://karelzak.blogspot.com

--
Chuck Lever
chuck[dot]lever[at]oracle[dot]com




2011-02-19 22:30:44

by Chuck Lever

[permalink] [raw]
Subject: Re: umount.nfs4 is disturbed by /proc/mounts


On Feb 17, 2011, at 3:44 PM, Jan Engelhardt wrote:

>
> Recent distributions start to have /etc/mtab being a symlink to
> /proc/mounts with util-linux 2.19. Subsequently, this annoys umount.nfs4
> from nfs-client-1.2.3 and emits a warning:
>
> jng-0:/ # grep nfs4 /proc/mounts
> 10.10.1.1:/home/ /home nfs4 rw,relatime,vers=4,rsize=1048576,wsize=1048576,namlen=255,hard,proto=tcp,port=0,timeo=600,retrans=2,sec=sys,clientaddr=10.10.1.212,minorversion=0,local_lock=none,addr=10.10.1.1 0 0
> jng-0:/ # umount /home
> umount.nfs4: invalid value for 'port=' option

With the latest nfs-utils (post 1.2.3) I can't reproduce this on my Fedora 13 client after replacing /etc/mtab with a symlink to /proc/mounts. I think we may have an upstream fix for this already:

commit dc08c702a6c7f824f317af561f491635ee898a71
Author: Chuck Lever <[email protected]>
Date: Wed Oct 13 13:55:10 2010 -0400

umount.nfs: Distinguish between nfs4 and nfs mounts

--
Chuck Lever
chuck[dot]lever[at]oracle[dot]com




2011-02-18 17:13:04

by Chuck Lever

[permalink] [raw]
Subject: Re: umount.nfs4 is disturbed by /proc/mounts


On Feb 17, 2011, at 3:44 PM, Jan Engelhardt wrote:

>
> Recent distributions start to have /etc/mtab being a symlink to
> /proc/mounts with util-linux 2.19. Subsequently, this annoys umount.nfs4
> from nfs-client-1.2.3 and emits a warning:
>
> jng-0:/ # grep nfs4 /proc/mounts
> 10.10.1.1:/home/ /home nfs4 rw,relatime,vers=4,rsize=1048576,wsize=1048576,namlen=255,hard,proto=tcp,port=0,timeo=600,retrans=2,sec=sys,clientaddr=10.10.1.212,minorversion=0,local_lock=none,addr=10.10.1.1 0 0
> jng-0:/ # umount /home
> umount.nfs4: invalid value for 'port=' option

I can take a look at this.

--
Chuck Lever
chuck[dot]lever[at]oracle[dot]com




2011-02-18 19:56:09

by Karel Zak

[permalink] [raw]
Subject: Re: umount.nfs4 is disturbed by /proc/mounts

On Fri, Feb 18, 2011 at 02:28:28PM -0500, Jeff Layton wrote:
> On Fri, 18 Feb 2011 09:11:54 -0800
> Chuck Lever <[email protected]> wrote:
>
> >
> > On Feb 17, 2011, at 3:44 PM, Jan Engelhardt wrote:
> >
> > >
> > > Recent distributions start to have /etc/mtab being a symlink to
> > > /proc/mounts with util-linux 2.19. Subsequently, this annoys umount.nfs4
> > > from nfs-client-1.2.3 and emits a warning:
> > >
> > > jng-0:/ # grep nfs4 /proc/mounts
> > > 10.10.1.1:/home/ /home nfs4 rw,relatime,vers=4,rsize=1048576,wsize=1048576,namlen=255,hard,proto=tcp,port=0,timeo=600,retrans=2,sec=sys,clientaddr=10.10.1.212,minorversion=0,local_lock=none,addr=10.10.1.1 0 0
> > > jng-0:/ # umount /home
> > > umount.nfs4: invalid value for 'port=' option
> >
> > I can take a look at this.
> >
>
> FWIW, I just fixed mount.cifs recently based on recommendations from
> Karel:
>
> http://git.samba.org/?p=cifs-utils.git;a=commitdiff;h=fba28cfe2f13dd8bdae3cec76178f42b001a40ca
>
> ...this is the same approach that /bin/mount uses, so you're probably
> safe to use a similar one for mount/umount.nfs.

This is not the same problem. The NFS umount/remount depends on
(userspace) values from /etc/mtab.

The solution is to link mount.nfs with libmount. I'm working on this
change... http://www.spinics.net/lists/linux-nfs/msg19165.html


I have already talked about mtab issues at fedora-devel list, see
http://lists.fedoraproject.org/pipermail/devel/2011-January/147995.html

The mount.nfs and pam_mount are on my TODO list for the next weeks.
If the changes in mount.nfs will be successful then I'd like to link
mount.cifs with libmount too.

Note that mount(8) in Fedora still supports regular mtab too, and
the regular mtab is still util-linux upstream default.

Karel

--
Karel Zak <[email protected]>
http://karelzak.blogspot.com

2011-02-18 19:24:34

by Jeff Layton

[permalink] [raw]
Subject: Re: umount.nfs4 is disturbed by /proc/mounts

On Fri, 18 Feb 2011 09:11:54 -0800
Chuck Lever <[email protected]> wrote:

>
> On Feb 17, 2011, at 3:44 PM, Jan Engelhardt wrote:
>
> >
> > Recent distributions start to have /etc/mtab being a symlink to
> > /proc/mounts with util-linux 2.19. Subsequently, this annoys umount.nfs4
> > from nfs-client-1.2.3 and emits a warning:
> >
> > jng-0:/ # grep nfs4 /proc/mounts
> > 10.10.1.1:/home/ /home nfs4 rw,relatime,vers=4,rsize=1048576,wsize=1048576,namlen=255,hard,proto=tcp,port=0,timeo=600,retrans=2,sec=sys,clientaddr=10.10.1.212,minorversion=0,local_lock=none,addr=10.10.1.1 0 0
> > jng-0:/ # umount /home
> > umount.nfs4: invalid value for 'port=' option
>
> I can take a look at this.
>

FWIW, I just fixed mount.cifs recently based on recommendations from
Karel:

http://git.samba.org/?p=cifs-utils.git;a=commitdiff;h=fba28cfe2f13dd8bdae3cec76178f42b001a40ca

...this is the same approach that /bin/mount uses, so you're probably
safe to use a similar one for mount/umount.nfs.

--
Jeff Layton <[email protected]>