-
Notifications
You must be signed in to change notification settings - Fork 108
Add cpu and gpu fan_speed_rpm #475
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
You don't mentioned your device, but I will assume it's new enough, like Intel core 11 gen+ or AMD 6 gen+. We have
You should not compare different types of fans.
Cooler Boost just bypasses fan curve completely. You should not account it as rule.
Do you have broken fan blade, or any other audible marker? Maybe you can get laser tachometer readings as proof? |
|
MSI Bravo 15 A4DDR I have trouble using sensors. I don't have CPU fan, and I only have GPU fan1 at 65535 RPM. On the other hand, isn't this the same case as for realtime_temperature also reported with sensors?
Yes, it illustrates realtime_fan_speed is not the real/current fan speed.
True, I was confused because I got no frequency spike at the exact rpm. But by looking again, I also get spikes above rpm. |
|
Okay, your model is "WMI1", so
How many fans you exactly have? If you have single fan, than it probably follows only CPU fan curve |
|
The laptop has 2 fans, one "CPU" and one GPU. GPU fan is often off. sensorsiGPU, CPU? GPU off GPU on |
|
I don't know why I kept this as a draft for so long. It's ready, expect it only contain address for my laptop.
If I understand correctly from your comments, msi-wmi-platform is for WMI2 and not implemented (yet?) for WMI1? Is it why I agree having access from standard API would be great. If you have an idea how I can help, let me know. |
I see this a reason to keep this as draft |
|
hwmon:
Same result with linux-lts. The feature is functional and ready. If you prefer not to support it, I'd appreciate your thoughts, so I can focus on troubleshooting hwmon in a discussion instead. |
Add the fan_speed_rpm entry to use the address 0xcc..0xcd (or 0xc8..0xc9) for CPU fan and 0xca..0xcb for GPU fan, like MControlCenter does.
Discussion with @mutchiko dmitry-s93/MControlCenter#267
realtime_fan_speed
The current
realtime_fan_speed(0x71 and 0x89) is the fan % target speed for fan mode.Ideally, it should include "target", "driving" or "control" in its name.
For example, when cooler boost is enabled, fan mode continues to change the target speed according to the temperature.
For now, I named the new value fan_speed_rpm to distinguish both.
About the magic number 480000
floor(480000/x)(e.g. target speed at 50% ⇒ read 160 ⇒ 3000 rpm)But now I wonder if MSI Center report is good. RPM above 4000 seems unreal.
By analyzing the audio with my CPU fan, I get
This is more likely 320000 for my CPU fan, with 100% speed targeting 4000 rpm.
https://download-2.msi.com/archive/mnu_exe/mb/MSICENTER.pdf slide 97, graphs show different RPM at 100% (~2100, 2702, and ~4500). It's also not impossible that they have different numerators, it could be a fan_speed_rpm_magic_number.