MangoHud icon indicating copy to clipboard operation
MangoHud copied to clipboard

Access to more sensors

Open shmerl opened this issue 1 year ago • 7 comments

Currently gpu_temp seems to be reading edge sensor value in case of AMD GPU. It would be useful to have access to more sensors, such as junction or really any lm-sensors sensor that's available so it could be a generic solution in case some new senors appear.

shmerl avatar Mar 14 '23 23:03 shmerl

That's known long-standing limitation. It doesn't use lm_sensors for reading gpu sensors and use hardcoded hwmon paths instead. About showing more amd gpu temps there were multiple attempts to make this happen:

  • There is old amdgpu-tj branch with added junction temperature sensor.
  • There is also not merged PR #841 which added support for both mem/junction temperatures.
  • I'm personally for a long time using my own variation which add separate gpu_fan/gpu_temp options which looks like this: mangohud-amd-temps

CkNoSFeRaTU avatar Mar 26 '23 20:03 CkNoSFeRaTU

It's probably useful to add actual generic way to read any of the lm-sensors, since it would give access to a whole bunch of them at once.

shmerl avatar Mar 26 '23 20:03 shmerl

Unlike lm-sensors, we don't have a lot of space to label each temp. If we just give access to all temps, how would you choose which ones you want? How do you know which one is which?

flightlessmango avatar Mar 26 '23 20:03 flightlessmango

It would be up to the user to decide how to label them. I.e. it could a generic advanced feature and those who would use it would presumably know what each sensor means. Probably also makes sense to allow configuring the measurement string, like °C, rpm and so on.

I.e. it's not the task of the hud to explain how to use sensors and what they mean because there are so many and new can appear too. It can just give access to them as an option and be flexible enough to make any custom labeling.

shmerl avatar Mar 26 '23 20:03 shmerl

That's known long-standing limitation. It doesn't use lm_sensors for reading gpu sensors and use hardcoded hwmon paths instead. About showing more amd gpu temps there were multiple attempts to make this happen:

* There is old [amdgpu-tj branch](https://github.com/flightlessmango/MangoHud/tree/amdgpu-tj) with added junction temperature sensor.

* There is also not merged PR [Additional amdgpu temps #841](https://github.com/flightlessmango/MangoHud/pull/841) which added support for both mem/junction temperatures.

* I'm personally for a long time using my own [variation](https://github.com/flightlessmango/MangoHud/files/11072878/mangohud-9999-amdgpu-fan-temp.patch.tar.gz) which add separate `gpu_fan`/`gpu_temp` options which looks like this:
  ![mangohud-amd-temps](https://user-images.githubusercontent.com/2136776/227800892-1ddab14f-29f3-406b-a7f9-53c9414e3074.png)

CkNoSFeRaTU How do I apply the patch file to multiple files? I can't find it in the man patch

hajosattila avatar May 02 '23 16:05 hajosattila

for example I have a very fancy PSU from corsair that reports this when using lm_sensors:

sensors
corsairpsu-hid-3-3
Adapter: HID adapter
v_in:        230.00 V  
v_out +12v:   11.95 V  (crit min =  +8.41 V, crit max = +15.59 V)
v_out +5v:     5.05 V  (crit min =  +3.50 V, crit max =  +6.50 V)
v_out +3.3v:   3.31 V  (crit min =  +2.31 V, crit max =  +4.30 V)
psu fan:        0 RPM
vrm temp:     +50.5°C  (crit = +70.0°C)
case temp:    +52.5°C  (crit = +70.0°C)
power total:  44.00 W  
power +12v:   28.00 W  
power +5v:    14.00 W  
power +3.3v:   3.00 W  
curr +12v:     3.25 A  (crit max = +112.50 A)
curr +5v:      2.81 A  (crit max = +40.00 A)
curr +3.3v:  1000.00 mA (crit max = +40.00 A)

It would be nice to have a way to put some sensors in some configuration file... to map the lmsensor name to an alias and put it in mangoapp or mangohud.

Right now I have amd gpu power consumption but it would be nice to have full system consumption using lm_sensors Thanks :)

mercuriete avatar May 29 '24 13:05 mercuriete

+1, would love a psu sensor reading

amigthea avatar Aug 06 '24 10:08 amigthea