Re: [PATCH 3/3] xhci: Don't create stream debugfs files with spinlock held.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2020-10-29 15:08:06 [+0200], Mathias Nyman wrote:
> Sebastian, I think you might be missing the first mail in this thread. 

thank you for clearing that up.

> https://lore.kernel.org/linux-usb/20201028203124.375344-4-mathias.nyman@xxxxxxxxxxxxxxx/
> 
> Let me do a quick recap to avoid confusion here. So far we have 2 issues.
> 
> 1. Lockdep issue #1, stream debugfs entry created with local interrupts disabled and xhci->lock held
>    Cause: 673d74683627b ("usb: xhci: add debugfs support for ep with stream") in v5.10-rc1
>    Fix:  [PATCH 3/3] xhci: Don't create stream debugfs files with spinlock held. (i.e. the mail we are replying to)
>    Comment: easily reproduced, enable lockdep and uas, mount debugfs and connect a UAS device.
>             discovered by both Hans and Mike
> .
> 2. Lockdep issue #2, adding entries to radix tree during (stream) ring expansion with interrupts disabled and xhci->lock held.
>    Cause: unknown, probably a patch since we started using radix trees for finding streams
>    Fix: unknown.
>    Comment: Discovered by Mike when testing fix for issue#1. I suspect it can be reproduced on 5.9 but is 
>    probably really hard as it involves ring expansion.
> 
> -Mathias

Sebastian



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux