summaryrefslogtreecommitdiffstats
path: root/samples/bpf/fds_example.c
diff options
context:
space:
mode:
authorKan Liang <kan.liang@intel.com>2016-12-13 16:29:44 +0100
committerArnaldo Carvalho de Melo <acme@redhat.com>2016-12-20 16:00:38 +0100
commited6c166cc7dc329736cace3affd2df984fb22ec8 (patch)
tree45f700ec74fe62b6ac93923749eaba5def503ec7 /samples/bpf/fds_example.c
parentperf annotate: Don't throw error for zero length symbols (diff)
downloadlinux-ed6c166cc7dc329736cace3affd2df984fb22ec8.tar.xz
linux-ed6c166cc7dc329736cace3affd2df984fb22ec8.zip
perf diff: Do not overwrite valid build id
Fixes a perf diff regression issue which was introduced by commit 5baecbcd9c9a ("perf symbols: we can now read separate debug-info files based on a build ID") The binary name could be same when perf diff different binaries. Build id is used to distinguish between them. However, the previous patch assumes the same binary name has same build id. So it overwrites the build id according to the binary name, regardless of whether the build id is set or not. Check the has_build_id in dso__load. If the build id is already set, use it. Before the fix: $ perf diff 1.perf.data 2.perf.data # Event 'cycles' # # Baseline Delta Shared Object Symbol # ........ ....... ................ ............................. # 99.83% -99.80% tchain_edit [.] f2 0.12% +99.81% tchain_edit [.] f3 0.02% -0.01% [ixgbe] [k] ixgbe_read_reg After the fix: $ perf diff 1.perf.data 2.perf.data # Event 'cycles' # # Baseline Delta Shared Object Symbol # ........ ....... ................ ............................. # 99.83% +0.10% tchain_edit [.] f3 0.12% -0.08% tchain_edit [.] f2 Signed-off-by: Kan Liang <kan.liang@intel.com> Cc: Andi Kleen <andi@firstfloor.org> CC: Dima Kogan <dima@secretsauce.net> Cc: Jiri Olsa <jolsa@redhat.com> Cc: Namhyung Kim <namhyung@kernel.org> Fixes: 5baecbcd9c9a ("perf symbols: we can now read separate debug-info files based on a build ID") Link: http://lkml.kernel.org/r/1481642984-13593-1-git-send-email-kan.liang@intel.com Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'samples/bpf/fds_example.c')
0 files changed, 0 insertions, 0 deletions