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

[BUG] LLL uses power-profiles-daemon but Fedora upstream has replaced it with tuned-ppd #279

Open
GloriousEggroll opened this issue Nov 13, 2024 · 3 comments

Comments

@GloriousEggroll
Copy link

https://fedoraproject.org/wiki/Changes/TunedAsTheDefaultPowerProfileManagementDaemon
This happened in 41

@johnfanv2
Copy link
Owner

@MrDuartePT Do you know any details about that?

We still expose the profile from ACPI here /sys/firmware/acpi/platform_profile? Shouldn't tuned also pick it up. The only difference is in the GUI, python, and deamon that might call the power profile manager.

@MrDuartePT
Copy link
Collaborator

MrDuartePT commented Nov 14, 2024

@MrDuartePT Do you know any details about that?

Not really I don't use fedora in my personal machine, need to give a look

We still expose the profile from ACPI here /sys/firmware/acpi/platform_profile? Shouldn't tuned also pick it up. The only difference is in the GUI, python, and deamon that might call the power profile manager.

For what i know /sys/firmware/acpi/platform_profile is the default location for platform_profile I use tlp on my machine and it can adjust the legion power profile.

The daemon was updated to not required PPD if I not wrong.

@MrDuartePT
Copy link
Collaborator

MrDuartePT commented Nov 14, 2024

Ok I was correct the daemon was update to not required ppd: ef427ef

@johnfanv2 what we could do is removed the toogle to enable ppd in the GUI or make it check if the system have PPD, TLP or tuned-ppd install.

I think all distro enable ppd by default anyway so the toogle might be not needed.

We also should update the README say it that the driver works with any PowerProfile Management daemon.

Edit: after reading seems that tuned uses platform_profile so everthing should work on our end, but let's wait until the change drop.

Thanks for the warning @GloriousEggroll

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants