Re: [PATCH 1/9] perf tools: Add build id shell test

From: Jiri Olsa
Date: Tue Oct 13 2020 - 15:15:23 EST


On Tue, Oct 13, 2020 at 01:13:40PM -0300, Arnaldo Carvalho de Melo wrote:
> Em Wed, Sep 30, 2020 at 07:15:04PM +0200, Jiri Olsa escreveu:
> > Adding test for build id cache that adds binary
> > with sha1 and md5 build ids and verifies it's
> > added properly.
> >
> > The test updates build id cache with perf record
> > and perf buildid-cache -a.
>
>
> [root@five ~]# perf test "build id"
> 82: build id cache operations : Skip
> [root@five ~]# set -o vi
> [root@five ~]# perf test -v "build id"
> 82: build id cache operations :
> --- start ---
> test child forked, pid 88384
> failed: no test binaries
> test child finished with -2
> ---- end ----
> build id cache operations: Skip
> [root@five ~]#

hm, output looks like older version of the test

>
> Also the other patches clashed with Namhyung's patch series, can you
> please check?

right, new api is different now

>
> I've just pushed what I have to acme/perf/core

I rebased and pushed perf/build_id_size branch, also will post new version

thanks,
jirka