On ma, 2016-07-25 at 18:32 +0100, Chris Wilson wrote: > diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.h b/drivers/gpu/drm/i915/i915_gem_gtt.h > index 529fb483afc8..d2206f40f7b2 100644 > --- a/drivers/gpu/drm/i915/i915_gem_gtt.h > +++ b/drivers/gpu/drm/i915/i915_gem_gtt.h > > +static inline bool i915_vma_is_active(const struct i915_vma *vma) > +{ > + return vma->active; > +} > + > +static inline void i915_vma_set_active(struct i915_vma *vma, > + unsigned int engine) > +{ > + vma->active |= BIT(engine); > +} > + > +static inline void i915_vma_unset_active(struct i915_vma *vma, > + unsigned int engine) > +{ > + vma->active &= ~BIT(engine); > +} > + > +static inline bool i915_vma_has_active_engine(const struct i915_vma *vma, > + unsigned int engine) > +{ > + return vma->active & BIT(engine); > +} > + Are these going to grow more complex? Otherwise looks fine, Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx