X-Git-Url: http://git.efficios.com/?a=blobdiff_plain;f=src%2Flib%2Ftrace-ir%2Ftrace-class.c;h=593f390efac2d4dfd5bf9adcbf467af6db5eccb8;hb=43c59509042845f8d42c3e99ec74d45fa2dc0908;hp=620e8f6daeb3375b0bc5bd3e006d3e1f82c5f3a9;hpb=0de083a28d550811823a5f61d9ed1989d785f48d;p=babeltrace.git diff --git a/src/lib/trace-ir/trace-class.c b/src/lib/trace-ir/trace-class.c index 620e8f6d..593f390e 100644 --- a/src/lib/trace-ir/trace-class.c +++ b/src/lib/trace-ir/trace-class.c @@ -21,18 +21,17 @@ * SOFTWARE. */ -#define BT_LOG_TAG "LIB/TRACE" +#define BT_LOG_TAG "LIB/TRACE-CLASS" #include "lib/logging.h" #include "lib/assert-pre.h" +#include "lib/assert-post.h" #include -#include #include #include "ctf-writer/functor.h" #include "ctf-writer/clock.h" #include "compat/compiler.h" #include -#include #include "lib/value.h" #include #include "compat/endian.h" @@ -41,6 +40,7 @@ #include #include #include +#include #include #include "clock-class.h" @@ -53,6 +53,7 @@ #include "stream.h" #include "trace.h" #include "utils.h" +#include "lib/value.h" #include "lib/func-status.h" struct bt_trace_class_destruction_listener_elem { @@ -69,12 +70,16 @@ void destroy_trace_class(struct bt_object *obj) struct bt_trace_class *tc = (void *) obj; BT_LIB_LOGD("Destroying trace class object: %!+T", tc); + BT_OBJECT_PUT_REF_AND_RESET(tc->user_attributes); + /* * Call destruction listener functions so that everything else * still exists in the trace class. */ if (tc->destruction_listeners) { uint64_t i; + const struct bt_error *saved_error; + BT_LIB_LOGD("Calling trace class destruction listener(s): %!+T", tc); /* @@ -89,6 +94,8 @@ void destroy_trace_class(struct bt_object *obj) */ tc->base.ref_count++; + saved_error = bt_current_thread_take_error(); + /* Call all the trace class destruction listeners */ for (i = 0; i < tc->destruction_listeners->len; i++) { struct bt_trace_class_destruction_listener_elem elem = @@ -97,16 +104,21 @@ void destroy_trace_class(struct bt_object *obj) if (elem.func) { elem.func(tc, elem.data); + BT_ASSERT_POST_NO_ERROR(); } /* * The destruction listener should not have kept a * reference to the trace class. */ - BT_ASSERT_PRE(tc->base.ref_count == 1, "Destruction listener kept a reference to the trace class being destroyed: %![tc-]+T", tc); + BT_ASSERT_POST(tc->base.ref_count == 1, "Destruction listener kept a reference to the trace class being destroyed: %![tc-]+T", tc); } g_array_free(tc->destruction_listeners, TRUE); tc->destruction_listeners = NULL; + + if (saved_error) { + BT_CURRENT_THREAD_MOVE_ERROR_AND_RESET(saved_error); + } } if (tc->stream_classes) { @@ -122,6 +134,7 @@ struct bt_trace_class *bt_trace_class_create(bt_self_component *self_comp) { struct bt_trace_class *tc = NULL; + BT_ASSERT_PRE_NO_ERROR(); BT_ASSERT_PRE_NON_NULL(self_comp, "Self component"); BT_LOGD_STR("Creating default trace class object."); tc = g_new0(struct bt_trace_class, 1); @@ -131,6 +144,12 @@ struct bt_trace_class *bt_trace_class_create(bt_self_component *self_comp) } bt_object_init_shared_with_parent(&tc->base, destroy_trace_class); + tc->user_attributes = bt_value_map_create(); + if (!tc->user_attributes) { + BT_LIB_LOGE_APPEND_CAUSE( + "Failed to create a map value object."); + goto error; + } tc->stream_classes = g_ptr_array_new_with_free_func( (GDestroyNotify) bt_object_try_spec_release); @@ -160,7 +179,7 @@ end: enum bt_trace_class_add_listener_status bt_trace_class_add_destruction_listener( const struct bt_trace_class *_tc, bt_trace_class_destruction_listener_func listener, - void *data, uint64_t *listener_id) + void *data, bt_listener_id *listener_id) { struct bt_trace_class *tc = (void *) _tc; uint64_t i; @@ -169,6 +188,7 @@ enum bt_trace_class_add_listener_status bt_trace_class_add_destruction_listener( .data = data, }; + BT_ASSERT_PRE_NO_ERROR(); BT_ASSERT_PRE_NON_NULL(tc, "Trace class"); BT_ASSERT_PRE_NON_NULL(listener, "Listener"); @@ -204,15 +224,16 @@ bool has_listener_id(const struct bt_trace_class *tc, uint64_t listener_id) BT_ASSERT(listener_id < tc->destruction_listeners->len); return (&g_array_index(tc->destruction_listeners, struct bt_trace_class_destruction_listener_elem, - listener_id))->func != NULL; + listener_id))->func; } enum bt_trace_class_remove_listener_status bt_trace_class_remove_destruction_listener( - const struct bt_trace_class *_tc, uint64_t listener_id) + const struct bt_trace_class *_tc, bt_listener_id listener_id) { struct bt_trace_class *tc = (void *) _tc; struct bt_trace_class_destruction_listener_elem *elem; + BT_ASSERT_PRE_NO_ERROR(); BT_ASSERT_PRE_NON_NULL(tc, "Trace class"); BT_ASSERT_PRE(has_listener_id(tc, listener_id), "Trace class has no such trace class destruction listener ID: " @@ -305,6 +326,33 @@ void bt_trace_class_set_assigns_automatic_stream_class_id(struct bt_trace_class "assignment property: %!+T", tc); } +const struct bt_value *bt_trace_class_borrow_user_attributes_const( + const struct bt_trace_class *trace_class) +{ + BT_ASSERT_PRE_DEV_NON_NULL(trace_class, "Trace class"); + return trace_class->user_attributes; +} + +struct bt_value *bt_trace_class_borrow_user_attributes( + struct bt_trace_class *trace_class) +{ + return (void *) bt_trace_class_borrow_user_attributes_const( + (void *) trace_class); +} + +void bt_trace_class_set_user_attributes(struct bt_trace_class *trace_class, + const struct bt_value *user_attributes) +{ + BT_ASSERT_PRE_NON_NULL(trace_class, "Trace class"); + BT_ASSERT_PRE_NON_NULL(user_attributes, "User attributes"); + BT_ASSERT_PRE(user_attributes->type == BT_VALUE_TYPE_MAP, + "User attributes object is not a map value object."); + BT_ASSERT_PRE_DEV_TRACE_CLASS_HOT(trace_class); + bt_object_put_ref_no_null_check(trace_class->user_attributes); + trace_class->user_attributes = (void *) user_attributes; + bt_object_get_ref_no_null_check(trace_class->user_attributes); +} + void bt_trace_class_get_ref(const struct bt_trace_class *trace_class) { bt_object_get_ref(trace_class);