src.ctf.fs: make ctf_fs_component::trace a unique_ptr
[babeltrace.git] / src / plugins / ctf / fs-src / fs.cpp
index 86b4c45ba636b19dc33abc8d9bf8f1425a1d6760..21e61e3783948f93536507e0f8bca68ef95a06f3 100644 (file)
 
 #include <babeltrace2/babeltrace.h>
 
-#define BT_COMP_LOG_SELF_COMP self_comp
-#define BT_LOG_OUTPUT_LEVEL   ((enum bt_log_level) log_level)
-#define BT_LOG_TAG            "PLUGIN/SRC.CTF.FS"
-#include "logging/comp-logging.h"
-
 #include "common/assert.h"
 #include "common/common.h"
 #include "common/uuid.h"
 
 #include "plugins/common/param-validation/param-validation.h"
 
-#include "../common/metadata/ctf-meta-configure-ir-trace.hpp"
-#include "../common/msg-iter/msg-iter.hpp"
+#include "../common/src/metadata/tsdl/ctf-meta-configure-ir-trace.hpp"
+#include "../common/src/msg-iter/msg-iter.hpp"
 #include "data-stream-file.hpp"
 #include "file.hpp"
 #include "fs.hpp"
 #include "metadata.hpp"
-#include "plugins/ctf/common/metadata/ctf-meta.hpp"
 #include "query.hpp"
 
 struct tracer_info
@@ -54,7 +48,7 @@ static void ctf_fs_msg_iter_data_destroy(struct ctf_fs_msg_iter_data *msg_iter_d
         ctf_fs_ds_group_medops_data_destroy(msg_iter_data->msg_iter_medops_data);
     }
 
-    g_free(msg_iter_data);
+    delete msg_iter_data;
 }
 
 static bt_message_iterator_class_next_method_status
