systemd: Disable Intel RAPL PL4 on resume/boot#26
Conversation
3a99d9d to
e07763b
Compare
e07763b to
0d8d9c7
Compare
|
I'm kind of wondering whether this is something that should be handled by PPD (https://gitlab.freedesktop.org/upower/power-profiles-daemon) instead of just setting it to "unlimited" unconditionally. Or whether this is the right package for it... but I guess we can add it here. |
I still have thermald not PPD. A later PR can disable our service once it’s confirmed those daemons update the value upon resume. Though, the on-resume problem never affected my computer.
I set it to unlimited to avoid complexities of someone else possibly having different values and requiring more calculation. Seems fine, and during normal operation, I didn't see a thermal difference from the correct value I determined.
The only other somewhat-reasonable target package from pkg.surfacelinux.com is the kernel. It’s weird shipping a service in the kernel package, and placing the logic inside the kernel code itself may encounter difficulty and opposition. I believe surface-control here is the correct package because of |
Yeah, I was more thinking about a new package for settings and config files, but for now I think we can keep it here. |
Fixes: linux-surface/linux-surface#706