Re: [PATCH] tools/xe-perf-recorder: Add mmio-trigger support

[Shekhar Chauhan <shekhar.chauhan@intel.com>]

On 6/30/2026 3:21, Dixit, Ashutosh wrote:
> On Thu, 18 Jun 2026 22:02:17 -0700, Shekhar Chauhan wrote:
>> Add mmio-trigger support to xe-perf-recorder, to justify the
>> whitelisting of OA MMIO Trigger Registers in XeKMD.
> We can't have these compile warnings:
>
> $ ninja -C build
> ninja: Entering directory `build'
> [4/952] Compiling C object tools/xe-perf/xe-perf-recorder.p/xe_perf_recorder.c.o
> ../tools/xe-perf/xe_perf_recorder.c:41:9: warning: ‘ALIGN’ redefined
>     41 | #define ALIGN(v, a) (((v) + (a)-1) & ~((a)-1))
>        |         ^~~~~
> In file included from ../lib/intel_batchbuffer.h:10,
>                   from ../tools/xe-perf/xe_perf_recorder.c:30:
> ../lib/drmtest.h:100:9: note: this is the location of the previous definition
>    100 | #define ALIGN(v, a) ALIGN_MASK(v, (typeof(v))(a) - 1)
>        |         ^~~~~
> ../tools/xe-perf/xe_perf_recorder.c:42:9: warning: ‘ARRAY_SIZE’ redefined
>     42 | #define ARRAY_SIZE(arr) (sizeof(arr)/sizeof((arr)[0]))
>        |         ^~~~~~~~~~
> ../lib/drmtest.h:91:9: note: this is the location of the previous definition
>     91 | #define ARRAY_SIZE(arr) (sizeof(arr)/sizeof(arr[0]))
>        |         ^~~~~~~~~~

Fixed in the new revision.

-shekhar

>> Signed-off-by: Shekhar Chauhan <shekhar.chauhan@intel.com>

-- 
Shekhar Chauhan
Linux Graphics Software Engineer
Intel Corporation