Skip to content

Commit

Permalink
Merge pull request intel#2519 from oneapi-src/revert-2481-fabio/fix_p…
Browse files Browse the repository at this point in the history
…otential_race_condition

Revert "Fix potential deadlock in the WaitEvent path of CmdBuffers"
  • Loading branch information
martygrant authored Jan 6, 2025
2 parents 545a01b + ceb0465 commit f0b3520
Showing 1 changed file with 6 additions and 9 deletions.
15 changes: 6 additions & 9 deletions source/adapters/level_zero/command_buffer.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1682,28 +1682,25 @@ ur_result_t enqueueWaitEventPath(ur_exp_command_buffer_handle_t CommandBuffer,
(ZeCopyCommandQueue, 1, &CommandBuffer->ZeCopyCommandList, nullptr));
}

ZE2UR_CALL(zeCommandListAppendBarrier,
(SignalCommandList->first, nullptr, 1,
&(CommandBuffer->ExecutionFinishedEvent->ZeEvent)));

// Reset the wait-event for the UR command-buffer that is signaled when its
// submission dependencies have been satisfied.
ZE2UR_CALL(zeCommandListAppendEventReset,
(SignalCommandList->first, CommandBuffer->WaitEvent->ZeEvent));

// Reset the all-reset-event for the UR command-buffer that is signaled when
// all events of the main command-list have been reset.
ZE2UR_CALL(zeCommandListAppendEventReset,
(SignalCommandList->first, CommandBuffer->AllResetEvent->ZeEvent));

if (DoProfiling) {
UR_CALL(appendProfilingQueries(CommandBuffer, SignalCommandList->first,
nullptr, nullptr));
*Event,
CommandBuffer->ExecutionFinishedEvent));
} else {
ZE2UR_CALL(zeCommandListAppendBarrier,
(SignalCommandList->first, (*Event)->ZeEvent, 1,
&(CommandBuffer->ExecutionFinishedEvent->ZeEvent)));
}

ZE2UR_CALL(zeCommandListAppendBarrier,
(SignalCommandList->first, (*Event)->ZeEvent, 0, nullptr));

UR_CALL(Queue->executeCommandList(SignalCommandList, false /*IsBlocking*/,
false /*OKToBatchCommand*/));

Expand Down

0 comments on commit f0b3520

Please sign in to comment.