@@ -62,7 +56,6 @@ ctf_fs_iterator_next_one(struct ctf_fs_msg_iter_data *msg_iter_data, const bt_me
 {
     bt_message_iterator_class_next_method_status status;
     enum ctf_msg_iter_status msg_iter_status;
-    bt_logging_level log_level = msg_iter_data->log_level;
 
     msg_iter_status = ctf_msg_iter_get_next_message(msg_iter_data->msg_iter, out_msg);
 
@@ -85,14 +78,14 @@ ctf_fs_iterator_next_one(struct ctf_fs_msg_iter_data *msg_iter_data, const bt_me
         bt_common_abort();
 
     case CTF_MSG_ITER_STATUS_ERROR:
-        BT_MSG_ITER_LOGE_APPEND_CAUSE(msg_iter_data->self_msg_iter,
-                                      "Failed to get next message from CTF message iterator.");
+        BT_CPPLOGE_APPEND_CAUSE_SPEC(msg_iter_data->logger,
+                                     "Failed to get next message from CTF message iterator.");
         status = BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_ERROR;
         break;
 
     case CTF_MSG_ITER_STATUS_MEMORY_ERROR:
-        BT_MSG_ITER_LOGE_APPEND_CAUSE(msg_iter_data->self_msg_iter,
-                                      "Failed to get next message from CTF message iterator.");
+        BT_CPPLOGE_APPEND_CAUSE_SPEC(msg_iter_data->logger,
+                                     "Failed to get next message from CTF message iterator.");
         status = BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_MEMORY_ERROR;
         break;
 
@@ -107,73 +100,86 @@ bt_message_iterator_class_next_method_status
 ctf_fs_iterator_next(bt_self_message_iterator *iterator, bt_message_array_const msgs,
                      uint64_t capacity, uint64_t *count)
 {
-    bt_message_iterator_class_next_method_status status;
-    struct ctf_fs_msg_iter_data *msg_iter_data =
-        (struct ctf_fs_msg_iter_data *) bt_self_message_iterator_get_data(iterator);
-    uint64_t i = 0;
+    try {
+        bt_message_iterator_class_next_method_status status;
+        struct ctf_fs_msg_iter_data *msg_iter_data =
+            (struct ctf_fs_msg_iter_data *) bt_self_message_iterator_get_data(iterator);
+        uint64_t i = 0;
 
-    if (G_UNLIKELY(msg_iter_data->next_saved_error)) {
-        /*
+        if (G_UNLIKELY(msg_iter_data->next_saved_error)) {
+            /*
          * Last time we were called, we hit an error but had some
          * messages to deliver, so we stashed the error here.  Return
          * it now.
          */
-        BT_CURRENT_THREAD_MOVE_ERROR_AND_RESET(msg_iter_data->next_saved_error);
-        status = msg_iter_data->next_saved_status;
-        goto end;
-    }
-
-    do {
-        status = ctf_fs_iterator_next_one(msg_iter_data, &msgs[i]);
-        if (status == BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_OK) {
-            i++;
+            BT_CURRENT_THREAD_MOVE_ERROR_AND_RESET(msg_iter_data->next_saved_error);
+            status = msg_iter_data->next_saved_status;
+            goto end;
         }
-    } while (i < capacity && status == BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_OK);
 
-    if (i > 0) {
-        /*
+        do {
+            status = ctf_fs_iterator_next_one(msg_iter_data, &msgs[i]);
+            if (status == BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_OK) {
+                i++;
+            }
+        } while (i < capacity && status == BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_OK);
+
+        if (i > 0) {
+            /*
          * Even if ctf_fs_iterator_next_one() returned something
          * else than BT_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_OK, we
          * accumulated message objects in the output
          * message array, so we need to return
          * BT_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_OK so that they are
-         * transfered to downstream. This other status occurs
+         * transferred to downstream. This other status occurs
          * again the next time muxer_msg_iter_do_next() is
          * called, possibly without any accumulated
          * message, in which case we'll return it.
          */
-        if (status < 0) {
-            /*
+            if (status < 0) {
+                /*
              * Save this error for the next _next call.  Assume that
              * this component always appends error causes when
              * returning an error status code, which will cause the
              * current thread error to be non-NULL.
              */
-            msg_iter_data->next_saved_error = bt_current_thread_take_error();
-            BT_ASSERT(msg_iter_data->next_saved_error);
-            msg_iter_data->next_saved_status = status;
-        }
+                msg_iter_data->next_saved_error = bt_current_thread_take_error();
+                BT_ASSERT(msg_iter_data->next_saved_error);
+                msg_iter_data->next_saved_status = status;
+            }
 
-        *count = i;
-        status = BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_OK;
-    }
+            *count = i;
+            status = BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_OK;
+        }
 
 end:
-    return status;
+        return status;
+        return status;
+    } catch (const std::bad_alloc&) {
+        return BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_MEMORY_ERROR;
+    } catch (const bt2::Error&) {
+        return BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_ERROR;
+    }
 }
 
 bt_message_iterator_class_seek_beginning_method_status
 ctf_fs_iterator_seek_beginning(bt_self_message_iterator *it)
 {
-    struct ctf_fs_msg_iter_data *msg_iter_data =
-        (struct ctf_fs_msg_iter_data *) bt_self_message_iterator_get_data(it);
+    try {
+        struct ctf_fs_msg_iter_data *msg_iter_data =
+            (struct ctf_fs_msg_iter_data *) bt_self_message_iterator_get_data(it);
 
-    BT_ASSERT(msg_iter_data);
+        BT_ASSERT(msg_iter_data);
 
-    ctf_msg_iter_reset(msg_iter_data->msg_iter);
-    ctf_fs_ds_group_medops_data_reset(msg_iter_data->msg_iter_medops_data);
+        ctf_msg_iter_reset(msg_iter_data->msg_iter);
+        ctf_fs_ds_group_medops_data_reset(msg_iter_data->msg_iter_medops_data);
 
-    return BT_MESSAGE_ITERATOR_CLASS_SEEK_BEGINNING_METHOD_STATUS_OK;
+        return BT_MESSAGE_ITERATOR_CLASS_SEEK_BEGINNING_METHOD_STATUS_OK;
+    } catch (const std::bad_alloc&) {
+        return BT_MESSAGE_ITERATOR_CLASS_SEEK_BEGINNING_METHOD_STATUS_MEMORY_ERROR;
+    } catch (const bt2::Error&) {
+        return BT_MESSAGE_ITERATOR_CLASS_SEEK_BEGINNING_METHOD_STATUS_ERROR;
+    }
 }
 
 void ctf_fs_iterator_finalize(bt_self_message_iterator *it)
@@ -204,70 +210,68 @@ ctf_fs_iterator_init(bt_self_message_iterator *self_msg_iter,
                      bt_self_message_iterator_configuration *config,
                      bt_self_component_port_output *self_port)
 {
-    struct ctf_fs_port_data *port_data;
-    struct ctf_fs_msg_iter_data *msg_iter_data = NULL;
-    bt_message_iterator_class_initialize_method_status status;
-    bt_logging_level log_level;
-    enum ctf_msg_iter_medium_status medium_status;
-    bt_self_component *self_comp = bt_self_message_iterator_borrow_component(self_msg_iter);
-
-    port_data = (struct ctf_fs_port_data *) bt_self_component_port_get_data(
-        bt_self_component_port_output_as_self_component_port(self_port));
-    BT_ASSERT(port_data);
-    log_level = port_data->ctf_fs->log_level;
-    msg_iter_data = g_new0(struct ctf_fs_msg_iter_data, 1);
-    if (!msg_iter_data) {
-        status = BT_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_MEMORY_ERROR;
-        goto error;
-    }
-
-    msg_iter_data->log_level = log_level;
-    msg_iter_data->self_comp = self_comp;
-    msg_iter_data->self_msg_iter = self_msg_iter;
-    msg_iter_data->ds_file_group = port_data->ds_file_group;
-
-    medium_status =
-        ctf_fs_ds_group_medops_data_create(msg_iter_data->ds_file_group, self_msg_iter, log_level,
-                                           &msg_iter_data->msg_iter_medops_data);
-    BT_ASSERT(medium_status == CTF_MSG_ITER_MEDIUM_STATUS_OK ||
-              medium_status == CTF_MSG_ITER_MEDIUM_STATUS_ERROR ||
-              medium_status == CTF_MSG_ITER_MEDIUM_STATUS_MEMORY_ERROR);
-    if (medium_status != CTF_MSG_ITER_MEDIUM_STATUS_OK) {
-        BT_MSG_ITER_LOGE_APPEND_CAUSE(self_msg_iter, "Failed to create ctf_fs_ds_group_medops");
-        status = ctf_msg_iter_medium_status_to_msg_iter_initialize_status(medium_status);
-        goto error;
-    }
+    try {
+        struct ctf_fs_port_data *port_data;
+        bt_message_iterator_class_initialize_method_status status;
+        enum ctf_msg_iter_medium_status medium_status;
+
+        port_data = (struct ctf_fs_port_data *) bt_self_component_port_get_data(
+            bt_self_component_port_output_as_self_component_port(self_port));
+        BT_ASSERT(port_data);
+
+        ctf_fs_msg_iter_data *msg_iter_data = new ctf_fs_msg_iter_data {self_msg_iter};
+        msg_iter_data->ds_file_group = port_data->ds_file_group;
+
+        medium_status = ctf_fs_ds_group_medops_data_create(msg_iter_data->ds_file_group,
+                                                           self_msg_iter, msg_iter_data->logger,
+                                                           &msg_iter_data->msg_iter_medops_data);
+        BT_ASSERT(medium_status == CTF_MSG_ITER_MEDIUM_STATUS_OK ||
+                  medium_status == CTF_MSG_ITER_MEDIUM_STATUS_ERROR ||
+                  medium_status == CTF_MSG_ITER_MEDIUM_STATUS_MEMORY_ERROR);
+        if (medium_status != CTF_MSG_ITER_MEDIUM_STATUS_OK) {
+            BT_CPPLOGE_APPEND_CAUSE_SPEC(msg_iter_data->logger,
+                                         "Failed to create ctf_fs_ds_group_medops");
+            status = ctf_msg_iter_medium_status_to_msg_iter_initialize_status(medium_status);
+            goto error;
+        }
 
-    msg_iter_data->msg_iter = ctf_msg_iter_create(
-        msg_iter_data->ds_file_group->ctf_fs_trace->metadata->tc,
-        bt_common_get_page_size(msg_iter_data->log_level) * 8, ctf_fs_ds_group_medops,
-        msg_iter_data->msg_iter_medops_data, msg_iter_data->log_level, self_comp, self_msg_iter);
-    if (!msg_iter_data->msg_iter) {
-        BT_COMP_LOGE_APPEND_CAUSE(self_comp, "Cannot create a CTF message iterator.");
-        status = BT_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_MEMORY_ERROR;
-        goto error;
-    }
+        msg_iter_data->msg_iter = ctf_msg_iter_create(
+            msg_iter_data->ds_file_group->ctf_fs_trace->metadata->tc,
+            bt_common_get_page_size(static_cast<int>(msg_iter_data->logger.level())) * 8,
+            ctf_fs_ds_group_medops, msg_iter_data->msg_iter_medops_data, self_msg_iter,
+            msg_iter_data->logger);
+        if (!msg_iter_data->msg_iter) {
+            BT_CPPLOGE_APPEND_CAUSE_SPEC(msg_iter_data->logger,
+                                         "Cannot create a CTF message iterator.");
+            status = BT_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_MEMORY_ERROR;
+            goto error;
+        }
 
-    /*
+        /*
      * This iterator can seek forward if its stream class has a default
      * clock class.
      */
-    if (msg_iter_data->ds_file_group->sc->default_clock_class) {
-        bt_self_message_iterator_configuration_set_can_seek_forward(config, true);
-    }
+        if (msg_iter_data->ds_file_group->sc->default_clock_class) {
+            bt_self_message_iterator_configuration_set_can_seek_forward(config, true);
+        }
 
-    bt_self_message_iterator_set_data(self_msg_iter, msg_iter_data);
-    msg_iter_data = NULL;
+        bt_self_message_iterator_set_data(self_msg_iter, msg_iter_data);
+        msg_iter_data = NULL;
 
-    status = BT_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_OK;
-    goto end;
+        status = BT_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_OK;
+        goto end;
 
 error:
-    bt_self_message_iterator_set_data(self_msg_iter, NULL);
+        bt_self_message_iterator_set_data(self_msg_iter, NULL);
 
 end:
-    ctf_fs_msg_iter_data_destroy(msg_iter_data);
-    return status;
+        ctf_fs_msg_iter_data_destroy(msg_iter_data);
+        return status;
+    } catch (const std::bad_alloc&) {
+        return BT_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_MEMORY_ERROR;
+    } catch (const bt2::Error&) {
+        return BT_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_ERROR;
+    }
 }
 
 static void ctf_fs_trace_destroy(struct ctf_fs_trace *ctf_fs_trace)
@@ -288,10 +292,15 @@ static void ctf_fs_trace_destroy(struct ctf_fs_trace *ctf_fs_trace)
 
     if (ctf_fs_trace->metadata) {
         ctf_fs_metadata_fini(ctf_fs_trace->metadata);
-        g_free(ctf_fs_trace->metadata);
+        delete ctf_fs_trace->metadata;
     }
 
-    g_free(ctf_fs_trace);
+    delete ctf_fs_trace;
+}
+
+void ctf_fs_trace_deleter::operator()(ctf_fs_trace * const trace) noexcept
+{
+    ctf_fs_trace_destroy(trace);
 }
 
 void ctf_fs_destroy(struct ctf_fs_component *ctf_fs)
@@ -300,13 +309,16 @@ void ctf_fs_destroy(struct ctf_fs_component *ctf_fs)
         return;
     }
 
-    ctf_fs_trace_destroy(ctf_fs->trace);
-
     if (ctf_fs->port_data) {
         g_ptr_array_free(ctf_fs->port_data, TRUE);
     }
 
-    g_free(ctf_fs);
+    delete ctf_fs;
+}
+
+void ctf_fs_component_deleter::operator()(ctf_fs_component *comp)
+{
+    ctf_fs_destroy(comp);
 }
 
 static void port_data_destroy(struct ctf_fs_port_data *port_data)
@@ -315,7 +327,7 @@ static void port_data_destroy(struct ctf_fs_port_data *port_data)
         return;
     }
 
-    g_free(port_data);
+    delete port_data;
 }
 
 static void port_data_destroy_notifier(void *data)
@@ -329,16 +341,10 @@ static void ctf_fs_trace_destroy_notifier(void *data)
     ctf_fs_trace_destroy(trace);
 }
 
-struct ctf_fs_component *ctf_fs_component_create(bt_logging_level log_level)
+ctf_fs_component::UP ctf_fs_component_create(const bt2c::Logger& parentLogger)
 {
-    struct ctf_fs_component *ctf_fs;
-
-    ctf_fs = g_new0(struct ctf_fs_component, 1);
-    if (!ctf_fs) {
-        goto error;
-    }
+    ctf_fs_component::UP ctf_fs {new ctf_fs_component {parentLogger}};
 
-    ctf_fs->log_level = log_level;
     ctf_fs->port_data = g_ptr_array_new_with_free_func(port_data_destroy_notifier);
     if (!ctf_fs->port_data) {
         goto error;
@@ -347,8 +353,7 @@ struct ctf_fs_component *ctf_fs_component_create(bt_logging_level log_level)
     goto end;
 
 error:
-    ctf_fs_destroy(ctf_fs);
-    ctf_fs = NULL;
+    ctf_fs.reset();
 
 end:
     return ctf_fs;
@@ -360,7 +365,7 @@ void ctf_fs_finalize(bt_self_component_source *component)
         bt_self_component_source_as_self_component(component)));
 }
 
-gchar *ctf_fs_make_port_name(struct ctf_fs_ds_file_group *ds_file_group)
+bt2c::GCharUP ctf_fs_make_port_name(struct ctf_fs_ds_file_group *ds_file_group)
 {
     GString *name = g_string_new(NULL);
 
@@ -401,7 +406,7 @@ gchar *ctf_fs_make_port_name(struct ctf_fs_ds_file_group *ds_file_group)
         g_string_append_printf(name, " | %s", ds_file_info->path->str);
     }
 
-    return g_string_free(name, FALSE);
+    return bt2c::GCharUP {g_string_free(name, FALSE)};
 }
 
 static int create_one_port_for_trace(struct ctf_fs_component *ctf_fs,
@@ -410,26 +415,19 @@ static int create_one_port_for_trace(struct ctf_fs_component *ctf_fs,
 {
     int ret = 0;
     struct ctf_fs_port_data *port_data = NULL;
-    gchar *port_name;
-    bt_logging_level log_level = ctf_fs->log_level;
-    bt_self_component *self_comp = bt_self_component_source_as_self_component(self_comp_src);
 
-    port_name = ctf_fs_make_port_name(ds_file_group);
+    bt2c::GCharUP port_name = ctf_fs_make_port_name(ds_file_group);
     if (!port_name) {
         goto error;
     }
 
-    BT_COMP_LOGI("Creating one port named `%s`", port_name);
+    BT_CPPLOGI_SPEC(ctf_fs->logger, "Creating one port named `{}`", port_name.get());
 
     /* Create output port for this file */
-    port_data = g_new0(struct ctf_fs_port_data, 1);
-    if (!port_data) {
-        goto error;
-    }
-
+    port_data = new ctf_fs_port_data;
     port_data->ctf_fs = ctf_fs;
     port_data->ds_file_group = ds_file_group;
-    ret = bt_self_component_source_add_output_port(self_comp_src, port_name, port_data, NULL);
+    ret = bt_self_component_source_add_output_port(self_comp_src, port_name.get(), port_data, NULL);
     if (ret) {
         goto error;
     }
@@ -442,8 +440,6 @@ error:
     ret = -1;
 
 end:
-    g_free(port_name);
-
     port_data_destroy(port_data);
     return ret;
 }
@@ -454,8 +450,6 @@ static int create_ports_for_trace(struct ctf_fs_component *ctf_fs,
 {
     int ret = 0;
     size_t i;
-    bt_logging_level log_level = ctf_fs_trace->log_level;
-    bt_self_component *self_comp = bt_self_component_source_as_self_component(self_comp_src);
 
     /* Create one output port for each stream file group */
     for (i = 0; i < ctf_fs_trace->ds_file_groups->len; i++) {
@@ -464,7 +458,7 @@ static int create_ports_for_trace(struct ctf_fs_component *ctf_fs,
 
         ret = create_one_port_for_trace(ctf_fs, ds_file_group, self_comp_src);
         if (ret) {
-            BT_COMP_LOGE_APPEND_CAUSE(self_comp, "Cannot create output port.");
+            BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs->logger, "Cannot create output port.");
             goto end;
         }
     }
@@ -483,18 +477,12 @@ static void ctf_fs_ds_file_info_destroy(struct ctf_fs_ds_file_info *ds_file_info
         g_string_free(ds_file_info->path, TRUE);
     }
 
-    g_free(ds_file_info);
+    delete ds_file_info;
 }
 
 static struct ctf_fs_ds_file_info *ctf_fs_ds_file_info_create(const char *path, int64_t begin_ns)
 {
-    struct ctf_fs_ds_file_info *ds_file_info;
-
-    ds_file_info = g_new0(struct ctf_fs_ds_file_info, 1);
-    if (!ds_file_info) {
-        goto end;
-    }
-
+    ctf_fs_ds_file_info *ds_file_info = new ctf_fs_ds_file_info;
     ds_file_info->path = g_string_new(path);
     if (!ds_file_info->path) {
         ctf_fs_ds_file_info_destroy(ds_file_info);
@@ -521,7 +509,7 @@ static void ctf_fs_ds_file_group_destroy(struct ctf_fs_ds_file_group *ds_file_gr
     ctf_fs_ds_index_destroy(ds_file_group->index);
 
     bt_stream_put_ref(ds_file_group->stream);
-    g_free(ds_file_group);
+    delete ds_file_group;
 }
 
 static struct ctf_fs_ds_file_group *ctf_fs_ds_file_group_create(struct ctf_fs_trace *ctf_fs_trace,
@@ -529,13 +517,7 @@ static struct ctf_fs_ds_file_group *ctf_fs_ds_file_group_create(struct ctf_fs_tr
                                                                 uint64_t stream_instance_id,
                                                                 struct ctf_fs_ds_index *index)
 {
-    struct ctf_fs_ds_file_group *ds_file_group;
-
-    ds_file_group = g_new0(struct ctf_fs_ds_file_group, 1);
-    if (!ds_file_group) {
-        goto error;
-    }
-
+    ctf_fs_ds_file_group *ds_file_group = new ctf_fs_ds_file_group;
     ds_file_group->ds_file_infos =
         g_ptr_array_new_with_free_func((GDestroyNotify) ctf_fs_ds_file_info_destroy);
     if (!ds_file_group->ds_file_infos) {
@@ -603,7 +585,7 @@ static void ds_file_group_insert_ds_file_info_sorted(struct ctf_fs_ds_file_group
 static bool ds_index_entries_equal(const struct ctf_fs_ds_index_entry *left,
                                    const struct ctf_fs_ds_index_entry *right)
 {
-    if (left->packet_size != right->packet_size) {
+    if (left->packetSize != right->packetSize) {
         return false;
     }
 
@@ -656,7 +638,7 @@ static void ds_index_insert_ds_index_entry_sorted(struct ctf_fs_ds_index *index,
     if (i == index->entries->len || !ds_index_entries_equal(entry, other_entry)) {
         array_insert(index->entries, entry, i);
     } else {
-        g_free(entry);
+        delete entry;
     }
 }
 
@@ -691,25 +673,23 @@ static int add_ds_file_to_ds_file_group(struct ctf_fs_trace *ctf_fs_trace, const
     struct ctf_msg_iter *msg_iter = NULL;
     struct ctf_stream_class *sc = NULL;
     struct ctf_msg_iter_packet_properties props;
-    bt_logging_level log_level = ctf_fs_trace->log_level;
-    bt_self_component *self_comp = ctf_fs_trace->self_comp;
-    bt_self_component_class *self_comp_class = ctf_fs_trace->self_comp_class;
 
     /*
      * Create a temporary ds_file to read some properties about the data
      * stream file.
      */
-    ds_file = ctf_fs_ds_file_create(ctf_fs_trace, NULL, path, log_level);
+    ds_file = ctf_fs_ds_file_create(ctf_fs_trace, NULL, path, ctf_fs_trace->logger);
     if (!ds_file) {
         goto error;
     }
 
     /* Create a temporary iterator to read the ds_file. */
-    msg_iter =
-        ctf_msg_iter_create(ctf_fs_trace->metadata->tc, bt_common_get_page_size(log_level) * 8,
-                            ctf_fs_ds_file_medops, ds_file, log_level, self_comp, NULL);
+    msg_iter = ctf_msg_iter_create(
+        ctf_fs_trace->metadata->tc,
+        bt_common_get_page_size(static_cast<int>(ctf_fs_trace->logger.level())) * 8,
+        ctf_fs_ds_file_medops, ds_file, nullptr, ctf_fs_trace->logger);
     if (!msg_iter) {
-        BT_COMP_LOGE_STR("Cannot create a CTF message iterator.");
+        BT_CPPLOGE_STR_SPEC(ctf_fs_trace->logger, "Cannot create a CTF message iterator.");
         goto error;
     }
 
@@ -717,9 +697,9 @@ static int add_ds_file_to_ds_file_group(struct ctf_fs_trace *ctf_fs_trace, const
 
     ret = ctf_msg_iter_get_packet_properties(msg_iter, &props);
     if (ret) {
-        BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(
-            self_comp, self_comp_class,
-            "Cannot get stream file's first packet's header and context fields (`%s`).", path);
+        BT_CPPLOGE_APPEND_CAUSE_SPEC(
+            ctf_fs_trace->logger,
+            "Cannot get stream file's first packet's header and context fields (`{}`).", path);
         goto error;
     }
 
@@ -734,9 +714,9 @@ static int add_ds_file_to_ds_file_group(struct ctf_fs_trace *ctf_fs_trace, const
             sc->default_clock_class->offset_seconds, sc->default_clock_class->offset_cycles,
             &begin_ns);
         if (ret) {
-            BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(
-                self_comp, self_comp_class,
-                "Cannot convert clock cycles to nanoseconds from origin (`%s`).", path);
+            BT_CPPLOGE_APPEND_CAUSE_SPEC(
+                ctf_fs_trace->logger,
+                "Cannot convert clock cycles to nanoseconds from origin (`{}`).", path);
             goto error;
         }
     }
@@ -748,9 +728,8 @@ static int add_ds_file_to_ds_file_group(struct ctf_fs_trace *ctf_fs_trace, const
 
     index = ctf_fs_ds_file_build_index(ds_file, ds_file_info, msg_iter);
     if (!index) {
-        BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp, self_comp_class,
-                                                "Failed to index CTF stream file \'%s\'",
-                                                ds_file->file->path->str);
+        BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs_trace->logger, "Failed to index CTF stream file \'{}\'",
+                                     ds_file->file->path->str);
         goto error;
     }
 
@@ -844,16 +823,13 @@ static int create_ds_file_groups(struct ctf_fs_trace *ctf_fs_trace)
     const char *basename;
     GError *error = NULL;
     GDir *dir = NULL;
-    bt_logging_level log_level = ctf_fs_trace->log_level;
-    bt_self_component *self_comp = ctf_fs_trace->self_comp;
-    bt_self_component_class *self_comp_class = ctf_fs_trace->self_comp_class;
 
     /* Check each file in the path directory, except specific ones */
     dir = g_dir_open(ctf_fs_trace->path->str, 0, &error);
     if (!dir) {
-        BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(
-            self_comp, self_comp_class, "Cannot open directory `%s`: %s (code %d)",
-            ctf_fs_trace->path->str, error->message, error->code);
+        BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs_trace->logger,
+                                     "Cannot open directory `{}`: {} (code {})",
+                                     ctf_fs_trace->path->str, error->message, error->code);
         goto error;
     }
 
@@ -862,23 +838,25 @@ static int create_ds_file_groups(struct ctf_fs_trace *ctf_fs_trace)
 
         if (strcmp(basename, CTF_FS_METADATA_FILENAME) == 0) {
             /* Ignore the metadata stream. */
-            BT_COMP_LOGI("Ignoring metadata file `%s" G_DIR_SEPARATOR_S "%s`",
-                         ctf_fs_trace->path->str, basename);
+            BT_CPPLOGI_SPEC(ctf_fs_trace->logger,
+                            "Ignoring metadata file `{}" G_DIR_SEPARATOR_S "{}`",
+                            ctf_fs_trace->path->str, basename);
             continue;
         }
 
         if (basename[0] == '.') {
-            BT_COMP_LOGI("Ignoring hidden file `%s" G_DIR_SEPARATOR_S "%s`",
-                         ctf_fs_trace->path->str, basename);
+            BT_CPPLOGI_SPEC(ctf_fs_trace->logger,
+                            "Ignoring hidden file `{}" G_DIR_SEPARATOR_S "{}`",
+                            ctf_fs_trace->path->str, basename);
             continue;
         }
 
         /* Create the file. */
-        file = ctf_fs_file_create(log_level, self_comp);
+        file = ctf_fs_file_create(ctf_fs_trace->logger);
         if (!file) {
-            BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(
-                self_comp, self_comp_class,
-                "Cannot create stream file object for file `%s" G_DIR_SEPARATOR_S "%s`",
+            BT_CPPLOGE_APPEND_CAUSE_SPEC(
+                ctf_fs_trace->logger,
+                "Cannot create stream file object for file `{}" G_DIR_SEPARATOR_S "{}`",
                 ctf_fs_trace->path->str, basename);
             goto error;
         }
@@ -887,7 +865,8 @@ static int create_ds_file_groups(struct ctf_fs_trace *ctf_fs_trace)
         g_string_append_printf(file->path, "%s" G_DIR_SEPARATOR_S "%s", ctf_fs_trace->path->str,
                                basename);
         if (!g_file_test(file->path->str, G_FILE_TEST_IS_REGULAR)) {
-            BT_COMP_LOGI("Ignoring non-regular file `%s`", file->path->str);
+            BT_CPPLOGI_SPEC(ctf_fs_trace->logger, "Ignoring non-regular file `{}`",
+                            file->path->str);
             ctf_fs_file_destroy(file);
             file = NULL;
             continue;
@@ -895,23 +874,23 @@ static int create_ds_file_groups(struct ctf_fs_trace *ctf_fs_trace)
 
         ret = ctf_fs_file_open(file, "rb");
         if (ret) {
-            BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(
-                self_comp, self_comp_class, "Cannot open stream file `%s`", file->path->str);
+            BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs_trace->logger, "Cannot open stream file `{}`",
+                                         file->path->str);
             goto error;
         }
 
         if (file->size == 0) {
             /* Skip empty stream. */
-            BT_COMP_LOGI("Ignoring empty file `%s`", file->path->str);
+            BT_CPPLOGI_SPEC(ctf_fs_trace->logger, "Ignoring empty file `{}`", file->path->str);
             ctf_fs_file_destroy(file);
             continue;
         }
 
         ret = add_ds_file_to_ds_file_group(ctf_fs_trace, file->path->str);
         if (ret) {
-            BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(
-                self_comp, self_comp_class, "Cannot add stream file `%s` to stream file group",
-                file->path->str);
+            BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs_trace->logger,
+                                         "Cannot add stream file `{}` to stream file group",
+                                         file->path->str);
             ctf_fs_file_destroy(file);
             goto error;
         }
@@ -937,8 +916,7 @@ end:
     return ret;
 }
 
-static int set_trace_name(bt_trace *trace, const char *name_suffix, bt_logging_level log_level,
-                          bt_self_component *self_comp)
+static int set_trace_name(bt_trace *trace, const char *name_suffix, const bt2c::Logger& logger)
 {
     int ret = 0;
     const bt_value *val;
@@ -946,7 +924,7 @@ static int set_trace_name(bt_trace *trace, const char *name_suffix, bt_logging_l
 
     name = g_string_new(NULL);
     if (!name) {
-        BT_COMP_LOGE_STR("Failed to allocate a GString.");
+        BT_CPPLOGE_STR_SPEC(logger, "Failed to allocate a GString.");
         ret = -1;
         goto end;
     }
@@ -983,36 +961,20 @@ end:
     return ret;
 }
 
-static struct ctf_fs_trace *ctf_fs_trace_create(bt_self_component *self_comp,
-                                                bt_self_component_class *self_comp_class,
-                                                const char *path, const char *name,
-                                                struct ctf_fs_metadata_config *metadata_config,
-                                                bt_logging_level log_level)
+static struct ctf_fs_trace *ctf_fs_trace_create(const char *path, const char *name,
+                                                const ctf::src::ClkClsCfg& clkClsCfg,
+                                                bt_self_component *selfComp,
+                                                const bt2c::Logger& parentLogger)
 {
-    struct ctf_fs_trace *ctf_fs_trace;
     int ret;
 
-    /* Only one of them must be set. */
-    BT_ASSERT(!self_comp != !self_comp_class);
-
-    ctf_fs_trace = g_new0(struct ctf_fs_trace, 1);
-    if (!ctf_fs_trace) {
-        goto end;
-    }
-
-    ctf_fs_trace->log_level = log_level;
-    ctf_fs_trace->self_comp = self_comp;
-    ctf_fs_trace->self_comp_class = self_comp_class;
+    ctf_fs_trace *ctf_fs_trace = new struct ctf_fs_trace(parentLogger);
     ctf_fs_trace->path = g_string_new(path);
     if (!ctf_fs_trace->path) {
         goto error;
     }
 
-    ctf_fs_trace->metadata = g_new0(struct ctf_fs_metadata, 1);
-    if (!ctf_fs_trace->metadata) {
-        goto error;
-    }
-
+    ctf_fs_trace->metadata = new ctf_fs_metadata;
     ctf_fs_metadata_init(ctf_fs_trace->metadata);
     ctf_fs_trace->ds_file_groups =
         g_ptr_array_new_with_free_func((GDestroyNotify) ctf_fs_ds_file_group_destroy);
@@ -1020,7 +982,7 @@ static struct ctf_fs_trace *ctf_fs_trace_create(bt_self_component *self_comp,
         goto error;
     }
 
-    ret = ctf_fs_metadata_set_trace_class(self_comp, ctf_fs_trace, metadata_config);
+    ret = ctf_fs_metadata_set_trace_class(selfComp, ctf_fs_trace, clkClsCfg);
     if (ret) {
         goto error;
     }
@@ -1038,7 +1000,7 @@ static struct ctf_fs_trace *ctf_fs_trace_create(bt_self_component *self_comp,
             goto error;
         }
 
-        ret = set_trace_name(ctf_fs_trace->trace, name, log_level, self_comp);
+        ret = set_trace_name(ctf_fs_trace->trace, name, ctf_fs_trace->logger);
         if (ret) {
             goto error;
         }
@@ -1086,47 +1048,42 @@ end:
 static int ctf_fs_component_create_ctf_fs_trace_one_path(struct ctf_fs_component *ctf_fs,
                                                          const char *path_param,
                                                          const char *trace_name, GPtrArray *traces,
-                                                         bt_self_component *self_comp,
-                                                         bt_self_component_class *self_comp_class)
+                                                         bt_self_component *selfComp)
 {
     struct ctf_fs_trace *ctf_fs_trace;
     int ret;
     GString *norm_path;
-    bt_logging_level log_level = ctf_fs->log_level;
 
     norm_path = bt_common_normalize_path(path_param, NULL);
     if (!norm_path) {
-        BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp, self_comp_class,
-                                                "Failed to normalize path: `%s`.", path_param);
+        BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs->logger, "Failed to normalize path: `{}`.", path_param);
         goto error;
     }
 
     ret = path_is_ctf_trace(norm_path->str);
     if (ret < 0) {
-        BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp, self_comp_class,
-                                                "Failed to check if path is a CTF trace: path=%s",
-                                                norm_path->str);
+        BT_CPPLOGE_APPEND_CAUSE_SPEC(
+            ctf_fs->logger, "Failed to check if path is a CTF trace: path={}", norm_path->str);
         goto error;
     } else if (ret == 0) {
-        BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(
-            self_comp, self_comp_class,
-            "Path is not a CTF trace (does not contain a metadata file): `%s`.", norm_path->str);
+        BT_CPPLOGE_APPEND_CAUSE_SPEC(
+            ctf_fs->logger, "Path is not a CTF trace (does not contain a metadata file): `{}`.",
+            norm_path->str);
         goto error;
     }
 
     // FIXME: Remove or ifdef for __MINGW32__
     if (strcmp(norm_path->str, "/") == 0) {
-        BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp, self_comp_class,
-                                                "Opening a trace in `/` is not supported.");
+        BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs->logger, "Opening a trace in `/` is not supported.");
         ret = -1;
         goto end;
     }
 
-    ctf_fs_trace = ctf_fs_trace_create(self_comp, self_comp_class, norm_path->str, trace_name,
-                                       &ctf_fs->metadata_config, log_level);
+    ctf_fs_trace = ctf_fs_trace_create(norm_path->str, trace_name, ctf_fs->clkClsCfg, selfComp,
+                                       ctf_fs->logger);
     if (!ctf_fs_trace) {
-        BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp, self_comp_class,
-                                                "Cannot create trace for `%s`.", norm_path->str);
+        BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs->logger, "Cannot create trace for `{}`.",
+                                     norm_path->str);
         goto error;
     }
 
@@ -1258,7 +1215,7 @@ static int merge_matching_ctf_fs_ds_file_groups(struct ctf_fs_trace *dest_trace,
                                                            src_group->sc->id);
             BT_ASSERT(sc);
 
-            index = ctf_fs_ds_index_create(dest_trace->log_level, dest_trace->self_comp);
+            index = ctf_fs_ds_index_create(dest_trace->logger);
             if (!index) {
                 ret = -1;
                 goto end;
@@ -1367,17 +1324,15 @@ static int decode_clock_snapshot_after_event(struct ctf_fs_trace *ctf_fs_trace,
     enum ctf_msg_iter_status iter_status = CTF_MSG_ITER_STATUS_OK;
     struct ctf_fs_ds_file *ds_file = NULL;
     struct ctf_msg_iter *msg_iter = NULL;
-    bt_logging_level log_level = ctf_fs_trace->log_level;
-    bt_self_component *self_comp = ctf_fs_trace->self_comp;
     int ret = 0;
 
     BT_ASSERT(ctf_fs_trace);
     BT_ASSERT(index_entry);
     BT_ASSERT(index_entry->path);
 
-    ds_file = ctf_fs_ds_file_create(ctf_fs_trace, NULL, index_entry->path, log_level);
+    ds_file = ctf_fs_ds_file_create(ctf_fs_trace, NULL, index_entry->path, ctf_fs_trace->logger);
     if (!ds_file) {
-        BT_COMP_LOGE_APPEND_CAUSE(self_comp, "Failed to create a ctf_fs_ds_file");
+        BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs_trace->logger, "Failed to create a ctf_fs_ds_file");
         ret = -1;
         goto end;
     }
@@ -1385,9 +1340,10 @@ static int decode_clock_snapshot_after_event(struct ctf_fs_trace *ctf_fs_trace,
     BT_ASSERT(ctf_fs_trace->metadata);
     BT_ASSERT(ctf_fs_trace->metadata->tc);
 
-    msg_iter =
-        ctf_msg_iter_create(ctf_fs_trace->metadata->tc, bt_common_get_page_size(log_level) * 8,
-                            ctf_fs_ds_file_medops, ds_file, log_level, self_comp, NULL);
+    msg_iter = ctf_msg_iter_create(
+        ctf_fs_trace->metadata->tc,
+        bt_common_get_page_size(static_cast<int>(ctf_fs_trace->logger.level())) * 8,
+        ctf_fs_ds_file_medops, ds_file, NULL, ctf_fs_trace->logger);
     if (!msg_iter) {
         /* ctf_msg_iter_create() logs errors. */
         ret = -1;
@@ -1401,7 +1357,7 @@ static int decode_clock_snapshot_after_event(struct ctf_fs_trace *ctf_fs_trace,
     ctf_msg_iter_set_dry_run(msg_iter, true);
 
     /* Seek to the beginning of the target packet. */
-    iter_status = ctf_msg_iter_seek(msg_iter, index_entry->offset);
+    iter_status = ctf_msg_iter_seek(msg_iter, index_entry->offset.bytes());
     if (iter_status) {
         /* ctf_msg_iter_seek() logs errors. */
         ret = -1;
@@ -1433,7 +1389,8 @@ static int decode_clock_snapshot_after_event(struct ctf_fs_trace *ctf_fs_trace,
     ret = bt_util_clock_cycles_to_ns_from_origin(
         *cs, default_cc->frequency, default_cc->offset_seconds, default_cc->offset_cycles, ts_ns);
     if (ret) {
-        BT_COMP_LOGE_APPEND_CAUSE(self_comp, "Failed to convert clock snapshot to timestamp");
+        BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs_trace->logger,
+                                     "Failed to convert clock snapshot to timestamp");
         goto end;
     }
 
@@ -1489,7 +1446,6 @@ static int fix_index_lttng_event_after_packet_bug(struct ctf_fs_trace *trace)
     int ret = 0;
     guint ds_file_group_i;
     GPtrArray *ds_file_groups = trace->ds_file_groups;
-    bt_logging_level log_level = trace->log_level;
 
     for (ds_file_group_i = 0; ds_file_group_i < ds_file_groups->len; ds_file_group_i++) {
         guint entry_i;
@@ -1544,8 +1500,8 @@ static int fix_index_lttng_event_after_packet_bug(struct ctf_fs_trace *trace)
                                                  &last_entry->timestamp_end,
                                                  &last_entry->timestamp_end_ns);
         if (ret) {
-            BT_COMP_LOGE_APPEND_CAUSE(
-                trace->self_comp,
+            BT_CPPLOGE_APPEND_CAUSE_SPEC(
+                trace->logger,
                 "Failed to decode stream's last packet to get its last event's clock snapshot.");
             goto end;
         }
@@ -1574,7 +1530,6 @@ static int fix_index_barectf_event_before_packet_bug(struct ctf_fs_trace *trace)
     int ret = 0;
     guint ds_file_group_i;
     GPtrArray *ds_file_groups = trace->ds_file_groups;
-    bt_logging_level log_level = trace->log_level;
 
     for (ds_file_group_i = 0; ds_file_group_i < ds_file_groups->len; ds_file_group_i++) {
         guint entry_i;
@@ -1608,8 +1563,8 @@ static int fix_index_barectf_event_before_packet_bug(struct ctf_fs_trace *trace)
                                                       &curr_entry->timestamp_begin,
                                                       &curr_entry->timestamp_begin_ns);
             if (ret) {
-                BT_COMP_LOGE_APPEND_CAUSE(trace->self_comp,
-                                          "Failed to decode first event's clock snapshot");
+                BT_CPPLOGE_APPEND_CAUSE_SPEC(trace->logger,
+                                             "Failed to decode first event's clock snapshot");
                 goto end;
             }
 
@@ -1647,7 +1602,6 @@ static int fix_index_lttng_crash_quirk(struct ctf_fs_trace *trace)
     int ret = 0;
     guint ds_file_group_idx;
     GPtrArray *ds_file_groups = trace->ds_file_groups;
-    bt_logging_level log_level = trace->log_level;
 
     for (ds_file_group_idx = 0; ds_file_group_idx < ds_file_groups->len; ds_file_group_idx++) {
         guint entry_idx;
@@ -1681,8 +1635,8 @@ static int fix_index_lttng_crash_quirk(struct ctf_fs_trace *trace)
                                                      &last_entry->timestamp_end,
                                                      &last_entry->timestamp_end_ns);
             if (ret) {
-                BT_COMP_LOGE_APPEND_CAUSE(trace->self_comp,
-                                          "Failed to decode last event's clock snapshot");
+                BT_CPPLOGE_APPEND_CAUSE_SPEC(trace->logger,
+                                             "Failed to decode last event's clock snapshot");
                 goto end;
             }
         }
@@ -1854,15 +1808,12 @@ static bool is_tracer_affected_by_lttng_crash_quirk(struct tracer_info *curr_tra
  * Looks for trace produced by known buggy tracers and fix up the index
  * produced earlier.
  */
-static int fix_packet_index_tracer_bugs(struct ctf_fs_component *ctf_fs,
-                                        bt_self_component *self_comp,
-                                        bt_self_component_class *self_comp_class)
+static int fix_packet_index_tracer_bugs(ctf_fs_trace *trace)
 {
     int ret = 0;
     struct tracer_info current_tracer_info;
-    bt_logging_level log_level = ctf_fs->log_level;
 
-    ret = extract_tracer_info(ctf_fs->trace, &current_tracer_info);
+    ret = extract_tracer_info(trace, &current_tracer_info);
     if (ret) {
         /*
          * A trace may not have all the necessary environment
@@ -1872,42 +1823,47 @@ static int fix_packet_index_tracer_bugs(struct ctf_fs_component *ctf_fs,
          * an error.
          */
         ret = 0;
-        BT_LOGI_STR("Cannot extract tracer information necessary to compare with buggy versions.");
+        BT_CPPLOGI_STR_SPEC(
+            trace->logger,
+            "Cannot extract tracer information necessary to compare with buggy versions.");
         goto end;
-        ;
     }
 
     /* Check if the trace may be affected by old tracer bugs. */
     if (is_tracer_affected_by_lttng_event_after_packet_bug(&current_tracer_info)) {
-        BT_LOGI_STR("Trace may be affected by LTTng tracer packet timestamp bug. Fixing up.");
-        ret = fix_index_lttng_event_after_packet_bug(ctf_fs->trace);
+        BT_CPPLOGI_STR_SPEC(
+            trace->logger,
+            "Trace may be affected by LTTng tracer packet timestamp bug. Fixing up.");
+        ret = fix_index_lttng_event_after_packet_bug(trace);
         if (ret) {
-            BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp, self_comp_class,
-                                                    "Failed to fix LTTng event-after-packet bug.");
+            BT_CPPLOGE_APPEND_CAUSE_SPEC(trace->logger,
+                                         "Failed to fix LTTng event-after-packet bug.");
             goto end;
         }
-        ctf_fs->trace->metadata->tc->quirks.lttng_event_after_packet = true;
+        trace->metadata->tc->quirks.lttng_event_after_packet = true;
     }
 
     if (is_tracer_affected_by_barectf_event_before_packet_bug(&current_tracer_info)) {
-        BT_LOGI_STR("Trace may be affected by barectf tracer packet timestamp bug. Fixing up.");
-        ret = fix_index_barectf_event_before_packet_bug(ctf_fs->trace);
+        BT_CPPLOGI_STR_SPEC(
+            trace->logger,
+            "Trace may be affected by barectf tracer packet timestamp bug. Fixing up.");
+        ret = fix_index_barectf_event_before_packet_bug(trace);
         if (ret) {
-            BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(
-                self_comp, self_comp_class, "Failed to fix barectf event-before-packet bug.");
+            BT_CPPLOGE_APPEND_CAUSE_SPEC(trace->logger,
+                                         "Failed to fix barectf event-before-packet bug.");
             goto end;
         }
-        ctf_fs->trace->metadata->tc->quirks.barectf_event_before_packet = true;
+        trace->metadata->tc->quirks.barectf_event_before_packet = true;
     }
 
     if (is_tracer_affected_by_lttng_crash_quirk(&current_tracer_info)) {
-        ret = fix_index_lttng_crash_quirk(ctf_fs->trace);
+        ret = fix_index_lttng_crash_quirk(trace);
         if (ret) {
-            BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp, self_comp_class,
-                                                    "Failed to fix lttng-crash timestamp quirks.");
+            BT_CPPLOGE_APPEND_CAUSE_SPEC(trace->logger,
+                                         "Failed to fix lttng-crash timestamp quirks.");
             goto end;
         }
-        ctf_fs->trace->metadata->tc->quirks.lttng_crash = true;
+        trace->metadata->tc->quirks.lttng_crash = true;
     }
 
 end:
@@ -1941,12 +1897,10 @@ static gint compare_strings(gconstpointer p_a, gconstpointer p_b)
 int ctf_fs_component_create_ctf_fs_trace(struct ctf_fs_component *ctf_fs,
                                          const bt_value *paths_value,
                                          const bt_value *trace_name_value,
-                                         bt_self_component *self_comp,
-                                         bt_self_component_class *self_comp_class)
+                                         bt_self_component *selfComp)
 {
     int ret = 0;
     uint64_t i;
-    bt_logging_level log_level = ctf_fs->log_level;
     GPtrArray *paths = NULL;
     GPtrArray *traces;
     const char *trace_name;
@@ -1956,15 +1910,13 @@ int ctf_fs_component_create_ctf_fs_trace(struct ctf_fs_component *ctf_fs,
 
     traces = g_ptr_array_new_with_free_func(ctf_fs_trace_destroy_notifier);
     if (!traces) {
-        BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp, self_comp_class,
-                                                "Failed to allocate a GPtrArray.");
+        BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs->logger, "Failed to allocate a GPtrArray.");
         goto error;
     }
 
     paths = g_ptr_array_new_with_free_func(g_free);
     if (!paths) {
-        BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp, self_comp_class,
-                                                "Failed to allocate a GPtrArray.");
+        BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs->logger, "Failed to allocate a GPtrArray.");
         goto error;
     }
 
@@ -1981,8 +1933,7 @@ int ctf_fs_component_create_ctf_fs_trace(struct ctf_fs_component *ctf_fs,
 
         input_copy = g_strdup(input);
         if (!input_copy) {
-            BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp, self_comp_class,
-                                                    "Failed to copy a string.");
+            BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs->logger, "Failed to copy a string.");
             goto error;
         }
 
@@ -1996,7 +1947,7 @@ int ctf_fs_component_create_ctf_fs_trace(struct ctf_fs_component *ctf_fs,
         const char *path = (const char *) g_ptr_array_index(paths, i);
 
         ret = ctf_fs_component_create_ctf_fs_trace_one_path(ctf_fs, path, trace_name, traces,
-                                                            self_comp, self_comp_class);
+                                                            selfComp);
         if (ret) {
             goto end;
         }
@@ -2016,9 +1967,9 @@ int ctf_fs_component_create_ctf_fs_trace(struct ctf_fs_component *ctf_fs,
             const uint8_t *this_trace_uuid = this_trace->metadata->tc->uuid;
 
             if (!this_trace->metadata->tc->is_uuid_set) {
-                BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(
-                    self_comp, self_comp_class,
-                    "Multiple traces given, but a trace does not have a UUID: path=%s",
+                BT_CPPLOGE_APPEND_CAUSE_SPEC(
+                    ctf_fs->logger,
+                    "Multiple traces given, but a trace does not have a UUID: path={}",
                     this_trace->path->str);
                 goto error;
             }
@@ -2030,35 +1981,33 @@ int ctf_fs_component_create_ctf_fs_trace(struct ctf_fs_component *ctf_fs,
                 bt_uuid_to_str(first_trace_uuid, first_trace_uuid_str);
                 bt_uuid_to_str(this_trace_uuid, this_trace_uuid_str);
 
-                BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(
-                    self_comp, self_comp_class,
-                    "Multiple traces given, but UUIDs don't match: "
-                    "first-trace-uuid=%s, first-trace-path=%s, "
-                    "trace-uuid=%s, trace-path=%s",
-                    first_trace_uuid_str, first_trace->path->str, this_trace_uuid_str,
-                    this_trace->path->str);
+                BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs->logger,
+                                             "Multiple traces given, but UUIDs don't match: "
+                                             "first-trace-uuid={}, first-trace-path={}, "
+                                             "trace-uuid={}, trace-path={}",
+                                             first_trace_uuid_str, first_trace->path->str,
+                                             this_trace_uuid_str, this_trace->path->str);
                 goto error;
             }
         }
 
         ret = merge_ctf_fs_traces((struct ctf_fs_trace **) traces->pdata, traces->len, &trace);
         if (ret) {
-            BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp, self_comp_class,
-                                                    "Failed to merge traces with the same UUID.");
+            BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs->logger,
+                                         "Failed to merge traces with the same UUID.");
             goto error;
         }
 
-        ctf_fs->trace = trace;
+        ctf_fs->trace.reset(trace);
     } else {
         /* Just one trace, it may or may not have a UUID, both are fine. */
-        ctf_fs->trace = (ctf_fs_trace *) traces->pdata[0];
+        ctf_fs->trace.reset((ctf_fs_trace *) traces->pdata[0]);
         traces->pdata[0] = NULL;
     }
 
-    ret = fix_packet_index_tracer_bugs(ctf_fs, self_comp, self_comp_class);
+    ret = fix_packet_index_tracer_bugs(ctf_fs->trace.get());
     if (ret) {
-        BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp, self_comp_class,
-                                                "Failed to fix packet index tracer bugs.");
+        BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs->logger, "Failed to fix packet index tracer bugs.");
     }
 
     /*
@@ -2120,8 +2069,6 @@ static int create_streams_for_trace(struct ctf_fs_trace *ctf_fs_trace)
     int ret;
     GString *name = NULL;
     guint i;
-    bt_logging_level log_level = ctf_fs_trace->log_level;
-    bt_self_component *self_comp = ctf_fs_trace->self_comp;
 
     for (i = 0; i < ctf_fs_trace->ds_file_groups->len; i++) {
         ctf_fs_ds_file_group *ds_file_group =
@@ -2132,38 +2079,34 @@ static int create_streams_for_trace(struct ctf_fs_trace *ctf_fs_trace)
             goto error;
         }
 
-        if (ds_file_group->sc->ir_sc) {
-            BT_ASSERT(ctf_fs_trace->trace);
-
-            if (ds_file_group->stream_id == UINT64_C(-1)) {
-                /* No stream ID: use 0 */
-                ds_file_group->stream = bt_stream_create_with_id(
-                    ds_file_group->sc->ir_sc, ctf_fs_trace->trace, ctf_fs_trace->next_stream_id);
-                ctf_fs_trace->next_stream_id++;
-            } else {
-                /* Specific stream ID */
-                ds_file_group->stream =
-                    bt_stream_create_with_id(ds_file_group->sc->ir_sc, ctf_fs_trace->trace,
-                                             (uint64_t) ds_file_group->stream_id);
-            }
+        BT_ASSERT(ds_file_group->sc->ir_sc);
+        BT_ASSERT(ctf_fs_trace->trace);
+
+        if (ds_file_group->stream_id == UINT64_C(-1)) {
+            /* No stream ID: use 0 */
+            ds_file_group->stream = bt_stream_create_with_id(
+                ds_file_group->sc->ir_sc, ctf_fs_trace->trace, ctf_fs_trace->next_stream_id);
+            ctf_fs_trace->next_stream_id++;
         } else {
-            ds_file_group->stream = NULL;
+            /* Specific stream ID */
+            ds_file_group->stream = bt_stream_create_with_id(
+                ds_file_group->sc->ir_sc, ctf_fs_trace->trace, (uint64_t) ds_file_group->stream_id);
         }
 
         if (!ds_file_group->stream) {
-            BT_COMP_LOGE_APPEND_CAUSE(self_comp,
-                                      "Cannot create stream for DS file group: "
-                                      "addr=%p, stream-name=\"%s\"",
-                                      ds_file_group, name->str);
+            BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs_trace->logger,
+                                         "Cannot create stream for DS file group: "
+                                         "addr={}, stream-name=\"{}\"",
+                                         fmt::ptr(ds_file_group), name->str);
             goto error;
         }
 
         ret = bt_stream_set_name(ds_file_group->stream, name->str);
         if (ret) {
-            BT_COMP_LOGE_APPEND_CAUSE(self_comp,
-                                      "Cannot set stream's name: "
-                                      "addr=%p, stream-name=\"%s\"",
-                                      ds_file_group->stream, name->str);
+            BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs_trace->logger,
+                                         "Cannot set stream's name: "
+                                         "addr={}, stream-name=\"{}\"",
+                                         fmt::ptr(ds_file_group->stream), name->str);
             goto error;
         }
 
