From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by gabe.freedesktop.org (Postfix) with ESMTPS id C2F9A10E248 for ; Tue, 25 Oct 2022 20:07:18 +0000 (UTC) From: Umesh Nerlige Ramappa To: igt-dev@lists.freedesktop.org Date: Tue, 25 Oct 2022 20:06:57 +0000 Message-Id: <20221025200709.83314-18-umesh.nerlige.ramappa@intel.com> In-Reply-To: <20221025200709.83314-1-umesh.nerlige.ramappa@intel.com> References: <20221025200709.83314-1-umesh.nerlige.ramappa@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [igt-dev] [PATCH 17/29] lib/i915/perf: fixup report validity List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: From: Lionel Landwerlin It appears that on Gfx12+ there is no more validity bit in the reports. Signed-off-by: Lionel Landwerlin Reviewed-by: Umesh Nerlige Ramappa --- lib/i915/perf_data_reader.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/lib/i915/perf_data_reader.c b/lib/i915/perf_data_reader.c index e69189ac..857bcd3a 100644 --- a/lib/i915/perf_data_reader.c +++ b/lib/i915/perf_data_reader.c @@ -45,13 +45,14 @@ oa_report_ctx_is_valid(const struct intel_perf_devinfo *devinfo, { const uint32_t *report = (const uint32_t *) _report; - if (devinfo->graphics_ver < 8) { + if (devinfo->graphics_ver < 8) return false; /* TODO */ - } else if (devinfo->graphics_ver == 8) { + if (devinfo->graphics_ver >= 12) + return true; /* Always valid */ + if (devinfo->graphics_ver == 8) return report[0] & (1ul << 25); - } else if (devinfo->graphics_ver > 8) { + if (devinfo->graphics_ver > 8) return report[0] & (1ul << 16); - } return false; } -- 2.25.1