Tracepoint API namespacing tracef

The ABI bump gives us the opportunity to namespace all public symbols
under the 'lttng_ust_' prefix. Namespace all API symbols and macros
under 'lttng_ust_' / 'LTTNG_UST_' and add compat macros to keep
compatibility with the previous API.

Change-Id: I1a16b9af1dfabb9483cdf3e5225ebe50fe691935
Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

diff --git a/doc/examples/demo-tracef/demo-tracef.c b/doc/examples/demo-tracef/demo-tracef.c
index 8af0a5837fee9a6c11b218e9a05691bfb6b12889..b90c07bd765fa7299551becc977435402b5d84d3 100644 (file)
--- a/doc/examples/demo-tracef/demo-tracef.c
+++ b/doc/examples/demo-tracef/demo-tracef.c
@@ -27,7 +27,7 @@ int main(int argc, char **argv)
 
        fprintf(stderr, "Tracing... ");
        for (i = 0; i < 5; i++) {
-               tracef("This is a \"%s\" formatted %d event %lx",
+               lttng_ust_tracef("This is a \"%s\" formatted %d event %lx",
                        str, i, l);
        }
        fprintf(stderr, " done.\n");

diff --git a/doc/examples/demo-tracef/demo-vtracef.c b/doc/examples/demo-tracef/demo-vtracef.c
index 8fe8bb36d27a4bd55da56ac4d0b1d5fa1155f24d..13564f41b0c0e17964ee6b73a716ff4833082c1b 100644 (file)
--- a/doc/examples/demo-tracef/demo-vtracef.c
+++ b/doc/examples/demo-tracef/demo-vtracef.c
@@ -20,7 +20,7 @@ void print_debug(const char* msg, ...)
        va_list ap;
 
        va_start(ap, msg);
-       vtracef(msg, ap);
+       lttng_ust_vtracef(msg, ap);
        va_end(ap);
 }
 

diff --git a/include/lttng/tracef.h b/include/lttng/tracef.h
index 4526d3d8e2b670cd7b9de5c5d0b29734f4e9f717..37f7ae478d9aa0ffd3fc2020d8f5d7055aca8f14 100644 (file)
--- a/include/lttng/tracef.h
+++ b/include/lttng/tracef.h
@@ -7,6 +7,7 @@
 #ifndef _LTTNG_UST_TRACEF_H
 #define _LTTNG_UST_TRACEF_H
 
+#include <lttng/ust-api-compat.h>
 #include <lttng/tp/lttng-ust-tracef.h>
 
 #ifdef __cplusplus
@@ -14,25 +15,31 @@ extern "C" {
 #endif
 
 extern
-void _lttng_ust_tracef(const char *fmt, ...)
+void lttng_ust__tracef(const char *fmt, ...)
        __attribute__((format(printf, 1, 2)));
 
 extern
-void _lttng_ust_vtracef(const char *fmt, va_list ap)
+void lttng_ust__vtracef(const char *fmt, va_list ap)
        __attribute__((format(printf, 1, 0)));
 
-#define tracef(fmt, ...)                                               \
+#define lttng_ust_tracef(fmt, ...)                                             \
        do {                                                            \
                LTTNG_UST_STAP_PROBEV(tracepoint_lttng_ust_tracef, event, ## __VA_ARGS__); \
                if (caa_unlikely(lttng_ust_tracepoint_lttng_ust_tracef___event.state)) \
-                       _lttng_ust_tracef(fmt, ## __VA_ARGS__);         \
+                       lttng_ust__tracef(fmt, ## __VA_ARGS__);         \
        } while (0)
 
-#define vtracef(fmt, ap)                                               \
+#define lttng_ust_vtracef(fmt, ap)                                             \
        do {                                                            \
                if (caa_unlikely(lttng_ust_tracepoint_lttng_ust_tracef___event.state)) \
-                       _lttng_ust_vtracef(fmt, ap);            \
+                       lttng_ust__vtracef(fmt, ap);            \
        } while (0)
+
+#if LTTNG_UST_COMPAT_API(0)
+#define tracef         lttng_ust_tracef
+#define vtracef                lttng_ust_vtracef
+#endif
+
 #ifdef __cplusplus
 }
 #endif

diff --git a/src/lib/lttng-ust/tracef.c b/src/lib/lttng-ust/tracef.c
index e1824e5c767148c4f36d0111e30447613adfdedf..e321ee07a497c3043243f7d8199a26d423743a5a 100644 (file)
--- a/src/lib/lttng-ust/tracef.c
+++ b/src/lib/lttng-ust/tracef.c
@@ -13,10 +13,10 @@
 #include "lttng-ust-tracef-provider.h"
 
 static inline
-void __lttng_ust_vtracef(const char *fmt, va_list ap)
+void lttng_ust___vtracef(const char *fmt, va_list ap)
        __attribute__((always_inline, format(printf, 1, 0)));
 static inline
-void __lttng_ust_vtracef(const char *fmt, va_list ap)
+void lttng_ust___vtracef(const char *fmt, va_list ap)
 {
        char *msg;
        const int len = vasprintf(&msg, fmt, ap);
@@ -31,25 +31,20 @@ end:
        return;
 }
 
-/*
- * FIXME: We should include <lttng/tracef.h> for the declarations here, but it
- * fails with tracepoint magic above my paygrade.
- */
-
-void _lttng_ust_vtracef(const char *fmt, va_list ap)
+void lttng_ust__vtracef(const char *fmt, va_list ap)
        __attribute__((format(printf, 1, 0)));
-void _lttng_ust_vtracef(const char *fmt, va_list ap)
+void lttng_ust__vtracef(const char *fmt, va_list ap)
 {
-       __lttng_ust_vtracef(fmt, ap);
+       lttng_ust___vtracef(fmt, ap);
 }
 
-void _lttng_ust_tracef(const char *fmt, ...)
+void lttng_ust__tracef(const char *fmt, ...)
        __attribute__((format(printf, 1, 2)));
-void _lttng_ust_tracef(const char *fmt, ...)
+void lttng_ust__tracef(const char *fmt, ...)
 {
        va_list ap;
 
        va_start(ap, fmt);
-       __lttng_ust_vtracef(fmt, ap);
+       lttng_ust___vtracef(fmt, ap);
        va_end(ap);
 }