From d50c99b71dd3db82bb4dec2ac89c63183cafec82 Mon Sep 17 00:00:00 2001
From: Andrew Tan <Andrew.Tan@windriver.com>
Date: Wed, 9 Nov 2022 17:39:07 +0000
Subject: [PATCH] Added CPU C-state to collect output

Added 'cpupower monitor' to collect_host
Output is saved to /var/extra/host.info.
Certain zt-proteus servers in our labs had C6 state enabled
which led to poor performance and timeouts occurring during bootstrap
process. The added information will help troubleshooting effort.

Test Plan:
PASS: Install and bootstrap AIO-SX, ran 'colllect all'.
PASS: Verify /var/extra/host.info file is updated on controller nodes.
PASS: Verified 'cpupower monitor' command on zt-proteus servers.

Closes-Bug: 1996238
Signed-off-by: Andrew Tan <Andrew.Tan@windriver.com>
Change-Id: I46970ee0ed032b508ced820feb5df142bd954d60
---
 tools/collector/debian-scripts/collect_host | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/tools/collector/debian-scripts/collect_host b/tools/collector/debian-scripts/collect_host
index 1de68ea6..9b99a824 100755
--- a/tools/collector/debian-scripts/collect_host
+++ b/tools/collector/debian-scripts/collect_host
@@ -237,6 +237,10 @@ function collect_extra()
         topology >> ${LOGFILE} 2>>${COLLECT_ERROR_LOG}
     fi
 
+    # CPU C-state power info
+    delimiter ${LOGFILE} "cpupower monitor"
+    cpupower monitor >> ${LOGFILE} 2>>${COLLECT_ERROR_LOG}
+    
     LOGFILE="${EXTRA_DIR}/memory.info"
     echo    "${hostname}: Memory Info .......: ${LOGFILE}"