X-Git-Url: http://git.efficios.com/?a=blobdiff_plain;f=cli%2Fbabeltrace.c;h=06f124615dd0621e2c4871c1946a13b109e138fe;hb=05e2128659970c32648a01255ed870449f05d518;hp=8e17c2a10458ced2cd046bae543be6caab2f9a2e;hpb=da91b29ad2964b85601e25843f1dca92f6c97406;p=babeltrace.git diff --git a/cli/babeltrace.c b/cli/babeltrace.c index 8e17c2a1..06f12461 100644 --- a/cli/babeltrace.c +++ b/cli/babeltrace.c @@ -1,8 +1,4 @@ /* - * babeltrace.c - * - * Babeltrace Trace Converter - * * Copyright 2010-2011 EfficiOS Inc. and Linux Foundation * * Author: Mathieu Desnoyers @@ -73,19 +69,20 @@ static const char* log_level_env_var_names[] = { }; /* Application's processing graph (weak) */ -static struct bt_graph *the_graph; -static struct bt_query_executor *the_query_executor; +static struct bt_private_graph *the_graph; +static struct bt_private_query_executor *the_query_executor; static bool canceled = false; GPtrArray *loaded_plugins; #ifdef __MINGW32__ + #include static BOOL WINAPI signal_handler(DWORD signal) { if (the_graph) { - bt_graph_cancel(the_graph); + bt_private_graph_cancel(the_graph); } canceled = true; @@ -100,7 +97,9 @@ void set_signal_handler(void) BT_LOGE("Failed to set the ctrl+c handler."); } } + #else /* __MINGW32__ */ + static void signal_handler(int signum) { @@ -109,11 +108,11 @@ void signal_handler(int signum) } if (the_graph) { - bt_graph_cancel(the_graph); + bt_private_graph_cancel(the_graph); } if (the_query_executor) { - bt_query_executor_cancel(the_query_executor); + bt_private_query_executor_cancel(the_query_executor); } canceled = true; @@ -133,12 +132,14 @@ void set_signal_handler(void) sigaction(SIGINT, &new_action, NULL); } } + #endif /* __MINGW32__ */ static void init_static_data(void) { - loaded_plugins = g_ptr_array_new_with_free_func(bt_object_put_ref); + loaded_plugins = g_ptr_array_new_with_free_func( + (GDestroyNotify) bt_object_put_ref); } static @@ -152,7 +153,7 @@ int create_the_query_executor(void) { int ret = 0; - the_query_executor = bt_query_executor_create(); + the_query_executor = bt_private_query_executor_create(); if (!the_query_executor) { BT_LOGE_STR("Cannot create a query executor."); ret = -1; @@ -169,11 +170,11 @@ void destroy_the_query_executor(void) static int query(struct bt_component_class *comp_cls, const char *obj, - struct bt_value *params, struct bt_value **user_result, + const struct bt_value *params, const struct bt_value **user_result, const char **fail_reason) { - struct bt_value *result = NULL; - enum bt_query_status status; + const struct bt_value *result = NULL; + enum bt_query_executor_status status; *fail_reason = "unknown error"; int ret = 0; @@ -195,21 +196,22 @@ int query(struct bt_component_class *comp_cls, const char *obj, } while (true) { - status = bt_query_executor_query(the_query_executor, comp_cls, - obj, params, &result); + status = bt_private_query_executor_query(the_query_executor, + comp_cls, obj, params, &result); switch (status) { - case BT_QUERY_STATUS_OK: + case BT_QUERY_EXECUTOR_STATUS_OK: goto ok; - case BT_QUERY_STATUS_AGAIN: + case BT_QUERY_EXECUTOR_STATUS_AGAIN: { const uint64_t sleep_time_us = 100000; /* Wait 100 ms and retry */ - BT_LOGV("Got BT_QUERY_STATUS_AGAIN: sleeping: " + BT_LOGV("Got BT_QUERY_EXECUTOR_STATUS_AGAIN: sleeping: " "time-us=%" PRIu64, sleep_time_us); if (usleep(sleep_time_us)) { - if (bt_query_executor_is_canceled(the_query_executor)) { + if (bt_query_executor_is_canceled( + bt_private_query_executor_as_query_executor(the_query_executor))) { BT_LOGI("Query was canceled by user: " "comp-cls-addr=%p, comp-cls-name=\"%s\", " "query-obj=\"%s\"", comp_cls, @@ -222,19 +224,21 @@ int query(struct bt_component_class *comp_cls, const char *obj, continue; } - case BT_QUERY_STATUS_EXECUTOR_CANCELED: + case BT_QUERY_EXECUTOR_STATUS_CANCELED: *fail_reason = "canceled by user"; goto error; - case BT_QUERY_STATUS_ERROR: - case BT_QUERY_STATUS_INVALID: + case BT_QUERY_EXECUTOR_STATUS_ERROR: goto error; - case BT_QUERY_STATUS_INVALID_OBJECT: + case BT_QUERY_EXECUTOR_STATUS_INVALID_OBJECT: *fail_reason = "invalid or unknown query object"; goto error; - case BT_QUERY_STATUS_INVALID_PARAMS: + case BT_QUERY_EXECUTOR_STATUS_INVALID_PARAMS: *fail_reason = "invalid query parameters"; goto error; - case BT_QUERY_STATUS_NOMEM: + case BT_QUERY_EXECUTOR_STATUS_UNSUPPORTED: + *fail_reason = "unsupported action"; + goto error; + case BT_QUERY_EXECUTOR_STATUS_NOMEM: *fail_reason = "not enough memory"; goto error; default: @@ -284,29 +288,31 @@ struct bt_plugin *find_plugin(const char *name) } } - return bt_object_get_ref(plugin); + bt_object_get_ref(plugin); + return plugin; } +typedef void *(*plugin_borrow_comp_cls_func_t)(struct bt_plugin *, + const char *); + static -struct bt_component_class *find_component_class(const char *plugin_name, +void *find_component_class_from_plugin(const char *plugin_name, const char *comp_class_name, - enum bt_component_class_type comp_class_type) + plugin_borrow_comp_cls_func_t plugin_borrow_comp_cls_func) { - struct bt_component_class *comp_class = NULL; + void *comp_class = NULL; struct bt_plugin *plugin; BT_LOGD("Finding component class: plugin-name=\"%s\", " - "comp-cls-name=\"%s\", comp-cls-type=%d", - plugin_name, comp_class_name, comp_class_type); + "comp-cls-name=\"%s\"", plugin_name, comp_class_name); plugin = find_plugin(plugin_name); - if (!plugin) { goto end; } - comp_class = bt_plugin_get_component_class_by_name_and_type(plugin, - comp_class_name, comp_class_type); + comp_class = plugin_borrow_comp_cls_func(plugin, comp_class_name); + bt_object_get_ref(comp_class); BT_OBJECT_PUT_REF_AND_RESET(plugin); end: @@ -315,13 +321,73 @@ end: BT_LOGD("Found component class: comp-cls-addr=%p", comp_class); } else { - BT_LOGD("Cannot find component class."); + BT_LOGD("Cannot find source component class."); } } return comp_class; } +static +struct bt_component_class_source *find_source_component_class( + const char *plugin_name, const char *comp_class_name) +{ + return (void *) find_component_class_from_plugin(plugin_name, + comp_class_name, + (plugin_borrow_comp_cls_func_t) + bt_plugin_borrow_source_component_class_by_name); +} + +static +struct bt_component_class_filter *find_filter_component_class( + const char *plugin_name, const char *comp_class_name) +{ + return (void *) find_component_class_from_plugin(plugin_name, + comp_class_name, + (plugin_borrow_comp_cls_func_t) + bt_plugin_borrow_filter_component_class_by_name); +} + +static +struct bt_component_class_sink *find_sink_component_class( + const char *plugin_name, const char *comp_class_name) +{ + return (void *) find_component_class_from_plugin(plugin_name, + comp_class_name, + (plugin_borrow_comp_cls_func_t) + bt_plugin_borrow_sink_component_class_by_name); +} + +static +struct bt_component_class *find_component_class(const char *plugin_name, + const char *comp_class_name, + enum bt_component_class_type comp_class_type) +{ + struct bt_component_class *comp_cls = NULL; + + switch (comp_class_type) { + case BT_COMPONENT_CLASS_TYPE_SOURCE: + comp_cls = bt_component_class_source_as_component_class( + find_source_component_class(plugin_name, + comp_class_name)); + break; + case BT_COMPONENT_CLASS_TYPE_FILTER: + comp_cls = bt_component_class_filter_as_component_class( + find_filter_component_class(plugin_name, + comp_class_name)); + break; + case BT_COMPONENT_CLASS_TYPE_SINK: + comp_cls = bt_component_class_sink_as_component_class( + find_sink_component_class(plugin_name, + comp_class_name)); + break; + default: + abort(); + } + + return comp_cls; +} + static void print_indent(FILE *fp, size_t indent) { @@ -342,7 +408,6 @@ const char *component_type_str(enum bt_component_class_type type) return "sink"; case BT_COMPONENT_CLASS_TYPE_FILTER: return "filter"; - case BT_COMPONENT_CLASS_TYPE_UNKNOWN: default: return "(unknown)"; } @@ -388,10 +453,10 @@ end: } static -void print_value(FILE *, struct bt_value *, size_t); +void print_value(FILE *, const struct bt_value *, size_t); static -void print_value_rec(FILE *, struct bt_value *, size_t); +void print_value_rec(FILE *, const struct bt_value *, size_t); struct print_map_value_data { size_t indent; @@ -399,7 +464,8 @@ struct print_map_value_data { }; static -bt_bool print_map_value(const char *key, struct bt_value *object, void *data) +bt_bool print_map_value(const char *key, const struct bt_value *object, + void *data) { struct print_map_value_data *print_map_value_data = data; @@ -430,7 +496,7 @@ bt_bool print_map_value(const char *key, struct bt_value *object, void *data) } static -void print_value_rec(FILE *fp, struct bt_value *value, size_t indent) +void print_value_rec(FILE *fp, const struct bt_value *value, size_t indent) { bt_bool bool_val; int64_t int_val; @@ -438,7 +504,6 @@ void print_value_rec(FILE *fp, struct bt_value *value, size_t indent) const char *str_val; int size; int i; - enum bt_value_status status; if (!value) { return; @@ -450,37 +515,25 @@ void print_value_rec(FILE *fp, struct bt_value *value, size_t indent) bt_common_color_reset()); break; case BT_VALUE_TYPE_BOOL: - status = bt_value_bool_get(value, &bool_val); - if (status != BT_VALUE_STATUS_OK) { - goto error; - } + bool_val = bt_value_bool_get(value); fprintf(fp, "%s%s%s%s\n", bt_common_color_bold(), bt_common_color_fg_cyan(), bool_val ? "yes" : "no", bt_common_color_reset()); break; case BT_VALUE_TYPE_INTEGER: - status = bt_value_integer_get(value, &int_val); - if (status != BT_VALUE_STATUS_OK) { - goto error; - } + int_val = bt_value_integer_get(value); fprintf(fp, "%s%s%" PRId64 "%s\n", bt_common_color_bold(), bt_common_color_fg_red(), int_val, bt_common_color_reset()); break; case BT_VALUE_TYPE_REAL: - status = bt_value_real_get(value, &dbl_val); - if (status != BT_VALUE_STATUS_OK) { - goto error; - } + dbl_val = bt_value_real_get(value); fprintf(fp, "%s%s%lf%s\n", bt_common_color_bold(), bt_common_color_fg_red(), dbl_val, bt_common_color_reset()); break; case BT_VALUE_TYPE_STRING: - status = bt_value_string_get(value, &str_val); - if (status != BT_VALUE_STATUS_OK) { - goto error; - } + str_val = bt_value_string_get(value); fprintf(fp, "%s%s%s%s\n", bt_common_color_bold(), bt_common_color_fg_green(), str_val, bt_common_color_reset()); @@ -498,8 +551,8 @@ void print_value_rec(FILE *fp, struct bt_value *value, size_t indent) } for (i = 0; i < size; i++) { - struct bt_value *element = - bt_value_array_borrow_element_by_index( + const struct bt_value *element = + bt_value_array_borrow_element_by_index_const( value, i); if (!element) { @@ -541,7 +594,7 @@ void print_value_rec(FILE *fp, struct bt_value *value, size_t indent) break; } - bt_value_map_foreach_entry(value, print_map_value, &data); + bt_value_map_foreach_entry_const(value, print_map_value, &data); break; } default: @@ -555,7 +608,7 @@ error: } static -void print_value(FILE *fp, struct bt_value *value, size_t indent) +void print_value(FILE *fp, const struct bt_value *value, size_t indent) { if (!bt_value_is_array(value) && !bt_value_is_map(value)) { print_indent(fp, indent); @@ -579,8 +632,7 @@ void print_bt_config_component(struct bt_config_component *bt_config_component) } fprintf(stderr, " Parameters:\n"); - print_value(stderr, - bt_value_borrow_from_private(bt_config_component->params), 8); + print_value(stderr, bt_config_component->params, 8); } static @@ -597,7 +649,7 @@ void print_bt_config_components(GPtrArray *array) } static -void print_plugin_paths(struct bt_value *plugin_paths) +void print_plugin_paths(const struct bt_value *plugin_paths) { fprintf(stderr, " Plugin paths:\n"); print_value(stderr, plugin_paths, 4); @@ -608,7 +660,7 @@ void print_cfg_run(struct bt_config *cfg) { size_t i; - print_plugin_paths(bt_value_borrow_from_private(cfg->plugin_paths)); + print_plugin_paths(cfg->plugin_paths); fprintf(stderr, " Source component instances:\n"); print_bt_config_components(cfg->cmd_data.run.sources); @@ -639,19 +691,19 @@ void print_cfg_run(struct bt_config *cfg) static void print_cfg_list_plugins(struct bt_config *cfg) { - print_plugin_paths(bt_value_borrow_from_private(cfg->plugin_paths)); + print_plugin_paths(cfg->plugin_paths); } static void print_cfg_help(struct bt_config *cfg) { - print_plugin_paths(bt_value_borrow_from_private(cfg->plugin_paths)); + print_plugin_paths(cfg->plugin_paths); } static void print_cfg_print_ctf_metadata(struct bt_config *cfg) { - print_plugin_paths(bt_value_borrow_from_private(cfg->plugin_paths)); + print_plugin_paths(cfg->plugin_paths); fprintf(stderr, " Path: %s\n", cfg->cmd_data.print_ctf_metadata.path->str); } @@ -659,7 +711,7 @@ void print_cfg_print_ctf_metadata(struct bt_config *cfg) static void print_cfg_print_lttng_live_sessions(struct bt_config *cfg) { - print_plugin_paths(bt_value_borrow_from_private(cfg->plugin_paths)); + print_plugin_paths(cfg->plugin_paths); fprintf(stderr, " URL: %s\n", cfg->cmd_data.print_lttng_live_sessions.url->str); } @@ -667,7 +719,7 @@ void print_cfg_print_lttng_live_sessions(struct bt_config *cfg) static void print_cfg_query(struct bt_config *cfg) { - print_plugin_paths(bt_value_borrow_from_private(cfg->plugin_paths)); + print_plugin_paths(cfg->plugin_paths); fprintf(stderr, " Object: `%s`\n", cfg->cmd_data.query.object->str); fprintf(stderr, " Component class:\n"); print_bt_config_component(cfg->cmd_data.query.cfg_component); @@ -719,9 +771,9 @@ void add_to_loaded_plugins(struct bt_plugin_set *plugin_set) for (i = 0; i < count; i++) { struct bt_plugin *plugin = - bt_plugin_set_get_plugin(plugin_set, i); + bt_plugin_set_borrow_plugin_by_index(plugin_set, i); struct bt_plugin *loaded_plugin = - find_plugin(bt_plugin_get_name(plugin)); + find_plugin(bt_plugin_get_name(plugin)); BT_ASSERT(plugin); @@ -737,15 +789,14 @@ void add_to_loaded_plugins(struct bt_plugin_set *plugin_set) /* Add to global array. */ BT_LOGD("Adding plugin to loaded plugins: plugin-path=\"%s\"", bt_plugin_get_name(plugin)); - g_ptr_array_add(loaded_plugins, bt_object_get_ref(plugin)); + bt_object_get_ref(plugin); + g_ptr_array_add(loaded_plugins, plugin); } - - bt_object_put_ref(plugin); } } static -int load_dynamic_plugins(struct bt_value *plugin_paths) +int load_dynamic_plugins(const struct bt_value *plugin_paths) { int nr_paths, i, ret = 0; @@ -759,18 +810,14 @@ int load_dynamic_plugins(struct bt_value *plugin_paths) BT_LOGI("Loading dynamic plugins."); for (i = 0; i < nr_paths; i++) { - struct bt_value *plugin_path_value = NULL; + const struct bt_value *plugin_path_value = NULL; const char *plugin_path; struct bt_plugin_set *plugin_set; - enum bt_value_status status; - plugin_path_value = bt_value_array_borrow_element_by_index( - plugin_paths, i); - status = bt_value_string_get(plugin_path_value, &plugin_path); - if (status != BT_VALUE_STATUS_OK) { - BT_LOGD_STR("Cannot get plugin path string."); - continue; - } + plugin_path_value = + bt_value_array_borrow_element_by_index_const( + plugin_paths, i); + plugin_path = bt_value_string_get(plugin_path_value); /* * Skip this if the directory does not exist because @@ -818,7 +865,7 @@ end: } static -int load_all_plugins(struct bt_value *plugin_paths) +int load_all_plugins(const struct bt_value *plugin_paths) { int ret = 0; @@ -894,10 +941,11 @@ int cmd_query(struct bt_config *cfg) { int ret = 0; struct bt_component_class *comp_cls = NULL; - struct bt_value *results = NULL; + const struct bt_value *results = NULL; const char *fail_reason = NULL; - comp_cls = find_component_class(cfg->cmd_data.query.cfg_component->plugin_name->str, + comp_cls = find_component_class( + cfg->cmd_data.query.cfg_component->plugin_name->str, cfg->cmd_data.query.cfg_component->comp_cls_name->str, cfg->cmd_data.query.cfg_component->type); if (!comp_cls) { @@ -920,7 +968,7 @@ int cmd_query(struct bt_config *cfg) } ret = query(comp_cls, cfg->cmd_data.query.object->str, - bt_value_borrow_from_private(cfg->cmd_data.query.cfg_component->params), + cfg->cmd_data.query.cfg_component->params, &results, &fail_reason); if (ret) { goto failed; @@ -959,12 +1007,36 @@ end: return ret; } +static +void print_component_class_help(const char *plugin_name, + struct bt_component_class *comp_cls) +{ + const char *comp_class_name = + bt_component_class_get_name(comp_cls); + const char *comp_class_description = + bt_component_class_get_description(comp_cls); + const char *comp_class_help = + bt_component_class_get_help(comp_cls); + enum bt_component_class_type type = + bt_component_class_get_type(comp_cls); + + print_plugin_comp_cls_opt(stdout, plugin_name, comp_class_name, type); + printf("\n"); + printf(" %sDescription%s: %s\n", bt_common_color_bold(), + bt_common_color_reset(), + comp_class_description ? comp_class_description : "(None)"); + + if (comp_class_help) { + printf("\n%s\n", comp_class_help); + } +} + static int cmd_help(struct bt_config *cfg) { int ret = 0; struct bt_plugin *plugin = NULL; - size_t i; + struct bt_component_class *needed_comp_cls = NULL; plugin = find_plugin(cfg->cmd_data.help.cfg_component->plugin_name->str); if (!plugin) { @@ -980,86 +1052,106 @@ int cmd_help(struct bt_config *cfg) } print_plugin_info(plugin); - printf(" %sComponent classes%s: %d\n", + printf(" %sSource component classes%s: %d\n", bt_common_color_bold(), bt_common_color_reset(), - (int) bt_plugin_get_component_class_count(plugin)); + (int) bt_plugin_get_source_component_class_count(plugin)); + printf(" %sFilter component classes%s: %d\n", + bt_common_color_bold(), + bt_common_color_reset(), + (int) bt_plugin_get_filter_component_class_count(plugin)); + printf(" %sSink component classes%s: %d\n", + bt_common_color_bold(), + bt_common_color_reset(), + (int) bt_plugin_get_sink_component_class_count(plugin)); + if (strlen(cfg->cmd_data.help.cfg_component->comp_cls_name->str) == 0) { + /* Plugin help only */ + goto end; + } - if (cfg->cmd_data.help.cfg_component->type != - BT_COMPONENT_CLASS_TYPE_UNKNOWN) { - struct bt_component_class *needed_comp_cls = - find_component_class( - cfg->cmd_data.help.cfg_component->plugin_name->str, - cfg->cmd_data.help.cfg_component->comp_cls_name->str, - cfg->cmd_data.help.cfg_component->type); + needed_comp_cls = find_component_class( + cfg->cmd_data.help.cfg_component->plugin_name->str, + cfg->cmd_data.help.cfg_component->comp_cls_name->str, + cfg->cmd_data.help.cfg_component->type); + if (!needed_comp_cls) { + BT_LOGE("Cannot find component class: plugin-name=\"%s\", " + "comp-cls-name=\"%s\", comp-cls-type=%d", + cfg->cmd_data.help.cfg_component->plugin_name->str, + cfg->cmd_data.help.cfg_component->comp_cls_name->str, + cfg->cmd_data.help.cfg_component->type); + fprintf(stderr, "\n%s%sCannot find component class %s", + bt_common_color_bold(), + bt_common_color_fg_red(), + bt_common_color_reset()); + print_plugin_comp_cls_opt(stderr, + cfg->cmd_data.help.cfg_component->plugin_name->str, + cfg->cmd_data.help.cfg_component->comp_cls_name->str, + cfg->cmd_data.help.cfg_component->type); + fprintf(stderr, "\n"); + ret = -1; + goto end; + } - if (!needed_comp_cls) { - BT_LOGE("Cannot find component class: plugin-name=\"%s\", " - "comp-cls-name=\"%s\", comp-cls-type=%d", - cfg->cmd_data.help.cfg_component->plugin_name->str, - cfg->cmd_data.help.cfg_component->comp_cls_name->str, - cfg->cmd_data.help.cfg_component->type); - fprintf(stderr, "\n%s%sCannot find component class %s", - bt_common_color_bold(), - bt_common_color_fg_red(), - bt_common_color_reset()); - print_plugin_comp_cls_opt(stderr, - cfg->cmd_data.help.cfg_component->plugin_name->str, - cfg->cmd_data.help.cfg_component->comp_cls_name->str, - cfg->cmd_data.help.cfg_component->type); - fprintf(stderr, "\n"); - ret = -1; - goto end; - } + printf("\n"); + print_component_class_help( + cfg->cmd_data.help.cfg_component->plugin_name->str, + needed_comp_cls); + +end: + bt_object_put_ref(needed_comp_cls); + bt_object_put_ref(plugin); + return ret; +} + +typedef void *(* plugin_borrow_comp_cls_by_index_func_t)(struct bt_plugin *, + uint64_t); +typedef struct bt_component_class *(* spec_comp_cls_borrow_comp_cls_func_t)( + void *); + +void cmd_list_plugins_print_component_classes(struct bt_plugin *plugin, + const char *cc_type_name, uint64_t count, + plugin_borrow_comp_cls_by_index_func_t borrow_comp_cls_by_index_func, + spec_comp_cls_borrow_comp_cls_func_t spec_comp_cls_borrow_comp_cls_func) +{ + uint64_t i; - bt_object_put_ref(needed_comp_cls); + if (count == 0) { + printf(" %s%s component classes%s: (none)\n", cc_type_name, + bt_common_color_bold(), + bt_common_color_reset()); + goto end; + } else { + printf(" %s%s component classes%s:\n", cc_type_name, + bt_common_color_bold(), + bt_common_color_reset()); } - for (i = 0; i < bt_plugin_get_component_class_count(plugin); i++) { - struct bt_component_class *comp_cls = - bt_plugin_get_component_class_by_index(plugin, i); + for (i = 0; i < count; i++) { + struct bt_component_class *comp_class = + spec_comp_cls_borrow_comp_cls_func( + borrow_comp_cls_by_index_func(plugin, i)); const char *comp_class_name = - bt_component_class_get_name(comp_cls); + bt_component_class_get_name(comp_class); const char *comp_class_description = - bt_component_class_get_description(comp_cls); - const char *comp_class_help = - bt_component_class_get_help(comp_cls); + bt_component_class_get_description(comp_class); enum bt_component_class_type type = - bt_component_class_get_type(comp_cls); - - BT_ASSERT(comp_cls); - - if (cfg->cmd_data.help.cfg_component->type != - BT_COMPONENT_CLASS_TYPE_UNKNOWN) { - if (strcmp(cfg->cmd_data.help.cfg_component->comp_cls_name->str, - comp_class_name) != 0 || - type != cfg->cmd_data.help.cfg_component->type) { - bt_object_put_ref(comp_cls); - continue; - } - } + bt_component_class_get_type(comp_class); - printf("\n"); + printf(" "); print_plugin_comp_cls_opt(stdout, - cfg->cmd_data.help.cfg_component->plugin_name->str, - comp_class_name, + bt_plugin_get_name(plugin), comp_class_name, type); - printf("\n"); - printf(" %sDescription%s: %s\n", bt_common_color_bold(), - bt_common_color_reset(), - comp_class_description ? comp_class_description : "(None)"); - if (comp_class_help) { - printf("\n%s\n", comp_class_help); + if (comp_class_description) { + printf(": %s", comp_class_description); } - bt_object_put_ref(comp_cls); + printf("\n"); } end: - bt_object_put_ref(plugin); - return ret; + return; } static @@ -1069,7 +1161,7 @@ int cmd_list_plugins(struct bt_config *cfg) int plugins_count, component_classes_count = 0, i; printf("From the following plugin paths:\n\n"); - print_value(stdout, bt_value_borrow_from_private(cfg->plugin_paths), 2); + print_value(stdout, cfg->plugin_paths, 2); printf("\n"); plugins_count = loaded_plugins->len; if (plugins_count == 0) { @@ -1080,7 +1172,10 @@ int cmd_list_plugins(struct bt_config *cfg) for (i = 0; i < plugins_count; i++) { struct bt_plugin *plugin = g_ptr_array_index(loaded_plugins, i); - component_classes_count += bt_plugin_get_component_class_count(plugin); + component_classes_count += + bt_plugin_get_source_component_class_count(plugin) + + bt_plugin_get_filter_component_class_count(plugin) + + bt_plugin_get_sink_component_class_count(plugin); } printf("Found %s%d%s component classes in %s%d%s plugins.\n", @@ -1092,47 +1187,28 @@ int cmd_list_plugins(struct bt_config *cfg) bt_common_color_reset()); for (i = 0; i < plugins_count; i++) { - int j; struct bt_plugin *plugin = g_ptr_array_index(loaded_plugins, i); - component_classes_count = - bt_plugin_get_component_class_count(plugin); printf("\n"); print_plugin_info(plugin); - - if (component_classes_count == 0) { - printf(" %sComponent classes%s: (none)\n", - bt_common_color_bold(), - bt_common_color_reset()); - } else { - printf(" %sComponent classes%s:\n", - bt_common_color_bold(), - bt_common_color_reset()); - } - - for (j = 0; j < component_classes_count; j++) { - struct bt_component_class *comp_class = - bt_plugin_get_component_class_by_index( - plugin, j); - const char *comp_class_name = - bt_component_class_get_name(comp_class); - const char *comp_class_description = - bt_component_class_get_description(comp_class); - enum bt_component_class_type type = - bt_component_class_get_type(comp_class); - - printf(" "); - print_plugin_comp_cls_opt(stdout, - bt_plugin_get_name(plugin), comp_class_name, - type); - - if (comp_class_description) { - printf(": %s", comp_class_description); - } - - printf("\n"); - bt_object_put_ref(comp_class); - } + cmd_list_plugins_print_component_classes(plugin, "Source", + bt_plugin_get_source_component_class_count(plugin), + (plugin_borrow_comp_cls_by_index_func_t) + bt_plugin_borrow_source_component_class_by_name, + (spec_comp_cls_borrow_comp_cls_func_t) + bt_component_class_source_as_component_class); + cmd_list_plugins_print_component_classes(plugin, "Filter", + bt_plugin_get_filter_component_class_count(plugin), + (plugin_borrow_comp_cls_by_index_func_t) + bt_plugin_borrow_filter_component_class_by_name, + (spec_comp_cls_borrow_comp_cls_func_t) + bt_component_class_filter_as_component_class); + cmd_list_plugins_print_component_classes(plugin, "Sink", + bt_plugin_get_sink_component_class_count(plugin), + (plugin_borrow_comp_cls_by_index_func_t) + bt_plugin_borrow_sink_component_class_by_name, + (spec_comp_cls_borrow_comp_cls_func_t) + bt_component_class_sink_as_component_class); } end: @@ -1144,10 +1220,10 @@ int cmd_print_lttng_live_sessions(struct bt_config *cfg) { int ret = 0; struct bt_component_class *comp_cls = NULL; - struct bt_value *results = NULL; - struct bt_private_value *params = NULL; - struct bt_value *map = NULL; - struct bt_value *v = NULL; + const struct bt_value *results = NULL; + struct bt_value *params = NULL; + const struct bt_value *map = NULL; + const struct bt_value *v = NULL; static const char * const plugin_name = "ctf"; static const char * const comp_cls_name = "lttng-live"; static const enum bt_component_class_type comp_cls_type = @@ -1174,19 +1250,19 @@ int cmd_print_lttng_live_sessions(struct bt_config *cfg) goto error; } - params = bt_private_value_map_create(); + params = bt_value_map_create(); if (!params) { goto error; } - ret = bt_private_value_map_insert_string_entry(params, "url", + ret = bt_value_map_insert_string_entry(params, "url", cfg->cmd_data.print_lttng_live_sessions.url->str); if (ret) { goto error; } - ret = query(comp_cls, "sessions", bt_value_borrow_from_private(params), - &results, &fail_reason); + ret = query(comp_cls, "sessions", params, + &results, &fail_reason); if (ret) { goto failed; } @@ -1220,7 +1296,7 @@ int cmd_print_lttng_live_sessions(struct bt_config *cfg) const char *url_text; int64_t timer_us, streams, clients; - map = bt_value_array_borrow_element_by_index(results, i); + map = bt_value_array_borrow_element_by_index_const(results, i); if (!map) { BT_LOGE_STR("Unexpected empty array entry."); goto error; @@ -1230,37 +1306,33 @@ int cmd_print_lttng_live_sessions(struct bt_config *cfg) goto error; } - v = bt_value_map_borrow_entry_value(map, "url"); + v = bt_value_map_borrow_entry_value_const(map, "url"); if (!v) { BT_LOGE_STR("Unexpected empty array \"url\" entry."); goto error; } - ret = bt_value_string_get(v, &url_text); - BT_ASSERT(ret == 0); + url_text = bt_value_string_get(v); fprintf(out_stream, "%s", url_text); - v = bt_value_map_borrow_entry_value(map, "timer-us"); + v = bt_value_map_borrow_entry_value_const(map, "timer-us"); if (!v) { BT_LOGE_STR("Unexpected empty array \"timer-us\" entry."); goto error; } - ret = bt_value_integer_get(v, &timer_us); - BT_ASSERT(ret == 0); + timer_us = bt_value_integer_get(v); fprintf(out_stream, " (timer = %" PRIu64 ", ", timer_us); - v = bt_value_map_borrow_entry_value(map, "stream-count"); + v = bt_value_map_borrow_entry_value_const(map, "stream-count"); if (!v) { BT_LOGE_STR("Unexpected empty array \"stream-count\" entry."); goto error; } - ret = bt_value_integer_get(v, &streams); - BT_ASSERT(ret == 0); + streams = bt_value_integer_get(v); fprintf(out_stream, "%" PRIu64 " stream(s), ", streams); - v = bt_value_map_borrow_entry_value(map, "client-count"); + v = bt_value_map_borrow_entry_value_const(map, "client-count"); if (!v) { BT_LOGE_STR("Unexpected empty array \"client-count\" entry."); goto error; } - ret = bt_value_integer_get(v, &clients); - BT_ASSERT(ret == 0); + clients = bt_value_integer_get(v); fprintf(out_stream, "%" PRIu64 " client(s) connected)\n", clients); } @@ -1300,9 +1372,9 @@ int cmd_print_ctf_metadata(struct bt_config *cfg) { int ret = 0; struct bt_component_class *comp_cls = NULL; - struct bt_value *results = NULL; - struct bt_private_value *params = NULL; - struct bt_value *metadata_text_value = NULL; + const struct bt_value *results = NULL; + struct bt_value *params = NULL; + const struct bt_value *metadata_text_value = NULL; const char *metadata_text = NULL; static const char * const plugin_name = "ctf"; static const char * const comp_cls_name = "fs"; @@ -1330,13 +1402,13 @@ int cmd_print_ctf_metadata(struct bt_config *cfg) goto end; } - params = bt_private_value_map_create(); + params = bt_value_map_create(); if (!params) { ret = -1; goto end; } - ret = bt_private_value_map_insert_string_entry(params, "path", + ret = bt_value_map_insert_string_entry(params, "path", cfg->cmd_data.print_ctf_metadata.path->str); if (ret) { ret = -1; @@ -1344,20 +1416,20 @@ int cmd_print_ctf_metadata(struct bt_config *cfg) } ret = query(comp_cls, "metadata-info", - bt_value_borrow_from_private(params), &results, &fail_reason); + params, &results, &fail_reason); if (ret) { goto failed; } - metadata_text_value = bt_value_map_borrow_entry_value(results, "text"); + metadata_text_value = bt_value_map_borrow_entry_value_const(results, + "text"); if (!metadata_text_value) { BT_LOGE_STR("Cannot find `text` string value in the resulting metadata info object."); ret = -1; goto end; } - ret = bt_value_string_get(metadata_text_value, &metadata_text); - BT_ASSERT(ret == 0); + metadata_text = bt_value_string_get(metadata_text_value); if (cfg->cmd_data.print_ctf_metadata.output_path->len > 0) { out_stream = @@ -1390,7 +1462,6 @@ failed: bt_common_color_reset()); end: - destroy_the_query_executor(); bt_object_put_ref(results); bt_object_put_ref(params); bt_object_put_ref(comp_cls); @@ -1457,10 +1528,16 @@ void trace_range_destroy(gpointer data) struct cmd_run_ctx { /* Owned by this */ - GHashTable *components; + GHashTable *src_components; + + /* Owned by this */ + GHashTable *flt_components; + + /* Owned by this */ + GHashTable *sink_components; /* Owned by this */ - struct bt_graph *graph; + struct bt_private_graph *graph; /* Weak */ struct bt_config *cfg; @@ -1522,26 +1599,33 @@ char *s_from_ns(int64_t ns) static int cmd_run_ctx_connect_upstream_port_to_downstream_component( - struct cmd_run_ctx *ctx, struct bt_component *upstream_comp, - struct bt_port *upstream_port, + struct cmd_run_ctx *ctx, + struct bt_component *upstream_comp, + struct bt_port_output *out_upstream_port, struct bt_config_connection *cfg_conn) { + typedef uint64_t (*input_port_count_func_t)(void *); + typedef struct bt_port_input *(*borrow_input_port_by_index_func_t)( + void *, uint64_t); + struct bt_port *upstream_port = + bt_port_output_as_port(out_upstream_port); + int ret = 0; GQuark downstreamp_comp_name_quark; - struct bt_component *downstream_comp; - int64_t downstream_port_count; + void *downstream_comp; + uint64_t downstream_port_count; uint64_t i; - int64_t (*port_count_fn)(struct bt_component *); - struct bt_port *(*port_by_index_fn)(struct bt_component *, uint64_t); + input_port_count_func_t port_count_fn; + borrow_input_port_by_index_func_t port_by_index_fn; enum bt_graph_status status = BT_GRAPH_STATUS_ERROR; bool insert_trimmer = false; - struct bt_private_value *trimmer_params = NULL; + struct bt_value *trimmer_params = NULL; char *intersection_begin = NULL; char *intersection_end = NULL; - struct bt_component *trimmer = NULL; - struct bt_component_class *trimmer_class = NULL; - struct bt_port *trimmer_input = NULL; - struct bt_port *trimmer_output = NULL; + struct bt_component_filter *trimmer = NULL; + struct bt_component_class_filter *trimmer_class = NULL; + struct bt_port_input *trimmer_input = NULL; + struct bt_port_output *trimmer_output = NULL; if (ctx->intersections && bt_component_get_class_type(upstream_comp) == @@ -1571,17 +1655,17 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component( } insert_trimmer = true; - trimmer_params = bt_private_value_map_create(); + trimmer_params = bt_value_map_create(); if (!trimmer_params) { goto error; } - status = bt_private_value_map_insert_string_entry( + status = bt_value_map_insert_string_entry( trimmer_params, "begin", intersection_begin); if (status != BT_VALUE_STATUS_OK) { goto error; } - status = bt_private_value_map_insert_string_entry( + status = bt_value_map_insert_string_entry( trimmer_params, "end", intersection_end); if (status != BT_VALUE_STATUS_OK) { @@ -1589,8 +1673,7 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component( } } - trimmer_class = find_component_class("utils", "trimmer", - BT_COMPONENT_CLASS_TYPE_FILTER); + trimmer_class = find_filter_component_class("utils", "trimmer"); if (!trimmer_class) { goto error; } @@ -1605,8 +1688,22 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component( downstreamp_comp_name_quark = g_quark_from_string( cfg_conn->downstream_comp_name->str); BT_ASSERT(downstreamp_comp_name_quark > 0); - downstream_comp = g_hash_table_lookup(ctx->components, + downstream_comp = g_hash_table_lookup(ctx->flt_components, GUINT_TO_POINTER(downstreamp_comp_name_quark)); + port_count_fn = (input_port_count_func_t) + bt_component_filter_get_input_port_count; + port_by_index_fn = (borrow_input_port_by_index_func_t) + bt_component_filter_borrow_input_port_by_index; + + if (!downstream_comp) { + downstream_comp = g_hash_table_lookup(ctx->sink_components, + GUINT_TO_POINTER(downstreamp_comp_name_quark)); + port_count_fn = (input_port_count_func_t) + bt_component_sink_get_input_port_count; + port_by_index_fn = (borrow_input_port_by_index_func_t) + bt_component_sink_borrow_input_port_by_index; + } + if (!downstream_comp) { BT_LOGE("Cannot find downstream component: comp-name=\"%s\", " "conn-arg=\"%s\"", cfg_conn->downstream_comp_name->str, @@ -1616,28 +1713,14 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component( goto error; } - if (bt_component_is_filter(downstream_comp)) { - port_count_fn = bt_component_filter_get_input_port_count; - port_by_index_fn = bt_component_filter_get_input_port_by_index; - } else if (bt_component_is_sink(downstream_comp)) { - port_count_fn = bt_component_sink_get_input_port_count; - port_by_index_fn = bt_component_sink_get_input_port_by_index; - } else { - /* - * Should never happen because the connections are - * validated before we get here. - */ - BT_LOGF("Invalid connection: downstream component is a source: " - "conn-arg=\"%s\"", cfg_conn->arg->str); - abort(); - } - downstream_port_count = port_count_fn(downstream_comp); BT_ASSERT(downstream_port_count >= 0); for (i = 0; i < downstream_port_count; i++) { - struct bt_port *downstream_port = + struct bt_port_input *in_downstream_port = port_by_index_fn(downstream_comp, i); + struct bt_port *downstream_port = + bt_port_input_as_port(in_downstream_port); const char *upstream_port_name; const char *downstream_port_name; @@ -1645,7 +1728,6 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component( /* Skip port if it's already connected. */ if (bt_port_is_connected(downstream_port)) { - bt_object_put_ref(downstream_port); BT_LOGD("Skipping downstream port: already connected: " "port-addr=%p, port-name=\"%s\"", downstream_port, @@ -1661,29 +1743,30 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component( if (!bt_common_star_glob_match( cfg_conn->downstream_port_glob->str, SIZE_MAX, downstream_port_name, SIZE_MAX)) { - bt_object_put_ref(downstream_port); continue; } if (insert_trimmer) { /* - * In order to insert the trimmer between the two - * components that were being connected, we create - * a connection configuration entry which describes - * a connection from the trimmer's output to the - * original input that was being connected. + * In order to insert the trimmer between the + * two components that were being connected, we + * create a connection configuration entry which + * describes a connection from the trimmer's + * output to the original input that was being + * connected. * - * Hence, the creation of the trimmer will cause the - * graph "new port" listener to establish all downstream - * connections as its output port is connected. We will - * then establish the connection between the original - * upstream source and the trimmer. + * Hence, the creation of the trimmer will cause + * the graph "new port" listener to establish + * all downstream connections as its output port + * is connected. We will then establish the + * connection between the original upstream + * source and the trimmer. */ char *trimmer_name = NULL; enum bt_graph_status graph_status; - ret = asprintf(&trimmer_name, "%s-%s", - "stream-intersection-trimmer", + ret = asprintf(&trimmer_name, + "stream-intersection-trimmer-%s", upstream_port_name); if (ret < 0) { goto error; @@ -1691,9 +1774,9 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component( ret = 0; ctx->connect_ports = false; - graph_status = bt_graph_add_component(ctx->graph, - trimmer_class, trimmer_name, - bt_value_borrow_from_private(trimmer_params), + graph_status = bt_private_graph_add_filter_component( + ctx->graph, trimmer_class, trimmer_name, + trimmer_params, &trimmer); free(trimmer_name); if (graph_status != BT_GRAPH_STATUS_OK) { @@ -1702,13 +1785,13 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component( BT_ASSERT(trimmer); trimmer_input = - bt_component_filter_get_input_port_by_index( + bt_component_filter_borrow_input_port_by_index( trimmer, 0); if (!trimmer_input) { goto error; } trimmer_output = - bt_component_filter_get_output_port_by_index( + bt_component_filter_borrow_output_port_by_index( trimmer, 0); if (!trimmer_output) { goto error; @@ -1718,18 +1801,18 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component( * Replace the current downstream port by the trimmer's * upstream port. */ - BT_OBJECT_MOVE_REF(downstream_port, trimmer_input); + in_downstream_port = trimmer_input; + downstream_port = + bt_port_input_as_port(in_downstream_port); downstream_port_name = bt_port_get_name( downstream_port); - if (!downstream_port_name) { - goto error; - } + BT_ASSERT(downstream_port_name); } /* We have a winner! */ - status = bt_graph_connect_ports(ctx->graph, - upstream_port, downstream_port, NULL); - BT_OBJECT_PUT_REF_AND_RESET(downstream_port); + status = bt_private_graph_connect_ports(ctx->graph, + out_upstream_port, in_downstream_port, NULL); + downstream_port = NULL; switch (status) { case BT_GRAPH_STATUS_OK: break; @@ -1794,7 +1877,9 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component( * original downstream port. */ ret = cmd_run_ctx_connect_upstream_port_to_downstream_component( - ctx, trimmer, trimmer_output, cfg_conn); + ctx, + bt_component_filter_as_component(trimmer), + trimmer_output, cfg_conn); if (ret) { goto error; } @@ -1828,14 +1913,12 @@ end: BT_OBJECT_PUT_REF_AND_RESET(trimmer_params); BT_OBJECT_PUT_REF_AND_RESET(trimmer_class); BT_OBJECT_PUT_REF_AND_RESET(trimmer); - BT_OBJECT_PUT_REF_AND_RESET(trimmer_input); - BT_OBJECT_PUT_REF_AND_RESET(trimmer_output); return ret; } static int cmd_run_ctx_connect_upstream_port(struct cmd_run_ctx *ctx, - struct bt_port *upstream_port) + struct bt_port_output *upstream_port) { int ret = 0; const char *upstream_port_name; @@ -1845,9 +1928,11 @@ int cmd_run_ctx_connect_upstream_port(struct cmd_run_ctx *ctx, BT_ASSERT(ctx); BT_ASSERT(upstream_port); - upstream_port_name = bt_port_get_name(upstream_port); + upstream_port_name = bt_port_get_name( + bt_port_output_as_port(upstream_port)); BT_ASSERT(upstream_port_name); - upstream_comp = bt_port_get_component(upstream_port); + upstream_comp = bt_port_borrow_component( + bt_port_output_as_port(upstream_port)); if (!upstream_comp) { BT_LOGW("Upstream port to connect is not part of a component: " "port-addr=%p, port-name=\"%s\"", @@ -1907,17 +1992,17 @@ error: ret = -1; end: - bt_object_put_ref(upstream_comp); return ret; } static -void graph_port_added_listener(struct bt_port *port, void *data) +void graph_output_port_added_listener(struct cmd_run_ctx *ctx, + struct bt_port_output *out_port) { - struct bt_component *comp = NULL; - struct cmd_run_ctx *ctx = data; + struct bt_component *comp; + struct bt_port *port = bt_port_output_as_port(out_port); - comp = bt_port_get_component(port); + comp = bt_port_borrow_component(port); BT_LOGI("Port added to a graph's component: comp-addr=%p, " "comp-name=\"%s\", port-addr=%p, port-name=\"%s\"", comp, comp ? bt_component_get_name(comp) : "", @@ -1937,66 +2022,30 @@ void graph_port_added_listener(struct bt_port *port, void *data) goto end; } - if (!bt_port_is_output(port)) { - BT_LOGI_STR("Skipping input port."); - goto end; - } - - if (cmd_run_ctx_connect_upstream_port(ctx, port)) { + if (cmd_run_ctx_connect_upstream_port(ctx, out_port)) { BT_LOGF_STR("Cannot connect upstream port."); fprintf(stderr, "Added port could not be connected: aborting\n"); abort(); } end: - bt_object_put_ref(comp); return; } static -void graph_port_removed_listener(struct bt_component *component, - struct bt_port *port, void *data) -{ - BT_LOGI("Port removed from a graph's component: comp-addr=%p, " - "comp-name=\"%s\", port-addr=%p, port-name=\"%s\"", - component, bt_component_get_name(component), - port, bt_port_get_name(port)); -} - -static -void graph_ports_connected_listener(struct bt_port *upstream_port, - struct bt_port *downstream_port, void *data) +void graph_source_output_port_added_listener( + struct bt_component_source *component, + struct bt_port_output *port, void *data) { - struct bt_component *upstream_comp = bt_port_get_component(upstream_port); - struct bt_component *downstream_comp = bt_port_get_component(downstream_port); - - BT_ASSERT(upstream_comp); - BT_ASSERT(downstream_comp); - BT_LOGI("Graph's component ports connected: " - "upstream-comp-addr=%p, upstream-comp-name=\"%s\", " - "upstream-port-addr=%p, upstream-port-name=\"%s\", " - "downstream-comp-addr=%p, downstream-comp-name=\"%s\", " - "downstream-port-addr=%p, downstream-port-name=\"%s\"", - upstream_comp, bt_component_get_name(upstream_comp), - upstream_port, bt_port_get_name(upstream_port), - downstream_comp, bt_component_get_name(downstream_comp), - downstream_port, bt_port_get_name(downstream_port)); - bt_object_put_ref(upstream_comp); - bt_object_put_ref(downstream_comp); + graph_output_port_added_listener(data, port); } static -void graph_ports_disconnected_listener( - struct bt_component *upstream_component, - struct bt_component *downstream_component, - struct bt_port *upstream_port, struct bt_port *downstream_port, - void *data) +void graph_filter_output_port_added_listener( + struct bt_component_filter *component, + struct bt_port_output *port, void *data) { - BT_LOGI("Graph's component ports disconnected: " - "upstream-port-addr=%p, upstream-port-name=\"%s\", " - "downstream-port-addr=%p, downstream-port-name=\"%s\"", - upstream_port, bt_port_get_name(upstream_port), - downstream_port, bt_port_get_name(downstream_port)); + graph_output_port_added_listener(data, port); } static @@ -2006,9 +2055,19 @@ void cmd_run_ctx_destroy(struct cmd_run_ctx *ctx) return; } - if (ctx->components) { - g_hash_table_destroy(ctx->components); - ctx->components = NULL; + if (ctx->src_components) { + g_hash_table_destroy(ctx->src_components); + ctx->src_components = NULL; + } + + if (ctx->flt_components) { + g_hash_table_destroy(ctx->flt_components); + ctx->flt_components = NULL; + } + + if (ctx->sink_components) { + g_hash_table_destroy(ctx->sink_components); + ctx->sink_components = NULL; } if (ctx->intersections) { @@ -2025,12 +2084,25 @@ static int cmd_run_ctx_init(struct cmd_run_ctx *ctx, struct bt_config *cfg) { int ret = 0; + enum bt_graph_status status; ctx->cfg = cfg; ctx->connect_ports = false; - ctx->components = g_hash_table_new_full(g_direct_hash, g_direct_equal, - NULL, bt_object_put_ref); - if (!ctx->components) { + ctx->src_components = g_hash_table_new_full(g_direct_hash, + g_direct_equal, NULL, (GDestroyNotify) bt_object_put_ref); + if (!ctx->src_components) { + goto error; + } + + ctx->flt_components = g_hash_table_new_full(g_direct_hash, + g_direct_equal, NULL, (GDestroyNotify) bt_object_put_ref); + if (!ctx->flt_components) { + goto error; + } + + ctx->sink_components = g_hash_table_new_full(g_direct_hash, + g_direct_equal, NULL, (GDestroyNotify) bt_object_put_ref); + if (!ctx->sink_components) { goto error; } @@ -2043,37 +2115,25 @@ int cmd_run_ctx_init(struct cmd_run_ctx *ctx, struct bt_config *cfg) } } - ctx->graph = bt_graph_create(); + ctx->graph = bt_private_graph_create(); if (!ctx->graph) { goto error; } the_graph = ctx->graph; - ret = bt_graph_add_port_added_listener(ctx->graph, - graph_port_added_listener, NULL, ctx); - if (ret < 0) { + status = bt_private_graph_add_source_component_output_port_added_listener( + ctx->graph, graph_source_output_port_added_listener, NULL, ctx, + NULL); + if (status != BT_GRAPH_STATUS_OK) { BT_LOGE_STR("Cannot add \"port added\" listener to graph."); goto error; } - ret = bt_graph_add_port_removed_listener(ctx->graph, - graph_port_removed_listener, NULL, ctx); - if (ret < 0) { - BT_LOGE_STR("Cannot add \"port removed\" listener to graph."); - goto error; - } - - ret = bt_graph_add_ports_connected_listener(ctx->graph, - graph_ports_connected_listener, NULL, ctx); - if (ret < 0) { - BT_LOGE_STR("Cannot add \"ports connected\" listener to graph."); - goto error; - } - - ret = bt_graph_add_ports_disconnected_listener(ctx->graph, - graph_ports_disconnected_listener, NULL, ctx); - if (ret < 0) { - BT_LOGE_STR("Cannot add \"ports disconnected\" listener to graph."); + status = bt_private_graph_add_filter_component_output_port_added_listener( + ctx->graph, graph_filter_output_port_added_listener, NULL, ctx, + NULL); + if (status != BT_GRAPH_STATUS_OK) { + BT_LOGE_STR("Cannot add \"port added\" listener to graph."); goto error; } @@ -2090,31 +2150,32 @@ end: static int set_stream_intersections(struct cmd_run_ctx *ctx, struct bt_config_component *cfg_comp, - struct bt_component_class *comp_cls) + struct bt_component_class_source *src_comp_cls) { int ret = 0; uint64_t trace_idx; int64_t trace_count; enum bt_value_status value_status; const char *path = NULL; - struct bt_value *component_path_value = NULL; - struct bt_private_value *query_params = NULL; - struct bt_value *query_result = NULL; - struct bt_value *trace_info = NULL; - struct bt_value *intersection_range = NULL; - struct bt_value *intersection_begin = NULL; - struct bt_value *intersection_end = NULL; - struct bt_value *stream_path_value = NULL; - struct bt_value *stream_paths = NULL; - struct bt_value *stream_infos = NULL; - struct bt_value *stream_info = NULL; + const struct bt_value *component_path_value = NULL; + struct bt_value *query_params = NULL; + const struct bt_value *query_result = NULL; + const struct bt_value *trace_info = NULL; + const struct bt_value *intersection_range = NULL; + const struct bt_value *intersection_begin = NULL; + const struct bt_value *intersection_end = NULL; + const struct bt_value *stream_path_value = NULL; + const struct bt_value *stream_paths = NULL; + const struct bt_value *stream_infos = NULL; + const struct bt_value *stream_info = NULL; struct port_id *port_id = NULL; struct trace_range *trace_range = NULL; const char *fail_reason = NULL; + struct bt_component_class *comp_cls = + bt_component_class_source_as_component_class(src_comp_cls); - component_path_value = bt_value_map_borrow_entry_value( - bt_value_borrow_from_private(cfg_comp->params), - "path"); + component_path_value = bt_value_map_borrow_entry_value(cfg_comp->params, + "path"); if (component_path_value && !bt_value_is_string(component_path_value)) { BT_LOGD("Cannot get path parameter: component-name=%s", cfg_comp->instance_name->str); @@ -2122,23 +2183,16 @@ int set_stream_intersections(struct cmd_run_ctx *ctx, goto error; } - value_status = bt_value_string_get(component_path_value, &path); - if (value_status != BT_VALUE_STATUS_OK) { - BT_LOGD("Cannot get path string value: component-name=%s", - cfg_comp->instance_name->str); - ret = -1; - goto error; - } - - query_params = bt_private_value_map_create(); + path = bt_value_string_get(component_path_value); + query_params = bt_value_map_create(); if (!query_params) { BT_LOGE_STR("Cannot create query parameters."); ret = -1; goto error; } - value_status = bt_private_value_map_insert_entry(query_params, "path", - component_path_value); + value_status = bt_value_map_insert_string_entry(query_params, "path", + path); if (value_status != BT_VALUE_STATUS_OK) { BT_LOGE_STR("Cannot insert path parameter in query parameter map."); ret = -1; @@ -2146,7 +2200,7 @@ int set_stream_intersections(struct cmd_run_ctx *ctx, } ret = query(comp_cls, "trace-info", - bt_value_borrow_from_private(query_params), &query_result, + query_params, &query_result, &fail_reason); if (ret) { BT_LOGD("Component class does not support the `trace-info` query: %s: " @@ -2177,7 +2231,7 @@ int set_stream_intersections(struct cmd_run_ctx *ctx, uint64_t stream_idx; int64_t stream_count; - trace_info = bt_value_array_borrow_element_by_index( + trace_info = bt_value_array_borrow_element_by_index_const( query_result, trace_idx); if (!trace_info || !bt_value_is_map(trace_info)) { ret = -1; @@ -2185,43 +2239,32 @@ int set_stream_intersections(struct cmd_run_ctx *ctx, goto error; } - intersection_range = bt_value_map_borrow_entry_value(trace_info, - "intersection-range-ns"); + intersection_range = bt_value_map_borrow_entry_value_const( + trace_info, "intersection-range-ns"); if (!intersection_range) { ret = -1; BT_LOGD_STR("Cannot retrieve \'intersetion-range-ns\' field from query result."); goto error; } - intersection_begin = bt_value_map_borrow_entry_value( - intersection_range, "begin"); + intersection_begin = bt_value_map_borrow_entry_value_const(intersection_range, + "begin"); if (!intersection_begin) { ret = -1; BT_LOGD_STR("Cannot retrieve intersection-range-ns \'begin\' field from query result."); goto error; } - intersection_end = bt_value_map_borrow_entry_value( - intersection_range, "end"); + intersection_end = bt_value_map_borrow_entry_value_const(intersection_range, + "end"); if (!intersection_end) { ret = -1; BT_LOGD_STR("Cannot retrieve intersection-range-ns \'end\' field from query result."); goto error; } - value_status = bt_value_integer_get(intersection_begin, &begin); - if (value_status != BT_VALUE_STATUS_OK) { - ret = -1; - BT_LOGD_STR("Cannot retrieve value of intersection-range-ns \'begin\' field from query result."); - goto error; - } - - value_status = bt_value_integer_get(intersection_end, &end); - if (value_status != BT_VALUE_STATUS_OK) { - ret = -1; - BT_LOGD_STR("Cannot retrieve value of intersection-range-ns \'end\' field from query result."); - goto error; - } + begin = bt_value_integer_get(intersection_begin); + end = bt_value_integer_get(intersection_end); if (begin < 0 || end < 0 || end < begin) { BT_LOGW("Invalid trace stream intersection values: " @@ -2232,8 +2275,8 @@ int set_stream_intersections(struct cmd_run_ctx *ctx, goto error; } - stream_infos = bt_value_map_borrow_entry_value(trace_info, - "streams"); + stream_infos = bt_value_map_borrow_entry_value_const(trace_info, + "streams"); if (!stream_infos || !bt_value_is_array(stream_infos)) { ret = -1; BT_LOGD_STR("Cannot retrieve stream information from trace in query result."); @@ -2283,7 +2326,7 @@ int set_stream_intersections(struct cmd_run_ctx *ctx, trace_range->intersection_range_begin_ns = begin; trace_range->intersection_range_end_ns = end; - stream_info = bt_value_array_borrow_element_by_index( + stream_info = bt_value_array_borrow_element_by_index_const( stream_infos, stream_idx); if (!stream_info || !bt_value_is_map(stream_info)) { ret = -1; @@ -2291,8 +2334,8 @@ int set_stream_intersections(struct cmd_run_ctx *ctx, goto error; } - stream_paths = bt_value_map_borrow_entry_value( - stream_info, "paths"); + stream_paths = bt_value_map_borrow_entry_value_const(stream_info, + "paths"); if (!stream_paths || !bt_value_is_array(stream_paths)) { ret = -1; BT_LOGD_STR("Cannot retrieve stream paths from trace in query result."); @@ -2300,7 +2343,7 @@ int set_stream_intersections(struct cmd_run_ctx *ctx, } stream_path_value = - bt_value_array_borrow_element_by_index( + bt_value_array_borrow_element_by_index_const( stream_paths, 0); if (!stream_path_value || !bt_value_is_string(stream_path_value)) { @@ -2309,13 +2352,7 @@ int set_stream_intersections(struct cmd_run_ctx *ctx, goto error; } - value_status = bt_value_string_get(stream_path_value, - &stream_path); - if (value_status != BT_VALUE_STATUS_OK) { - ret = -1; - goto error; - } - + stream_path = bt_value_string_get(stream_path_value); port_id->port_name = strdup(stream_path); if (!port_id->port_name) { ret = -1; @@ -2353,8 +2390,8 @@ int cmd_run_ctx_create_components_from_config_components( struct cmd_run_ctx *ctx, GPtrArray *cfg_components) { size_t i; - struct bt_component_class *comp_cls = NULL; - struct bt_component *comp = NULL; + void *comp_cls = NULL; + void *comp = NULL; int ret = 0; for (i = 0; i < cfg_components->len; i++) { @@ -2362,8 +2399,26 @@ int cmd_run_ctx_create_components_from_config_components( g_ptr_array_index(cfg_components, i); GQuark quark; - comp_cls = find_component_class(cfg_comp->plugin_name->str, - cfg_comp->comp_cls_name->str, cfg_comp->type); + switch (cfg_comp->type) { + case BT_COMPONENT_CLASS_TYPE_SOURCE: + comp_cls = find_source_component_class( + cfg_comp->plugin_name->str, + cfg_comp->comp_cls_name->str); + break; + case BT_COMPONENT_CLASS_TYPE_FILTER: + comp_cls = find_filter_component_class( + cfg_comp->plugin_name->str, + cfg_comp->comp_cls_name->str); + break; + case BT_COMPONENT_CLASS_TYPE_SINK: + comp_cls = find_sink_component_class( + cfg_comp->plugin_name->str, + cfg_comp->comp_cls_name->str); + break; + default: + abort(); + } + if (!comp_cls) { BT_LOGE("Cannot find component class: plugin-name=\"%s\", " "comp-cls-name=\"%s\", comp-cls-type=%d", @@ -2382,9 +2437,29 @@ int cmd_run_ctx_create_components_from_config_components( goto error; } - ret = bt_graph_add_component(ctx->graph, comp_cls, - cfg_comp->instance_name->str, - bt_value_borrow_from_private(cfg_comp->params), &comp); + switch (cfg_comp->type) { + case BT_COMPONENT_CLASS_TYPE_SOURCE: + ret = bt_private_graph_add_source_component(ctx->graph, + comp_cls, cfg_comp->instance_name->str, + cfg_comp->params, + (void *) &comp); + break; + case BT_COMPONENT_CLASS_TYPE_FILTER: + ret = bt_private_graph_add_filter_component(ctx->graph, + comp_cls, cfg_comp->instance_name->str, + cfg_comp->params, + (void *) &comp); + break; + case BT_COMPONENT_CLASS_TYPE_SINK: + ret = bt_private_graph_add_sink_component(ctx->graph, + comp_cls, cfg_comp->instance_name->str, + cfg_comp->params, + (void *) &comp); + break; + default: + abort(); + } + if (ret) { BT_LOGE("Cannot create component: plugin-name=\"%s\", " "comp-cls-name=\"%s\", comp-cls-type=%d, " @@ -2412,8 +2487,24 @@ int cmd_run_ctx_create_components_from_config_components( comp, cfg_comp->instance_name->str); quark = g_quark_from_string(cfg_comp->instance_name->str); BT_ASSERT(quark > 0); - g_hash_table_insert(ctx->components, - GUINT_TO_POINTER(quark), comp); + + switch (cfg_comp->type) { + case BT_COMPONENT_CLASS_TYPE_SOURCE: + g_hash_table_insert(ctx->src_components, + GUINT_TO_POINTER(quark), comp); + break; + case BT_COMPONENT_CLASS_TYPE_FILTER: + g_hash_table_insert(ctx->flt_components, + GUINT_TO_POINTER(quark), comp); + break; + case BT_COMPONENT_CLASS_TYPE_SINK: + g_hash_table_insert(ctx->sink_components, + GUINT_TO_POINTER(quark), comp); + break; + default: + abort(); + } + comp = NULL; BT_OBJECT_PUT_REF_AND_RESET(comp_cls); } @@ -2467,25 +2558,27 @@ end: return ret; } +typedef uint64_t (*output_port_count_func_t)(void *); +typedef struct bt_port_output *(*borrow_output_port_by_index_func_t)( + void *, uint64_t); + static int cmd_run_ctx_connect_comp_ports(struct cmd_run_ctx *ctx, - struct bt_component *comp, - int64_t (*port_count_fn)(struct bt_component *), - struct bt_port *(*port_by_index_fn)(struct bt_component *, uint64_t)) + void *comp, output_port_count_func_t port_count_fn, + borrow_output_port_by_index_func_t port_by_index_fn) { int ret = 0; - int64_t count; + uint64_t count; uint64_t i; count = port_count_fn(comp); BT_ASSERT(count >= 0); for (i = 0; i < count; i++) { - struct bt_port *upstream_port = port_by_index_fn(comp, i); + struct bt_port_output *upstream_port = port_by_index_fn(comp, i); BT_ASSERT(upstream_port); ret = cmd_run_ctx_connect_upstream_port(ctx, upstream_port); - bt_object_put_ref(upstream_port); if (ret) { goto end; } @@ -2503,28 +2596,27 @@ int cmd_run_ctx_connect_ports(struct cmd_run_ctx *ctx) gpointer g_name_quark, g_comp; ctx->connect_ports = true; - g_hash_table_iter_init(&iter, ctx->components); + g_hash_table_iter_init(&iter, ctx->src_components); while (g_hash_table_iter_next(&iter, &g_name_quark, &g_comp)) { - int64_t (*port_count_fn)(struct bt_component *); - struct bt_port *(*port_by_index_fn)(struct bt_component *, uint64_t); - - if (bt_component_is_source(g_comp)) { - port_count_fn = - bt_component_source_get_output_port_count; - port_by_index_fn = - bt_component_source_get_output_port_by_index; - } else if (bt_component_is_filter(g_comp)) { - port_count_fn = - bt_component_filter_get_output_port_count; - port_by_index_fn = - bt_component_filter_get_output_port_by_index; - } else { - continue; + ret = cmd_run_ctx_connect_comp_ports(ctx, g_comp, + (output_port_count_func_t) + bt_component_source_get_output_port_count, + (borrow_output_port_by_index_func_t) + bt_component_source_borrow_output_port_by_index); + if (ret) { + goto end; } + } - ret = cmd_run_ctx_connect_comp_ports(ctx, - g_comp, port_count_fn, port_by_index_fn); + g_hash_table_iter_init(&iter, ctx->flt_components); + + while (g_hash_table_iter_next(&iter, &g_name_quark, &g_comp)) { + ret = cmd_run_ctx_connect_comp_ports(ctx, g_comp, + (output_port_count_func_t) + bt_component_filter_get_output_port_count, + (borrow_output_port_by_index_func_t) + bt_component_filter_borrow_output_port_by_index); if (ret) { goto end; } @@ -2538,20 +2630,22 @@ static inline const char *bt_graph_status_str(enum bt_graph_status status) { switch (status) { - case BT_GRAPH_STATUS_CANCELED: - return "BT_GRAPH_STATUS_CANCELED"; - case BT_GRAPH_STATUS_AGAIN: - return "BT_GRAPH_STATUS_AGAIN"; - case BT_GRAPH_STATUS_END: - return "BT_GRAPH_STATUS_END"; case BT_GRAPH_STATUS_OK: return "BT_GRAPH_STATUS_OK"; - case BT_GRAPH_STATUS_INVALID: - return "BT_GRAPH_STATUS_INVALID"; - case BT_GRAPH_STATUS_NO_SINK: - return "BT_GRAPH_STATUS_NO_SINK"; + case BT_GRAPH_STATUS_END: + return "BT_GRAPH_STATUS_END"; + case BT_GRAPH_STATUS_AGAIN: + return "BT_GRAPH_STATUS_AGAIN"; + case BT_GRAPH_STATUS_COMPONENT_REFUSES_PORT_CONNECTION: + return "BT_GRAPH_STATUS_COMPONENT_REFUSES_PORT_CONNECTION"; + case BT_GRAPH_STATUS_CANCELED: + return "BT_GRAPH_STATUS_CANCELED"; case BT_GRAPH_STATUS_ERROR: return "BT_GRAPH_STATUS_ERROR"; + case BT_GRAPH_STATUS_NO_SINK: + return "BT_GRAPH_STATUS_NO_SINK"; + case BT_GRAPH_STATUS_NOMEM: + return "BT_GRAPH_STATUS_NOMEM"; default: return "(unknown)"; } @@ -2607,7 +2701,7 @@ int cmd_run(struct bt_config *cfg) /* Run the graph */ while (true) { - enum bt_graph_status graph_status = bt_graph_run(ctx.graph); + enum bt_graph_status graph_status = bt_private_graph_run(ctx.graph); /* * Reset console in case something messed with console @@ -2616,7 +2710,7 @@ int cmd_run(struct bt_config *cfg) printf("%s", bt_common_color_reset()); fflush(stdout); fprintf(stderr, "%s", bt_common_color_reset()); - BT_LOGV("bt_graph_run() returned: status=%s", + BT_LOGV("bt_private_graph_run() returned: status=%s", bt_graph_status_str(graph_status)); switch (graph_status) { @@ -2626,7 +2720,8 @@ int cmd_run(struct bt_config *cfg) BT_LOGI_STR("Graph was canceled by user."); goto error; case BT_GRAPH_STATUS_AGAIN: - if (bt_graph_is_canceled(ctx.graph)) { + if (bt_graph_is_canceled( + bt_private_graph_as_graph(ctx.graph))) { BT_LOGI_STR("Graph was canceled by user."); goto error; } @@ -2637,14 +2732,15 @@ int cmd_run(struct bt_config *cfg) cfg->cmd_data.run.retry_duration_us); if (usleep(cfg->cmd_data.run.retry_duration_us)) { - if (bt_graph_is_canceled(ctx.graph)) { + if (bt_graph_is_canceled( + bt_private_graph_as_graph(ctx.graph))) { BT_LOGI_STR("Graph was canceled by user."); goto error; } } } break; - case BT_COMPONENT_STATUS_END: + case BT_GRAPH_STATUS_END: goto end; default: BT_LOGE_STR("Graph failed to complete successfully"); @@ -2859,8 +2955,7 @@ int main(int argc, const char **argv) print_cfg(cfg); if (cfg->command_needs_plugins) { - ret = load_all_plugins( - bt_value_borrow_from_private(cfg->plugin_paths)); + ret = load_all_plugins(cfg->plugin_paths); if (ret) { BT_LOGE("Failed to load plugins: ret=%d", ret); retcode = 1;