On Wed, Sep 14, 2016 at 12:44:04PM +0300, Joonas Lahtinen wrote: > On ke, 2016-09-14 at 07:52 +0100, Chris Wilson wrote: > > -static inline bool > > -i915_gem_object_has_active_engine(const struct drm_i915_gem_object *obj, > > - int engine) > > -{ > > - return obj->flags & BIT(engine + I915_BO_ACTIVE_SHIFT); > > + return obj->active_count; > > our type is bool, so !!obj->active_count; That's the beauty of using bool, !! is implied on the (bool)integer conversion. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx