Skip to content

Commit a0db607

Browse files
Jaime ArteagaCompute-Runtime-Automation
authored andcommitted
Remove unnecessary flush in event profiling
Remove flush before appending appendWriteKernelTimestamp, since the associated PC is only to ensure kernel execution has completed in the device, so flush to the host is not required. Signed-off-by: Jaime Arteaga <[email protected]>
1 parent be2a87f commit a0db607

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

level_zero/core/source/cmdlist/cmdlist_hw.inl

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1523,7 +1523,6 @@ void CommandListCoreFamily<gfxCoreFamily>::appendEventForProfiling(ze_event_hand
15231523
} else {
15241524

15251525
NEO::PipeControlArgs args = {};
1526-
args.dcFlushEnable = true;
15271526

15281527
NEO::MemorySynchronizationCommands<GfxFamily>::addPipeControl(*commandContainer.getCommandStream(), args);
15291528
appendWriteKernelTimestamp(hEvent, beforeWalker, true);

level_zero/core/test/unit_tests/sources/cmdlist/test_cmdlist_append_launch_kernel.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright (C) 2020 Intel Corporation
2+
* Copyright (C) 2020-2021 Intel Corporation
33
*
44
* SPDX-License-Identifier: MIT
55
*
@@ -337,7 +337,7 @@ HWTEST2_F(CommandListAppendLaunchKernel, givenTimestampEventsWhenAppendingKernel
337337
{
338338
auto cmd = genCmdCast<PIPE_CONTROL *>(*itor);
339339
EXPECT_TRUE(cmd->getCommandStreamerStallEnable());
340-
EXPECT_TRUE(cmd->getDcFlushEnable());
340+
EXPECT_FALSE(cmd->getDcFlushEnable());
341341
}
342342
itor++;
343343

0 commit comments

Comments
 (0)