Skip to content

Commit

Permalink
trace: no need to call trace_backend_init in different branches now
Browse files Browse the repository at this point in the history
original idea to split calling locations was to spawn tracing thread
in the final child process according to

    commit 8a745f2
    Author: Michael Mueller
    Date:   Mon Sep 23 16:36:54 2013 +0200

os_daemonize is now on top of both locations. Drop unneeded ifs.

Signed-off-by: Denis V. Lunev <den@openvz.org>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Message-id: 1452174932-28657-5-git-send-email-den@openvz.org
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
  • Loading branch information
Denis V. Lunev authored and stefanhaRH committed Feb 3, 2016
1 parent 41fc57e commit f246b86
Showing 1 changed file with 2 additions and 10 deletions.
12 changes: 2 additions & 10 deletions vl.c
Expand Up @@ -4110,10 +4110,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
Expand Down Expand Up @@ -4657,12 +4655,6 @@ int main(int argc, char **argv, char **envp)

os_setup_post();

if (is_daemonized()) {
if (!trace_init_backends()) {
exit(1);
}
}

main_loop();
replay_disable_events();

Expand Down

0 comments on commit f246b86

Please sign in to comment.