Re: [PATCH] kernel.spec: enable building of perf with libnuma support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Oct 24, 2014 at 10:22 AM, Josh Boyer <jwboyer@xxxxxxxxxxxxxxxxx> wrote:
> On Fri, Oct 24, 2014 at 10:19 AM, Dan Horák <dan@xxxxxxxx> wrote:
>> On Fri, 24 Oct 2014 10:10:11 -0400
>> Josh Boyer <jwboyer@xxxxxxxxxxxxxxxxx> wrote:
>>
>>> On Fri, Oct 24, 2014 at 12:01:21PM -0200, Arnaldo Carvalho de Melo
>>> wrote:
>>> > Em Fri, Oct 24, 2014 at 09:51:35AM -0400, Josh Boyer escreveu:
>>> > > On Thu, Oct 23, 2014 at 06:51:50PM +0200, Petr Holasek wrote:
>>> > > > Patch makes perf package dependent on numactl-devel and so
>>> > > > enables numa subsystem benchmarks for perf bench command.
>>> > > > numactl-devel is added to the buildroot just for x86 archs,
>>> > > > because we don't ship libnuma for any others architectures.
>>> > > >
>>> > > > Can be easily reproduced by calling 'perf bench numa mem -a':
>>> > > >
>>> > > > - current output:
>>> > > >
>>> > > > 'Unknown subsystem:numa'
>>> > > >
>>> > > > - built with this patch:
>>> > > >
>>> > > > 'Running numa/mem benchmark...'
>>> > > >
>>> > > > Signed-off-by: Petr Holasek <pholasek@xxxxxxxxxx>
>>> > >
>>> > > Thanks.  I'll look at getting this into today's rawhide build.
>>> >
>>> > FYI, checked with Jiri Olsa and he tells me that in RHEL7 it is:
>>> >
>>> > %if %{with_perf}
>>> > BuildRequires: elfutils-devel zlib-devel binutils-devel bison
>>> > BuildRequires: audit-libs-devel
>>> > %ifnarch s390 s390x
>>> > BuildRequires: numactl-devel
>>> > %endif
>>> > %endif
>>>
>>> Thanks.  I'll leave it as-is until the secondary arch teams request
>>> numa support.
>>
>> my opinion is to go with the RHEL %ifnarch s390 s390x - we (at least
>> ppc + s390) want to be close to the enterprise kernel
>
> But then I have to do work instead of just running 'git am' :).  OK,
> OK.  I'll tweak it to match RHEL.

And of course, this failed to build.  Petr had it correct originally
for primary koji, as numactl is only available for i686 and x86_64.
It's missing on armv7hl.  Sigh.

http://koji.fedoraproject.org/koji/buildinfo?buildID=587593

josh
_______________________________________________
kernel mailing list
kernel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/kernel





[Index of Archives]     [Fedora General Discussion]     [Older Fedora Users Archive]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Tux]     [Yosemite News]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [USB]     [Asterisk PBX]

  Powered by Linux