Skip to content

Add test ioq operations ordering - #22740

Open
mianowski wants to merge 3 commits into
intel:syclfrom
mianowski:feat/URT-753-test-ioq-operations-ordering
Open

Add test ioq operations ordering#22740
mianowski wants to merge 3 commits into
intel:syclfrom
mianowski:feat/URT-753-test-ioq-operations-ordering

Conversation

@mianowski
mianowski requested review from a team as code owners July 23, 2026 13:21
@mianowski
mianowski requested a review from pbalcer July 23, 2026 13:21
@mianowski
mianowski marked this pull request as draft July 23, 2026 13:29
@mianowski
mianowski force-pushed the feat/URT-753-test-ioq-operations-ordering branch from 9d78623 to 515e734 Compare July 23, 2026 14:30
@mianowski
mianowski marked this pull request as ready for review July 23, 2026 15:52
@bratpiorka
bratpiorka requested a review from Copilot July 28, 2026 08:07

Copilot AI 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.

Pull request overview

Adds a new Unified Runtime conformance test intended to validate correct ordering of USM operations on an in-order queue when UR_QUEUE_FLAG_DISCARD_EVENTS is enabled (ported from the referenced SYCL E2E test).

Changes:

  • Introduce a new conformance test exercising interleaved USM memcpy/fill and kernel launches under discard-events queue semantics.
  • Add the new test source to the enqueue conformance test CMake target.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
unified-runtime/test/conformance/enqueue/urEnqueueUSMOperationsOrderingIOQ.cpp New conformance test covering USM operation ordering on discard-events queues across multiple USM allocation types.
unified-runtime/test/conformance/enqueue/CMakeLists.txt Registers the new test file in the enqueue conformance test build.

Comment on lines +33 to +35
kernel_name = uur::KernelsEnvironment::instance->GetEntryPointNames(
"cpy_and_mult_usm")[0];
ASSERT_FALSE(kernel_name.empty());
Comment on lines +132 to +134
EXPECT_SUCCESS(alloc_one(&values1));
EXPECT_SUCCESS(alloc_one(&values2));
EXPECT_SUCCESS(alloc_one(&values3));
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.

2 participants