On 12/22/06, Oliver Neukum <[EMAIL PROTECTED]> wrote: > Am Freitag, 22. Dezember 2006 17:18 schrieb Dmitry Torokhov: > > Hi, > > > > On 12/20/06, Greg KH <[EMAIL PROTECTED]> wrote: > > > > dev = usb_get_intfdata (interface); > > > - usb_set_intfdata(interface, NULL); > > > device_remove_file(&interface->dev, &dev_attr_speed); > > > + usb_set_intfdata(interface, NULL); > > > usb_put_dev(dev->udev); > > > kfree(dev); > > > } > > > > What does this change buy us? device_remove_file is not synchronous; > > If there is a process holding this attribute open whle disconnect is > > running it still may read/write to it after interface was set to NULL > > and dev freed. > > > > BTW, what is this device used for? Or maybe I don't want to know... > > Hi, > > right now, it merely shrinks the window. > However, device_remove_file() working asynchronous is a design bug. > When would you safely call kfree? Therefore I've made a patch that makes > it synchronous. Unfortunately it is fairly invasive to sysfs. Greg is > including it into -mm. This will close the race for good. >
Do you know the name of the patch? -- Dmitry ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ linux-usb-devel@lists.sourceforge.net To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
