Re: usbd does not use detach

In message: <20030814110113.4d238ddd.eaja_at_erols.com>
            Eric Jacobs <eaja_at_erols.com> writes:
: #    DETACH_FORCE: Clients using the device must be disconnected,
: #        typically by revoking open file descriptors. May not
: #        return EBUSY due to client activitiy, but may return
: #        that or other errors due to hardware problems or
: #        limitations.
: #
: #    DETACH_EJECTED: This call is made from a lower-level bus   
: #        driver when the device has been physically removed from
: #        the system. Like DETACH_FORCE, except that drivers can
: #        avoid attemping (and failing) to reset the hardware
: #        state. This request must succeed.

These two are redundant.  Devices can already ask the bridge driver if
the device is still present on the bus.  Smart drivers already do
this, but most of the drivers in the tree are dumb.  You also have to
deal with device disappearance in ISRs since it is possible for the
device to go away while the device is in the middle of the ISR.  Some
bus technologies also allow interrupt entry when a card/device is
ejected.

: In addition, the DETACH_FORCE and DETACH_EJECTED flags could
: be mapped to appropriate flag values for the other subsystems, such
: as MNT_FORCE and (a new) MNT_EJECTED flag for VFS.

The problem is that when you are detaching a device, it is gone when
you return from the detach routine.  It can be hard to know what
buffers (disk, network, etc) in the system refer a given newbus device
because there's not a one to one mapping for the device_t to dev_t
that the rest of the system uses.  Devices may or may not know about
buffers that are outstanding.  Work would be needed in the buf/bio
system to reference cound the dev_t so that when the driver destroys
it, it doesn't go completely away until the reference count goes to
zero.  However, doing that may have unfavorable performance impacts.

: > i manually umount the device before unpluging it.
: 
: That is the only safe way to do it for now.

Agreed.

Warner