lkml.org 
[lkml]   [2015]   [May]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: arm/arm64 perf build issue with mainline
Em Mon, May 11, 2015 at 02:33:04PM +0200, Ingo Molnar escreveu:
>
> * Will Deacon <will.deacon@arm.com> wrote:
>
> > On Mon, May 11, 2015 at 09:21:20AM +0100, Ingo Molnar wrote:
> > > * David Ahern <david.ahern@oracle.com> wrote:
> > > > On 4/23/15 5:29 AM, Will Deacon wrote:
> > > > >Commit 6428c59a97de ("perf tools: Set JOBS based on CPU or processor")
> > > > >causes weird behaviour on arm/arm64 platforms because we use the "CPU"
> > > > >prefix for things like:
> > > > >
> > > > >CPU implementer : 0x41
> > > > >CPU architecture: 8
> > > > >CPU variant : 0x0
> > > > >CPU part : 0xd03
> > > > >CPU revision : 0
> > > > >
> > > > >in /proc/cpuinfo. Consequently, a 6 core machine ends up doing:
> > > > >
> > > > >will@confinement-loaf:~/linux/tools/perf$ make
> > > > > BUILD: Doing 'make -j36' parallel build
> > > > >
> > > > >which is a little overwhelming. Any chance we can predicate the extra
> > > > >part of the regex on $(ARCH) being sparc?
> > >
> > > That regex needs to be fixed or replaced with a more robust 'number of
> > > CPUs on the system' discovery method.
> >
> > That was already proposed here (as part of the fallback from getconf):
> >
> > https://lkml.kernel.org/r/20150427190356.GD474@krava.redhat.com
> >
> > but I'm not sure what happened to the patch.
>
> Sending out the latest/best version as a reminder for Arnaldo will
> sure help it along.
>
> Acked-by: Ingo Molnar <mingo@kernel.org>

IIRC it was merged already, lemme check...

- Arnaldo


\
 
 \ /
  Last update: 2015-05-11 18:21    [W:0.046 / U:14.188 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site