Skip to content

MI300X build script - #4

Merged
sean-baccas merged 3 commits into
mainfrom
HenriqueBR/mi300x-build
Aug 18, 2026
Merged

MI300X build script#4
sean-baccas merged 3 commits into
mainfrom
HenriqueBR/mi300x-build

Conversation

@Heinrich-BR

Copy link
Copy Markdown
Contributor

Added a build script that works for the MI300X partition on the CSD3 system. It builds with HIP-aware OpenMPI.

@Heinrich-BR
Heinrich-BR force-pushed the HenriqueBR/mi300x-build branch from 3462dd4 to 7d7b213 Compare April 17, 2026 09:09
@Heinrich-BR

Copy link
Copy Markdown
Contributor Author

@alexanderianblair the test that's failing has nothing to do with the PR, I believe.

@sean-baccas

Copy link
Copy Markdown
Contributor

All good from my side, apart from some issues with headers for CEED's JIT backend. Henrique has come up with a fix, which I'm testing atm. But network problems on csd3 are making this very slow

@sean-baccas

Copy link
Copy Markdown
Contributor

@Heinrich-BR @alexanderianblair Last time I had a go at this, I had an issue with the stdint header not being included in some part of CEED's just-in-time compilation stuff (I can fetch the exact location and error message when CSD3 is back online). The result is that it fails at runtime when running with CEED backend - it tries to do some JIT compilation, and fails with a compilation error. Its problem is something to do with int32_t not being correctly namespaced - the hip compiler was expecting something like hip::int32_t

My solution was to just copy #include <stdint> into the source file where the error occurs. This fixes everything, and does not require recompiling libCEED since it's only an issue at JIT compilation time. This works, but it isn't really something that can be added to this script.

What do you think is a good solution to this? The only thing I can think of is it to:

  • Fork petsc
  • Set this fork of petsc to pull a fork of libCEED (already made one here)
  • Fork moose
  • Set this fork of moose to pull the fork of petsc

But this is very ugly. Any ideas?

Comment thread scripts/build_csd3_mi300x.sh Outdated
Comment thread scripts/build_csd3_mi300x.sh Outdated

@sean-baccas sean-baccas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Heinrich-BR @alexanderianblair I've finally managed to get the build working on csd3 with these two minor changes. Please apply these changes and I'll be happy!

@Heinrich-BR Heinrich-BR mentioned this pull request Aug 18, 2026
Heinrich-BR and others added 3 commits August 18, 2026 12:51
Linting

More linting
Co-authored-by: Sean Baccas <sean.baccas@ukaea.uk>
@Heinrich-BR
Heinrich-BR force-pushed the HenriqueBR/mi300x-build branch from 7ef481e to 0bfc0a3 Compare August 18, 2026 11:52
@sean-baccas
sean-baccas self-requested a review August 18, 2026 12:26
@sean-baccas
sean-baccas merged commit fa9a4f7 into main Aug 18, 2026
10 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.

2 participants