From: Tvrtko Ursulin <tvrtko.ursu...@intel.com>

Adding a cross-check with ABI config name space and not just relying on
sysfs names.

Signed-off-by: Tvrtko Ursulin <tvrtko.ursu...@intel.com>
Cc: Dmitry Rogozhkin <dmitry.v.rogozh...@intel.com>
---
 tools/intel_gpu_top.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/tools/intel_gpu_top.c b/tools/intel_gpu_top.c
index 41c59a72c09d..81c724d1fe1c 100644
--- a/tools/intel_gpu_top.c
+++ b/tools/intel_gpu_top.c
@@ -376,6 +376,12 @@ static struct engines *discover_engines(char *device)
                        break;
                }
 
+               /* Double check config is an engine config. */
+               if (engine->busy.config >= __I915_PMU_OTHER(0)) {
+                       free((void *)engine->name);
+                       continue;
+               }
+
                engine->class = (engine->busy.config &
                                 (__I915_PMU_OTHER(0) - 1)) >>
                                I915_PMU_CLASS_SHIFT;
-- 
2.32.0

Reply via email to