-
Notifications
You must be signed in to change notification settings - Fork 346
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hitting non-empty VmaDeviceMemoryBlock assert #276
Comments
Hello and thanks for reporting that! |
Hi Marek,
|
Hello and sorry for long time in responses. |
No problem! I had a play around with disabling some of the additional vma debugging features we use and the issue went away. |
Hi and thanks for checking that! |
No problem.
We're not explicitly enabling any other features but we will set VmaAllocatorCreateInfo::flags based on the implementations supported features. If it helps I'm using Linux + clang-14.
If we can figure out the root cause that'd be preferable as using VMA_DEBUG_MARGIN + VMA_DEBUG_INITIALIZE_ALLOCATIONS/VMA_DEBUG_DETECT_CORRUPTION have helped us debug other issues previously. |
Hi, the problem that is appearing here is caused by improper merging of free blocks during their deletion in your case. When creating new allocation with debug margin there is block placed behind the actual allocation indicating requested margin. This seems to be causing the whole merging problem. I could dig deeper to that if you could share with me 3 more logs (same as previous JSON dump):
Maybe that would indicate what is happening next, as it would show me the layout of memory before corruption. |
We have the following pattern:
I'm attaching:
after_first_allocation_after_deletion.txt |
Hi and thanks for that logs, they really helped us! |
Thanks for the proposed fix! Unfortunately I still see the same assert :(. If I additionally enable VMA_HEAVY_ASSERT by setting it to VMA_ASSERT I then hit the following assert |
Hi, sorry to hear that. |
Sorry for the delay - I've been OoO for the last week. |
We're hitting the metadata->IsEmpty() assert on shutdown inside VmaDeviceMemoryBlock::Destroy however VmaBlockMetadata_TLSF::DebugLogAllAllocations() isn't printing a list of unfreed allocations - we've defined VMA_DEBUG_LOG.
I see that VmaBlockMetadata_TLSF::DebugLogAllAllocation is getting skipped due to the !block->IsFree() check.
I've copied the VMA specific part of our logs below. The number of allocations seems to line up perfectly with the number of allocations freed. Are we missing anything obvious?
Note that I modified the allocation/freed VMA_DEBUG_LOGs to include the allocation handle to help debug this but all seems fine.
We've (PVRCarbon: https://developer.imaginationtech.com/pvrcarbon/) been using v3.0.0 since March (Vma in general a lot longer) and we've not hit this issue before. I also tried v3.0.1 this morning and see the same behaviour.
It looks like its a particular path is being taken to cause this. Have you got any suggestions?
vmaCreateAllocator
vmaCreateAllocator
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=7680
Created new block #0 Size=33554432, allocation=140737127085760
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=768
Returned from existing block #0, allocation: 140737127085840
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=398272
Returned from existing block #0, allocation: 140737127085920
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=120348
Returned from existing block #0, allocation: 140737127086000
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=120348
Returned from existing block #0, allocation: 140737127086080
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=2368
Returned from existing block #0, allocation: 140737127086160
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=432
Returned from existing block #0, allocation: 140737127086240
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=432
Returned from existing block #0, allocation: 140737127086320
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=35552
Returned from existing block #0, allocation: 140737127086400
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=4932
Returned from existing block #0, allocation: 140737127086480
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=4932
Returned from existing block #0, allocation: 140737127086560
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=35616
Returned from existing block #0, allocation: 140737127086640
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=4932
Returned from existing block #0, allocation: 140737127086720
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=4932
Returned from existing block #0, allocation: 140737127086800
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=56800
Returned from existing block #0, allocation: 140737127086880
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=16752
Returned from existing block #0, allocation: 140737127086960
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=16752
Returned from existing block #0, allocation: 140737127087040
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=768
Returned from existing block #0, allocation: 140737127087120
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=72
Returned from existing block #0, allocation: 140737127087200
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=72
Returned from existing block #0, allocation: 140737127087280
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=5120
Returned from existing block #0, allocation: 140737127087360
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=1368
Returned from existing block #0, allocation: 140737127087440
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=1368
Returned from existing block #0, allocation: 140737127087520
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=5120
Returned from existing block #0, allocation: 140737127087600
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=1368
Returned from existing block #0, allocation: 140737127087680
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=1368
Returned from existing block #0, allocation: 140737127087760
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=40320
Returned from existing block #0, allocation: 140737127087840
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=4188
Returned from existing block #0, allocation: 140737127087920
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=4188
Returned from existing block #0, allocation: 140737127088000
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=4194304
Returned from existing block #0, allocation: 140737127088080
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=1048576
Returned from existing block #0, allocation: 140737127088160
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=262144
Returned from existing block #0, allocation: 140737127088240
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=65536
Returned from existing block #0, allocation: 140737127088320
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=16384
Returned from existing block #0, allocation: 140737127088400
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=4096
Returned from existing block #0, allocation: 140737127088480
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=1024
Returned from existing block #0, allocation: 140737127088560
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=256
Returned from existing block #0, allocation: 140737127088640
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=64
Returned from existing block #0, allocation: 140737127088720
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=16
Returned from existing block #0, allocation: 140737127088800
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=4
Returned from existing block #0, allocation: 140737127088880
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=1048576
Returned from existing block #0, allocation: 140737127088960
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=262144
Returned from existing block #0, allocation: 140737127089040
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=65536
Returned from existing block #0, allocation: 140737127089120
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=16384
Returned from existing block #0, allocation: 140737127089200
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=4096
Returned from existing block #0, allocation: 140737127089280
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=1024
Returned from existing block #0, allocation: 140737127089360
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=256
Returned from existing block #0, allocation: 140737127089440
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=64
Returned from existing block #0, allocation: 140737127089520
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=16
Returned from existing block #0, allocation: 140737127089600
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=4
Returned from existing block #0, allocation: 140737127089680
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=2097152
Returned from existing block #0, allocation: 140737127089760
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=524288
Returned from existing block #0, allocation: 140737127089840
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=131072
Returned from existing block #0, allocation: 140737127089920
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=32768
Returned from existing block #0, allocation: 140737127090000
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=8192
Returned from existing block #0, allocation: 140737127090080
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=2048
Returned from existing block #0, allocation: 140737127090160
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=512
Returned from existing block #0, allocation: 140737127090240
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=128
Returned from existing block #0, allocation: 140737127090320
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=32
Returned from existing block #0, allocation: 140737127090400
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=8
Returned from existing block #0, allocation: 140737127090480
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=4
Returned from existing block #0, allocation: 140737127090560
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=16384
Returned from existing block #0, allocation: 140737127090640
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=16384
Returned from existing block #0, allocation: 140737127090720
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=356088
Returned from existing block #0, allocation: 140737127090800
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=88800
Returned from existing block #0, allocation: 140737127090880
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=22000
Returned from existing block #0, allocation: 140737127090960
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=5400
Returned from existing block #0, allocation: 140737127091040
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=1300
Returned from existing block #0, allocation: 140737127091120
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=288
Returned from existing block #0, allocation: 140737127091200
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=72
Returned from existing block #0, allocation: 140737127091280
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=12
Returned from existing block #0, allocation: 140737127091360
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=4
Returned from existing block #0, allocation: 140737127091440
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=131072
Returned from existing block #0, allocation: 140737127091520
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=20
Returned from existing block #0, allocation: 140737127091600
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=20
Returned from existing block #0, allocation: 140737127091680
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=20
Returned from existing block #0, allocation: 140737127091760
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127090800
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127090880
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127090960
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127091040
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127091120
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127091200
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127091280
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127091360
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127091440
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127091520
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=196596
Returned from existing block #0, allocation: 140737127091520
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127085920
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127086080
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127086160
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127086320
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127086400
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127086560
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127086640
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127086800
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127086880
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127087040
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127087120
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127087280
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127087360
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127087520
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127087600
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127087760
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127087840
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127088000
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127088080
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127088160
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127088240
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127088320
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127088400
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127088480
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127088560
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127088640
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127088720
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127088800
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127088880
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127088960
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127089040
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127089120
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127089200
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127089280
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127089360
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127089440
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127089520
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127089600
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127089680
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127089760
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127089840
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127089920
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127090000
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127090080
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127090160
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127090240
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127090320
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127090400
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127090480
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127090560
vmaAllocateMemoryForBuffer
AllocateMemory: MemoryTypeIndex=0, AllocationCount=1, Size=4587520
Returned from existing block #0, allocation: 140737127090560
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127090560
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127091680
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127091760
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127085760
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127085840
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127091520
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127091600
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127090640
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127090720
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127086000
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127086240
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127086480
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127086720
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127086960
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127087200
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127087440
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127087680
vmaFreeMemory
Freed from MemoryTypeIndex=0, allocation: 140737127087920
vmaDestroyAllocator
The text was updated successfully, but these errors were encountered: