recent USB MFCs cause panics
Julian Elischer
julian at elischer.org
Thu May 5 14:56:33 PDT 2005
Warner Losh wrote:
>From: Julian Elischer <julian at elischer.org>
>Subject: Re: recent USB MFCs cause panics
>Date: Thu, 05 May 2005 14:24:03 -0700
>
>
>
>>Julian Elischer wrote:
>>
>>try:
>>
>>in usb_port.h
>>comment out line 425 (as below)
>>
>>422
>> 423 #define config_detach(dev, flag) \
>> 424 do { \
>> 425 /* device_detach(dev); */ \
>> 426 free(device_get_ivars(dev), M_USB); \
>> 427 device_delete_child(device_get_parent(dev), dev); \
>> 428 } while (0);
>> 429
>>
>>
>
>Commenting it out is lame... I fixed this in current in uhub.c as
>well as here... Since 'dev' is 0 here, I'm unsure that commenting it
>out will fix the problem because the next line frees it....
>
>
yes I noticed that..
the next line doesn't free it, it frees the ivars
which I don't think is the same thing..
the problem is that the 5.0 code does the device_delete_child() (as you
see above)
where 4.x did it in the device_detach()
so with this merge I get the worst of both worlds..
the answer is to make uhub.c not call it's bus_child_detached() method
(as 5.0 doesn't)
or to make it a null function, as it clears the subdev entry which
causes this problem.
(see teh next email.)
>Warner
>_______________________________________________
>freebsd-usb at freebsd.org mailing list
>http://lists.freebsd.org/mailman/listinfo/freebsd-usb
>To unsubscribe, send any mail to "freebsd-usb-unsubscribe at freebsd.org"
>
>
More information about the freebsd-usb
mailing list