cvs commit: src/sys/kern vfs_bio.c vfs_mount.c

Remko Lodder remko at FreeBSD.org
Thu Dec 27 19:03:21 UTC 2007


Aryeh M. Friedman wrote:
> Remko Lodder wrote:
>> 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!
> 
> Can someone give me a quick overview of what a super page is (I assume
> it is meant to improve paging performance)
> 

It is totally unrelated to this commit.

-- 
/"\   Best regards,                      | remko at FreeBSD.org
\ /   Remko Lodder                       | remko at EFnet
 X    http://www.evilcoder.org/          |
/ \   ASCII Ribbon Campaign              | Against HTML Mail and News


More information about the cvs-all mailing list