Skip to content

Conversation

@bbernhar
Copy link
Contributor

@bbernhar bbernhar commented Jun 24, 2022

Fixes a crash when SlabMemoryAllocator nears OOM and attempts to use the smallest slab size possible, which could be too small to use for the allocation. This fix change it to return kInvalidSize which fails allocation.

@bbernhar bbernhar added the bug Something isn't working label Jun 24, 2022
@github-actions github-actions bot added Frontend Changes to common or GPU API agnostic code. Test Changes in tests. labels Jun 24, 2022
@bbernhar bbernhar changed the title Make SlabMemoryAllocator::TryAllocateMemory fail when there is no available memory and no more slabs can be reused. Fix SlabMemoryAllocator from failing when near OOM. Jun 24, 2022
Fixes a crash when SlabMemoryAllocator nears OOM and attempts to use the smallest slab size possible, which could be too small to use for the allocation. This fix makes SlabMemoryAllocator::FindNextFreeSlabOfSize return kInvalidSize.
@bbernhar bbernhar changed the title Fix SlabMemoryAllocator from failing when near OOM. Fix SlabMemoryAllocator crash when near OOM. Jun 24, 2022
@bbernhar bbernhar merged commit 714d273 into main Jun 24, 2022
@bbernhar bbernhar deleted the fix_slab_size branch June 24, 2022 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working Frontend Changes to common or GPU API agnostic code. Test Changes in tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants