/*
- * 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:
+ * SPDX-License-Identifier: MIT
*
- * The above copyright notice and this permission notice shall be included in
- * all copies or substantial portions of the Software.
- *
- * 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)
#include <stdlib.h>
#include <string.h>
-#include "muxer.h"
+#include "plugins/common/muxing/muxing.h"
+#include "plugins/common/param-validation/param-validation.h"
-#define ASSUME_ABSOLUTE_CLOCK_CLASSES_PARAM_NAME "assume-absolute-clock-classes"
+#include "muxer.h"
struct muxer_comp {
/* Weak refs */
unsigned int next_port_num;
size_t available_input_ports;
bool initializing_muxer_msg_iter;
- bool assume_absolute_clock_classes;
bt_logging_level log_level;
};
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;
* MUXER_MSG_ITER_CLOCK_CLASS_EXPECTATION_NOT_ABS_SPEC_UUID.
*/
bt_uuid_t expected_clock_class_uuid;
+
+ /*
+ * Saved error. If we hit an error in the _next method, but have some
+ * messages ready to return, we save the error here and return it on
+ * the next _next call.
+ */
+ bt_message_iterator_class_next_method_status next_saved_status;
+ const struct bt_error *next_saved_error;
};
static
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) {
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 =
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.");
}
static
-bt_value *get_default_params(struct muxer_comp *muxer_comp)
-{
- bt_value *params;
- int ret;
-
- params = bt_value_map_create();
- if (!params) {
- BT_COMP_LOGE_STR("Cannot create a map value object.");
- goto error;
- }
-
- ret = bt_value_map_insert_bool_entry(params,
- ASSUME_ABSOLUTE_CLOCK_CLASSES_PARAM_NAME, false);
- if (ret) {
- BT_COMP_LOGE_STR("Cannot add boolean value to map value object.");
- goto error;
- }
-
- goto end;
-
-error:
- BT_VALUE_PUT_REF_AND_RESET(params);
-
-end:
- return params;
-}
-
-static
-int configure_muxer_comp(struct muxer_comp *muxer_comp,
- const bt_value *params)
-{
- bt_value *default_params = NULL;
- bt_value *real_params = NULL;
- const bt_value *assume_absolute_clock_classes = NULL;
- int ret = 0;
- bt_bool bool_val;
-
- default_params = get_default_params(muxer_comp);
- if (!default_params) {
- BT_COMP_LOGE("Cannot get default parameters: "
- "muxer-comp-addr=%p", muxer_comp);
- goto error;
- }
-
- ret = bt_value_map_extend(default_params, params, &real_params);
- if (ret) {
- BT_COMP_LOGE("Cannot extend default parameters map value: "
- "muxer-comp-addr=%p, def-params-addr=%p, "
- "params-addr=%p", muxer_comp, default_params,
- params);
- goto error;
- }
-
- assume_absolute_clock_classes = bt_value_map_borrow_entry_value(real_params,
- ASSUME_ABSOLUTE_CLOCK_CLASSES_PARAM_NAME);
- if (assume_absolute_clock_classes &&
- !bt_value_is_bool(assume_absolute_clock_classes)) {
- BT_COMP_LOGE("Expecting a boolean value for the `%s` parameter: "
- "muxer-comp-addr=%p, value-type=%s",
- ASSUME_ABSOLUTE_CLOCK_CLASSES_PARAM_NAME, muxer_comp,
- bt_common_value_type_string(
- bt_value_get_type(assume_absolute_clock_classes)));
- goto error;
- }
-
- bool_val = bt_value_bool_get(assume_absolute_clock_classes);
- muxer_comp->assume_absolute_clock_classes = (bool) bool_val;
- BT_COMP_LOGI("Configured muxer component: muxer-comp-addr=%p, "
- "assume-absolute-clock-classes=%d",
- muxer_comp, muxer_comp->assume_absolute_clock_classes);
- goto end;
-
-error:
- ret = -1;
-
-end:
- bt_value_put_ref(default_params);
- bt_value_put_ref(real_params);
- return ret;
-}
+struct bt_param_validation_map_value_entry_descr muxer_params[] = {
+ BT_PARAM_VALIDATION_MAP_VALUE_ENTRY_END
+};
BT_HIDDEN
-bt_component_class_init_method_status muxer_init(
+bt_component_class_initialize_method_status muxer_init(
bt_self_component_filter *self_comp_flt,
+ bt_self_component_filter_configuration *config,
const bt_value *params, void *init_data)
{
- int ret;
- bt_component_class_init_method_status status =
- BT_COMPONENT_CLASS_INIT_METHOD_STATUS_OK;
+ bt_component_class_initialize_method_status status;
bt_self_component_add_port_status add_port_status;
bt_self_component *self_comp =
bt_self_component_filter_as_self_component(self_comp_flt);
struct muxer_comp *muxer_comp = g_new0(struct muxer_comp, 1);
bt_logging_level log_level = bt_component_get_logging_level(
bt_self_component_as_component(self_comp));
+ enum bt_param_validation_status validation_status;
+ gchar *validate_error = NULL;
BT_COMP_LOG_CUR_LVL(BT_LOG_INFO, log_level, self_comp,
"Initializing muxer component: "
if (!muxer_comp) {
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;
goto error;
}
muxer_comp->log_level = log_level;
muxer_comp->self_comp = self_comp;
muxer_comp->self_comp_flt = self_comp_flt;
- ret = configure_muxer_comp(muxer_comp, params);
- if (ret) {
- BT_COMP_LOGE("Cannot configure muxer component: "
- "muxer-comp-addr=%p, params-addr=%p",
- muxer_comp, params);
+
+ validation_status = bt_param_validation_validate(params,
+ muxer_params, &validate_error);
+ if (validation_status == BT_PARAM_VALIDATION_STATUS_MEMORY_ERROR) {
+ status = BT_COMPONENT_CLASS_INITIALIZE_METHOD_STATUS_MEMORY_ERROR;
+ goto error;
+ } else if (validation_status == BT_PARAM_VALIDATION_STATUS_VALIDATION_ERROR) {
+ status = BT_COMPONENT_CLASS_INITIALIZE_METHOD_STATUS_ERROR;
+ BT_COMP_LOGE_APPEND_CAUSE(self_comp, "%s", validate_error);
goto error;
}
"muxer-comp-addr=%p, status=%s",
muxer_comp,
bt_common_func_status_string(add_port_status));
- if (add_port_status ==
- BT_SELF_COMPONENT_ADD_PORT_STATUS_MEMORY_ERROR) {
- status = BT_COMPONENT_CLASS_INIT_METHOD_STATUS_MEMORY_ERROR;
- } else {
- status = BT_COMPONENT_CLASS_INIT_METHOD_STATUS_ERROR;
- }
-
+ status = (int) add_port_status;
goto error;
}
"muxer-comp-addr=%p, status=%s",
muxer_comp,
bt_common_func_status_string(add_port_status));
- if (add_port_status ==
- BT_SELF_COMPONENT_ADD_PORT_STATUS_MEMORY_ERROR) {
- status = BT_COMPONENT_CLASS_INIT_METHOD_STATUS_MEMORY_ERROR;
- } else {
- status = BT_COMPONENT_CLASS_INIT_METHOD_STATUS_ERROR;
- }
-
+ status = (int) add_port_status;
goto error;
}
"comp-addr=%p, params-addr=%p, muxer-comp-addr=%p",
self_comp, params, muxer_comp);
+ status = BT_COMPONENT_CLASS_INITIALIZE_METHOD_STATUS_OK;
goto end;
error:
destroy_muxer_comp(muxer_comp);
bt_self_component_set_data(self_comp, NULL);
- if (status == BT_COMPONENT_CLASS_INIT_METHOD_STATUS_OK) {
- status = BT_COMPONENT_CLASS_INIT_METHOD_STATUS_ERROR;
- }
-
end:
+ g_free(validate_error);
return status;
}
}
static
-bt_self_component_port_input_message_iterator *
+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 *self_port,
+ 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 *msg_iter =
- NULL;
+ bt_message_iterator_create_from_message_iterator_status
+ status;
BT_ASSERT(port);
BT_ASSERT(bt_port_is_connected(port));
// 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_message_iterator_create_from_message_iterator(
+ muxer_msg_iter->self_msg_iter, self_port, msg_iter);
+ if (status != BT_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));
port, bt_port_get_name(port), msg_iter);
end:
- return msg_iter;
+ return status;
}
static
-bt_component_class_message_iterator_next_method_status muxer_upstream_msg_iter_next(
+bt_message_iterator_class_next_method_status muxer_upstream_msg_iter_next(
struct muxer_upstream_msg_iter *muxer_upstream_msg_iter,
bool *is_ended)
{
- struct muxer_comp *muxer_comp =
- muxer_upstream_msg_iter->muxer_comp;
- bt_component_class_message_iterator_next_method_status status;
+ struct muxer_comp *muxer_comp = muxer_upstream_msg_iter->muxer_comp;
+ bt_message_iterator_class_next_method_status status;
bt_message_iterator_next_status input_port_iter_status;
bt_message_array_const msgs;
uint64_t i;
"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",
* valid: it must be considered for muxing operations.
*/
BT_COMP_LOGD_STR("Validated upstream message iterator wrapper.");
- BT_ASSERT(count > 0);
+ BT_ASSERT_DBG(count > 0);
/* Move messages to our queue */
for (i = 0; i < count; i++) {
g_queue_push_tail(muxer_upstream_msg_iter->msgs,
(void *) msgs[i]);
}
- status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_OK;
+ status = BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_OK;
break;
case BT_MESSAGE_ITERATOR_NEXT_STATUS_AGAIN:
/*
* valid anymore. Return
* BT_MESSAGE_ITERATOR_NEXT_STATUS_AGAIN immediately.
*/
- status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_AGAIN;
+ status = BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_AGAIN;
break;
case BT_MESSAGE_ITERATOR_NEXT_STATUS_END: /* Fall-through. */
/*
* message.
*/
*is_ended = true;
- status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_OK;
+ status = BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_OK;
+ break;
+ case BT_MESSAGE_ITERATOR_NEXT_STATUS_ERROR:
+ case BT_MESSAGE_ITERATOR_NEXT_STATUS_MEMORY_ERROR:
+ /* Error status code */
+ BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+ "Upstream iterator's next method returned an error: status=%s",
+ bt_common_func_status_string(input_port_iter_status));
+ status = (int) input_port_iter_status;
break;
default:
- /* Error or unsupported status code */
- BT_COMP_LOGE("Error or unsupported status code: "
- "status-code=%d", input_port_iter_status);
- status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_ERROR;
+ /* Unsupported status code */
+ BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+ "Unsupported status code: status=%s",
+ bt_common_func_status_string(input_port_iter_status));
+ status = BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_ERROR;
break;
}
const bt_stream_class *stream_class = NULL;
bt_message_type msg_type;
- BT_ASSERT(msg);
- BT_ASSERT(ts_ns);
+ BT_ASSERT_DBG(msg);
+ BT_ASSERT_DBG(ts_ns);
BT_COMP_LOGD("Getting message's timestamp: "
"muxer-msg-iter-addr=%p, msg-addr=%p, "
"last-returned-ts=%" PRId64,
switch (msg_type) {
case BT_MESSAGE_TYPE_EVENT:
- BT_ASSERT(bt_message_event_borrow_stream_class_default_clock_class_const(
+ BT_ASSERT_DBG(bt_message_event_borrow_stream_class_default_clock_class_const(
msg));
clock_snapshot = bt_message_event_borrow_default_clock_snapshot_const(
msg);
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:
const uint8_t *cc_uuid;
const char *cc_name;
- BT_ASSERT(clock_class);
+ BT_ASSERT_DBG(clock_class);
cc_uuid = bt_clock_class_get_uuid(clock_class);
cc_name = bt_clock_class_get_name(clock_class);
if (muxer_msg_iter->clock_class_expectation ==
MUXER_MSG_ITER_CLOCK_CLASS_EXPECTATION_ANY) {
/*
- * This is the first clock class that this muxer
- * message iterator encounters. Its properties
- * determine what to expect for the whole lifetime of
- * the iterator without a true
- * `assume-absolute-clock-classes` parameter.
+ * This is the first clock class that this muxer message
+ * iterator encounters. Its properties determine what to expect
+ * for the whole lifetime of the iterator.
*/
if (bt_clock_class_origin_is_unix_epoch(clock_class)) {
/* Expect absolute clock classes */
}
}
- if (!muxer_comp->assume_absolute_clock_classes) {
- 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, "
- "but got a non-absolute one: "
- "clock-class-addr=%p, clock-class-name=\"%s\"",
- clock_class, cc_name);
- goto error;
- }
- 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, "
- "but got an absolute one: "
- "clock-class-addr=%p, clock-class-name=\"%s\"",
- clock_class, cc_name);
- goto error;
- }
-
- if (cc_uuid) {
- BT_COMP_LOGE("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 "\"",
- clock_class, cc_name, BT_UUID_FMT_VALUES(cc_uuid));
- goto error;
- }
- 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, "
- "but got an absolute one: "
- "clock-class-addr=%p, clock-class-name=\"%s\"",
- clock_class, cc_name);
- goto error;
- }
+ 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, "
+ "but got a non-absolute one: "
+ "clock-class-addr=%p, clock-class-name=\"%s\"",
+ clock_class, cc_name);
+ goto error;
+ }
+ 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, "
+ "but got an absolute one: "
+ "clock-class-addr=%p, clock-class-name=\"%s\"",
+ clock_class, cc_name);
+ goto error;
+ }
- if (!cc_uuid) {
- BT_COMP_LOGE("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);
- goto error;
- }
+ if (cc_uuid) {
+ BT_COMP_LOGE("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 "\"",
+ clock_class, cc_name, BT_UUID_FMT_VALUES(cc_uuid));
+ goto error;
+ }
+ 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, "
+ "but got an absolute one: "
+ "clock-class-addr=%p, clock-class-name=\"%s\"",
+ clock_class, cc_name);
+ goto error;
+ }
- 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, "
- "but got one with different UUID: "
- "clock-class-addr=%p, clock-class-name=\"%s\", "
- "expected-uuid=\"" BT_UUID_FMT "\", "
- "uuid=\"" BT_UUID_FMT "\"",
- clock_class, cc_name,
- BT_UUID_FMT_VALUES(muxer_msg_iter->expected_clock_class_uuid),
- BT_UUID_FMT_VALUES(cc_uuid));
- goto error;
- }
- break;
- case MUXER_MSG_ITER_CLOCK_CLASS_EXPECTATION_NONE:
- BT_COMP_LOGE("Expecting no clock class, but got one: "
+ if (!cc_uuid) {
+ BT_COMP_LOGE("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);
goto error;
- default:
- /* Unexpected */
- BT_COMP_LOGF("Unexpected clock class expectation: "
- "expectation-code=%d",
- muxer_msg_iter->clock_class_expectation);
- abort();
}
+
+ 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, "
+ "but got one with different UUID: "
+ "clock-class-addr=%p, clock-class-name=\"%s\", "
+ "expected-uuid=\"" BT_UUID_FMT "\", "
+ "uuid=\"" BT_UUID_FMT "\"",
+ clock_class, cc_name,
+ BT_UUID_FMT_VALUES(muxer_msg_iter->expected_clock_class_uuid),
+ BT_UUID_FMT_VALUES(cc_uuid));
+ goto error;
+ }
+ break;
+ case MUXER_MSG_ITER_CLOCK_CLASS_EXPECTATION_NONE:
+ BT_COMP_LOGE("Expecting no clock class, but got one: "
+ "clock-class-addr=%p, clock-class-name=\"%s\"",
+ clock_class, cc_name);
+ goto error;
+ default:
+ /* Unexpected */
+ BT_COMP_LOGF("Unexpected clock class expectation: "
+ "expectation-code=%d",
+ muxer_msg_iter->clock_class_expectation);
+ bt_common_abort();
}
goto end;
/* 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),
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
+ * This function finds the youngest available message amongst the
+ * non-ended upstream message iterators and returns the upstream
+ * message iterator which has it, or
+ * BT_MESSAGE_ITERATOR_STATUS_END if there's no available
+ * message.
+ *
+ * This function does NOT:
+ *
+ * * Update any upstream message iterator.
+ * * Check the upstream message iterators to retry.
+ *
+ * On sucess, this function sets *muxer_upstream_msg_iter to the
+ * upstream message iterator of which the current message is
+ * the youngest, and sets *ts_ns to its time.
*/
static
-int compare_messages_by_type(struct messages_to_compare *msgs)
+bt_message_iterator_class_next_method_status
+muxer_msg_iter_youngest_upstream_msg_iter(
+ struct muxer_comp *muxer_comp,
+ struct muxer_msg_iter *muxer_msg_iter,
+ struct muxer_upstream_msg_iter **muxer_upstream_msg_iter,
+ int64_t *ts_ns)
{
- 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);
+ size_t i;
+ int ret;
+ int64_t youngest_ts_ns = INT64_MAX;
+ bt_message_iterator_class_next_method_status status =
+ BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_OK;
- return message_type_weight(right_msg_type) -
- message_type_weight(left_msg_type);
-}
+ BT_ASSERT_DBG(muxer_comp);
+ BT_ASSERT_DBG(muxer_msg_iter);
+ BT_ASSERT_DBG(muxer_upstream_msg_iter);
+ *muxer_upstream_msg_iter = NULL;
-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;
+ for (i = 0; i < muxer_msg_iter->active_muxer_upstream_msg_iters->len;
+ i++) {
+ const bt_message *msg;
+ struct muxer_upstream_msg_iter *cur_muxer_upstream_msg_iter =
+ g_ptr_array_index(
+ muxer_msg_iter->active_muxer_upstream_msg_iters,
+ i);
+ int64_t msg_ts_ns;
- left_event_class = bt_event_borrow_class_const(left_event);
- right_event_class = bt_event_borrow_class_const(right_event);
+ if (!cur_muxer_upstream_msg_iter->msg_iter) {
+ /* This upstream message iterator is ended */
+ BT_COMP_LOGT("Skipping ended upstream message iterator: "
+ "muxer-upstream-msg-iter-wrap-addr=%p",
+ cur_muxer_upstream_msg_iter);
+ continue;
+ }
- left_event_class_id = bt_event_class_get_id(left_event_class);
- right_event_class_id = bt_event_class_get_id(right_event_class);
+ BT_ASSERT_DBG(cur_muxer_upstream_msg_iter->msgs->length > 0);
+ msg = g_queue_peek_head(cur_muxer_upstream_msg_iter->msgs);
+ BT_ASSERT_DBG(msg);
- 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;
- }
+ if (G_UNLIKELY(bt_message_get_type(msg) ==
+ BT_MESSAGE_TYPE_STREAM_BEGINNING)) {
+ ret = validate_new_stream_clock_class(
+ muxer_msg_iter, muxer_comp,
+ bt_message_stream_beginning_borrow_stream_const(
+ msg));
+ if (ret) {
+ /*
+ * validate_new_stream_clock_class() logs
+ * errors.
+ */
+ status = BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_ERROR;
+ goto end;
+ }
+ } else if (G_UNLIKELY(bt_message_get_type(msg) ==
+ BT_MESSAGE_TYPE_MESSAGE_ITERATOR_INACTIVITY)) {
+ const bt_clock_snapshot *cs;
- 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;
+ 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));
+ if (ret) {
+ /* validate_clock_class() logs errors */
+ status = BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_ERROR;
+ 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;
+ ret = get_msg_ts_ns(muxer_comp, muxer_msg_iter, msg,
+ muxer_msg_iter->last_returned_ts_ns, &msg_ts_ns);
if (ret) {
+ /* get_msg_ts_ns() logs errors */
+ *muxer_upstream_msg_iter = NULL;
+ status = BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_ERROR;
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
- * message iterator which has it, or
- * BT_MESSAGE_ITERATOR_STATUS_END if there's no available
- * message.
- *
- * This function does NOT:
- *
- * * Update any upstream message iterator.
- * * Check the upstream message iterators to retry.
- *
- * On sucess, this function sets *muxer_upstream_msg_iter to the
- * upstream message iterator of which the current message is
- * the youngest, and sets *ts_ns to its time.
- */
-static
-bt_component_class_message_iterator_next_method_status
-muxer_msg_iter_youngest_upstream_msg_iter(
- struct muxer_comp *muxer_comp,
- struct muxer_msg_iter *muxer_msg_iter,
- struct muxer_upstream_msg_iter **muxer_upstream_msg_iter,
- int64_t *ts_ns)
-{
- size_t i;
- int ret;
- int64_t youngest_ts_ns = INT64_MAX;
- bt_component_class_message_iterator_next_method_status status =
- BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_OK;
-
- BT_ASSERT(muxer_comp);
- BT_ASSERT(muxer_msg_iter);
- BT_ASSERT(muxer_upstream_msg_iter);
- *muxer_upstream_msg_iter = NULL;
-
- for (i = 0; i < muxer_msg_iter->active_muxer_upstream_msg_iters->len;
- i++) {
- const bt_message *msg;
- struct muxer_upstream_msg_iter *cur_muxer_upstream_msg_iter =
- g_ptr_array_index(
- muxer_msg_iter->active_muxer_upstream_msg_iters,
- i);
- int64_t msg_ts_ns;
-
- if (!cur_muxer_upstream_msg_iter->msg_iter) {
- /* This upstream message iterator is ended */
- BT_COMP_LOGT("Skipping ended upstream message iterator: "
- "muxer-upstream-msg-iter-wrap-addr=%p",
- cur_muxer_upstream_msg_iter);
- continue;
- }
-
- BT_ASSERT(cur_muxer_upstream_msg_iter->msgs->length > 0);
- msg = g_queue_peek_head(cur_muxer_upstream_msg_iter->msgs);
- BT_ASSERT(msg);
-
- if (G_UNLIKELY(bt_message_get_type(msg) ==
- BT_MESSAGE_TYPE_STREAM_BEGINNING)) {
- ret = validate_new_stream_clock_class(
- muxer_msg_iter, muxer_comp,
- bt_message_stream_beginning_borrow_stream_const(
- msg));
- if (ret) {
- /*
- * validate_new_stream_clock_class() logs
- * errors.
- */
- status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_ERROR;
- goto end;
- }
- } else if (G_UNLIKELY(bt_message_get_type(msg) ==
- BT_MESSAGE_TYPE_MESSAGE_ITERATOR_INACTIVITY)) {
- const bt_clock_snapshot *cs;
-
- cs = bt_message_message_iterator_inactivity_borrow_default_clock_snapshot_const(
- msg);
- ret = validate_clock_class(muxer_msg_iter, muxer_comp,
- bt_clock_snapshot_borrow_clock_class_const(cs));
- if (ret) {
- /* validate_clock_class() logs errors */
- status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_ERROR;
- goto end;
- }
- }
-
- ret = get_msg_ts_ns(muxer_comp, muxer_msg_iter, msg,
- muxer_msg_iter->last_returned_ts_ns, &msg_ts_ns);
- if (ret) {
- /* get_msg_ts_ns() logs errors */
- *muxer_upstream_msg_iter = NULL;
- status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_ERROR;
- goto end;
- }
-
- if (msg_ts_ns < youngest_ts_ns) {
- *muxer_upstream_msg_iter =
- cur_muxer_upstream_msg_iter;
- youngest_ts_ns = msg_ts_ns;
- *ts_ns = youngest_ts_ns;
- } else if (msg_ts_ns == youngest_ts_ns) {
- /*
- * The currently selected message to be sent downstream
- * next has the exact same timestamp that of the
- * current candidate message. We must break the tie
- * in a predictable manner.
- */
- const bt_message *selected_msg = g_queue_peek_head(
- (*muxer_upstream_msg_iter)->msgs);
- BT_COMP_LOGD_STR("Two of the next message candidates have the same timestamps, pick one deterministically.");
-
- /*
- * Order the messages in an arbitrary but determinitic
- * way.
- */
- ret = compare_messages(msg, selected_msg);
- if (ret < 0) {
- /*
- * The `msg` should go first. Update the next
- * iterator and the current timestamp.
- */
- *muxer_upstream_msg_iter =
- cur_muxer_upstream_msg_iter;
- youngest_ts_ns = msg_ts_ns;
- *ts_ns = youngest_ts_ns;
- } else if (ret == 0) {
- /* Unable to pick which one should go first. */
- BT_COMP_LOGW("Cannot deterministically pick next upstream message iterator because they have identical next messages: "
- "muxer-upstream-msg-iter-wrap-addr=%p"
- "cur-muxer-upstream-msg-iter-wrap-addr=%p",
- *muxer_upstream_msg_iter,
- cur_muxer_upstream_msg_iter);
- }
+ /*
+ * 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;
+ *ts_ns = youngest_ts_ns;
+ } else if (msg_ts_ns == youngest_ts_ns) {
+ /*
+ * The currently selected message to be sent downstream
+ * next has the exact same timestamp that of the
+ * current candidate message. We must break the tie
+ * in a predictable manner.
+ */
+ const bt_message *selected_msg = g_queue_peek_head(
+ (*muxer_upstream_msg_iter)->msgs);
+ BT_COMP_LOGD_STR("Two of the next message candidates have the same timestamps, pick one deterministically.");
+
+ /*
+ * Order the messages in an arbitrary but determinitic
+ * way.
+ */
+ ret = common_muxing_compare_messages(msg, selected_msg);
+ if (ret < 0) {
+ /*
+ * The `msg` should go first. Update the next
+ * iterator and the current timestamp.
+ */
+ *muxer_upstream_msg_iter =
+ cur_muxer_upstream_msg_iter;
+ youngest_ts_ns = msg_ts_ns;
+ *ts_ns = youngest_ts_ns;
+ } else if (ret == 0) {
+ /* Unable to pick which one should go first. */
+ BT_COMP_LOGW("Cannot deterministically pick next upstream message iterator because they have identical next messages: "
+ "muxer-upstream-msg-iter-wrap-addr=%p"
+ "cur-muxer-upstream-msg-iter-wrap-addr=%p",
+ *muxer_upstream_msg_iter,
+ cur_muxer_upstream_msg_iter);
+ }
}
}
if (!*muxer_upstream_msg_iter) {
- status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_END;
+ status = BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_END;
*ts_ns = INT64_MIN;
}
}
static
-bt_component_class_message_iterator_next_method_status
+bt_message_iterator_class_next_method_status
validate_muxer_upstream_msg_iter(
struct muxer_upstream_msg_iter *muxer_upstream_msg_iter,
bool *is_ended)
{
- struct muxer_comp *muxer_comp =
- muxer_upstream_msg_iter->muxer_comp;
- bt_component_class_message_iterator_next_method_status status =
- BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_OK;
+ struct muxer_comp *muxer_comp = muxer_upstream_msg_iter->muxer_comp;
+ bt_message_iterator_class_next_method_status status;
BT_COMP_LOGD("Validating muxer's upstream message iterator wrapper: "
"muxer-upstream-msg-iter-wrap-addr=%p",
"queue-len=%u, upstream-msg-iter-addr=%p",
muxer_upstream_msg_iter->msgs->length,
muxer_upstream_msg_iter->msg_iter);
+ status = BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_OK;
goto end;
}
}
static
-bt_component_class_message_iterator_next_method_status
+bt_message_iterator_class_next_method_status
validate_muxer_upstream_msg_iters(
struct muxer_msg_iter *muxer_msg_iter)
{
struct muxer_comp *muxer_comp = muxer_msg_iter->muxer_comp;
- bt_component_class_message_iterator_next_method_status status =
- BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_OK;
+ bt_message_iterator_class_next_method_status status;
size_t i;
BT_COMP_LOGD("Validating muxer's upstream message iterator wrappers: "
status = validate_muxer_upstream_msg_iter(
muxer_upstream_msg_iter, &is_ended);
- if (status != BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_OK) {
+ if (status != BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_OK) {
if (status < 0) {
- BT_COMP_LOGE("Cannot validate muxer's upstream message iterator wrapper: "
+ BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+ "Cannot validate muxer's upstream message iterator wrapper: "
"muxer-msg-iter-addr=%p, "
"muxer-upstream-msg-iter-wrap-addr=%p",
muxer_msg_iter,
}
}
+ status = BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_OK;
+
end:
return status;
}
static inline
-bt_component_class_message_iterator_next_method_status muxer_msg_iter_do_next_one(
+bt_message_iterator_class_next_method_status muxer_msg_iter_do_next_one(
struct muxer_comp *muxer_comp,
struct muxer_msg_iter *muxer_msg_iter,
const bt_message **msg)
{
- bt_component_class_message_iterator_next_method_status status;
+ 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_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_OK) {
+ if (status != BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_OK) {
/* validate_muxer_upstream_msg_iters() logs details */
goto end;
}
status = muxer_msg_iter_youngest_upstream_msg_iter(muxer_comp,
muxer_msg_iter, &muxer_upstream_msg_iter,
&next_return_ts);
- if (status < 0 || status == BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_END) {
+ if (status < 0 || status == BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_END) {
if (status < 0) {
- BT_COMP_LOGE("Cannot find the youngest upstream message iterator wrapper: "
+ BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+ "Cannot find the youngest upstream message iterator wrapper: "
"status=%s",
bt_common_func_status_string(status));
} else {
}
if (next_return_ts < muxer_msg_iter->last_returned_ts_ns) {
- BT_COMP_LOGE("Youngest upstream message iterator wrapper's timestamp is less than muxer's message iterator's last returned timestamp: "
+ BT_COMP_LOGE_APPEND_CAUSE(muxer_comp->self_comp,
+ "Youngest upstream message iterator wrapper's timestamp is less than muxer's message iterator's last returned timestamp: "
"muxer-msg-iter-addr=%p, ts=%" PRId64 ", "
"last-returned-ts=%" PRId64,
muxer_msg_iter, next_return_ts,
muxer_msg_iter->last_returned_ts_ns);
- status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_ERROR;
+ status = BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_ERROR;
goto end;
}
"muxer-upstream-msg-iter-wrap-addr=%p, "
"ts=%" PRId64,
muxer_msg_iter, muxer_upstream_msg_iter, next_return_ts);
- BT_ASSERT(status == BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_OK);
- BT_ASSERT(muxer_upstream_msg_iter);
+ BT_ASSERT_DBG(status ==
+ BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_OK);
+ BT_ASSERT_DBG(muxer_upstream_msg_iter);
/*
* Consume from the queue's head: other side
* (muxer_upstream_msg_iter_next()) writes to the tail.
*/
*msg = g_queue_pop_head(muxer_upstream_msg_iter->msgs);
- BT_ASSERT(*msg);
+ BT_ASSERT_DBG(*msg);
muxer_msg_iter->last_returned_ts_ns = next_return_ts;
end:
}
static
-bt_component_class_message_iterator_next_method_status muxer_msg_iter_do_next(
+bt_message_iterator_class_next_method_status muxer_msg_iter_do_next(
struct muxer_comp *muxer_comp,
struct muxer_msg_iter *muxer_msg_iter,
bt_message_array_const msgs, uint64_t capacity,
uint64_t *count)
{
- bt_component_class_message_iterator_next_method_status status =
- BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_OK;
+ bt_message_iterator_class_next_method_status status;
uint64_t i = 0;
- while (i < capacity && status == BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_OK) {
+ if (G_UNLIKELY(muxer_msg_iter->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(muxer_msg_iter->next_saved_error);
+ status = muxer_msg_iter->next_saved_status;
+ goto end;
+ }
+
+ do {
status = muxer_msg_iter_do_next_one(muxer_comp,
muxer_msg_iter, &msgs[i]);
- if (status == BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_OK) {
+ 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) {
/*
* called, possibly without any accumulated
* message, in which case we'll return it.
*/
+ 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.
+ */
+ muxer_msg_iter->next_saved_error = bt_current_thread_take_error();
+ BT_ASSERT(muxer_msg_iter->next_saved_error);
+ muxer_msg_iter->next_saved_status = status;
+ }
+
*count = i;
- status = BT_COMPONENT_CLASS_MESSAGE_ITERATOR_NEXT_METHOD_STATUS_OK;
+ status = BT_MESSAGE_ITERATOR_CLASS_NEXT_METHOD_STATUS_OK;
}
+end:
return status;
}
}
static
-int muxer_msg_iter_init_upstream_iterators(struct muxer_comp *muxer_comp,
- struct muxer_msg_iter *muxer_msg_iter)
+bt_message_iterator_class_initialize_method_status
+muxer_msg_iter_init_upstream_iterators(struct muxer_comp *muxer_comp,
+ struct muxer_msg_iter *muxer_msg_iter,
+ struct bt_self_message_iterator_configuration *config)
{
int64_t count;
int64_t i;
- int ret = 0;
+ bt_message_iterator_class_initialize_method_status status;
+ bool can_seek_forward = true;
count = bt_component_filter_get_input_port_count(
bt_self_component_filter_as_component_filter(
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_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_OK;
goto end;
}
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_message_iterator_create_from_message_iterator_status
+ msg_iter_status;
+ int int_status;
BT_ASSERT(self_port);
port = bt_self_component_port_as_port(
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_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(
+ bt_message_iterator_put_ref(
upstream_msg_iter);
- if (ret) {
+ if (int_status) {
+ status = BT_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_ERROR;
/* muxer_msg_iter_add_upstream_msg_iter() logs errors */
goto end;
}
+
+ can_seek_forward = can_seek_forward &&
+ bt_message_iterator_can_seek_forward(
+ upstream_msg_iter);
}
+ /*
+ * This iterator can seek forward if all of its iterators can seek
+ * forward.
+ */
+ bt_self_message_iterator_configuration_set_can_seek_forward(
+ config, can_seek_forward);
+
+ status = BT_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_OK;
+
end:
- return ret;
+ return status;
}
BT_HIDDEN
-bt_component_class_message_iterator_init_method_status muxer_msg_iter_init(
+bt_message_iterator_class_initialize_method_status muxer_msg_iter_init(
bt_self_message_iterator *self_msg_iter,
- bt_self_component_filter *self_comp,
+ bt_self_message_iterator_configuration *config,
bt_self_component_port_output *port)
{
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_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(
- bt_self_component_filter_as_self_component(self_comp));
+ muxer_comp = bt_self_component_get_data(self_comp);
BT_ASSERT(muxer_comp);
BT_COMP_LOGD("Initializing muxer component's message iterator: "
"comp-addr=%p, muxer-comp-addr=%p, msg-iter-addr=%p",
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_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_ERROR;
goto error;
}
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_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_MEMORY_ERROR;
goto error;
}
(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_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_MEMORY_ERROR;
goto error;
}
(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_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_MEMORY_ERROR;
goto error;
}
- ret = muxer_msg_iter_init_upstream_iterators(muxer_comp,
- muxer_msg_iter);
- if (ret) {
+ 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: "
"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;
}
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;
}
BT_HIDDEN
-bt_component_class_message_iterator_next_method_status muxer_msg_iter_next(
+bt_message_iterator_class_next_method_status muxer_msg_iter_next(
bt_self_message_iterator *self_msg_iter,
bt_message_array_const msgs, uint64_t capacity,
uint64_t *count)
{
- bt_component_class_message_iterator_next_method_status status;
+ bt_message_iterator_class_next_method_status status;
struct muxer_msg_iter *muxer_msg_iter =
bt_self_message_iterator_get_data(self_msg_iter);
bt_self_component *self_comp = NULL;
struct muxer_comp *muxer_comp = NULL;
- BT_ASSERT(muxer_msg_iter);
+ BT_ASSERT_DBG(muxer_msg_iter);
self_comp = bt_self_message_iterator_borrow_component(
self_msg_iter);
- BT_ASSERT(self_comp);
+ BT_ASSERT_DBG(self_comp);
muxer_comp = bt_self_component_get_data(self_comp);
- BT_ASSERT(muxer_comp);
+ BT_ASSERT_DBG(muxer_comp);
BT_COMP_LOGT("Muxer component's message iterator's \"next\" method called: "
"comp-addr=%p, muxer-comp-addr=%p, muxer-msg-iter-addr=%p, "
"msg-iter-addr=%p",
}
static inline
-bt_bool muxer_upstream_msg_iters_can_all_seek_beginning(
- GPtrArray *muxer_upstream_msg_iters)
+bt_message_iterator_class_can_seek_beginning_method_status
+muxer_upstream_msg_iters_can_all_seek_beginning(
+ GPtrArray *muxer_upstream_msg_iters, bt_bool *can_seek)
{
+ bt_message_iterator_class_can_seek_beginning_method_status status =
+ BT_MESSAGE_ITERATOR_CLASS_CAN_SEEK_BEGINNING_METHOD_STATUS_OK;
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_message_iterator_can_seek_beginning(
+ upstream_msg_iter->msg_iter, can_seek);
+ if (status != BT_MESSAGE_ITERATOR_CLASS_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_message_iterator_class_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_message_iterator_class_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_MESSAGE_ITERATOR_CLASS_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
-bt_component_class_message_iterator_seek_beginning_method_status muxer_msg_iter_seek_beginning(
+bt_message_iterator_class_seek_beginning_method_status muxer_msg_iter_seek_beginning(
bt_self_message_iterator *self_msg_iter)
{
struct muxer_msg_iter *muxer_msg_iter =
bt_self_message_iterator_get_data(self_msg_iter);
- bt_component_class_message_iterator_seek_beginning_method_status status =
- BT_COMPONENT_CLASS_MESSAGE_ITERATOR_SEEK_BEGINNING_METHOD_STATUS_OK;
+ bt_message_iterator_class_seek_beginning_method_status status =
+ BT_MESSAGE_ITERATOR_CLASS_SEEK_BEGINNING_METHOD_STATUS_OK;
bt_message_iterator_seek_beginning_status seek_beg_status;
uint64_t i;
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;
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;
projects / babeltrace.git / blobdiff