Skip to content

[UR][L0] Disable command-buffer immediate append path #17097

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
Feb 21, 2025

Conversation

fabiomestre
Copy link
Contributor

@fabiomestre fabiomestre commented Feb 20, 2025

There has been some synchronization issues found in the implementation of zeCommandListImmediateAppendCommandListsExp. Since the immediate append path relies on this entrypoint, we need to disable it until the issues are fixed.

@fabiomestre fabiomestre force-pushed the fabio/disable_immediate_append branch from d5c8713 to 1c65f26 Compare February 21, 2025 11:29
@fabiomestre fabiomestre changed the title debug PR [UR][L0] Disable immediate append path for command-buffers Feb 21, 2025
@fabiomestre fabiomestre changed the title [UR][L0] Disable immediate append path for command-buffers [UR][L0] Disable command-buffer immediate append path Feb 21, 2025
@fabiomestre fabiomestre marked this pull request as ready for review February 21, 2025 12:01
@fabiomestre fabiomestre requested a review from a team as a code owner February 21, 2025 12:01
@fabiomestre fabiomestre force-pushed the fabio/disable_immediate_append branch from 1c65f26 to 34927a6 Compare February 21, 2025 12:04
@fabiomestre fabiomestre force-pushed the fabio/disable_immediate_append branch from 34927a6 to 4766a86 Compare February 21, 2025 13:35
@fabiomestre
Copy link
Contributor Author

@intel/llvm-gatekeepers This should be ready to merge

@martygrant martygrant merged commit 1515afa into intel:sycl Feb 21, 2025
28 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