CPU temp reporting

[AhnHEL]

@meyergru  It looks like that second CPU0 belongs to dev.pchtherm.0 but then why does the formatting in the widget express it as CPU0?

[franco]

I merged this now since I haven't seen a better approach yet:

https://github.com/opnsense/core/commit/eded37411f

It would be nice if configd had a "prefetch" and "serve expired" type of metric but in the average case this is more than enough. A suboptimal workaround can be discussed, but it will always look a bit strange in my opinion (running random actions at boot to fix an edge case on certain hardware).


Cheers,
Franco

[AhnHEL]

@franco I applied the patch, can it omit the tjmax graph outputs?  Its not easy on the eyes.  I'm still having issue with the pchtherm.0 section that shows up as another CPU0.

[franco]

Can you dump your

# configctl system sensors

here?

The 100 degree ones look like thresholds from Intel CPU maybe. I don't have the hardware to verify.

The widget is another issue as the patch only deals with the backend. It would be nicer if the widget would not do too much magic here guestimating where the reading comes from and making similar readings collapsible maybe.

But we can get there step by step. Thanks for testing!


Cheers,
Franco

[AhnHEL]

Thank you, yes.

# configctl system sensors
dev.cpu.0.coretemp.tjmax
dev.cpu.0.temperature
dev.cpu.1.coretemp.tjmax
dev.cpu.1.temperature
dev.cpu.10.coretemp.tjmax
dev.cpu.10.temperature
dev.cpu.11.coretemp.tjmax
dev.cpu.11.temperature
dev.cpu.2.coretemp.tjmax
dev.cpu.2.temperature
dev.cpu.3.coretemp.tjmax
dev.cpu.3.temperature
dev.cpu.4.coretemp.tjmax
dev.cpu.4.temperature
dev.cpu.5.coretemp.tjmax
dev.cpu.5.temperature
dev.cpu.6.coretemp.tjmax
dev.cpu.6.temperature
dev.cpu.7.coretemp.tjmax
dev.cpu.7.temperature
dev.cpu.8.coretemp.tjmax
dev.cpu.8.temperature
dev.cpu.9.coretemp.tjmax
dev.cpu.9.temperature
dev.pchtherm.0.ctt
dev.pchtherm.0.pmtemp
dev.pchtherm.0.t0temp
dev.pchtherm.0.t1temp
dev.pchtherm.0.t2temp
dev.pchtherm.0.temperature