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

Enable cmd-buf local memory update CTS tests on L0 #2631

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

EwanC
Copy link
Contributor

@EwanC EwanC commented Jan 28, 2025

The CTS tests for updating local memory in command-buffer kernel commands are currently skipped on Level-Zero. This feature is now available in L0 drivers so the tests can be enabled.

There is a failure in LocalMemoryUpdateTest.UpdateParametersSmallerLocalSize with incorrect results. Mark the individual test as failed as it needs further investigation.

The CTS tests for updating local memory in command-buffer kernel
commands are currently skipped on Level-Zero. This feature is now
available in L0 drivers so the tests can be enabled.

There is a failure in `LocalMemoryUpdateTest.UpdateParametersSmallerLocalSize`
with incorrect results. Mark the individual test as failed as it needs further
investigation.
@github-actions github-actions bot added the conformance Conformance test suite issues. label Jan 28, 2025
@EwanC EwanC marked this pull request as ready for review January 28, 2025 17:11
@EwanC EwanC requested a review from a team as a code owner January 28, 2025 17:11
@EwanC EwanC requested a review from reble January 28, 2025 17:11
Copy link
Contributor

@reble reble left a comment

Choose a reason for hiding this comment

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

LGTM

@EwanC EwanC added the ready to merge Added to PR's which are ready to merge label Jan 29, 2025
@aarongreig aarongreig merged commit b048cc2 into oneapi-src:main Jan 30, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conformance Conformance test suite issues. ready to merge Added to PR's which are ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants