Re: [PATCH 22/24] perf tools: Build programs to copy 32-bit compatibility VDSOs

[Arnaldo Carvalho de Melo]

Em Fri, Aug 15, 2014 at 10:08:57PM +0300, Adrian Hunter escreveu:

> perf tools copy VDSO out of memory.  However, on 64-bit machines there
> may be 32-bit compatibility VDOs also.  To copy those requires
> separate 32-bit executables.  This patch adds to the build additional
> programs perf-read-vdso32 and perf-read-vdsox32 for 32-bit and x32
> respectively.
 
> Signed-off-by: Adrian Hunter <adrian.hunter@xxxxxxxxx>
<SNIP>
>  tools/perf/config/feature-checks/test-compile.c |  4 +++
<SNIP>
>  8 files changed, 149 insertions(+), 37 deletions(-)
<SNIP>
>  create mode 100644 tools/perf/config/feature-checks/test-compile.c
<SNIP>


In resolving conflicts that happen because I haven't merged some other
files (thread_stack, etC) I noticed that this patch introduces something
that is explicitely listed in a .gitignore file:

[acme@zoo linux]$ git show d316f16dddd8c7780c727bbd507338e4147c9280
commit d316f16dddd8c7780c727bbd507338e4147c9280
Author: Alexander Shishkin <alexander.shishkin@xxxxxxxxxxxxxxx>
Date:   Fri Aug 15 22:08:59 2014 +0300

    perf tools: Add feature checks to .gitignore

<SNIP>

+++ b/tools/perf/.gitignore
@@ -24,3 +24,4 @@ config.mak.autogen
 *-flex.*
 *.pyc
 *.pyo
+config/feature-checks/test-*
[acme@zoo linux]$


So I think we should use some different name for this test-compile.c
file?

I'll leave those two csets out of this pull req, so that you can rebase
what is left on top of what is being merged and we can do one more
progress step in having all this merged, ok?

I mean:

>> Up-to-date versions can be found from 15 August:
>>
>>      patchwork:
>>
>>              https://patchwork.kernel.org/patch/4729131/
>>              https://patchwork.kernel.org/patch/4729111/


- Arnaldo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/