Skip to content

Commit 32eb611

Browse files
committed
fabtests/efa: Remove the skip for dmabuf_only
Today, fabtests will skip test if the `check_cuda_hmem` binary thinks DMABUF is the only way to register cuda buffer, while test is not passing `--do-dmabuf-reg-for-hmem`, it will silently skip the test. This is wrong because efa provider supports registering with dmabuf when it's supported, even if application is not using FI_MR_DMABUF API Signed-off-by: Shi Jin <sjina@amazon.com>
1 parent 102872c commit 32eb611

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

fabtests/pytest/efa/conftest.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -147,11 +147,6 @@ def cuda_memory_type_validation(cmdline_args):
147147
pytest.fail("CUDA memory support never initialized")
148148

149149
do_dmabuf = cmdline_args.do_dmabuf_reg_for_hmem
150-
if (do_dmabuf is None and
151-
cuda_support == CudaMemorySupport.DMA_BUF_ONLY):
152-
error = "User specified CUDA without dmabuf but hardware only supports DMA_BUF_ONLY"
153-
print(f"CUDA validation failed: {error}")
154-
pytest.skip(error)
155150

156151
print(f"Correctly defined dma buf mode {do_dmabuf} and return {cuda_support}!")
157152

0 commit comments

Comments
 (0)