Message ID | 20190415114307.1356-1-tvrtko.ursulin@linux.intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | drm/i915/icl: Whitelist GEN9_SLICE_COMMON_ECO_CHICKEN1 | expand |
Quoting Tvrtko Ursulin (2019-04-15 12:43:07) > From: Tvrtko Ursulin <tvrtko.ursulin@intel.com> > > WaEnableStateCacheRedirectToCS context workaround configures the L3 cache > to benefit 3d workloads but media has different requirements. > > Whitelist the register to allow media re-configuring it to their liking. > > Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> > Cc: kevin.ma@intel.com > Cc: xiaogang.li@intel.com > --- > drivers/gpu/drm/i915/intel_workarounds.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/gpu/drm/i915/intel_workarounds.c b/drivers/gpu/drm/i915/intel_workarounds.c > index ccaf63679435..6458d161204f 100644 > --- a/drivers/gpu/drm/i915/intel_workarounds.c > +++ b/drivers/gpu/drm/i915/intel_workarounds.c > @@ -1050,6 +1050,9 @@ static void icl_whitelist_build(struct i915_wa_list *w) > > /* WaAllowUMDToModifySamplerMode:icl */ > whitelist_reg(w, GEN10_SAMPLER_MODE); > + > + /* WaEnableStateCacheRedirectToCS:icl */ > + whitelist_reg(w, GEN9_SLICE_COMMON_ECO_CHICKEN1); Have we checked this is context saved? -Chris
On 15/04/2019 12:45, Chris Wilson wrote: > Quoting Tvrtko Ursulin (2019-04-15 12:43:07) >> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com> >> >> WaEnableStateCacheRedirectToCS context workaround configures the L3 cache >> to benefit 3d workloads but media has different requirements. >> >> Whitelist the register to allow media re-configuring it to their liking. >> >> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> >> Cc: kevin.ma@intel.com >> Cc: xiaogang.li@intel.com >> --- >> drivers/gpu/drm/i915/intel_workarounds.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/drivers/gpu/drm/i915/intel_workarounds.c b/drivers/gpu/drm/i915/intel_workarounds.c >> index ccaf63679435..6458d161204f 100644 >> --- a/drivers/gpu/drm/i915/intel_workarounds.c >> +++ b/drivers/gpu/drm/i915/intel_workarounds.c >> @@ -1050,6 +1050,9 @@ static void icl_whitelist_build(struct i915_wa_list *w) >> >> /* WaAllowUMDToModifySamplerMode:icl */ >> whitelist_reg(w, GEN10_SAMPLER_MODE); >> + >> + /* WaEnableStateCacheRedirectToCS:icl */ >> + whitelist_reg(w, GEN9_SLICE_COMMON_ECO_CHICKEN1); > > Have we checked this is context saved? It is listed as such. But I haven't checked on actual hardware. Regards, Tvrtko
Quoting Tvrtko Ursulin (2019-04-15 13:25:08) > > On 15/04/2019 12:45, Chris Wilson wrote: > > Quoting Tvrtko Ursulin (2019-04-15 12:43:07) > >> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com> > >> > >> WaEnableStateCacheRedirectToCS context workaround configures the L3 cache > >> to benefit 3d workloads but media has different requirements. > >> > >> Whitelist the register to allow media re-configuring it to their liking. > >> > >> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> > >> Cc: kevin.ma@intel.com > >> Cc: xiaogang.li@intel.com > >> --- > >> drivers/gpu/drm/i915/intel_workarounds.c | 3 +++ > >> 1 file changed, 3 insertions(+) > >> > >> diff --git a/drivers/gpu/drm/i915/intel_workarounds.c b/drivers/gpu/drm/i915/intel_workarounds.c > >> index ccaf63679435..6458d161204f 100644 > >> --- a/drivers/gpu/drm/i915/intel_workarounds.c > >> +++ b/drivers/gpu/drm/i915/intel_workarounds.c > >> @@ -1050,6 +1050,9 @@ static void icl_whitelist_build(struct i915_wa_list *w) > >> > >> /* WaAllowUMDToModifySamplerMode:icl */ > >> whitelist_reg(w, GEN10_SAMPLER_MODE); > >> + > >> + /* WaEnableStateCacheRedirectToCS:icl */ > >> + whitelist_reg(w, GEN9_SLICE_COMMON_ECO_CHICKEN1); > > > > Have we checked this is context saved? > > It is listed as such. But I haven't checked on actual hardware. Just posted a selftest that should allow us to verify the whitelist in future. -Chris
diff --git a/drivers/gpu/drm/i915/intel_workarounds.c b/drivers/gpu/drm/i915/intel_workarounds.c index ccaf63679435..6458d161204f 100644 --- a/drivers/gpu/drm/i915/intel_workarounds.c +++ b/drivers/gpu/drm/i915/intel_workarounds.c @@ -1050,6 +1050,9 @@ static void icl_whitelist_build(struct i915_wa_list *w) /* WaAllowUMDToModifySamplerMode:icl */ whitelist_reg(w, GEN10_SAMPLER_MODE); + + /* WaEnableStateCacheRedirectToCS:icl */ + whitelist_reg(w, GEN9_SLICE_COMMON_ECO_CHICKEN1); } void intel_engine_init_whitelist(struct intel_engine_cs *engine)