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

Update HIP CI build ROCm 4.2 -> 5.0 #682

Merged
merged 7 commits into from May 28, 2022

Conversation

dalg24
Copy link
Contributor

@dalg24 dalg24 commented May 20, 2022

CMake 3.18.5 -> 3.22.4
Boost 1.72.0 -> 1.79.0
Google Benmchark 1.5.0 -> 1.6.1

CMake 3.18.5 -> 3.22.4
Boost 1.72.0 -> 1.79.0
Google Benmchark 1.5.0 -> 1.6.1
@dalg24 dalg24 added the testing Anything to do with tests and CI label May 20, 2022
@aprokop
Copy link
Contributor

aprokop commented May 20, 2022

Expired CMake key.

@dalg24
Copy link
Contributor Author

dalg24 commented May 20, 2022

No that is not the issue. See kokkos/kokkos#4601

@aprokop
Copy link
Contributor

aprokop commented May 20, 2022

Starting from 1.76.0, the place we download boost from (e.g., 1.79.0) do not carry .asc signature files.

jfrog stopped providing .asc files after 1.75.0.
@dalg24
Copy link
Contributor Author

dalg24 commented May 24, 2022

Test this please

@dalg24
Copy link
Contributor Author

dalg24 commented May 24, 2022

test this please

@dalg24
Copy link
Contributor Author

dalg24 commented May 26, 2022

Retest this please

3 similar comments
@dalg24
Copy link
Contributor Author

dalg24 commented May 26, 2022

Retest this please

@dalg24
Copy link
Contributor Author

dalg24 commented May 26, 2022

Retest this please

@dalg24
Copy link
Contributor Author

dalg24 commented May 27, 2022

Retest this please

aprokop
aprokop previously approved these changes May 27, 2022
@dalg24
Copy link
Contributor Author

dalg24 commented May 27, 2022

Retest this please

@dalg24
Copy link
Contributor Author

dalg24 commented May 27, 2022

I wasn't able to reproduce the HIP failure (cmake not finding MPI)
Would someone else be able to try as well?

@aprokop aprokop dismissed their stale review May 27, 2022 15:04

The success check was not actually successful.

@aprokop
Copy link
Contributor

aprokop commented May 27, 2022

Hmm, HIP took 40 minutes to build. Wonder if it's related to #652. In the previous HIP 4.2 test build we had

--amdgpu-target=gfx908 -xhip --offload-arch=gfx900 --offload-arch=gfx906 --offload-arch=gfx908

The new 5.0 test build has

--amdgpu-target=gfx906 -x hip --offload-arch=gfx900 --offload-arch=gfx906 --offload-arch=gfx908 --offload-arch=gfx90a --offload-arch=gfx1030

@aprokop
Copy link
Contributor

aprokop commented May 28, 2022

With building just one target, the build now takes about 15 minutes. Still longer than it used to be, but it's acceptable.

@aprokop aprokop merged commit 3741744 into arborx:master May 28, 2022
@dalg24 dalg24 deleted the update_hip_ci_build branch May 29, 2022 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing Anything to do with tests and CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants