Skip to contentSkip to conversation

intel: Fix GTT entry setup for aub dump on recent fulsim

PATCH New Zhenyu Wang 1 participant 1 message Dec 29, 2014

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 intel: Fix GTT entry setup for aub dump on recent fulsim

On recent fulsim GTT entry setup for aub dump seems to need mem type as GTT_ENTRY instead of NONLOCAL. NONLOCAL would write data in main memory space which seems wrong on recent fulsim. GTT_ENTRY write would setup GTT memory pool and other required internal buffers. With this I can run aub dump on latest fulsim release without crash.

---
 intel/intel_bufmgr_gem.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
intel/intel_bufmgr_gem.c +2 -1
index 14e92c9..3076111 100644
@@ -3180,7 +3180,8 @@ drm_intel_bufmgr_gem_set_aub_dump(drm_intel_bufmgr *bufmgr, int enable)
 
 	/* Set up the GTT. The max we can handle is 256M */
 	aub_out(bufmgr_gem, CMD_AUB_TRACE_HEADER_BLOCK | ((bufmgr_gem->gen >= 8 ? 6 : 5) - 2));
-	aub_out(bufmgr_gem, AUB_TRACE_MEMTYPE_NONLOCAL | 0 | AUB_TRACE_OP_DATA_WRITE);
+	/* Need to use GTT_ENTRY type for recent fulsim */
+	aub_out(bufmgr_gem, AUB_TRACE_MEMTYPE_GTT_ENTRY | 0 | AUB_TRACE_OP_DATA_WRITE);
 	aub_out(bufmgr_gem, 0); /* subtype */
 	aub_out(bufmgr_gem, 0); /* offset */
 	aub_out(bufmgr_gem, gtt_size); /* size */
Signature · 6 lines
2.1.4

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx