Skip to content

Conversation

nrspruit
Copy link
Contributor

  • fix multi device event creation to be enqueued as a barrier waiting on the original event keeping the multi device event valid.
  • Avoid wait on events on a multi device event given driver in order due to p2p errors for driver in order lists. Barrier from multi device event creation will guarantee in order execution.

… in order wait list

- fix multi device event creation to be enqueued as a barrier waiting on
  the original event keeping the multi device event valid.
- Avoid wait on events on a multi device event given driver in order due
  to p2p errors for driver in order lists. Barrier from multi device
event creation will guarantee in order execution.

Signed-off-by: Spruit, Neil R <neil.r.spruit@intel.com>
@nrspruit nrspruit requested a review from a team as a code owner September 25, 2024 20:30
@github-actions github-actions bot added the level-zero L0 adapter specific issues label Sep 25, 2024
nrspruit added a commit to nrspruit/llvm that referenced this pull request Sep 25, 2024
…from in order wait list

-pre-commit PR for oneapi-src/unified-runtime#2134

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
@nrspruit nrspruit added the v0.10.x Include in the v0.10.x release label Sep 25, 2024
Copy link
Contributor

@pbalcer pbalcer left a comment

Choose a reason for hiding this comment

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

I like this, makes sense!

@pbalcer pbalcer merged commit ce0325d into oneapi-src:main Sep 26, 2024
73 of 74 checks passed
steffenlarsen pushed a commit to intel/llvm that referenced this pull request Sep 27, 2024
…from in order wait list (#15516)

-pre-commit PR for
oneapi-src/unified-runtime#2134

---------

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
Co-authored-by: Piotr Balcer <piotr.balcer@intel.com>
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 v0.10.x Include in the v0.10.x release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants