lib/drmtest: __ prefix for drmtest_run_subtest

Now that callers must setup the longjmp target (we can't do that in
run_subtest itself since that callframe won't survive) tests shouldn't
call this function directly any more.

Make this clear by adding a __ prefix.

Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

diff --git a/lib/drmtest.c b/lib/drmtest.c
index 7e3c1d8..5f1b4ae 100644 (file)
--- a/lib/drmtest.c
+++ b/lib/drmtest.c
@@ -717,7 +717,7 @@ out:
  * outside of places protected by drmtest_run_subtest checks - the piglit
  * runner adds every line to the subtest list.
  */
-bool drmtest_run_subtest(const char *subtest_name)
+bool __drmtest_run_subtest(const char *subtest_name)
 {
        assert(in_subtest == false);
 

diff --git a/lib/drmtest.h b/lib/drmtest.h
index b687411..0182e09 100644 (file)
--- a/lib/drmtest.h
+++ b/lib/drmtest.h
@@ -98,8 +98,8 @@ void drmtest_progress(const char *header, uint64_t i, uint64_t total);
 /* subtest infrastructure */
 jmp_buf drmtest_subtest_jmpbuf;
 void drmtest_subtest_init(int argc, char **argv);
-bool drmtest_run_subtest(const char *subtest_name);
-#define drmtest_subtest_block(name) for (; drmtest_run_subtest((name)) && \
+bool __drmtest_run_subtest(const char *subtest_name);
+#define drmtest_subtest_block(name) for (; __drmtest_run_subtest((name)) && \
                                         (setjmp(drmtest_subtest_jmpbuf) == 0); \
                                         drmtest_success())
 bool drmtest_only_list_subtests(void);