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

[ci] Add AMDGPU relected ci #6743

Merged
merged 15 commits into from
Dec 6, 2022
Merged

Conversation

galeselee
Copy link
Contributor

@galeselee galeselee commented Nov 26, 2022

Issue: #6434

Brief Summary

Add the logic of docker for AMDGPU ci

  1. The llvm used which only contains AMDGPU and X86 targets is from docker image
  2. Using AMDGPU in docker requires that /dev/kfd and the directory /dev/dri be mounted on. For dev user, there is no permission to access these character devices by default.
  3. TI_WITH_CUDA=OFF
  4. TI_RUN_RELEASE_TESTS=OFF
  5. Currently only run cpu-relected test

@netlify
Copy link

netlify bot commented Nov 26, 2022

Deploy Preview for docsite-preview canceled.

Name Link
🔨 Latest commit c237676
🔍 Latest deploy log https://app.netlify.com/sites/docsite-preview/deploys/638ee77725246b00097279ac

@galeselee galeselee marked this pull request as draft November 26, 2022 11:58
@galeselee galeselee marked this pull request as ready for review December 1, 2022 19:47
@galeselee galeselee marked this pull request as draft December 1, 2022 20:22
@galeselee galeselee requested a review from feisuzhu December 2, 2022 07:29
@galeselee galeselee marked this pull request as ready for review December 2, 2022 07:30
@feisuzhu
Copy link
Contributor

feisuzhu commented Dec 6, 2022

/rebase

.github/workflows/scripts/unix-build.sh Outdated Show resolved Hide resolved
.github/workflows/scripts/unix_test.sh Outdated Show resolved Hide resolved
.github/workflows/scripts/unix_test.sh Outdated Show resolved Hide resolved
.github/workflows/testing.yml Outdated Show resolved Hide resolved
.github/workflows/testing.yml Outdated Show resolved Hide resolved
@galeselee
Copy link
Contributor Author

/rebase

Copy link
Contributor

@feisuzhu feisuzhu left a comment

Choose a reason for hiding this comment

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

The rest looks right, approving

.github/workflows/testing.yml Outdated Show resolved Hide resolved
@galeselee
Copy link
Contributor Author

/rebase

@galeselee galeselee merged commit 7be40a5 into taichi-dev:master Dec 6, 2022
@galeselee galeselee deleted the amdgpu_ci branch December 31, 2022 04:09
quadpixels pushed a commit to quadpixels/taichi that referenced this pull request May 13, 2023
Issue: taichi-dev#6434

### Brief Summary
Add the logic of docker for AMDGPU ci
1. The llvm used which only contains AMDGPU and X86 targets is from
docker image
2. Using AMDGPU in docker requires that `/dev/kfd` and the directory
`/dev/dri` be mounted on. For `dev` user, there is no permission to
access these character devices by default.
3. `TI_WITH_CUDA=OFF`
4. `TI_RUN_RELEASE_TESTS=OFF`
5. Currently only run cpu-relected test
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