> On Thu, 26 Dec 2013, Du, ChangbinX wrote: > > > I can reproduce issue by adding a delay just after > > usb_set_intfdata(intf, NULL) (echo -1 > bConfigurationValue to trigger > hub_dissconnect())without your patch. > > > > After patch applied, cannot reproduce and didn't found any other issue. > Patch works well. > > > > Alan, need I update patch to v2 or you will do it? > > Changbin, after looking more closely I realized there was a second aspect to > this race: recursively_mark_NOTATTACHED uses hub->ports[i] while > hub_disconnect removes the port devices. You ought to be able to cause > an oops by inserting a delay just after the loop where > usb_hub_remove_port_device is called. > > The updated patch below should fix both problems. Can you test it? > > Alan Stern > Ok, I'll test it today or tomorrow. Please wait my response. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
