From: Tomeu Vizoso Date: Fri, 17 Dec 2021 13:57:27 +0000 (+0100) Subject: anv/tests: Free BO cache and device mutex X-Git-Tag: upstream/22.3.5~14226 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c9adcb605103d248dee991492939489478fbb8bd;p=platform%2Fupstream%2Fmesa.git anv/tests: Free BO cache and device mutex Was getting ASAN errors in CI when trying to add ANV to the debian-testing job: ==10993==ERROR: LeakSanitizer: detected memory leaks Direct leak of 4194304 byte(s) in 64 object(s) allocated from: #0 0x7f763c1bda3c in __interceptor_posix_memalign ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:226 #1 0x55f43d28627f in os_malloc_aligned ../src/util/os_memory_aligned.h:58 #2 0x55f43d28627f in _util_sparse_array_node_alloc ../src/util/sparse_array.c:107 #3 0x55f43d28627f in util_sparse_array_get ../src/util/sparse_array.c:143 #4 0x55f43d1fdaba in anv_device_lookup_bo ../src/intel/vulkan/anv_private.h:1335 #5 0x55f43d1fdaba in anv_device_import_bo_from_host_ptr ../src/intel/vulkan/anv_allocator.c:1843 #6 0x55f43d1ff571 in anv_block_pool_expand_range ../src/intel/vulkan/anv_allocator.c:534 #7 0x55f43d1ffcb5 in anv_block_pool_init ../src/intel/vulkan/anv_allocator.c:417 #8 0x55f43d18f082 in run_test ../src/intel/vulkan/tests/block_pool_no_free.c:123 #9 0x55f43d1862b6 in main ../src/intel/vulkan/tests/block_pool_no_free.c:152 #10 0x7f763b942d09 in __libc_start_main ../csu/libc-start.c:308 Signed-off-by: Tomeu Vizoso Reviewed-by: Lionel Landwerlin Part-of: --- diff --git a/src/intel/vulkan/tests/block_pool_grow_first.c b/src/intel/vulkan/tests/block_pool_grow_first.c index a993fda..d2d2b4f 100644 --- a/src/intel/vulkan/tests/block_pool_grow_first.c +++ b/src/intel/vulkan/tests/block_pool_grow_first.c @@ -63,4 +63,6 @@ int main(void) memset(map, 22, block_size); anv_block_pool_finish(&pool); + anv_bo_cache_finish(&device.bo_cache); + pthread_mutex_destroy(&device.mutex); } diff --git a/src/intel/vulkan/tests/block_pool_no_free.c b/src/intel/vulkan/tests/block_pool_no_free.c index e2c99e9..f7cc85e 100644 --- a/src/intel/vulkan/tests/block_pool_no_free.c +++ b/src/intel/vulkan/tests/block_pool_no_free.c @@ -143,6 +143,7 @@ static void run_test() validate_monotonic(block_ptrs); anv_block_pool_finish(&pool); + anv_bo_cache_finish(&device.bo_cache); pthread_mutex_destroy(&device.mutex); } diff --git a/src/intel/vulkan/tests/state_pool.c b/src/intel/vulkan/tests/state_pool.c index 30180cc..0b685ca 100644 --- a/src/intel/vulkan/tests/state_pool.c +++ b/src/intel/vulkan/tests/state_pool.c @@ -55,5 +55,6 @@ int main(void) anv_state_pool_finish(&state_pool); } + anv_bo_cache_finish(&device.bo_cache); pthread_mutex_destroy(&device.mutex); } diff --git a/src/intel/vulkan/tests/state_pool_free_list_only.c b/src/intel/vulkan/tests/state_pool_free_list_only.c index 8d27f65..b5e5146 100644 --- a/src/intel/vulkan/tests/state_pool_free_list_only.c +++ b/src/intel/vulkan/tests/state_pool_free_list_only.c @@ -64,5 +64,6 @@ int main(void) run_state_pool_test(&state_pool); anv_state_pool_finish(&state_pool); + anv_bo_cache_finish(&device.bo_cache); pthread_mutex_destroy(&device.mutex); } diff --git a/src/intel/vulkan/tests/state_pool_no_free.c b/src/intel/vulkan/tests/state_pool_no_free.c index b844686..0424546 100644 --- a/src/intel/vulkan/tests/state_pool_no_free.c +++ b/src/intel/vulkan/tests/state_pool_no_free.c @@ -109,6 +109,7 @@ static void run_test() } anv_state_pool_finish(&state_pool); + anv_bo_cache_finish(&device.bo_cache); pthread_mutex_destroy(&device.mutex); } diff --git a/src/intel/vulkan/tests/state_pool_padding.c b/src/intel/vulkan/tests/state_pool_padding.c index d6703cf..54e8bc9 100644 --- a/src/intel/vulkan/tests/state_pool_padding.c +++ b/src/intel/vulkan/tests/state_pool_padding.c @@ -75,4 +75,6 @@ int main(void) ASSERT(state.offset == pool_size); anv_state_pool_finish(&state_pool); + anv_bo_cache_finish(&device.bo_cache); + pthread_mutex_destroy(&device.mutex); }