Re: [PATCH v2 2/3] drm/panthor: Fix ordering in _irq_suspend()

2024-03-26 Thread Boris Brezillon
On Tue, 26 Mar 2024 09:58:17 + Liviu Dudau wrote: > On Mon, Mar 25, 2024 at 07:02:13PM +0100, Boris Brezillon wrote: > > On Mon, 25 Mar 2024 17:16:16 + > > Liviu Dudau wrote: > > > > > On Mon, Mar 25, 2024 at 02:57:04PM +0100, Boris Brezillon wrote: > > > > Make sure we set

Re: [PATCH v2 2/3] drm/panthor: Fix ordering in _irq_suspend()

2024-03-26 Thread Liviu Dudau
On Mon, Mar 25, 2024 at 07:02:13PM +0100, Boris Brezillon wrote: > On Mon, 25 Mar 2024 17:16:16 + > Liviu Dudau wrote: > > > On Mon, Mar 25, 2024 at 02:57:04PM +0100, Boris Brezillon wrote: > > > Make sure we set suspended=true last to avoid generating an irq storm > > > in the unlikely case

Re: [PATCH v2 2/3] drm/panthor: Fix ordering in _irq_suspend()

2024-03-25 Thread Boris Brezillon
On Mon, 25 Mar 2024 17:16:16 + Liviu Dudau wrote: > On Mon, Mar 25, 2024 at 02:57:04PM +0100, Boris Brezillon wrote: > > Make sure we set suspended=true last to avoid generating an irq storm > > in the unlikely case where an IRQ happens between the suspended=true > > assignment and the

Re: [PATCH v2 2/3] drm/panthor: Fix ordering in _irq_suspend()

2024-03-25 Thread Liviu Dudau
On Mon, Mar 25, 2024 at 02:57:04PM +0100, Boris Brezillon wrote: > Make sure we set suspended=true last to avoid generating an irq storm > in the unlikely case where an IRQ happens between the suspended=true > assignment and the _INT_MASK update. > > v2: > - New patch > > Reported-by: Steven

Re: [PATCH v2 2/3] drm/panthor: Fix ordering in _irq_suspend()

2024-03-25 Thread Steven Price
On 25/03/2024 13:57, Boris Brezillon wrote: > Make sure we set suspended=true last to avoid generating an irq storm > in the unlikely case where an IRQ happens between the suspended=true > assignment and the _INT_MASK update. > > v2: > - New patch > > Reported-by: Steven Price > Signed-off-by:

[PATCH v2 2/3] drm/panthor: Fix ordering in _irq_suspend()

2024-03-25 Thread Boris Brezillon
Make sure we set suspended=true last to avoid generating an irq storm in the unlikely case where an IRQ happens between the suspended=true assignment and the _INT_MASK update. v2: - New patch Reported-by: Steven Price Signed-off-by: Boris Brezillon --- drivers/gpu/drm/panthor/panthor_device.h