AuroraSupport/AuroraRuntime
2
1
Fork 0
Code Issues 27 Pull Requests Releases Wiki Activity

[*] WoA acbd6266 cont (?). I think I'm still getting messed up by over-optimizations as opposed to there being a genuine bug in the primitives. Made the accesses to the comparison volatile during deep sleeps, whereas before we were just casting to regular integer pointers or performing a memcmp.

Browse Source 
00:29:11:820	15>  1 functions were new in current compilation
00:29:11:820	15>  51 functions had inline decision re-evaluated but remain unchanged
(it did *something*)
This commit is contained in:
Reece Wilson 2024-02-08 01:55:16 +00:00
parent 899e505b1e
commit 12fd03a6dd
1 changed files with 13 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
Source/Threading/AuWakeOnAddress.cpp
View File

@ -80,6 +80,15 @@ namespace Aurora::Threading
this->Release(); this->Release();
} }
static void FlushWaitBufferPAddressCache()
{
#if defined(MemoryBarrier)
MemoryBarrier();
#else
asm volatile ("" : : : "memory");
#endif
}
bool WaitEntry::SleepOn(WaitState &state) bool WaitEntry::SleepOn(WaitState &state)
{ {
#if !defined(WOA_SEMAPHORE_MODE) #if !defined(WOA_SEMAPHORE_MODE)
@ -98,6 +107,8 @@ namespace Aurora::Threading
while (uNow < uEndTime) while (uNow < uEndTime)
{ {
FlushWaitBufferPAddressCache();
if (!WaitBuffer::Compare(this->pAddress, this->uSize, state)) if (!WaitBuffer::Compare(this->pAddress, this->uSize, state))
{ {
return true; return true;
@ -154,6 +165,8 @@ namespace Aurora::Threading
this->variable.WaitForSignalNsEx(&this->mutex, 0, false); this->variable.WaitForSignalNsEx(&this->mutex, 0, false);
#endif #endif
} }
FlushWaitBufferPAddressCache();
} }
return true; return true;