X-Git-Url: http://git.lttng.org/?a=blobdiff_plain;f=liblttng-ust%2Flttng-context-vtid.c;h=0a29021d4b104b3f4cb94deea5d72069dddc7145;hb=8306baea3da8b1f429e14218b1ef7413ee56ed19;hp=81508515ba56c3dee817c92f34b4c5513a3df444;hpb=9501d22f55b3a23f382f631c99b2325d1419b5da;p=lttng-ust.git

diff --git a/liblttng-ust/lttng-context-vtid.c b/liblttng-ust/lttng-context-vtid.c
index 81508515..0a29021d 100644
--- a/liblttng-ust/lttng-context-vtid.c
+++ b/liblttng-ust/lttng-context-vtid.c
@@ -20,19 +20,22 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
+#define _LGPL_SOURCE
+#include <stddef.h>
 #include <sys/types.h>
 #include <unistd.h>
 #include <lttng/ust-events.h>
 #include <lttng/ust-tracer.h>
 #include <lttng/ringbuffer-config.h>
 #include <lttng/ust-tid.h>
-#include "ltt-tracer-core.h"
+#include <urcu/tls-compat.h>
+#include "lttng-tracer-core.h"
 
 /*
  * We cache the result to ensure we don't trigger a system call for
  * each event.
  */
-static __thread pid_t cached_vtid;
+static DEFINE_URCU_TLS(pid_t, cached_vtid);
 
 /*
  * Upon fork or clone, the TID assigned to our thread is not the same as
@@ -41,11 +44,11 @@ static __thread pid_t cached_vtid;
  */
 void lttng_context_vtid_reset(void)
 {
-	cached_vtid = 0;
+	CMM_STORE_SHARED(URCU_TLS(cached_vtid), 0);
 }
 
 static
-size_t vtid_get_size(size_t offset)
+size_t vtid_get_size(struct lttng_ctx_field *field, size_t offset)
 {
 	size_t size = 0;
 
@@ -54,15 +57,35 @@ size_t vtid_get_size(size_t offset)
 	return size;
 }
 
+static inline
+pid_t wrapper_getvtid(void)
+{
+	pid_t vtid;
+
+	vtid = CMM_LOAD_SHARED(URCU_TLS(cached_vtid));
+	if (caa_unlikely(!vtid)) {
+		vtid = lttng_gettid();
+		CMM_STORE_SHARED(URCU_TLS(cached_vtid), vtid);
+	}
+	return vtid;
+}
+
 static
 void vtid_record(struct lttng_ctx_field *field,
 		 struct lttng_ust_lib_ring_buffer_ctx *ctx,
-		 struct ltt_channel *chan)
+		 struct lttng_channel *chan)
+{
+	pid_t vtid = wrapper_getvtid();
+
+	lib_ring_buffer_align_ctx(ctx, lttng_alignof(vtid));
+	chan->ops->event_write(ctx, &vtid, sizeof(vtid));
+}
+
+static
+void vtid_get_value(struct lttng_ctx_field *field,
+		struct lttng_ctx_value *value)
 {
-	if (caa_unlikely(!cached_vtid))
-		cached_vtid = gettid();
-	lib_ring_buffer_align_ctx(ctx, lttng_alignof(cached_vtid));
-	chan->ops->event_write(ctx, &cached_vtid, sizeof(cached_vtid));
+	value->u.s64 = wrapper_getvtid();
 }
 
 int lttng_add_vtid_to_ctx(struct lttng_ctx **ctx)
@@ -86,6 +109,8 @@ int lttng_add_vtid_to_ctx(struct lttng_ctx **ctx)
 	field->event_field.type.u.basic.integer.encoding = lttng_encode_none;
 	field->get_size = vtid_get_size;
 	field->record = vtid_record;
+	field->get_value = vtid_get_value;
+	lttng_context_update(*ctx);
 	return 0;
 }
 
@@ -94,5 +119,5 @@ int lttng_add_vtid_to_ctx(struct lttng_ctx **ctx)
  */
 void lttng_fixup_vtid_tls(void)
 {
-	asm volatile ("" : : "m" (cached_vtid));
+	asm volatile ("" : : "m" (URCU_TLS(cached_vtid)));
 }