Skip to content

CUDA: Skip multi-GPU copy test with peer access disabled - #7266

Merged
sklam merged 1 commit into
numba:masterfrom
gmarkall:issue-7253
Aug 11, 2021
Merged

sklam merged 1 commit into
numba:masterfrom
gmarkall:issue-7253

Conversation

@gmarkall

@gmarkall gmarkall commented Aug 3, 2021

Copy link
Copy Markdown
Member

When peer access is disabled (e.g. with one GPU in WDDM mode and one in TCC mode), a device-to-device copy between the two fails, causing TestMultiGPUContext.test_with_context to fail. We fix this issue by moving the test of device-to-device copy into its own test method, which is skipped when peer access is disabled.

Fixes #7253.

When peer access is disabled (e.g. with one GPU in WDDM mode and one in
TCC mode), a device-to-device copy between the two fails, causing
`TestMultiGPUContext.test_with_context` to fail. We fix this issue by
moving the test of device-to-device copy into its own test method, which
is skipped when peer access is disabled.

Fixes numba#7253.
@gmarkall gmarkall added 2 - In Progress CUDA CUDA related issue/PR Effort - short Short size effort needed labels Aug 3, 2021
@gmarkall gmarkall added this to the Numba 0.55 RC milestone Aug 3, 2021

@stuartarchibald stuartarchibald 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.

Thanks for the patch, looks good.

@stuartarchibald stuartarchibald added 4 - Waiting on CI Review etc done, waiting for CI to finish Pending BuildFarm For PRs that have been reviewed but pending a push through our buildfarm and removed 3 - Ready for Review labels Aug 3, 2021
@stuartarchibald

Copy link
Copy Markdown
Contributor

Buildfarm ID: numba_smoketest_cuda_yaml_87.

@stuartarchibald

Copy link
Copy Markdown
Contributor

Buildfarm ID: numba_smoketest_cuda_yaml_87.

Passed. Test system has 2x GPUs according to nvidia-smi.

@stuartarchibald stuartarchibald added 5 - Ready to merge Review and testing done, is ready to merge BuildFarm Passed For PRs that have been through the buildfarm and passed and removed Pending BuildFarm For PRs that have been reviewed but pending a push through our buildfarm 4 - Waiting on CI Review etc done, waiting for CI to finish labels Aug 10, 2021
@sklam
sklam merged commit 21f44da into numba:master Aug 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5 - Ready to merge Review and testing done, is ready to merge BuildFarm Passed For PRs that have been through the buildfarm and passed CUDA CUDA related issue/PR Effort - short Short size effort needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CUDA: Multi-GPU tests failing on Windows

3 participants