Re: [PATCH] rt-tests - allow build without NUMA

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

 



On Mon, Sep 26, 2011 at 01:16:19PM -0700, Frank Rowand wrote:
> On 09/23/11 14:09, Clark Williams wrote:
> > On Fri, 23 Sep 2011 21:46:32 +0200
> > Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> wrote:
> > 
> >> On Fri, Sep 23, 2011 at 12:26:03PM -0700, Frank Rowand wrote:
> >>>
> >>>
> > 
> > 
> > How about this?
> > 
> > diff --git a/Makefile b/Makefile
> > index 4f51154..4776391 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -14,7 +14,11 @@ bindir  ?= $(prefix)/bin
> >  mandir ?= $(prefix)/share/man
> >  srcdir ?= $(prefix)/src
> >  
> > -NUMA   := 1
> > +machinetype = $(shell uname -m | \
> > +    sed -e 's/i.86/i386/' -e 's/mips.*/mips/' -e 's/ppc.*/powerpc/')
> > +ifneq ($(filter x86_64 i386 ia64 mips powerpc,$(machinetype)),)
> > +NUMA   := 1
> > +endif
> >  
> >  CFLAGS = -D_GNU_SOURCE -Wall -Wno-nonnull -Isrc/include
> >  
> > @@ -26,7 +30,7 @@ else
> >         CFLAGS  += -O0 -g
> >  endif
> >  
> > -ifdef NUMA
> > +ifeq ($(NUMA),1)
> >         CFLAGS += -DNUMA
> >         NUMA_LIBS = -lnuma
> >  endif
> 
> That works for me.
> 
> The change from Uwe doesn't solve the NUMA issue for me because I
> cross-compile, but I am quite happy to type:
Yeah, my patch was meant on top of your's. So I get exactly what I
wanted by Clark :-)

Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
--
To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [RT Stable]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]

  Powered by Linux