[Intel-gfx] [PATCH 2/6] drm/i915: Break out common code from gen8_gt_irq_handler

Chris Wilson chris at chris-wilson.co.uk
Tue Oct 20 05:40:20 PDT 2015


On Tue, Oct 20, 2015 at 02:12:21PM +0200, Daniel Vetter wrote:
> On Tue, Oct 20, 2015 at 10:23:52AM +0100, Nick Hoath wrote:
> > Break out common code from gen8_gt_irq_handler and put it in to
> > an always inlined function. gcc optimises out the shift at compile
> > time. (Thomas Daniel/Daniel Vetter/Chris Wilson)
> > 
> > Issue: VIZ-4277
> > Signed-off-by: Nick Hoath <nicholas.hoath at intel.com>
> > Cc: Thomas Daniel <thomas.daniel at intel.com>
> > Cc: Daniel Vetter <daniel.vetter at ffwll.ch>
> > Cc: Chris Wilson <chris at chris-wilson.co.uk>
> 
> Merged the first two patches to dinq, thanks.
> -Daniel
> 
> > ---
> >  drivers/gpu/drm/i915/i915_irq.c | 40 ++++++++++++++++++++--------------------
> >  1 file changed, 20 insertions(+), 20 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
> > index fbf9153..7837f5e 100644
> > --- a/drivers/gpu/drm/i915/i915_irq.c
> > +++ b/drivers/gpu/drm/i915/i915_irq.c
> > @@ -1290,6 +1290,16 @@ static void snb_gt_irq_handler(struct drm_device *dev,
> >  		ivybridge_parity_error_irq_handler(dev, gt_iir);
> >  }
> >  
> > +static __always_inline void
> > +		gen8_cs_irq_handler(struct intel_engine_cs *ring, u32 iir,
> > +				    int test_shift)

Pray tell you fixed the formatting first?
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list