IntelCPUSensor: Fix bug in hidden sensors update
Hidden sensors like Tcontrol are not populated to DBus. Therefore if we
try to use generic 'updateValue' function on such sensors, it could
call 'assertThresholds' function which in turn would fail with a
message "trying to set uninitialized interface".
To get rid of such messages from log hidden sensors need to update
their 'value' property directly.
Tested:
- Before
Multiple messages "trying to set uninitialized interface" are printed
to the journal
- After
No messages "trying to set uninitialized interface" are present in
the journal.
Change-Id: I74ac32b431565defbd99aa16732bcd27d51cc3d3
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
diff --git a/src/intel-cpu/IntelCPUSensor.cpp b/src/intel-cpu/IntelCPUSensor.cpp
index 46ca8ee..0c555c3 100644
--- a/src/intel-cpu/IntelCPUSensor.cpp
+++ b/src/intel-cpu/IntelCPUSensor.cpp
@@ -160,7 +160,14 @@
else
{
markAvailable(false);
- updateValue(std::numeric_limits<double>::quiet_NaN());
+ if (show)
+ {
+ updateValue(std::numeric_limits<double>::quiet_NaN());
+ }
+ else
+ {
+ value = std::numeric_limits<double>::quiet_NaN();
+ }
restartRead();
return;
}