[RFC PATCH v1 4/6] USB: EHCI: don't release ehci->lock if URB giveback in tasklet context

Ming Lei ming.lei at canonical.com
Tue Jun 18 23:13:49 EDT 2013


On Wed, Jun 19, 2013 at 12:43 AM, Alan Stern <stern at rowland.harvard.edu> wrote:
> On Tue, 18 Jun 2013, Ming Lei wrote:
>
>> If HCD_BH is set for HC driver's flags, URB giveback will be
>> done in tasklet context instead of interrupt context, so the
>> ehci->lock needn't to be released any more before calling
>> usb_hcd_giveback_urb().
>
> It is premature to do this now.  This should be part of the 6/6 patch,

It is fine since HCD_BH isn't enabled.

> when it won't be necessary to test hcd_giveback_urb_in_bh().

Keeping it is easy to compare test results between running complete()
in tasklet and in hard irq handler.

Also it might be helpful for out of tree drivers.

But it isn't a big deal, I can merge it to 6/6 if you care.

Thanks,
--
Ming Lei



More information about the linux-arm-kernel mailing list