Re: [PATCH] serial: imx: drop workaround for forced irq threading

2021-03-24 Thread Uwe Kleine-König
Hello Sebastian, On Tue, Mar 23, 2021 at 10:04:13AM +0100, Sebastian Andrzej Siewior wrote: > On 2021-03-23 08:34:47 [+0100], Uwe Kleine-König wrote: > > On Mon, Mar 22, 2021 at 09:48:36PM +0100, Sebastian Andrzej Siewior wrote: > > > On 2021-03-22 14:40:32 [+0100], Uwe Kleine-König wrote: > > >

Re: [PATCH] serial: imx: drop workaround for forced irq threading

2021-03-24 Thread Uwe Kleine-König
Hello Johan, On Tue, Mar 23, 2021 at 03:37:29PM +0100, Johan Hovold wrote: > On Mon, Mar 22, 2021 at 02:40:32PM +0100, Uwe Kleine-König wrote: > > On Mon, Mar 22, 2021 at 02:20:57PM +0100, Johan Hovold wrote: > > > On Mon, Mar 22, 2021 at 12:55:36PM +0100, Uwe Kleine-König wrote: > > > > On Mon,

Re: [PATCH] serial: imx: drop workaround for forced irq threading

2021-03-23 Thread Johan Hovold
On Mon, Mar 22, 2021 at 02:40:32PM +0100, Uwe Kleine-König wrote: > Hello Johan, > > On Mon, Mar 22, 2021 at 02:20:57PM +0100, Johan Hovold wrote: > > On Mon, Mar 22, 2021 at 12:55:36PM +0100, Uwe Kleine-König wrote: > > > On Mon, Mar 22, 2021 at 12:39:18PM +0100, Sebastian Andrzej Siewior wrote:

Re: [PATCH] serial: imx: drop workaround for forced irq threading

2021-03-23 Thread Sebastian Andrzej Siewior
On 2021-03-23 08:34:47 [+0100], Uwe Kleine-König wrote: > Hello Sebastian, Hi Uwe, > On Mon, Mar 22, 2021 at 09:48:36PM +0100, Sebastian Andrzej Siewior wrote: > > On 2021-03-22 14:40:32 [+0100], Uwe Kleine-König wrote: > > > From a strictly logically point of view you indeed cannot. But if you

Re: [PATCH] serial: imx: drop workaround for forced irq threading

2021-03-23 Thread Uwe Kleine-König
Hello Sebastian, On Mon, Mar 22, 2021 at 09:48:36PM +0100, Sebastian Andrzej Siewior wrote: > On 2021-03-22 14:40:32 [+0100], Uwe Kleine-König wrote: > > From a strictly logically point of view you indeed cannot. But if you go > > to the street and say to people there that they can park their car

Re: [PATCH] serial: imx: drop workaround for forced irq threading

2021-03-22 Thread Sebastian Andrzej Siewior
On 2021-03-22 14:40:32 [+0100], Uwe Kleine-König wrote: > From a strictly logically point of view you indeed cannot. But if you go > to the street and say to people there that they can park their car in > this street free of charge between Monday and Friday, I expect that most > of them will

Re: [PATCH] serial: imx: drop workaround for forced irq threading

2021-03-22 Thread Uwe Kleine-König
Hello Johan, On Mon, Mar 22, 2021 at 02:20:57PM +0100, Johan Hovold wrote: > On Mon, Mar 22, 2021 at 12:55:36PM +0100, Uwe Kleine-König wrote: > > On Mon, Mar 22, 2021 at 12:39:18PM +0100, Sebastian Andrzej Siewior wrote: > > > On 2021-03-22 12:34:02 [+0100], Uwe Kleine-König wrote: > > > > On

Re: [PATCH] serial: imx: drop workaround for forced irq threading

2021-03-22 Thread Johan Hovold
On Mon, Mar 22, 2021 at 12:55:36PM +0100, Uwe Kleine-König wrote: > Hallo Sebastian, > > On Mon, Mar 22, 2021 at 12:39:18PM +0100, Sebastian Andrzej Siewior wrote: > > On 2021-03-22 12:34:02 [+0100], Uwe Kleine-König wrote: > > > On Mon, Mar 22, 2021 at 12:10:36PM +0100, Johan Hovold wrote: > > >

Re: [PATCH] serial: imx: drop workaround for forced irq threading

2021-03-22 Thread Uwe Kleine-König
Hallo Sebastian, On Mon, Mar 22, 2021 at 12:39:18PM +0100, Sebastian Andrzej Siewior wrote: > On 2021-03-22 12:34:02 [+0100], Uwe Kleine-König wrote: > > On Mon, Mar 22, 2021 at 12:10:36PM +0100, Johan Hovold wrote: > > > Force-threaded interrupt handlers used to run with interrupts enabled, > >

Re: [PATCH] serial: imx: drop workaround for forced irq threading

2021-03-22 Thread Sebastian Andrzej Siewior
On 2021-03-22 12:34:02 [+0100], Uwe Kleine-König wrote: > On Mon, Mar 22, 2021 at 12:10:36PM +0100, Johan Hovold wrote: > > Force-threaded interrupt handlers used to run with interrupts enabled, > > something which could lead to deadlocks in case a threaded handler > > shared a lock with code

Re: [PATCH] serial: imx: drop workaround for forced irq threading

2021-03-22 Thread Uwe Kleine-König
On Mon, Mar 22, 2021 at 12:10:36PM +0100, Johan Hovold wrote: > Force-threaded interrupt handlers used to run with interrupts enabled, > something which could lead to deadlocks in case a threaded handler > shared a lock with code running in hard interrupt context (e.g. timer > callbacks) and did

[PATCH] serial: imx: drop workaround for forced irq threading

2021-03-22 Thread Johan Hovold
Force-threaded interrupt handlers used to run with interrupts enabled, something which could lead to deadlocks in case a threaded handler shared a lock with code running in hard interrupt context (e.g. timer callbacks) and did not explicitly disable interrupts. This was specifically the case for