Re: [patch V2 3/6] genirq: Add optional hardware synchronization for shutdown

2019-07-01 Thread Thomas Gleixner
On Mon, 1 Jul 2019, Peter Zijlstra wrote: > On Fri, Jun 28, 2019 at 01:11:51PM +0200, Thomas Gleixner wrote: > > But that does not catch the case where the interrupt is on flight at the > > hardware level but not yet serviced by the target CPU. That creates an > > interesing race condition: > > >

Re: [patch V2 3/6] genirq: Add optional hardware synchronization for shutdown

2019-07-01 Thread Peter Zijlstra
On Fri, Jun 28, 2019 at 01:11:51PM +0200, Thomas Gleixner wrote: > But that does not catch the case where the interrupt is on flight at the > hardware level but not yet serviced by the target CPU. That creates an > interesing race condition: > + * It does not check whether there is an interrupt

Re: [patch V2 3/6] genirq: Add optional hardware synchronization for shutdown

2019-07-01 Thread Marc Zyngier
On 28/06/2019 12:11, Thomas Gleixner wrote: > free_irq() ensures that no hardware interrupt handler is executing on a > different CPU before actually releasing resources and deactivating the > interrupt completely in a domain hierarchy. > > But that does not catch the case where the interrupt is o

[patch V2 3/6] genirq: Add optional hardware synchronization for shutdown

2019-06-28 Thread Thomas Gleixner
free_irq() ensures that no hardware interrupt handler is executing on a different CPU before actually releasing resources and deactivating the interrupt completely in a domain hierarchy. But that does not catch the case where the interrupt is on flight at the hardware level but not yet serviced by