flt-utils.muxer: use BT_COMP_LOGE_APPEND_CAUSE
[babeltrace.git] / src / plugins / utils / muxer / muxer.c
index 0c08f8289a298091542117c1aa33f137d4e2e292..f340f38d227a77592b7fe30474a0be0a0b73a928 100644 (file)
@@ -1,23 +1,7 @@
 /*
- * Copyright 2017 Philippe Proulx <pproulx@efficios.com>
- *
- * Permission is hereby granted, free of charge, to any person obtaining a copy
- * of this software and associated documentation files (the "Software"), to deal
- * in the Software without restriction, including without limitation the rights
- * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
- * copies of the Software, and to permit persons to whom the Software is
- * furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice shall be included in
- * all copies or substantial portions of the Software.
+ * SPDX-License-Identifier: MIT
  *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
- * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
- * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
- * SOFTWARE.
+ * Copyright 2017 Philippe Proulx <pproulx@efficios.com>
  */
 
 #define BT_COMP_LOG_SELF_COMP (muxer_comp->self_comp)
@@ -56,7 +40,7 @@ struct muxer_upstream_msg_iter {
        struct muxer_comp *muxer_comp;
 
        /* Owned by this, NULL if ended */
-       bt_self_component_port_input_message_iterator *msg_iter;
+       bt_message_iterator *msg_iter;
 
        /* Contains `const bt_message *`, owned by this */
        GQueue *msgs;
@@ -143,7 +127,7 @@ void destroy_muxer_upstream_msg_iter(
                muxer_upstream_msg_iter,
                muxer_upstream_msg_iter->msg_iter,
                muxer_upstream_msg_iter->msgs->length);
-       bt_self_component_port_input_message_iterator_put_ref(
+       bt_message_iterator_put_ref(
                muxer_upstream_msg_iter->msg_iter);
 
        if (muxer_upstream_msg_iter->msgs) {
@@ -156,7 +140,7 @@ void destroy_muxer_upstream_msg_iter(
 
 static
 int muxer_msg_iter_add_upstream_msg_iter(struct muxer_msg_iter *muxer_msg_iter,
-               bt_self_component_port_input_message_iterator *self_msg_iter)
+               bt_message_iterator *self_msg_iter)
 {
        int ret = 0;
        struct muxer_upstream_msg_iter *muxer_upstream_msg_iter =
@@ -164,16 +148,18 @@ int muxer_msg_iter_add_upstream_msg_iter(struct muxer_msg_iter *muxer_msg_iter,
        struct muxer_comp *muxer_comp = muxer_msg_iter->muxer_comp;
 
        if (!muxer_upstream_msg_iter) {
-               BT_COMP_LOGE_STR("Failed to allocate one muxer's upstream message iterator wrapper.");
+               BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                       "Failed to allocate one muxer's upstream message iterator wrapper.");
                goto error;
        }
 
        muxer_upstream_msg_iter->muxer_comp = muxer_comp;
        muxer_upstream_msg_iter->msg_iter = self_msg_iter;
-       bt_self_component_port_input_message_iterator_get_ref(muxer_upstream_msg_iter->msg_iter);
+       bt_message_iterator_get_ref(muxer_upstream_msg_iter->msg_iter);
        muxer_upstream_msg_iter->msgs = g_queue_new();
        if (!muxer_upstream_msg_iter->msgs) {
-               BT_COMP_LOGE_STR("Failed to allocate a GQueue.");
+               BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                       "Failed to allocate a GQueue.");
                goto error;
        }
 
@@ -187,7 +173,7 @@ int muxer_msg_iter_add_upstream_msg_iter(struct muxer_msg_iter *muxer_msg_iter,
        goto end;
 
 error:
-       g_free(muxer_upstream_msg_iter);
+       destroy_muxer_upstream_msg_iter(muxer_upstream_msg_iter);
        ret = -1;
 
 end:
@@ -207,7 +193,7 @@ bt_self_component_add_port_status add_available_input_port(
        BT_ASSERT(muxer_comp);
        port_name = g_string_new("in");
        if (!port_name) {
-               BT_COMP_LOGE_STR("Failed to allocate a GString.");
+               BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp, "Failed to allocate a GString.");
                status = BT_SELF_COMPONENT_ADD_PORT_STATUS_MEMORY_ERROR;
                goto end;
        }
@@ -216,7 +202,8 @@ bt_self_component_add_port_status add_available_input_port(
        status = bt_self_component_filter_add_input_port(
                self_comp, port_name->str, NULL, NULL);
        if (status != BT_SELF_COMPONENT_ADD_PORT_STATUS_OK) {
-               BT_COMP_LOGE("Cannot add input port to muxer component: "
+               BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                       "Cannot add input port to muxer component: "
                        "port-name=\"%s\", comp-addr=%p, status=%s",
                        port_name->str, self_comp,
                        bt_common_func_status_string(status));
@@ -281,9 +268,15 @@ bt_component_class_initialize_method_status muxer_init(
                "comp-addr=%p, params-addr=%p", self_comp, params);
 
        if (!muxer_comp) {
+               /*
+                * Don't use BT_COMP_LOGE_APPEND_CAUSE, as `muxer_comp` is not
+                * initialized.
+                */
                BT_COMP_LOG_CUR_LVL(BT_LOG_ERROR, log_level, self_comp,
                        "Failed to allocate one muxer component.");
-               status = BT_COMPONENT_CLASS_INITIALIZE_METHOD_STATUS_MEMORY_ERROR;
+               BT_CURRENT_THREAD_ERROR_APPEND_CAUSE_FROM_COMPONENT(self_comp,
+                       "Failed to allocate one muxer component.");
+                       status = BT_COMPONENT_CLASS_INITIALIZE_METHOD_STATUS_MEMORY_ERROR;
                goto error;
        }
 
@@ -305,20 +298,20 @@ bt_component_class_initialize_method_status muxer_init(
        bt_self_component_set_data(self_comp, muxer_comp);
        add_port_status = add_available_input_port(self_comp_flt);
        if (add_port_status != BT_SELF_COMPONENT_ADD_PORT_STATUS_OK) {
-               BT_COMP_LOGE("Cannot ensure that at least one muxer component's input port is available: "
+               BT_COMP_LOGE_APPEND_CAUSE(self_comp,
+                       "Cannot ensure that at least one muxer component's input port is available: "
                        "muxer-comp-addr=%p, status=%s",
-                       muxer_comp,
-                       bt_common_func_status_string(add_port_status));
+                       muxer_comp, bt_common_func_status_string(add_port_status));
                status = (int) add_port_status;
                goto error;
        }
 
        add_port_status = create_output_port(self_comp_flt);
        if (add_port_status != BT_SELF_COMPONENT_ADD_PORT_STATUS_OK) {
-               BT_COMP_LOGE("Cannot create muxer component's output port: "
+               BT_COMP_LOGE_APPEND_CAUSE(self_comp,
+                       "Cannot create muxer component's output port: "
                        "muxer-comp-addr=%p, status=%s",
-                       muxer_comp,
-                       bt_common_func_status_string(add_port_status));
+                       muxer_comp, bt_common_func_status_string(add_port_status));
                status = (int) add_port_status;
                goto error;
        }
@@ -351,16 +344,16 @@ void muxer_finalize(bt_self_component_filter *self_comp)
 }
 
 static
-bt_self_component_port_input_message_iterator_create_from_message_iterator_status
+bt_message_iterator_create_from_message_iterator_status
 create_msg_iter_on_input_port(struct muxer_comp *muxer_comp,
                struct muxer_msg_iter *muxer_msg_iter,
                bt_self_component_port_input *self_port,
-               bt_self_component_port_input_message_iterator **msg_iter)
+               bt_message_iterator **msg_iter)
 {
        const bt_port *port = bt_self_component_port_as_port(
                bt_self_component_port_input_as_self_component_port(
                        self_port));
-       bt_self_component_port_input_message_iterator_create_from_message_iterator_status
+       bt_message_iterator_create_from_message_iterator_status
                status;
 
        BT_ASSERT(port);
@@ -369,10 +362,11 @@ create_msg_iter_on_input_port(struct muxer_comp *muxer_comp,
        // TODO: Advance the iterator to >= the time of the latest
        //       returned message by the muxer message
        //       iterator which creates it.
-       status = bt_self_component_port_input_message_iterator_create_from_message_iterator(
+       status = bt_message_iterator_create_from_message_iterator(
                muxer_msg_iter->self_msg_iter, self_port, msg_iter);
-       if (status != BT_SELF_COMPONENT_PORT_INPUT_MESSAGE_ITERATOR_CREATE_FROM_MESSAGE_ITERATOR_STATUS_OK) {
-               BT_COMP_LOGE("Cannot create upstream message iterator on input port: "
+       if (status != BT_MESSAGE_ITERATOR_CREATE_FROM_MESSAGE_ITERATOR_STATUS_OK) {
+               BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                       "Cannot create upstream message iterator on input port: "
                        "port-addr=%p, port-name=\"%s\"",
                        port, bt_port_get_name(port));
                goto end;
@@ -402,7 +396,7 @@ bt_message_iterator_class_next_method_status muxer_upstream_msg_iter_next(
                "muxer-upstream-msg-iter-wrap-addr=%p, msg-iter-addr=%p",
                muxer_upstream_msg_iter,
                muxer_upstream_msg_iter->msg_iter);
-       input_port_iter_status = bt_self_component_port_input_message_iterator_next(
+       input_port_iter_status = bt_message_iterator_next(
                muxer_upstream_msg_iter->msg_iter, &msgs, &count);
        BT_COMP_LOGD("Upstream message iterator's \"next\" method returned: "
                "status=%s",
@@ -580,7 +574,7 @@ int get_msg_ts_ns(struct muxer_comp *muxer_comp,
 
                break;
        case BT_MESSAGE_TYPE_MESSAGE_ITERATOR_INACTIVITY:
-               clock_snapshot = bt_message_message_iterator_inactivity_borrow_default_clock_snapshot_const(
+               clock_snapshot = bt_message_message_iterator_inactivity_borrow_clock_snapshot_const(
                        msg);
                break;
        default:
@@ -592,7 +586,8 @@ int get_msg_ts_ns(struct muxer_comp *muxer_comp,
 
        ret = bt_clock_snapshot_get_ns_from_origin(clock_snapshot, ts_ns);
        if (ret) {
-               BT_COMP_LOGE("Cannot get nanoseconds from Epoch of clock snapshot: "
+               BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                       "Cannot get nanoseconds from Epoch of clock snapshot: "
                        "clock-snapshot-addr=%p", clock_snapshot);
                goto error;
        }
@@ -667,7 +662,8 @@ int validate_clock_class(struct muxer_msg_iter *muxer_msg_iter,
        switch (muxer_msg_iter->clock_class_expectation) {
        case MUXER_MSG_ITER_CLOCK_CLASS_EXPECTATION_ABSOLUTE:
                if (!bt_clock_class_origin_is_unix_epoch(clock_class)) {
-                       BT_COMP_LOGE("Expecting an absolute clock class, "
+                       BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                               "Expecting an absolute clock class, "
                                "but got a non-absolute one: "
                                "clock-class-addr=%p, clock-class-name=\"%s\"",
                                clock_class, cc_name);
@@ -676,7 +672,8 @@ int validate_clock_class(struct muxer_msg_iter *muxer_msg_iter,
                break;
        case MUXER_MSG_ITER_CLOCK_CLASS_EXPECTATION_NOT_ABS_NO_UUID:
                if (bt_clock_class_origin_is_unix_epoch(clock_class)) {
-                       BT_COMP_LOGE("Expecting a non-absolute clock class with no UUID, "
+                       BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                               "Expecting a non-absolute clock class with no UUID, "
                                "but got an absolute one: "
                                "clock-class-addr=%p, clock-class-name=\"%s\"",
                                clock_class, cc_name);
@@ -684,7 +681,8 @@ int validate_clock_class(struct muxer_msg_iter *muxer_msg_iter,
                }
 
                if (cc_uuid) {
-                       BT_COMP_LOGE("Expecting a non-absolute clock class with no UUID, "
+                       BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                               "Expecting a non-absolute clock class with no UUID, "
                                "but got one with a UUID: "
                                "clock-class-addr=%p, clock-class-name=\"%s\", "
                                "uuid=\"" BT_UUID_FMT "\"",
@@ -694,7 +692,8 @@ int validate_clock_class(struct muxer_msg_iter *muxer_msg_iter,
                break;
        case MUXER_MSG_ITER_CLOCK_CLASS_EXPECTATION_NOT_ABS_SPEC_UUID:
                if (bt_clock_class_origin_is_unix_epoch(clock_class)) {
-                       BT_COMP_LOGE("Expecting a non-absolute clock class with a specific UUID, "
+                       BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                               "Expecting a non-absolute clock class with a specific UUID, "
                                "but got an absolute one: "
                                "clock-class-addr=%p, clock-class-name=\"%s\"",
                                clock_class, cc_name);
@@ -702,7 +701,8 @@ int validate_clock_class(struct muxer_msg_iter *muxer_msg_iter,
                }
 
                if (!cc_uuid) {
-                       BT_COMP_LOGE("Expecting a non-absolute clock class with a specific UUID, "
+                       BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                               "Expecting a non-absolute clock class with a specific UUID, "
                                "but got one with no UUID: "
                                "clock-class-addr=%p, clock-class-name=\"%s\"",
                                clock_class, cc_name);
@@ -710,7 +710,8 @@ int validate_clock_class(struct muxer_msg_iter *muxer_msg_iter,
                }
 
                if (bt_uuid_compare(muxer_msg_iter->expected_clock_class_uuid, cc_uuid) != 0) {
-                       BT_COMP_LOGE("Expecting a non-absolute clock class with a specific UUID, "
+                       BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                               "Expecting a non-absolute clock class with a specific UUID, "
                                "but got one with different UUID: "
                                "clock-class-addr=%p, clock-class-name=\"%s\", "
                                "expected-uuid=\"" BT_UUID_FMT "\", "
@@ -722,7 +723,8 @@ int validate_clock_class(struct muxer_msg_iter *muxer_msg_iter,
                }
                break;
        case MUXER_MSG_ITER_CLOCK_CLASS_EXPECTATION_NONE:
-               BT_COMP_LOGE("Expecting no clock class, but got one: "
+               BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                       "Expecting no clock class, but got one: "
                        "clock-class-addr=%p, clock-class-name=\"%s\"",
                        clock_class, cc_name);
                goto error;
@@ -761,7 +763,8 @@ int validate_new_stream_clock_class(struct muxer_msg_iter *muxer_msg_iter,
                                MUXER_MSG_ITER_CLOCK_CLASS_EXPECTATION_NONE;
                } else if (muxer_msg_iter->clock_class_expectation !=
                                MUXER_MSG_ITER_CLOCK_CLASS_EXPECTATION_NONE) {
-                       BT_COMP_LOGE("Expecting stream class without a default clock class: "
+                       BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                               "Expecting stream class without a default clock class: "
                                "stream-class-addr=%p, stream-class-name=\"%s\", "
                                "stream-class-id=%" PRIu64,
                                stream_class, bt_stream_class_get_name(stream_class),
@@ -852,7 +855,7 @@ muxer_msg_iter_youngest_upstream_msg_iter(
                                BT_MESSAGE_TYPE_MESSAGE_ITERATOR_INACTIVITY)) {
                        const bt_clock_snapshot *cs;
 
-                       cs = bt_message_message_iterator_inactivity_borrow_default_clock_snapshot_const(
+                       cs = bt_message_message_iterator_inactivity_borrow_clock_snapshot_const(
                                msg);
                        ret = validate_clock_class(muxer_msg_iter, muxer_comp,
                                bt_clock_snapshot_borrow_clock_class_const(cs));
@@ -1039,7 +1042,8 @@ bt_message_iterator_class_next_method_status muxer_msg_iter_do_next_one(
 {
        bt_message_iterator_class_next_method_status status;
        struct muxer_upstream_msg_iter *muxer_upstream_msg_iter = NULL;
-       int64_t next_return_ts;
+       /* Initialize to avoid -Wmaybe-uninitialized warning with gcc 4.8. */
+       int64_t next_return_ts = 0;
 
        status = validate_muxer_upstream_msg_iters(muxer_msg_iter);
        if (status != BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_OK) {
@@ -1216,12 +1220,12 @@ muxer_msg_iter_init_upstream_iterators(struct muxer_comp *muxer_comp,
        }
 
        for (i = 0; i < count; i++) {
-               bt_self_component_port_input_message_iterator *upstream_msg_iter;
+               bt_message_iterator *upstream_msg_iter;
                bt_self_component_port_input *self_port =
                        bt_self_component_filter_borrow_input_port_by_index(
                                muxer_comp->self_comp_flt, i);
                const bt_port *port;
-               bt_self_component_port_input_message_iterator_create_from_message_iterator_status
+               bt_message_iterator_create_from_message_iterator_status
                        msg_iter_status;
                int int_status;
 
@@ -1238,7 +1242,7 @@ muxer_msg_iter_init_upstream_iterators(struct muxer_comp *muxer_comp,
 
                msg_iter_status = create_msg_iter_on_input_port(muxer_comp,
                        muxer_msg_iter, self_port, &upstream_msg_iter);
-               if (msg_iter_status != BT_SELF_COMPONENT_PORT_INPUT_MESSAGE_ITERATOR_CREATE_FROM_MESSAGE_ITERATOR_STATUS_OK) {
+               if (msg_iter_status != BT_MESSAGE_ITERATOR_CREATE_FROM_MESSAGE_ITERATOR_STATUS_OK) {
                        /* create_msg_iter_on_input_port() logs errors */
                        status = (int) msg_iter_status;
                        goto end;
@@ -1246,7 +1250,7 @@ muxer_msg_iter_init_upstream_iterators(struct muxer_comp *muxer_comp,
 
                int_status = muxer_msg_iter_add_upstream_msg_iter(muxer_msg_iter,
                        upstream_msg_iter);
-               bt_self_component_port_input_message_iterator_put_ref(
+               bt_message_iterator_put_ref(
                        upstream_msg_iter);
                if (int_status) {
                        status = BT_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_ERROR;
@@ -1255,7 +1259,7 @@ muxer_msg_iter_init_upstream_iterators(struct muxer_comp *muxer_comp,
                }
 
                can_seek_forward = can_seek_forward &&
-                       bt_self_component_port_input_message_iterator_can_seek_forward(
+                       bt_message_iterator_can_seek_forward(
                                upstream_msg_iter);
        }
 
@@ -1276,12 +1280,13 @@ BT_HIDDEN
 bt_message_iterator_class_initialize_method_status muxer_msg_iter_init(
                bt_self_message_iterator *self_msg_iter,
                bt_self_message_iterator_configuration *config,
-               bt_self_component *self_comp,
                bt_self_component_port_output *port)
 {
        struct muxer_comp *muxer_comp = NULL;
        struct muxer_msg_iter *muxer_msg_iter = NULL;
        bt_message_iterator_class_initialize_method_status status;
+       bt_self_component *self_comp =
+               bt_self_message_iterator_borrow_component(self_msg_iter);
 
        muxer_comp = bt_self_component_get_data(self_comp);
        BT_ASSERT(muxer_comp);
@@ -1295,7 +1300,8 @@ bt_message_iterator_class_initialize_method_status muxer_msg_iter_init(
                 * creates a muxer message iterator while creating
                 * another muxer message iterator (same component).
                 */
-               BT_COMP_LOGE("Recursive initialization of muxer component's message iterator: "
+               BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                       "Recursive initialization of muxer component's message iterator: "
                        "comp-addr=%p, muxer-comp-addr=%p, msg-iter-addr=%p",
                        self_comp, muxer_comp, self_msg_iter);
                status = BT_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_ERROR;
@@ -1305,7 +1311,8 @@ bt_message_iterator_class_initialize_method_status muxer_msg_iter_init(
        muxer_comp->initializing_muxer_msg_iter = true;
        muxer_msg_iter = g_new0(struct muxer_msg_iter, 1);
        if (!muxer_msg_iter) {
-               BT_COMP_LOGE_STR("Failed to allocate one muxer component's message iterator.");
+               BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                       "Failed to allocate one muxer component's message iterator.");
                status = BT_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_MEMORY_ERROR;
                goto error;
        }
@@ -1317,7 +1324,7 @@ bt_message_iterator_class_initialize_method_status muxer_msg_iter_init(
                g_ptr_array_new_with_free_func(
                        (GDestroyNotify) destroy_muxer_upstream_msg_iter);
        if (!muxer_msg_iter->active_muxer_upstream_msg_iters) {
-               BT_COMP_LOGE_STR("Failed to allocate a GPtrArray.");
+               BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp, "Failed to allocate a GPtrArray.");
                status = BT_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_MEMORY_ERROR;
                goto error;
        }
@@ -1326,7 +1333,7 @@ bt_message_iterator_class_initialize_method_status muxer_msg_iter_init(
                g_ptr_array_new_with_free_func(
                        (GDestroyNotify) destroy_muxer_upstream_msg_iter);
        if (!muxer_msg_iter->ended_muxer_upstream_msg_iters) {
-               BT_COMP_LOGE_STR("Failed to allocate a GPtrArray.");
+               BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp, "Failed to allocate a GPtrArray.");
                status = BT_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_MEMORY_ERROR;
                goto error;
        }
@@ -1334,7 +1341,8 @@ bt_message_iterator_class_initialize_method_status muxer_msg_iter_init(
        status = muxer_msg_iter_init_upstream_iterators(muxer_comp,
                muxer_msg_iter, config);
        if (status) {
-               BT_COMP_LOGE("Cannot initialize connected input ports for muxer component's message iterator: "
+               BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                       "Cannot initialize connected input ports for muxer component's message iterator: "
                        "comp-addr=%p, muxer-comp-addr=%p, "
                        "muxer-msg-iter-addr=%p, msg-iter-addr=%p, ret=%d",
                        self_comp, muxer_comp, muxer_msg_iter,
@@ -1406,7 +1414,8 @@ bt_message_iterator_class_next_method_status muxer_msg_iter_next(
        status = muxer_msg_iter_do_next(muxer_comp, muxer_msg_iter,
                msgs, capacity, count);
        if (status < 0) {
-               BT_COMP_LOGE("Cannot get next message: "
+               BT_COMP_LOGE_APPEND_CAUSE(self_comp,
+                       "Cannot get next message: "
                        "comp-addr=%p, muxer-comp-addr=%p, muxer-msg-iter-addr=%p, "
                        "msg-iter-addr=%p, status=%s",
                        self_comp, muxer_comp, muxer_msg_iter, self_msg_iter,
@@ -1434,9 +1443,9 @@ bt_component_class_port_connected_method_status muxer_input_port_connected(
 
        add_port_status = add_available_input_port(self_comp);
        if (add_port_status) {
-               BT_COMP_LOGE("Cannot add one muxer component's input port: "
-                       "status=%s",
-                       bt_common_func_status_string(status));
+               BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                       "Cannot add one muxer component's input port: status=%s",
+                       bt_common_func_status_string(add_port_status));
 
                if (add_port_status ==
                                BT_SELF_COMPONENT_ADD_PORT_STATUS_MEMORY_ERROR) {
@@ -1455,6 +1464,7 @@ end:
 static inline
 bt_message_iterator_class_can_seek_beginning_method_status
 muxer_upstream_msg_iters_can_all_seek_beginning(
+               struct muxer_comp *muxer_comp,
                GPtrArray *muxer_upstream_msg_iters, bt_bool *can_seek)
 {
        bt_message_iterator_class_can_seek_beginning_method_status status =
@@ -1464,9 +1474,12 @@ muxer_upstream_msg_iters_can_all_seek_beginning(
        for (i = 0; i < muxer_upstream_msg_iters->len; i++) {
                struct muxer_upstream_msg_iter *upstream_msg_iter =
                        muxer_upstream_msg_iters->pdata[i];
-               status = (int) bt_self_component_port_input_message_iterator_can_seek_beginning(
+               status = (int) bt_message_iterator_can_seek_beginning(
                        upstream_msg_iter->msg_iter, can_seek);
                if (status != BT_MESSAGE_ITERATOR_CLASS_CAN_SEEK_BEGINNING_METHOD_STATUS_OK) {
+                       BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+                               "Failed to determine whether upstream message iterator can seek beginning: "
+                               "msg-iter-addr=%p", upstream_msg_iter->msg_iter);
                        goto end;
                }
 
@@ -1491,6 +1504,7 @@ muxer_msg_iter_can_seek_beginning(
        bt_message_iterator_class_can_seek_beginning_method_status status;
 
        status = muxer_upstream_msg_iters_can_all_seek_beginning(
+               muxer_msg_iter->muxer_comp,
                muxer_msg_iter->active_muxer_upstream_msg_iters, can_seek);
        if (status != BT_MESSAGE_ITERATOR_CLASS_CAN_SEEK_BEGINNING_METHOD_STATUS_OK) {
                goto end;
@@ -1501,6 +1515,7 @@ muxer_msg_iter_can_seek_beginning(
        }
 
        status = muxer_upstream_msg_iters_can_all_seek_beginning(
+               muxer_msg_iter->muxer_comp,
                muxer_msg_iter->ended_muxer_upstream_msg_iters, can_seek);
 
 end:
@@ -1524,7 +1539,7 @@ bt_message_iterator_class_seek_beginning_method_status muxer_msg_iter_seek_begin
                struct muxer_upstream_msg_iter *upstream_msg_iter =
                        muxer_msg_iter->ended_muxer_upstream_msg_iters->pdata[i];
 
-               seek_beg_status = bt_self_component_port_input_message_iterator_seek_beginning(
+               seek_beg_status = bt_message_iterator_seek_beginning(
                        upstream_msg_iter->msg_iter);
                if (seek_beg_status != BT_MESSAGE_ITERATOR_SEEK_BEGINNING_STATUS_OK) {
                        status = (int) seek_beg_status;
@@ -1540,7 +1555,7 @@ bt_message_iterator_class_seek_beginning_method_status muxer_msg_iter_seek_begin
                struct muxer_upstream_msg_iter *upstream_msg_iter =
                        muxer_msg_iter->active_muxer_upstream_msg_iters->pdata[i];
 
-               seek_beg_status = bt_self_component_port_input_message_iterator_seek_beginning(
+               seek_beg_status = bt_message_iterator_seek_beginning(
                        upstream_msg_iter->msg_iter);
                if (seek_beg_status != BT_MESSAGE_ITERATOR_SEEK_BEGINNING_STATUS_OK) {
                        status = (int) seek_beg_status;
This page took 0.029722 seconds and 4 git commands to generate.