Re: [igt-dev] [PATCH i-g-t v4 27/31] lib/i915/perf: Update MTL OA timestamp and EU thread config

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

On Tue, 21 Mar 2023 17:05:19 -0700, Umesh Nerlige Ramappa wrote:
>
> MTL has configuration similar to DG2 for OA timestamp and threads per
> EU. Update the same for MTL.
>
> Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
> ---
>  lib/i915/perf.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/lib/i915/perf.c b/lib/i915/perf.c
> index d8624dc0..24ef3819 100644
> --- a/lib/i915/perf.c
> +++ b/lib/i915/perf.c
> @@ -432,6 +432,14 @@ intel_perf_for_devinfo(uint32_t device_id,
>		else
>			return unsupported_i915_perf_platform(perf);
>	} else if (devinfo->is_meteorlake) {
> +		perf->devinfo.eu_threads_count = 8;
> +		/* OA reports have the timestamp value shifted to the
> +		 * right by 1 bits, it also means we cannot use the
> +		 * top bit for comparison.
> +		 */
> +		perf->devinfo.oa_timestamp_shift = -1;
> +		perf->devinfo.oa_timestamp_mask = 0x7fffffff;
> +

Is it possible to add a Bspec reference for this? Otherwise:

Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com>