@@ -2203,18 +2146,16 @@ static bt_param_validation_map_value_entry_descr fs_params_entries_descr[] = {
     BT_PARAM_VALIDATION_MAP_VALUE_ENTRY_END};
 
 bool read_src_fs_parameters(const bt_value *params, const bt_value **inputs,
-                            const bt_value **trace_name, struct ctf_fs_component *ctf_fs,
-                            bt_self_component *self_comp, bt_self_component_class *self_comp_class)
+                            const bt_value **trace_name, struct ctf_fs_component *ctf_fs)
 {
     bool ret;
     const bt_value *value;
-    bt_logging_level log_level = ctf_fs->log_level;
     enum bt_param_validation_status validate_value_status;
     gchar *error = NULL;
 
     validate_value_status = bt_param_validation_validate(params, fs_params_entries_descr, &error);
     if (validate_value_status != BT_PARAM_VALIDATION_STATUS_OK) {
-        BT_COMP_OR_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp, self_comp_class, "%s", error);
+        BT_CPPLOGE_APPEND_CAUSE_SPEC(ctf_fs->logger, "{}", error);
         ret = false;
         goto end;
     }
@@ -2225,19 +2166,19 @@ bool read_src_fs_parameters(const bt_value *params, const bt_value **inputs,
     /* clock-class-offset-s parameter */
     value = bt_value_map_borrow_entry_value_const(params, "clock-class-offset-s");
     if (value) {
-        ctf_fs->metadata_config.clock_class_offset_s = bt_value_integer_signed_get(value);
+        ctf_fs->clkClsCfg.offsetSec = bt_value_integer_signed_get(value);
     }
 
     /* clock-class-offset-ns parameter */
     value = bt_value_map_borrow_entry_value_const(params, "clock-class-offset-ns");
     if (value) {
-        ctf_fs->metadata_config.clock_class_offset_ns = bt_value_integer_signed_get(value);
+        ctf_fs->clkClsCfg.offsetNanoSec = bt_value_integer_signed_get(value);
     }
 
     /* force-clock-class-origin-unix-epoch parameter */
     value = bt_value_map_borrow_entry_value_const(params, "force-clock-class-origin-unix-epoch");
     if (value) {
-        ctf_fs->metadata_config.force_clock_class_origin_unix_epoch = bt_value_bool_get(value);
+        ctf_fs->clkClsCfg.forceOriginIsUnixEpoch = bt_value_bool_get(value);
     }
 
     /* trace-name parameter */
@@ -2250,48 +2191,36 @@ end:
     return ret;
 }
 
