Messages in this thread |  | | Date | Thu, 13 Sep 2012 11:48:26 +0200 | From | Robert Richter <> | Subject | Re: [PATCH] perf record: Add missing perf_hpp__init for pipe-mode |
| |
On 13.09.12 13:14:30, Namhyung Kim wrote: > From: Namhyung Kim <namhyung.kim@lge.com> > > The perf_hpp__init() function was only called from setup_browser() so > that the pipe-mode missed the initialization thus didn't respond to > related options. Fix it. > > Reported-by: Robert Richter <robert.richter@amd.com> > Signed-off-by: Namhyung Kim <namhyung@kernel.org> > --- > tools/perf/builtin-report.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-)
This patch fixes it.
Thanks,
-Robert
> > diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c > index 97b2e6300f4c..279155a47d1c 100644 > --- a/tools/perf/builtin-report.c > +++ b/tools/perf/builtin-report.c > @@ -689,8 +689,10 @@ int cmd_report(int argc, const char **argv, const char *prefix __maybe_unused) > > if (strcmp(report.input_name, "-") != 0) > setup_browser(true); > - else > + else { > use_browser = 0; > + perf_hpp__init(false, false); > + } > > /* > * Only in the newt browser we are doing integrated annotation, > -- > 1.7.11.4 > >
-- Advanced Micro Devices, Inc. Operating System Research Center
|  |