On Tue, 15 May 2018, Michel Thierry <michel.thie...@intel.com> wrote: > On 5/15/2018 10:13 AM, Jani Nikula wrote: >> On Mon, 14 May 2018, Michel Thierry <michel.thie...@intel.com> wrote: >>> Factor in clear values wherever required while updating destination >>> min/max. >> >> Hi Michel, please elaborate what the intention here is. >> > > Hi Jani, isn't the intention of all the workarounds to prevent gpu > hangs?
Err, sorry for the riddles, I meant with [BACKPORT v4.17-rc5] etc. :) Is this in dinq already? Commit id? BR, Jani. > >> BR, >> Jani. >> >> >> >>> >>> References: HSDES#1604444184 >>> Signed-off-by: Michel Thierry <michel.thie...@intel.com> >>> Cc: mesa-...@lists.freedesktop.org >>> Cc: Mika Kuoppala <mika.kuopp...@linux.intel.com> >>> Cc: Oscar Mateo <oscar.ma...@intel.com> >>> Reviewed-by: Mika Kuoppala <mika.kuopp...@linux.intel.com> >>> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk> >>> Link: >>> https://patchwork.freedesktop.org/patch/msgid/20180510200708.18097-1-michel.thie...@intel.com >>> Cc: sta...@vger.kernel.org >>> Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com> >>> --- >>> drivers/gpu/drm/i915/i915_reg.h | 3 +++ >>> drivers/gpu/drm/i915/intel_engine_cs.c | 4 ++++ >>> 2 files changed, 7 insertions(+) >>> >>> diff --git a/drivers/gpu/drm/i915/i915_reg.h >>> b/drivers/gpu/drm/i915/i915_reg.h >>> index e6a8c0ee7df1..8a69a9275e28 100644 >>> --- a/drivers/gpu/drm/i915/i915_reg.h >>> +++ b/drivers/gpu/drm/i915/i915_reg.h >>> @@ -7326,6 +7326,9 @@ enum { >>> #define SLICE_ECO_CHICKEN0 _MMIO(0x7308) >>> #define PIXEL_MASK_CAMMING_DISABLE (1 << 14) >>> >>> +#define GEN9_WM_CHICKEN3 _MMIO(0x5588) >>> +#define GEN9_FACTOR_IN_CLR_VAL_HIZ (1 << 9) >>> + >>> /* WaCatErrorRejectionIssue */ >>> #define GEN7_SQ_CHICKEN_MBCUNIT_CONFIG _MMIO(0x9030) >>> #define GEN7_SQ_CHICKEN_MBCUNIT_SQINTMOB (1<<11) >>> diff --git a/drivers/gpu/drm/i915/intel_engine_cs.c >>> b/drivers/gpu/drm/i915/intel_engine_cs.c >>> index 4ba139c27fba..f7c25828d3bb 100644 >>> --- a/drivers/gpu/drm/i915/intel_engine_cs.c >>> +++ b/drivers/gpu/drm/i915/intel_engine_cs.c >>> @@ -1149,6 +1149,10 @@ static int gen9_init_workarounds(struct >>> intel_engine_cs *engine) >>> WA_SET_FIELD_MASKED(GEN8_CS_CHICKEN1, GEN9_PREEMPT_GPGPU_LEVEL_MASK, >>> GEN9_PREEMPT_GPGPU_COMMAND_LEVEL); >>> >>> + /* WaClearHIZ_WM_CHICKEN3:bxt,glk */ >>> + if (IS_GEN9_LP(dev_priv)) >>> + WA_SET_BIT_MASKED(GEN9_WM_CHICKEN3, GEN9_FACTOR_IN_CLR_VAL_HIZ); >>> + >>> /* WaVFEStateAfterPipeControlwithMediaStateClear:skl,bxt,glk,cfl */ >>> ret = wa_ring_whitelist_reg(engine, GEN9_CTX_PREEMPT_REG); >>> if (ret) >> -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
