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

Added support Emerald Rapids #667

Merged
merged 4 commits into from
Mar 18, 2025
Merged

Added support Emerald Rapids #667

merged 4 commits into from
Mar 18, 2025

Conversation

vxirau
Copy link

@vxirau vxirau commented Mar 17, 2025

Hello!

I followed the instructions in https://github.com/RRZE-HPC/likwid/wiki/AddX86Support to add Emerald Rapids support.

Events are the same as for Sapphire according to perfmon intel events.

Let me know if there is something else that should be reviewed.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@TomTheBear
Copy link
Member

Hi, looks very good. Thanks for adding it.

One important change is missing though. You should add it to src/power.c (around here). Otherwise the ENERGY group will not work.

Some minor additions:

  • For sysFeatures, add the arch to the list here
  • For sysFeatures rapl component, EmeraldRapids probably uses the same energy units (for conversion) as SapphireRapids, so you can add EmeraldRapids here and here
  • For support for the fixed TOPDOWN_SLOTS event, you should add it to the condition

Any comments on AddX86Support? Something unclear? I'm happy to update the page.

@vxirau
Copy link
Author

vxirau commented Mar 17, 2025

Hi @TomTheBear !

Thank you for your response. I just made the changes you mentioned.

The AddX86Support guide was very clear and helpful! The only thing I didn't get from the guide was the sysFeatures being something mandatory to change! But of course it was easy to add.

I've also updated the README to reflect the change. Should I update the changelog too? As mentioned in the wiki?

What about the wiki itself? Something like https://github.com/RRZE-HPC/likwid/wiki/SapphireRapids ?

Thanks!

@TomTheBear
Copy link
Member

SysFeatures is a quite recent development. It is still experimental and has to be explicitly enabled at build time. I added it to the page but I have to first write better documentation for extending SysFeatures. At the moment, it describes the changes not well.

Please add it to the changelog, I forgot to mention that.

There is also the architecture docs in doc/archs and the architecture page in the wiki. You can do that or leave it to me. It's just copying the pages and changing some identifiers.

@vxirau
Copy link
Author

vxirau commented Mar 18, 2025

Not sure how you want the wiki done! I believe the rest is OK :)

@TomTheBear
Copy link
Member

It's close to perfection 🥇 . Thanks :)

I'll add the wiki page

@TomTheBear TomTheBear merged commit 9fddf85 into RRZE-HPC:master Mar 18, 2025
2 checks passed
@TomTheBear
Copy link
Member

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