From: Daniel Vetter <daniel.vetter@ffwll.ch>
Date: Mon, 12 Aug 2013 09:57:32 +0000 (+0200)
Subject: lib/drmtest: __ prefix for drmtest_run_subtest
X-Git-Tag: intel-gpu-tools-1.4~249
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=16024841a12f71d52aea12603d9fb1cb63124fb6;p=profile%2Fextras%2Fintel-gpu-tools.git

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
--- 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
--- 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);