Skip to content

Conversation

@danielzgtg
Copy link
Contributor

@danielzgtg danielzgtg force-pushed the feat/raplService branch 3 times, most recently from 3a99d9d to e07763b Compare January 15, 2026 03:25
@danielzgtg danielzgtg changed the title systemd: Disable Intel RAPL P4 on resume/boot systemd: Disable Intel RAPL PL4 on resume/boot Jan 15, 2026
@qzed
Copy link
Member

qzed commented Jan 18, 2026

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.

@danielzgtg
Copy link
Contributor Author

PPD

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.

unlimited

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.

whether this is the right package

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 surface profile. A Linux ACPI platform profile seemingly controls the Surface Pro 7 fan only, while on Windows, the corresponding power plan controls CPU throttling identical/similar to the register cleared here.

@qzed
Copy link
Member

qzed commented Jan 18, 2026

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.

Yeah, I was more thinking about a new package for settings and config files, but for now I think we can keep it here.

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

Successfully merging this pull request may close these issues.

Surface Pro 7 has CPU throttled to 1500 MHz after resume from suspend

2 participants