Skip to content

Commit b0a40e2

Browse files
authored
[UR][CTS] Skip failing command-buffer OpenCL CTS tests (#19138)
buffer read/write CTS tests aren't supported for the OpenCL adapter, because there is a gap in cl_khr_command_buffer that blocks implementing the required UR entry-points KhronosGroup/OpenCL-Docs#1281 is the cl_khr_command_buffer issue progressing this.
1 parent 0ab45d9 commit b0a40e2

File tree

8 files changed

+28
-1
lines changed

8 files changed

+28
-1
lines changed

unified-runtime/test/conformance/exp_command_buffer/copy.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -116,6 +116,10 @@ UUR_DEVICE_TEST_SUITE_WITH_PARAM(
116116
printMemcpyTestString<urCommandBufferMemcpyCommandsTest>);
117117

118118
TEST_P(urCommandBufferMemcpyCommandsTest, Buffer) {
119+
// Buffer read & write not supported on OpenCL
120+
// See https://github.com/KhronosGroup/OpenCL-Docs/issues/1281
121+
UUR_KNOWN_FAILURE_ON(uur::OpenCL{});
122+
119123
std::vector<uint8_t> input(size);
120124
std::iota(input.begin(), input.end(), 1);
121125

unified-runtime/test/conformance/exp_command_buffer/enqueue.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -194,6 +194,9 @@ TEST_P(urEnqueueCommandBufferExpTest, SerializeInOrOutOfOrderQueue) {
194194
// Tests releasing command-buffer while it is still executing relying
195195
// on synchronization during urCommandBufferReleaseExp call.
196196
TEST_P(urEnqueueCommandBufferExpTest, EnqueueAndRelease) {
197+
// https://github.com/intel/llvm/issues/19139
198+
UUR_KNOWN_FAILURE_ON(uur::OpenCL{});
199+
197200
ASSERT_SUCCESS(urEnqueueCommandBufferExp(
198201
in_or_out_of_order_queue, cmd_buf_handle, 0, nullptr, nullptr));
199202

unified-runtime/test/conformance/exp_command_buffer/read.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,10 @@ struct testParametersRead {
1515
struct urCommandBufferReadCommandsTest
1616
: uur::command_buffer::urCommandBufferExpTestWithParam<testParametersRead> {
1717
void SetUp() override {
18+
// Buffer read not supported on OpenCL
19+
// see https://github.com/KhronosGroup/OpenCL-Docs/issues/1281
20+
UUR_KNOWN_FAILURE_ON(uur::OpenCL{});
21+
1822
UUR_RETURN_ON_FATAL_FAILURE(
1923
uur::command_buffer::urCommandBufferExpTestWithParam<
2024
testParametersRead>::SetUp());

unified-runtime/test/conformance/exp_command_buffer/rect_read.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,10 @@ struct urCommandBufferAppendMemBufferReadRectTestWithParam
7575
: public uur::command_buffer::urCommandBufferExpTestWithParam<
7676
uur::test_parameters_t> {
7777
void SetUp() override {
78+
// Buffer read not supported on OpenCL
79+
// see https://github.com/KhronosGroup/OpenCL-Docs/issues/1281
80+
UUR_KNOWN_FAILURE_ON(uur::OpenCL{});
81+
7882
UUR_RETURN_ON_FATAL_FAILURE(
7983
uur::command_buffer::urCommandBufferExpTestWithParam<
8084
uur::test_parameters_t>::SetUp());

unified-runtime/test/conformance/exp_command_buffer/rect_write.cpp

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,10 @@ struct urCommandBufferAppendMemBufferWriteRectTestWithParam
7575
uur::test_parameters_t> {
7676

7777
void SetUp() override {
78+
// Buffer write not supported on OpenCL
79+
// see https://github.com/KhronosGroup/OpenCL-Docs/issues/1281
80+
UUR_KNOWN_FAILURE_ON(uur::OpenCL{});
81+
7882
UUR_RETURN_ON_FATAL_FAILURE(
7983
uur::command_buffer::urCommandBufferExpTestWithParam<
8084
uur::test_parameters_t>::SetUp());
@@ -151,4 +155,4 @@ TEST_P(urCommandBufferAppendMemBufferWriteRectTestWithParam, Success) {
151155

152156
// Verify the results.
153157
EXPECT_EQ(expected, output);
154-
}
158+
}

unified-runtime/test/conformance/exp_command_buffer/release.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@ using urCommandBufferReleaseExpTest =
1212
UUR_INSTANTIATE_DEVICE_TEST_SUITE(urCommandBufferReleaseExpTest);
1313

1414
TEST_P(urCommandBufferReleaseExpTest, Success) {
15+
// https://github.com/intel/llvm/issues/19139
16+
UUR_KNOWN_FAILURE_ON(uur::OpenCL{});
1517
EXPECT_SUCCESS(urCommandBufferRetainExp(cmd_buf_handle));
1618

1719
uint32_t prev_ref_count = 0;

unified-runtime/test/conformance/exp_command_buffer/retain.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@ using urCommandBufferRetainExpTest =
1212
UUR_INSTANTIATE_DEVICE_TEST_SUITE(urCommandBufferRetainExpTest);
1313

1414
TEST_P(urCommandBufferRetainExpTest, Success) {
15+
// https://github.com/intel/llvm/issues/19139
16+
UUR_KNOWN_FAILURE_ON(uur::OpenCL{});
1517
uint32_t prev_ref_count = 0;
1618
EXPECT_SUCCESS(uur::GetObjectReferenceCount(cmd_buf_handle, prev_ref_count));
1719

unified-runtime/test/conformance/exp_command_buffer/write.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,10 @@ struct urCommandBufferWriteCommandsTest
1616
: uur::command_buffer::urCommandBufferExpTestWithParam<
1717
testParametersWrite> {
1818
void SetUp() override {
19+
// Buffer write not supported on OpenCL
20+
// see https://github.com/KhronosGroup/OpenCL-Docs/issues/1281
21+
UUR_KNOWN_FAILURE_ON(uur::OpenCL{});
22+
1923
UUR_RETURN_ON_FATAL_FAILURE(
2024
uur::command_buffer::urCommandBufferExpTestWithParam<
2125
testParametersWrite>::SetUp());

0 commit comments

Comments
 (0)