error: Simplify error sink setup
qemu_error_sink can either point to a monitor or a file. In practice, it always points to the current monitor if we have one, else to stderr. Simply route errors to the current monitor or else to stderr, and remove qemu_error_sink along with the functions to control it. Actually, the old code switches the sink slightly later, in handle_user_command() and handle_qmp_command(), than it gets switched now, implicitly, by setting the current monitor in monitor_read() and monitor_control_read(). Likewise, it switches back slightly earlier (same places). Doesn't make a difference, because there are no calls of qemu_error() in between.stable-0.13
parent
b4a51f7f5d
commit
6e4f984cb9
|
@ -3971,8 +3971,6 @@ static void handle_user_command(Monitor *mon, const char *cmdline)
|
||||||
if (!cmd)
|
if (!cmd)
|
||||||
goto out;
|
goto out;
|
||||||
|
|
||||||
qemu_errors_to_mon(mon);
|
|
||||||
|
|
||||||
if (monitor_handler_is_async(cmd)) {
|
if (monitor_handler_is_async(cmd)) {
|
||||||
user_async_cmd_handler(mon, cmd, qdict);
|
user_async_cmd_handler(mon, cmd, qdict);
|
||||||
} else if (monitor_handler_ported(cmd)) {
|
} else if (monitor_handler_ported(cmd)) {
|
||||||
|
@ -3984,8 +3982,6 @@ static void handle_user_command(Monitor *mon, const char *cmdline)
|
||||||
if (monitor_has_error(mon))
|
if (monitor_has_error(mon))
|
||||||
monitor_print_error(mon);
|
monitor_print_error(mon);
|
||||||
|
|
||||||
qemu_errors_to_previous();
|
|
||||||
|
|
||||||
out:
|
out:
|
||||||
QDECREF(qdict);
|
QDECREF(qdict);
|
||||||
}
|
}
|
||||||
|
@ -4387,7 +4383,6 @@ static void handle_qmp_command(JSONMessageParser *parser, QList *tokens)
|
||||||
const char *cmd_name, *info_item;
|
const char *cmd_name, *info_item;
|
||||||
|
|
||||||
args = NULL;
|
args = NULL;
|
||||||
qemu_errors_to_mon(mon);
|
|
||||||
|
|
||||||
obj = json_parser_parse(tokens, NULL);
|
obj = json_parser_parse(tokens, NULL);
|
||||||
if (!obj) {
|
if (!obj) {
|
||||||
|
@ -4468,7 +4463,6 @@ err_out:
|
||||||
monitor_protocol_emitter(mon, NULL);
|
monitor_protocol_emitter(mon, NULL);
|
||||||
out:
|
out:
|
||||||
QDECREF(args);
|
QDECREF(args);
|
||||||
qemu_errors_to_previous();
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
76
qemu-error.c
76
qemu-error.c
|
@ -2,70 +2,17 @@
|
||||||
#include "monitor.h"
|
#include "monitor.h"
|
||||||
#include "sysemu.h"
|
#include "sysemu.h"
|
||||||
|
|
||||||
typedef struct QemuErrorSink QemuErrorSink;
|
|
||||||
struct QemuErrorSink {
|
|
||||||
enum {
|
|
||||||
ERR_SINK_FILE,
|
|
||||||
ERR_SINK_MONITOR,
|
|
||||||
} dest;
|
|
||||||
union {
|
|
||||||
FILE *fp;
|
|
||||||
Monitor *mon;
|
|
||||||
};
|
|
||||||
QemuErrorSink *previous;
|
|
||||||
};
|
|
||||||
|
|
||||||
static QemuErrorSink *qemu_error_sink;
|
|
||||||
|
|
||||||
void qemu_errors_to_file(FILE *fp)
|
|
||||||
{
|
|
||||||
QemuErrorSink *sink;
|
|
||||||
|
|
||||||
sink = qemu_mallocz(sizeof(*sink));
|
|
||||||
sink->dest = ERR_SINK_FILE;
|
|
||||||
sink->fp = fp;
|
|
||||||
sink->previous = qemu_error_sink;
|
|
||||||
qemu_error_sink = sink;
|
|
||||||
}
|
|
||||||
|
|
||||||
void qemu_errors_to_mon(Monitor *mon)
|
|
||||||
{
|
|
||||||
QemuErrorSink *sink;
|
|
||||||
|
|
||||||
sink = qemu_mallocz(sizeof(*sink));
|
|
||||||
sink->dest = ERR_SINK_MONITOR;
|
|
||||||
sink->mon = mon;
|
|
||||||
sink->previous = qemu_error_sink;
|
|
||||||
qemu_error_sink = sink;
|
|
||||||
}
|
|
||||||
|
|
||||||
void qemu_errors_to_previous(void)
|
|
||||||
{
|
|
||||||
QemuErrorSink *sink;
|
|
||||||
|
|
||||||
assert(qemu_error_sink != NULL);
|
|
||||||
sink = qemu_error_sink;
|
|
||||||
qemu_error_sink = sink->previous;
|
|
||||||
qemu_free(sink);
|
|
||||||
}
|
|
||||||
|
|
||||||
void qemu_error(const char *fmt, ...)
|
void qemu_error(const char *fmt, ...)
|
||||||
{
|
{
|
||||||
va_list args;
|
va_list args;
|
||||||
|
|
||||||
assert(qemu_error_sink != NULL);
|
va_start(args, fmt);
|
||||||
switch (qemu_error_sink->dest) {
|
if (cur_mon) {
|
||||||
case ERR_SINK_FILE:
|
monitor_vprintf(cur_mon, fmt, args);
|
||||||
va_start(args, fmt);
|
} else {
|
||||||
vfprintf(qemu_error_sink->fp, fmt, args);
|
vfprintf(stderr, fmt, args);
|
||||||
va_end(args);
|
|
||||||
break;
|
|
||||||
case ERR_SINK_MONITOR:
|
|
||||||
va_start(args, fmt);
|
|
||||||
monitor_vprintf(qemu_error_sink->mon, fmt, args);
|
|
||||||
va_end(args);
|
|
||||||
break;
|
|
||||||
}
|
}
|
||||||
|
va_end(args);
|
||||||
}
|
}
|
||||||
|
|
||||||
void qemu_error_internal(const char *file, int linenr, const char *func,
|
void qemu_error_internal(const char *file, int linenr, const char *func,
|
||||||
|
@ -74,19 +21,14 @@ void qemu_error_internal(const char *file, int linenr, const char *func,
|
||||||
va_list va;
|
va_list va;
|
||||||
QError *qerror;
|
QError *qerror;
|
||||||
|
|
||||||
assert(qemu_error_sink != NULL);
|
|
||||||
|
|
||||||
va_start(va, fmt);
|
va_start(va, fmt);
|
||||||
qerror = qerror_from_info(file, linenr, func, fmt, &va);
|
qerror = qerror_from_info(file, linenr, func, fmt, &va);
|
||||||
va_end(va);
|
va_end(va);
|
||||||
|
|
||||||
switch (qemu_error_sink->dest) {
|
if (cur_mon) {
|
||||||
case ERR_SINK_FILE:
|
monitor_set_error(cur_mon, qerror);
|
||||||
|
} else {
|
||||||
qerror_print(qerror);
|
qerror_print(qerror);
|
||||||
QDECREF(qerror);
|
QDECREF(qerror);
|
||||||
break;
|
|
||||||
case ERR_SINK_MONITOR:
|
|
||||||
monitor_set_error(qemu_error_sink->mon, qerror);
|
|
||||||
break;
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
3
sysemu.h
3
sysemu.h
|
@ -73,9 +73,6 @@ int qemu_savevm_state_complete(Monitor *mon, QEMUFile *f);
|
||||||
void qemu_savevm_state_cancel(Monitor *mon, QEMUFile *f);
|
void qemu_savevm_state_cancel(Monitor *mon, QEMUFile *f);
|
||||||
int qemu_loadvm_state(QEMUFile *f);
|
int qemu_loadvm_state(QEMUFile *f);
|
||||||
|
|
||||||
void qemu_errors_to_file(FILE *fp);
|
|
||||||
void qemu_errors_to_mon(Monitor *mon);
|
|
||||||
void qemu_errors_to_previous(void);
|
|
||||||
void qemu_error(const char *fmt, ...) __attribute__ ((format(printf, 1, 2)));
|
void qemu_error(const char *fmt, ...) __attribute__ ((format(printf, 1, 2)));
|
||||||
void qemu_error_internal(const char *file, int linenr, const char *func,
|
void qemu_error_internal(const char *file, int linenr, const char *func,
|
||||||
const char *fmt, ...)
|
const char *fmt, ...)
|
||||||
|
|
Loading…
Reference in New Issue