Date: Mon, 30 May 2011 09:58:49 -0400 (EDT) From: Rick Macklem <rmacklem@uoguelph.ca> To: Kostik Belousov <kostikbel@gmail.com> Cc: svn-src-head@freebsd.org, Rick Macklem <rmacklem@freebsd.org>, svn-src-all@freebsd.org, src-committers@freebsd.org, Robert Watson <rwatson@freebsd.org> Subject: Re: svn commit: r222466 - head/sbin/umount Message-ID: <336367081.983085.1306763929731.JavaMail.root@erie.cs.uoguelph.ca> In-Reply-To: <20110530130751.GV48734@deviant.kiev.zoral.com.ua>
next in thread | previous in thread | raw e-mail | index | archive | help
> On Mon, May 30, 2011 at 01:48:53PM +0100, Robert Watson wrote: > > On Sun, 29 May 2011, Rick Macklem wrote: > > > > > Modify the umount(8) command so that it doesn't do > > > a sync(2) syscall before unmount(2) for the "-f" case. > > > This avoids a forced dismount from getting stuck for > > > an NFS mountpoint in sync() when the server is not > > > responsive. With this commit, forced dismounts should > > > normally work for the NFS clients, but can take up to > > > about 1minute to complete. > > > > I'm actually a bit confused about why umount(8) calls sync(2) at > > all: > > surely it's the responsibility of the file system, rather than the > > userland > > tool, to ensure consistency subject to file system configuration and > > unmount-time flags? > This call is from the same department as triple-sync before reboot, > IMO. Hehe. I'm so old, I do two syncs, as required by 6th Edition.:-) I assumed the sync() was meant to be an optimization (given the comment for it) in the sense that it would get the writes of dirty blocks started "right away". However, given the short period of time from the the sync(2) call to the unmount(2) call, I'm not convinced it makes a significant difference. (I thought of just getting rid of it, but figured it was harmless for the non "-f" case and might matter for a buggy fs that doesn't get the unmount(2) quite right. ie. Same argument as doing the triple-sync, just to be sure.) rick
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?336367081.983085.1306763929731.JavaMail.root>