Re: Specific support for Intel Atom architecture

From: Willy Tarreau
Date: Sun May 03 2009 - 01:39:50 EST


On Thu, Apr 30, 2009 at 10:10:08AM -0700, H. Peter Anvin wrote:
> Ingo Molnar wrote:
> >> diff --git a/arch/x86/Makefile_32.cpu b/arch/x86/Makefile_32.cpu
> >> index 80177ec..07a11b0 100644
> >> --- a/arch/x86/Makefile_32.cpu
> >> +++ b/arch/x86/Makefile_32.cpu
> >> @@ -33,6 +33,7 @@ cflags-$(CONFIG_MCYRIXIII) += $(call cc-option,-march=c3,-march=i486) $(align)-f
> >> cflags-$(CONFIG_MVIAC3_2) += $(call cc-option,-march=c3-2,-march=i686)
> >> cflags-$(CONFIG_MVIAC7) += -march=i686
> >> cflags-$(CONFIG_MCORE2) += -march=i686 $(call tune,core2)
> >> +cflags-$(CONFIG_MATOM) += -march=atom $(call tune,atom)
> >>
>
> There should be a fallback option used here rather than requiring a new
> gcc, e.g. something like:
>
> $(call cc-option,-march=atom,-march=i686)

if it's an in-order architecture, wouldn't it be better to tune for i386
or i486 instead ?

Willy

--
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/