Re: [PATCH v4 3/9] drm/xe: Add a trace point for GPU work period

["Dixit, Ashutosh" <ashutosh.dixit@intel.com>]

On Thu, 02 Oct 2025 07:42:33 -0700, Rodrigo Vivi wrote:
>
> > diff --git a/include/trace/gpu_work_period.h b/include/trace/gpu_work_period.h
> > new file mode 100644
> > index 000000000000..e06467625705
> > --- /dev/null
> > +++ b/include/trace/gpu_work_period.h
> > @@ -0,0 +1,59 @@
> > +/* SPDX-License-Identifier: MIT */
> > +/*
> > + * Copyright © 2025 Intel Corporation
> > + */
> > +
> > +#undef TRACE_SYSTEM
> > +#define TRACE_SYSTEM power
> > +
> > +#if !defined(_TRACE_GPU_WORK_PERIOD_H) || defined(TRACE_HEADER_MULTI_READ)
> > +#define _TRACE_GPU_WORK_PERIOD_H
> > +
> > +#include <linux/tracepoint.h>
> > +
> > +TRACE_EVENT(gpu_work_period,
>
> oh! not again please!
> Stop trying to brute force this.
>
> If it is a xe trace you need to add 'xe_' prefix.
>
> If you really need the 'gpu_' prefix because of Android, this
> needs to go through the drm layer, not a driver specific.

Also, drm layer patches need to be sent to the following mail lists:

	intel-gfx@lists.freedesktop.org
	dri-devel@lists.freedesktop.org
	intel-xe@lists.freedesktop.org

Also, see previous similar comments on the gpu_frequency tracepoint series.

Thanks.
--
Ashutosh