[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/11] trace: no need to call trace_backend_init
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 04/11] trace: no need to call trace_backend_init in different branches now |
Date: |
Mon, 26 Oct 2015 18:18:55 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 26/10/2015 10:10, Denis V. Lunev wrote:
> original idea to split calling locations was to trace tracing thread
> in the final child process according to
>
> commit 8a745f2a9296ad2cf6bda33534ed298f2625a4ad
> Author: Michael Mueller
> Date: Mon Sep 23 16:36:54 2013 +0200
>
> os_daemonize is now on top of both locations. Drop unneeded ifs.
Oh, that's nice! :-)
Reviewed-by: Paolo Bonzini <address@hidden>
Paolo
> Signed-off-by: Denis V. Lunev <address@hidden>
> ---
> vl.c | 12 ++----------
> 1 file changed, 2 insertions(+), 10 deletions(-)
>
> diff --git a/vl.c b/vl.c
> index b1ddd3f..7ce9d70 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -4083,10 +4083,8 @@ int main(int argc, char **argv, char **envp)
> qemu_set_log(mask);
> }
>
> - if (!is_daemonized()) {
> - if (!trace_init_backends()) {
> - exit(1);
> - }
> + if (!trace_init_backends()) {
> + exit(1);
> }
>
> /* If no data_dir is specified then try to find it relative to the
> @@ -4635,12 +4633,6 @@ int main(int argc, char **argv, char **envp)
>
> os_setup_post();
>
> - if (is_daemonized()) {
> - if (!trace_init_backends()) {
> - exit(1);
> - }
> - }
> -
> main_loop();
> bdrv_close_all();
> pause_all_vcpus();
>
- [Qemu-devel] [PATCH v2 00/11] simplify usage of tracepoints, and connect them to logging, Denis V. Lunev, 2015/10/26
- [Qemu-devel] [PATCH 02/11] trace: split trace_init_events out of trace_init_backends, Denis V. Lunev, 2015/10/26
- [Qemu-devel] [PATCH 05/11] trace: add "-trace enable=...", Denis V. Lunev, 2015/10/26
- [Qemu-devel] [PATCH 03/11] trace: split trace_init_file out of trace_init_backends, Denis V. Lunev, 2015/10/26
- [Qemu-devel] [PATCH 04/11] trace: no need to call trace_backend_init in different branches now, Denis V. Lunev, 2015/10/26
- Re: [Qemu-devel] [PATCH 04/11] trace: no need to call trace_backend_init in different branches now,
Paolo Bonzini <=
- [Qemu-devel] [PATCH 01/11] trace: fix documentation, Denis V. Lunev, 2015/10/26
- [Qemu-devel] [PATCH 08/11] log: move qemu-log.c into util/ directory, Denis V. Lunev, 2015/10/26
- [Qemu-devel] [PATCH 06/11] trace: add "-trace help", Denis V. Lunev, 2015/10/26
- [Qemu-devel] [PATCH 09/11] trace: convert stderr backend to log, Denis V. Lunev, 2015/10/26