X-Git-Url: http://git.efficios.com/?a=blobdiff_plain;f=src%2Fcli%2Fbabeltrace2.c;h=0b026f78207d81791b2c133104a0ac50a52f8789;hb=d24d56638469189904fb6ddbb3c725817b3e9417;hp=4f786c2e3d6ebad141c22b0aa599b0a0a0ede92e;hpb=55478183b5a100be028ed476718f0773ec7b4af5;p=babeltrace.git diff --git a/src/cli/babeltrace2.c b/src/cli/babeltrace2.c index 4f786c2e..0b026f78 100644 --- a/src/cli/babeltrace2.c +++ b/src/cli/babeltrace2.c @@ -49,25 +49,8 @@ * modules. */ static const char* log_level_env_var_names[] = { - "BABELTRACE_COMMON_LOG_LEVEL", - "BABELTRACE_COMPAT_LOG_LEVEL", - "BABELTRACE_CTFSER_LOG_LEVEL", - "BABELTRACE_FD_CACHE_LOG_LEVEL", - "BABELTRACE_FLT_LTTNG_UTILS_DEBUG_INFO_LOG_LEVEL", - "BABELTRACE_FLT_UTILS_COUNTER_LOG_LEVEL", - "BABELTRACE_FLT_UTILS_MUXER_LOG_LEVEL", - "BABELTRACE_FLT_UTILS_TRIMMER_LOG_LEVEL", - "BABELTRACE_PLUGIN_CTF_BFCR_LOG_LEVEL", "BABELTRACE_PLUGIN_CTF_METADATA_LOG_LEVEL", - "BABELTRACE_PLUGIN_CTF_MSG_ITER_LOG_LEVEL", - "BABELTRACE_PLUGIN_CTF_UTILS_LOG_LEVEL", "BABELTRACE_PYTHON_BT2_LOG_LEVEL", - "BABELTRACE_SINK_CTF_FS_LOG_LEVEL", - "BABELTRACE_SINK_TEXT_PRETTY_LOG_LEVEL", - "BABELTRACE_SINK_TEXT_DETAILS_LOG_LEVEL", - "BABELTRACE_SRC_CTF_FS_LOG_LEVEL", - "BABELTRACE_SRC_CTF_LTTNG_LIVE_LOG_LEVEL", - "BABELTRACE_SRC_TEXT_DMESG_LOG_LEVEL", NULL, }; @@ -97,7 +80,7 @@ static void set_signal_handler(void) { if (!SetConsoleCtrlHandler(signal_handler, TRUE)) { - BT_LOGE("Failed to set the ctrl+c handler."); + BT_LOGE("Failed to set the Ctrl+C handler."); } } @@ -172,12 +155,12 @@ void destroy_the_query_executor(void) } static -int query(const bt_component_class *comp_cls, const char *obj, - const bt_value *params, const bt_value **user_result, - const char **fail_reason) +int query(struct bt_config *cfg, const bt_component_class *comp_cls, + const char *obj, const bt_value *params, + const bt_value **user_result, const char **fail_reason) { const bt_value *result = NULL; - bt_query_executor_status status; + bt_query_executor_query_status query_status; *fail_reason = "unknown error"; int ret = 0; @@ -199,17 +182,18 @@ int query(const bt_component_class *comp_cls, const char *obj, } while (true) { - status = bt_query_executor_query(the_query_executor, - comp_cls, obj, params, &result); - switch (status) { - case BT_QUERY_EXECUTOR_STATUS_OK: + query_status = bt_query_executor_query( + the_query_executor, comp_cls, obj, params, + cfg->log_level, &result); + switch (query_status) { + case BT_QUERY_EXECUTOR_QUERY_STATUS_OK: goto ok; - case BT_QUERY_EXECUTOR_STATUS_AGAIN: + case BT_QUERY_EXECUTOR_QUERY_STATUS_AGAIN: { const uint64_t sleep_time_us = 100000; /* Wait 100 ms and retry */ - BT_LOGV("Got BT_QUERY_EXECUTOR_STATUS_AGAIN: sleeping: " + BT_LOGD("Got BT_QUERY_EXECUTOR_QUERY_STATUS_AGAIN: sleeping: " "time-us=%" PRIu64, sleep_time_us); if (usleep(sleep_time_us)) { @@ -226,25 +210,27 @@ int query(const bt_component_class *comp_cls, const char *obj, continue; } - case BT_QUERY_EXECUTOR_STATUS_CANCELED: + case BT_QUERY_EXECUTOR_QUERY_STATUS_CANCELED: *fail_reason = "canceled by user"; goto error; - case BT_QUERY_EXECUTOR_STATUS_ERROR: + case BT_QUERY_EXECUTOR_QUERY_STATUS_ERROR: goto error; - case BT_QUERY_EXECUTOR_STATUS_INVALID_OBJECT: + case BT_QUERY_EXECUTOR_QUERY_STATUS_INVALID_OBJECT: *fail_reason = "invalid or unknown query object"; goto error; - case BT_QUERY_EXECUTOR_STATUS_INVALID_PARAMS: + case BT_QUERY_EXECUTOR_QUERY_STATUS_INVALID_PARAMS: *fail_reason = "invalid query parameters"; goto error; - case BT_QUERY_EXECUTOR_STATUS_UNSUPPORTED: + case BT_QUERY_EXECUTOR_QUERY_STATUS_UNSUPPORTED: *fail_reason = "unsupported action"; goto error; - case BT_QUERY_EXECUTOR_STATUS_NOMEM: + case BT_QUERY_EXECUTOR_QUERY_STATUS_MEMORY_ERROR: *fail_reason = "not enough memory"; goto error; default: - BT_LOGF("Unknown query status: status=%d", status); + BT_LOGF("Unknown query status: status=%s", + bt_common_func_status_string( + query_status)); abort(); } } @@ -270,7 +256,7 @@ const bt_plugin *find_plugin(const char *name) const bt_plugin *plugin = NULL; BT_ASSERT(name); - BT_LOGD("Finding plugin: name=\"%s\"", name); + BT_LOGI("Finding plugin: name=\"%s\"", name); for (i = 0; i < loaded_plugins->len; i++) { plugin = g_ptr_array_index(loaded_plugins, i); @@ -282,12 +268,11 @@ const bt_plugin *find_plugin(const char *name) plugin = NULL; } - if (BT_LOG_ON_DEBUG) { - if (plugin) { - BT_LOGD("Found plugin: plugin-addr=%p", plugin); - } else { - BT_LOGD("Cannot find plugin."); - } + if (plugin) { + BT_LOGI("Found plugin: name=\"%s\", plugin-addr=%p", + name, plugin); + } else { + BT_LOGI("Cannot find plugin: name=\"%s\"", name); } bt_plugin_get_ref(plugin); @@ -305,7 +290,7 @@ const void *find_component_class_from_plugin(const char *plugin_name, const void *comp_class = NULL; const bt_plugin *plugin; - BT_LOGD("Finding component class: plugin-name=\"%s\", " + BT_LOGI("Finding component class: plugin-name=\"%s\", " "comp-cls-name=\"%s\"", plugin_name, comp_class_name); plugin = find_plugin(plugin_name); @@ -318,13 +303,13 @@ const void *find_component_class_from_plugin(const char *plugin_name, BT_PLUGIN_PUT_REF_AND_RESET(plugin); end: - if (BT_LOG_ON_DEBUG) { - if (comp_class) { - BT_LOGD("Found component class: comp-cls-addr=%p", - comp_class); - } else { - BT_LOGD("Cannot find source component class."); - } + if (comp_class) { + BT_LOGI("Found component class: plugin-name=\"%s\", " + "comp-cls-name=\"%s\"", plugin_name, comp_class_name); + } else { + BT_LOGI("Cannot find source component class: " + "plugin-name=\"%s\", comp-cls-name=\"%s\"", + plugin_name, comp_class_name); } return comp_class; @@ -735,9 +720,9 @@ void print_cfg(struct bt_config *cfg) return; } - BT_LOGI_STR("Configuration:"); - fprintf(stderr, " Debug mode: %s\n", cfg->debug ? "yes" : "no"); - fprintf(stderr, " Verbose mode: %s\n", cfg->verbose ? "yes" : "no"); + BT_LOGI_STR("CLI configuration:"); + BT_LOGI(" Debug mode: %s\n", cfg->debug ? "yes" : "no"); + BT_LOGI(" Verbose mode: %s\n", cfg->verbose ? "yes" : "no"); switch (cfg->command) { case BT_CONFIG_COMMAND_RUN: @@ -810,12 +795,13 @@ int load_dynamic_plugins(const bt_value *plugin_paths) goto end; } - BT_LOGI("Loading dynamic plugins."); + BT_LOGI_STR("Loading dynamic plugins."); for (i = 0; i < nr_paths; i++) { const bt_value *plugin_path_value = NULL; const char *plugin_path; - const bt_plugin_set *plugin_set; + const bt_plugin_set *plugin_set = NULL; + bt_plugin_find_all_from_dir_status status; plugin_path_value = bt_value_array_borrow_element_by_index_const( @@ -828,18 +814,26 @@ int load_dynamic_plugins(const bt_value *plugin_paths) * directory. */ if (!g_file_test(plugin_path, G_FILE_TEST_IS_DIR)) { - BT_LOGV("Skipping nonexistent directory path: " + BT_LOGI("Skipping nonexistent directory path: " "path=\"%s\"", plugin_path); continue; } - plugin_set = bt_plugin_find_all_from_dir(plugin_path, false); - if (!plugin_set) { - BT_LOGD("Unable to load dynamic plugins: path=\"%s\"", + status = bt_plugin_find_all_from_dir(plugin_path, BT_FALSE, + BT_FALSE, &plugin_set); + if (status < 0) { + BT_LOGE("Unable to load dynamic plugins from directory: " + "path=\"%s\"", plugin_path); + continue; + } else if (status == + BT_PLUGIN_FIND_ALL_FROM_DIR_STATUS_NOT_FOUND) { + BT_LOGI("No plugins found in directory: path=\"%s\"", plugin_path); continue; } + BT_ASSERT(status == BT_PLUGIN_FIND_ALL_FROM_DIR_STATUS_OK); + BT_ASSERT(plugin_set); add_to_loaded_plugins(plugin_set); bt_plugin_set_put_ref(plugin_set); } @@ -852,17 +846,25 @@ int load_static_plugins(void) { int ret = 0; const bt_plugin_set *plugin_set; + bt_plugin_find_all_from_static_status status; BT_LOGI("Loading static plugins."); - plugin_set = bt_plugin_find_all_from_static(); - if (!plugin_set) { + status = bt_plugin_find_all_from_static(BT_FALSE, &plugin_set); + if (status < 0) { BT_LOGE("Unable to load static plugins."); ret = -1; goto end; + } else if (status == + BT_PLUGIN_FIND_ALL_FROM_STATIC_STATUS_NOT_FOUND) { + BT_LOGI("No static plugins found."); + goto end; } + BT_ASSERT(status == BT_PLUGIN_FIND_ALL_FROM_STATIC_STATUS_OK); + BT_ASSERT(plugin_set); add_to_loaded_plugins(plugin_set); bt_plugin_set_put_ref(plugin_set); + end: return ret; } @@ -970,7 +972,7 @@ int cmd_query(struct bt_config *cfg) goto end; } - ret = query(comp_cls, cfg->cmd_data.query.object->str, + ret = query(cfg, comp_cls, cfg->cmd_data.query.object->str, cfg->cmd_data.query.cfg_component->params, &results, &fail_reason); if (ret) { @@ -1266,7 +1268,7 @@ int cmd_print_lttng_live_sessions(struct bt_config *cfg) goto error; } - ret = query(comp_cls, "sessions", params, + ret = query(cfg, comp_cls, "sessions", params, &results, &fail_reason); if (ret) { goto failed; @@ -1420,7 +1422,7 @@ int cmd_print_ctf_metadata(struct bt_config *cfg) goto end; } - ret = query(comp_cls, "metadata-info", + ret = query(cfg, comp_cls, "metadata-info", params, &results, &fail_reason); if (ret) { goto failed; @@ -1625,7 +1627,8 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component( uint64_t i; input_port_count_func_t port_count_fn; borrow_input_port_by_index_func_t port_by_index_fn; - bt_graph_status status = BT_GRAPH_STATUS_ERROR; + bt_graph_connect_ports_status connect_ports_status = + BT_GRAPH_CONNECT_PORTS_STATUS_OK; bool insert_trimmer = false; bt_value *trimmer_params = NULL; char *intersection_begin = NULL; @@ -1651,7 +1654,7 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component( range = (struct trace_range *) g_hash_table_lookup( ctx->intersections, &port_id); if (range) { - bt_value_status status; + bt_value_map_insert_entry_status insert_status; intersection_begin = s_from_ns( range->intersection_range_begin_ns); @@ -1668,15 +1671,15 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component( goto error; } - status = bt_value_map_insert_string_entry( + insert_status = bt_value_map_insert_string_entry( trimmer_params, "begin", intersection_begin); - if (status != BT_VALUE_STATUS_OK) { + if (insert_status < 0) { goto error; } - status = bt_value_map_insert_string_entry( + insert_status = bt_value_map_insert_string_entry( trimmer_params, "end", intersection_end); - if (status != BT_VALUE_STATUS_OK) { + if (insert_status < 0) { goto error; } } @@ -1735,7 +1738,7 @@ 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_LOGD("Skipping downstream port: already connected: " + BT_LOGI("Skipping downstream port: already connected: " "port-addr=%p, port-name=\"%s\"", downstream_port, bt_port_get_name(downstream_port)); @@ -1770,7 +1773,7 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component( * source and the trimmer. */ char *trimmer_name = NULL; - bt_graph_status graph_status; + bt_graph_add_component_status add_comp_status; ret = asprintf(&trimmer_name, "stream-intersection-trimmer-%s", @@ -1781,11 +1784,13 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component( ret = 0; ctx->connect_ports = false; - graph_status = bt_graph_add_filter_component( + add_comp_status = bt_graph_add_filter_component( ctx->graph, trimmer_class, trimmer_name, - trimmer_params, &trimmer); + trimmer_params, ctx->cfg->log_level, + &trimmer); free(trimmer_name); - if (graph_status != BT_GRAPH_STATUS_OK) { + if (add_comp_status != + BT_GRAPH_ADD_COMPONENT_STATUS_OK) { goto error; } BT_ASSERT(trimmer); @@ -1816,33 +1821,14 @@ int cmd_run_ctx_connect_upstream_port_to_downstream_component( } /* We have a winner! */ - status = bt_graph_connect_ports(ctx->graph, + connect_ports_status = bt_graph_connect_ports(ctx->graph, out_upstream_port, in_downstream_port, NULL); downstream_port = NULL; - switch (status) { - case BT_GRAPH_STATUS_OK: + switch (connect_ports_status) { + case BT_GRAPH_CONNECT_PORTS_STATUS_OK: break; - case BT_GRAPH_STATUS_CANCELED: + case BT_GRAPH_CONNECT_PORTS_STATUS_CANCELED: BT_LOGI_STR("Graph was canceled by user."); - status = BT_GRAPH_STATUS_OK; - break; - case BT_GRAPH_STATUS_COMPONENT_REFUSES_PORT_CONNECTION: - BT_LOGE("A component refused a connection to one of its ports: " - "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\", " - "conn-arg=\"%s\"", - upstream_comp, bt_component_get_name(upstream_comp), - upstream_port, bt_port_get_name(upstream_port), - downstream_comp, cfg_conn->downstream_comp_name->str, - downstream_port, downstream_port_name, - cfg_conn->arg->str); - fprintf(stderr, - "A component refused a connection to one of its ports (`%s` to `%s`): %s\n", - bt_port_get_name(upstream_port), - downstream_port_name, - cfg_conn->arg->str); break; default: BT_LOGE("Cannot create connection: graph refuses to connect ports: " @@ -2002,13 +1988,14 @@ end: } static -bt_graph_listener_status +bt_graph_listener_func_status graph_output_port_added_listener(struct cmd_run_ctx *ctx, const bt_port_output *out_port) { const bt_component *comp; const bt_port *port = bt_port_output_as_port_const(out_port); - bt_graph_listener_status ret = BT_GRAPH_LISTENER_STATUS_OK; + bt_graph_listener_func_status ret = + BT_GRAPH_LISTENER_FUNC_STATUS_OK; comp = bt_port_borrow_component_const(port); BT_LOGI("Port added to a graph's component: comp-addr=%p, " @@ -2033,7 +2020,7 @@ graph_output_port_added_listener(struct cmd_run_ctx *ctx, 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"); - ret = BT_GRAPH_LISTENER_STATUS_ERROR; + ret = BT_GRAPH_LISTENER_FUNC_STATUS_ERROR; goto end; } @@ -2042,7 +2029,7 @@ end: } static -bt_graph_listener_status graph_source_output_port_added_listener( +bt_graph_listener_func_status graph_source_output_port_added_listener( const bt_component_source *component, const bt_port_output *port, void *data) { @@ -2050,7 +2037,7 @@ bt_graph_listener_status graph_source_output_port_added_listener( } static -bt_graph_listener_status graph_filter_output_port_added_listener( +bt_graph_listener_func_status graph_filter_output_port_added_listener( const bt_component_filter *component, const bt_port_output *port, void *data) { @@ -2093,7 +2080,7 @@ static int cmd_run_ctx_init(struct cmd_run_ctx *ctx, struct bt_config *cfg) { int ret = 0; - bt_graph_status status; + bt_graph_add_component_status add_comp_status; ctx->cfg = cfg; ctx->connect_ports = false; @@ -2130,18 +2117,18 @@ int cmd_run_ctx_init(struct cmd_run_ctx *ctx, struct bt_config *cfg) } the_graph = ctx->graph; - status = bt_graph_add_source_component_output_port_added_listener( + add_comp_status = bt_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) { + if (add_comp_status != BT_GRAPH_ADD_COMPONENT_STATUS_OK) { BT_LOGE_STR("Cannot add \"port added\" listener to graph."); goto error; } - status = bt_graph_add_filter_component_output_port_added_listener( + add_comp_status = bt_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) { + if (add_comp_status != BT_GRAPH_ADD_COMPONENT_STATUS_OK) { BT_LOGE_STR("Cannot add \"port added\" listener to graph."); goto error; } @@ -2178,7 +2165,7 @@ int set_stream_intersections(struct cmd_run_ctx *ctx, const bt_component_class *comp_cls = bt_component_class_source_as_component_class_const(src_comp_cls); - ret = query(comp_cls, "trace-info", + ret = query(ctx->cfg, comp_cls, "trace-info", cfg_comp->params, &query_result, &fail_reason); if (ret) { @@ -2297,14 +2284,14 @@ int set_stream_intersections(struct cmd_run_ctx *ctx, stream_infos, stream_idx); if (!stream_info || !bt_value_is_map(stream_info)) { ret = -1; - BT_LOGD_STR("Cannot retrieve stream informations from trace in query result."); + BT_LOGE_STR("Cannot retrieve stream informations from trace in query result."); goto error; } port_name = bt_value_map_borrow_entry_value_const(stream_info, "port-name"); if (!port_name || !bt_value_is_string(port_name)) { ret = -1; - BT_LOGD_STR("Cannot retrieve port name in query result."); + BT_LOGE_STR("Cannot retrieve port name in query result."); goto error; } @@ -2391,23 +2378,25 @@ int cmd_run_ctx_create_components_from_config_components( goto error; } + BT_ASSERT(cfg_comp->log_level >= BT_LOG_TRACE); + switch (cfg_comp->type) { case BT_COMPONENT_CLASS_TYPE_SOURCE: ret = bt_graph_add_source_component(ctx->graph, comp_cls, cfg_comp->instance_name->str, - cfg_comp->params, + cfg_comp->params, cfg_comp->log_level, (void *) &comp); break; case BT_COMPONENT_CLASS_TYPE_FILTER: ret = bt_graph_add_filter_component(ctx->graph, comp_cls, cfg_comp->instance_name->str, - cfg_comp->params, + cfg_comp->params, cfg_comp->log_level, (void *) &comp); break; case BT_COMPONENT_CLASS_TYPE_SINK: ret = bt_graph_add_sink_component(ctx->graph, comp_cls, cfg_comp->instance_name->str, - cfg_comp->params, + cfg_comp->params, cfg_comp->log_level, (void *) &comp); break; default: @@ -2579,29 +2568,6 @@ end: return ret; } -static inline -const char *bt_graph_status_str(bt_graph_status status) -{ - switch (status) { - case BT_GRAPH_STATUS_OK: - return "BT_GRAPH_STATUS_OK"; - 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_NOMEM: - return "BT_GRAPH_STATUS_NOMEM"; - default: - return "(unknown)"; - } -} - static int cmd_run(struct bt_config *cfg) { @@ -2652,7 +2618,7 @@ int cmd_run(struct bt_config *cfg) /* Run the graph */ while (true) { - bt_graph_status graph_status = bt_graph_run(ctx.graph); + bt_graph_run_status run_status = bt_graph_run(ctx.graph); /* * Reset console in case something messed with console @@ -2661,23 +2627,23 @@ 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_graph_status_str(graph_status)); + BT_LOGT("bt_graph_run() returned: status=%s", + bt_common_func_status_string(run_status)); - switch (graph_status) { - case BT_GRAPH_STATUS_OK: + switch (run_status) { + case BT_GRAPH_RUN_STATUS_OK: break; - case BT_GRAPH_STATUS_CANCELED: + case BT_GRAPH_RUN_STATUS_CANCELED: BT_LOGI_STR("Graph was canceled by user."); goto error; - case BT_GRAPH_STATUS_AGAIN: + case BT_GRAPH_RUN_STATUS_AGAIN: if (bt_graph_is_canceled(ctx.graph)) { BT_LOGI_STR("Graph was canceled by user."); goto error; } if (cfg->cmd_data.run.retry_duration_us > 0) { - BT_LOGV("Got BT_GRAPH_STATUS_AGAIN: sleeping: " + BT_LOGT("Got BT_GRAPH_RUN_STATUS_AGAIN: sleeping: " "time-us=%" PRIu64, cfg->cmd_data.run.retry_duration_us); @@ -2689,7 +2655,7 @@ int cmd_run(struct bt_config *cfg) } } break; - case BT_GRAPH_STATUS_END: + case BT_GRAPH_RUN_STATUS_END: goto end; default: BT_LOGE_STR("Graph failed to complete successfully"); @@ -2754,10 +2720,10 @@ void set_auto_log_levels(struct bt_config *cfg) */ if (getenv("BABELTRACE_DEBUG") && strcmp(getenv("BABELTRACE_DEBUG"), "1") == 0) { - cfg->log_level = 'V'; + cfg->log_level = BT_LOG_TRACE; } else if (getenv("BABELTRACE_VERBOSE") && strcmp(getenv("BABELTRACE_VERBOSE"), "1") == 0) { - cfg->log_level = 'I'; + cfg->log_level = BT_LOG_INFO; } /* @@ -2766,45 +2732,21 @@ void set_auto_log_levels(struct bt_config *cfg) * --verbose. So: * * --verbose: INFO log level - * --debug: VERBOSE log level (includes DEBUG, which is - * is less verbose than VERBOSE in the internal + * --debug: TRACE log level (includes DEBUG, which is + * is less verbose than TRACE in the internal * logging framework) */ - if (!getenv("BABELTRACE_LOGGING_GLOBAL_LEVEL")) { + if (!getenv("LIBBABELTRACE2_INIT_LOG_LEVEL")) { if (cfg->verbose) { - bt_logging_set_global_level(BT_LOGGING_LEVEL_INFO); + bt_logging_set_global_level(BT_LOG_INFO); } else if (cfg->debug) { - bt_logging_set_global_level(BT_LOGGING_LEVEL_VERBOSE); + bt_logging_set_global_level(BT_LOG_TRACE); } else { /* * Set library's default log level if not * explicitly specified. */ - switch (cfg->log_level) { - case 'N': - bt_logging_set_global_level(BT_LOGGING_LEVEL_NONE); - break; - case 'V': - bt_logging_set_global_level(BT_LOGGING_LEVEL_VERBOSE); - break; - case 'D': - bt_logging_set_global_level(BT_LOGGING_LEVEL_DEBUG); - break; - case 'I': - bt_logging_set_global_level(BT_LOGGING_LEVEL_INFO); - break; - case 'W': - bt_logging_set_global_level(BT_LOGGING_LEVEL_WARN); - break; - case 'E': - bt_logging_set_global_level(BT_LOGGING_LEVEL_ERROR); - break; - case 'F': - bt_logging_set_global_level(BT_LOGGING_LEVEL_FATAL); - break; - default: - abort(); - } + bt_logging_set_global_level(cfg->log_level); } } @@ -2812,37 +2754,13 @@ void set_auto_log_levels(struct bt_config *cfg) if (cfg->verbose) { bt_cli_log_level = BT_LOG_INFO; } else if (cfg->debug) { - bt_cli_log_level = BT_LOG_VERBOSE; + bt_cli_log_level = BT_LOG_TRACE; } else { /* * Set CLI's default log level if not explicitly * specified. */ - switch (cfg->log_level) { - case 'N': - bt_cli_log_level = BT_LOG_NONE; - break; - case 'V': - bt_cli_log_level = BT_LOG_VERBOSE; - break; - case 'D': - bt_cli_log_level = BT_LOG_DEBUG; - break; - case 'I': - bt_cli_log_level = BT_LOG_INFO; - break; - case 'W': - bt_cli_log_level = BT_LOG_WARN; - break; - case 'E': - bt_cli_log_level = BT_LOG_ERROR; - break; - case 'F': - bt_cli_log_level = BT_LOG_FATAL; - break; - default: - abort(); - } + bt_cli_log_level = cfg->log_level; } } @@ -2851,9 +2769,9 @@ void set_auto_log_levels(struct bt_config *cfg) while (*env_var_name) { if (!getenv(*env_var_name)) { if (cfg->verbose) { - g_setenv(*env_var_name, "I", 1); + g_setenv(*env_var_name, "INFO", 1); } else if (cfg->debug) { - g_setenv(*env_var_name, "V", 1); + g_setenv(*env_var_name, "TRACE", 1); } else { char val[2] = { 0 }; @@ -2861,7 +2779,8 @@ void set_auto_log_levels(struct bt_config *cfg) * Set module's default log level if not * explicitly specified. */ - val[0] = cfg->log_level; + val[0] = bt_log_get_letter_from_level( + cfg->log_level); g_setenv(*env_var_name, val, 1); } }