[PATCH v2 9/9] drm/xe: Remove unused GT TLB invalidation trace points

Summers, Stuart stuart.summers at intel.com
Fri Jul 11 21:13:05 UTC 2025


On Wed, 2025-07-02 at 16:42 -0700, Matthew Brost wrote:
> Remove unused GT TLB invalidation trace points after converting to
> used
> GT TLB invalidation jobs. Tracepoint removed were used during early
> bring up of unstable driver, with a stable driver no need to replace
> with new tracepoints.

Makes sense to me. IMO the g2h/h2g messages are more interesting here
for debug anyway.

Reviewed-by: Stuart Summers <stuart.summers at intel.com>

> 
> Signed-off-by: Matthew Brost <matthew.brost at intel.com>
> ---
>  drivers/gpu/drm/xe/xe_trace.h | 16 ----------------
>  1 file changed, 16 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_trace.h
> b/drivers/gpu/drm/xe/xe_trace.h
> index b4a3577df70c..21486a6f693a 100644
> --- a/drivers/gpu/drm/xe/xe_trace.h
> +++ b/drivers/gpu/drm/xe/xe_trace.h
> @@ -45,22 +45,6 @@ DECLARE_EVENT_CLASS(xe_gt_tlb_invalidation_fence,
>                               __get_str(dev), __entry->fence,
> __entry->seqno)
>  );
>  
> -DEFINE_EVENT(xe_gt_tlb_invalidation_fence,
> xe_gt_tlb_invalidation_fence_create,
> -            TP_PROTO(struct xe_device *xe, struct
> xe_gt_tlb_invalidation_fence *fence),
> -            TP_ARGS(xe, fence)
> -);
> -
> -DEFINE_EVENT(xe_gt_tlb_invalidation_fence,
> -            xe_gt_tlb_invalidation_fence_work_func,
> -            TP_PROTO(struct xe_device *xe, struct
> xe_gt_tlb_invalidation_fence *fence),
> -            TP_ARGS(xe, fence)
> -);
> -
> -DEFINE_EVENT(xe_gt_tlb_invalidation_fence,
> xe_gt_tlb_invalidation_fence_cb,
> -            TP_PROTO(struct xe_device *xe, struct
> xe_gt_tlb_invalidation_fence *fence),
> -            TP_ARGS(xe, fence)
> -);
> -
>  DEFINE_EVENT(xe_gt_tlb_invalidation_fence,
> xe_gt_tlb_invalidation_fence_send,
>              TP_PROTO(struct xe_device *xe, struct
> xe_gt_tlb_invalidation_fence *fence),
>              TP_ARGS(xe, fence)



More information about the Intel-xe mailing list