Bug 4665 - umount.cifs: gives up too quickly when mtab is locked by another process
Summary: umount.cifs: gives up too quickly when mtab is locked by another process
Status: RESOLVED FIXED
Alias: None
Product: CifsVFS
Classification: Unclassified
Component: user space tools (show other bugs)
Version: 2.6
Hardware: Other Linux
: P3 normal
Target Milestone: ---
Assignee: Steve French
QA Contact:
URL: http://bugs.debian.org/cgi-bin/bugrep...
Keywords:
Depends on:
Blocks:
 
Reported: 2007-05-27 09:40 UTC by Christian Perrier (dead mail address)
Modified: 2009-07-27 12:42 UTC (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Christian Perrier (dead mail address) 2007-05-27 09:40:29 UTC
From one of our Debian users:

The mtab locking code gives up too easily. If it can't obtain the lock
it just bails out without retrying, sometimes resulting in a dirty mtab
with the mount entry still there.

This causes autofs to think that the mount is still active and it
doesn't try to mount the autofs entry again when accessed.

I looked in the source code for umount (package: mount) and umount.cifs
(package: smbfs) for how each does the mtab locking. umount tries to get
the lock for a few seconds before giving up. umount.cifs just gives up.
So, if there are two umount processes running and one of them is
umount.cifs, and they both need to update mtab, umount.cifs will leave
its mtab entry there because it wasn't patient enough when trying to get
the lock.
Comment 1 Debian samba package maintainers (PUBLIC MAILING LIST) 2009-01-03 12:37:46 UTC
I think this bug is fixed in 3.3.0rc2, at least from the comments in the code

Christian Perrier
Comment 2 shirishpargaonkar@gmail.com 2009-07-26 15:08:38 UTC
Can you please verify if this bug still persists?
Comment 3 Debian samba package maintainers (PUBLIC MAILING LIST) 2009-07-26 15:45:00 UTC
Confirmed as fixed in 3.3.0 as I write (privately) to Steve French. Sorry for forgetting to come back to this bug et mention this here.

Original mail sent to Steve:

Steve (French), this is samba bug #4665 (umount.cifs: gives up too quickly when mtab is locked by another process)

Just a small mail to confirm that this bug is still here in 3.2.5 and
subsequent releases, but is fixed in 3.3.0 pre-releases where the
locking code has been apparently entirely rewritten.

There are probably several other bugs (reported in Debian and
forwarded to Samba's Bugzilla) related to mtab that are fixed in 3.3.*
as well.
Comment 4 Jeff Layton 2009-07-27 12:42:27 UTC
Closing as fixed.

Also note that with the changes to the default behavior when mount.cifs is setuid, there is virtually no reason to ship umount.cifs anymore.