Thanks, I'll forward the patches to quipper team. On Tue, Apr 21, 2015 at 8:47 PM, Andi Kleen <a...@firstfloor.org> wrote: > On Wed, Apr 22, 2015 at 05:15:47AM +0200, Andi Kleen wrote: >> On Tue, Apr 21, 2015 at 01:52:18PM -0700, Dehao Chen wrote: >> > Andi, >> > >> > Thanks for the patches. Turns out that the first 3 patches are already >> > in, the correct upstream quipper repository is: >> > >> > https://chromium.googlesource.com/chromiumos/platform2/+/master/chromiumos-wide-profiling/ >> > >> > The last 3 patches seem to be local hacks. Do you want any of them in? >> > >> > I just did a batch sync with quipper head. Please let me know if this >> > solves the perf problem. >> >> Still outdated: >> >> F0421 20:13:16.221422 22297 perf_reader.cc:1614] Check failed: attr_size <= >> sizeof(perf_event_attr) (104 vs. 96) > > It converts with the attached patches, but there's still some problem > parsing the data: > > % ./create_gcov -binary loop -gcov_version 1 -gcov loop.gcda -gcov_version > 0x500e > % gcc50 -O2 -fprofile-use loop.c > loop.c:1:0: warning: '/home/andi/src/autofdo/loop.gcda' is version ', > expected version '500e' > %
You need to use -fauto-profile=loop.gcda instead of "-fprofile-use", which is only for instrumentation based FDO. Dehao > > -Andi >