Re: [PATCH 00/12] Stitch LBR call stack (Perf Tools)

[Liang, Kan]

On 3/9/2020 9:27 AM, Arnaldo Carvalho de Melo wrote:
> Em Fri, Mar 06, 2020 at 02:13:15PM -0500, Liang, Kan escreveu:
> > On 3/6/2020 4:39 AM, Jiri Olsa wrote:
> > > On Fri, Feb 28, 2020 at 08:29:59AM -0800, kan.liang@xxxxxxxxxxxxxxx wrote:
> > >
> > > SNIP
> > >
> > > > Kan Liang (12):
> > > >     perf tools: Add hw_idx in struct branch_stack
> > > >     perf tools: Support PERF_SAMPLE_BRANCH_HW_INDEX
> > > >     perf header: Add check for event attr
> > > >     perf pmu: Add support for PMU capabilities
> > >
> > > hi,
> > > I'm getting compile error:
> > >
> > > 	util/pmu.c: In function âperf_pmu__caps_parseâ:
> > > 	util/pmu.c:1620:32: error: â%sâ directive output may be truncated writing up to 255 bytes into a region of size between 0 and 4095 [-Werror=format-truncation=]
> > > 	 1620 |   snprintf(path, PATH_MAX, "%s/%s", caps_path, name);
> > > 	      |                                ^~
> > > 	In file included from /usr/include/stdio.h:867,
> > > 			 from util/pmu.c:12:
> > > 	/usr/include/bits/stdio2.h:67:10: note: â__builtin___snprintf_chkâ output between 2 and 4352 bytes into a destination of size 4096
> > > 	   67 |   return __builtin___snprintf_chk (__s, __n, __USE_FORTIFY_LEVEL - 1,
> > > 	      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > 	   68 |        __bos (__s), __fmt, __va_arg_pack ());
> > > 	      |        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > 	cc1: all warnings being treated as errors
> > >
> > > 	[jolsa@krava perf]$ gcc --version
> > > 	gcc (GCC) 9.2.1 20190827 (Red Hat 9.2.1-1)
> >
> > My GCC version is too old. I will send V2 later to fix the error.
>
> So I stopped at the patch just before the one introducing this problem,
> i.e. now I have:
>
> [acme@seventh perf]$ git log --oneline -10
> 5100c2b77049 (HEAD -> perf/core, five/perf/core, acme/perf/core) perf header: Add check for unexpected use of reserved membrs in event attr
> 1d2fc2bd7c1c perf evsel: Support PERF_SAMPLE_BRANCH_HW_INDEX
> 1fa65c5092da perf tools: Add hw_idx in struct branch_stack
> 6339998d22ec tools headers UAPI: Update tools's copy of linux/perf_event.h
> 401d61cbd4d4 tools lib traceevent: Remove extra '\n' in print_event_time()
> 76ce02651dab libperf: Add counting example
> dabce16bd292 perf annotate: Get rid of annotation->nr_jumps
> 357a5d24c471 perf llvm: Add debug hint message about missing kernel-devel package
> 1af62ce61cd8 perf stat: Show percore counts in per CPU output
> 7982a8985150 tools lib api fs: Move cgroupsfs_find_mountpoint()
> [acme@seventh perf]$
>
> Please continue from there, I'll process some other patchsets,

Sure. I will re-base my patchset on top of it.

Thanks,
Kan