cudf icon indicating copy to clipboard operation
cudf copied to clipboard

Refine `CudaTest.testCudaException` in case throwing wrong type of CudaError under aarch64

Open sperlingxx opened this issue 1 year ago • 3 comments

Fix #15705

  1. Replacing Cuda.memset(Long.MAX_VALUE, (byte) 0, 1024) with Cuda.freePinned(-1L), the previous one throws fatal CUDAError cudaErrorIllegalAddress instead of nonFatal CUDAError cudaErrorInvalidValue under aarch64, while the later one throwing the correct kind of error.

  2. Enable the test case when Sanitizer is ON

sperlingxx avatar May 09 '24 09:05 sperlingxx

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

copy-pr-bot[bot] avatar May 09 '24 09:05 copy-pr-bot[bot]

Verified on my local arm64 rocky8 container, unit tests can PASS!

NvTimLiu avatar May 09 '24 12:05 NvTimLiu

/ok to test

NvTimLiu avatar May 09 '24 12:05 NvTimLiu

/ok to test

jlowe avatar May 10 '24 13:05 jlowe

/merge

jlowe avatar May 10 '24 13:05 jlowe