Re: [PATCH] USB: complain if userspace resets an active endpoint

2014-02-20 Thread Alan Stern
On Wed, 19 Feb 2014, Greg KH wrote: > > + pr_warn("Process %d (%s) called USBDEVFS_%s for active endpoint > > %s ep %02x\n", > > + task_pid_nr(current), current->comm, > > + ioctl_name, dev_name(&udev->dev), epnum); > > +} > > If you

Re: [PATCH] USB: complain if userspace resets an active endpoint

2014-02-19 Thread Greg KH
On Wed, Feb 19, 2014 at 03:32:12PM -0500, Alan Stern wrote: > It is an error for a driver to call usb_clear_halt() or > usb_reset_endpoint() while there are URBs queued for the endpoint, > because the end result is not well defined. At the time the endpoint > gets reset, it may or may not be activ

[PATCH] USB: complain if userspace resets an active endpoint

2014-02-19 Thread Alan Stern
It is an error for a driver to call usb_clear_halt() or usb_reset_endpoint() while there are URBs queued for the endpoint, because the end result is not well defined. At the time the endpoint gets reset, it may or may not be actively running. As far as I know, no kernel drivers do this. But some