So tracepoint.h (in applications) can just dlopen
liblttng-ust-tracepoint without having to load the full liblttng-ust.
Now liblttng-ust is only needed by tracepoint probes.
This is a first step to fix the deadlock between the dynamic linker
mutex and ust mutex occurring when liblttng-ust is dlopened (due to lazy
symbol resolving of Thread-Local Storage (TLS)).
Discourage dlopen of liblttng-ust (and of tracepoint probes) in the
README.
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
needed.
- Example:
- tests/demo/ demo.c tp*.c ust_tests_demo*.h demo-trace
needed.
- Example:
- tests/demo/ demo.c tp*.c ust_tests_demo*.h demo-trace
+ - Note about dlopen() usage: due to locking side-effects due to the
+ way libc lazily resolves Thread-Local Storage (TLS) symbols when a
+ library is dlopen'd, linking the tracepoint probe or liblttng-ust
+ with dlopen() is discouraged. They should be linked with the
+ application using "-llibname" or loaded with LD_PRELOAD.
- Enable instrumentation and control tracing with the "lttng" command
from lttng-tools. See lttng-tools doc/quickstart.txt.
- Enable instrumentation and control tracing with the "lttng" command
from lttng-tools. See lttng-tools doc/quickstart.txt.
if (__tracepoint_registered++)
return;
if (__tracepoint_registered++)
return;
- liblttngust_handle = dlopen("liblttng-ust.so.0", RTLD_NOW | RTLD_GLOBAL);
+ liblttngust_handle = dlopen("liblttng-ust-tracepoint.so.0", RTLD_NOW | RTLD_GLOBAL);
if (!liblttngust_handle)
return;
tracepoint_register_lib =
if (!liblttngust_handle)
return;
tracepoint_register_lib =
noinst_LTLIBRARIES = liblttng-ust-runtime.la liblttng-ust-support.la
noinst_LTLIBRARIES = liblttng-ust-runtime.la liblttng-ust-support.la
-lib_LTLIBRARIES = liblttng-ust.la
+lib_LTLIBRARIES = liblttng-ust-tracepoint.la liblttng-ust.la
+
+liblttng_ust_tracepoint_la_SOURCES = \
+ tracepoint.c \
+ tracepoint-internal.h \
+ ltt-tracer-core.h \
+ jhash.h \
+ error.h
+liblttng_ust_tracepoint_la_LIBADD = \
+ -lurcu-bp
+liblttng_ust_tracepoint_la_LDFLAGS = -no-undefined -version-info $(LTTNG_UST_LIBRARY_VERSION)
+liblttng_ust_tracepoint_la_CFLAGS = -DUST_COMPONENT="liblttng_ust_tracepoint" -fno-strict-aliasing
liblttng_ust_runtime_la_SOURCES = \
lttng-ust-comm.c \
liblttng_ust_runtime_la_SOURCES = \
lttng-ust-comm.c \
lttng-context-procname.c \
ltt-context.c \
ltt-events.c \
lttng-context-procname.c \
ltt-context.c \
ltt-events.c \
tracepoint-internal.h \
clock.h \
compat.h \
tracepoint-internal.h \
clock.h \
compat.h \
-lpthread \
-lrt \
-luuid \
-lpthread \
-lrt \
-luuid \
+ -llttng-ust-tracepoint \
$(top_builddir)/snprintf/libustsnprintf.la \
$(top_builddir)/liblttng-ust-comm/liblttng-ust-comm.la \
liblttng-ust-runtime.la liblttng-ust-support.la
$(top_builddir)/snprintf/libustsnprintf.la \
$(top_builddir)/liblttng-ust-comm/liblttng-ust-comm.la \
liblttng-ust-runtime.la liblttng-ust-support.la
static int initialized;
static void (*new_tracepoint_cb)(struct tracepoint *);
static int initialized;
static void (*new_tracepoint_cb)(struct tracepoint *);
+/*
+ * tracepoint_mutex nests inside UST mutex.
+ *
+ * Note about interaction with fork/clone: UST does not hold the
+ * tracepoint mutex across fork/clone because it is either:
+ * - nested within UST mutex, in which case holding the UST mutex across
+ * fork/clone suffice,
+ * - taken by a library constructor, which should never race with a
+ * fork/clone if the application is expected to continue running with
+ * the same memory layout (no following exec()).
+ */
+static pthread_mutex_t tracepoint_mutex = PTHREAD_MUTEX_INITIALIZER;
+
/*
* libraries that contain tracepoints (struct tracepoint_lib).
/*
* libraries that contain tracepoints (struct tracepoint_lib).
- * Protected by UST lock.
+ * Protected by tracepoint mutex.
*/
static CDS_LIST_HEAD(libs);
/*
*/
static CDS_LIST_HEAD(libs);
/*
- * The UST lock protects the library tracepoints, the hash table, and
+ * The tracepoint mutex protects the library tracepoints, the hash table, and
- * All calls to the tracepoint API must be protected by the UST lock,
+ * All calls to the tracepoint API must be protected by the tracepoint mutex,
* excepts calls to tracepoint_register_lib and
* excepts calls to tracepoint_register_lib and
- * tracepoint_unregister_lib, which take the UST lock themselves.
+ * tracepoint_unregister_lib, which take the tracepoint mutex themselves.
*/
/*
* Tracepoint hash table, containing the active tracepoints.
*/
/*
* Tracepoint hash table, containing the active tracepoints.
- * Protected by ust lock.
+ * Protected by tracepoint mutex.
*/
#define TRACEPOINT_HASH_BITS 6
#define TRACEPOINT_TABLE_SIZE (1 << TRACEPOINT_HASH_BITS)
*/
#define TRACEPOINT_HASH_BITS 6
#define TRACEPOINT_TABLE_SIZE (1 << TRACEPOINT_HASH_BITS)
* Note about RCU :
* It is used to to delay the free of multiple probes array until a quiescent
* state is reached.
* Note about RCU :
* It is used to to delay the free of multiple probes array until a quiescent
* state is reached.
- * Tracepoint entries modifications are protected by the ust lock.
+ * Tracepoint entries modifications are protected by the tracepoint mutex.
*/
struct tracepoint_entry {
struct cds_hlist_node hlist;
*/
struct tracepoint_entry {
struct cds_hlist_node hlist;
/*
* Get tracepoint if the tracepoint is present in the tracepoint hash table.
/*
* Get tracepoint if the tracepoint is present in the tracepoint hash table.
- * Must be called with ust lock held.
+ * Must be called with tracepoint mutex held.
* Returns NULL if not present.
*/
static struct tracepoint_entry *get_tracepoint(const char *name)
* Returns NULL if not present.
*/
static struct tracepoint_entry *get_tracepoint(const char *name)
/*
* Add the tracepoint to the tracepoint hash table. Must be called with
/*
* 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 *name)
{
/*
* Remove the tracepoint from the tracepoint hash table. Must be called with
/*
* Remove the tracepoint from the tracepoint hash table. Must be called with
+ * tracepoint mutex held.
*/
static void remove_tracepoint(struct tracepoint_entry *e)
{
*/
static void remove_tracepoint(struct tracepoint_entry *e)
{
*
* Returns 0 if ok, error value on error.
* The probe address must at least be aligned on the architecture pointer size.
*
* Returns 0 if ok, error value on error.
* The probe address must at least be aligned on the architecture pointer size.
- * Called with the UST lock held.
+ * Called with the tracepoint mutex held.
*/
int __tracepoint_probe_register(const char *name, void *probe, void *data)
{
void *old;
*/
int __tracepoint_probe_register(const char *name, void *probe, void *data)
{
void *old;
+ pthread_mutex_lock(&tracepoint_mutex);
old = tracepoint_add_probe(name, probe, data);
old = tracepoint_add_probe(name, probe, data);
- if (IS_ERR(old))
- return PTR_ERR(old);
+ if (IS_ERR(old)) {
+ ret = PTR_ERR(old);
+ goto end;
+ }
tracepoint_update_probes(); /* may update entry */
release_probes(old);
tracepoint_update_probes(); /* may update entry */
release_probes(old);
+end:
+ pthread_mutex_unlock(&tracepoint_mutex);
+ return ret;
}
static void *tracepoint_remove_probe(const char *name, void *probe, void *data)
}
static void *tracepoint_remove_probe(const char *name, void *probe, void *data)
* @name: tracepoint name
* @probe: probe function pointer
* @probe: probe data pointer
* @name: tracepoint name
* @probe: probe function pointer
* @probe: probe data pointer
- *
- * Called with the UST lock held.
*/
int __tracepoint_probe_unregister(const char *name, void *probe, void *data)
{
void *old;
*/
int __tracepoint_probe_unregister(const char *name, void *probe, void *data)
{
void *old;
+ pthread_mutex_lock(&tracepoint_mutex);
old = tracepoint_remove_probe(name, probe, data);
old = tracepoint_remove_probe(name, probe, data);
- if (IS_ERR(old))
- return PTR_ERR(old);
-
+ if (IS_ERR(old)) {
+ ret = PTR_ERR(old);
+ goto end;
+ }
tracepoint_update_probes(); /* may update entry */
release_probes(old);
tracepoint_update_probes(); /* may update entry */
release_probes(old);
+end:
+ pthread_mutex_unlock(&tracepoint_mutex);
+ return ret;
}
static void tracepoint_add_old_probes(void *old)
}
static void tracepoint_add_old_probes(void *old)
* @probe: probe handler
*
* caller must call tracepoint_probe_update_all()
* @probe: probe handler
*
* caller must call tracepoint_probe_update_all()
- * Called with the UST lock held.
*/
int tracepoint_probe_register_noupdate(const char *name, void *probe,
void *data)
{
void *old;
*/
int tracepoint_probe_register_noupdate(const char *name, void *probe,
void *data)
{
void *old;
+ pthread_mutex_lock(&tracepoint_mutex);
old = tracepoint_add_probe(name, probe, data);
if (IS_ERR(old)) {
old = tracepoint_add_probe(name, probe, data);
if (IS_ERR(old)) {
+ ret = PTR_ERR(old);
+ goto end;
}
tracepoint_add_old_probes(old);
}
tracepoint_add_old_probes(old);
+end:
+ pthread_mutex_unlock(&tracepoint_mutex);
+ return ret;
* @probe: probe function pointer
*
* caller must call tracepoint_probe_update_all()
* @probe: probe function pointer
*
* caller must call tracepoint_probe_update_all()
- * Called with the UST lock held.
+ * Called with the tracepoint mutex held.
*/
int tracepoint_probe_unregister_noupdate(const char *name, void *probe,
void *data)
{
void *old;
*/
int tracepoint_probe_unregister_noupdate(const char *name, void *probe,
void *data)
{
void *old;
+ pthread_mutex_lock(&tracepoint_mutex);
old = tracepoint_remove_probe(name, probe, data);
if (IS_ERR(old)) {
old = tracepoint_remove_probe(name, probe, data);
if (IS_ERR(old)) {
+ ret = PTR_ERR(old);
+ goto end;
}
tracepoint_add_old_probes(old);
}
tracepoint_add_old_probes(old);
+end:
+ pthread_mutex_unlock(&tracepoint_mutex);
+ return ret;
}
/**
* tracepoint_probe_update_all - update tracepoints
}
/**
* tracepoint_probe_update_all - update tracepoints
- * Called with the UST lock held.
*/
void tracepoint_probe_update_all(void)
{
CDS_LIST_HEAD(release_probes);
struct tp_probes *pos, *next;
*/
void tracepoint_probe_update_all(void)
{
CDS_LIST_HEAD(release_probes);
struct tp_probes *pos, *next;
+ pthread_mutex_lock(&tracepoint_mutex);
}
if (!cds_list_empty(&old_probes))
cds_list_replace_init(&old_probes, &release_probes);
}
if (!cds_list_empty(&old_probes))
cds_list_replace_init(&old_probes, &release_probes);
synchronize_rcu();
free(pos);
}
synchronize_rcu();
free(pos);
}
+end:
+ pthread_mutex_unlock(&tracepoint_mutex);
}
void tracepoint_set_new_tracepoint_cb(void (*cb)(struct tracepoint *))
}
void tracepoint_set_new_tracepoint_cb(void (*cb)(struct tracepoint *))
pl->tracepoints_start = tracepoints_start;
pl->tracepoints_count = tracepoints_count;
pl->tracepoints_start = tracepoints_start;
pl->tracepoints_count = tracepoints_count;
+ pthread_mutex_lock(&tracepoint_mutex);
/*
* We sort the libs by struct lib pointer address.
*/
/*
* We sort the libs by struct lib pointer address.
*/
/* TODO: update just the loaded lib */
lib_update_tracepoints();
/* TODO: update just the loaded lib */
lib_update_tracepoints();
+ pthread_mutex_unlock(&tracepoint_mutex);
DBG("just registered a tracepoints section from %p and having %d tracepoints",
tracepoints_start, tracepoints_count);
DBG("just registered a tracepoints section from %p and having %d tracepoints",
tracepoints_start, tracepoints_count);
struct tracepoint_lib *lib;
int tracepoints_count;
struct tracepoint_lib *lib;
int tracepoints_count;
+ pthread_mutex_lock(&tracepoint_mutex);
cds_list_for_each_entry(lib, &libs, list) {
if (lib->tracepoints_start == tracepoints_start) {
struct tracepoint_lib *lib2free = lib;
cds_list_for_each_entry(lib, &libs, list) {
if (lib->tracepoints_start == tracepoints_start) {
struct tracepoint_lib *lib2free = lib;
DBG("just unregistered a tracepoints section from %p",
tracepoints_start);
end:
DBG("just unregistered a tracepoints section from %p",
tracepoints_start);
end:
+ pthread_mutex_unlock(&tracepoint_mutex);