Re: [PATCH] um: Add winch to winch_handlers before registering winch IRQ

2024-04-23 Thread Richard Weinberger
- Ursprüngliche Mail - > Von: "Roberto Sassu" > An: "Johannes Berg" , "richard" , > "anton ivanov" > > CC: "linux-kernel" , "linux-um" > , "Roberto Sassu" > > Gesendet: Dienstag, 23. April

Re: [PATCH] um: Add winch to winch_handlers before registering winch IRQ

2024-04-23 Thread Roberto Sassu
On Thu, 2024-03-28 at 09:25 +0100, Johannes Berg wrote: > On Thu, 2024-03-07 at 11:49 +0100, Roberto Sassu wrote: > > From: Roberto Sassu > > > > Registering a winch IRQ is racy, an interrupt may occur before the winch is > > added to the winch_handlers list. > > > > If that happens, register_wi

Re: [PATCH] um: Add winch to winch_handlers before registering winch IRQ

2024-03-28 Thread Johannes Berg
On Thu, 2024-03-07 at 11:49 +0100, Roberto Sassu wrote: > From: Roberto Sassu > > Registering a winch IRQ is racy, an interrupt may occur before the winch is > added to the winch_handlers list. > > If that happens, register_winch_irq() adds to that list a winch that is > scheduled to be (or has

Re: [PATCH] um: Add winch to winch_handlers before registering winch IRQ

2024-03-28 Thread Roberto Sassu
On 3/7/2024 1:43 PM, Roberto Sassu wrote: On Thu, 2024-03-07 at 11:49 +0100, Roberto Sassu wrote: From: Roberto Sassu Registering a winch IRQ is racy, an interrupt may occur before the winch is added to the winch_handlers list. If that happens, register_winch_irq() adds to that list a winch t

Re: [PATCH] um: Add winch to winch_handlers before registering winch IRQ

2024-03-07 Thread Roberto Sassu
On Thu, 2024-03-07 at 11:49 +0100, Roberto Sassu wrote: > From: Roberto Sassu > > Registering a winch IRQ is racy, an interrupt may occur before the winch is > added to the winch_handlers list. > > If that happens, register_winch_irq() adds to that list a winch that is > scheduled to be (or has

[PATCH] um: Add winch to winch_handlers before registering winch IRQ

2024-03-07 Thread Roberto Sassu
From: Roberto Sassu Registering a winch IRQ is racy, an interrupt may occur before the winch is added to the winch_handlers list. If that happens, register_winch_irq() adds to that list a winch that is scheduled to be (or has already been) freed, causing a panic later in winch_cleanup(). Avoid