Skip to content

[SYCL] Enable discard_events mode for the Level Zero #6533

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.

Sign up for GitHub

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 7 commits into from
Aug 22, 2022
Merged

[SYCL] Enable discard_events mode for the Level Zero #6533

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
99549f4
[SYCL] Enable discard_events mode for the Level Zero
againull Jul 28, 2022
8e09c76
Fix accidental mistake
againull Aug 5, 2022
432ccef
Fix formatting
againull Aug 5, 2022
c30be94
Update PI version
againull Aug 10, 2022
a0d9e50
Address review comments
againull Aug 18, 2022
28e0a66
Merge remote-tracking branch 'origin/sycl' into discard
againull Aug 19, 2022
6fce540
Fix typos
againull Aug 19, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix accidental mistake
  • Loading branch information
@againull
againull committed Aug 5, 2022
commit 8e09c76ca2b367f15ae80a8e225f35cc3826e880
9 changes: 5 additions & 4 deletions sycl/plugins/level_zero/pi_level_zero.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -670,7 +670,8 @@ ze_result_t ZeCall::doCall(ze_result_t ZeResult, const char *ZeName,
// \param Event a pointer to hold the newly created pi_event
// \param CommandType various command type determined by the caller
// \param CommandList is the command list where the event is added
// \param ForceHostVisible tells if the event must be created in
// \param IsInternal tells if the event is internal, i.e. visible in the L0
// plugin only. \param ForceHostVisible tells if the event must be created in
// the host-visible pool
inline static pi_result createEventAndAssociateQueue(
pi_queue Queue, pi_event *Event, pi_command_type CommandType,
Expand Down Expand Up @@ -5465,9 +5466,9 @@ _pi_event::getOrCreateHostVisibleEvent(ze_event_handle_t &ZeHostVisibleEvent) {
return Res;

// Create a "proxy" host-visible event.
auto Res = createEventAndAssociateQueue(Queue, &HostVisibleEvent,
PI_COMMAND_TYPE_USER, CommandList);
// HostVisibleEvent->CleanedUp = true;
auto Res = createEventAndAssociateQueue(
Queue, &HostVisibleEvent, PI_COMMAND_TYPE_USER, CommandList,
/* IsInternal */ false, /* ForceHostVisible */ true);
if (Res != PI_SUCCESS)
return Res;

Expand Down