Date: Thu, 27 Dec 2007 18:10:46 +0100 From: Remko Lodder <remko@FreeBSD.org> To: Warner Losh <imp@FreeBSD.org> Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern vfs_bio.c vfs_mount.c Message-ID: <4773DC96.9040307@FreeBSD.org> In-Reply-To: <200712271638.lBRGcTu9071266@repoman.freebsd.org> References: <200712271638.lBRGcTu9071266@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Warner Losh wrote: > imp 2007-12-27 16:38:29 UTC > > FreeBSD src repository > > Modified files: > sys/kern vfs_bio.c vfs_mount.c > Log: > A partial solution to some of the 'pull the umass device with a > mounted FS' problems. These are more along the lines of 'avoiding an > avoidable panic' than a complete solution to removable devices. We > now close the barn door after the horse has gotten lose and has been > hit by a truck, as it were. The barn no longer catches fire in this > case, but the horse is still dead :-). > > The vfs_bio.c fix causes us not to put a failed write back into the > dirty pool if the error returned was ENXIO. In that case, the buffer > is treated like any other clean buffer that's being retured. ENXIO > means the device isn't there anymore and will never be there again in > the future, so retrying is futile. > > The vfs_mount.c fix treats 'ENXIO' as success for unmounting a file > system. If the device is gone, retrying later won't help and we'll > never be able to unmount the device. > > These two are part of a larger patch set submitted by the author. The > other patches will be forth coming. I added comments to these two > patches. > > Submitted by: Henrik Gulbrandsen > Reviewed by: phk@ > PR: usb/46176 (partial) > > Revision Changes Path > 1.531 +4 -0 src/sys/kern/vfs_bio.c > 1.269 +7 -2 src/sys/kern/vfs_mount.c Hurray! -- /"\ Best regards, | remko@FreeBSD.org \ / Remko Lodder | remko@EFnet X http://www.evilcoder.org/ | / \ ASCII Ribbon Campaign | Against HTML Mail and News
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4773DC96.9040307>