<div dir="ltr">Hello, <div>is there any chance to get review?</div><div>In the goal is idea to get a simple mechanism to control resources freeing (any extra memory allocations, files etc.) of any piglit-test on the exit.</div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 21, 2018 at 12:18 PM Sergii Romantsov <<a href="mailto:sergii.romantsov@gmail.com">sergii.romantsov@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Each test can be initialized ('init') and displayed ('dipslay').<br>
During initialization can be allocated resources.<br>
But there is no common unified mechanism to free resources on test<br>
finishing, except, probably, registaration of callabck with 'atexit'.<br>
<br>
To the piglit-configuration added callback-function 'deinit' that<br>
will be called on the test end.<br>
<br>
CC: Brian Paul <<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.com</a>><br>
CC: Timothy Arceri <<a href="mailto:tarceri@itsqueeze.com" target="_blank">tarceri@itsqueeze.com</a>><br>
Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=108819" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=108819</a><br>
Signed-off-by: Sergii Romantsov <<a href="mailto:sergii.romantsov@globallogic.com" target="_blank">sergii.romantsov@globallogic.com</a>><br>
---<br>
 tests/util/piglit-framework-gl.h                     | 3 +++<br>
 tests/util/piglit-framework-gl/piglit_gl_framework.c | 2 ++<br>
 2 files changed, 5 insertions(+)<br>
<br>
diff --git a/tests/util/piglit-framework-gl.h b/tests/util/piglit-framework-gl.h<br>
index a4ecddb..2dfbad2 100644<br>
--- a/tests/util/piglit-framework-gl.h<br>
+++ b/tests/util/piglit-framework-gl.h<br>
@@ -192,6 +192,9 @@ struct piglit_gl_test_config {<br>
        enum piglit_result<br>
        (*display)(void);<br>
<br>
+       void<br>
+       (*deinit)(void);<br>
+<br>
        /**<br>
         * List of subtests supported by this test case<br>
         *<br>
diff --git a/tests/util/piglit-framework-gl/piglit_gl_framework.c b/tests/util/piglit-framework-gl/piglit_gl_framework.c<br>
index 37c0677..bdb9dbd 100644<br>
--- a/tests/util/piglit-framework-gl/piglit_gl_framework.c<br>
+++ b/tests/util/piglit-framework-gl/piglit_gl_framework.c<br>
@@ -116,5 +116,7 @@ piglit_gl_framework_init(struct piglit_gl_framework *gl_fw,<br>
 void<br>
 piglit_gl_framework_teardown(struct piglit_gl_framework *gl_fw)<br>
 {<br>
+       if (gl_fw->test_config->deinit)<br>
+               gl_fw->test_config->deinit();<br>
        return;<br>
 }<br>
-- <br>
2.7.4<br>
<br>
</blockquote></div>