The temperature cannot be read

[songlinshu]

hi
The temperature cannot be read:

use sysinfo::{NetworkExt, NetworksExt, ProcessExt, System, SystemExt,ComponentExt};
fn main() {
  
   let mut sys = System::new_all();  
  
        for component in sys.components_mut() {
           
            
            
            println!("{:?}", component.temperature());
        }
     
 
}

=============
Default host: x86_64-pc-windows-msvc
rustup home: d:\01rustlang.rustup

installed toolchains

stable-i686-pc-windows-gnu
stable-i686-pc-windows-msvc
stable-x86_64-pc-windows-gnu
stable-x86_64-pc-windows-msvc
nightly-x86_64-pc-windows-gnu
nightly-x86_64-pc-windows-msvc (default)

installed targets for active toolchain

i686-pc-windows-gnu
i686-pc-windows-msvc
x86_64-pc-windows-msvc

active toolchain

nightly-x86_64-pc-windows-msvc (default)
rustc 1.58.0-nightly (514b38779 2021-10-22)

Microsoft Windows [Version 10.0.17763.2268]
Windows server 2019

[GuillaumeGomez]

On windows, you need to run sysinfo with Administrator priviledges to get this information. It's indicated in the documentation here.

[songlinshu]

Already an administrator

[GuillaumeGomez]

Then you'll need to look at the code to see what the Windows API is not happy about I guess... The code is here. Good luck. :)

[NotQuiteApex]

Walking through with the VS Code debugger it appears to be stuck on this bit. I'm testing this on Windows 11, I'm not very familiar with the Windows API myself, but it's odd that it thinks it lacks permissions when running as administrator (I set up the winres crate to require elevated perms, so there's no way it'd get to that point without them).

[GuillaumeGomez]

I have no idea. Windows is the platform I use the less and its API are by far the most tricky. In this case, I don't have a windows 11 so I can't do much...

[NotQuiteApex]

I did some more debugging. The HRESULT (_r) returned by the enumerator next call returns a 0x8004100C, which maps to the constant WBEM_E_NOT_SUPPORTED. Very odd, considering there are plenty of other apps out there that can read my CPU temperature just fine!

[NotQuiteApex]

It appears that the method used in the crate isn't even the correct method for reading CPU temperatures, or at the very least it is for sensors that are on the motherboard and not the CPU. See here.

[GuillaumeGomez]

Thanks for the debugging! Interested into sending a fix by any chance?

[NotQuiteApex]

I will have some spare time in the coming weeks, however it appears the solution may involve reading sensors in a variety of different methods depending on the CPU, hard drive, and motherboard. See all the code it takes for LibreHardwareMonitor to read from sensors on Windows.

[GuillaumeGomez]

It's very welcome, thanks! Just to prevent you from doing too much: please test with one CPU (yours?) to see how big it would be. If too big, I think it's simply not worth it for such a small feature.