cvs commit: src/sys/cam/scsi scsi_cd.c scsi_da.c src/sys/geom geom_disk.c geom_disk.h geom_subr.c

M. Warner Losh imp at bsdimp.com
Sat Nov 19 20:41:24 GMT 2005


In message: <XFMail.20051119104237.jdp at polstra.com>
            John Polstra <jdp at polstra.com> writes:
: On 19-Nov-2005 Jeremie Le Hen wrote:
: >>   This fix adds a new disk_gone() function which is called by CAM when a
: >>   drive goes away.  It orphans all of the providers associated with the
: >>   drive, setting an error condition of ENXIO in each one.  In addition,
: >>   we prevent a re-taste on last close for writing if an error condition
: >>   has been set in the provider.
: > 
: > This commit stirred my memory up about panicking when hot-unplugging
: > an USB key while it is still mounted.  I have not been able to track
: > HEAD source changes attentively for some time recently but it
: > appears from the above description that this awkard behaviour has been
: > replaced by a smarter one (IOW, return ENXIO on all pending filesystem
: > actions).
: > 
: > Thanks in advance for explanations.
: 
: This commit may or may not fix those panics -- I don't really know.
: There is a lot that can go wrong if you remove a mounted filesystem
: from the system.

Meaning no offense to John, I somewhat doubt that his fix will help
Jeremie's problems.  The problems with mounted file systems and their
disks going away is pervasive in the filesystem (and other) layers and
will require a bunch of work to fix.

Warner


More information about the cvs-src mailing list