Re: [PATCH v2 2/8] perf tools: Introduce perf_hpp__setup_hists_formats()

[Namhyung Kim]

On Fri, Mar 04, 2016 at 06:59:23PM -0300, Arnaldo Carvalho de Melo wrote:
> Em Fri, Mar 04, 2016 at 06:58:22PM -0300, Arnaldo Carvalho de Melo escreveu:
> > Em Fri, Mar 04, 2016 at 11:59:36PM +0900, Namhyung Kim escreveu:
> > > The perf_hpp__setup_hists_formats() is to build hists-specific output
> > > formats (and sort keys).  Currently it's only used in order to build the
> > > output format in a hierarchy with same sort keys, but it could be used
> > > with different sort keys in non-hierarchy mode later.
> > 
> > After applying this one:
> 
> > rebuild it with DEBUG=1, run it on gdb, press 'q' to exit 'perf top':
>  
> > (gdb) run top --stdio --hierarchy
> > Program received signal SIGSEGV, Segmentation fault.
> > 0x000000000051632b in hists_evsel__exit (evsel=0x19b95b0) at util/hist.c:2143
> > 2143			perf_hpp_list__for_each_format_safe(&node->hpp, fmt, pos) {
> > Missing separate debuginfos, use: dnf debuginfo-install bzip2-libs-1.0.6-19.fc23.x86_64 glibc-2.22-10.fc23.x86_64 libunwind-1.1-10.fc23.x86_64 nss-softokn-freebl-3.22.0-1.0.fc23.x86_64 numactl-libs-2.0.10-3.fc23.x86_64 perl-libs-5.22.1-350.fc23.x86_64 python-libs-2.7.10-8.fc23.x86_64 slang-2.3.0-4.fc23.x86_64
> 
> Please continue from acme/perf/core, the first patch is there already.

OK, I'll take a look at the problem.

Thanks,
Namhyung