| Summary: | Power Profile incorrectly shown when hardware does not support it | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | TraceyC <kdedev> |
| Component: | Power and Battery widget | Assignee: | Plasma Bugs List <plasma-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | kde, natalie_clarius, nate |
| Priority: | NOR | ||
| Version First Reported In: | 6.3.5 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Solus | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/powerdevil/-/commit/eb1f73c3e6146d5bb3d6237754f7c28d9a43e57f | Version Fixed In: | 6.3.6 |
| Sentry Crash Report: | |||
|
Description
TraceyC
2025-05-27 21:09:51 UTC
Confirming, since I can reproduce A bit complicated. We basically treat this situation as a distro bug; the PPD package *should* be pre-installed, and is even called out as such in https://community.kde.org/Distributions/Packaging_Recommendations#3rd-party_packages. The message in the applet is designed to subtly alert the user that their distro forgot something, and that they can fix it themselves. We have similar messages throughout Plasma, most notably in Discover, because Plasma is complex and it's easy for distros to forget things. The only time this message does not make sense IMO is on distros that are tied to hardware that it's known to not support PPD. An example is SteamOS. In that case, it probbly makes sense to patch out the message. Other than that case, I fail to see how this isn't a distro bug. > For systems where the CPU is not supported by power-profiles-daemon, telling the user to install > it is counter-productive. power-profiles-daemon conflicts with other governers like schedutil, > which supports some CPUs the former doesn't. We don't formally support that though. PPD is the thing we support. What do you think we should do here? If the applet is telling the user to install power-profiles-daemon, but it *is* installed, but won't work for the CPU, how would a distro fix that? This seems like a bug in the applet. Oh wait, PPD is installed but the widget thinks it isn't? Hmm, yes that would be a bug. A possibly relevant merge request was started @ https://invent.kde.org/plasma/powerdevil/-/merge_requests/554 Git commit 8aca3d2bf22117924ff10dc7f897ca3966830786 by Nate Graham. Committed on 28/05/2025 at 02:53. Pushed by ngraham into branch 'master'. applets/batterymonitor: don't ask to install PPD when already installed A logic error in this code resulted in the "try installing PPD" message appearing for people with PPD installed but using using hardware that doesn't support PPD yet. FIXED-IN: 6.3.6 M +1 -1 applets/batterymonitor/package/contents/ui/PowerProfileItem.qml https://invent.kde.org/plasma/powerdevil/-/commit/8aca3d2bf22117924ff10dc7f897ca3966830786 Git commit 60f7e31049fdc8b6b36771ac3d177c9b93abf471 by Nate Graham. Committed on 28/05/2025 at 13:42. Pushed by ngraham into branch 'Plasma/6.4'. applets/batterymonitor: don't ask to install PPD when already installed A logic error in this code resulted in the "try installing PPD" message appearing for people with PPD installed but using using hardware that doesn't support PPD yet. FIXED-IN: 6.3.6 (cherry picked from commit 8aca3d2bf22117924ff10dc7f897ca3966830786) Co-authored-by: Nate Graham <nate@kde.org> M +1 -1 applets/batterymonitor/package/contents/ui/PowerProfileItem.qml https://invent.kde.org/plasma/powerdevil/-/commit/60f7e31049fdc8b6b36771ac3d177c9b93abf471 Git commit eb1f73c3e6146d5bb3d6237754f7c28d9a43e57f by Nate Graham. Committed on 28/05/2025 at 13:43. Pushed by ngraham into branch 'Plasma/6.3'. applets/batterymonitor: don't ask to install PPD when already installed A logic error in this code resulted in the "try installing PPD" message appearing for people with PPD installed but using using hardware that doesn't support PPD yet. FIXED-IN: 6.3.6 (cherry picked from commit 8aca3d2bf22117924ff10dc7f897ca3966830786) Co-authored-by: Nate Graham <nate@kde.org> M +1 -1 applets/batterymonitor/package/contents/ui/PowerProfileItem.qml https://invent.kde.org/plasma/powerdevil/-/commit/eb1f73c3e6146d5bb3d6237754f7c28d9a43e57f |