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

uncore: Allow to configure frequency limits using percent #611

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sgruszka
Copy link
Contributor

@sgruszka sgruszka commented Mar 11, 2024

Using direct frequency values in only useful in specialistic profiles
where we know target platform. Add 'max_freq_pct' and 'min_freq_pct'
parameters that allow to specify limit of uncore frequency as percent of hardware
maximum/minimum frequency. Such parameters can be used in generic profiles
like 'balanced' in portable way, for example:

[uncore]
max_freq_pct=90

@sgruszka sgruszka changed the title uncore: Allow to configure maximum frequency using percent uncore: Allow to configure frequency limits using percent Mar 14, 2024
@sgruszka sgruszka marked this pull request as ready for review March 14, 2024 13:00
@sgruszka sgruszka marked this pull request as draft March 14, 2024 14:53
Using direct frequency values in only useful in specialistic profiles
where we know target platform. Add 'max_freq_pct' and 'min_freq_pct'
parameters that allow to specify limit of uncore frequency as percent of
hardware maximum/minimum frequency. Such parameters can be used in generic
profiles like 'balanced' in portable way, for example:

[uncore]
max_freq_pct=90

Signed-off-by: Stanislaw Gruszka <stanislaw.gruszka@linux.intel.com>
@sgruszka sgruszka marked this pull request as ready for review March 15, 2024 14:32
Copy link
Contributor

@yarda yarda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add ASCII doc inline comments describing plugin parameters like other plugins have. Especially mention what happened in case of conflicting setting (i.e. khZ vs percentage).

Otherwise the code LGTM.

@yarda
Copy link
Contributor

yarda commented May 23, 2024

What about using unit suffix to distinguish between percentage and kHz? Then we could avoid the possible conflict between kHz and percentage setting. E.g. we used something similar in readahead option of disk plugin, where the user can set the value in sectors or kiBs.

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.

None yet

2 participants