Messages in this thread | | | Date | Thu, 2 Oct 2014 12:13:07 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [PATCH 2/5] perf tools: Move callchain_param to util object in to fix python test |
| |
Em Thu, Oct 02, 2014 at 05:08:27PM +0200, Jiri Olsa escreveu: > On Thu, Oct 02, 2014 at 11:59:49AM -0300, Arnaldo Carvalho de Melo wrote: > > Em Wed, Oct 01, 2014 at 06:00:26PM +0200, Jiri Olsa escreveu: > > > In following commit we changed the location of callchains data: > > > b016b5ba554b perf tools: Move callchain config from record_opts to callchain_param > > > > > > Now all callchains stuff stays in callchain_param struct, > > > which adds its dependency for evsel.c object and breaks > > > python perf.so usage (unresolved callchain_param). > > > > > > Moving callchain_param into callchain.c and adding it into > > > python-ext-sources unleash just another dependency hell, > > > so I ended up adding callchain_param into util.c for now. > > > > Against what branch is the patch? I am not finding b016b5ba554b in none > > [jolsa@krava perf]$ git describe --contains b016b5ba554b > fatal: cannot describe 'b016b5ba554ba38c74241ef8739f8fde8878d3eb' > [jolsa@krava perf]$ git show --oneline b016b5ba554b | head -1 > b016b5ba554b perf tools: Move callchain config from record_opts to callchain_param > [jolsa@krava perf]$ git show 72a128aa083a > /tmp/p > [jolsa@krava perf]$ git show b016b5ba554b > /tmp/p1 > [jolsa@krava perf]$ diff -puw /tmp/p /tmp/p1 > --- /tmp/p 2014-10-02 17:06:20.072958720 +0200 > +++ /tmp/p1 2014-10-02 17:06:26.035946113 +0200 > @@ -1,4 +1,4 @@ > -commit 72a128aa083a7f4cc4f800718aaae05d9c698e26 > +commit b016b5ba554ba38c74241ef8739f8fde8878d3eb > Author: Namhyung Kim <namhyung@kernel.org> > Date: Tue Sep 23 10:01:41 2014 +0900 > > > I guess I took some version before your rebase > > > of the branches in my linux git repo and the python 'perf test' entry > > works as expected: > > > > [root@zoo ~]# perf test python > > 17: Try 'use perf' in python, checking link problems : Ok > > [root@zoo ~]# > > hum.. yours acme/perf/core is failing for me: > > [jolsa@krava perf]$ ./perf test python > 17: Try 'use perf' in python, checking link problems : FAILED! > [jolsa@krava perf]$ git show --oneline HEAD | head -1 > 281f92f233a5 perf record: Fix error message for --filter option not coming after tracepoint > > maybe you need 'make clean && make' ?
Yeap, I somehow thought that I had that covered :-\
Reproduced, yeah, 72a128aa083a7f4cc4f800718aaae05d9c698e26 is in, and I'll take your patch, thanks.
Will update the comment to refer to the right cset.
- Arnaldo
| |