All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Phil Endecott" <phil_wueww_endecott@chezphil.org>
To: "Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] Mention Intel Atom in Kconfig.cpu
Date: Mon, 13 Oct 2008 20:22:22 +0100	[thread overview]
Message-ID: <1223925742287@dmwebmail.dmwebmail.chezphil.org> (raw)
In-Reply-To: <20081013101722.2a1b3afa@infradead.org>

Arjan van de Ven wrotes:
> On Mon, 13 Oct 2008 17:02:30 +0300
> Adrian Bunk <bunk <at> kernel.org> wrote:
> 
> > On Mon, Oct 13, 2008 at 09:30:14AM -0400, Arjan van de Ven wrote:
> > > On Mon, 13 Oct 2008 15:30:51 +0200
> > > Andi Kleen <andi <at> firstfloor.org> wrote:
> > > > 
> > > > > Core2 instruction set with tune=generic is still the best to
> > > > > set.
> > > > 
> > > > Not sure that is true. These option are mostly for the compiler.
> > > 
> > > exactly, and our benchmarks show that tune=generic is best right now
> > > for Atom. 
> > > (586 scheduling sounds nice, but the pipelines are rather different.
> > > And the benchmarks don't lie..  
> > 
> > That sounds a bit dangerous since tune=generic is documented to
> > change the semantics between gcc versions to better fit more recent
> > CPUs (there's even a small difference between gcc 4.2 and gcc 4.3):
> > 
> 
> reality is that tune=generic avoids the things that are "really bad"
> for a wide generation of cpus; the world of x86 is such that there
> really are many common things that are good for the vast majority of
> the cpus out there (or at least neutral). 
> 
> Future versions of GCC might have a specific ATOM model. Until they do,
> tune=generic is the right thing based on tests over a few gcc versions.
> Yes it's a bit fluid, but no gcc isn't going to suddenly go do stupid
> things for currently mass-sold cpus.

Well, if the Intel experts can't even agree, what hope do I have of 
getting it right :-(   I chose Core2 because I read somewhere that Atom 
was "feature compatible" with it, but of course that doesn't say 
anything about the optimal optimisations.  I trust that someone will 
update Kconfig.cpu with their idea of the right choice for Atom eventually.

(Maybe there should be a way to auto-suggest the right setting for a 
native build based on /proc/cpuinfo?  I think the gcc build process can 
do something like that.)

Cheers,  Phil.





      reply	other threads:[~2008-10-13 19:22 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-13 10:58 [PATCH] Mention Intel Atom in Kconfig.cpu Phil Endecott
2008-10-13 12:13 ` [PATCH] Mention Intel Atom in Kconfig.cpu (less garbled this time) Phil Endecott
2008-10-13 15:36   ` J.A. Magallón
2008-10-13 12:28 ` [PATCH] Mention Intel Atom in Kconfig.cpu Andi Kleen
2008-10-13 13:18   ` Arjan van de Ven
2008-10-13 13:30     ` Andi Kleen
2008-10-13 13:30       ` Arjan van de Ven
2008-10-13 13:53         ` Andi Kleen
2008-10-13 14:02         ` Adrian Bunk
2008-10-13 14:17           ` Arjan van de Ven
2008-10-13 19:22             ` Phil Endecott [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1223925742287@dmwebmail.dmwebmail.chezphil.org \
    --to=phil_wueww_endecott@chezphil.org \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.