[Intel-gfx] [PATCH i-g-t] tests/gem_fenced_exec_thrash.c: Fix memory leaks

Derek Morton derek.j.morton at intel.com
Fri Jun 26 03:22:30 PDT 2015


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 <derek.j.morton at intel.com>
---
 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 75f251c..c0e820f 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;
-- 
1.9.1



More information about the Intel-gfx mailing list