Skip to content

[UR][Graph] in-order USM Memcpy regression CTS test #18241

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

Merged
merged 1 commit into from
May 1, 2025

Conversation

EwanC
Copy link
Contributor

@EwanC EwanC commented Apr 29, 2025

Create a UR CTS test based on the E2E test Graph/RecordReplay/usm_copy_in_order.cpp to help debug #18169

Disabled for Level-Zero v1 adapter (passes on v2) as test fails with:

[ RUN      ] urCommandBufferUSMCopyInOrderTest.Success/Intel_R__oneAPI_Unified_Runtime_over_Level_Zero__Intel_R__Data_Center_GPU_Max_1100_ID0ID____________________
/home/test-user/actions-runner/sycl-ur-01/_work/llvm/llvm/unified-runtime/test/conformance/exp_command_buffer/regression/usm_copy.cpp:140: Failure
Expected equality of these values:
  result3
    Which is: 3656
  output[i]
    Which is: 170

This fails both with an without driver in-order command-lists used, indicating a separate problem in the command-buffer L0 v1 adapter code that needs investigated and debugger to re-enable this test.

@EwanC EwanC force-pushed the ewan/ur_memcpy_in-order branch from cbfe079 to 0ec5c2a Compare April 30, 2025 08:13
@EwanC EwanC marked this pull request as ready for review April 30, 2025 09:10
@EwanC EwanC requested a review from a team as a code owner April 30, 2025 09:10
@EwanC EwanC requested a review from Bensuo April 30, 2025 09:10
@EwanC
Copy link
Contributor Author

EwanC commented May 1, 2025

@intel/llvm-gatekeepers This is good to merge, thanks

@steffenlarsen steffenlarsen merged commit 07649c3 into intel:sycl May 1, 2025
16 checks passed
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.

3 participants