lib: add bt_{graph,query_executor}_add_interrupter()
[babeltrace.git] / src / lib / graph / graph.c
index 734ebb7ea1906c077356936efed93aefc2423f19..192ec9e59489e01e5c61bdacbb05609224e4aa39 100644 (file)
@@ -26,6 +26,7 @@
 
 #include "common/assert.h"
 #include "lib/assert-pre.h"
+#include "lib/assert-post.h"
 #include <babeltrace2/graph/graph.h>
 #include <babeltrace2/graph/graph-const.h>
 #include <babeltrace2/graph/component-source-const.h>
 #include "component-sink.h"
 #include "connection.h"
 #include "graph.h"
+#include "interrupter.h"
 #include "message/event.h"
 #include "message/packet.h"
 
-typedef enum bt_graph_listener_status (*port_added_func_t)(
-               const void *, const void *, void *);
+typedef enum bt_graph_listener_func_status
+(*port_added_func_t)(const void *, const void *, void *);
 
-typedef enum bt_graph_listener_status (*ports_connected_func_t)(
-               const void *, const void *, const void *, const void *, void *);
-
-typedef enum bt_self_component_status (*comp_init_method_t)(const void *,
+typedef enum bt_graph_listener_func_status
+(*ports_connected_func_t)(const void *, const void *, const void *,
                const void *, void *);
 
+typedef enum bt_component_class_init_method_status
+(*comp_init_method_t)(const void *, const void *, void *);
+
 struct bt_graph_listener {
        bt_graph_listener_removed_func removed;
        void *data;
@@ -76,7 +79,8 @@ struct bt_graph_listener_ports_connected {
        do {                                                            \
                _listeners = g_array_new(FALSE, TRUE, sizeof(_type));   \
                if (!(_listeners)) {                                    \
-                       BT_LOGE_STR("Failed to allocate one GArray.");  \
+                       BT_LIB_LOGE_APPEND_CAUSE(                       \
+                               "Failed to allocate one GArray.");      \
                }                                                       \
        } while (0)
 
@@ -130,12 +134,7 @@ void destroy_graph(struct bt_object *obj)
         */
        BT_LIB_LOGI("Destroying graph: %!+g", graph);
        obj->ref_count++;
-
-       /*
-        * Cancel the graph to disallow some operations, like creating
-        * message iterators and adding ports to components.
-        */
-       (void) bt_graph_cancel((void *) graph);
+       graph->config_state = BT_GRAPH_CONFIGURATION_STATE_DESTROYING;
 
        /* Call all remove listeners */
        CALL_REMOVE_LISTENERS(struct bt_graph_listener_port_added,
@@ -172,6 +171,14 @@ void destroy_graph(struct bt_object *obj)
                graph->components = NULL;
        }
 
+       if (graph->interrupters) {
+               BT_LOGD_STR("Putting interrupters.");
+               g_ptr_array_free(graph->interrupters, TRUE);
+               graph->interrupters = NULL;
+       }
+
+       BT_OBJECT_PUT_REF_AND_RESET(graph->default_interrupter);
+
        if (graph->sinks_to_consume) {
                g_queue_free(graph->sinks_to_consume);
                graph->sinks_to_consume = NULL;
@@ -262,7 +269,7 @@ struct bt_graph *bt_graph_create(void)
        BT_LOGI_STR("Creating graph object.");
        graph = g_new0(struct bt_graph, 1);
        if (!graph) {
-               BT_LOGE_STR("Failed to allocate one graph.");
+               BT_LIB_LOGE_APPEND_CAUSE("Failed to allocate one graph.");
                goto end;
        }
 
@@ -270,18 +277,18 @@ struct bt_graph *bt_graph_create(void)
        graph->connections = g_ptr_array_new_with_free_func(
                (GDestroyNotify) bt_object_try_spec_release);
        if (!graph->connections) {
-               BT_LOGE_STR("Failed to allocate one GPtrArray.");
+               BT_LIB_LOGE_APPEND_CAUSE("Failed to allocate one GPtrArray.");
                goto error;
        }
        graph->components = g_ptr_array_new_with_free_func(
                (GDestroyNotify) bt_object_try_spec_release);
        if (!graph->components) {
-               BT_LOGE_STR("Failed to allocate one GPtrArray.");
+               BT_LIB_LOGE_APPEND_CAUSE("Failed to allocate one GPtrArray.");
                goto error;
        }
        graph->sinks_to_consume = g_queue_new();
        if (!graph->sinks_to_consume) {
-               BT_LOGE_STR("Failed to allocate one GQueue.");
+               BT_LIB_LOGE_APPEND_CAUSE("Failed to allocate one GQueue.");
                goto error;
        }
 
@@ -350,12 +357,28 @@ struct bt_graph *bt_graph_create(void)
                goto error;
        }
 
+       graph->interrupters = g_ptr_array_new_with_free_func(
+               (GDestroyNotify) bt_object_put_no_null_check);
+       if (!graph->interrupters) {
+               BT_LIB_LOGE_APPEND_CAUSE("Failed to allocate one GPtrArray.");
+               goto error;
+       }
+
+       graph->default_interrupter = bt_interrupter_create();
+       if (!graph->default_interrupter) {
+               BT_LIB_LOGE_APPEND_CAUSE(
+                       "Failed to create one interrupter object.");
+               goto error;
+       }
+
+       bt_graph_add_interrupter(graph, graph->default_interrupter);
        ret = bt_object_pool_initialize(&graph->event_msg_pool,
                (bt_object_pool_new_object_func) bt_message_event_new,
                (bt_object_pool_destroy_object_func) destroy_message_event,
                graph);
        if (ret) {
-               BT_LOGE("Failed to initialize event message pool: ret=%d",
+               BT_LIB_LOGE_APPEND_CAUSE(
+                       "Failed to initialize event message pool: ret=%d",
                        ret);
                goto error;
        }
@@ -365,7 +388,8 @@ struct bt_graph *bt_graph_create(void)
                (bt_object_pool_destroy_object_func) destroy_message_packet_begin,
                graph);
        if (ret) {
-               BT_LOGE("Failed to initialize packet beginning message pool: ret=%d",
+               BT_LIB_LOGE_APPEND_CAUSE(
+                       "Failed to initialize packet beginning message pool: ret=%d",
                        ret);
                goto error;
        }
@@ -375,7 +399,8 @@ struct bt_graph *bt_graph_create(void)
                (bt_object_pool_destroy_object_func) destroy_message_packet_end,
                graph);
        if (ret) {
-               BT_LOGE("Failed to initialize packet end message pool: ret=%d",
+               BT_LIB_LOGE_APPEND_CAUSE(
+                       "Failed to initialize packet end message pool: ret=%d",
                        ret);
                goto error;
        }
@@ -392,26 +417,25 @@ error:
        goto end;
 }
 
-enum bt_graph_status bt_graph_connect_ports(
+enum bt_graph_connect_ports_status bt_graph_connect_ports(
                struct bt_graph *graph,
                const struct bt_port_output *upstream_port_out,
                const struct bt_port_input *downstream_port_in,
                const struct bt_connection **user_connection)
 {
-       enum bt_graph_status status = BT_GRAPH_STATUS_OK;
-       enum bt_graph_listener_status listener_status;
+       enum bt_graph_connect_ports_status status = BT_FUNC_STATUS_OK;
+       enum bt_graph_listener_func_status listener_status;
        struct bt_connection *connection = NULL;
        struct bt_port *upstream_port = (void *) upstream_port_out;
        struct bt_port *downstream_port = (void *) downstream_port_in;
        struct bt_component *upstream_component = NULL;
        struct bt_component *downstream_component = NULL;
-       enum bt_self_component_status component_status;
+       enum bt_component_class_port_connected_method_status port_connected_status;
        bool init_can_consume;
 
        BT_ASSERT_PRE_NON_NULL(graph, "Graph");
        BT_ASSERT_PRE_NON_NULL(upstream_port, "Upstream port");
        BT_ASSERT_PRE_NON_NULL(downstream_port, "Downstream port port");
-       BT_ASSERT_PRE(!graph->canceled, "Graph is canceled: %!+g", graph);
        BT_ASSERT_PRE(
                graph->config_state == BT_GRAPH_CONFIGURATION_STATE_CONFIGURING,
                "Graph is not in the \"configuring\" state: %!+g", graph);
@@ -435,51 +459,12 @@ enum bt_graph_status bt_graph_connect_ports(
        downstream_component = bt_port_borrow_component_inline(
                (void *) downstream_port);
 
-       /*
-        * At this point the ports are not connected yet. Both
-        * components need to accept an eventual connection to their
-        * port by the other port before we continue.
-        */
-       BT_LIB_LOGD("Asking upstream component to accept the connection: "
-               "%![comp-]+c", upstream_component);
-       component_status = bt_component_accept_port_connection(
-               upstream_component, (void *) upstream_port,
-               (void *) downstream_port);
-       if (component_status != BT_SELF_COMPONENT_STATUS_OK) {
-               if (component_status == BT_SELF_COMPONENT_STATUS_REFUSE_PORT_CONNECTION) {
-                       BT_LOGD_STR("Upstream component refused the connection.");
-               } else {
-                       BT_LOGW("Cannot ask upstream component to accept the connection: "
-                               "status=%s", bt_self_component_status_string(component_status));
-               }
-
-               status = (int) component_status;
-               goto end;
-       }
-
-       BT_LIB_LOGD("Asking downstream component to accept the connection: "
-               "%![comp-]+c", downstream_component);
-       component_status = bt_component_accept_port_connection(
-               downstream_component, (void *) downstream_port,
-               (void *) upstream_port);
-       if (component_status != BT_SELF_COMPONENT_STATUS_OK) {
-               if (component_status == BT_SELF_COMPONENT_STATUS_REFUSE_PORT_CONNECTION) {
-                       BT_LOGD_STR("Downstream component refused the connection.");
-               } else {
-                       BT_LOGW("Cannot ask downstream component to accept the connection: "
-                               "status=%s", bt_self_component_status_string(component_status));
-               }
-
-               status = (int) component_status;
-               goto end;
-       }
-
        BT_LOGD_STR("Creating connection.");
        connection = bt_connection_create(graph, (void *) upstream_port,
                (void *) downstream_port);
        if (!connection) {
-               BT_LOGW("Cannot create connection object.");
-               status = BT_GRAPH_STATUS_NOMEM;
+               BT_LIB_LOGE_APPEND_CAUSE("Cannot create connection object.");
+               status = BT_FUNC_STATUS_MEMORY_ERROR;
                goto end;
        }
 
@@ -496,17 +481,22 @@ enum bt_graph_status bt_graph_connect_ports(
         */
        BT_LIB_LOGD("Notifying upstream component that its port is connected: "
                "%![comp-]+c, %![port-]+p", upstream_component, upstream_port);
-       component_status = bt_component_port_connected(upstream_component,
+       port_connected_status = bt_component_port_connected(upstream_component,
                (void *) upstream_port, (void *) downstream_port);
-       if (component_status != BT_SELF_COMPONENT_STATUS_OK) {
-               BT_LIB_LOGW("Error while notifying upstream component that its port is connected: "
-                       "status=%s, %![graph-]+g, %![up-comp-]+c, "
-                       "%![down-comp-]+c, %![up-port-]+p, %![down-port-]+p",
-                       bt_self_component_status_string(component_status),
-                       graph, upstream_component, downstream_component,
-                       upstream_port, downstream_port);
+       if (port_connected_status != BT_FUNC_STATUS_OK) {
+               if (port_connected_status < 0) {
+                       BT_LIB_LOGW_APPEND_CAUSE(
+                               "Upstream component's \"port connected\" method failed: "
+                               "status=%s, %![graph-]+g, %![up-comp-]+c, "
+                               "%![down-comp-]+c, %![up-port-]+p, %![down-port-]+p",
+                               bt_common_func_status_string(
+                                       port_connected_status),
+                               graph, upstream_component, downstream_component,
+                               upstream_port, downstream_port);
+               }
+
                bt_connection_end(connection, true);
-               status = (int) component_status;
+               status = (int) port_connected_status;
                goto end;
        }
 
@@ -514,17 +504,22 @@ enum bt_graph_status bt_graph_connect_ports(
        BT_LIB_LOGD("Notifying downstream component that its port is connected: "
                "%![comp-]+c, %![port-]+p", downstream_component,
                downstream_port);
-       component_status = bt_component_port_connected(downstream_component,
+       port_connected_status = bt_component_port_connected(downstream_component,
                (void *) downstream_port, (void *) upstream_port);
-       if (component_status != BT_SELF_COMPONENT_STATUS_OK) {
-               BT_LIB_LOGW("Error while notifying downstream component that its port is connected: "
-                       "status=%s, %![graph-]+g, %![up-comp-]+c, "
-                       "%![down-comp-]+c, %![up-port-]+p, %![down-port-]+p",
-                       bt_self_component_status_string(component_status),
-                       graph, upstream_component, downstream_component,
-                       upstream_port, downstream_port);
+       if (port_connected_status != BT_FUNC_STATUS_OK) {
+               if (port_connected_status < 0) {
+                       BT_LIB_LOGW_APPEND_CAUSE(
+                               "Downstream component's \"port connected\" method failed: "
+                               "status=%s, %![graph-]+g, %![up-comp-]+c, "
+                               "%![down-comp-]+c, %![up-port-]+p, %![down-port-]+p",
+                               bt_common_func_status_string(
+                                       port_connected_status),
+                               graph, upstream_component, downstream_component,
+                               upstream_port, downstream_port);
+               }
+
                bt_connection_end(connection, true);
-               status = (int) component_status;
+               status = (int) port_connected_status;
                goto end;
        }
 
@@ -535,7 +530,17 @@ enum bt_graph_status bt_graph_connect_ports(
         */
        BT_LOGD_STR("Notifying graph's user that new component ports are connected.");
        listener_status = bt_graph_notify_ports_connected(graph, upstream_port, downstream_port);
-       if (listener_status != BT_GRAPH_LISTENER_STATUS_OK) {
+       if (listener_status != BT_FUNC_STATUS_OK) {
+               if (listener_status < 0) {
+                       BT_LIB_LOGW_APPEND_CAUSE(
+                               "Graph \"ports connected\" listener failed: "
+                               "status=%d, %![graph-]+g, %![up-comp-]+c, "
+                               "%![down-comp-]+c, %![up-port-]+p, %![down-port-]+p",
+                               listener_status, graph,
+                               upstream_component, downstream_component,
+                               upstream_port, downstream_port);
+               }
+
                status = (int) listener_status;
                goto end;
        }
@@ -554,7 +559,7 @@ enum bt_graph_status bt_graph_connect_ports(
        }
 
 end:
-       if (status != BT_GRAPH_STATUS_OK) {
+       if (status != BT_FUNC_STATUS_OK) {
                bt_graph_make_faulty(graph);
        }
 
@@ -565,35 +570,42 @@ end:
 }
 
 static inline
-enum bt_graph_status consume_graph_sink(struct bt_component_sink *comp)
+int consume_graph_sink(struct bt_component_sink *comp)
 {
-       enum bt_self_component_status comp_status;
+       enum bt_component_class_sink_consume_method_status consume_status;
        struct bt_component_class_sink *sink_class = NULL;
 
        BT_ASSERT(comp);
        sink_class = (void *) comp->parent.class;
        BT_ASSERT(sink_class->methods.consume);
        BT_LIB_LOGD("Calling user's consume method: %!+c", comp);
-       comp_status = sink_class->methods.consume((void *) comp);
+       consume_status = sink_class->methods.consume((void *) comp);
        BT_LOGD("User method returned: status=%s",
-               bt_self_component_status_string(comp_status));
-       BT_ASSERT_PRE(comp_status == BT_SELF_COMPONENT_STATUS_OK ||
-               comp_status == BT_SELF_COMPONENT_STATUS_END ||
-               comp_status == BT_SELF_COMPONENT_STATUS_AGAIN ||
-               comp_status == BT_SELF_COMPONENT_STATUS_ERROR ||
-               comp_status == BT_SELF_COMPONENT_STATUS_NOMEM,
+               bt_common_func_status_string(consume_status));
+       BT_ASSERT_POST_DEV(consume_status == BT_FUNC_STATUS_OK ||
+               consume_status == BT_FUNC_STATUS_END ||
+               consume_status == BT_FUNC_STATUS_AGAIN ||
+               consume_status == BT_FUNC_STATUS_ERROR ||
+               consume_status == BT_FUNC_STATUS_MEMORY_ERROR,
                "Invalid component status returned by consuming method: "
-               "status=%s", bt_self_component_status_string(comp_status));
-       if (comp_status < 0) {
-               BT_LOGW_STR("Consume method failed.");
+               "status=%s", bt_common_func_status_string(consume_status));
+       if (consume_status) {
+               if (consume_status < 0) {
+                       BT_LIB_LOGW_APPEND_CAUSE(
+                               "Component's \"consume\" method failed: "
+                               "status=%s, %![comp-]+c",
+                               bt_common_func_status_string(consume_status),
+                               comp);
+               }
+
                goto end;
        }
 
        BT_LIB_LOGD("Consumed from sink: %![comp-]+c, status=%s",
-               comp, bt_self_component_status_string(comp_status));
+               comp, bt_common_func_status_string(consume_status));
 
 end:
-       return (int) comp_status;
+       return consume_status;
 }
 
 /*
@@ -602,14 +614,14 @@ end:
  * still something to consume afterwards.
  */
 static inline
-enum bt_graph_status consume_sink_node(struct bt_graph *graph, GList *node)
+int consume_sink_node(struct bt_graph *graph, GList *node)
 {
-       enum bt_graph_status status;
+       int status;
        struct bt_component_sink *sink;
 
        sink = node->data;
        status = consume_graph_sink(sink);
-       if (G_UNLIKELY(status != BT_GRAPH_STATUS_END)) {
+       if (G_UNLIKELY(status != BT_FUNC_STATUS_END)) {
                g_queue_push_tail_link(graph->sinks_to_consume, node);
                goto end;
        }
@@ -619,21 +631,21 @@ enum bt_graph_status consume_sink_node(struct bt_graph *graph, GList *node)
 
        /* Don't forward an END status if there are sinks left to consume. */
        if (!g_queue_is_empty(graph->sinks_to_consume)) {
-               status = BT_GRAPH_STATUS_OK;
+               status = BT_FUNC_STATUS_OK;
                goto end;
        }
 
 end:
        BT_LIB_LOGD("Consumed sink node: %![comp-]+c, status=%s",
-               sink, bt_graph_status_string(status));
+               sink, bt_common_func_status_string(status));
        return status;
 }
 
 BT_HIDDEN
-enum bt_graph_status bt_graph_consume_sink_no_check(struct bt_graph *graph,
+int bt_graph_consume_sink_no_check(struct bt_graph *graph,
                struct bt_component_sink *sink)
 {
-       enum bt_graph_status status;
+       int status;
        GList *sink_node;
        int index;
 
@@ -642,7 +654,7 @@ enum bt_graph_status bt_graph_consume_sink_no_check(struct bt_graph *graph,
 
        if (g_queue_is_empty(graph->sinks_to_consume)) {
                BT_LOGD_STR("Graph's sink queue is empty: end of graph.");
-               status = BT_GRAPH_STATUS_END;
+               status = BT_FUNC_STATUS_END;
                goto end;
        }
 
@@ -650,7 +662,7 @@ enum bt_graph_status bt_graph_consume_sink_no_check(struct bt_graph *graph,
        if (index < 0) {
                BT_LIB_LOGD("Sink component is not marked as consumable: "
                        "component sink is ended: %![comp-]+c", sink);
-               status = BT_GRAPH_STATUS_END;
+               status = BT_FUNC_STATUS_END;
                goto end;
        }
 
@@ -663,19 +675,19 @@ end:
 }
 
 static inline
-enum bt_graph_status consume_no_check(struct bt_graph *graph)
+int consume_no_check(struct bt_graph *graph)
 {
-       enum bt_graph_status status = BT_GRAPH_STATUS_OK;
+       int status = BT_FUNC_STATUS_OK;
        struct bt_component *sink;
        GList *current_node;
 
-       BT_ASSERT_PRE(graph->has_sink,
+       BT_ASSERT_PRE_DEV(graph->has_sink,
                "Graph has no sink component: %!+g", graph);
        BT_LIB_LOGD("Making next sink component consume: %![graph-]+g", graph);
 
        if (G_UNLIKELY(g_queue_is_empty(graph->sinks_to_consume))) {
                BT_LOGD_STR("Graph's sink queue is empty: end of graph.");
-               status = BT_GRAPH_STATUS_END;
+               status = BT_FUNC_STATUS_END;
                goto end;
        }
 
@@ -688,15 +700,15 @@ end:
        return status;
 }
 
-enum bt_graph_status bt_graph_consume(struct bt_graph *graph)
+enum bt_graph_consume_status bt_graph_consume(struct bt_graph *graph)
 {
-       enum bt_graph_status status;
+       enum bt_graph_consume_status status;
 
-       BT_ASSERT_PRE_NON_NULL(graph, "Graph");
-       BT_ASSERT_PRE(!graph->canceled, "Graph is canceled: %!+g", graph);
-       BT_ASSERT_PRE(graph->can_consume,
+       BT_ASSERT_PRE_DEV_NON_NULL(graph, "Graph");
+       BT_ASSERT_PRE_DEV(graph->can_consume,
                "Cannot consume graph in its current state: %!+g", graph);
-       BT_ASSERT_PRE(graph->config_state != BT_GRAPH_CONFIGURATION_STATE_FAULTY,
+       BT_ASSERT_PRE_DEV(graph->config_state !=
+               BT_GRAPH_CONFIGURATION_STATE_FAULTY,
                "Graph is in a faulty state: %!+g", graph);
        bt_graph_set_can_consume(graph, false);
        status = bt_graph_configure(graph);
@@ -712,12 +724,11 @@ end:
        return status;
 }
 
-enum bt_graph_status bt_graph_run(struct bt_graph *graph)
+enum bt_graph_run_status bt_graph_run(struct bt_graph *graph)
 {
-       enum bt_graph_status status;
+       enum bt_graph_run_status status;
 
        BT_ASSERT_PRE_NON_NULL(graph, "Graph");
-       BT_ASSERT_PRE(!graph->canceled, "Graph is canceled: %!+g", graph);
        BT_ASSERT_PRE(graph->can_consume,
                "Cannot consume graph in its current state: %!+g", graph);
        BT_ASSERT_PRE(graph->config_state != BT_GRAPH_CONFIGURATION_STATE_FAULTY,
@@ -733,20 +744,20 @@ enum bt_graph_status bt_graph_run(struct bt_graph *graph)
 
        do {
                /*
-                * Check if the graph is canceled at each iteration. If
-                * the graph was canceled by another thread or by a
-                * signal handler, this is not a warning nor an error,
-                * it was intentional: log with a DEBUG level only.
+                * Check if the graph is interrupted at each iteration.
+                * If the graph was interrupted by another thread or by
+                * a signal handler, this is NOT a warning nor an error;
+                * it was intentional: log with an INFO level only.
                 */
-               if (G_UNLIKELY(graph->canceled)) {
-                       BT_LIB_LOGI("Stopping the graph: graph is canceled: "
-                               "%!+g", graph);
-                       status = BT_GRAPH_STATUS_CANCELED;
+               if (G_UNLIKELY(bt_graph_is_interrupted(graph))) {
+                       BT_LIB_LOGI("Stopping the graph: "
+                               "graph was interrupted: %!+g", graph);
+                       status = BT_FUNC_STATUS_AGAIN;
                        goto end;
                }
 
                status = consume_no_check(graph);
-               if (G_UNLIKELY(status == BT_GRAPH_STATUS_AGAIN)) {
+               if (G_UNLIKELY(status == BT_FUNC_STATUS_AGAIN)) {
                        /*
                         * If AGAIN is received and there are multiple
                         * sinks, go ahead and consume from the next
@@ -759,23 +770,23 @@ enum bt_graph_status bt_graph_run(struct bt_graph *graph)
                         * sleep for an arbitrary amount of time.
                         */
                        if (graph->sinks_to_consume->length > 1) {
-                               status = BT_GRAPH_STATUS_OK;
+                               status = BT_FUNC_STATUS_OK;
                        }
                }
-       } while (status == BT_GRAPH_STATUS_OK);
+       } while (status == BT_FUNC_STATUS_OK);
 
        if (g_queue_is_empty(graph->sinks_to_consume)) {
-               status = BT_GRAPH_STATUS_END;
+               status = BT_FUNC_STATUS_END;
        }
 
 end:
        BT_LIB_LOGI("Graph ran: %![graph-]+g, status=%s", graph,
-               bt_graph_status_string(status));
+               bt_common_func_status_string(status));
        bt_graph_set_can_consume(graph, true);
        return status;
 }
 
-enum bt_graph_status
+enum bt_graph_add_listener_status
 bt_graph_add_source_component_output_port_added_listener(
                struct bt_graph *graph,
                bt_graph_source_component_output_port_added_listener_func func,
@@ -807,10 +818,10 @@ bt_graph_add_source_component_output_port_added_listener(
                *out_listener_id = listener_id;
        }
 
-       return BT_GRAPH_STATUS_OK;
+       return BT_FUNC_STATUS_OK;
 }
 
-enum bt_graph_status
+enum bt_graph_add_listener_status
 bt_graph_add_filter_component_output_port_added_listener(
                struct bt_graph *graph,
                bt_graph_filter_component_output_port_added_listener_func func,
@@ -842,10 +853,10 @@ bt_graph_add_filter_component_output_port_added_listener(
                *out_listener_id = listener_id;
        }
 
-       return BT_GRAPH_STATUS_OK;
+       return BT_FUNC_STATUS_OK;
 }
 
-enum bt_graph_status
+enum bt_graph_add_listener_status
 bt_graph_add_filter_component_input_port_added_listener(
                struct bt_graph *graph,
                bt_graph_filter_component_input_port_added_listener_func func,
@@ -877,10 +888,10 @@ bt_graph_add_filter_component_input_port_added_listener(
                *out_listener_id = listener_id;
        }
 
-       return BT_GRAPH_STATUS_OK;
+       return BT_FUNC_STATUS_OK;
 }
 
-enum bt_graph_status
+enum bt_graph_add_listener_status
 bt_graph_add_sink_component_input_port_added_listener(
                struct bt_graph *graph,
                bt_graph_sink_component_input_port_added_listener_func func,
@@ -912,10 +923,10 @@ bt_graph_add_sink_component_input_port_added_listener(
                *out_listener_id = listener_id;
        }
 
-       return BT_GRAPH_STATUS_OK;
+       return BT_FUNC_STATUS_OK;
 }
 
-enum bt_graph_status
+enum bt_graph_add_listener_status
 bt_graph_add_source_filter_component_ports_connected_listener(
                struct bt_graph *graph,
                bt_graph_source_filter_component_ports_connected_listener_func func,
@@ -948,10 +959,10 @@ bt_graph_add_source_filter_component_ports_connected_listener(
                *out_listener_id = listener_id;
        }
 
-       return BT_GRAPH_STATUS_OK;
+       return BT_FUNC_STATUS_OK;
 }
 
-enum bt_graph_status
+enum bt_graph_add_listener_status
 bt_graph_add_source_sink_component_ports_connected_listener(
                struct bt_graph *graph,
                bt_graph_source_sink_component_ports_connected_listener_func func,
@@ -984,10 +995,10 @@ bt_graph_add_source_sink_component_ports_connected_listener(
                *out_listener_id = listener_id;
        }
 
-       return BT_GRAPH_STATUS_OK;
+       return BT_FUNC_STATUS_OK;
 }
 
-enum bt_graph_status
+enum bt_graph_add_listener_status
 bt_graph_add_filter_filter_component_ports_connected_listener(
                struct bt_graph *graph,
                bt_graph_filter_filter_component_ports_connected_listener_func func,
@@ -1020,10 +1031,10 @@ bt_graph_add_filter_filter_component_ports_connected_listener(
                *out_listener_id = listener_id;
        }
 
-       return BT_GRAPH_STATUS_OK;
+       return BT_FUNC_STATUS_OK;
 }
 
-enum bt_graph_status
+enum bt_graph_add_listener_status
 bt_graph_add_filter_sink_component_ports_connected_listener(
                struct bt_graph *graph,
                bt_graph_filter_sink_component_ports_connected_listener_func func,
@@ -1056,17 +1067,17 @@ bt_graph_add_filter_sink_component_ports_connected_listener(
                *out_listener_id = listener_id;
        }
 
-       return BT_GRAPH_STATUS_OK;
+       return BT_FUNC_STATUS_OK;
 }
 
 BT_HIDDEN
-enum bt_graph_listener_status bt_graph_notify_port_added(
+enum bt_graph_listener_func_status bt_graph_notify_port_added(
                struct bt_graph *graph, struct bt_port *port)
 {
        uint64_t i;
        GArray *listeners;
        struct bt_component *comp;
-       enum bt_graph_listener_status status = BT_GRAPH_LISTENER_STATUS_OK;
+       enum bt_graph_listener_func_status status = BT_FUNC_STATUS_OK;
 
        BT_ASSERT(graph);
        BT_ASSERT(port);
@@ -1127,7 +1138,7 @@ enum bt_graph_listener_status bt_graph_notify_port_added(
 
                BT_ASSERT(listener->func);
                status = listener->func(comp, port, listener->base.data);
-               if (status != BT_GRAPH_LISTENER_STATUS_OK) {
+               if (status != BT_FUNC_STATUS_OK) {
                        goto end;
                }
        }
@@ -1137,7 +1148,7 @@ end:
 }
 
 BT_HIDDEN
-enum bt_graph_listener_status bt_graph_notify_ports_connected(
+enum bt_graph_listener_func_status bt_graph_notify_ports_connected(
                struct bt_graph *graph, struct bt_port *upstream_port,
                struct bt_port *downstream_port)
 {
@@ -1145,7 +1156,7 @@ enum bt_graph_listener_status bt_graph_notify_ports_connected(
        GArray *listeners;
        struct bt_component *upstream_comp;
        struct bt_component *downstream_comp;
-       enum bt_graph_listener_status status = BT_GRAPH_LISTENER_STATUS_OK;
+       enum bt_graph_listener_func_status status = BT_FUNC_STATUS_OK;
 
        BT_ASSERT(graph);
        BT_ASSERT(upstream_port);
@@ -1205,7 +1216,7 @@ enum bt_graph_listener_status bt_graph_notify_ports_connected(
                BT_ASSERT(listener->func);
                status = listener->func(upstream_comp, downstream_comp,
                        upstream_port, downstream_port, listener->base.data);
-               if (status != BT_GRAPH_LISTENER_STATUS_OK) {
+               if (status != BT_FUNC_STATUS_OK) {
                        goto end;
                }
        }
@@ -1214,21 +1225,6 @@ end:
        return status;
 }
 
-enum bt_graph_status bt_graph_cancel(struct bt_graph *graph)
-{
-
-       BT_ASSERT_PRE_NON_NULL(graph, "Graph");
-       graph->canceled = true;
-       BT_LIB_LOGI("Canceled graph: %!+i", graph);
-       return BT_GRAPH_STATUS_OK;
-}
-
-bt_bool bt_graph_is_canceled(const struct bt_graph *graph)
-{
-       BT_ASSERT_PRE_NON_NULL(graph, "Graph");
-       return graph->canceled ? BT_TRUE : BT_FALSE;
-}
-
 BT_HIDDEN
 void bt_graph_remove_connection(struct bt_graph *graph,
                struct bt_connection *connection)
@@ -1240,7 +1236,6 @@ void bt_graph_remove_connection(struct bt_graph *graph,
        g_ptr_array_remove(graph->connections, connection);
 }
 
-BT_ASSERT_PRE_FUNC
 static inline
 bool component_name_exists(struct bt_graph *graph, const char *name)
 {
@@ -1264,7 +1259,7 @@ end:
 }
 
 static
-enum bt_graph_status add_component_with_init_method_data(
+int add_component_with_init_method_data(
                struct bt_graph *graph,
                struct bt_component_class *comp_cls,
                comp_init_method_t init_method,
@@ -1272,8 +1267,8 @@ enum bt_graph_status add_component_with_init_method_data(
                void *init_method_data, bt_logging_level log_level,
                struct bt_component **user_component)
 {
-       enum bt_graph_status graph_status = BT_GRAPH_STATUS_OK;
-       enum bt_self_component_status comp_status;
+       int status = BT_FUNC_STATUS_OK;
+       enum bt_component_class_init_method_status init_status;
        struct bt_component *component = NULL;
        int ret;
        bool init_can_consume;
@@ -1282,7 +1277,6 @@ enum bt_graph_status add_component_with_init_method_data(
        BT_ASSERT(comp_cls);
        BT_ASSERT_PRE_NON_NULL(graph, "Graph");
        BT_ASSERT_PRE_NON_NULL(name, "Name");
-       BT_ASSERT_PRE(!graph->canceled, "Graph is canceled: %!+g", graph);
        BT_ASSERT_PRE(
                graph->config_state == BT_GRAPH_CONFIGURATION_STATE_CONFIGURING,
                "Graph is not in the \"configuring\" state: %!+g", graph);
@@ -1302,8 +1296,9 @@ enum bt_graph_status add_component_with_init_method_data(
        if (!params) {
                new_params = bt_value_map_create();
                if (!new_params) {
-                       BT_LOGE_STR("Cannot create empty map value object.");
-                       graph_status = BT_GRAPH_STATUS_NOMEM;
+                       BT_LIB_LOGE_APPEND_CAUSE(
+                               "Cannot create empty map value object.");
+                       status = BT_FUNC_STATUS_MEMORY_ERROR;
                        goto end;
                }
 
@@ -1312,9 +1307,10 @@ enum bt_graph_status add_component_with_init_method_data(
 
        ret = bt_component_create(comp_cls, name, log_level, &component);
        if (ret) {
-               BT_LOGE("Cannot create empty component object: ret=%d",
+               BT_LIB_LOGE_APPEND_CAUSE(
+                       "Cannot create empty component object: ret=%d",
                        ret);
-               graph_status = BT_GRAPH_STATUS_NOMEM;
+               status = BT_FUNC_STATUS_MEMORY_ERROR;
                goto end;
        }
 
@@ -1329,12 +1325,19 @@ enum bt_graph_status add_component_with_init_method_data(
 
        if (init_method) {
                BT_LOGD_STR("Calling user's initialization method.");
-               comp_status = init_method(component, params, init_method_data);
+               init_status = init_method(component, params, init_method_data);
                BT_LOGD("User method returned: status=%s",
-                       bt_self_component_status_string(comp_status));
-               if (comp_status != BT_SELF_COMPONENT_STATUS_OK) {
-                       BT_LOGW_STR("Initialization method failed.");
-                       graph_status = (int) comp_status;
+                       bt_common_func_status_string(init_status));
+               if (init_status != BT_FUNC_STATUS_OK) {
+                       if (init_status < 0) {
+                               BT_LIB_LOGW_APPEND_CAUSE(
+                                       "Component initialization method failed: "
+                                       "status=%s, %![comp-]+c",
+                                       bt_common_func_status_string(init_status),
+                                       component);
+                       }
+
+                       status = init_status;
                        bt_component_set_graph(component, NULL);
                        g_ptr_array_remove_fast(graph->components, component);
                        goto end;
@@ -1376,7 +1379,7 @@ enum bt_graph_status add_component_with_init_method_data(
        }
 
 end:
-       if (graph_status != BT_GRAPH_STATUS_OK) {
+       if (status != BT_FUNC_STATUS_OK) {
                bt_graph_make_faulty(graph);
        }
 
@@ -1384,10 +1387,10 @@ end:
        bt_object_put_ref(new_params);
        (void) init_can_consume;
        bt_graph_set_can_consume(graph, init_can_consume);
-       return graph_status;
+       return status;
 }
 
-enum bt_graph_status
+enum bt_graph_add_component_status
 bt_graph_add_source_component_with_init_method_data(
                struct bt_graph *graph,
                const struct bt_component_class_source *comp_cls,
@@ -1401,7 +1404,7 @@ bt_graph_add_source_component_with_init_method_data(
                name, params, init_method_data, log_level, (void *) component);
 }
 
-enum bt_graph_status bt_graph_add_source_component(
+enum bt_graph_add_component_status bt_graph_add_source_component(
                struct bt_graph *graph,
                const struct bt_component_class_source *comp_cls,
                const char *name, const struct bt_value *params,
@@ -1412,7 +1415,7 @@ enum bt_graph_status bt_graph_add_source_component(
                graph, comp_cls, name, params, NULL, log_level, component);
 }
 
-enum bt_graph_status
+enum bt_graph_add_component_status
 bt_graph_add_filter_component_with_init_method_data(
                struct bt_graph *graph,
                const struct bt_component_class_filter *comp_cls,
@@ -1426,7 +1429,7 @@ bt_graph_add_filter_component_with_init_method_data(
                name, params, init_method_data, log_level, (void *) component);
 }
 
-enum bt_graph_status bt_graph_add_filter_component(
+enum bt_graph_add_component_status bt_graph_add_filter_component(
                struct bt_graph *graph,
                const struct bt_component_class_filter *comp_cls,
                const char *name, const struct bt_value *params,
@@ -1437,7 +1440,7 @@ enum bt_graph_status bt_graph_add_filter_component(
                graph, comp_cls, name, params, NULL, log_level, component);
 }
 
-enum bt_graph_status
+enum bt_graph_add_component_status
 bt_graph_add_sink_component_with_init_method_data(
                struct bt_graph *graph,
                const struct bt_component_class_sink *comp_cls,
@@ -1451,7 +1454,7 @@ bt_graph_add_sink_component_with_init_method_data(
                name, params, init_method_data, log_level, (void *) component);
 }
 
-enum bt_graph_status bt_graph_add_sink_component(
+enum bt_graph_add_component_status bt_graph_add_sink_component(
                struct bt_graph *graph,
                const struct bt_component_class_sink *comp_cls,
                const char *name, const struct bt_value *params,
@@ -1486,7 +1489,8 @@ int bt_graph_remove_unconnected_component(struct bt_graph *graph,
                BT_ASSERT(port);
 
                if (bt_port_is_connected(port)) {
-                       BT_LIB_LOGW("Cannot remove component from graph: "
+                       BT_LIB_LOGW_APPEND_CAUSE(
+                               "Cannot remove component from graph: "
                                "an input port is connected: "
                                "%![graph-]+g, %![comp-]+c, %![port-]+p",
                                graph, component, port);
@@ -1503,7 +1507,8 @@ int bt_graph_remove_unconnected_component(struct bt_graph *graph,
                BT_ASSERT(port);
 
                if (bt_port_is_connected(port)) {
-                       BT_LIB_LOGW("Cannot remove component from graph: "
+                       BT_LIB_LOGW_APPEND_CAUSE(
+                               "Cannot remove component from graph: "
                                "an output port is connected: "
                                "%![graph-]+g, %![comp-]+c, %![port-]+p",
                                graph, component, port);
@@ -1555,6 +1560,32 @@ void bt_graph_add_message(struct bt_graph *graph,
        g_ptr_array_add(graph->messages, msg);
 }
 
+BT_HIDDEN
+bool bt_graph_is_interrupted(const struct bt_graph *graph)
+{
+       BT_ASSERT(graph);
+       return bt_interrupter_array_any_is_set(graph->interrupters);
+}
+
+enum bt_graph_add_interrupter_status bt_graph_add_interrupter(
+               struct bt_graph *graph, const struct bt_interrupter *intr)
+{
+       BT_ASSERT_PRE_NON_NULL(graph, "Graph");
+       BT_ASSERT_PRE_NON_NULL(intr, "Interrupter");
+       g_ptr_array_add(graph->interrupters, (void *) intr);
+       bt_object_get_no_null_check(intr);
+       BT_LIB_LOGD("Added interrupter to graph: %![graph-]+g, %![intr-]+z",
+               graph, intr);
+       return BT_FUNC_STATUS_OK;
+}
+
+void bt_graph_interrupt(struct bt_graph *graph)
+{
+       BT_ASSERT_PRE_NON_NULL(graph, "Graph");
+       bt_interrupter_set(graph->default_interrupter);
+       BT_LIB_LOGI("Interrupted graph: %!+g", graph);
+}
+
 void bt_graph_get_ref(const struct bt_graph *graph)
 {
        bt_object_get_ref(graph);
This page took 0.039754 seconds and 4 git commands to generate.