izzy/UnrealEngineUWP
0
0
Fork 0
mirror of https://github.com/izzy2lost/UnrealEngineUWP.git synced 2026-03-26 18:15:20 -07:00
Code Issues Packages Projects Releases Wiki Activity

Use exchange instead of CAS to gain ownership of the linked list of new trace events.

Browse Source 
#rb johan.berg
#rnx

[CL 15872372 by Martin Ridgers in ue5-main branch]
This commit is contained in:
Martin Ridgers
2021-03-31 04:24:02 -04:00
parent 4c5985ddb2
commit 5313a089ed
1 changed files with 2 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Engine/Source/Runtime/TraceLog/Private/Trace/EventNode.cpp
+2 -16
View File
@@ -40,18 +40,12 @@ const FEventNode* FEventNode::FIter::GetNext()
////////////////////////////////////////////////////////////////////////////////
FEventNode::FIter FEventNode::ReadNew()
{
FEventNode* EventList = AtomicLoadRelaxed(&GNewEventList);
FEventNode* EventList = AtomicExchangeAcquire(&GNewEventList, (FEventNode*)nullptr);
if (EventList == nullptr)
{
return {};
}
while (!AtomicCompareExchangeAcquire(&GNewEventList, (FEventNode*)nullptr, EventList))
{
PlatformYield();
EventList = AtomicLoadRelaxed(&GNewEventList);
}
if (GEventListHead == nullptr)
{
GEventListHead = EventList;
@@ -187,15 +181,7 @@ void FEventNode::OnConnect()
return;
}
for (;; PlatformYield())
{
FEventNode* Node = AtomicLoadRelaxed(&GNewEventList);
if (AtomicCompareExchangeRelaxed(&GNewEventList, GEventListHead, Node))
{
GEventListTail->Next = Node;
break;
}
}
GEventListTail->Next = AtomicExchangeAcquire(&GNewEventList, GEventListHead);
GEventListHead = GEventListTail = nullptr;
}