[PATCH 07/11] drm/i915: Clarify irq_lock locking, irq handlers

2014-09-16 Thread Daniel Vetter
On Tue, Sep 16, 2014 at 11:21:52AM +0100, Chris Wilson wrote: > On Mon, Sep 15, 2014 at 02:35:07PM +0200, Daniel Vetter wrote: > > irq handlers always run with interrupts locally disabled, so > > plain spinlocks is all we need. I've also reviewed again that they > > all follow the _irq_handler post

[PATCH 07/11] drm/i915: Clarify irq_lock locking, irq handlers

2014-09-16 Thread Chris Wilson
On Tue, Sep 16, 2014 at 12:28:18PM +0200, Daniel Vetter wrote: > On Tue, Sep 16, 2014 at 11:21:52AM +0100, Chris Wilson wrote: > > On Mon, Sep 15, 2014 at 02:35:07PM +0200, Daniel Vetter wrote: > > > irq handlers always run with interrupts locally disabled, so > > > plain spinlocks is all we need.

[PATCH 07/11] drm/i915: Clarify irq_lock locking, irq handlers

2014-09-16 Thread Chris Wilson
On Mon, Sep 15, 2014 at 02:35:07PM +0200, Daniel Vetter wrote: > irq handlers always run with interrupts locally disabled, so > plain spinlocks is all we need. I've also reviewed again that they > all follow the _irq_handler postfix convention. Hmm, we still have the full irq dance inside the reg

[PATCH 07/11] drm/i915: Clarify irq_lock locking, irq handlers

2014-09-15 Thread Daniel Vetter
irq handlers always run with interrupts locally disabled, so plain spinlocks is all we need. I've also reviewed again that they all follow the _irq_handler postfix convention. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/i915/i915_irq.c | 15 ++- 1 file changed, 6 insertions(+),