* SOFTWARE.
*/
+#define BT_LOG_TAG "CONNECTION"
+#include <babeltrace/lib-logging-internal.h>
+
#include <babeltrace/graph/notification-iterator-internal.h>
#include <babeltrace/graph/component-internal.h>
#include <babeltrace/graph/component-source-internal.h>
{
struct bt_connection *connection = container_of(obj,
struct bt_connection, base);
+ size_t i;
+
+ BT_LOGD("Destroying connection: addr=%p", connection);
+
+ /*
+ * Make sure that each notification iterator which was created
+ * for this connection is finalized before we destroy it. Once a
+ * notification iterator is finalized, all its method return
+ * NULL or the BT_NOTIFICATION_ITERATOR_STATUS_CANCELED status.
+ *
+ * Because connections are destroyed before components within a
+ * graph, this ensures that notification iterators are always
+ * finalized before their upstream component.
+ */
+ if (connection->iterators) {
+ for (i = 0; i < connection->iterators->len; i++) {
+ struct bt_notification_iterator *iterator =
+ g_ptr_array_index(connection->iterators, i);
+
+ BT_LOGD("Finalizing notification iterator created by this connection: "
+ "iter-addr=%p", iterator);
+ bt_notification_iterator_finalize(iterator);
+
+ /*
+ * Make sure this iterator does not try to
+ * remove itself from this connection's
+ * iterators on destruction because this
+ * connection won't exist anymore.
+ */
+ bt_notification_iterator_set_connection(iterator,
+ NULL);
+ }
+
+ g_ptr_array_free(connection->iterators, TRUE);
+ }
/*
* No bt_put on ports as a connection only holds _weak_ references
g_free(connection);
}
+static
+void bt_connection_try_remove_from_graph(struct bt_connection *connection)
+{
+ void *graph = bt_object_borrow_parent(&connection->base);
+
+ if (connection->base.ref_count.count > 0 ||
+ connection->downstream_port ||
+ connection->upstream_port ||
+ connection->iterators->len > 0) {
+ return;
+ }
+
+ /*
+ * At this point we know that:
+ *
+ * 1. The connection is ended (ports were disconnected).
+ * 2. All the notification iterators that this connection
+ * created, if any, are finalized.
+ * 3. The connection's reference count is 0, so only the
+ * parent (graph) owns this connection after this call.
+ *
+ * In other words, no other object than the graph knows this
+ * connection.
+ *
+ * It is safe to remove the connection from the graph, therefore
+ * destroying it.
+ */
+ BT_LOGD("Removing self from graph's connections: "
+ "graph-addr=%p, conn-addr=%p", graph, connection);
+ bt_graph_remove_connection(graph, connection);
+}
+
+static
+void bt_connection_parent_is_owner(struct bt_object *obj)
+{
+ struct bt_connection *connection = container_of(obj,
+ struct bt_connection, base);
+
+ bt_connection_try_remove_from_graph(connection);
+}
+
struct bt_connection *bt_connection_from_private_connection(
struct bt_private_connection *private_connection)
{
struct bt_connection *connection = NULL;
if (bt_port_get_type(upstream_port) != BT_PORT_TYPE_OUTPUT) {
+ BT_LOGW_STR("Invalid parameter: upstream port is not an output port.");
goto end;
}
if (bt_port_get_type(downstream_port) != BT_PORT_TYPE_INPUT) {
+ BT_LOGW_STR("Invalid parameter: downstream port is not an input port.");
goto end;
}
+ BT_LOGD("Creating connection: "
+ "graph-addr=%p, upstream-port-addr=%p, uptream-port-name=\"%s\", "
+ "downstream-port-addr=%p, downstream-port-name=\"%s\"",
+ graph, upstream_port, bt_port_get_name(upstream_port),
+ downstream_port, bt_port_get_name(downstream_port));
connection = g_new0(struct bt_connection, 1);
if (!connection) {
+ BT_LOGE_STR("Failed to allocate one connection.");
goto end;
}
bt_object_init(connection, bt_connection_destroy);
+ bt_object_set_parent_is_owner_listener(connection,
+ bt_connection_parent_is_owner);
+ connection->iterators = g_ptr_array_new();
+ if (!connection->iterators) {
+ BT_LOGE_STR("Failed to allocate a GPtrArray.");
+ BT_PUT(connection);
+ goto end;
+ }
+
/* Weak references are taken, see comment in header. */
connection->upstream_port = upstream_port;
connection->downstream_port = downstream_port;
+ BT_LOGD_STR("Setting upstream port's connection.");
bt_port_set_connection(upstream_port, connection);
+ BT_LOGD_STR("Setting downstream port's connection.");
bt_port_set_connection(downstream_port, connection);
bt_object_set_parent(connection, &graph->base);
+ BT_LOGD("Created connection: "
+ "graph-addr=%p, upstream-port-addr=%p, uptream-port-name=\"%s\", "
+ "downstream-port-addr=%p, downstream-port-name=\"%s\", "
+ "conn-addr=%p",
+ graph, upstream_port, bt_port_get_name(upstream_port),
+ downstream_port, bt_port_get_name(downstream_port),
+ connection);
+
end:
return connection;
}
struct bt_component *upstream_comp = NULL;
struct bt_port *downstream_port = conn->downstream_port;
struct bt_port *upstream_port = conn->upstream_port;
- struct bt_graph *graph = (void *) bt_object_get_parent(conn);
+ struct bt_graph *graph = (void *) bt_object_borrow_parent(conn);
+ size_t i;
if (downstream_port) {
downstream_comp = bt_port_get_component(downstream_port);
}
if (downstream_comp) {
+ /* bt_component_port_disconnected() logs details */
bt_component_port_disconnected(downstream_comp,
downstream_port);
}
if (upstream_comp) {
+ /* bt_component_port_disconnected() logs details */
bt_component_port_disconnected(upstream_comp, upstream_port);
}
assert(graph);
+ /* bt_graph_notify_ports_disconnected() logs details */
bt_graph_notify_ports_disconnected(graph, upstream_comp,
downstream_comp, upstream_port, downstream_port);
bt_put(downstream_comp);
bt_put(upstream_comp);
- bt_put(graph);
+
+ /*
+ * Because this connection is ended, finalize (cancel) each
+ * notification iterator created from it.
+ */
+ for (i = 0; i < conn->iterators->len; i++) {
+ struct bt_notification_iterator *iterator =
+ g_ptr_array_index(conn->iterators, i);
+
+ BT_LOGD("Finalizing notification iterator created by this ended connection: "
+ "conn-addr=%p, iter-addr=%p", conn, iterator);
+ bt_notification_iterator_finalize(iterator);
+
+ /*
+ * Make sure this iterator does not try to remove itself
+ * from this connection's iterators on destruction
+ * because this connection won't exist anymore.
+ */
+ bt_notification_iterator_set_connection(iterator,
+ NULL);
+ }
+
+ g_ptr_array_set_size(conn->iterators, 0);
+ bt_connection_try_remove_from_graph(conn);
}
struct bt_port *bt_connection_get_upstream_port(
struct bt_notification_iterator *
bt_private_connection_create_notification_iterator(
- struct bt_private_connection *private_connection)
+ struct bt_private_connection *private_connection,
+ const enum bt_notification_type *notification_types)
{
- enum bt_notification_iterator_status ret_iterator;
enum bt_component_class_type upstream_comp_class_type;
struct bt_notification_iterator *iterator = NULL;
struct bt_port *upstream_port = NULL;
struct bt_component_class *upstream_comp_class = NULL;
struct bt_connection *connection = NULL;
bt_component_class_notification_iterator_init_method init_method = NULL;
+ static const enum bt_notification_type all_notif_types[] = {
+ BT_NOTIFICATION_TYPE_ALL,
+ BT_NOTIFICATION_TYPE_SENTINEL,
+ };
if (!private_connection) {
+ BT_LOGW_STR("Invalid parameter: private connection is NULL.");
goto error;
}
- connection = bt_connection_from_private(private_connection);
+ if (!notification_types) {
+ BT_LOGD_STR("No notification types: subscribing to all notifications.");
+ notification_types = all_notif_types;
+ }
+ connection = bt_connection_from_private(private_connection);
if (!connection->upstream_port || !connection->downstream_port) {
+ BT_LOGW("Invalid parameter: connection is ended: "
+ "conn-addr=%p", connection);
goto error;
}
upstream_component = bt_port_get_component(upstream_port);
assert(upstream_component);
upstream_comp_class = upstream_component->class;
-
- if (!upstream_component) {
- goto error;
- }
-
+ BT_LOGD("Creating notification iterator from connection: "
+ "conn-addr=%p, upstream-port-addr=%p, "
+ "upstream-port-name=\"%s\", upstream-comp-addr=%p, "
+ "upstream-comp-name=\"%s\"",
+ connection, connection->upstream_port,
+ bt_port_get_name(connection->upstream_port),
+ upstream_component, bt_component_get_name(upstream_component));
upstream_comp_class_type =
bt_component_get_class_type(upstream_component);
if (upstream_comp_class_type != BT_COMPONENT_CLASS_TYPE_SOURCE &&
upstream_comp_class_type != BT_COMPONENT_CLASS_TYPE_FILTER) {
/* Unsupported operation. */
+ BT_LOGW("Upstream component's class is not a source or filter component class: "
+ "comp-class-type=%s",
+ bt_component_class_type_string(upstream_comp_class_type));
goto error;
}
iterator = bt_notification_iterator_create(upstream_component,
- upstream_port);
+ upstream_port, notification_types, connection);
if (!iterator) {
+ BT_LOGW("Cannot create notification iterator from connection.");
goto error;
}
}
if (init_method) {
- enum bt_notification_iterator_status status = init_method(
+ enum bt_notification_iterator_status status;
+
+ BT_LOGD("Calling user's initialization method: iter-addr=%p",
+ iterator);
+ status = init_method(
bt_private_notification_iterator_from_notification_iterator(iterator),
bt_private_port_from_port(upstream_port));
+ BT_LOGD("User method returned: status=%s",
+ bt_notification_iterator_status_string(status));
if (status < 0) {
+ BT_LOGW_STR("Initialization method failed.");
goto error;
}
}
- ret_iterator = bt_notification_iterator_validate(iterator);
- if (ret_iterator != BT_NOTIFICATION_ITERATOR_STATUS_OK) {
- goto error;
- }
-
+ g_ptr_array_add(connection->iterators, iterator);
+ BT_LOGD("Created notification iterator from connection: "
+ "conn-addr=%p, upstream-port-addr=%p, "
+ "upstream-port-name=\"%s\", upstream-comp-addr=%p, "
+ "upstream-comp-name=\"%s\", iter-addr=%p",
+ connection, connection->upstream_port,
+ bt_port_get_name(connection->upstream_port),
+ upstream_component, bt_component_get_name(upstream_component),
+ iterator);
goto end;
error:
bt_put(upstream_component);
return iterator;
}
+
+BT_HIDDEN
+void bt_connection_remove_iterator(struct bt_connection *conn,
+ struct bt_notification_iterator *iterator)
+{
+ g_ptr_array_remove(conn->iterators, iterator);
+ BT_LOGV("Removed notification iterator from connection: "
+ "conn-addr=%p, iter-addr=%p", conn, iterator);
+ bt_connection_try_remove_from_graph(conn);
+}