-
-
Notifications
You must be signed in to change notification settings - Fork 122
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
Add ABI support for HPE's MPT and HMPT implementations #580
Conversation
I suspect there is no chance that we can get a copy for CI/Yggdrasil? |
It'd be nice to list the new ABIs in https://juliaparallel.org/MPI.jl/dev/configuration/#Configuration-2 🙂 |
It's all closed source and if you dig through the docs, there's license servers etc. mentioned everywhere. Thus I think the answer is "no" 😞 |
Thanks for the hint! Fixed in a36dc35. |
# 3) determine the abi from the implementation + version | ||
if (impl == "MPICH" && version >= v"3.1" || | ||
impl == "IntelMPI" && version > v"2014" || | ||
impl == "MVAPICH" && version >= v"2" || | ||
impl == "CrayMPICH" && version >= v"7") | ||
impl == "CrayMPICH" && version >= v"7" || | ||
# https://www.mpich.org/abi/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The comment should go at the line below?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure. The website specifically lists which MPI libraries starting which version are compatible to MPICH. HMPT is not listed on that website, however, I know from the docs that it is supposed to be compatible. Thus I tried to exclude it by putting it below the existing comment, but I can also move it up again. What do you think?
Just a word of warning, you will likely run into JuliaPackaging/JLLWrappers.jl#40 Try adding LAMMPS.jl or LAMMPS_jll |
OK, so I am still running into issues I cannot explain: When I am trying to run the most simple MPI program with this branch,
I have not seen this error before with the registered versions of MPI.jl. To reproduce, here's what I did:
Has anyone seen something like this before with the current @JBlaschke Could you try to reproduce this on PM? |
@sloede That buffer size 140444502511616 looks like a pointer (0x00007fbbc8aed000). Maybe your Julia declarations get confused between the value of and pointer to a constant? |
That's a good point. I could create an MWE by running MPI_SHEPHERD=true \
mpirun -n 1 julia -e '
required = Cint(2);
provided = Ref{Cint}();
ccall((:MPI_Init_thread, :libmpi), Cint, (Ptr{Cint},Ptr{Cvoid}, Cint, Ref{Cint}), C_NULL, C_NULL, required, provided)' Note that the correct
Unfortunately, I do not see or understand where my mistake could be. Even less do I understand why this fails now and in the new @vchuravy Was there anything changed in how the MPI library is loaded/prepared between the last registered version and the current |
With all errors I have encountered so far being resolved (thanks to #592), from my side this PR is ready for review. @JBlaschke have you had a chance to test this out on Perlmutter yet? |
The MPT ABI is based on
mpi.h
andmpio.h
for MPT v2.23.cc @JBlaschke