#include <lttng/ust-compiler.h>
#include <lttng/ust-tracer.h>
+#define LTTNG_UST_TRACEPOINT_NAME_LEN_MAX 256
+
#ifdef LTTNG_UST_HAVE_SDT_INTEGRATION
/*
* Instead of using SDT_USE_VARIADIC from 'sys/sdt.h', use our own namespaced
*/
#include <sys/sdt.h>
-#define _LTTNG_SDT_NARG(...) \
- __LTTNG_SDT_NARG(__VA_ARGS__, 12,11,10,9,8,7,6,5,4,3,2,1,0)
+#define _LTTNG_UST_SDT_NARG(...) \
+ __LTTNG_UST_SDT_NARG(__VA_ARGS__, 12,11,10,9,8,7,6,5,4,3,2,1,0)
-#define __LTTNG_SDT_NARG(_0,_1,_2,_3,_4,_5,_6,_7,_8,_9,_10,_11,_12, N, ...) N
+#define __LTTNG_UST_SDT_NARG(_0,_1,_2,_3,_4,_5,_6,_7,_8,_9,_10,_11,_12, N, ...) N
-#define _LTTNG_SDT_PROBE_N(provider, name, N, ...) \
+#define _LTTNG_UST_SDT_PROBE_N(provider, name, N, ...) \
_SDT_PROBE(provider, name, N, (__VA_ARGS__))
-#define LTTNG_STAP_PROBEV(provider, name, ...) \
- _LTTNG_SDT_PROBE_N(provider, name, _LTTNG_SDT_NARG(0, ##__VA_ARGS__), ##__VA_ARGS__)
+#define LTTNG_UST_STAP_PROBEV(provider, name, ...) \
+ _LTTNG_UST_SDT_PROBE_N(provider, name, _LTTNG_UST_SDT_NARG(0, ##__VA_ARGS__), ##__VA_ARGS__)
#else
-#define LTTNG_STAP_PROBEV(...)
+#define LTTNG_UST_STAP_PROBEV(...)
#endif
#ifdef __cplusplus
#define tracepoint(provider, name, ...) \
do { \
- LTTNG_STAP_PROBEV(provider, name, ## __VA_ARGS__); \
+ LTTNG_UST_STAP_PROBEV(provider, name, ## __VA_ARGS__); \
if (tracepoint_enabled(provider, name)) \
do_tracepoint(provider, name, __VA_ARGS__); \
} while (0)
#define _TP_ARGS_DATA_VAR(...) _TP_DATA_VAR_N(_TP_NARGS(0, ##__VA_ARGS__), ##__VA_ARGS__)
#define _TP_PARAMS(...) __VA_ARGS__
+/*
+ * sizeof(#_provider) - 1 : length of the provider string (excluding \0).
+ * sizeof(#_name) - 1 : length of the name string (excluding \0).
+ * + 1 : separator between provider and event name.
+ *
+ * Upper bound (inclusive) is LTTNG_UST_TRACEPOINT_NAME_LEN_MAX - 1 to
+ * account for \0.
+ *
+ * The comparison is:
+ * left hand side: sizeof(#_provider) - 1 + sizeof(#_name) - 1 + 1
+ * right hand side: LTTNG_UST_TRACEPOINT_NAME_LEN_MAX - 1
+ * operator: <= (inclusive)
+ * Simplified in the code below.
+ */
+#define lttng_ust_tracepoint_validate_name_len(_provider, _name) \
+ lttng_ust_static_assert(sizeof(#_provider) + sizeof(#_name) <= LTTNG_UST_TRACEPOINT_NAME_LEN_MAX, \
+ "Tracepoint name length is too long", \
+ Tracepoint_name_length_is_too_long)
+
/*
* The tracepoint cb is marked always inline so we can distinguish
* between caller's ip addresses within the probe using the return
end: \
tp_rcu_read_unlock(); \
} \
-static inline lttng_ust_notrace \
+static inline \
void __tracepoint_register_##_provider##___##_name(char *name, \
- void (*func)(void), void *data); \
+ void (*func)(void), void *data) \
+ lttng_ust_notrace; \
static inline \
void __tracepoint_register_##_provider##___##_name(char *name, \
void (*func)(void), void *data) \
__tracepoint_probe_register(name, func, data, \
__tracepoint_##_provider##___##_name.signature); \
} \
-static inline lttng_ust_notrace \
+static inline \
void __tracepoint_unregister_##_provider##___##_name(char *name, \
- void (*func)(void), void *data); \
+ void (*func)(void), void *data) \
+ lttng_ust_notrace; \
static inline \
void __tracepoint_unregister_##_provider##___##_name(char *name, \
void (*func)(void), void *data) \
}
#ifndef _LGPL_SOURCE
-static inline void lttng_ust_notrace
-__tracepoint__init_urcu_sym(void);
+static inline void
+__tracepoint__init_urcu_sym(void)
+ lttng_ust_notrace;
static inline void
__tracepoint__init_urcu_sym(void)
{
"tp_rcu_dereference_sym"));
}
#else
-static inline void lttng_ust_notrace
-__tracepoint__init_urcu_sym(void);
+static inline void
+__tracepoint__init_urcu_sym(void)
+ lttng_ust_notrace;
static inline void
__tracepoint__init_urcu_sym(void)
{
__tracepoint__init_urcu_sym();
}
-static void lttng_ust_notrace __attribute__((destructor))
-__tracepoints__destroy(void);
+static void
+__tracepoints__destroy(void)
+ lttng_ust_notrace __attribute__((destructor));
static void
__tracepoints__destroy(void)
{
#define _TP_EXTRACT_STRING(...) #__VA_ARGS__
#define _DEFINE_TRACEPOINT(_provider, _name, _args) \
+ lttng_ust_tracepoint_validate_name_len(_provider, _name); \
extern int __tracepoint_provider_##_provider; \
static const char __tp_strtab_##_provider##___##_name[] \
__attribute__((section("__tracepoints_strings"))) = \
}
}
-static void lttng_ust_notrace __attribute__((destructor))
-__tracepoints__ptrs_destroy(void);
+static void
+__tracepoints__ptrs_destroy(void)
+ lttng_ust_notrace __attribute__((destructor));
static void
__tracepoints__ptrs_destroy(void)
{