Skip to content

Comments

arch: Change numeric serialization for CPU profiles#80

Merged
olivereanderson merged 1 commit intocyberus-technology:gardenlinuxfrom
olivereanderson:cpu-profiles-change-numeric-serialization
Feb 10, 2026
Merged

arch: Change numeric serialization for CPU profiles#80
olivereanderson merged 1 commit intocyberus-technology:gardenlinuxfrom
olivereanderson:cpu-profiles-change-numeric-serialization

Conversation

@olivereanderson
Copy link

This PR makes some changes to the way we serialize CPUID leaf and sub-leaves in the CPU profiles. When debugging CPU profiles I noticed that it is easier to deal with shorter hex strings for the leaves and that it would also be nice to have the sub-leaf ranges specified in terms of hex as well.

We do however want to keep serializing the adjustments as 10 byte hex strings which we do not change here (only the code).

We don't bother regenerating profiles in this PR, because there is no functional change, and we have to regenerate them in #70 anyway.

@olivereanderson olivereanderson force-pushed the cpu-profiles-change-numeric-serialization branch 3 times, most recently from 876c3c1 to a6cb0eb Compare February 10, 2026 07:58
@olivereanderson olivereanderson self-assigned this Feb 10, 2026
From working with serialized CPU profile data we noticed the following:

1. It is easier to read shorter hex strings when looking up CPUID
leaves in the serialized CPU profile data.
2. It is preferable to also have sub-leaf ranges hex encoded.
3. We still want to keep serializing the adjustments to 10 character
hex strings.

Some of the convenience functions introduced here will also be utilized
in the upcoming MSR adjustments PR in order to serialize register
addresses.

Signed-off-by: Oliver Anderson <oliver.anderson@cyberus-technology.de>
On-behalf-of: SAP oliver.anderson@sap.com
@olivereanderson olivereanderson force-pushed the cpu-profiles-change-numeric-serialization branch from a6cb0eb to 9192e4a Compare February 10, 2026 09:57
@olivereanderson olivereanderson merged commit 2cff8a7 into cyberus-technology:gardenlinux Feb 10, 2026
11 checks passed
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.

3 participants