flightlessmango / MangoHud

A Vulkan and OpenGL overlay for monitoring FPS, temperatures, CPU/GPU load and more. Discord: https://discordapp.com/invite/Gj5YmBb
MIT License
6.47k stars 287 forks source link

On first gen Ryzen, CPU temps are reported incorrectly on 6.7.4 #1228

Open BlueGoliath opened 8 months ago

BlueGoliath commented 8 months ago

On first gen Ryzen(1800X), CPU temps are reported 20 degrees higher than they should be. This may not sound like an issue as first-generation Ryzen CPUs have a 20-degree offset, but it's not what MangoHud reported before.

Strangely, CPU-X reports the correct temps. Maybe MangoHud should switch to whatever CPU-X is using?

flightlessmango commented 8 months ago

Can you identify which sensor mangohud is using through sensors ?

BlueGoliath commented 8 months ago

I think it's Tctl here:

k10temp-pci-00c3
Adapter: PCI adapter
Tctl:         +49.2°C  
Tdie:         +29.2°C  

Here's everything:

Adapter: ISA adapter
Vcore:                 680.00 mV (min =  +0.00 V, max =  +1.74 V)
in1:                     1.27 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
AVCC:                    3.30 V  (min =  +2.98 V, max =  +3.63 V)
+3.3V:                   3.30 V  (min =  +2.98 V, max =  +3.63 V)
in4:                     1.86 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
in5:                     1.18 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
in6:                     1.37 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
3VSB:                    3.44 V  (min =  +2.98 V, max =  +3.63 V)
Vbat:                    3.26 V  (min =  +2.70 V, max =  +3.63 V)
in9:                     0.00 V  (min =  +0.00 V, max =  +0.00 V)
in10:                  736.00 mV (min =  +0.00 V, max =  +0.00 V)  ALARM
in11:                    1.08 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
in12:                    1.68 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
in13:                  928.00 mV (min =  +0.00 V, max =  +0.00 V)  ALARM
in14:                    1.82 V  (min =  +0.00 V, max =  +0.00 V)  ALARM
fan1:                  1313 RPM  (min =    0 RPM)
fan2:                   955 RPM  (min =    0 RPM)
fan3:                   918 RPM  (min =    0 RPM)
fan4:                     0 RPM  (min =    0 RPM)
fan5:                     0 RPM  (min =    0 RPM)
SYSTIN:                 +32.0°C  (high =  +0.0°C, hyst =  +0.0°C)  ALARM  sensor = thermistor
CPUTIN:                 +32.0°C  (high = +80.0°C, hyst = +75.0°C)  sensor = thermistor
AUXTIN0:                 +7.0°C    sensor = thermistor
AUXTIN1:                +41.0°C    sensor = thermistor
AUXTIN2:                +22.0°C    sensor = thermistor
AUXTIN3:                -26.0°C    sensor = thermistor
SMBUSMASTER 0:          +49.0°C  
PCH_CHIP_CPU_MAX_TEMP:   +0.0°C  
PCH_CHIP_TEMP:           +0.0°C  
PCH_CPU_TEMP:            +0.0°C  
PCH_MCH_TEMP:            +0.0°C  
PCH_DIM0_TEMP:           +0.0°C  
TSI0_TEMP:              +49.2°C  
intrusion0:            ALARM
intrusion1:            ALARM
beep_enable:           disabled

k10temp-pci-00c3
Adapter: PCI adapter
Tctl:         +49.2°C
Tdie:         +29.2°C  

iwlwifi_1-virtual-0
Adapter: Virtual device
temp1:            N/A 
flightlessmango commented 6 months ago

Looking at the code it should be using Tdie as long as we can find it, otherwise use Tctl. Clearly from your sensors output both should be present, so it should work. A possible work around could be to install zenpower