Skip to content
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

Command-buffer queue compatibility test update #2230

Merged
merged 2 commits into from
Feb 11, 2025

Conversation

EwanC
Copy link
Contributor

@EwanC EwanC commented Jan 16, 2025

Update cl_khr_command_buffer tests to reflect changes from KhronosGroup/OpenCL-Docs#1292

  • Moves negative test for CL_DEVICE_COMMAND_BUFFER_SUPPORTED_QUEUE_PROPERTIES_KHR from command-buffer creation to enqueue.
  • Moves negative test for CL_DEVICE_COMMAND_BUFFER_REQUIRED_QUEUE_PROPERTIES_KHR from command-buffer creation to enqueue.
  • Introduces a negative test for CL_INVALID_DEVICE on command-buffer enqueue for new error condition in spec. Although it requires a context to be contain more than 1 device, which I'm not sure if possible in current test framework.
  • Introduces a new test that created a command-buffer using a queue without the profiling property set, then enqueues the command-buffer to a queue with the profiling property set.
  • Introduces a new test that creates a command-buffer with an in-order queue, enqueued on an out-of-order queue.
  • Introduces a new test that creates a command-buffer with an out-of-order queue, enqueued on an in-order queue.

Copy link
Contributor

@bashbaug bashbaug left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One additional note - I believe the negative_enqueue_queue_not_compatible test is invalid now, and can be removed.

@EwanC EwanC force-pushed the command_buffer_queue_refactor branch 2 times, most recently from d293fcc to 9c663aa Compare February 6, 2025 11:10
Update cl_khr_command_buffer tests to reflect changes from
KhronosGroup/OpenCL-Docs#1292

Required headers change
KhronosGroup/OpenCL-Headers#271 to enable
tests with version 0.9.7
@EwanC EwanC force-pushed the command_buffer_queue_refactor branch from 9c663aa to 5999c5a Compare February 6, 2025 11:26
bashbaug
bashbaug previously approved these changes Feb 6, 2025
Copy link
Contributor

@bashbaug bashbaug left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The new tests LGTM.

I still think the negative_enqueue_queue_not_compatible test is invalid now, but would you prefer to remove it in a separate PR?

Test is no longer valid, and has been superseded by
newly added tests
@EwanC
Copy link
Contributor Author

EwanC commented Feb 7, 2025

The new tests LGTM.

I still think the negative_enqueue_queue_not_compatible test is invalid now, but would you prefer to remove it in a separate PR?

Apologies! I overlooked that bit of feedback. I agree, pushed a new commit to remove that test.

@bashbaug
Copy link
Contributor

Merging as discussed in the February 11th teleconference.

@bashbaug bashbaug merged commit 044ec98 into KhronosGroup:main Feb 11, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants