commit c5b223fe4b379478485ca95af56dbf76416acaa3
author Adam Sawicki <Adam.Sawicki@amd.com> Mon Aug 27 11:53:02 2018 +0200
committer Adam Sawicki <Adam.Sawicki@amd.com> Mon Aug 27 11:53:02 2018 +0200
tree 88c6c89f3d3a8106147a5b68a3f27ee50d2d52a6
parent a5d5ffafe2a38ceb6a8f8c3f36336d7cd75ccd25

VmaAllocator_T::FreeMemory: Fixed synchronization bug for cases when an allocation becomes lost at the same time as it is being freed.

src/vk_mem_alloc.h

diff --git a/src/vk_mem_alloc.h b/src/vk_mem_alloc.h
index aef6b1b..2975335 100644
--- a/src/vk_mem_alloc.h
+++ b/src/vk_mem_alloc.h
@@ -11542,8 +11542,7 @@
 {

     VMA_ASSERT(allocation);

 

-    if(allocation->CanBecomeLost() == false ||

-        allocation->GetLastUseFrameIndex() != VMA_FRAME_INDEX_LOST)

+    if(TouchAllocation(allocation))

     {

         if(VMA_DEBUG_INITIALIZE_ALLOCATIONS)

         {

@@ -11762,7 +11761,7 @@
         Warning: This is a carefully designed algorithm.

         Do not modify unless you really know what you're doing :)

         */

-        uint32_t localCurrFrameIndex = m_CurrentFrameIndex.load();

+        const uint32_t localCurrFrameIndex = m_CurrentFrameIndex.load();

         uint32_t localLastUseFrameIndex = hAllocation->GetLastUseFrameIndex();

         for(;;)

         {