Fix: pass private data to context callbacks

[lttng-ust.git] / liblttng-ust / event-notifier-notification.c

diff --git a/liblttng-ust/event-notifier-notification.c b/liblttng-ust/event-notifier-notification.c
index 199e615b15fe0e065d375b3a9991a0ba0c79fe0f..b3e2c94f1bd7a2d7180ad64256bcb325ea393e85 100644 (file)
--- a/liblttng-ust/event-notifier-notification.c
+++ b/liblttng-ust/event-notifier-notification.c
@@ -8,9 +8,13 @@
 
 #include <assert.h>
 #include <errno.h>
+#include <limits.h>
+
 #include <lttng/ust-endian.h>
 #include <usterr-signal-safe.h>
+#include <urcu/rculist.h>
 
+#include "lttng-tracer-core.h"
 #include "ust-events-internal.h"
 #include "../libmsgpack/msgpack.h"
 #include "lttng-bytecode.h"
@@ -60,7 +64,7 @@ void capture_enum(struct lttng_msgpack_writer *writer,
 
 static
 int64_t capture_sequence_element_signed(uint8_t *ptr,
-               struct lttng_ust_type_integer *integer_type)
+               const struct lttng_ust_type_integer *integer_type)
 {
        int64_t value;
        unsigned int size = integer_type->size;
@@ -109,7 +113,7 @@ int64_t capture_sequence_element_signed(uint8_t *ptr,
 
 static
 uint64_t capture_sequence_element_unsigned(uint8_t *ptr,
-               struct lttng_ust_type_integer *integer_type)
+               const struct lttng_ust_type_integer *integer_type)
 {
        uint64_t value;
        unsigned int size = integer_type->size;
@@ -160,8 +164,8 @@ static
 void capture_sequence(struct lttng_msgpack_writer *writer,
                struct lttng_interpreter_output *output)
 {
-       struct lttng_ust_type_integer *integer_type;
-       struct lttng_ust_type_common *nested_type;
+       const struct lttng_ust_type_integer *integer_type;
+       const struct lttng_ust_type_common *nested_type;
        uint8_t *ptr;
        bool signedness;
        int i;
@@ -357,7 +361,8 @@ void notification_send(struct lttng_event_notifier_notification *notif,
 
 void lttng_event_notifier_notification_send(
                struct lttng_ust_event_notifier *event_notifier,
-               const char *stack_data)
+               const char *stack_data,
+               struct lttng_ust_notification_ctx *notif_ctx)
 {
        /*
         * This function is called from the probe, we must do dynamic
@@ -367,7 +372,7 @@ void lttng_event_notifier_notification_send(
 
        notification_init(&notif, event_notifier);
 
-       if (caa_unlikely(!cds_list_empty(&event_notifier->capture_bytecode_runtime_head))) {
+       if (caa_unlikely(notif_ctx->eval_capture)) {
                struct lttng_ust_bytecode_runtime *capture_bc_runtime;
 
                /*
@@ -376,12 +381,12 @@ void lttng_event_notifier_notification_send(
                 * `output` parameter to the capture buffer. If the interpreter
                 * fails, append an empty capture to the buffer.
                 */
-               cds_list_for_each_entry(capture_bc_runtime,
-                               &event_notifier->capture_bytecode_runtime_head, node) {
+               cds_list_for_each_entry_rcu(capture_bc_runtime,
+                               &event_notifier->priv->capture_bytecode_runtime_head, node) {
                        struct lttng_interpreter_output output;
 
-                       if (capture_bc_runtime->interpreter_funcs.capture(capture_bc_runtime,
-                                       stack_data, &output) & LTTNG_INTERPRETER_RECORD_FLAG)
+                       if (capture_bc_runtime->interpreter_func(capture_bc_runtime,
+                                       stack_data, &output) == LTTNG_UST_BYTECODE_INTERPRETER_OK)
                                notification_append_capture(&notif, &output);
                        else
                                notification_append_empty_capture(&notif);