lib: make can_seek_beginning and can_seek_ns_from_origin methods return a status
[babeltrace.git] / src / plugins / utils / muxer / muxer.c
index f92f8326b14c8bfc9c5254940358371ef665bb6c..d6ca31bb63b435383c45c1ee55d5505425b44472 100644 (file)
@@ -36,6 +36,8 @@
 #include <stdlib.h>
 #include <string.h>
 
+#include "plugins/common/muxing/muxing.h"
+
 #include "muxer.h"
 
 #define ASSUME_ABSOLUTE_CLOCK_CLASSES_PARAM_NAME       "assume-absolute-clock-classes"
@@ -428,16 +430,17 @@ void muxer_finalize(bt_self_component_filter *self_comp)
 }
 
 static
-bt_self_component_port_input_message_iterator *
+bt_self_component_port_input_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 *self_port,
+               bt_self_component_port_input_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 *msg_iter =
-               NULL;
+       bt_self_component_port_input_message_iterator_create_from_message_iterator_status
+               status;
 
        BT_ASSERT(port);
        BT_ASSERT(bt_port_is_connected(port));
@@ -445,9 +448,9 @@ 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.
-       msg_iter = bt_self_component_port_input_message_iterator_create_from_message_iterator(
-               muxer_msg_iter->self_msg_iter, self_port);
-       if (!msg_iter) {
+       status = bt_self_component_port_input_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: "
                        "port-addr=%p, port-name=\"%s\"",
                        port, bt_port_get_name(port));
@@ -459,7 +462,7 @@ create_msg_iter_on_input_port(struct muxer_comp *muxer_comp,
                port, bt_port_get_name(port), msg_iter);
 
 end:
-       return msg_iter;
+       return status;
 }
 
 static
@@ -831,8 +834,9 @@ int validate_new_stream_clock_class(struct muxer_msg_iter *muxer_msg_iter,
                        /* Expect no clock class */
                        muxer_msg_iter->clock_class_expectation =
                                MUXER_MSG_ITER_CLOCK_CLASS_EXPECTATION_NONE;
-               } else {
-                       BT_COMP_LOGE("Expecting stream class with a default clock class: "
+               } 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: "
                                "stream-class-addr=%p, stream-class-name=\"%s\", "
                                "stream-class-id=%" PRIu64,
                                stream_class, bt_stream_class_get_name(stream_class),
@@ -849,895 +853,6 @@ end:
        return ret;
 }
 
