cvs commit: src/sys/net if.c src/sys/dev/nmdm nmdm.c src/sys/dev/usb ugen.c uhid.c ulpt.c ums.c uscanner.c src/sys/netgraph/bluetooth/dr

[Jeff Roberson]

On Sun, 28 Sep 2003, Poul-Henning Kamp wrote:

> phk         2003/09/28 13:48:13 PDT
>
>   FreeBSD src repository
>
>   Modified files:
>     sys/net              if.c
>     sys/dev/nmdm         nmdm.c
>     sys/dev/usb          ugen.c uhid.c ulpt.c ums.c uscanner.c
>     sys/netgraph/bluetooth/drivers/ubt ng_ubt.c
>     sys/netgraph/bluetooth/drivers/ubtbcmfw ubtbcmfw.c
>   Log:
>   I don't know from where the notion that device driver should or
>   even could call VOP_REVOKE() on vnodes associated with its dev_t's
>   has originated, but it stops right here.
>
>   If there are things people belive destroy_dev() needs to learn how to
>   do, please tell me about it, preferably with a reproducible test case.
>
>   Include <sys/uio.h> in bluetooth code rather than rely on <sys/vnode.h>
>   to do so.
>
>   The fact that some of the USB code needs to include <sys/vnode.h>
>   still disturbs me greatly, but I do not have time to chase that.
>
>   Revision  Changes    Path
>   1.16      +2 -2      src/sys/dev/nmdm/nmdm.c
>   1.77      +0 -8      src/sys/dev/usb/ugen.c
>   1.63      +0 -6      src/sys/dev/usb/uhid.c
>   1.59      +0 -10     src/sys/dev/usb/ulpt.c
>   1.62      +0 -5      src/sys/dev/usb/ums.c
>   1.45      +0 -4      src/sys/dev/usb/uscanner.c
>   1.165     +1 -1      src/sys/net/if.c
>   1.8       +1 -14     src/sys/netgraph/bluetooth/drivers/ubt/ng_ubt.c
>   1.2       +1 -15     src/sys/netgraph/bluetooth/drivers/ubtbcmfw/ubtbcmfw.c
>

Unfortunately this functionality was actually required for usb to function
properly.  Did you test this with some removable devices?

Thanks,
Jeff