Skip to content
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

Thermald tanks performance on ASUS UX3402Z. Need to use --adaptive #46064

Open
lcubeddu opened this issue Sep 15, 2023 · 1 comment
Open

Thermald tanks performance on ASUS UX3402Z. Need to use --adaptive #46064

lcubeddu opened this issue Sep 15, 2023 · 1 comment
Labels
bug Something isn't working needs-testing Testing a PR or reproducing an issue needed

Comments

@lcubeddu
Copy link

Is this a new report?

Yes

System Info

Void 6.5.2_1 x86_64 GenuineIntel notuptodate FFF

Package(s) Affected

thermald-2.5.4_1

Does a report exist for this bug with the project's home (upstream) and/or another distro?

No response

Expected behaviour

thermald is not supposed to affect performance this much.

Actual behaviour

Hello, I've encountered an issue with thermald, I don't know if we could really call this a bug but thermald needs some ACPI info about max temperatures on newer laptops/ultrabooks.

The ACPI info was not read, so on my laptop thermald would throttle heavily (cf. https://www.reddit.com/r/voidlinux/comments/16gtkl5/whole_system_slower_with_dbus_service/?utm_source=share&utm_medium=web2x&context=3)

Upstream people have added the "--adaptive" option to thermald. I simply added this option in the runit run file.

Up to you to decide if this should be distributed on void-packages.

The thermald help does say "adaptive mode: use adaptive performance tables if available", so it shouldn't affect users who miss it.

Should we add this option ?

Steps to reproduce

Hardware specific bug
enable dbus service
enable thermald service
lower performances.

enable dbus service
enable thermald service with --adaptive
no performance issue.

@lcubeddu lcubeddu added bug Something isn't working needs-testing Testing a PR or reproducing an issue needed labels Sep 15, 2023
@classabbyamp
Copy link
Member

that service should probably have [ -r ./conf ] && . ./conf and add $OPTS to the last line so you can pass custom options set in thr conf file

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs-testing Testing a PR or reproducing an issue needed
Projects
None yet
Development

No branches or pull requests

2 participants