Skip to content

Commit 2eb0964

Browse files
icklejnikula
authored andcommitted
drm/i915: Restore relaxed padding (OCL_OOB_SUPPRES_ENABLE) for skl+
This bit was fliped on for "syncing dependencies between camera and graphics". BSpec has no recollection why, and it is causing unrecoverable GPU hangs with Vulkan compute workloads. From BSpec, setting bit5 to 0 enables relaxed padding requirements for buffers, 1D and 2D non-array, non-MSAA, non-mip-mapped linear surfaces; and *must* be set to 0h on skl+ to ensure "Out of Bounds" case is suppressed. Reported-by: Jason Ekstrand <[email protected]> Suggested-by: Jason Ekstrand <[email protected]> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110998 Fixes: 8424171 ("drm/i915/gen9: h/w w/a: syncing dependencies between camera and graphics") Signed-off-by: Chris Wilson <[email protected]> Tested-by: [email protected] Cc: Jason Ekstrand <[email protected]> Cc: Mika Kuoppala <[email protected]> Cc: <[email protected]> # v4.1+ Reviewed-by: Mika Kuoppala <[email protected]> Link: https://patchwork.freedesktop.org/patch/msgid/[email protected] (cherry picked from commit 9d7b01e) Signed-off-by: Jani Nikula <[email protected]>
1 parent bb1a71f commit 2eb0964

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

drivers/gpu/drm/i915/gt/intel_workarounds.c

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -308,11 +308,6 @@ static void gen9_ctx_workarounds_init(struct intel_engine_cs *engine,
308308
FLOW_CONTROL_ENABLE |
309309
PARTIAL_INSTRUCTION_SHOOTDOWN_DISABLE);
310310

311-
/* Syncing dependencies between camera and graphics:skl,bxt,kbl */
312-
if (!IS_COFFEELAKE(i915))
313-
WA_SET_BIT_MASKED(HALF_SLICE_CHICKEN3,
314-
GEN9_DISABLE_OCL_OOB_SUPPRESS_LOGIC);
315-
316311
/* WaEnableYV12BugFixInHalfSliceChicken7:skl,bxt,kbl,glk,cfl */
317312
/* WaEnableSamplerGPGPUPreemptionSupport:skl,bxt,kbl,cfl */
318313
WA_SET_BIT_MASKED(GEN9_HALF_SLICE_CHICKEN7,

0 commit comments

Comments
 (0)