[Intel-gfx] [PATCH v2] drm/i915: Store preemption capability in engine->flags

Chris Wilson chris at chris-wilson.co.uk
Wed Apr 4 08:25:54 UTC 2018


Quoting Daniele Ceraolo Spurio (2018-04-03 22:52:35)
> 
> 
> On 03/04/18 11:35, Chris Wilson wrote:
> > Let's avoid having to delve down the pointer chain to see if the i915
> > device has support for preemption and store that on the engine, which
> > made the decision in the first place!
> > 
> > v2: Refactor common preemption policy between execlists/guc.
> > 
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> > Cc: Tomasz Lis <tomasz.lis at intel.com>
> > Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
> > Cc: Michał Winiarski <michal.winiarski at intel.com>
> > Cc: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
> 
> Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
> 
> <snip>
> 
> > +static inline bool
> > +__execlists_need_preempt(int prio, int last)
> 
> Nitpick: this fits on a single line

Fixed up and pushed, thanks. The refactor saves me doing in a later
patch for the third user :)
-Chris


More information about the Intel-gfx mailing list