From faddb5d497178f7ca257408bf970f3aec3cca59c Mon Sep 17 00:00:00 2001
From: Felix DeGrood <felix.j.degrood@intel.com>
Date: Mon, 3 Nov 2025 20:54:28 +0000
Subject: [PATCH] anv/pps: remove assert for double init

pps initializes perf counter multiple times, once from
GpuDataSource::register_data_source and once from
GpuDataSource::OnSetup. This is fine, except we should replace
failing assert with skip on second call.

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/38224>
---
 src/intel/ds/intel_pps_driver.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/intel/ds/intel_pps_driver.cc b/src/intel/ds/intel_pps_driver.cc
index 09fcba27f27..8a038c3cceb 100644
--- a/src/intel/ds/intel_pps_driver.cc
+++ b/src/intel/ds/intel_pps_driver.cc
@@ -73,7 +73,8 @@ bool IntelDriver::init_perfcnt()
     */
    this->clock_id = intel_pps_clock_id(drm_device.gpu_num);
 
-   assert(!perf && "Intel perf should not be initialized at this point");
+   if (perf)
+      return true;
 
    perf = std::make_unique<IntelPerf>(drm_device.fd);