intel / intel/S0ixSelftestTool

Do not hardcode package cstates

Open
#29 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Shell
Stars
213
Forks
29
PR merge metrics
No merged PRs in 30d

Description

Different platforms may support different package cstate.
Instead of hardcoded package cstates to check in turbostat, we should
a. blindly check all possible package cstates, and leverage turbostat to give meaningful output (turbostat has per-platform profile which should expose available package cstates only)
b. check CPUID.5, it exposes sub-cstates for supported cstates only, and we can leverage that info.

Contributor guide

Open the contributing guide

Research direction

Start by locating the turbostat package-cstate checks in the shell code and review how turbostat exposes per-platform profiles. Compare that path with the CPUID.5 information described in the issue. Done means supported package cstates are detected per platform without hardcoded checks and turbostat output remains meaningful.

Written by the indexing model from the issue text.

Assessment

Tech stack
linux, shell
Domain
operating-systems, tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.