[Intel-gfx] [PATCH I-g-t 2/2] Mediafill/skl: Remove redundant field to fix GPU hang

Zhao Yakui yakui.zhao at intel.com
Thu Nov 6 09:02:31 CET 2014


After applying the commit(982f7eb238a0898c456e0574dee7c4507738d75f), the OUT_RELOC is
updated on Broadwell and later, which is to handle the 64-bit field of gfx address
internally. In such case some commands should be fixed, otherwise GPU hang will
be triggered when running gem_media_fill.
(It is already fixed on Broadwell)

Signed-off-by: Zhao Yakui <yakui.zhao at intel.com>
---
 lib/media_fill_gen9.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/lib/media_fill_gen9.c b/lib/media_fill_gen9.c
index 6c6ba89..3fd2181 100644
--- a/lib/media_fill_gen9.c
+++ b/lib/media_fill_gen9.c
@@ -205,12 +205,10 @@ gen9_emit_state_base_address(struct intel_batchbuffer *batch)
 
 	/* surface */
 	OUT_RELOC(batch->bo, I915_GEM_DOMAIN_SAMPLER, 0, BASE_ADDRESS_MODIFY);
-	OUT_BATCH(0);
 
 	/* dynamic */
 	OUT_RELOC(batch->bo, I915_GEM_DOMAIN_RENDER | I915_GEM_DOMAIN_INSTRUCTION,
 		0, BASE_ADDRESS_MODIFY);
-	OUT_BATCH(0);
 
 	/* indirect */
 	OUT_BATCH(0);
@@ -218,7 +216,6 @@ gen9_emit_state_base_address(struct intel_batchbuffer *batch)
 
 	/* instruction */
 	OUT_RELOC(batch->bo, I915_GEM_DOMAIN_INSTRUCTION, 0, BASE_ADDRESS_MODIFY);
-	OUT_BATCH(0);
 
 	/* general state buffer size */
 	OUT_BATCH(0xfffff000 | 1);
-- 
1.8.5.dirty




More information about the Intel-gfx mailing list