gate apple gpu collectors + revert readme change

This commit is contained in:
henrygd
2026-02-18 14:57:41 -05:00
parent f792f9b102
commit f17ffc21b8
2 changed files with 12 additions and 7 deletions

View File

@@ -708,13 +708,18 @@ func (gm *GPUManager) resolveLegacyCollectorPriority(caps gpuCapabilities) []col
priorities = append(priorities, collectorSourceIntelGpuTop)
}
// Apple collectors are currently opt-in only for testing.
// Enable them with GPU_COLLECTOR=macmon or GPU_COLLECTOR=powermetrics.
// TODO: uncomment below when Apple collectors are confirmed to be working.
//
// Prefer macmon on macOS (no sudo). Fall back to powermetrics if present.
if caps.hasMacmon {
priorities = append(priorities, collectorSourceMacmon)
} else if caps.hasPowermetrics {
priorities = append(priorities, collectorSourcePowermetrics)
}
// Keep nvtop as a legacy last resort only when no vendor collector exists.
// if caps.hasMacmon {
// priorities = append(priorities, collectorSourceMacmon)
// } else if caps.hasPowermetrics {
// priorities = append(priorities, collectorSourcePowermetrics)
// }
// Keep nvtop as a last resort only when no vendor collector exists.
if len(priorities) == 0 && caps.hasNvtop {
priorities = append(priorities, collectorSourceNVTop)
}