fix: ignore metrics that use ref-cycles when ref-cycles not supported

harp-intel · harp-intel · commit a1728e350b87 · 2025-11-07T15:27:31.000-08:00
Signed-off-by: Harper, Jason M &lt;jason.m.harper@intel.com&gt;
diff --git a/cmd/metrics/loader_perfmon_event_core.go b/cmd/metrics/loader_perfmon_event_core.go
@@ -120,7 +120,7 @@ func (event CoreEvent) IsCollectable(metadata Metadata) bool {
 			return false // Off-core response events are not supported in process or cgroup scope
 		}
 	}
-	if !metadata.SupportsRefCycles && strings.Contains(event.EventName, "ref-cycles") {
+	if !metadata.SupportsRefCycles && strings.Contains(event.EventName, "CPU_CLK_UNHALTED.REF_TSC") { // AKA ref-cycles
 		slog.Debug("Ref-cycles events not supported", slog.String("event", event.EventName))
 		return false // ref-cycles events are not supported
 	}

Original file line number	Diff line number	Diff line change
`@@ -120,7 +120,7 @@ func (event CoreEvent) IsCollectable(metadata Metadata) bool {`
`120`	`120`	`return false // Off-core response events are not supported in process or cgroup scope`
`121`	`121`	`}`
`122`	`122`	`}`
`123`		`- if !metadata.SupportsRefCycles && strings.Contains(event.EventName, "ref-cycles") {`
	`123`	`+ if !metadata.SupportsRefCycles && strings.Contains(event.EventName, "CPU_CLK_UNHALTED.REF_TSC") { // AKA ref-cycles`
`124`	`124`	`slog.Debug("Ref-cycles events not supported", slog.String("event", event.EventName))`
`125`	`125`	`return false // ref-cycles events are not supported`
`126`	`126`	`}`