-static struct ctf_fs_component *ctf_fs_create(const bt_value *params,
-                                              bt_self_component_source *self_comp_src)
+static ctf_fs_component::UP ctf_fs_create(const bt_value *params,
+                                          bt_self_component_source *self_comp_src)
 {
-    struct ctf_fs_component *ctf_fs = NULL;
     const bt_value *inputs_value;
     const bt_value *trace_name_value;
     bt_self_component *self_comp = bt_self_component_source_as_self_component(self_comp_src);
 
-    ctf_fs = ctf_fs_component_create(
-        bt_component_get_logging_level(bt_self_component_as_component(self_comp)));
+    ctf_fs_component::UP ctf_fs = ctf_fs_component_create(
+        bt2c::Logger {bt2::SelfSourceComponent {self_comp_src}, "PLUGIN/SRC.CTF.FS/COMP"});
     if (!ctf_fs) {
-        goto error;
+        return nullptr;
     }
 
-    if (!read_src_fs_parameters(params, &inputs_value, &trace_name_value, ctf_fs, self_comp,
-                                NULL)) {
-        goto error;
+    if (!read_src_fs_parameters(params, &inputs_value, &trace_name_value, ctf_fs.get())) {
+        return nullptr;
     }
 
-    bt_self_component_set_data(self_comp, ctf_fs);
-
-    if (ctf_fs_component_create_ctf_fs_trace(ctf_fs, inputs_value, trace_name_value, self_comp,
-                                             NULL)) {
-        goto error;
+    if (ctf_fs_component_create_ctf_fs_trace(ctf_fs.get(), inputs_value, trace_name_value,
+                                             self_comp)) {
+        return nullptr;
     }
 
-    if (create_streams_for_trace(ctf_fs->trace)) {
-        goto error;
+    if (create_streams_for_trace(ctf_fs->trace.get())) {
+        return nullptr;
     }
 
-    if (create_ports_for_trace(ctf_fs, ctf_fs->trace, self_comp_src)) {
-        goto error;
+    if (create_ports_for_trace(ctf_fs.get(), ctf_fs->trace.get(), self_comp_src)) {
+        return nullptr;
     }
 
-    goto end;
-
-error:
-    ctf_fs_destroy(ctf_fs);
-    ctf_fs = NULL;
-    bt_self_component_set_data(self_comp, NULL);
-
-end:
     return ctf_fs;
 }
 
@@ -2299,39 +2228,55 @@ bt_component_class_initialize_method_status ctf_fs_init(bt_self_component_source
                                                         bt_self_component_source_configuration *,
                                                         const bt_value *params, void *)
 {
-    struct ctf_fs_component *ctf_fs;
-    bt_component_class_initialize_method_status ret =
-        BT_COMPONENT_CLASS_INITIALIZE_METHOD_STATUS_OK;
+    try {
+        bt_component_class_initialize_method_status ret =
+            BT_COMPONENT_CLASS_INITIALIZE_METHOD_STATUS_OK;
 
-    ctf_fs = ctf_fs_create(params, self_comp_src);
-    if (!ctf_fs) {
-        ret = BT_COMPONENT_CLASS_INITIALIZE_METHOD_STATUS_ERROR;
-    }
+        ctf_fs_component::UP ctf_fs = ctf_fs_create(params, self_comp_src);
+        if (!ctf_fs) {
+            ret = BT_COMPONENT_CLASS_INITIALIZE_METHOD_STATUS_ERROR;
+        }
 
-    return ret;
+        bt_self_component_set_data(bt_self_component_source_as_self_component(self_comp_src),
+                                   ctf_fs.release());
+        return ret;
+    } catch (const std::bad_alloc&) {
+        return BT_COMPONENT_CLASS_INITIALIZE_METHOD_STATUS_MEMORY_ERROR;
+    } catch (const bt2::Error&) {
+        return BT_COMPONENT_CLASS_INITIALIZE_METHOD_STATUS_ERROR;
+    }
 }
 
-bt_component_class_query_method_status ctf_fs_query(bt_self_component_class_source *comp_class,
+bt_component_class_query_method_status ctf_fs_query(bt_self_component_class_source *comp_class_src,
                                                     bt_private_query_executor *priv_query_exec,
                                                     const char *object, const bt_value *params,
                                                     __attribute__((unused)) void *method_data,
                                                     const bt_value **result)
 {
-    bt_component_class_query_method_status status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_OK;
-    bt_logging_level log_level = bt_query_executor_get_logging_level(
-        bt_private_query_executor_as_query_executor_const(priv_query_exec));
-
-    if (strcmp(object, "metadata-info") == 0) {
-        status = metadata_info_query(comp_class, params, log_level, result);
-    } else if (strcmp(object, "babeltrace.trace-infos") == 0) {
-        status = trace_infos_query(comp_class, params, log_level, result);
-    } else if (!strcmp(object, "babeltrace.support-info")) {
-        status = support_info_query(comp_class, params, log_level, result);
-    } else {
-        BT_LOGE("Unknown query object `%s`", object);
-        status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_UNKNOWN_OBJECT;
-        goto end;
+    try {
+        bt2c::Logger logger {bt2::SelfComponentClass {comp_class_src},
+                             bt2::PrivateQueryExecutor {priv_query_exec},
+                             "PLUGIN/SRC.CTF.FS/QUERY"};
+        bt2::ConstMapValue paramsObj(params);
+        bt2::Value::Shared resultObj;
+
+        if (strcmp(object, "metadata-info") == 0) {
+            resultObj = metadata_info_query(paramsObj, logger);
+        } else if (strcmp(object, "babeltrace.trace-infos") == 0) {
+            resultObj = trace_infos_query(paramsObj, logger);
+        } else if (!strcmp(object, "babeltrace.support-info")) {
+            resultObj = support_info_query(paramsObj, logger);
+        } else {
+            BT_CPPLOGE_SPEC(logger, "Unknown query object `{}`", object);
+            return BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_UNKNOWN_OBJECT;
+        }
+
+        *result = resultObj.release().libObjPtr();
+
+        return BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_OK;
+    } catch (const std::bad_alloc&) {
+        return BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_MEMORY_ERROR;
+    } catch (const bt2::Error&) {
+        return BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_ERROR;
     }
-end:
-    return status;
 }
This page took 0.045386 seconds and 4 git commands to generate.