On Wed, 23 May 2007, Oliver Neukum wrote: > del_timer_sync(&usbhid->io_retry); > + cancel_work_sync(&usbhid->restart_work); > flush_scheduled_work();
Hi Oliver, isn't the call to flush_scheduled_work() after cancel_work_sync() redundant? Also, we could very probably use the hotplug-safer cancel_work_sync() elsewhere in your patch where you introduce flush_scheduled_work(), right? Thanks, -- Jiri Kosina SUSE Labs ------------------------------------------------------------------------- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/ _______________________________________________ linux-usb-devel@lists.sourceforge.net To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
