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

Lucas De Marchi lucas.demarchi at intel.com
Wed Mar 13 19:21:30 UTC 2024


On Wed, Mar 13, 2024 at 02:50:52AM +0000, Shuicheng Lin wrote:
>The fence print in xe_gt_tlb_invalidation_fence and xe_hw_fence
>is with "%p", change fence print in xe_sched_job to "%p" also.
>
>Cc: Lucas De Marchi <lucas.demarchi at intel.com>
>Signed-off-by: Shuicheng Lin <shuicheng.lin at intel.com>

Reviewed-by: Lucas De Marchi <lucas.demarchi at intel.com>

thanks
Lucas De Marchi

>---
> drivers/gpu/drm/xe/xe_trace.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/drivers/gpu/drm/xe/xe_trace.h b/drivers/gpu/drm/xe/xe_trace.h
>index 4ddc55527f9a..2d3beaffc308 100644
>--- a/drivers/gpu/drm/xe/xe_trace.h
>+++ b/drivers/gpu/drm/xe/xe_trace.h
>@@ -258,7 +258,7 @@ DECLARE_EVENT_CLASS(xe_sched_job,
> 			     __field(u32, guc_state)
> 			     __field(u32, flags)
> 			     __field(int, error)
>-			     __field(u64, fence)
>+			     __field(struct dma_fence *, fence)
> 			     __field(u64, batch_addr)
> 			     ),
>
>@@ -269,11 +269,11 @@ DECLARE_EVENT_CLASS(xe_sched_job,
> 			   atomic_read(&job->q->guc->state);
> 			   __entry->flags = job->q->flags;
> 			   __entry->error = job->fence->error;
>-			   __entry->fence = (unsigned long)job->fence;
>+			   __entry->fence = job->fence;
> 			   __entry->batch_addr = (u64)job->batch_addr[0];
> 			   ),
>
>-		    TP_printk("fence=0x%016llx, seqno=%u, guc_id=%d, batch_addr=0x%012llx, guc_state=0x%x, flags=0x%x, error=%d",
>+		    TP_printk("fence=%p, seqno=%u, guc_id=%d, batch_addr=0x%012llx, guc_state=0x%x, flags=0x%x, error=%d",
> 			      __entry->fence, __entry->seqno, __entry->guc_id,
> 			      __entry->batch_addr, __entry->guc_state,
> 			      __entry->flags, __entry->error)
>-- 
>2.25.1
>


More information about the Intel-xe mailing list