On 13.09.12 13:14:30, Namhyung Kim wrote: > From: Namhyung Kim <namhyung.kim@xxxxxxx> > > 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@xxxxxxx> > Signed-off-by: Namhyung Kim <namhyung@xxxxxxxxxx> > --- > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-tip-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html