On Sat, Jun 20, 2015 at 10:09:37AM +0100, Stefan Wahren wrote:
>
> > Dave Martin <Dave.Martin@xxxxxxx> hat am 18. Juni 2015 um 15:54 geschrieben:
> >
> >
> > pl011_tx_softirq() currently uses spin_{,un}lock(), which are not
> > sufficient to inhibit pl011_int() from being triggered by a local
> > IRQ and trying to re-take the same lock. This can lead to
> > deadlocks.
> >
> > This patch uses the _irq() locking variants instead to ensure that
> > pl011_int() handling for a given port is deferred until any
> > pl011_tx_softirq() work for that port is complete.
> >
> > Fixes: 734745caeb9f serial/amba-pl011: Activate TX IRQ passively
> > Signed-off-by: Dave Martin <Dave.Martin@xxxxxxx>
> > Tested-by: Robin Murphy <robin.murphy@xxxxxxx>
> > Cc: stable <stable@xxxxxxxxxxxxxxx> # 4.1
> >
>
> I also get info messages about inconsistent lock state on my mx28 board,
> after applying the patch the messages disappeared.
>
> Tested-by: Stefan Wahren <stefan.wahren@xxxxxxxx>
Thanks for the confirmation.
Cheers
---Dave
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
