Re: [PATCH] drm/xe: Remove unused xe_vma_flush tracepoint

[Steven Rostedt <rostedt@goodmis.org>]

On Tue,  3 Nov 2026 00:11:48 +0530
Torra Sowmya <sowmyatorra@gmail.com> wrote:

> Commit f3e9b1f43458 ("drm/xe: Remove async worker and rework sync
> binds") removed the async worker and its calls to
> trace_xe_vma_flush(), leaving the event definition unused.
> 
> Remove the definition from xe_trace_bo.h to avoid generating code
> and data for an event with no remaining callers.
> 
> With the tested x86-64 configuration, size reports a reduction of
> 452 bytes in xe_trace_bo.o, from 20262 to 19810 bytes: 156 bytes
> in text and 296 bytes in data.

Since this event and the events in your previous patch were removed by
the same commit, please fold the two patches into one.

We want to remove events that their uses were removed together.

Thanks,

-- Steve


> 
> Signed-off-by: Torra Sowmya <sowmyatorra@gmail.com>
> ---
>  drivers/gpu/drm/xe/xe_trace_bo.h | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_trace_bo.h b/drivers/gpu/drm/xe/xe_trace_bo.h
> index 86323cf3be2c..72c4107adc13 100644
> --- a/drivers/gpu/drm/xe/xe_trace_bo.h
> +++ b/drivers/gpu/drm/xe/xe_trace_bo.h
> @@ -115,11 +115,6 @@ DECLARE_EVENT_CLASS(xe_vma,
>  			      __entry->end, __entry->ptr)
>  )
>  
> -DEFINE_EVENT(xe_vma, xe_vma_flush,
> -	     TP_PROTO(struct xe_vma *vma),
> -	     TP_ARGS(vma)
> -);
> -
>  DEFINE_EVENT(xe_vma, xe_vma_pagefault,
>  	     TP_PROTO(struct xe_vma *vma),
>  	     TP_ARGS(vma)