more verbose telemetry logging

Author: planetchili

IntelPresentMon/ControlLib/ControlLib.vcxproj

@@ -206,6 +206,7 @@
     <ClInclude Include="Exceptions.h" />
     <ClInclude Include="IntelPowerTelemetryAdapter.h" />
     <ClInclude Include="Logging.h" />
+    <ClInclude Include="LoggingVerbose.h" />
     <ClInclude Include="MacroHelpers.h" />
     <ClInclude Include="nvapi.h" />
     <ClInclude Include="NvapiWrapper.h" />

IntelPresentMon/ControlLib/ControlLib.vcxproj.filters

@@ -79,6 +79,7 @@
     <ClInclude Include="ctlpvttemp_api.h">
       <Filter>Intel</Filter>
     </ClInclude>
+    <ClInclude Include="LoggingVerbose.h" />
   </ItemGroup>
   <ItemGroup>
     <ClCompile Include="NvapiWrapper.cpp">

IntelPresentMon/ControlLib/IntelPowerTelemetryAdapter.cpp

@@ -33,6 +33,8 @@ namespace pwr::intel
 
     bool IntelPowerTelemetryAdapter::Sample() noexcept
     {
+        pmlog_verb(v::gpu)("Sample called");
+
         LARGE_INTEGER qpc;
         QueryPerformanceCounter(&qpc);
         bool success = true;
@@ -473,9 +475,10 @@ namespace pwr::intel
                     currentSample.vramReadBandwidth,
                     pm_gpu_power_telemetry_info.gpu_mem_read_bandwidth_bps,
                     GpuTelemetryCapBits::gpu_mem_read_bandwidth);
-                pmlog_verb(v::gpu)(std::format("VR read BW bSupported [{}] type [{}] units [{}] data [{}]",
+                pmlog_verb(v::gpu)(std::format("VRAM read BW bSupported [{}] type [{}] units [{}] data_64 [{}] data_double [{}]",
                     currentSample.vramReadBandwidth.bSupported, (int)currentSample.vramReadBandwidth.type,
-                    (int)currentSample.vramReadBandwidth.units, currentSample.vramReadBandwidth.value.datau64));
+                    (int)currentSample.vramReadBandwidth.units, currentSample.vramReadBandwidth.value.datau64,
+                    currentSample.vramReadBandwidth.value.datadouble));
                 if (result != CTL_RESULT_SUCCESS ||
                     !(HasTelemetryCapBit(GpuTelemetryCapBits::gpu_mem_read_bandwidth))) {
                     useNewBandwidthTelemetry = false;
@@ -488,9 +491,10 @@ namespace pwr::intel
                     currentSample.vramWriteBandwidth,
                     pm_gpu_power_telemetry_info.gpu_mem_write_bandwidth_bps,
                     GpuTelemetryCapBits::gpu_mem_write_bandwidth);
-                pmlog_verb(v::gpu)(std::format("VR write BW bSupported [{}] type [{}] units [{}] data [{}]",
+                pmlog_verb(v::gpu)(std::format("VRAM write BW bSupported [{}] type [{}] units [{}] data_64 [{}] data_double [{}]",
                     currentSample.vramWriteBandwidth.bSupported, (int)currentSample.vramWriteBandwidth.type,
-                    (int)currentSample.vramWriteBandwidth.units, currentSample.vramWriteBandwidth.value.datau64));
+                    (int)currentSample.vramWriteBandwidth.units, currentSample.vramWriteBandwidth.value.datau64,
+                    currentSample.vramReadBandwidth.value.datadouble));
                 if (result != CTL_RESULT_SUCCESS ||
                     !(HasTelemetryCapBit(GpuTelemetryCapBits::gpu_mem_write_bandwidth))) {
                     useNewBandwidthTelemetry = false;

IntelPresentMon/ControlLib/Logging.h

@@ -5,15 +5,7 @@
 #include <format>
 #include "igcl_api.h"
 #include "../CommonUtilities/log/Log.h"
-
-namespace pwr::v
-{
-#ifndef VVV_GPU_TELEMETRY // system that reads power, temperature, etc. telemetry of graphics adapter devices
-    inline constexpr bool gpu = false;
-#else
-    inline constexpr bool gpu = true;
-#endif	
-}
+#include "LoggingVerbose.h"
 
 namespace pwr::log
 {

IntelPresentMon/ControlLib/LoggingVerbose.h

@@ -0,0 +1,10 @@
+#pragma once
+
+namespace pwr::v
+{
+#ifndef VVV_GPU_TELEMETRY // system that reads power, temperature, etc. telemetry of graphics adapter devices
+    inline constexpr bool gpu = false;
+#else
+    inline constexpr bool gpu = true;
+#endif	
+}