From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MbsGr-0004He-SV for qemu-devel@nongnu.org; Fri, 14 Aug 2009 04:36:21 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MbsGm-0004H1-SX for qemu-devel@nongnu.org; Fri, 14 Aug 2009 04:36:21 -0400 Received: from [199.232.76.173] (port=49685 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MbsGm-0004Gy-LI for qemu-devel@nongnu.org; Fri, 14 Aug 2009 04:36:16 -0400 Received: from mx2.redhat.com ([66.187.237.31]:41550) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1MbsGm-0005J1-3N for qemu-devel@nongnu.org; Fri, 14 Aug 2009 04:36:16 -0400 Received: from int-mx2.corp.redhat.com (int-mx2.corp.redhat.com [172.16.27.26]) by mx2.redhat.com (8.13.8/8.13.8) with ESMTP id n7E8aFdx026521 for ; Fri, 14 Aug 2009 04:36:15 -0400 From: Gerd Hoffmann Date: Fri, 14 Aug 2009 10:36:06 +0200 Message-Id: <1250238968-2114-2-git-send-email-kraxel@redhat.com> In-Reply-To: <1250238968-2114-1-git-send-email-kraxel@redhat.com> References: <1250238968-2114-1-git-send-email-kraxel@redhat.com> Subject: [Qemu-devel] [PATCH 2/4] add qemu_error() + friends List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Gerd Hoffmann This patch adds some functions for error reporting to address the problem that error messages should be routed to different destinations depending on the context of the caller, i.e. monitor command errors should go to the monitor, command line errors to stderr. qemu_error() is a printf-like function to report errors. qemu_errors_to_file() and qemu_errors_to_mon() switch the destination for the error message to the specified file or monitor. When setting a new destination the old one will be kept. One can switch back using qemu_errors_to_previous(). i.e. it works like a stack. main() calls qemu_errors_to_file(stderr), so errors go to stderr by default. monitor callbacks are wrapped into qemu_errors_to_mon() + qemu_errors_to_previous(), so any errors triggered by monitor commands will go to the monitor. Each thread has its own error message destination. qemu-kvm probably should add a qemu_errors_to_file(stderr) call to the i/o-thread initialization code. Signed-off-by: Gerd Hoffmann --- monitor.c | 71 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++- sysemu.h | 5 ++++ vl.c | 1 + 3 files changed, 76 insertions(+), 1 deletions(-) diff --git a/monitor.c b/monitor.c index 362322b..cb87c88 100644 --- a/monitor.c +++ b/monitor.c @@ -2839,6 +2839,7 @@ static void monitor_handle_command(Monitor *mon, const char *cmdline) goto fail; } + qemu_errors_to_mon(mon); switch(nb_args) { case 0: handler_0 = cmd->handler; @@ -2890,8 +2891,10 @@ static void monitor_handle_command(Monitor *mon, const char *cmdline) break; default: monitor_printf(mon, "unsupported number of arguments: %d\n", nb_args); - goto fail; + break; } + qemu_errors_to_previous(); + fail: for(i = 0; i < MAX_ARGS; i++) qemu_free(str_allocated[i]); @@ -3256,3 +3259,69 @@ void monitor_read_bdrv_key_start(Monitor *mon, BlockDriverState *bs, if (err && completion_cb) completion_cb(opaque, err); } + +typedef struct QemuErrorSink QemuErrorSink; +struct QemuErrorSink { + enum { + ERR_SINK_FILE, + ERR_SINK_MONITOR, + } dest; + union { + FILE *fp; + Monitor *mon; + }; + QemuErrorSink *previous; +}; + +static __thread 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, ...) +{ + va_list args; + + assert(qemu_error_sink != NULL); + switch (qemu_error_sink->dest) { + case ERR_SINK_FILE: + va_start(args, fmt); + vfprintf(qemu_error_sink->fp, 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; + } +} diff --git a/sysemu.h b/sysemu.h index dffb2f1..449dc83 100644 --- a/sysemu.h +++ b/sysemu.h @@ -65,6 +65,11 @@ int qemu_savevm_state_complete(QEMUFile *f); int qemu_savevm_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))); + #ifdef _WIN32 /* Polling handling */ diff --git a/vl.c b/vl.c index 8b2b289..c1b11cd 100644 --- a/vl.c +++ b/vl.c @@ -4815,6 +4815,7 @@ int main(int argc, char **argv, char **envp) CPUState *env; int show_vnc_port = 0; + qemu_errors_to_file(stderr); qemu_cache_utils_init(envp); LIST_INIT (&vm_change_state_head); -- 1.6.2.5