From c69b13578399915f71f4ccc10cae188c31381c6a Mon Sep 17 00:00:00 2001 From: Derek Morton Date: Fri, 26 Jun 2015 11:36:00 +0100 Subject: [PATCH] igt/gem_fenced_exec_thrash: Fix memory leak between tests gem_fenced_exec_thrash was not freeing any resources between subtests. On 1Gb android systems this resulted in the test failing with an OOM error. Added cleanup code to free BOs at the end of each subtest. Signed-off-by: Derek Morton --- tests/gem_fenced_exec_thrash.c | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/tests/gem_fenced_exec_thrash.c b/tests/gem_fenced_exec_thrash.c index 75f251cc..c0e820f8 100644 --- a/tests/gem_fenced_exec_thrash.c +++ b/tests/gem_fenced_exec_thrash.c @@ -215,6 +215,18 @@ static void run_test(int fd, int num_fences, int expected_errno, if (flags & INTERRUPTIBLE) igt_stop_signal_helper(); + + /* Cleanup */ + for (n = 0; n < 2*num_fences; n++) + gem_close(fd, exec[0][n].handle); + + for (i = 0; i < 2; i++) + gem_close(fd, exec[i][2*num_fences].handle); + + if (flags & BUSY_LOAD) { + intel_batchbuffer_free(batch); + drm_intel_bufmgr_destroy(bufmgr); + } } int fd;