On 12/14/2016 6:25 AM, John Stultz wrote:
> On Tue, Dec 13, 2016 at 3:51 PM, John Youn <John.Youn@xxxxxxxxxxxx> wrote:
>> On 12/13/2016 3:23 PM, John Stultz wrote:
>>> I've found trying to boot Linus' HEAD today on HiKey, I'm hitting the
>>> following WARNINGS/panics, which I bisected down to being introduced
>>> with 5f54c54b0ba8 ("Add DDMA chain pointers to dwc2_hsotg_ep
>>> structure").
>>>
>>> It seems the issue is calling dma_alloc_coherent() in
>>> dwc2_hsotg_ep_enable() isn't safe, as we may get there from irq
>>> context with irqs off.
>>>
>>> Any thoughts on a fix?
>>>
>>
>> Hi John,
>>
>> Check this:
>> http://marc.info/?l=linux-usb&m=148058361526211&w=2
>
> So digging further here, it seems that patch addresses the allocation
> side, but I'm also seeing warnings on the free side, as irqs are
> disabled in dwc2_hsotg_vbus_session() before calling
> dwc2_hsotg_disconnect().
>
> (Note that this is triggered using the extcon patch I have, but the
> problematic logic seems to be pre-existing).
>
> thanks
> -john
Hi John Stultz,
Related to free side, we have fix which will be submitted after internal
review and testing. You'll see it here soon.
Thanks,
Vardan.
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
