Re: [PATCH] perf record: Add missing perf_hpp__init for pipe-mode

2012-09-13 Thread Robert Richter
On 13.09.12 13:14:30, Namhyung Kim wrote: > From: Namhyung Kim > > 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 > Signed-off-by: Namhyung

Re: [PATCH] perf record: Add missing perf_hpp__init for pipe-mode

2012-09-13 Thread Robert Richter
On 13.09.12 13:14:30, Namhyung Kim wrote: From: Namhyung Kim namhyung@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

Re: [PATCH] perf record: Add missing perf_hpp__init for pipe-mode

2012-09-12 Thread Namhyung Kim
Oops, please do s/record/report/ on the subject line, sorry! ;-) Also I don't see the issue on the other users of the setup_browser() - i.e. perf annotate and perf top - since they call it unconditionally. Thanks, Namhyung -- To unsubscribe from this list: send the line "unsubscribe

[PATCH] perf record: Add missing perf_hpp__init for pipe-mode

2012-09-12 Thread Namhyung Kim
From: Namhyung Kim 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 Signed-off-by: Namhyung Kim --- tools/perf/builtin-report.c | 4 +++- 1 file

[PATCH] perf record: Add missing perf_hpp__init for pipe-mode

2012-09-12 Thread Namhyung Kim
From: Namhyung Kim namhyung@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.rich...@amd.com Signed-off-by: Namhyung Kim

Re: [PATCH] perf record: Add missing perf_hpp__init for pipe-mode

2012-09-12 Thread Namhyung Kim
Oops, please do s/record/report/ on the subject line, sorry! ;-) Also I don't see the issue on the other users of the setup_browser() - i.e. perf annotate and perf top - since they call it unconditionally. Thanks, Namhyung -- To unsubscribe from this list: send the line unsubscribe linux-kernel