From 10f97718c353e101c64fa60fcde91e1550e39957 Mon Sep 17 00:00:00 2001 From: Jason Ekstrand Date: Fri, 2 Oct 2015 16:24:42 -0700 Subject: anv/allocator: Add a sanity assertion in state stream finish. We assert that the block offset we got while walking the list of blocks is actually a multiple of the block size. If something goes wrong and the GPU decides to stomp on the surface state buffer we can end up getting corruptions in our list of blocks. This assertion makes such corruptions a crash with a meaningful message rather than an infinite loop. --- src/vulkan/anv_allocator.c | 1 + 1 file changed, 1 insertion(+) (limited to 'src/vulkan/anv_allocator.c') diff --git a/src/vulkan/anv_allocator.c b/src/vulkan/anv_allocator.c index 05126305b51..db04a2c236c 100644 --- a/src/vulkan/anv_allocator.c +++ b/src/vulkan/anv_allocator.c @@ -697,6 +697,7 @@ anv_state_stream_finish(struct anv_state_stream *stream) block = stream->current_block; while (block != NULL_BLOCK) { + assert(block % stream->block_pool->block_size == 0); sb = stream->block_pool->map + block; next_block = VG_NOACCESS_READ(&sb->next); VG(VALGRIND_MEMPOOL_FREE(stream, VG_NOACCESS_READ(&sb->_vg_ptr))); -- cgit v1.2.3