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

Fix RAPL DRAM energy unit for Sapphire Rapids (SPR) #643

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

chriswasser
Copy link
Contributor

Hi @TomTheBear,

this PR directly addresses the issue reported in #642. Although the implemented changes are really minor, I tested it on our system and the reported numbers are indeed ~4 times higher (new 61 uJ / old 15,3 uJ = 3,9869281).

Greetings

Christian

@ipatix ipatix merged commit 6a00df8 into RRZE-HPC:master Nov 5, 2024
2 checks passed
@ipatix
Copy link
Contributor

ipatix commented Nov 5, 2024

Good catch. Thanks for the discovery. The energy unit has indeed changed for Sapphire Rapids.

@chriswasser
Copy link
Contributor Author

Hi Michael,

Thanks for the quick feedback. And sorry to have addressed Thomas directly - I did not know he had a new helping hand in maintaining LIKWID 😃 Glad to see the continued development 👌

Greetings from Aachen,

Christian

@hservatg
Copy link

hservatg commented Nov 5, 2024

Certainly.

According to Table 2-52 (Additional MSRs Supported by the 4th and 5th Generation Intel Xeon Scalable Processor Families) from the Intel SDM (as of October 2024 - Volume 4) one can read for Register Address: 619h, 1561 -- bits 31:0

Energy in 61 micro-joules. Requires BIOS configuration to enable DRAM RAPL mode 0 (Direct VR)

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