Re: [PATCH] irqchip/alpine-msi: Fix freeing of interrupts on allocation error path

2020-12-11 Thread Marc Zyngier
On Sun, 29 Nov 2020 13:55:25 +, Marc Zyngier wrote: > The alpine-msi driver has an interesting allocation error handling, > where it frees the same interrupts repeatedly. Hilarity follows. > > This code is probably never executed, but let's fix it nonetheless. Applied to irq/irqchip-next,

Re: [PATCH] irqchip/alpine-msi: Fix freeing of interrupts on allocation error path

2020-11-30 Thread Antoine Tenart
Hello Marc, Quoting Marc Zyngier (2020-11-29 14:55:25) > The alpine-msi driver has an interesting allocation error handling, > where it frees the same interrupts repeatedly. Hilarity follows. That's interesting indeed... > This code is probably never executed, but let's fix it nonetheless. > >

[PATCH] irqchip/alpine-msi: Fix freeing of interrupts on allocation error path

2020-11-29 Thread Marc Zyngier
The alpine-msi driver has an interesting allocation error handling, where it frees the same interrupts repeatedly. Hilarity follows. This code is probably never executed, but let's fix it nonetheless. Fixes: e6b78f2c3e14 ("irqchip: Add the Alpine MSIX interrupt controller") Cc: Tsahee Zidenberg