Multiple fixes and man page update#112
Open
maciejwieczorretman wants to merge 11 commits intointel:mainfrom
Open
Multiple fixes and man page update#112maciejwieczorretman wants to merge 11 commits intointel:mainfrom
maciejwieczorretman wants to merge 11 commits intointel:mainfrom
Conversation
cgroup isolation mode doesn't work in LPMD because when writing 'isolated' to cpuset.cpus.partition file the expectation is that cpuset.cpus will be a subset of cpuset.cpus.exclusive of that cgroup. The simple fix is to write the same content to cpuset.cpus and cpuset.cpus.exclusive when switching into the isolated mode. However a problem can appear if there are any other cgroups on the system that have exclusive cpus claimed. For one LPMD will just shut down if it can't set affinity into any cpu it wants during core type discovery phase. Aside from that using this simple version of the fix will later result in creating an invalid cgroup partition. Signed-off-by: Maciej Wieczor-Retman <maciej.wieczor-retman@intel.com>
…val writes When WLT proxy is enabled it's supposed to set it's own polling timeout. The problem is in the state machine an early return happens - that disables polling timeout - when the non-proxy polling is disabled. Given that it has nothing to do with the proxy polling it should not interfere in the proxy setting it's own timeouts. Change order of the early returns so that it happens after the WLT proxy check - thereby not overwriting the interval assigned by the proxy code. Signed-off-by: Maciej Wieczor-Retman <maciej.wieczor-retman@intel.com>
If one profile, let's say the balanced one, is set to enable AUTO LPMD, and another one like the performance one, has LPMD OFF, then a deadlock might happen when using the wlt proxy system. Basically when going into the OFF state, polling gets disabled and there is no specific code flow to get it enabled again after switching back to an AUTO state. This only affects the wlt proxy as other polling and non-polling systems seem to be able to get themselves back up again. Reinitialize the polling interval when switching power profiles. Signed-off-by: Maciej Wieczor-Retman <maciej.wieczor-retman@intel.com>
ITMT became broken when the enabling file was moved to the debugfs. The data in the file also changed from printing out 0 or 1 to printing out N or Y characters. However 0 and 1 are still accepted for writing. Change the file that's being accesses to the debugfs and add helpers for the Y/N format. Signed-off-by: Maciej Wieczor-Retman <maciej.wieczor-retman@intel.com>
The cpu name in the config file is described as having 16 cores while the specific tags in the config file try to enable higher core indices than 16. Correct the indices. Signed-off-by: Maciej Wieczor-Retman <maciej.wieczor-retman@intel.com>
Get the readme source to a readable format for editing. Signed-off-by: Maciej Wieczor-Retman <maciej.wieczor-retman@intel.com>
Make clear how to start the program to avoid confusion around the default behavior. Signed-off-by: Maciej Wieczor-Retman <maciej.wieczor-retman@intel.com>
Update the main man page with new informations about LPMD. Mention what are the expectation towards the user, clear up intel_lpmd_control section since it has it's own man page and add how to enable the system service persistently in the examples section. Signed-off-by: Maciej Wieczor-Retman <maciej.wieczor-retman@intel.com>
Cleanup the long lines and link in the config man page at the end. Signed-off-by: Maciej Wieczor-Retman <maciej.wieczor-retman@intel.com>
Cleanup some topics that expanded since they were written. Make general cleanups and small corrections to typos or style. Signed-off-by: Maciej Wieczor-Retman <maciej.wieczor-retman@intel.com>
Some options are not currently doing anything. Move them out of the proper part of the man page to avoid confusing users. Signed-off-by: Maciej Wieczor-Retman <maciej.wieczor-retman@intel.com>
This was referenced Apr 8, 2026
ptr1337
reviewed
Apr 9, 2026
| Run intel_lpmd as a service with logs directed to system journal | ||
|
|
||
| .TP | ||
| .B sudo systemctl enable intel_lpmd.service --now |
There was a problem hiding this comment.
Should this not be:
sudo systemctl enable --now intel_lpmd.service
Author
There was a problem hiding this comment.
I think both work, for some reason I've always put '--now' at the end when configuring anything on arch linux. If there will be any changes to make I'll change it so it's a bit more tidy. Otherwise I'll fix it when I'll be updating man pages the next time
|
Looked a bit over and looking good! Ill deploy it to the CachyOS fork (with the profile changes) and will do some testing. Thanks for working on this. |
Author
Thanks for checking it out and testing! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bugs fixed:
Man pages update: