fix: all functions have declarations (-Wmissing-prototypes -Wold-style-definition)
[lttng-ust.git] / liblttng-ust / tracepoint.c
index cf7cf63c992d16e90647aa5856d82059bf05c2db..00fcc740101b5f443d77fba5b8596f3053643302 100644 (file)
@@ -1,21 +1,9 @@
 /*
+ * SPDX-License-Identifier: LGPL-2.1-only
+ *
  * Copyright (C) 2008-2011 Mathieu Desnoyers
  * Copyright (C) 2009 Pierre-Marc Fournier
  *
- * This library is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Lesser General Public
- * License as published by the Free Software Foundation;
- * version 2.1 of the License.
- *
- * This library is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public
- * License along with this library; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301 USA
- *
  * Ported to userspace by Pierre-Marc Fournier.
  */
 
 #include <urcu/system.h>
 
 #include <lttng/tracepoint.h>
-#include <lttng/ust-abi.h>     /* for LTTNG_UST_SYM_NAME_LEN */
+#include <lttng/ust-abi.h>     /* for LTTNG_UST_ABI_SYM_NAME_LEN */
 
 #include <usterr-signal-safe.h>
-#include <helper.h>
+#include <ust-helper.h>
 
 #include "tracepoint-internal.h"
 #include "lttng-tracer-core.h"
@@ -60,15 +48,6 @@ static int initialized;
  */
 static int tracepoint_destructors_state = 1;
 
-/*
- * Expose the now deprecated symbol __tracepoints__disable_destructors for
- * backward compatibility of applications built against old versions of
- * lttng-ust. We need to keep __tracepoints__disable_destructors up to date
- * within the new destructor disabling API because old applications read this
- * symbol directly.
- */
-int __tracepoints__disable_destructors __attribute__((weak));
-
 static void (*new_tracepoint_cb)(struct lttng_ust_tracepoint *);
 
 /*
@@ -151,72 +130,6 @@ struct callsite_entry {
        bool tp_entry_callsite_ref; /* Has a tp_entry took a ref on this callsite */
 };
 
-static int tracepoint_v1_api_used;
-static void (*lttng_ust_liburcu_bp_synchronize_rcu)(void);
-static void (*lttng_ust_liburcu_bp_rcu_read_lock)(void);
-static void (*lttng_ust_liburcu_bp_rcu_read_unlock)(void);
-void (*lttng_ust_liburcu_bp_before_fork)(void);
-void (*lttng_ust_liburcu_bp_after_fork_parent)(void);
-void (*lttng_ust_liburcu_bp_after_fork_child)(void);
-
-static bool lttng_ust_tracepoint_v1_used(void)
-{
-       return uatomic_read(&tracepoint_v1_api_used);
-}
-
-static void lttng_ust_tracepoint_set_v1_used(void)
-{
-       if (!lttng_ust_tracepoint_v1_used()) {
-               /*
-                * Perform dlsym here rather than lazily on first use to
-                * eliminate nesting of dynamic loader lock (used within
-                * dlsym) inside the ust lock.
-                */
-               if (!lttng_ust_liburcu_bp_synchronize_rcu) {
-                       lttng_ust_liburcu_bp_synchronize_rcu = URCU_FORCE_CAST(void (*)(void),
-                               dlsym(RTLD_DEFAULT, "synchronize_rcu_bp"));
-                       if (!lttng_ust_liburcu_bp_synchronize_rcu)
-                               abort();
-               }
-               if (!lttng_ust_liburcu_bp_before_fork) {
-                       lttng_ust_liburcu_bp_before_fork = URCU_FORCE_CAST(void (*)(void),
-                               dlsym(RTLD_DEFAULT, "rcu_bp_before_fork"));
-                       if (!lttng_ust_liburcu_bp_before_fork)
-                               abort();
-               }
-               if (!lttng_ust_liburcu_bp_after_fork_parent) {
-                       lttng_ust_liburcu_bp_after_fork_parent = URCU_FORCE_CAST(void (*)(void),
-                               dlsym(RTLD_DEFAULT, "rcu_bp_after_fork_parent"));
-                       if (!lttng_ust_liburcu_bp_after_fork_parent)
-                               abort();
-               }
-               if (!lttng_ust_liburcu_bp_after_fork_child) {
-                       lttng_ust_liburcu_bp_after_fork_child = URCU_FORCE_CAST(void (*)(void),
-                               dlsym(RTLD_DEFAULT, "rcu_bp_after_fork_child"));
-                       if (!lttng_ust_liburcu_bp_after_fork_child)
-                               abort();
-               }
-               if (!lttng_ust_liburcu_bp_rcu_read_lock) {
-                       lttng_ust_liburcu_bp_rcu_read_lock = URCU_FORCE_CAST(void (*)(void),
-                               dlsym(RTLD_DEFAULT, "rcu_read_lock_bp"));
-                       if (!lttng_ust_liburcu_bp_rcu_read_lock)
-                               abort();
-               }
-               if (!lttng_ust_liburcu_bp_rcu_read_unlock) {
-                       lttng_ust_liburcu_bp_rcu_read_unlock = URCU_FORCE_CAST(void (*)(void),
-                               dlsym(RTLD_DEFAULT, "rcu_read_unlock_bp"));
-                       if (!lttng_ust_liburcu_bp_rcu_read_unlock)
-                               abort();
-               }
-
-               /* Fixup URCU bp TLS. */
-               lttng_ust_liburcu_bp_rcu_read_lock();
-               lttng_ust_liburcu_bp_rcu_read_unlock();
-
-               uatomic_set(&tracepoint_v1_api_used, 1);
-       }
-}
-
 /* coverity[+alloc] */
 static void *allocate_probes(int count)
 {
@@ -232,7 +145,7 @@ static void release_probes(void *old)
        if (old) {
                struct tp_probes *tp_probes = caa_container_of(old,
                        struct tp_probes, probes[0]);
-               lttng_ust_synchronize_trace();
+               lttng_ust_urcu_synchronize_rcu();
                free(tp_probes);
        }
 }
