Moved the CLIENT_tstCALL[0123] requests out of valgrind.h into vg_skin.h,
because there was no point exposing them to clients, as they don't know the
ThreadState type.

Also, removed the LOGMESSAGE request type, replaced it with calls to
VG_(message) via the generic VALGRIND_NON_SIMD_CALL2.

In fact, almost every single pthread client request could be removed in this
same way.  That would result in less code, which would be nice... yeah, real
nice.


git-svn-id: svn://svn.valgrind.org/valgrind/trunk@1584 a5019735-40e9-0310-863c-91ae7b9d1cf9
diff --git a/coregrind/vg_include.h b/coregrind/vg_include.h
index 2d2131c..ca70eef 100644
--- a/coregrind/vg_include.h
+++ b/coregrind/vg_include.h
@@ -448,8 +448,17 @@
    request codes.  A few, publically-visible, request codes are also
    defined in valgrind.h, and similar headers for some skins. */
 
-#define VG_USERREQ__MALLOC              0x2001
-#define VG_USERREQ__FREE                0x2002
+#define VG_USERREQ__MALLOC                  0x2001
+#define VG_USERREQ__FREE                    0x2002
+
+/* 
+In vg_skin.h, so skins can use it.
+Call an arbitrary function with ThreadState as the first arg.
+#define VG_USERREQ__CLIENT_tstCALL0         0x2101
+#define VG_USERREQ__CLIENT_tstCALL1         0x2102
+#define VG_USERREQ__CLIENT_tstCALL2         0x2103
+#define VG_USERREQ__CLIENT_tstCALL3         0x2104
+*/
 
 /* (Fn, Arg): Create a new thread and run Fn applied to Arg in it.  Fn
    MUST NOT return -- ever.  Eventually it will do either __QUIT or
@@ -524,17 +533,13 @@
 #define VG_USERREQ__GET_PTHREAD_TRACE_LEVEL 0x3101
 /* Log a pthread error from client-space.  Cosmetic. */
 #define VG_USERREQ__PTHREAD_ERROR           0x3102
-/* 
-In vg_skin.h, so skins can use it.
-Write a string to the logging sink. 
-#define VG_USERREQ__LOGMESSAGE              0x3103
-*/
 
 /* 
 In vg_constants.h:
 #define VG_USERREQ__SIGNAL_RETURNS          0x4001
 */
 
+
 /* The scheduler does need to know the address of it so it can be
    called at program exit. */
 extern void VG_(__libc_freeres_wrapper)( void );