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

Remaining Issues with HIP #787

Open
5 of 8 tasks
bmhan12 opened this issue Feb 8, 2022 · 7 comments
Open
5 of 8 tasks

Remaining Issues with HIP #787

bmhan12 opened this issue Feb 8, 2022 · 7 comments
Assignees
Labels
Build system Issues related to Axom's build system Hip Issues related to Hip Reviewed TPL Issues related to Axom's third party libraries

Comments

@bmhan12
Copy link
Contributor

bmhan12 commented Feb 8, 2022

Tasks from #778 that still remain:

  • The shared TPLs are currently located at /usr/WS1/axom/libs/toss_4_x86_64_ib_cray_temp_dir/. The expected build_src.py default, /usr/WS1/axom/libs/toss_4_x86_64_ib_cray/, is a private user-only directory from someone that has left. @white238 is seeing if someone at LC can delete the directory.
  • The installed examples (using-with-[cmake/blt/make]) fail and are disabled for hip. These examples are also failing for CUDA and were already disabled.
  • There is a Fortran executable (quest_inout_interface_F_ex) and unit test (quest_inout_interface_mpi_F_test) that fails to compile with amdclang in a Debug build (File: src/axom/quest/examples/CMakeLists.txt). They are disabled under these conditions.
  • blt is not updated to develop with the hip changes. Still have issue with -lhip_runtime linkage error. This branch tracks the current PR and has latest blt with blt::hip.

EDIT:

  • Fix spack configs to work with new uberenv/clingo concretizer

EDIT x2:

EDIT x3:

@bmhan12 bmhan12 added TPL Issues related to Axom's third party libraries Build system Issues related to Axom's build system Hip Issues related to Hip labels Feb 8, 2022
@bmhan12 bmhan12 self-assigned this Feb 8, 2022
@rhornung67
Copy link
Member

@bmhan12 you are using the latest BLT develop with amdclang++? what is being built when the -lhip_runtime error shows up?

@bmhan12
Copy link
Contributor Author

bmhan12 commented Feb 8, 2022

@rhornung67 With the latest blt develop and hipcc, I've seen the error appear with any CMake target that depends on umpire or RAJA (e.g. "umpire_smoke_test").

@white238
Copy link
Member

white238 commented Feb 8, 2022

@rhornung67 With the latest blt develop and hipcc, I've seen it appear with any CMake target that depends on umpire or RAJA (e.g. "umpire_smoke_test").

This is more than likely because the Umpire release we are using is not exporting their hip_runtime target properly. This has been probably fixed in the new Umpire release. After both RAJA and Umpire have ironed out their BLT HIP support, I will help you with this.

We could also fake it. This makes a noop fake target in CMake:

add_library(hip_runtime INTERFACE)

Just stick it at the base of Axom somewhere with a giant TODO

@rhornung67
Copy link
Member

rhornung67 commented Feb 8, 2022

Actually, I ran across that and the issue was some "very clever" stuff in camp. Here is the camp branch I am using to test the new BLT in RAJA: https://github.com/LLNL/camp/tree/raja-blt-update This isn't a complete fix and I am waiting on a response from Tom for something better. @bmhan12 you may want to try that camp branch and this RAJA branch that I have working now: https://github.com/LLNL/RAJA/tree/blt-update. Umpire develop has the new BLT stuff in it. Umpire develop and the camp and RAJA branches I linked here are all using the same commit of BLT develop.

@white238
Copy link
Member

white238 commented Feb 8, 2022

That makes more sense. Umpire is usually very good about exporting their targets properly.

@rhornung67
Copy link
Member

camp's cmake stuff is too fancy(!) for me....

@kennyweiss
Copy link
Member

Bump. Is this still an issue? It might be worth reinvestigating the open tasks on this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build system Issues related to Axom's build system Hip Issues related to Hip Reviewed TPL Issues related to Axom's third party libraries
Projects
None yet
Development

No branches or pull requests

4 participants