Now that we bump the ABI, it's a good opportunity to split the
tracepoint "name" field, which is a concatenation of
#provider_name ':' #event_name
into separate provider and name fields.
Reject tracepoints and callsites which are larger than the maximum
length supported by LTTng-UST rather than silently truncating them.
This emits a warning visible with LTTNG_UST_DEBUG=1 to allow pinpointing
erroneous users.
Note that as long as tracepoints are defined statically with the
tracepoint API, the length of the provider and event names is validated
with a build-time check.
This means the core of tracepoint.c is now agnostic to the length of the
tracepoint provider/event name. If there is ever a change to the
LTTng-UST protocol allowing larger length for those, changing the
length validations will be straightforward.
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Change-Id: I770cf57e1f19ec6f4d8c1c7b05e537a176a046cb
struct lttng_ust_tracepoint {
uint32_t struct_size;
struct lttng_ust_tracepoint {
uint32_t struct_size;
+ const char *provider_name;
+ const char *event_name;
int state;
struct lttng_ust_tracepoint_probe *probes;
int *tracepoint_provider_ref;
int state;
struct lttng_ust_tracepoint_probe *probes;
int *tracepoint_provider_ref;
tp_rcu_read_unlock(); \
} \
static inline \
tp_rcu_read_unlock(); \
} \
static inline \
-void __tracepoint_register_##_provider##___##_name(char *name, \
+void __tracepoint_register_##_provider##___##_name(char *provider_name, char *event_name, \
void (*func)(void), void *data) \
lttng_ust_notrace; \
static inline \
void (*func)(void), void *data) \
lttng_ust_notrace; \
static inline \
-void __tracepoint_register_##_provider##___##_name(char *name, \
+void __tracepoint_register_##_provider##___##_name(char *provider_name, char *event_name, \
void (*func)(void), void *data) \
{ \
void (*func)(void), void *data) \
{ \
- __tracepoint_probe_register(name, func, data, \
+ __tracepoint_probe_register(provider_name, event_name, func, data, \
__tracepoint_##_provider##___##_name.signature); \
} \
static inline \
__tracepoint_##_provider##___##_name.signature); \
} \
static inline \
-void __tracepoint_unregister_##_provider##___##_name(char *name, \
+void __tracepoint_unregister_##_provider##___##_name(char *provider_name, char *event_name, \
void (*func)(void), void *data) \
lttng_ust_notrace; \
static inline \
void (*func)(void), void *data) \
lttng_ust_notrace; \
static inline \
-void __tracepoint_unregister_##_provider##___##_name(char *name, \
+void __tracepoint_unregister_##_provider##___##_name(char *provider_name, char *event_name, \
void (*func)(void), void *data) \
{ \
void (*func)(void), void *data) \
{ \
- __tracepoint_probe_unregister(name, func, data); \
+ __tracepoint_probe_unregister(provider_name, event_name, func, data); \
-extern int __tracepoint_probe_register(const char *name, void (*func)(void),
- void *data, const char *signature);
-extern int __tracepoint_probe_unregister(const char *name, void (*func)(void),
- void *data);
+extern int __tracepoint_probe_register(const char *provider_name, const char *event_name,
+ void (*func)(void), void *data, const char *signature);
+extern int __tracepoint_probe_unregister(const char *provider_name, const char *event_name,
+ void (*func)(void), void *data);
/*
* tracepoint dynamic linkage handling (callbacks). Hidden visibility:
/*
* tracepoint dynamic linkage handling (callbacks). Hidden visibility:
#define _DEFINE_TRACEPOINT(_provider, _name, _args) \
lttng_ust_tracepoint_validate_name_len(_provider, _name); \
extern int __tracepoint_provider_##_provider; \
#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[] \
+ static const char __tp_provider_strtab_##_provider##___##_name[] \
__attribute__((section("__tracepoints_strings"))) = \
__attribute__((section("__tracepoints_strings"))) = \
- #_provider ":" #_name; \
+ #_provider; \
+ static const char __tp_name_strtab_##_provider##___##_name[] \
+ __attribute__((section("__tracepoints_strings"))) = \
+ #_name; \
struct lttng_ust_tracepoint __tracepoint_##_provider##___##_name \
__attribute__((section("__tracepoints"))) = { \
sizeof(struct lttng_ust_tracepoint), \
struct lttng_ust_tracepoint __tracepoint_##_provider##___##_name \
__attribute__((section("__tracepoints"))) = { \
sizeof(struct lttng_ust_tracepoint), \
- __tp_strtab_##_provider##___##_name, \
+ __tp_provider_strtab_##_provider##___##_name, \
+ __tp_name_strtab_##_provider##___##_name, \
0, \
NULL, \
_TRACEPOINT_UNDEFINED_REF(_provider), \
0, \
NULL, \
_TRACEPOINT_UNDEFINED_REF(_provider), \
* These symbols are ABI between liblttng-ust-tracepoint and liblttng-ust,
* which is why they are not hidden and not part of the public API.
*/
* These symbols are ABI between liblttng-ust-tracepoint and liblttng-ust,
* which is why they are not hidden and not part of the public API.
*/
-int lttng_ust_tp_probe_register_queue_release(const char *name,
+int lttng_ust_tp_probe_register_queue_release(const char *provider_name, const char *event_name,
void (*func)(void), void *data, const char *signature);
void (*func)(void), void *data, const char *signature);
-int lttng_ust_tp_probe_unregister_queue_release(const char *name,
+int lttng_ust_tp_probe_unregister_queue_release(const char *provider_name, const char *event_name,
void (*func)(void), void *data);
void lttng_ust_tp_probe_prune_release_queue(void);
void (*func)(void), void *data);
void lttng_ust_tp_probe_prune_release_queue(void);
int refcount; /* Number of times armed. 0 if disarmed. */
int callsite_refcount; /* how many libs use this tracepoint */
char *signature;
int refcount; /* Number of times armed. 0 if disarmed. */
int callsite_refcount; /* how many libs use this tracepoint */
char *signature;
+ char *provider_name;
+ char *event_name;
"Tracepoint name max length mismatch between UST ABI and tracepoint API",
Tracepoint_name_max_length_mismatch);
"Tracepoint name max length mismatch between UST ABI and tracepoint API",
Tracepoint_name_max_length_mismatch);
+static
+bool lttng_ust_tp_validate_event_name(const struct lttng_ust_tracepoint *tp)
+{
+ if (strlen(tp->provider_name) + 1 +
+ strlen(tp->event_name) >= LTTNG_UST_TRACEPOINT_NAME_LEN_MAX)
+ return false;
+ return true;
+}
+
/* coverity[+alloc] */
static void *allocate_probes(int count)
{
/* coverity[+alloc] */
static void *allocate_probes(int count)
{
* Must be called with tracepoint mutex held.
* Returns NULL if not present.
*/
* Must be called with tracepoint mutex held.
* Returns NULL if not present.
*/
-static struct tracepoint_entry *get_tracepoint(const char *name)
+static struct tracepoint_entry *get_tracepoint(const char *provider_name, const char *event_name)
{
struct cds_hlist_head *head;
struct cds_hlist_node *node;
struct tracepoint_entry *e;
{
struct cds_hlist_head *head;
struct cds_hlist_node *node;
struct tracepoint_entry *e;
- size_t name_len = strlen(name);
- if (name_len > LTTNG_UST_ABI_SYM_NAME_LEN - 1) {
- WARN("Truncating tracepoint name %s which exceeds size limits of %u chars", name, LTTNG_UST_ABI_SYM_NAME_LEN - 1);
- name_len = LTTNG_UST_ABI_SYM_NAME_LEN - 1;
- }
- hash = jhash(name, name_len, 0);
+ hash = jhash(provider_name, strlen(provider_name), 0) ^
+ jhash(event_name, strlen(event_name), 0);
head = &tracepoint_table[hash & (TRACEPOINT_TABLE_SIZE - 1)];
cds_hlist_for_each_entry(e, node, head, hlist) {
head = &tracepoint_table[hash & (TRACEPOINT_TABLE_SIZE - 1)];
cds_hlist_for_each_entry(e, node, head, hlist) {
- if (!strncmp(name, e->name, LTTNG_UST_ABI_SYM_NAME_LEN - 1))
+ if (!strcmp(event_name, e->event_name) && !strcmp(provider_name, e->provider_name))
* Add the tracepoint to the tracepoint hash table. Must be called with
* tracepoint mutex held.
*/
* Add the tracepoint to the tracepoint hash table. Must be called with
* tracepoint mutex held.
*/
-static struct tracepoint_entry *add_tracepoint(const char *name,
+static struct tracepoint_entry *add_tracepoint(const char *provider_name, const char *event_name,
const char *signature)
{
struct cds_hlist_head *head;
struct cds_hlist_node *node;
struct tracepoint_entry *e;
const char *signature)
{
struct cds_hlist_head *head;
struct cds_hlist_node *node;
struct tracepoint_entry *e;
- size_t name_len = strlen(name);
size_t sig_len = strlen(signature);
size_t sig_len = strlen(signature);
- size_t sig_off, name_off;
+ size_t sig_off, provider_name_off, event_name_off;
+ size_t provider_name_len = strlen(provider_name);
+ size_t event_name_len = strlen(event_name);
- if (name_len > LTTNG_UST_ABI_SYM_NAME_LEN - 1) {
- WARN("Truncating tracepoint name %s which exceeds size limits of %u chars", name, LTTNG_UST_ABI_SYM_NAME_LEN - 1);
- name_len = LTTNG_UST_ABI_SYM_NAME_LEN - 1;
- }
- hash = jhash(name, name_len, 0);
+ hash = jhash(provider_name, provider_name_len, 0) ^
+ jhash(event_name, event_name_len, 0);
head = &tracepoint_table[hash & (TRACEPOINT_TABLE_SIZE - 1)];
cds_hlist_for_each_entry(e, node, head, hlist) {
head = &tracepoint_table[hash & (TRACEPOINT_TABLE_SIZE - 1)];
cds_hlist_for_each_entry(e, node, head, hlist) {
- if (!strncmp(name, e->name, LTTNG_UST_ABI_SYM_NAME_LEN - 1)) {
- DBG("tracepoint %s busy", name);
+ if (!strcmp(event_name, e->event_name) && !strcmp(provider_name, e->provider_name)) {
+ DBG("tracepoint \"%s:%s\" busy", provider_name, event_name);
return ERR_PTR(-EEXIST); /* Already there */
}
}
return ERR_PTR(-EEXIST); /* Already there */
}
}
* Using zmalloc here to allocate a variable length elements: name and
* signature. Could cause some memory fragmentation if overused.
*/
* Using zmalloc here to allocate a variable length elements: name and
* signature. Could cause some memory fragmentation if overused.
*/
- name_off = sizeof(struct tracepoint_entry);
- sig_off = name_off + name_len + 1;
+ provider_name_off = sizeof(struct tracepoint_entry);
+ event_name_off = provider_name_off + provider_name_len + 1;
+ sig_off = event_name_off + event_name_len + 1;
- e = zmalloc(sizeof(struct tracepoint_entry) + name_len + 1 + sig_len + 1);
+ e = zmalloc(sizeof(struct tracepoint_entry) + provider_name_len + 1 +
+ event_name_len + 1 + sig_len + 1);
if (!e)
return ERR_PTR(-ENOMEM);
if (!e)
return ERR_PTR(-ENOMEM);
- e->name = (char *) e + name_off;
- memcpy(e->name, name, name_len + 1);
- e->name[name_len] = '\0';
+ e->provider_name = (char *) e + provider_name_off;
+ memcpy(e->provider_name, provider_name, provider_name_len + 1);
+ e->provider_name[provider_name_len] = '\0';
+
+ e->event_name = (char *) e + event_name_off;
+ memcpy(e->event_name, event_name, event_name_len + 1);
+ e->event_name[event_name_len] = '\0';
e->signature = (char *) e + sig_off;
memcpy(e->signature, signature, sig_len + 1);
e->signature = (char *) e + sig_off;
memcpy(e->signature, signature, sig_len + 1);
static void set_tracepoint(struct tracepoint_entry **entry,
struct lttng_ust_tracepoint *elem, int active)
{
static void set_tracepoint(struct tracepoint_entry **entry,
struct lttng_ust_tracepoint *elem, int active)
{
- WARN_ON(strncmp((*entry)->name, elem->name, LTTNG_UST_ABI_SYM_NAME_LEN - 1) != 0);
+ WARN_ON(strcmp((*entry)->provider_name, elem->provider_name) != 0);
+ WARN_ON(strcmp((*entry)->event_name, elem->event_name) != 0);
/*
* Check that signatures match before connecting a probe to a
* tracepoint. Warn the user if they don't.
/*
* Check that signatures match before connecting a probe to a
* tracepoint. Warn the user if they don't.
/* Only print once, don't flood console. */
if (!warned) {
WARN("Tracepoint signature mismatch, not enabling one or more tracepoints. Ensure that the tracepoint probes prototypes match the application.");
/* Only print once, don't flood console. */
if (!warned) {
WARN("Tracepoint signature mismatch, not enabling one or more tracepoints. Ensure that the tracepoint probes prototypes match the application.");
- WARN("Tracepoint \"%s\" signatures: call: \"%s\" vs probe: \"%s\".",
- elem->name, elem->signature, (*entry)->signature);
+ WARN("Tracepoint \"%s:%s\" signatures: call: \"%s\" vs probe: \"%s\".",
+ elem->provider_name, elem->event_name, elem->signature, (*entry)->signature);
warned = 1;
}
/* Don't accept connecting non-matching signatures. */
warned = 1;
}
/* Don't accept connecting non-matching signatures. */
{
struct cds_hlist_head *head;
struct callsite_entry *e;
{
struct cds_hlist_head *head;
struct callsite_entry *e;
- const char *name = tp->name;
- size_t name_len = strlen(name);
uint32_t hash;
struct tracepoint_entry *tp_entry;
uint32_t hash;
struct tracepoint_entry *tp_entry;
- if (name_len > LTTNG_UST_ABI_SYM_NAME_LEN - 1) {
- WARN("Truncating tracepoint name %s which exceeds size limits of %u chars", name, LTTNG_UST_ABI_SYM_NAME_LEN - 1);
- name_len = LTTNG_UST_ABI_SYM_NAME_LEN - 1;
+ if (!lttng_ust_tp_validate_event_name(tp)) {
+ WARN("Rejecting tracepoint name \"%s:%s\" which exceeds size limits of %u chars",
+ tp->provider_name, tp->event_name, LTTNG_UST_TRACEPOINT_NAME_LEN_MAX - 1);
+ return;
- hash = jhash(name, name_len, 0);
+ hash = jhash(tp->provider_name, strlen(tp->provider_name), 0) ^
+ jhash(tp->event_name, strlen(tp->event_name), 0);
head = &callsite_table[hash & (CALLSITE_TABLE_SIZE - 1)];
e = zmalloc(sizeof(struct callsite_entry));
if (!e) {
head = &callsite_table[hash & (CALLSITE_TABLE_SIZE - 1)];
e = zmalloc(sizeof(struct callsite_entry));
if (!e) {
- PERROR("Unable to add callsite for tracepoint \"%s\"", name);
+ PERROR("Unable to add callsite for tracepoint \"%s:%s\"", tp->provider_name, tp->event_name);
return;
}
cds_hlist_add_head(&e->hlist, head);
e->tp = tp;
cds_list_add(&e->node, &lib->callsites);
return;
}
cds_hlist_add_head(&e->hlist, head);
e->tp = tp;
cds_list_add(&e->node, &lib->callsites);
- tp_entry = get_tracepoint(name);
+ tp_entry = get_tracepoint(tp->provider_name, tp->event_name);
if (!tp_entry)
return;
tp_entry->callsite_refcount++;
if (!tp_entry)
return;
tp_entry->callsite_refcount++;
{
struct tracepoint_entry *tp_entry;
{
struct tracepoint_entry *tp_entry;
- tp_entry = get_tracepoint(e->tp->name);
+ tp_entry = get_tracepoint(e->tp->provider_name, e->tp->event_name);
if (tp_entry) {
if (e->tp_entry_callsite_ref)
tp_entry->callsite_refcount--;
if (tp_entry) {
if (e->tp_entry_callsite_ref)
tp_entry->callsite_refcount--;
* tracepoint entry.
* Must be called with tracepoint mutex held.
*/
* tracepoint entry.
* Must be called with tracepoint mutex held.
*/
-static void tracepoint_sync_callsites(const char *name)
+static void tracepoint_sync_callsites(const char *provider_name, const char *event_name)
+ struct tracepoint_entry *tp_entry;
struct cds_hlist_head *head;
struct cds_hlist_node *node;
struct callsite_entry *e;
struct cds_hlist_head *head;
struct cds_hlist_node *node;
struct callsite_entry *e;
- size_t name_len = strlen(name);
- struct tracepoint_entry *tp_entry;
- tp_entry = get_tracepoint(name);
- if (name_len > LTTNG_UST_ABI_SYM_NAME_LEN - 1) {
- WARN("Truncating tracepoint name %s which exceeds size limits of %u chars", name, LTTNG_UST_ABI_SYM_NAME_LEN - 1);
- name_len = LTTNG_UST_ABI_SYM_NAME_LEN - 1;
- }
- hash = jhash(name, name_len, 0);
+ tp_entry = get_tracepoint(provider_name, event_name);
+ hash = jhash(provider_name, strlen(provider_name), 0) ^
+ jhash(event_name, strlen(event_name), 0);
head = &callsite_table[hash & (CALLSITE_TABLE_SIZE - 1)];
cds_hlist_for_each_entry(e, node, head, hlist) {
struct lttng_ust_tracepoint *tp = e->tp;
head = &callsite_table[hash & (CALLSITE_TABLE_SIZE - 1)];
cds_hlist_for_each_entry(e, node, head, hlist) {
struct lttng_ust_tracepoint *tp = e->tp;
- if (strncmp(name, tp->name, LTTNG_UST_ABI_SYM_NAME_LEN - 1))
+ if (strcmp(event_name, tp->event_name))
+ continue;
+ if (strcmp(provider_name, tp->provider_name))
continue;
if (tp_entry) {
if (!e->tp_entry_callsite_ref) {
continue;
if (tp_entry) {
if (!e->tp_entry_callsite_ref) {
for (iter = begin; iter < end; iter++) {
if (!*iter)
continue; /* skip dummy */
for (iter = begin; iter < end; iter++) {
if (!*iter)
continue; /* skip dummy */
+ if (!(*iter)->provider_name || !(*iter)->event_name) {
+ disable_tracepoint(*iter);
+ continue;
+ }
+ if (!lttng_ust_tp_validate_event_name(*iter)) {
+ WARN("Rejecting tracepoint name \"%s:%s\" which exceeds size limits of %u chars",
+ (*iter)->provider_name, (*iter)->event_name, LTTNG_UST_TRACEPOINT_NAME_LEN_MAX - 1);
disable_tracepoint(*iter);
continue;
}
disable_tracepoint(*iter);
continue;
}
- mark_entry = get_tracepoint((*iter)->name);
+ mark_entry = get_tracepoint((*iter)->provider_name, (*iter)->event_name);
if (mark_entry) {
set_tracepoint(&mark_entry, *iter,
!!mark_entry->refcount);
if (mark_entry) {
set_tracepoint(&mark_entry, *iter,
!!mark_entry->refcount);
for (iter = begin; iter < end; iter++) {
if (!*iter)
continue; /* skip dummy */
for (iter = begin; iter < end; iter++) {
if (!*iter)
continue; /* skip dummy */
+ if (!(*iter)->provider_name || !(*iter)->event_name) {
continue;
}
add_callsite(lib, *iter);
continue;
}
add_callsite(lib, *iter);
}
static struct lttng_ust_tracepoint_probe *
}
static struct lttng_ust_tracepoint_probe *
-tracepoint_add_probe(const char *name, void (*probe)(void), void *data,
- const char *signature)
+tracepoint_add_probe(const char *provider_name, const char *event_name,
+ void (*probe)(void), void *data, const char *signature)
{
struct tracepoint_entry *entry;
struct lttng_ust_tracepoint_probe *old;
{
struct tracepoint_entry *entry;
struct lttng_ust_tracepoint_probe *old;
- entry = get_tracepoint(name);
+ entry = get_tracepoint(provider_name, event_name);
if (entry) {
if (strcmp(entry->signature, signature) != 0) {
if (entry) {
if (strcmp(entry->signature, signature) != 0) {
- ERR("Tracepoint and probe signature do not match.");
+ ERR("Tracepoint \"%s:%s\": tracepoint and probe signature do not match: \"%s\" vs \"%s\".",
+ provider_name, event_name, entry->signature, signature);
return ERR_PTR(-EINVAL);
}
} else {
return ERR_PTR(-EINVAL);
}
} else {
- entry = add_tracepoint(name, signature);
+ entry = add_tracepoint(provider_name, event_name, signature);
if (IS_ERR(entry))
return (struct lttng_ust_tracepoint_probe *)entry;
}
if (IS_ERR(entry))
return (struct lttng_ust_tracepoint_probe *)entry;
}
/**
* __tracepoint_probe_register - Connect a probe to a tracepoint
/**
* __tracepoint_probe_register - Connect a probe to a tracepoint
- * @name: tracepoint name
+ * @name: tracepoint provider name
+ * @name: tracepoint event name
* @probe: probe handler
*
* Returns 0 if ok, error value on error.
* The probe address must at least be aligned on the architecture pointer size.
* Called with the tracepoint mutex held.
*/
* @probe: probe handler
*
* Returns 0 if ok, error value on error.
* The probe address must at least be aligned on the architecture pointer size.
* Called with the tracepoint mutex held.
*/
-int __tracepoint_probe_register(const char *name, void (*probe)(void),
- void *data, const char *signature)
+int __tracepoint_probe_register(const char *provider_name, const char *event_name,
+ void (*probe)(void), void *data, const char *signature)
{
void *old;
int ret = 0;
{
void *old;
int ret = 0;
- DBG("Registering probe to tracepoint %s", name);
+ DBG("Registering probe to tracepoint \"%s:%s\"", provider_name, event_name);
pthread_mutex_lock(&tracepoint_mutex);
pthread_mutex_lock(&tracepoint_mutex);
- old = tracepoint_add_probe(name, probe, data, signature);
+ old = tracepoint_add_probe(provider_name, event_name, probe, data, signature);
if (IS_ERR(old)) {
ret = PTR_ERR(old);
goto end;
}
if (IS_ERR(old)) {
ret = PTR_ERR(old);
goto end;
}
- tracepoint_sync_callsites(name);
+ tracepoint_sync_callsites(provider_name, event_name);
release_probes(old);
end:
pthread_mutex_unlock(&tracepoint_mutex);
release_probes(old);
end:
pthread_mutex_unlock(&tracepoint_mutex);
* calling lttng_ust_tp_probe_register_queue_release() one or multiple
* times to ensure it does not leak memory.
*/
* calling lttng_ust_tp_probe_register_queue_release() one or multiple
* times to ensure it does not leak memory.
*/
-int lttng_ust_tp_probe_register_queue_release(const char *name,
+int lttng_ust_tp_probe_register_queue_release(const char *provider_name, const char *event_name,
void (*probe)(void), void *data, const char *signature)
{
void *old;
int ret = 0;
void (*probe)(void), void *data, const char *signature)
{
void *old;
int ret = 0;
- DBG("Registering probe to tracepoint %s. Queuing release.", name);
+ DBG("Registering probe to tracepoint \"%s:%s\". Queuing release.", provider_name, event_name);
pthread_mutex_lock(&tracepoint_mutex);
pthread_mutex_lock(&tracepoint_mutex);
- old = tracepoint_add_probe(name, probe, data, signature);
+ old = tracepoint_add_probe(provider_name, event_name, probe, data, signature);
if (IS_ERR(old)) {
ret = PTR_ERR(old);
goto end;
}
if (IS_ERR(old)) {
ret = PTR_ERR(old);
goto end;
}
- tracepoint_sync_callsites(name);
+ tracepoint_sync_callsites(provider_name, event_name);
tracepoint_release_queue_add_old_probes(old);
end:
pthread_mutex_unlock(&tracepoint_mutex);
return ret;
}
tracepoint_release_queue_add_old_probes(old);
end:
pthread_mutex_unlock(&tracepoint_mutex);
return ret;
}
-static void *tracepoint_remove_probe(const char *name, void (*probe)(void),
- void *data)
+static void *tracepoint_remove_probe(const char *provider_name, const char *event_name,
+ void (*probe)(void), void *data)
{
struct tracepoint_entry *entry;
void *old;
{
struct tracepoint_entry *entry;
void *old;
- entry = get_tracepoint(name);
+ entry = get_tracepoint(provider_name, event_name);
if (!entry)
return ERR_PTR(-ENOENT);
old = tracepoint_entry_remove_probe(entry, probe, data);
if (!entry)
return ERR_PTR(-ENOENT);
old = tracepoint_entry_remove_probe(entry, probe, data);
/**
* tracepoint_probe_unregister - Disconnect a probe from a tracepoint
/**
* tracepoint_probe_unregister - Disconnect a probe from a tracepoint
- * @name: tracepoint name
+ * @provider_name: tracepoint provider name
+ * @event_name: tracepoint event name
* @probe: probe function pointer
* @probe: probe data pointer
*/
* @probe: probe function pointer
* @probe: probe data pointer
*/
-int __tracepoint_probe_unregister(const char *name, void (*probe)(void),
- void *data)
+int __tracepoint_probe_unregister(const char *provider_name, const char *event_name,
+ void (*probe)(void), void *data)
{
void *old;
int ret = 0;
{
void *old;
int ret = 0;
- DBG("Un-registering probe from tracepoint %s", name);
+ DBG("Un-registering probe \"%s:%s\" from tracepoint %p", provider_name, event_name, probe);
pthread_mutex_lock(&tracepoint_mutex);
pthread_mutex_lock(&tracepoint_mutex);
- old = tracepoint_remove_probe(name, probe, data);
+ old = tracepoint_remove_probe(provider_name, event_name, probe, data);
if (IS_ERR(old)) {
ret = PTR_ERR(old);
goto end;
}
if (IS_ERR(old)) {
ret = PTR_ERR(old);
goto end;
}
- tracepoint_sync_callsites(name);
+ tracepoint_sync_callsites(provider_name, event_name);
release_probes(old);
end:
pthread_mutex_unlock(&tracepoint_mutex);
release_probes(old);
end:
pthread_mutex_unlock(&tracepoint_mutex);
* calling lttng_ust_tp_probe_unregister_queue_release() one or multiple
* times to ensure it does not leak memory.
*/
* calling lttng_ust_tp_probe_unregister_queue_release() one or multiple
* times to ensure it does not leak memory.
*/
-int lttng_ust_tp_probe_unregister_queue_release(const char *name,
+int lttng_ust_tp_probe_unregister_queue_release(const char *provider_name, const char *event_name,
void (*probe)(void), void *data)
{
void *old;
int ret = 0;
void (*probe)(void), void *data)
{
void *old;
int ret = 0;
- DBG("Un-registering probe from tracepoint %s. Queuing release.", name);
+ DBG("Un-registering probe from tracepoint \"%s:%s\". Queuing release.", provider_name, event_name);
pthread_mutex_lock(&tracepoint_mutex);
pthread_mutex_lock(&tracepoint_mutex);
- old = tracepoint_remove_probe(name, probe, data);
+ old = tracepoint_remove_probe(provider_name, event_name, probe, data);
if (IS_ERR(old)) {
ret = PTR_ERR(old);
goto end;
}
if (IS_ERR(old)) {
ret = PTR_ERR(old);
goto end;
}
- tracepoint_sync_callsites(name);
+ tracepoint_sync_callsites(provider_name, event_name);
tracepoint_release_queue_add_old_probes(old);
end:
pthread_mutex_unlock(&tracepoint_mutex);
tracepoint_release_queue_add_old_probes(old);
end:
pthread_mutex_unlock(&tracepoint_mutex);
*
* caller must call tracepoint_probe_update_all()
*/
*
* caller must call tracepoint_probe_update_all()
*/
-int tracepoint_probe_register_noupdate(const char *name, void (*probe)(void),
- void *data, const char *signature)
+int tracepoint_probe_register_noupdate(const char *provider_name, const char *event_name,
+ void (*probe)(void), void *data, const char *signature)
{
void *old;
int ret = 0;
pthread_mutex_lock(&tracepoint_mutex);
{
void *old;
int ret = 0;
pthread_mutex_lock(&tracepoint_mutex);
- old = tracepoint_add_probe(name, probe, data, signature);
+ old = tracepoint_add_probe(provider_name, event_name, probe, data, signature);
if (IS_ERR(old)) {
ret = PTR_ERR(old);
goto end;
if (IS_ERR(old)) {
ret = PTR_ERR(old);
goto end;
* caller must call tracepoint_probe_update_all()
* Called with the tracepoint mutex held.
*/
* caller must call tracepoint_probe_update_all()
* Called with the tracepoint mutex held.
*/
-int tracepoint_probe_unregister_noupdate(const char *name, void (*probe)(void),
- void *data)
+int tracepoint_probe_unregister_noupdate(const char *provider_name, const char *event_name,
+ void (*probe)(void), void *data)
{
void *old;
int ret = 0;
{
void *old;
int ret = 0;
- DBG("Un-registering probe from tracepoint %s", name);
+ DBG("Un-registering probe from tracepoint \"%s:%s\"", provider_name, event_name);
pthread_mutex_lock(&tracepoint_mutex);
pthread_mutex_lock(&tracepoint_mutex);
- old = tracepoint_remove_probe(name, probe, data);
+ old = tracepoint_remove_probe(provider_name, event_name, probe, data);
if (IS_ERR(old)) {
ret = PTR_ERR(old);
goto end;
if (IS_ERR(old)) {
ret = PTR_ERR(old);
goto end;
int i;
for (i = 0; i < tracepoints_count; i++) {
int i;
for (i = 0; i < tracepoints_count; i++) {
- DBG("registered tracepoint: %s", tracepoints_start[i]->name);
+ if (!lttng_ust_tp_validate_event_name(tracepoints_start[i]))
+ continue;
+ DBG("registered tracepoint: \"%s:%s\"",
+ tracepoints_start[i]->provider_name,
+ tracepoints_start[i]->event_name);
struct cds_list_head callsites;
};
struct cds_list_head callsites;
};
-int tracepoint_probe_register_noupdate(const char *name,
+int tracepoint_probe_register_noupdate(const char *provider_name, const char *event_name,
void (*callback)(void), void *priv,
const char *signature)
__attribute__((visibility("hidden")));
void (*callback)(void), void *priv,
const char *signature)
__attribute__((visibility("hidden")));
-int tracepoint_probe_unregister_noupdate(const char *name,
+int tracepoint_probe_unregister_noupdate(const char *provider_name, const char *event_name,
void (*callback)(void), void *priv)
__attribute__((visibility("hidden")));
void (*callback)(void), void *priv)
__attribute__((visibility("hidden")));
{
int ret;
const struct lttng_ust_event_desc *desc;
{
int ret;
const struct lttng_ust_event_desc *desc;
- char name[LTTNG_UST_ABI_SYM_NAME_LEN];
assert(event->priv->registered == 0);
desc = event->priv->desc;
assert(event->priv->registered == 0);
desc = event->priv->desc;
- lttng_ust_format_event_name(desc, name);
- ret = lttng_ust_tp_probe_register_queue_release(name,
+ ret = lttng_ust_tp_probe_register_queue_release(desc->probe_desc->provider_name,
+ desc->event_name,
desc->probe_callback,
event, desc->signature);
WARN_ON_ONCE(ret);
desc->probe_callback,
event, desc->signature);
WARN_ON_ONCE(ret);
{
int ret;
const struct lttng_ust_event_desc *desc;
{
int ret;
const struct lttng_ust_event_desc *desc;
- char name[LTTNG_UST_ABI_SYM_NAME_LEN];
assert(event->priv->registered == 1);
desc = event->priv->desc;
assert(event->priv->registered == 1);
desc = event->priv->desc;
- lttng_ust_format_event_name(desc, name);
- ret = lttng_ust_tp_probe_unregister_queue_release(name,
+ ret = lttng_ust_tp_probe_unregister_queue_release(desc->probe_desc->provider_name,
+ desc->event_name,
desc->probe_callback,
event);
WARN_ON_ONCE(ret);
desc->probe_callback,
event);
WARN_ON_ONCE(ret);