@@ -343,14 +256,14 @@ static struct tracepoint_entry *get_tracepoint(const char *name)
        size_t name_len = strlen(name);
        uint32_t hash;
 
-       if (name_len > LTTNG_UST_SYM_NAME_LEN - 1) {
-               WARN("Truncating tracepoint name %s which exceeds size limits of %u chars", name, LTTNG_UST_SYM_NAME_LEN - 1);
-               name_len = LTTNG_UST_SYM_NAME_LEN - 1;
+       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);
        head = &tracepoint_table[hash & (TRACEPOINT_TABLE_SIZE - 1)];
        cds_hlist_for_each_entry(e, node, head, hlist) {
-               if (!strncmp(name, e->name, LTTNG_UST_SYM_NAME_LEN - 1))
+               if (!strncmp(name, e->name, LTTNG_UST_ABI_SYM_NAME_LEN - 1))
                        return e;
        }
        return NULL;
@@ -371,14 +284,14 @@ static struct tracepoint_entry *add_tracepoint(const char *name,
        size_t sig_off, name_off;
        uint32_t hash;
 
-       if (name_len > LTTNG_UST_SYM_NAME_LEN - 1) {
-               WARN("Truncating tracepoint name %s which exceeds size limits of %u chars", name, LTTNG_UST_SYM_NAME_LEN - 1);
-               name_len = LTTNG_UST_SYM_NAME_LEN - 1;
+       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);
        head = &tracepoint_table[hash & (TRACEPOINT_TABLE_SIZE - 1)];
        cds_hlist_for_each_entry(e, node, head, hlist) {
-               if (!strncmp(name, e->name, LTTNG_UST_SYM_NAME_LEN - 1)) {
+               if (!strncmp(name, e->name, LTTNG_UST_ABI_SYM_NAME_LEN - 1)) {
                        DBG("tracepoint %s busy", name);
                        return ERR_PTR(-EEXIST);        /* Already there */
                }
@@ -426,7 +339,7 @@ static void remove_tracepoint(struct tracepoint_entry *e)
 static void set_tracepoint(struct tracepoint_entry **entry,
        struct lttng_ust_tracepoint *elem, int active)
 {
-       WARN_ON(strncmp((*entry)->name, elem->name, LTTNG_UST_SYM_NAME_LEN - 1) != 0);
+       WARN_ON(strncmp((*entry)->name, elem->name, LTTNG_UST_ABI_SYM_NAME_LEN - 1) != 0);
        /*
         * Check that signatures match before connecting a probe to a
         * tracepoint. Warn the user if they don't.
@@ -481,9 +394,9 @@ static void add_callsite(struct tracepoint_lib * lib, struct lttng_ust_tracepoin
        uint32_t hash;
        struct tracepoint_entry *tp_entry;
 
-       if (name_len > LTTNG_UST_SYM_NAME_LEN - 1) {
-               WARN("Truncating tracepoint name %s which exceeds size limits of %u chars", name, LTTNG_UST_SYM_NAME_LEN - 1);
-               name_len = LTTNG_UST_SYM_NAME_LEN - 1;
+       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);
        head = &callsite_table[hash & (CALLSITE_TABLE_SIZE - 1)];
@@ -538,16 +451,16 @@ static void tracepoint_sync_callsites(const char *name)
        struct tracepoint_entry *tp_entry;
 
        tp_entry = get_tracepoint(name);
-       if (name_len > LTTNG_UST_SYM_NAME_LEN - 1) {
-               WARN("Truncating tracepoint name %s which exceeds size limits of %u chars", name, LTTNG_UST_SYM_NAME_LEN - 1);
-               name_len = LTTNG_UST_SYM_NAME_LEN - 1;
+       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);
        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_SYM_NAME_LEN - 1))
+               if (strncmp(name, tp->name, LTTNG_UST_ABI_SYM_NAME_LEN - 1))
                        continue;
                if (tp_entry) {
                        if (!e->tp_entry_callsite_ref) {
@@ -706,10 +619,10 @@ end:
 
 /*
  * Caller needs to invoke __tracepoint_probe_release_queue() after
- * calling __tracepoint_probe_register_queue_release() one or multiple
+ * calling lttng_ust_tp_probe_register_queue_release() one or multiple
  * times to ensure it does not leak memory.
  */
-int __tracepoint_probe_register_queue_release(const char *name,
+int lttng_ust_tp_probe_register_queue_release(const char *name,
                void (*probe)(void), void *data, const char *signature)
 {
        void *old;
@@ -777,10 +690,10 @@ end:
 
 /*
  * Caller needs to invoke __tracepoint_probe_release_queue() after
- * calling __tracepoint_probe_unregister_queue_release() one or multiple
+ * calling lttng_ust_tp_probe_unregister_queue_release() one or multiple
  * times to ensure it does not leak memory.
  */
-int __tracepoint_probe_unregister_queue_release(const char *name,
+int lttng_ust_tp_probe_unregister_queue_release(const char *name,
                void (*probe)(void), void *data)
 {
        void *old;
@@ -801,7 +714,7 @@ end:
        return ret;
 }
 
-void __tracepoint_probe_prune_release_queue(void)
+void lttng_ust_tp_probe_prune_release_queue(void)
 {
        CDS_LIST_HEAD(release_probes);
        struct tp_probes *pos, *next;
@@ -816,7 +729,7 @@ void __tracepoint_probe_prune_release_queue(void)
        release_queue_need_update = 0;
 
        /* Wait for grace period between all sync_callsites and free. */
-       lttng_ust_synchronize_trace();
+       lttng_ust_urcu_synchronize_rcu();
 
        cds_list_for_each_entry_safe(pos, next, &release_probes, u.list) {
                cds_list_del(&pos->u.list);
@@ -907,7 +820,7 @@ void tracepoint_probe_update_all(void)
 
        tracepoint_update_probes();
        /* Wait for grace period between update_probes and free. */
-       lttng_ust_synchronize_trace();
+       lttng_ust_urcu_synchronize_rcu();
        cds_list_for_each_entry_safe(pos, next, &release_probes, u.list) {
                cds_list_del(&pos->u.list);
                free(pos);
@@ -916,11 +829,6 @@ end:
        pthread_mutex_unlock(&tracepoint_mutex);
 }
 
-void tracepoint_set_new_tracepoint_cb(void (*cb)(struct lttng_ust_tracepoint *))
-{
-       new_tracepoint_cb = cb;
-}
-
 static void new_tracepoints(struct lttng_ust_tracepoint * const *start,
                            struct lttng_ust_tracepoint * const *end)
 {
@@ -946,12 +854,14 @@ static void new_tracepoints(struct lttng_ust_tracepoint * const *start,
  * against recent liblttng-ust headers require a recent liblttng-ust
  * runtime for those tracepoints to be taken into account.
  */
-int tracepoint_register_lib2(struct lttng_ust_tracepoint * const *tracepoints_start,
+int tracepoint_register_lib(struct lttng_ust_tracepoint * const *tracepoints_start,
+                            int tracepoints_count);
+int tracepoint_register_lib(struct lttng_ust_tracepoint * const *tracepoints_start,
                             int tracepoints_count)
 {
        struct tracepoint_lib *pl, *iter;
 
-       init_tracepoint();
+       lttng_ust_tp_init();
 
        pl = (struct tracepoint_lib *) zmalloc(sizeof(struct tracepoint_lib));
        if (!pl) {
@@ -984,7 +894,7 @@ lib_added:
 
        DBG("just registered a tracepoints section from %p and having %d tracepoints",
                tracepoints_start, tracepoints_count);
-       if (ust_debug()) {
+       if (ust_err_debug_enabled()) {
                int i;
 
                for (i = 0; i < tracepoints_count; i++) {
@@ -995,15 +905,8 @@ lib_added:
        return 0;
 }
 
-/* Exposed for backward compatibility with old instrumented applications. */
-int tracepoint_register_lib(struct lttng_ust_tracepoint * const *tracepoints_start,
-                            int tracepoints_count)
-{
-       lttng_ust_tracepoint_set_v1_used();
-       return tracepoint_register_lib2(tracepoints_start, tracepoints_count);
-}
-
-int tracepoint_unregister_lib2(struct lttng_ust_tracepoint * const *tracepoints_start)
+int tracepoint_unregister_lib(struct lttng_ust_tracepoint * const *tracepoints_start);
+int tracepoint_unregister_lib(struct lttng_ust_tracepoint * const *tracepoints_start)
 {
        struct tracepoint_lib *lib;
 
@@ -1029,13 +932,6 @@ int tracepoint_unregister_lib2(struct lttng_ust_tracepoint * const *tracepoints_
        return 0;
 }
 
-/* Exposed for backward compatibility with old instrumented applications. */
-int tracepoint_unregister_lib(struct lttng_ust_tracepoint * const *tracepoints_start)
-{
-       lttng_ust_tracepoint_set_v1_used();
-       return tracepoint_unregister_lib2(tracepoints_start);
-}
-
 /*
  * Report in debug message whether the compiler correctly supports weak
  * hidden symbols. This test checks that the address associated with two
@@ -1058,15 +954,15 @@ static void check_weak_hidden(void)
                        "DIFFERENT addresses");
 }
 
-void init_tracepoint(void)
+void lttng_ust_tp_init(void)
 {
        if (uatomic_xchg(&initialized, 1) == 1)
                return;
-       init_usterr();
+       ust_err_init();
        check_weak_hidden();
 }
 
-void exit_tracepoint(void)
+void lttng_ust_tp_exit(void)
 {
        initialized = 0;
 }
@@ -1078,16 +974,19 @@ void exit_tracepoint(void)
 #undef tp_rcu_read_unlock
 #undef tp_rcu_dereference
 
+void tp_rcu_read_lock(void);
 void tp_rcu_read_lock(void)
 {
        lttng_ust_urcu_read_lock();
 }
 
+void tp_rcu_read_unlock(void);
 void tp_rcu_read_unlock(void)
 {
        lttng_ust_urcu_read_unlock();
 }
 
+void *tp_rcu_dereference_sym(void *p);
 void *tp_rcu_dereference_sym(void *p)
 {
        return lttng_ust_rcu_dereference(p);
@@ -1104,6 +1003,7 @@ void *tp_rcu_dereference_sym(void *p)
  * dlopen(3) and dlsym(3) to get an handle on the
  * tp_disable_destructors and tp_get_destructors_state symbols below.
  */
+void tp_disable_destructors(void);
 void tp_disable_destructors(void)
 {
        uatomic_set(&tracepoint_destructors_state, 0);
@@ -1113,36 +1013,8 @@ void tp_disable_destructors(void)
  * Returns 1 if the destructors are enabled and should be executed.
  * Returns 0 if the destructors are disabled.
  */
+int tp_get_destructors_state(void);
 int tp_get_destructors_state(void)
 {
        return uatomic_read(&tracepoint_destructors_state);
 }
-
-void lttng_ust_synchronize_trace(void)
-{
-       lttng_ust_urcu_synchronize_rcu();
-       /*
-        * For legacy tracepoint instrumentation, also wait for urcu-bp
-        * grace period.
-        */
-       if (lttng_ust_liburcu_bp_synchronize_rcu)
-               lttng_ust_liburcu_bp_synchronize_rcu();
-}
-
-/*
- * Create the wrapper symbols for legacy v1 API.
- */
-void tp_rcu_read_lock_bp(void)
-{
-       lttng_ust_urcu_read_lock();
-}
-
-void tp_rcu_read_unlock_bp(void)
-{
-       lttng_ust_urcu_read_unlock();
-}
-
-void *tp_rcu_dereference_sym_bp(void *p)
-{
-       return lttng_ust_rcu_dereference(p);
-}
This page took 0.028381 seconds and 4 git commands to generate.