From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by gabe.freedesktop.org (Postfix) with ESMTPS id 03EE310E20C for ; Wed, 22 Mar 2023 00:05:28 +0000 (UTC) From: Umesh Nerlige Ramappa To: igt-dev@lists.freedesktop.org, Ashutosh Dixit , kamil.konieczny@linux.intel.com Date: Tue, 21 Mar 2023 17:05:02 -0700 Message-Id: <20230322000523.3310187-11-umesh.nerlige.ramappa@intel.com> In-Reply-To: <20230322000523.3310187-1-umesh.nerlige.ramappa@intel.com> References: <20230322000523.3310187-1-umesh.nerlige.ramappa@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [igt-dev] [PATCH i-g-t v4 10/31] i915/perf: Add OAM format type List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Lionel G Landwerlin Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: Similar to OAG (Global) and OAR (Render), OAM (Observability Architecture for Media) adds some new OA formats that require different metrics configurations for B and C counters. Don't do a sanity check for B and C counters for OAM formats until such metrics are available. v2: Improve commit message (Kamil) Signed-off-by: Umesh Nerlige Ramappa Reviewed-by: Kamil Konieczny --- tests/i915/perf.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/i915/perf.c b/tests/i915/perf.c index d116667d..a2bc6dc1 100644 --- a/tests/i915/perf.c +++ b/tests/i915/perf.c @@ -91,6 +91,7 @@ struct accumulator { enum { OAG, OAR, + OAM, MAX_OA_TYPE, }; @@ -1022,7 +1023,7 @@ gen8_sanity_check_test_oa_reports(const uint32_t *oa_report0, /* The TestOa metric set defines all B counters to be a * multiple of the gpu clock */ - if (format.n_b) { + if (format.n_b && (format.oa_type == OAG || format.oa_type == OAR)) { if (clock_delta > 0) { b = rpt1_b[0] - rpt0_b[0]; igt_debug("B0: delta = %"PRIu32"\n", b); -- 2.36.1