Re: umount.nfs4 is disturbed by /proc/mounts

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 <chuck.lever@xxxxxxxxxx> 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  <kzak@xxxxxxxxxx>
> http://karelzak.blogspot.com

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



--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux