X-Git-Url: http://git.efficios.com/?a=blobdiff_plain;f=src%2Flib%2Flib-logging.c;h=dcfee70ff92dcf0daa3af70076e2524281afb73a;hb=188edac1113dbbb29030681dbde2de0ada742499;hp=93a2c989f5aa0adf7b9d911741e127bedc214b92;hpb=51375aa95adf2f601f74a5a1f1665ca685584388;p=babeltrace.git diff --git a/src/lib/lib-logging.c b/src/lib/lib-logging.c index 93a2c989..dcfee70f 100644 --- a/src/lib/lib-logging.c +++ b/src/lib/lib-logging.c @@ -20,11 +20,8 @@ * SOFTWARE. */ -/* - * This is just to satisfy the preprocessor check in "lib/logging.h": - * this file does not log anything. - */ -#define BT_LOG_TAG "" +#define BT_LOG_TAG "LIB/LIB-LOGGING" +#include "lib/logging.h" #include #include @@ -38,6 +35,7 @@ #include #include #include +#include #include "logging.h" #include "assert-pre.h" @@ -58,7 +56,6 @@ #include "graph/message/message.h" #include "graph/message/message-iterator-inactivity.h" #include "graph/message/packet.h" -#include "graph/message/stream-activity.h" #include "graph/message/stream.h" #include "graph/port.h" #include "plugin/plugin.h" @@ -76,6 +73,8 @@ #include "trace-ir/trace-class.h" #include "trace-ir/trace.h" #include "trace-ir/utils.h" +#include "error.h" +#include "assert-pre.h" #define LIB_LOGGING_BUF_SIZE (4096 * 4) @@ -451,21 +450,12 @@ static inline void format_field_path(char **buf_ch, bool extended, static inline void format_trace_class(char **buf_ch, bool extended, const char *prefix, const struct bt_trace_class *trace_class) { - if (trace_class->name.value) { - BUF_APPEND(", %sname=\"%s\"", - PRFIELD(trace_class->name.value)); - } - if (!extended) { return; } BUF_APPEND(", %sis-frozen=%d", PRFIELD(trace_class->frozen)); - if (trace_class->uuid.value) { - BUF_APPEND_UUID(trace_class->uuid.value); - } - if (trace_class->stream_classes) { BUF_APPEND(", %sstream-class-count=%u", PRFIELD(trace_class->stream_classes->len)); @@ -488,6 +478,10 @@ static inline void format_trace(char **buf_ch, bool extended, return; } + if (trace->uuid.value) { + BUF_APPEND_UUID(trace->uuid.value); + } + BUF_APPEND(", %sis-frozen=%d", PRFIELD(trace->frozen)); if (trace->streams) { @@ -930,28 +924,14 @@ static inline void format_message(char **buf_ch, bool extended, msg_stream->stream); } - break; - } - case BT_MESSAGE_TYPE_STREAM_ACTIVITY_BEGINNING: - case BT_MESSAGE_TYPE_STREAM_ACTIVITY_END: - { - const struct bt_message_stream_activity *msg_stream_activity = - (const void *) msg; - - if (msg_stream_activity->stream) { - SET_TMP_PREFIX("stream-"); - format_stream(buf_ch, true, tmp_prefix, - msg_stream_activity->stream); - } - BUF_APPEND(", %sdefault-cs-state=%s", - PRFIELD(bt_message_stream_activity_clock_snapshot_state_string( - msg_stream_activity->default_cs_state))); + PRFIELD(bt_message_stream_clock_snapshot_state_string( + msg_stream->default_cs_state))); - if (msg_stream_activity->default_cs) { + if (msg_stream->default_cs_state == BT_MESSAGE_STREAM_CLOCK_SNAPSHOT_STATE_KNOWN) { SET_TMP_PREFIX("default-cs-"); format_clock_snapshot(buf_ch, true, tmp_prefix, - msg_stream_activity->default_cs); + msg_stream->default_cs); } break; @@ -1281,6 +1261,66 @@ static inline void format_plugin(char **buf_ch, bool extended, } } +static inline void format_error_cause(char **buf_ch, bool extended, + const char *prefix, const struct bt_error_cause *cause) +{ + const struct bt_error_cause_component_class_id *comp_class_id = NULL; + + BUF_APPEND(", %sactor-type=%s, %smodule-name=\"%s\"", + PRFIELD(bt_error_cause_actor_type_string(cause->actor_type)), + PRFIELD_GSTRING(cause->module_name)); + + if (!extended) { + return; + } + + BUF_APPEND(", %spartial-msg=\"%.32s\"", + PRFIELD_GSTRING(cause->message)); + + switch (cause->actor_type) { + case BT_ERROR_CAUSE_ACTOR_TYPE_COMPONENT: + { + const struct bt_error_cause_component_actor *spec_cause = + (const void *) cause; + + BUF_APPEND(", %scomp-name=\"%s\"", + PRFIELD_GSTRING(spec_cause->comp_name)); + comp_class_id = &spec_cause->comp_class_id; + break; + } + case BT_ERROR_CAUSE_ACTOR_TYPE_COMPONENT_CLASS: + { + const struct bt_error_cause_component_class_actor *spec_cause = + (const void *) cause; + + comp_class_id = &spec_cause->comp_class_id; + break; + } + case BT_ERROR_CAUSE_ACTOR_TYPE_MESSAGE_ITERATOR: + { + const struct bt_error_cause_message_iterator_actor *spec_cause = + (const void *) cause; + + BUF_APPEND(", %scomp-name=\"%s\", %scomp-out-port-name=\"%s\"", + PRFIELD_GSTRING(spec_cause->comp_name), + PRFIELD_GSTRING(spec_cause->output_port_name)); + comp_class_id = &spec_cause->comp_class_id; + break; + } + default: + break; + } + + if (comp_class_id) { + BUF_APPEND(", %scomp-cls-type=%s, %scomp-cls-name=\"%s\", " + "%splugin-name=\"%s\"", + PRFIELD(bt_component_class_type_string( + comp_class_id->type)), + PRFIELD_GSTRING(comp_class_id->name), + PRFIELD_GSTRING(comp_class_id->plugin_name)); + } +} + static inline void handle_conversion_specifier_bt(void *priv_data, char **buf_ch, size_t avail_size, const char **out_fmt_ch, va_list *args) @@ -1402,6 +1442,9 @@ static inline void handle_conversion_specifier_bt(void *priv_data, case 'O': format_object(buf_ch, extended, prefix, obj); break; + case 'r': + format_error_cause(buf_ch, extended, prefix, obj); + break; default: abort(); } @@ -1423,3 +1466,41 @@ void bt_lib_log(const char *func, const char *file, unsigned line, va_end(args); _bt_log_write_d(func, file, line, lvl, tag, "%s", lib_logging_buf); } + +void bt_lib_maybe_log_and_append_cause(const char *func, const char *file, + unsigned line, int lvl, const char *tag, + const char *fmt, ...) +{ + va_list args; + bt_current_thread_error_append_cause_status status; + + BT_ASSERT(fmt); + va_start(args, fmt); + bt_common_custom_vsnprintf(lib_logging_buf, LIB_LOGGING_BUF_SIZE, '!', + handle_conversion_specifier_bt, NULL, fmt, &args); + va_end(args); + + /* Log conditionally, but always append the error cause */ + if (BT_LOG_ON(lvl)) { + _bt_log_write_d(func, file, line, lvl, tag, "%s", + lib_logging_buf); + } + + status = bt_current_thread_error_append_cause_from_unknown( + "Babeltrace library", file, line, "%s", lib_logging_buf); + if (status) { + /* + * Worst case: this error cause is not appended to the + * current thread's error. + * + * We can accept this as it's an almost impossible + * scenario and returning an error here would mean you + * need to check the return value of each + * BT_LIB_LOG*_APPEND_CAUSE() macro and that would be + * cumbersome. + */ + BT_LOGE("Cannot append error cause to current thread's " + "error object: status=%s", + bt_common_func_status_string(status)); + } +}