[PATCH] drm/xe/xe_tracer: Align fence output format in ftrace log

Lucas De Marchi lucas.demarchi at intel.com
Tue Mar 12 12:59:50 UTC 2024


On Tue, Mar 12, 2024 at 06:53:38AM +0000, Shuicheng Lin wrote:
>The fence print in xe_sched_job class is with "0x" prefix.

it's actually xe_sched_job that got missed in commit
7a975748d4dc ("drm/xe: Use pointers in trace events")

Lucas De Marchi

>Add the "0x" prefix for other fence output, so that they
>could be searched in the same way.
>
>Signed-off-by: Shuicheng Lin <shuicheng.lin at intel.com>
>---
> drivers/gpu/drm/xe/xe_trace.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/drivers/gpu/drm/xe/xe_trace.h b/drivers/gpu/drm/xe/xe_trace.h
>index 4ddc55527f9a..c3ac3b2bad3a 100644
>--- a/drivers/gpu/drm/xe/xe_trace.h
>+++ b/drivers/gpu/drm/xe/xe_trace.h
>@@ -36,7 +36,7 @@ DECLARE_EVENT_CLASS(xe_gt_tlb_invalidation_fence,
> 			   __entry->seqno = fence->seqno;
> 			   ),
>
>-		    TP_printk("fence=%p, seqno=%d",
>+		    TP_printk("fence=0x%p, seqno=%d",
> 			      __entry->fence, __entry->seqno)
> );
>
>@@ -359,7 +359,7 @@ DECLARE_EVENT_CLASS(xe_hw_fence,
> 			   __entry->fence = fence;
> 			   ),
>
>-		    TP_printk("ctx=0x%016llx, fence=%p, seqno=%u",
>+		    TP_printk("ctx=0x%016llx, fence=0x%p, seqno=%u",
> 			      __entry->ctx, __entry->fence, __entry->seqno)
> );
>
>-- 
>2.25.1
>


More information about the Intel-xe mailing list