[PATCH i-g-t 27/28] xe/oa/oa-metricset-codegen: Fix XE_OA_FORMAT_PEC64u64 offsets

Umesh Nerlige Ramappa umesh.nerlige.ramappa at intel.com
Thu Jun 20 23:44:37 UTC 2024


On Thu, Jun 20, 2024 at 01:00:52PM -0700, Ashutosh Dixit wrote:
>From: José Roberto de Souza <jose.souza at intel.com>
>
>XE_OA_FORMAT_PEC64u64 only has pec_offset.
>
>Signed-off-by: José Roberto de Souza <jose.souza at intel.com>
>---
> lib/xe/oa-configs/oa-metricset-codegen.py | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
>diff --git a/lib/xe/oa-configs/oa-metricset-codegen.py b/lib/xe/oa-configs/oa-metricset-codegen.py
>index f4a008d690..5fa12f9caf 100644
>--- a/lib/xe/oa-configs/oa-metricset-codegen.py
>+++ b/lib/xe/oa-configs/oa-metricset-codegen.py
>@@ -145,11 +145,7 @@ def generate_metric_sets(args, gen):
>                 metric_set->perf_raw_size = 576;
>                 metric_set->gpu_time_offset = 0;
>                 metric_set->gpu_clock_offset = 1;
>-                metric_set->a_offset = 2;
>-                // metric_set->b_offset = metric_set->a_offset + 38;
>-                // metric_set->c_offset = metric_set->b_offset + 8;
>-                metric_set->perfcnt_offset = metric_set->a_offset + 64;
>-

Okay, you can ignore my earlier comment to remove the above lines, since 
it is removed here.

Reviewed-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa at intel.com>

>+                metric_set->pec_offset = 2;
>             """))
>         else:
>             c(textwrap.dedent("""\
>-- 
>2.41.0
>


More information about the igt-dev mailing list