On su, 2016-08-07 at 15:45 +0100, Chris Wilson wrote: > @@ -596,11 +594,9 @@ unsigned int intel_kick_waiters(struct drm_i915_private *i915) > * RCU lock, i.e. as we call wake_up_process() we must be holding the > * rcu_read_lock(). > */ Comment is not relevant to this piece of code anymore? Looks like more proper RCU code; Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> Regards, Joonas > - rcu_read_lock(); > for_each_engine(engine, i915) > if (unlikely(intel_engine_wakeup(engine))) > mask |= intel_engine_flag(engine); > - rcu_read_unlock(); > > return mask; > } -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
