Commit c65908c3 authored by José Roberto de Souza's avatar José Roberto de Souza
Browse files

drm/xe: Remove double new lines in devcoredump 



Right now devcoredump has a new line between '**** GuC CT ****' and
'H2G CTB (all sizes in DW):' while other sections don't have.

v2: remove double new line after IPEHR

Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: Maarten Lankhorst <dev@lankhorst.se>
Cc: Stuart Summers <stuart.summers@intel.com>
Reviewed-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
Reviewed-by: default avatarStuart Summers <stuart.summers@intel.com>
Signed-off-by: default avatarJosé Roberto de Souza <jose.souza@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240123204454.246788-1-jose.souza@intel.com
parent ab5ae65f

drivers/gpu/drm/xe/xe_guc_ct.c

+2 −2
Original line number Diff line number Diff line
@@ -1366,7 +1366,7 @@ void xe_guc_ct_snapshot_print(struct xe_guc_ct_snapshot *snapshot, 
		return;



	if (snapshot->ct_enabled) {

		drm_puts(p, "\nH2G CTB (all sizes in DW):\n");

		drm_puts(p, "H2G CTB (all sizes in DW):\n");

		guc_ctb_snapshot_print(&snapshot->h2g, p);



		drm_puts(p, "\nG2H CTB (all sizes in DW):\n");
@@ -1375,7 +1375,7 @@ void xe_guc_ct_snapshot_print(struct xe_guc_ct_snapshot *snapshot, 
		drm_printf(p, "\tg2h outstanding: %d\n",

			   snapshot->g2h_outstanding);

	} else {

		drm_puts(p, "\nCT disabled\n");

		drm_puts(p, "CT disabled\n");

	}

}

drivers/gpu/drm/xe/xe_hw_engine.c

+1 −1
Original line number Diff line number Diff line
@@ -856,7 +856,7 @@ void xe_hw_engine_snapshot_print(struct xe_hw_engine_snapshot *snapshot, 
	drm_printf(p, "\tDMA_FADDR: 0x%08x_%08x\n",

		   snapshot->reg.ring_dma_fadd_udw,

		   snapshot->reg.ring_dma_fadd);

	drm_printf(p, "\tIPEHR: 0x%08x\n\n", snapshot->reg.ipehr);

	drm_printf(p, "\tIPEHR: 0x%08x\n", snapshot->reg.ipehr);

	if (snapshot->class == XE_ENGINE_CLASS_COMPUTE)

		drm_printf(p, "\tRCU_MODE: 0x%08x\n",

			   snapshot->reg.rcu_mode);