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.

Sign up for GitHub

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

Jump to bottom

As a GEOPM Researcher I would like to be able to set additional HWP controls (activity window, EPP, pkg request) #2456

Open
lhlawson opened this issue Jul 15, 2022 · 0 comments
Open

As a GEOPM Researcher I would like to be able to set additional HWP controls (activity window, EPP, pkg request) #2456

lhlawson opened this issue Jul 15, 2022 · 0 comments
Labels
2.1 feature Description of a new feature as a story

Comments

@lhlawson
Copy link
Contributor

lhlawson commented Jul 15, 2022 •
edited

Is your feature request related to a problem? Please describe.
I'm always frustrated when I need to set the activity window or EPP value of HWP and can't access it via GEOPM.

Describe the solution you'd like
Addition of support for all HWP registers exposed on a platform. This requires additional CPUID checks for each register and in some cases fields of HWP registers. From the SDM Volume 3 the fields, registers, and related CPUID checks are:
image

Describe alternatives you've considered
Not supporting these fields & registers
@lhlawson lhlawson added feature Description of a new feature as a story 2.1 labels Jul 15, 2022
bgeltz added a commit to bgeltz/geopm that referenced this issue Jul 18, 2022
@bgeltz
Purge unused HWP MSRs
fd2066d 
- The purged MSRs require that additional CPUID bits are checked
  in leaf 6 for support.
- The fields that were marked as not writable also have CPUID
  bits that need to be checked to see if they are indeed writable.
- Related to geopm#2456.

Signed-off-by: Brad Geltz <brad.geltz@intel.com>
@bgeltz bgeltz mentioned this issue Jul 18, 2022
Merged
bgeltz added a commit that referenced this issue Jul 18, 2022
@bgeltz
Purge unused HWP MSRs
1ff43e2 
- The purged MSRs require that additional CPUID bits are checked
  in leaf 6 for support.
- The fields that were marked as not writable also have CPUID
  bits that need to be checked to see if they are indeed writable.
- Related to #2456.

Signed-off-by: Brad Geltz <brad.geltz@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
2.1 feature Description of a new feature as a story
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@lhlawson