perf kvm: Remove invocation of setup/exit_browser()
authorNamhyung Kim <namhyung@kernel.org>
Thu, 10 Dec 2015 03:00:55 +0000 (12:00 +0900)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Thu, 10 Dec 2015 17:35:35 +0000 (14:35 -0300)
Calling setup_browser(false) with use_browser = 0 is meaningless.
Just get rid of it.  This is necessary to remove the browser
dependency from usage_with_options() and friends.

Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Reviewed-by: Josh Poimboeuf <jpoimboe@redhat.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1449716459-23004-4-git-send-email-namhyung@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/builtin-kvm.c

index dd94b4ca22131a211729082925bbbf082d19c73e..031f9f55c2819f0fb4f529ef61e239df5b216b3f 100644 (file)
@@ -1351,7 +1351,6 @@ static int kvm_events_live(struct perf_kvm_stat *kvm,
        disable_buildid_cache();
 
        use_browser = 0;
-       setup_browser(false);
 
        if (argc) {
                argc = parse_options(argc, argv, live_options,
@@ -1409,8 +1408,6 @@ static int kvm_events_live(struct perf_kvm_stat *kvm,
        err = kvm_events_live_report(kvm);
 
 out:
-       exit_browser(0);
-
        if (kvm->session)
                perf_session__delete(kvm->session);
        kvm->session = NULL;