intel/compute-runtime
1
0
Fork 0
mirror of https://github.com/intel/compute-runtime.git synced 2026-01-05 09:09:04 +08:00
Code Issues Packages Projects Releases Wiki Activity

Revert "fix: fallback ioq appendSignalEvent to appendBarrier"

Browse Source 
This reverts commit 8427aa9c7b.

Signed-off-by: Compute-Runtime-Validation <compute-runtime-validation@intel.com>
This commit is contained in:
Compute-Runtime-Validation
2025-09-18 01:53:41 +02:00
committed by Compute-Runtime-Automation
parent 9c41fe996d
commit 40eee63ede
3 changed files with 6 additions and 43 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
level_zero/core/source/cmdlist/cmdlist_hw.inl
View File

@@ -3056,16 +3056,11 @@ inline ze_result_t CommandListCoreFamily<gfxCoreFamily>::addEventsToCmdList(uint
template <GFXCORE_FAMILY gfxCoreFamily>
ze_result_t CommandListCoreFamily<gfxCoreFamily>::appendSignalEvent(ze_event_handle_t hEvent, bool relaxedOrderingDispatch) {
auto event = Event::fromHandle(hEvent);
if (event->isCounterBased()) {
return appendBarrier(hEvent, 0, nullptr, relaxedOrderingDispatch);
}
if (this->isInOrderExecutionEnabled()) {
handleInOrderImplicitDependencies(relaxedOrderingDispatch, false);
}
auto event = Event::fromHandle(hEvent);
event->resetKernelCountAndPacketUsedCount();
if (!handleCounterBasedEventOperations(event, false)) {
@@ -4245,10 +4240,8 @@ ze_result_t CommandListCoreFamily<gfxCoreFamily>::appendBarrier(ze_event_handle_
appendSynchronizedDispatchInitializationSection();
Event *signalEvent = nullptr;
bool hostVisibleEvent = false;
if (hSignalEvent) {
signalEvent = Event::fromHandle(hSignalEvent);
hostVisibleEvent = signalEvent->isSignalScope(ZE_EVENT_SCOPE_FLAG_HOST);
}
if (!handleCounterBasedEventOperations(signalEvent, false)) {
@@ -4277,10 +4270,7 @@ ze_result_t CommandListCoreFamily<gfxCoreFamily>::appendBarrier(ze_event_handle_
}
addToMappedEventList(signalEvent);
bool skipPipeControl = this->isInOrderExecutionEnabled() && !getDcFlushRequired(hostVisibleEvent);
appendSignalEventPostWalker(signalEvent, nullptr, nullptr, skipPipeControl, false, isCopyOnly(false));
appendSignalEventPostWalker(signalEvent, nullptr, nullptr, this->isInOrderExecutionEnabled(), false, isCopyOnly(false));
if (isInOrderExecutionEnabled()) {
appendSignalInOrderDependencyCounter(signalEvent, false, false, false);

31
level_zero/core/test/unit_tests/sources/cmdlist/test_in_order_cmdlist_1.cpp
View File

@@ -1154,7 +1154,7 @@ HWTEST_F(InOrderCmdListTests, givenDependencyFromDifferentRootDeviceWhenAppendCa
auto immCmdList0 = createCmdList(device0);
auto immCmdList1 = createCmdList(device1);
immCmdList0->appendLaunchKernel(kernel->toHandle(), groupCount, eventH, 0, nullptr, launchParams);
immCmdList0->appendSignalEvent(eventH, false);
auto &cmdContainer1 = immCmdList1->getCmdContainer();
auto cmdStream = cmdContainer1.getCommandStream();
@@ -1195,7 +1195,6 @@ HWTEST_F(InOrderCmdListTests, givenTimestmapEventWhenProgrammingBarrierThenDontA
auto eventPool = createEvents<FamilyType>(1, true);
auto eventHandle = events[0]->toHandle();
events[0]->signalScope = 0;
auto immCmdList = createImmCmdList<FamilyType::gfxCoreFamily>();
@@ -2245,7 +2244,7 @@ HWTEST2_F(InOrderCmdListTests, givenRelaxedOrderingEnabledWhenSignalEventCalledT
auto immCmdList1 = createImmCmdList<FamilyType::gfxCoreFamily>();
auto immCmdList2 = createImmCmdList<FamilyType::gfxCoreFamily>();
auto eventPool = createEvents<FamilyType>(2, true);
auto eventPool = createEvents<FamilyType>(2, false);
events[1]->makeCounterBasedInitiallyDisabled(eventPool->getAllocation());
auto nonCbEvent = events[1]->toHandle();
@@ -2966,32 +2965,6 @@ HWTEST2_F(InOrderCmdListTests, givenRelaxedOrderingWhenProgrammingTimestampEvent
}
}
HWTEST_F(InOrderCmdListTests, givenCbEventWhenAppendSignalEventCalledThenFallbackToAppendBarrier) {
class MyMockCmdList : public WhiteBox<L0::CommandListCoreFamilyImmediate<FamilyType::gfxCoreFamily>> {
public:
using BaseClass = WhiteBox<L0::CommandListCoreFamilyImmediate<FamilyType::gfxCoreFamily>>;
using BaseClass::BaseClass;
ze_result_t appendBarrier(ze_event_handle_t hSignalEvent, uint32_t numWaitEvents, ze_event_handle_t *phWaitEvents, bool relaxedOrderingDispatch) override {
appendBarrierCalled++;
return BaseClass::appendBarrier(hSignalEvent, numWaitEvents, phWaitEvents, relaxedOrderingDispatch);
}
uint32_t appendBarrierCalled = 0;
};
auto immCmdList = createImmCmdListImpl<FamilyType::gfxCoreFamily, MyMockCmdList>(false);
auto eventPool = createEvents<FamilyType>(2, true);
events[0]->makeCounterBasedInitiallyDisabled(eventPool->getAllocation());
immCmdList->appendSignalEvent(events[0]->toHandle(), false);
EXPECT_EQ(0u, immCmdList->appendBarrierCalled);
immCmdList->appendSignalEvent(events[1]->toHandle(), false);
EXPECT_EQ(1u, immCmdList->appendBarrierCalled);
}
HWTEST2_F(InOrderCmdListTests, givenRelaxedOrderingWhenProgrammingTimestampEventCbThenClearOnHstAndChainWithSyncAllocSignalingAsTwoSeparateSubmissions, IsAtLeastXeHpcCore) {
class MyMockCmdList : public WhiteBox<L0::CommandListCoreFamilyImmediate<FamilyType::gfxCoreFamily>> {
public:

4
level_zero/core/test/unit_tests/sources/cmdlist/test_in_order_cmdlist_2.cpp
View File

@@ -2441,11 +2441,11 @@ struct StandaloneInOrderTimestampAllocationTests : public InOrderCmdListFixture
HWTEST_F(StandaloneInOrderTimestampAllocationTests, givenSignalScopeEventWhenSignalEventIsCalledThenProgramPipeControl) {
using PIPE_CONTROL = typename FamilyType::PIPE_CONTROL;
auto eventPool = createEvents<FamilyType>(2, true);
auto eventPool = createEvents<FamilyType>(2, false);
auto cmdList = createImmCmdList<FamilyType::gfxCoreFamily>();
events[0]->signalScope = ZE_EVENT_SCOPE_FLAG_HOST;
events[0]->signalScope = true;
events[1]->signalScope = false;
auto cmdStream = cmdList->getCmdContainer().getCommandStream();