Skip to content

Commit 0524532

Browse files
committed
[CTS] Fix virtual mem enqueue test
`urKernelSetArgPointer` doesn't take the argument pointer directly, it takes a pointer to the argument pointer. This fixes a crash in the CTS on CUDA.
1 parent da3ee42 commit 0524532

File tree

2 files changed

+1
-2
lines changed

2 files changed

+1
-2
lines changed

test/conformance/enqueue/enqueue_adapter_cuda.match

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,4 +55,3 @@ urEnqueueKernelLaunchKernelSubGroupTest.Success/NVIDIA_CUDA_BACKEND___{{.*}}_
5555
{{OPT}}urEnqueueUSMMemcpy2DNegativeTest.InvalidEventWaitList/NVIDIA_CUDA_BACKEND___{{.*}}___pitch__1__width__1__height__1
5656
{{OPT}}urEnqueueUSMPrefetchWithParamTest.Success/NVIDIA_CUDA_BACKEND___{{.*}}___UR_USM_MIGRATION_FLAG_DEFAULT
5757
{{OPT}}urEnqueueUSMPrefetchWithParamTest.CheckWaitEvent/NVIDIA_CUDA_BACKEND___{{.*}}___UR_USM_MIGRATION_FLAG_DEFAULT
58-
{{OPT}}{{Segmentation fault|Aborted}}

test/conformance/enqueue/urEnqueueKernelLaunch.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -364,7 +364,7 @@ TEST_P(urEnqueueKernelLaunchWithVirtualMemory, Success) {
364364
size_t global_size = alloc_size / sizeof(uint32_t);
365365
uint32_t fill_val = 42;
366366

367-
ASSERT_SUCCESS(urKernelSetArgPointer(kernel, 0, nullptr, virtual_ptr));
367+
ASSERT_SUCCESS(urKernelSetArgPointer(kernel, 0, nullptr, &virtual_ptr));
368368
ASSERT_SUCCESS(
369369
urKernelSetArgValue(kernel, 1, sizeof(fill_val), nullptr, &fill_val));
370370

0 commit comments

Comments
 (0)