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

[L0] Fix immediate command list use in Command Queues #1802

Merged
merged 1 commit into from
Jul 5, 2024

Conversation

nrspruit
Copy link
Contributor

  • Keep track of all immediate command lists ensuring that they are not used in regular queues after they are placed back into the command list map.

@nrspruit nrspruit requested a review from a team as a code owner June 28, 2024 01:12
@nrspruit nrspruit requested review from pbalcer and igchor and removed request for a team June 28, 2024 01:12
@github-actions github-actions bot added the level-zero L0 adapter specific issues label Jun 28, 2024
nrspruit added a commit to nrspruit/llvm that referenced this pull request Jun 28, 2024
pre-commit PR for oneapi-src/unified-runtime#1802

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
@nrspruit nrspruit added the ready to merge Added to PR's which are ready to merge label Jun 28, 2024
@kbenzie
Copy link
Contributor

kbenzie commented Jul 4, 2024

There's a failure in the Regression/commandlist/gpu.cpp test. I'll update the branches to see if it goes away.

Edit: That's gone away 👍

- Keep track of all immediate command lists ensuring that they are not
  used in regular queues after they are placed back into the command
list map.

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
@kbenzie kbenzie force-pushed the fix_immediate_cmdlist_reuse branch from d847710 to 665d4a6 Compare July 4, 2024 09:35
@aarongreig aarongreig merged commit 731376d into oneapi-src:main Jul 5, 2024
51 checks passed
martygrant pushed a commit to intel/llvm that referenced this pull request Jul 5, 2024
pre-commit PR for
oneapi-src/unified-runtime#1802

---------

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
Co-authored-by: Kenneth Benzie (Benie) <k.benzie@codeplay.com>
Co-authored-by: Aaron Greig <aaron.greig@codeplay.com>
pbalcer pushed a commit to pbalcer/unified-runtime that referenced this pull request Jul 5, 2024
…t_reuse

[L0] Fix immediate command list use in Command Queues
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
level-zero L0 adapter specific 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.

5 participants