All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: cbe-oss-dev@ozlabs.org
Cc: linuxppc-dev@ozlabs.org
Subject: Re: [Cbe-oss-dev] [RFC, PATCH] selection of CPU optimization
Date: Thu, 12 Apr 2007 19:26:03 +0200	[thread overview]
Message-ID: <200704121926.03723.arnd@arndb.de> (raw)
In-Reply-To: <981B07C8-D35B-4581-8FC7-DCA58430A081@kernel.crashing.org>

On Thursday 12 April 2007, Kumar Gala wrote:

> > +config PPC_CPU_ANY32
> > +	bool "Generic 32 bit PowerPC"
> > +	depends on PPC_6xx || PPC_8xx || 4xx || E200 || E500
> > +	select PPC_ENABLE_ALTIVEC
> 
> What is PPC_ENABLE_ALTIVEC used for?

The idea was that PPC_ENABLE_ALTIVEC is selected when PPC_ALTIVEC
_can_ be enabled, while PPC_ALTIVEC is selected by some CPUs
directly.

I now remember that I had not done this part right in the patch
I posted. The correct behavior should be something like

choice
	prompt "Processor level"

config PPC_CPU_740
	bool "G3 or higher"
	select PPC_ENABLE_ALTIVEC
	# altivec is possible

config PPC_CPU_86xx
	bool "86xx"
	select PPC_ALTIVEC
	# altivec is required

config PPC_CPU_4xx
	bool "4xx"
	# altivec is not possible

endchoice

config PPC_ENABLE_ALTIVEC
	bool

config PPC_ALTIVEC
	bool "Enable the use of Altivec"
	depends on PPC_ENABLE_ALTIVEC

I'll fix this up in the next version, which should also
be split into simpler patches.

> > +config PPC_CPU_FLAG
> > +	string
> > +	default "" if PPC_CPU_UNSPECIFIED
> > +	default "-mcpu=powerpc64" if PPC_CPU_ANY64
> > +	default "-mcpu=rs64" if PPC_CPU_RS64
> > +	default "-mcpu=power3" if PPC_CPU_POWER3
> > +	default "-mcpu=power4" if PPC_CPU_POWER4
> > +	default "-mcpu=power5" if PPC_CPU_POWER5
> > +	default "-mcpu=power5+" if PPC_CPU_POWER5PLUS
> > +	default "-mcpu=power6" if PPC_CPU_POWER6
> > +	default "-mcpu=970" if PPC_CPU_970
> > +	default "" if PPC_CPU_CELLBE
> > +	default "" if PPC_CPU_PA6T
> > +	default "-mcpu=powerpc" if PPC_CPU_ANY32
> > +	default "-mcpu=601" if PPC_CPU_601
> > +	default "-mcpu=603" if PPC_CPU_603
> > +	default "-mcpu=604" if PPC_CPU_604
> > +	default "-mcpu=750" if PPC_CPU_750
> > +	default "-mcpu=7400" if PPC_CPU_7400
> > +	default "-mcpu=7450" if PPC_CPU_7450
> > +	default "-mcpu=801" if PPC_CPU_801
> > +	default "-mcpu=821" if PPC_CPU_821
> > +	default "-mcpu=823" if PPC_CPU_823
> > +	default "-mcpu=860" if PPC_CPU_860
> > +	default "-mcpu=403" if PPC_CPU_403
> > +	default "-mcpu=405" if PPC_CPU_405
> > +	default "-mcpu=440" if PPC_CPU_440
> > +	default "" if PPC_CPU_E200
> > +	default "" if PPC_CPU_E500
> 
> I think -mcpu=8540 will work for e500 (same for tune)

Interestingly, -mcpu=8540 is known to gcc, according to
gcc-4.0.3 -dumpspecs, but is not listed in the documentation.
There is also -me500, while -me200 seems to be known only
to gas but not gcc.

What would be the right options to pass on e200 and on
pa6t?

> > +config PPC_OF_PLATFORM_PCI
> > +	bool
> > +	depends on PPC64 # not supported on 32 bits yet
> > +	default n
> 
> Should this really be in here? I don't see how OF PCI has anything to  
> do with CPU/core.

Right.

	Arnd <><

  reply	other threads:[~2007-04-12 17:26 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-10 11:15 [patch 3/3] cell: prevent alignment interrupt on local store Akinobu Mita
2007-04-10 12:52 ` Segher Boessenkool
2007-04-11  3:06   ` Akinobu Mita
2007-04-10 21:22 ` Benjamin Herrenschmidt
2007-04-11  2:56   ` Akinobu Mita
2007-04-11  3:30     ` Benjamin Herrenschmidt
2007-04-11 21:03       ` Segher Boessenkool
2007-04-12  4:23         ` Olaf Hering
2007-04-12  5:26           ` Benjamin Herrenschmidt
2007-04-12  6:33             ` Olaf Hering
2007-04-12  6:38               ` Benjamin Herrenschmidt
2007-04-12  8:31                 ` Gabriel Paubert
2007-04-12  8:48                   ` Benjamin Herrenschmidt
2007-04-12  6:50           ` Segher Boessenkool
2007-04-12  6:57             ` [Cbe-oss-dev] " Michael Ellerman
2007-04-12  7:07               ` Segher Boessenkool
2007-04-12 18:43           ` Arnd Bergmann
2007-04-12 18:55             ` Arnd Bergmann
2007-04-12 19:57               ` Segher Boessenkool
2007-04-12 19:52             ` Segher Boessenkool
2007-04-12 13:01   ` [RFC, PATCH] selection of CPU optimization Arnd Bergmann
2007-04-12 16:45     ` Kumar Gala
2007-04-12 17:26       ` Arnd Bergmann [this message]
2007-04-12 18:17         ` [Cbe-oss-dev] " Kumar Gala
2007-04-12 19:25           ` Arnd Bergmann
2007-04-12 20:04           ` Olof Johansson
2007-04-12 20:01             ` Segher Boessenkool
2007-04-12 20:22               ` Olof Johansson
2007-04-12 20:22                 ` Segher Boessenkool
2007-04-12 19:50         ` Segher Boessenkool
2007-04-13  0:10           ` Arnd Bergmann
2007-04-13  2:03             ` Olof Johansson
2007-04-13 18:43             ` Segher Boessenkool

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=200704121926.03723.arnd@arndb.de \
    --to=arnd@arndb.de \
    --cc=cbe-oss-dev@ozlabs.org \
    --cc=linuxppc-dev@ozlabs.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.