[Intel-gfx] [PATCH 1/7] drm/i915/execlists: Move RPCS setup to context pin
Joonas Lahtinen
joonas.lahtinen at linux.intel.com
Tue Jan 8 13:11:56 UTC 2019
Quoting Tvrtko Ursulin (2019-01-08 13:22:46)
> From: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
>
> Configuring RPCS in context image just before pin is sufficient and will
> come extra handy in one of the following patches.
>
> v2:
> * Split image setup a bit differently. (Chris Wilson)
>
> v3:
> * Update context image after reset as well - otherwise the application
> of pinned default state clears the RPCS.
>
> v4:
> * Use local variable throughout the function. (Chris Wilson)
>
> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
> Suggested-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Chris Wilson <chris at chris-wilson.co.uk>
> Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
<SNIP>
> +static u32 make_rpcs(struct drm_i915_private *dev_priv);
This will be gotten rid of in upcoming patches, so ok.
> if (rcs) {
> regs[CTX_LRI_HEADER_2] = MI_LOAD_REGISTER_IMM(1);
> - CTX_REG(regs, CTX_R_PWR_CLK_STATE, GEN8_R_PWR_CLK_STATE,
> - make_rpcs(dev_priv));
> + CTX_REG(regs, CTX_R_PWR_CLK_STATE, GEN8_R_PWR_CLK_STATE, 0);
While here, we could poison the value with something nasty as code flow
assumes it to be overwritten in next call.
Reviewed-by: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
Regards, Joonas
More information about the Intel-gfx
mailing list