-
Notifications
You must be signed in to change notification settings - Fork 62
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
100% CPU load on >= linux-4.19 when /proc/acpi/ibm/thermal is used #60
Comments
Can someone with this problem please post the output of: |
The output is the same as is was before kernel update: |
Huh. That is very strange. I'd really like to know what's causing the input to fail there. Can you show me the hex dump with: |
|
For me, thinkfan shows 100% CPU load (occupying one CPU core only) on a similar configuration (Thinkpad x201, kernel 4.20.10-arch1-1-ARCH), with the latest version of thinkfan from AUR. |
Output without thinkfan running: And with thinkfan running: The configuration is an unaltered /usr/share/doc/thinkfan/examples/thinkfan.conf.simple copied to /etc/thinkfan.conf. |
I was faced same problem, eat 100% of one core, and rise temp up to 70C. Just downgrade to 0.9.3 version "solved" this issue. Snippet (click to open)
Yep, here is:
Thanks, I was use thinkfan many many years, hope you fix this soon )) UPD.
|
OK, so I dug out an old system that still has |
Fortunately thinkfan freezes before any fan speed is set, so a least it won't overheat your CPU ;-) |
Ah well, I was looking at a portion of the code that already dealt with that newline char that suddenly popped up in |
not fixed :( |
The Archlinux AUR package “thinkfan” uses “thinkfan-1.0.1.tar.gz”: there the error is not fixed. |
Yeah, I'm pretty sure it should be fixed in the master now. @kubriel and possibly others: If thinkfan still freezes for you, please do a debug build:
Run the debuggable binary on the terminal
And then go to another terminal to send a
Thinkfan should print a backtrace as it crashes. Please post that here. |
On my Thinkpad X220, Archlinux. AUR package “thinkfan” is buggy. And “thinkfan-git” (AUR) is working/fixed. Thanks! |
@superherointj I'm the maintainer of the thinkfan AUR package. I'm just waiting for an official version containing the fix to be released, and I will update the package accordingly. thinkfan-git is working because it pulls the latest changes from the master branch. |
OK great, then I'll do a bugfix release. |
Thinkfan stopped working after 4.18 → 4.19 kernel upgrade. It loads CPU to 100% due to infinite loop.
I'm using simple config without any significant modifications, thus thinkfan reads temperatures from
/proc/acpi/ibm/thermals
. For some reason,!f.eof()
check doesn't work, so it tries to read temperatures infinitely, reading zeros when EOF is reached. I tried to change the condition and it worked just fine:Thinkfan 1.0.1, Parabola GNU/Linux-libre, Lenovo ThinkPad X200.
The text was updated successfully, but these errors were encountered: