Quoting Tvrtko Ursulin (2018-11-30 17:44:10)
>  u64 intel_engine_get_active_head(const struct intel_engine_cs *engine)
> diff --git a/drivers/gpu/drm/i915/intel_lrc.c 
> b/drivers/gpu/drm/i915/intel_lrc.c
> index dfafc3f710d6..4eead104cd9c 100644
> --- a/drivers/gpu/drm/i915/intel_lrc.c
> +++ b/drivers/gpu/drm/i915/intel_lrc.c
> @@ -2316,6 +2316,7 @@ int logical_render_ring_init(struct intel_engine_cs 
> *engine)
>                           ret);
>         }
>  
> +       intel_whitelist_workarounds_init(engine);
>         intel_engine_workarounds_init(engine);

So to play devil's advocate:

intel_engine_init_whitelist(engine);
intel_engine_init_workarounds(engine);

object_verb_subject
        object = intel_engine
        verb = init
        subject / subcluase.

I guess there's one more to see how it fits into the pattern...
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to