> -----Original Message----- > From: Robert Baldyga [mailto:[email protected]] > Sent: Friday, December 12, 2014 2:17 PM > To: [email protected] > Cc: [email protected]; [email protected]; linux- > [email protected]; [email protected]; > [email protected]; [email protected]; Robert Baldyga > Subject: [PATCH] usb: gadget: udc-core: call udc_stop() before > gadget unbind > > As usb function drivers assumes that all usb request will be > completed > before function unbind call, we should supply such behavior. In > some > cases ep_disable() won't kill all request effectively, because some > IN requests can be in running state. In such situation it's > possible > to have unbind function called before last request completion, > which > can cause problems. > > For example unbinding f_ecm function while request on 'notify' > endpoint > is not completed, ends up NULL pointer dereference in unbind() > function. > > usb_gadget_udc_stop() call causes completion of all requests so if > it's > called before gadget unbind there is no risk that some of requests > will > stay uncompleted. > > Signed-off-by: Robert Baldyga <[email protected]>
This finally solves issue described before in [1] Tested-by: Krzysztof Opasiak <[email protected]> Footnotes: 1 - https://lkml.org/lkml/2014/12/9/283 -- Krzysztof Opasiak Samsung R&D Institute Poland Samsung Electronics [email protected] -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