-struct message_to_compare {
-       const bt_message *msg;
-       const bt_trace *trace;
-       const bt_stream *stream;
-};
-
-struct messages_to_compare {
-       struct message_to_compare left;
-       struct message_to_compare right;
-};
-
-static
-int message_type_weight(const bt_message_type msg_type)
-{
-       int weight;
-
-       switch (msg_type) {
-       case BT_MESSAGE_TYPE_STREAM_BEGINNING:
-               weight = 7;
-               break;
-       case BT_MESSAGE_TYPE_PACKET_BEGINNING:
-               weight = 6;
-               break;
-       case BT_MESSAGE_TYPE_EVENT:
-               weight = 5;
-               break;
-       case BT_MESSAGE_TYPE_DISCARDED_EVENTS:
-               weight = 4;
-               break;
-       case BT_MESSAGE_TYPE_PACKET_END:
-               weight = 3;
-               break;
-       case BT_MESSAGE_TYPE_MESSAGE_ITERATOR_INACTIVITY:
-               weight = 2;
-               break;
-       case BT_MESSAGE_TYPE_DISCARDED_PACKETS:
-               weight = 1;
-               break;
-       case BT_MESSAGE_TYPE_STREAM_END:
-               weight = 0;
-               break;
-       default:
-               abort();
-       }
-
-       return weight;
-}
-
-/*
- * Compare 2 messages to order them in a determinitic way based on their
- * types.
- * Returns -1 is left mesage must go first
- * Returns 1 is right mesage must go first
- */
-static
-int compare_messages_by_type(struct messages_to_compare *msgs)
-{
-       bt_message_type left_msg_type = bt_message_get_type(msgs->left.msg);
-       bt_message_type right_msg_type = bt_message_get_type(msgs->right.msg);
-
-       return message_type_weight(right_msg_type) -
-               message_type_weight(left_msg_type);
-}
-
-static
-int compare_events(const bt_event *left_event, const bt_event *right_event)
-{
-       int ret;
-       const bt_event_class *left_event_class, *right_event_class;
-       uint64_t left_event_class_id, right_event_class_id;
-       const char *left_event_class_name, *right_event_class_name,
-                       *left_event_class_emf_uri, *right_event_class_emf_uri;
-       bt_event_class_log_level left_event_class_log_level, right_event_class_log_level;
-       bt_property_availability left_log_level_avail, right_log_level_avail;
-
-       left_event_class = bt_event_borrow_class_const(left_event);
-       right_event_class = bt_event_borrow_class_const(right_event);
-
-       left_event_class_id = bt_event_class_get_id(left_event_class);
-       right_event_class_id = bt_event_class_get_id(right_event_class);
-
-       if (left_event_class_id > right_event_class_id) {
-               ret = 1;
-               goto end;
-       } else if (left_event_class_id < right_event_class_id) {
-               ret = -1;
-               goto end;
-       }
-
-       left_event_class_name = bt_event_class_get_name(left_event_class);
-       right_event_class_name = bt_event_class_get_name(right_event_class);
-       if (left_event_class_name && right_event_class_name) {
-               ret = strcmp(left_event_class_name, right_event_class_name);
-               if (ret != 0) {
-                       goto end;
-               }
-       } else if (!left_event_class_name && right_event_class_name) {
-               ret = -1;
-               goto end;
-       } else if (left_event_class_name && !right_event_class_name) {
-               ret = 1;
-               goto end;
-       }
-
-       left_log_level_avail = bt_event_class_get_log_level(left_event_class,
-               &left_event_class_log_level);
-       right_log_level_avail = bt_event_class_get_log_level(right_event_class,
-               &right_event_class_log_level);
-
-       if (left_log_level_avail == BT_PROPERTY_AVAILABILITY_AVAILABLE &&
-                       right_log_level_avail == BT_PROPERTY_AVAILABILITY_AVAILABLE) {
-               ret = left_event_class_log_level - right_event_class_log_level;
-               if (ret) {
-                       goto end;
-               }
-       } else if (left_log_level_avail == BT_PROPERTY_AVAILABILITY_AVAILABLE &&
-                       right_log_level_avail == BT_PROPERTY_AVAILABILITY_NOT_AVAILABLE) {
-               ret = -1;
-               goto end;
-       } else if (left_log_level_avail == BT_PROPERTY_AVAILABILITY_NOT_AVAILABLE &&
-                       right_log_level_avail == BT_PROPERTY_AVAILABILITY_AVAILABLE) {
-               ret = 1;
-               goto end;
-       }
-
-       left_event_class_emf_uri = bt_event_class_get_emf_uri(left_event_class);
-       right_event_class_emf_uri = bt_event_class_get_emf_uri(right_event_class);
-       if (left_event_class_emf_uri && right_event_class_emf_uri) {
-               ret = strcmp(left_event_class_emf_uri, right_event_class_emf_uri);
-               if (ret != 0) {
-                       goto end;
-               }
-       } else if (!left_event_class_emf_uri && right_event_class_emf_uri) {
-               ret = -1;
-               goto end;
-       } else if (left_event_class_emf_uri && !right_event_class_emf_uri) {
-               ret = 1;
-               goto end;
-       }
-
-end:
-       return ret;
-}
-
-static
-int compare_streams(const bt_stream *left_stream, const bt_stream *right_stream)
-{
-       int ret = 0;
-       const char *left_stream_name, *right_stream_name,
-             *left_stream_class_name, *right_stream_class_name;
-       const bt_stream_class *left_stream_class, *right_stream_class;
-
-       /*
-        * No need to compare stream id as it was checked earlier and if we are
-        * here it means they are identical or both absent.
-        */
-       BT_ASSERT(bt_stream_get_id(left_stream) ==
-               bt_stream_get_id(right_stream));
-
-       /* Compare stream name. */
-       left_stream_name = bt_stream_get_name(left_stream);
-       right_stream_name = bt_stream_get_name(right_stream);
-
-       if (left_stream_name && right_stream_name) {
-               ret = strcmp(left_stream_name, right_stream_name);
-               if (ret != 0) {
-                       goto end;
-               }
-       } else if (!left_stream_name && right_stream_name) {
-               ret = -1;
-               goto end;
-       } else if (left_stream_name && !right_stream_name) {
-               ret = 1;
-               goto end;
-       }
-
-       left_stream_class = bt_stream_borrow_class_const(left_stream);
-       right_stream_class = bt_stream_borrow_class_const(right_stream);
-
-       /*
-        * No need to compare stream class id as it was checked earlier and if
-        * we are here it means they are identical.
-        */
-       BT_ASSERT(bt_stream_class_get_id(left_stream_class) ==
-               bt_stream_class_get_id(right_stream_class));
-
-       /* Compare stream class name. */
-       left_stream_class_name = bt_stream_class_get_name(left_stream_class);
-       right_stream_class_name = bt_stream_class_get_name(right_stream_class);
-
-       if (left_stream_class_name && right_stream_class_name) {
-               ret = strcmp(left_stream_class_name, right_stream_class_name);
-               if (ret != 0) {
-                       goto end;
-               }
-       } else if (!left_stream_class_name && right_stream_class_name) {
-               ret = -1;
-               goto end;
-       } else if (left_stream_class_name && !right_stream_class_name) {
-               ret = 1;
-               goto end;
-       }
-
-       /* Compare stream class automatic event class id assignment. */
-       if (bt_stream_class_assigns_automatic_event_class_id(left_stream_class) &&
-                       !bt_stream_class_assigns_automatic_event_class_id(right_stream_class)) {
-               ret = 1;
-               goto end;
-       } else if (!bt_stream_class_assigns_automatic_event_class_id(left_stream_class) &&
-                       bt_stream_class_assigns_automatic_event_class_id(right_stream_class)) {
-               ret = -1;
-               goto end;
-       }
-
-       /* Compare stream class automatic stream id assignment. */
-       if (bt_stream_class_assigns_automatic_stream_id(left_stream_class) &&
-                       !bt_stream_class_assigns_automatic_stream_id(right_stream_class)) {
-               ret = 1;
-               goto end;
-       } else if (!bt_stream_class_assigns_automatic_stream_id(left_stream_class) &&
-                       bt_stream_class_assigns_automatic_stream_id(right_stream_class)) {
-               ret = -1;
-               goto end;
-       }
-
-       /* Compare stream class support of discarded events. */
-       if (bt_stream_class_supports_discarded_events(left_stream_class) &&
-                       !bt_stream_class_supports_discarded_events(right_stream_class)) {
-               ret = 1;
-               goto end;
-       } else if (!bt_stream_class_supports_discarded_events(left_stream_class) &&
-                       bt_stream_class_supports_discarded_events(right_stream_class)) {
-               ret = -1;
-               goto end;
-       }
-
-       /* Compare stream class discarded events default clock snapshot. */
-       if (bt_stream_class_discarded_events_have_default_clock_snapshots(left_stream_class) &&
-                       !bt_stream_class_discarded_events_have_default_clock_snapshots(right_stream_class)) {
-               ret = 1;
-               goto end;
-       } else if (!bt_stream_class_discarded_events_have_default_clock_snapshots(left_stream_class) &&
-                       bt_stream_class_discarded_events_have_default_clock_snapshots(right_stream_class)) {
-               ret = -1;
-               goto end;
-       }
-
-       /* Compare stream class support of packets. */
-       if (bt_stream_class_supports_packets(left_stream_class) &&
-                       !bt_stream_class_supports_packets(right_stream_class)) {
-               ret = 1;
-               goto end;
-       } else if (!bt_stream_class_supports_packets(left_stream_class) &&
-                       bt_stream_class_supports_packets(right_stream_class)) {
-               ret = -1;
-               goto end;
-       }
-
-       if (!bt_stream_class_supports_packets(left_stream_class)) {
-               /* Skip all packet related checks. */
-               goto end;
-       }
-
-       /*
-        * Compare stream class presence of discarded packets beginning default
-        * clock snapshot.
-        */
-       if (bt_stream_class_packets_have_beginning_default_clock_snapshot(left_stream_class) &&
-                       !bt_stream_class_packets_have_beginning_default_clock_snapshot(right_stream_class)) {
-               ret = 1;
-               goto end;
-       } else if (!bt_stream_class_packets_have_beginning_default_clock_snapshot(left_stream_class) &&
-                       bt_stream_class_packets_have_beginning_default_clock_snapshot(right_stream_class)) {
-               ret = -1;
-               goto end;
-       }
-
-       /*
-        * Compare stream class presence of discarded packets end default clock
-        * snapshot.
-        */
-       if (bt_stream_class_packets_have_end_default_clock_snapshot(left_stream_class) &&
-                       !bt_stream_class_packets_have_end_default_clock_snapshot(right_stream_class)) {
-               ret = 1;
-               goto end;
-       } else if (!bt_stream_class_packets_have_end_default_clock_snapshot(left_stream_class) &&
-                       bt_stream_class_packets_have_end_default_clock_snapshot(right_stream_class)) {
-               ret = -1;
-               goto end;
-       }
-
-       /* Compare stream class support of discarded packets. */
-       if (bt_stream_class_supports_discarded_packets(left_stream_class) &&
-                       !bt_stream_class_supports_discarded_packets(right_stream_class)) {
-               ret = 1;
-               goto end;
-       } else if (!bt_stream_class_supports_discarded_packets(left_stream_class) &&
-                       bt_stream_class_supports_discarded_packets(right_stream_class)) {
-               ret = -1;
-               goto end;
-       }
-
-       /* Compare stream class discarded packets default clock snapshot. */
-       if (bt_stream_class_discarded_packets_have_default_clock_snapshots(left_stream_class) &&
-                       !bt_stream_class_discarded_packets_have_default_clock_snapshots(right_stream_class)) {
-               ret = 1;
-               goto end;
-       } else if (!bt_stream_class_discarded_packets_have_default_clock_snapshots(left_stream_class) &&
-                       bt_stream_class_discarded_packets_have_default_clock_snapshots(right_stream_class)) {
-               ret = -1;
-               goto end;
-       }
-
-end:
-       return ret;
-}
-
-static
-int compare_clock_snapshots_and_clock_classes(const bt_clock_snapshot *left_cs,
-               const bt_clock_snapshot *right_cs)
-{
-       int ret;
-       uint64_t left_freq, right_freq, left_prec, right_prec;
-       uint64_t left_cs_value, right_cs_value;
-       const bt_clock_class *left_clock_class, *right_clock_class;
-       const char *left_clock_class_name, *right_clock_class_name;
-       left_cs_value = bt_clock_snapshot_get_value(left_cs);
-       right_cs_value = bt_clock_snapshot_get_value(right_cs);
-       bt_uuid left_clock_class_uuid, right_clock_class_uuid;
-
-       ret = left_cs_value - right_cs_value;
-       if (ret != 0) {
-               goto end;
-       }
-
-       left_clock_class = bt_clock_snapshot_borrow_clock_class_const(left_cs);
-       right_clock_class = bt_clock_snapshot_borrow_clock_class_const(right_cs);
-
-       left_clock_class_uuid = bt_clock_class_get_uuid(left_clock_class);
-       right_clock_class_uuid = bt_clock_class_get_uuid(right_clock_class);
-
-       if (left_clock_class_uuid && !right_clock_class_uuid) {
-               ret = -1;
-               goto end;
-       } else if (!left_clock_class_uuid && right_clock_class_uuid) {
-               ret = 1;
-               goto end;
-       } else if (left_clock_class_uuid && right_clock_class_uuid) {
-               ret = bt_uuid_compare(left_clock_class_uuid,
-                       right_clock_class_uuid);
-               if (ret != 0) {
-                       goto end;
-               }
-       }
-
-
-       left_clock_class_name = bt_clock_class_get_name(left_clock_class);
-       right_clock_class_name = bt_clock_class_get_name(right_clock_class);
-
-       if (left_clock_class_name && !right_clock_class_name) {
-               ret = -1;
-               goto end;
-       } else if (!left_clock_class_name && right_clock_class_name) {
-               ret = 1;
-               goto end;
-       } else if (left_clock_class_name && right_clock_class_name) {
-               ret = strcmp(left_clock_class_name, right_clock_class_name);
-               if (ret != 0) {
-                       goto end;
-               }
-       }
-
-       left_freq = bt_clock_class_get_frequency(left_clock_class);
-       right_freq = bt_clock_class_get_frequency(right_clock_class);
-
-       ret = right_freq - left_freq;
-       if (ret != 0) {
-               goto end;
-       }
-
-       left_prec = bt_clock_class_get_precision(left_clock_class);
-       right_prec = bt_clock_class_get_precision(right_clock_class);
-
-       ret = right_prec - left_prec;
-       if (ret != 0) {
-               goto end;
-       }
-
-end:
-       return ret;
-}
-
-static
-const bt_stream *borrow_stream(const bt_message *msg)
-{
-       bt_message_type msg_type = bt_message_get_type(msg);
-       const bt_stream *stream = NULL;
-       const bt_packet *packet = NULL;
-       const bt_event *event = NULL;
-
-       switch (msg_type) {
-       case BT_MESSAGE_TYPE_STREAM_BEGINNING:
-               stream = bt_message_stream_beginning_borrow_stream_const(msg);
-               break;
-       case BT_MESSAGE_TYPE_STREAM_END:
-               stream = bt_message_stream_end_borrow_stream_const(msg);
-               break;
-       case BT_MESSAGE_TYPE_PACKET_BEGINNING:
-               packet = bt_message_packet_beginning_borrow_packet_const(msg);
-               stream = bt_packet_borrow_stream_const(packet);
-               break;
-       case BT_MESSAGE_TYPE_PACKET_END:
-               packet = bt_message_packet_end_borrow_packet_const(msg);
-               stream = bt_packet_borrow_stream_const(packet);
-               break;
-       case BT_MESSAGE_TYPE_EVENT:
-               event = bt_message_event_borrow_event_const(msg);
-               stream = bt_event_borrow_stream_const(event);
-               break;
-       case BT_MESSAGE_TYPE_DISCARDED_EVENTS:
-               stream = bt_message_discarded_events_borrow_stream_const(msg);
-               break;
-       case BT_MESSAGE_TYPE_DISCARDED_PACKETS:
-               stream = bt_message_discarded_packets_borrow_stream_const(msg);
-               break;
-       case BT_MESSAGE_TYPE_MESSAGE_ITERATOR_INACTIVITY:
-               goto end;
-       default:
-               abort();
-       }
-
-end:
-       return stream;
-}
-
-static
-const bt_trace *borrow_trace(const bt_message *msg)
-{
-       const bt_trace *trace = NULL;
-       const bt_stream *stream = NULL;
-
-       stream = borrow_stream(msg);
-       if (stream) {
-               trace = bt_stream_borrow_trace_const(stream);
-       }
-
-       return trace;
-}
-
-static
-int compare_messages_by_trace_name(struct messages_to_compare *msgs)
-{
-       int ret = 0;
-       const char *left_trace_name = NULL, *right_trace_name = NULL;
-
-       if (msgs->left.trace && !msgs->right.trace) {
-               ret = -1;
-               goto end;
-       }
-
-       if (!msgs->left.trace && msgs->right.trace) {
-               ret = 1;
-               goto end;
-       }
-
-       if (!msgs->left.trace && !msgs->right.trace) {
-               ret = 0;
-               goto end;
-       }
-
-       left_trace_name = bt_trace_get_name(msgs->left.trace);
-       right_trace_name = bt_trace_get_name(msgs->right.trace);
-
-       if (left_trace_name && !right_trace_name) {
-               ret = -1;
-               goto end;
-       }
-
-       if (!left_trace_name && right_trace_name) {
-               ret = 1;
-               goto end;
-       }
-
-       if (!left_trace_name && !right_trace_name) {
-               ret = 0;
-               goto end;
-       }
-
-       ret = strcmp(left_trace_name, right_trace_name);
-end:
-       return ret;
-}
-
-static
-int compare_messages_by_trace_uuid(struct messages_to_compare *msgs)
-{
-       int ret = 0;
-       bt_uuid left_trace_uuid = NULL, right_trace_uuid = NULL;
-
-       if (msgs->left.trace && !msgs->right.trace) {
-               ret = -1;
-               goto end;
-       }
-
-       if (!msgs->left.trace && msgs->right.trace) {
-               ret = 1;
-               goto end;
-       }
-
-       if (!msgs->left.trace && !msgs->right.trace) {
-               ret = 0;
-               goto end;
-       }
-
-       left_trace_uuid = bt_trace_get_uuid(msgs->left.trace);
-       right_trace_uuid = bt_trace_get_uuid(msgs->right.trace);
-
-       if (left_trace_uuid && !right_trace_uuid) {
-               ret = -1;
-               goto end;
-       }
-
-       if (!left_trace_uuid && right_trace_uuid) {
-               ret = 1;
-               goto end;
-       }
-
-       if (!left_trace_uuid && !right_trace_uuid) {
-               ret = 0;
-               goto end;
-       }
-
-       ret = bt_uuid_compare(left_trace_uuid, right_trace_uuid);
-end:
-       return ret;
-}
-
-static
-int compare_messages_by_stream_class_id(struct messages_to_compare *msgs)
-{
-       int ret = 0;
-       uint64_t left_stream_class_id = 0, right_stream_class_id = 0;
-
-       if (msgs->left.stream && !msgs->right.stream) {
-               ret = -1;
-               goto end;
-       }
-
-       if (!msgs->left.stream && msgs->right.stream) {
-               ret = 1;
-               goto end;
-       }
-
-       if (!msgs->left.stream && !msgs->right.stream) {
-               ret = 0;
-               goto end;
-       }
-
-       left_stream_class_id = bt_stream_class_get_id(
-               bt_stream_borrow_class_const(msgs->left.stream));
-
-       right_stream_class_id = bt_stream_class_get_id(
-               bt_stream_borrow_class_const(msgs->right.stream));
-
-       if (left_stream_class_id == right_stream_class_id) {
-               ret = 0;
-               goto end;
-       }
-
-       ret = (left_stream_class_id < right_stream_class_id) ? -1 : 1;
-
-end:
-       return ret;
-}
-
-static
-int compare_messages_by_stream_id(struct messages_to_compare *msgs)
-{
-       int ret = 0;
-       uint64_t left_stream_id = 0, right_stream_id = 0;
-
-       if (msgs->left.stream && !msgs->right.stream) {
-               ret = -1;
-               goto end;
-       }
-
-       if (!msgs->left.stream && msgs->right.stream) {
-               ret = 1;
-               goto end;
-       }
-
-       if (!msgs->left.stream && !msgs->right.stream) {
-               ret = 0;
-               goto end;
-       }
-
-       left_stream_id = bt_stream_get_id(msgs->left.stream);
-       right_stream_id = bt_stream_get_id(msgs->right.stream);
-
-       if (left_stream_id == right_stream_id) {
-               ret = 0;
-               goto end;
-       }
-
-       ret = (left_stream_id < right_stream_id) ? -1 : 1;
-
-end:
-       return ret;
-}
-
-static
-int compare_messages_same_type(struct messages_to_compare *msgs)
-{
-       int ret = 0;
-
-       /*
-        * Both messages are of the same type, we must compare characterics of
-        * the messages such as the attributes of the event in a event message.
-        */
-       BT_ASSERT(bt_message_get_type(msgs->left.msg) ==
-               bt_message_get_type(msgs->right.msg));
-
-       switch (bt_message_get_type(msgs->left.msg)) {
-       case BT_MESSAGE_TYPE_STREAM_BEGINNING:
-               ret = compare_streams(msgs->left.stream, msgs->right.stream);
-               if (ret) {
-                       goto end;
-               }
-       case BT_MESSAGE_TYPE_STREAM_END:
-               ret = compare_streams(msgs->left.stream, msgs->right.stream);
-               if (ret) {
-                       goto end;
-               }
-
-               break;
-       case BT_MESSAGE_TYPE_PACKET_BEGINNING:
-               ret = compare_streams(msgs->left.stream, msgs->right.stream);
-               if (ret) {
-                       goto end;
-               }
-
-               break;
-       case BT_MESSAGE_TYPE_PACKET_END:
-               ret = compare_streams(msgs->left.stream, msgs->right.stream);
-               if (ret) {
-                       goto end;
-               }
-
-               break;
-       case BT_MESSAGE_TYPE_EVENT:
-       {
-               const bt_event *left_event, *right_event;
-               left_event = bt_message_event_borrow_event_const(msgs->left.msg);
-               right_event = bt_message_event_borrow_event_const(msgs->right.msg);
-
-               ret = compare_events(left_event, right_event);
-               if (ret) {
-                       goto end;
-               }
-
-               ret = compare_streams(msgs->left.stream, msgs->right.stream);
-               if (ret) {
-                       goto end;
-               }
-               break;
-       }
-       case BT_MESSAGE_TYPE_DISCARDED_EVENTS:
-       {
-               const bt_stream_class *left_stream_class;
-               bt_property_availability left_event_count_avail,
-                                        right_event_count_avail;
-               uint64_t left_event_count, right_event_count;
-
-               /*
-                * Compare streams first to check if there is a
-                * mismatch about discarded event related configuration
-                * in the stream class.
-                */
-               ret = compare_streams(msgs->left.stream, msgs->right.stream);
-               if (ret) {
-                       goto end;
-               }
-
-               left_stream_class = bt_stream_borrow_class_const(msgs->left.stream);
-               if (bt_stream_class_discarded_events_have_default_clock_snapshots(
-                               left_stream_class)) {
-                       const bt_clock_snapshot *left_beg_cs =
-                               bt_message_discarded_events_borrow_beginning_default_clock_snapshot_const(msgs->left.msg);
-                       const bt_clock_snapshot *right_beg_cs =
-                               bt_message_discarded_events_borrow_beginning_default_clock_snapshot_const(msgs->right.msg);
-                       const bt_clock_snapshot *left_end_cs =
-                               bt_message_discarded_events_borrow_end_default_clock_snapshot_const(msgs->left.msg);
-                       const bt_clock_snapshot *right_end_cs =
-                               bt_message_discarded_events_borrow_end_default_clock_snapshot_const(msgs->right.msg);
-
-                       ret = compare_clock_snapshots_and_clock_classes(
-                               left_beg_cs, right_beg_cs);
-                       if (ret) {
-                               goto end;
-                       }
-
-                       ret = compare_clock_snapshots_and_clock_classes(
-                               left_end_cs, right_end_cs);
-                       if (ret) {
-                               goto end;
-                       }
-               }
-
-               left_event_count_avail =
-                       bt_message_discarded_events_get_count(
-                               msgs->left.msg, &left_event_count);
-               right_event_count_avail =
-                       bt_message_discarded_events_get_count(
-                               msgs->right.msg, &right_event_count);
-               if (left_event_count_avail == BT_PROPERTY_AVAILABILITY_AVAILABLE &&
-                       right_event_count_avail == BT_PROPERTY_AVAILABILITY_AVAILABLE) {
-                       ret = left_event_count - right_event_count;
-                       if (ret != 0) {
-                               goto end;
-                       }
-               } else if (left_event_count_avail == BT_PROPERTY_AVAILABILITY_AVAILABLE &&
-                       right_event_count_avail == BT_PROPERTY_AVAILABILITY_NOT_AVAILABLE) {
-                       ret = -1;
-                       goto end;
-               } else if (left_event_count_avail == BT_PROPERTY_AVAILABILITY_NOT_AVAILABLE &&
-                       right_event_count_avail == BT_PROPERTY_AVAILABILITY_AVAILABLE) {
-                       ret = 1;
-                       goto end;
-               }
-
-               break;
-       }
-       case BT_MESSAGE_TYPE_DISCARDED_PACKETS:
-       {
-               const bt_stream_class *left_stream_class;
-               bt_property_availability left_packet_count_avail,
-                                        right_packet_count_avail;
-               uint64_t left_packet_count, right_packet_count;
-
-               /*
-                * Compare streams first to check if there is a
-                * mismatch about discarded packets related
-                * configuration in the stream class.
-                */
-               ret = compare_streams(msgs->left.stream, msgs->right.stream);
-               if (ret) {
-                       goto end;
-               }
-
-               left_stream_class = bt_stream_borrow_class_const(msgs->left.stream);
-
-               if (bt_stream_class_discarded_packets_have_default_clock_snapshots(
-                               left_stream_class)) {
-                       const bt_clock_snapshot *left_beg_cs =
-                               bt_message_discarded_packets_borrow_beginning_default_clock_snapshot_const(msgs->left.msg);
-                       const bt_clock_snapshot *right_beg_cs =
-                               bt_message_discarded_packets_borrow_beginning_default_clock_snapshot_const(msgs->right.msg);
-                       const bt_clock_snapshot *left_end_cs =
-                               bt_message_discarded_packets_borrow_end_default_clock_snapshot_const(msgs->left.msg);
-                       const bt_clock_snapshot *right_end_cs =
-                               bt_message_discarded_packets_borrow_end_default_clock_snapshot_const(msgs->right.msg);
-
-                       ret = compare_clock_snapshots_and_clock_classes(
-                               left_beg_cs, right_beg_cs);
-                       if (ret) {
-                               goto end;
-                       }
-
-                       ret = compare_clock_snapshots_and_clock_classes(
-                               left_end_cs, right_end_cs);
-                       if (ret) {
-                               goto end;
-                       }
-               }
-
-               left_packet_count_avail = bt_message_discarded_packets_get_count(
-                       msgs->left.msg, &left_packet_count);
-               right_packet_count_avail = bt_message_discarded_packets_get_count(
-                       msgs->right.msg, &right_packet_count);
-               if (left_packet_count_avail == BT_PROPERTY_AVAILABILITY_AVAILABLE &&
-                       right_packet_count_avail == BT_PROPERTY_AVAILABILITY_AVAILABLE) {
-                       ret = left_packet_count - right_packet_count;
-                       if (ret != 0) {
-                               goto end;
-                       }
-               } else if (left_packet_count_avail == BT_PROPERTY_AVAILABILITY_AVAILABLE &&
-                       right_packet_count_avail == BT_PROPERTY_AVAILABILITY_NOT_AVAILABLE) {
-                       ret = -1;
-                       goto end;
-               } else if (left_packet_count_avail == BT_PROPERTY_AVAILABILITY_NOT_AVAILABLE &&
-                       right_packet_count_avail == BT_PROPERTY_AVAILABILITY_AVAILABLE) {
-                       ret = 1;
-                       goto end;
-               }
-
-               break;
-       }
-       case BT_MESSAGE_TYPE_MESSAGE_ITERATOR_INACTIVITY:
-       {
-               const bt_clock_snapshot *left_cs =
-                       bt_message_message_iterator_inactivity_borrow_default_clock_snapshot_const(msgs->left.msg);
-               const bt_clock_snapshot *right_cs =
-                       bt_message_message_iterator_inactivity_borrow_default_clock_snapshot_const(msgs->right.msg);
-
-               ret = compare_clock_snapshots_and_clock_classes(
-                       left_cs, right_cs);
-               if (ret != 0) {
-                       goto end;
-               }
-
-               break;
-       }
-       default:
-               abort();
-       }
-
-end:
-       return ret;
-}
-
-static
-int compare_messages(const bt_message *left_msg, const bt_message *right_msg)
-{
-       int ret = 0;
-       struct messages_to_compare msgs;
-
-       BT_ASSERT(left_msg != right_msg);
-
-       msgs.left.msg = left_msg;
-       msgs.left.trace = borrow_trace(left_msg);
-       msgs.left.stream = borrow_stream(left_msg);
-
-       msgs.right.msg = right_msg;
-       msgs.right.trace = borrow_trace(right_msg);
-       msgs.right.stream = borrow_stream(right_msg);
-
-       /* Same timestamp: compare trace names. */
-       ret = compare_messages_by_trace_name(&msgs);
-       if (ret) {
-               goto end;
-       }
-
-       /* Same timestamp and trace name: compare trace UUIDs. */
-       ret = compare_messages_by_trace_uuid(&msgs);
-       if (ret) {
-               goto end;
-       }
-
-       /*
-        * Same timestamp, trace name, and trace UUID: compare stream class
-        * IDs.
-        */
-       ret = compare_messages_by_stream_class_id(&msgs);
-       if (ret) {
-               goto end;
-       }
-
-       /*
-        * Same timestamp, trace name, trace UUID, and stream class ID: compare
-        * stream IDs.
-        */
-       ret = compare_messages_by_stream_id(&msgs);
-       if (ret) {
-               goto end;
-       }
-
-       if (bt_message_get_type(msgs.left.msg) !=
-                       bt_message_get_type(msgs.right.msg)) {
-               /*
-                * The messages are of different type, we order (arbitrarily)
-                * in the following way:
-                * SB < PB < EV < DE < MI < PE < DP < SE
-                */
-               ret = compare_messages_by_type(&msgs);
-               if (ret) {
-                       goto end;
-               }
-       } else {
-               /* The messages are of the same type. */
-               ret = compare_messages_same_type(&msgs);
-               if (ret) {
-                       goto end;
-               }
-       }
-
-end:
-       return ret;
-}
-
 /*
  * This function finds the youngest available message amongst the
  * non-ended upstream message iterators and returns the upstream
@@ -1832,7 +947,13 @@ muxer_msg_iter_youngest_upstream_msg_iter(
                        goto end;
                }
 
-               if (msg_ts_ns < youngest_ts_ns) {
+               /*
+                * Update the current message iterator if it has not been set
+                * yet, or if its current message has a timestamp smaller than
+                * the previously selected youngest message.
+                */
+               if (G_UNLIKELY(*muxer_upstream_msg_iter == NULL) ||
+                               msg_ts_ns < youngest_ts_ns) {
                        *muxer_upstream_msg_iter =
                                cur_muxer_upstream_msg_iter;
                        youngest_ts_ns = msg_ts_ns;
@@ -1852,7 +973,7 @@ muxer_msg_iter_youngest_upstream_msg_iter(
                         * Order the messages in an arbitrary but determinitic
                         * way.
                         */
-                       ret = compare_messages(msg, selected_msg);
+                       ret = common_muxing_compare_messages(msg, selected_msg);
                        if (ret < 0) {
                                /*
                                 * The `msg` should go first. Update the next
@@ -2121,12 +1242,13 @@ void destroy_muxer_msg_iter(struct muxer_msg_iter *muxer_msg_iter)
 }
 
 static
-int muxer_msg_iter_init_upstream_iterators(struct muxer_comp *muxer_comp,
+bt_component_class_message_iterator_init_method_status
+muxer_msg_iter_init_upstream_iterators(struct muxer_comp *muxer_comp,
                struct muxer_msg_iter *muxer_msg_iter)
 {
        int64_t count;
        int64_t i;
-       int ret = 0;
+       bt_component_class_message_iterator_init_method_status status;
 
        count = bt_component_filter_get_input_port_count(
                bt_self_component_filter_as_component_filter(
@@ -2135,6 +1257,7 @@ int muxer_msg_iter_init_upstream_iterators(struct muxer_comp *muxer_comp,
                BT_COMP_LOGD("No input port to initialize for muxer component's message iterator: "
                        "muxer-comp-addr=%p, muxer-msg-iter-addr=%p",
                        muxer_comp, muxer_msg_iter);
+               status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_INIT_METHOD_STATUS_OK;
                goto end;
        }
 
@@ -2144,6 +1267,9 @@ int muxer_msg_iter_init_upstream_iterators(struct muxer_comp *muxer_comp,
                        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
+                       msg_iter_status;
+               int int_status;
 
                BT_ASSERT(self_port);
                port = bt_self_component_port_as_port(
@@ -2156,27 +1282,29 @@ int muxer_msg_iter_init_upstream_iterators(struct muxer_comp *muxer_comp,
                        continue;
                }
 
-               upstream_msg_iter = create_msg_iter_on_input_port(muxer_comp,
-                       muxer_msg_iter, self_port);
-               if (!upstream_msg_iter) {
+               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) {
                        /* create_msg_iter_on_input_port() logs errors */
-                       BT_ASSERT(!upstream_msg_iter);
-                       ret = -1;
+                       status = (int) msg_iter_status;
                        goto end;
                }
 
-               ret = muxer_msg_iter_add_upstream_msg_iter(muxer_msg_iter,
+               int_status = muxer_msg_iter_add_upstream_msg_iter(muxer_msg_iter,
                        upstream_msg_iter);
                bt_self_component_port_input_message_iterator_put_ref(
                        upstream_msg_iter);
-               if (ret) {
+               if (int_status) {
+                       status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_INIT_METHOD_STATUS_ERROR;
                        /* muxer_msg_iter_add_upstream_msg_iter() logs errors */
                        goto end;
                }
        }
 
+       status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_INIT_METHOD_STATUS_OK;
+
 end:
-       return ret;
+       return status;
 }
 
 BT_HIDDEN
@@ -2187,9 +1315,7 @@ bt_component_class_message_iterator_init_method_status muxer_msg_iter_init(
 {
        struct muxer_comp *muxer_comp = NULL;
        struct muxer_msg_iter *muxer_msg_iter = NULL;
-       bt_component_class_message_iterator_init_method_status status =
-               BT_COMPONENT_CLASS_MESSAGE_ITERATOR_INIT_METHOD_STATUS_OK;
-       int ret;
+       bt_component_class_message_iterator_init_method_status status;
 
        muxer_comp = bt_self_component_get_data(
                bt_self_component_filter_as_self_component(self_comp));
@@ -2207,6 +1333,7 @@ bt_component_class_message_iterator_init_method_status muxer_msg_iter_init(
                BT_COMP_LOGE("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_COMPONENT_CLASS_MESSAGE_ITERATOR_INIT_METHOD_STATUS_ERROR;
                goto error;
        }
 
@@ -2214,6 +1341,7 @@ bt_component_class_message_iterator_init_method_status muxer_msg_iter_init(
        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.");
+               status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_INIT_METHOD_STATUS_MEMORY_ERROR;
                goto error;
        }
 
@@ -2225,6 +1353,7 @@ bt_component_class_message_iterator_init_method_status muxer_msg_iter_init(
                        (GDestroyNotify) destroy_muxer_upstream_msg_iter);
        if (!muxer_msg_iter->active_muxer_upstream_msg_iters) {
                BT_COMP_LOGE_STR("Failed to allocate a GPtrArray.");
+               status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_INIT_METHOD_STATUS_MEMORY_ERROR;
                goto error;
        }
 
@@ -2233,17 +1362,18 @@ bt_component_class_message_iterator_init_method_status muxer_msg_iter_init(
                        (GDestroyNotify) destroy_muxer_upstream_msg_iter);
        if (!muxer_msg_iter->ended_muxer_upstream_msg_iters) {
                BT_COMP_LOGE_STR("Failed to allocate a GPtrArray.");
+               status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_INIT_METHOD_STATUS_MEMORY_ERROR;
                goto error;
        }
 
-       ret = muxer_msg_iter_init_upstream_iterators(muxer_comp,
+       status = muxer_msg_iter_init_upstream_iterators(muxer_comp,
                muxer_msg_iter);
-       if (ret) {
+       if (status) {
                BT_COMP_LOGE("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,
-                       self_msg_iter, ret);
+                       self_msg_iter, status);
                goto error;
        }
 
@@ -2257,7 +1387,6 @@ bt_component_class_message_iterator_init_method_status muxer_msg_iter_init(
 error:
        destroy_muxer_msg_iter(muxer_msg_iter);
        bt_self_message_iterator_set_data(self_msg_iter, NULL);
-       status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_INIT_METHOD_STATUS_ERROR;
 
 end:
        muxer_comp->initializing_muxer_msg_iter = false;
@@ -2359,49 +1488,57 @@ end:
 }
 
 static inline
-bt_bool muxer_upstream_msg_iters_can_all_seek_beginning(
-               GPtrArray *muxer_upstream_msg_iters)
+bt_component_class_message_iterator_can_seek_beginning_method_status
+muxer_upstream_msg_iters_can_all_seek_beginning(
+               GPtrArray *muxer_upstream_msg_iters, bt_bool *can_seek)
 {
+       bt_component_class_message_iterator_can_seek_beginning_method_status status;
        uint64_t i;
-       bt_bool ret = BT_TRUE;
 
        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(
+                       upstream_msg_iter->msg_iter, can_seek);
+               if (status != BT_COMPONENT_CLASS_MESSAGE_ITERATOR_CAN_SEEK_BEGINNING_METHOD_STATUS_OK) {
+                       goto end;
+               }
 
-               if (!bt_self_component_port_input_message_iterator_can_seek_beginning(
-                               upstream_msg_iter->msg_iter)) {
-                       ret = BT_FALSE;
+               if (!*can_seek) {
                        goto end;
                }
        }
 
+       *can_seek = BT_TRUE;
+
 end:
-       return ret;
+       return status;
 }
 
 BT_HIDDEN
-bt_bool muxer_msg_iter_can_seek_beginning(
-               bt_self_message_iterator *self_msg_iter)
+bt_component_class_message_iterator_can_seek_beginning_method_status
+muxer_msg_iter_can_seek_beginning(
+               bt_self_message_iterator *self_msg_iter, bt_bool *can_seek)
 {
        struct muxer_msg_iter *muxer_msg_iter =
                bt_self_message_iterator_get_data(self_msg_iter);
-       bt_bool ret = BT_TRUE;
+       bt_component_class_message_iterator_can_seek_beginning_method_status status;
 
-       if (!muxer_upstream_msg_iters_can_all_seek_beginning(
-                       muxer_msg_iter->active_muxer_upstream_msg_iters)) {
-               ret = BT_FALSE;
+       status = muxer_upstream_msg_iters_can_all_seek_beginning(
+               muxer_msg_iter->active_muxer_upstream_msg_iters, can_seek);
+       if (status != BT_COMPONENT_CLASS_MESSAGE_ITERATOR_CAN_SEEK_BEGINNING_METHOD_STATUS_OK) {
                goto end;
        }
 
-       if (!muxer_upstream_msg_iters_can_all_seek_beginning(
-                       muxer_msg_iter->ended_muxer_upstream_msg_iters)) {
-               ret = BT_FALSE;
+       if (!*can_seek) {
                goto end;
        }
 
+       status = muxer_upstream_msg_iters_can_all_seek_beginning(
+               muxer_msg_iter->ended_muxer_upstream_msg_iters, can_seek);
+
 end:
-       return ret;
+       return status;
 }
 
 BT_HIDDEN
This page took 0.035722 seconds and 4 git commands to generate.