Re: [PATCH v2] perf tools: don't adjust symbols in vDSO

2015-07-09 Thread Will Deacon
Hi Adrian, On Tue, Jun 30, 2015 at 10:57:04AM +0100, Adrian Hunter wrote: > On 29/06/15 16:52, Will Deacon wrote: > > On Mon, Jun 29, 2015 at 02:23:51PM +0100, Adrian Hunter wrote: > >> On 29/06/15 15:30, Adrian Hunter wrote: > >>> On 29/06/15 12:02, Adrian Hunter wrote: > On 27/06/15 12:10,

Re: [PATCH v2] perf tools: don't adjust symbols in vDSO

2015-06-30 Thread Will Deacon
On Tue, Jun 30, 2015 at 10:57:04AM +0100, Adrian Hunter wrote: > On 29/06/15 16:52, Will Deacon wrote: > > On Mon, Jun 29, 2015 at 02:23:51PM +0100, Adrian Hunter wrote: > >> On 29/06/15 15:30, Adrian Hunter wrote: > >>> On 29/06/15 12:02, Adrian Hunter wrote: > On 27/06/15 12:10, Will Deacon

Re: [PATCH v2] perf tools: don't adjust symbols in vDSO

2015-06-30 Thread Adrian Hunter
On 29/06/15 16:52, Will Deacon wrote: > On Mon, Jun 29, 2015 at 02:23:51PM +0100, Adrian Hunter wrote: >> On 29/06/15 15:30, Adrian Hunter wrote: >>> On 29/06/15 12:02, Adrian Hunter wrote: On 27/06/15 12:10, Will Deacon wrote: > @@ -824,6 +823,14 @@ int dso__load_sym(struct dso *dso, stru

Re: [PATCH v2] perf tools: don't adjust symbols in vDSO

2015-06-29 Thread a...@redhat.com
Em Mon, Jun 29, 2015 at 02:52:39PM +0100, Will Deacon escreveu: > On Mon, Jun 29, 2015 at 02:23:51PM +0100, Adrian Hunter wrote: > > On 29/06/15 15:30, Adrian Hunter wrote: > > > On 29/06/15 12:02, Adrian Hunter wrote: > > >> On 27/06/15 12:10, Will Deacon wrote: > > >>> @@ -824,6 +823,14 @@ int ds

Re: [PATCH v2] perf tools: don't adjust symbols in vDSO

2015-06-29 Thread Will Deacon
On Mon, Jun 29, 2015 at 02:23:51PM +0100, Adrian Hunter wrote: > On 29/06/15 15:30, Adrian Hunter wrote: > > On 29/06/15 12:02, Adrian Hunter wrote: > >> On 27/06/15 12:10, Will Deacon wrote: > >>> @@ -824,6 +823,14 @@ int dso__load_sym(struct dso *dso, struct map *map, > >>> sec = syms_ss->symta

Re: [PATCH v2] perf tools: don't adjust symbols in vDSO

2015-06-29 Thread Adrian Hunter
On 29/06/15 15:30, Adrian Hunter wrote: > On 29/06/15 12:02, Adrian Hunter wrote: >> On 27/06/15 12:10, Will Deacon wrote: >>> Commit 922d0e4d9f04 ("perf tools: Adjust symbols in VDSO") changed the >>> ELF symbol parsing so that the vDSO is treated the same as ET_EXEC and >>> ET_REL binaries despit

Re: [PATCH v2] perf tools: don't adjust symbols in vDSO

2015-06-29 Thread Adrian Hunter
On 29/06/15 12:02, Adrian Hunter wrote: > On 27/06/15 12:10, Will Deacon wrote: >> Commit 922d0e4d9f04 ("perf tools: Adjust symbols in VDSO") changed the >> ELF symbol parsing so that the vDSO is treated the same as ET_EXEC and >> ET_REL binaries despite being an ET_DYN. This was a partial workarou

Re: [PATCH v2] perf tools: don't adjust symbols in vDSO

2015-06-29 Thread Adrian Hunter
On 27/06/15 12:10, Will Deacon wrote: > Commit 922d0e4d9f04 ("perf tools: Adjust symbols in VDSO") changed the > ELF symbol parsing so that the vDSO is treated the same as ET_EXEC and > ET_REL binaries despite being an ET_DYN. This was a partial workaround > to deal with older x86 vDSOs being preli

[PATCH v2] perf tools: don't adjust symbols in vDSO

2015-06-27 Thread Will Deacon
Commit 922d0e4d9f04 ("perf tools: Adjust symbols in VDSO") changed the ELF symbol parsing so that the vDSO is treated the same as ET_EXEC and ET_REL binaries despite being an ET_DYN. This was a partial workaround to deal with older x86 vDSOs being prelinked at a high address that didn't correspond