All of lore.kernel.org
 help / color / mirror / Atom feed
From: Saul Wold <sgw@linux.intel.com>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 0/7 v2] Cleanup and document tuning files
Date: Wed, 04 Apr 2012 09:59:58 -0700	[thread overview]
Message-ID: <4F7C7E0E.1000105@linux.intel.com> (raw)
In-Reply-To: <cover.1333481624.git.mark.hatle@windriver.com>

On 04/03/2012 01:07 PM, Mark Hatle wrote:
> (v2 only MIPS changed -- slight revision in the README to resolve comments
> from Phil Blundell)
>
> Also this fixes Yocto bugzilla # 2234.
>
> ----
>
> This is a documentation and cleanup of the tuning files.  Every tuning listed
> in AVAILTUNES was built for core-image-minimal!  The following is a list of
> failure conditions with explanations:
>
> ep9312 - binutils - gas fails to recognize -march=ep9312 as valid
>           (see binutils armv5e note below)
>
> cortexm3 - gcc reports the target CPU does not support ARM mode
> cortexr4 - gcc reports the target CPU does not support ARM mode
>
> mips64* - These are not yet supported by the OE compilers
>
> (SH is new and experimental, based on the existing tunings)
> sh3* - These are missing a patch in the compiler:
>         sh3-installfix-fixheaders.patch
>
> sh*eb - Big endian varieties are defined, but not supported
>
> x32 ABI variants for core2-64 and x86-64 - needs x32 layer
>
>
> Multilibs verified to work:
>
> x86-64&  x86
>
> ppce5500&  ppc64e5500
>
> ---
>
> Note armv5e&  binutils-gas:
>
> Finally there is a small patch to binutils gas that adds armv5e as a valid
> architecture, otherwise all of the armv5e* tunings fail, even though gcc
> says they are valid architectures.
>
> ---
>
> Future work:
>
> I'm still not completely happy with the ARM tunings, but it's too disruptive
> to change them at this time.  I'd like to move to something closer to how
> MIPS is configured.  Define various "variants" for the various tunings,
> instead of simply defining the ARMPKGARCH.  The current method prevents us
> from having a single build that can build packages optimized for armv5te and
> armv7a.  While this is not likely to be a huge problem for the time being,
> it may lead to problems for feed based systems that can support multiple
> ARM architecture families.
>
> The following changes since commit 38ee88e3b32b7444d7f1eb64f1b4f69a48fe0458:
>
>    gdb: build with expat, add missing RRECOMMENDS_gdbserver (2012-04-03 14:10:44 +0100)
>
> are available in the git repository at:
>    git://git.pokylinux.org/poky-contrib mhatle/tunings
>    http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=mhatle/tunings
>
> Mark Hatle (7):
>    conf/machine/include/README: Add readme to explain cpu tunings
>    conf/machine/include: Cleanup IA tunings to match README
>    conf/machine/include: Cleanup MIPS tunings to match README
>    conf/machine/include: Cleanup PowerPC tunings to match README
>    conf/machine/include: Cleanup ARM tunings to match README
>    conf/machine/include: Update SH tunings to match README
>    binutils: Inform binutils that armv5e really is valid!
>
>   meta/conf/bitbake.conf                             |    1 -
>   meta/conf/machine/include/README                   |   77 ++++++++++++++++++++
>   meta/conf/machine/include/arm/README               |   40 ++++++++++
>   meta/conf/machine/include/arm/arch-arm.inc         |    2 +-
>   meta/conf/machine/include/arm/arch-armv5.inc       |    2 +-
>   meta/conf/machine/include/arm/arch-armv7a.inc      |    6 +-
>   meta/conf/machine/include/ia32/README              |   24 ++++++
>   meta/conf/machine/include/ia32/arch-ia32.inc       |   26 ++++---
>   meta/conf/machine/include/mips/README              |   36 +++++++++
>   meta/conf/machine/include/mips/arch-mips.inc       |   38 +++++++++-
>   meta/conf/machine/include/powerpc/README           |   17 +++++
>   meta/conf/machine/include/powerpc/arch-powerpc.inc |   15 ++--
>   .../machine/include/powerpc/arch-powerpc64.inc     |    1 +
>   meta/conf/machine/include/sh/README                |   11 +++
>   meta/conf/machine/include/sh/arch-sh.inc           |    9 +++
>   meta/conf/machine/include/tune-c3.inc              |   12 +++-
>   meta/conf/machine/include/tune-core2.inc           |   16 +++--
>   meta/conf/machine/include/tune-cortexm3.inc        |    2 +-
>   meta/conf/machine/include/tune-cortexr4.inc        |    2 +-
>   meta/conf/machine/include/tune-i586.inc            |    8 +--
>   meta/conf/machine/include/tune-mips32.inc          |   17 +++--
>   meta/conf/machine/include/tune-ppc603e.inc         |    4 +-
>   meta/conf/machine/include/tune-ppce300c2.inc       |    2 +-
>   meta/conf/machine/include/tune-ppce500.inc         |   10 ++-
>   meta/conf/machine/include/tune-ppce500mc.inc       |    4 +-
>   meta/conf/machine/include/tune-ppce500v2.inc       |   10 ++-
>   meta/conf/machine/include/tune-ppce5500.inc        |    4 +-
>   meta/conf/machine/include/tune-sh3.inc             |   19 ++++-
>   meta/conf/machine/include/tune-sh4.inc             |   32 ++++++++-
>   .../binutils/binutils/binutils-armv5e.patch        |   20 +++++
>   meta/recipes-devtools/binutils/binutils_2.22.bb    |    3 +-
>   31 files changed, 401 insertions(+), 69 deletions(-)
>   create mode 100644 meta/conf/machine/include/README
>   create mode 100644 meta/conf/machine/include/arm/README
>   create mode 100644 meta/conf/machine/include/ia32/README
>   create mode 100644 meta/conf/machine/include/mips/README
>   create mode 100644 meta/conf/machine/include/powerpc/README
>   create mode 100644 meta/conf/machine/include/sh/README
>   create mode 100644 meta/conf/machine/include/sh/arch-sh.inc
>   create mode 100644 meta/recipes-devtools/binutils/binutils/binutils-armv5e.patch
>
Merged into OE-Core

Thanks
	Sau!

>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>



      parent reply	other threads:[~2012-04-04 17:09 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-03 19:47 [PATCH 0/7] Cleanup and document tuning files Mark Hatle
2012-04-03 19:47 ` [PATCH 1/7] conf/machine/include/README: Add readme to explain cpu tunings Mark Hatle
2012-04-04  0:40   ` Chris Larson
2012-04-04  1:58   ` Otavio Salvador
2012-04-03 19:47 ` [PATCH 2/7] conf/machine/include: Cleanup IA tunings to match README Mark Hatle
2012-04-03 19:47 ` [PATCH 3/7] conf/machine/include: Cleanup MIPS " Mark Hatle
2012-04-03 19:51   ` Phil Blundell
2012-04-03 19:57     ` Mark Hatle
2012-04-04 22:10   ` Andreas Oberritter
2012-04-05  4:17     ` Khem Raj
2012-04-06 17:33       ` Mark Hatle
2012-04-06 21:30         ` Khem Raj
2012-04-07  0:10           ` Mark Hatle
2012-04-08 21:34             ` Andreas Oberritter
2012-04-09 15:17               ` Mark Hatle
2012-04-09 15:56                 ` Koen Kooi
2012-04-09 16:03                   ` Mark Hatle
2012-04-09 20:06                 ` Andreas Oberritter
2012-04-09 20:25                   ` Mark Hatle
2012-04-09 20:51                     ` Andreas Oberritter
2012-04-09 21:00                     ` Mark Hatle
2012-04-09 21:03                     ` Phil Blundell
2012-04-09 21:21                       ` ARM tunings was " Mark Hatle
2012-04-09 21:30                         ` Phil Blundell
2012-04-09 21:44                           ` Mark Hatle
2012-04-10  9:23                             ` Phil Blundell
2012-04-10 17:39                               ` Mark Hatle
2012-04-10 19:33                                 ` Phil Blundell
2012-04-09 22:19                     ` Khem Raj
2012-04-03 19:47 ` [PATCH 4/7] conf/machine/include: Cleanup PowerPC " Mark Hatle
2012-04-04 18:02   ` Matthew McClintock
2012-04-04 19:57     ` Mark Hatle
2012-04-04 18:03   ` Matthew McClintock
2012-04-04 19:59     ` Mark Hatle
2012-04-03 19:47 ` [PATCH 5/7] conf/machine/include: Cleanup ARM " Mark Hatle
2012-04-03 19:47 ` [PATCH 6/7] conf/machine/include: Update SH " Mark Hatle
2012-04-03 19:47 ` [PATCH 7/7] binutils: Inform binutils that armv5e really is valid! Mark Hatle
2012-04-07  8:03   ` Khem Raj
2012-04-04 16:59 ` Saul Wold [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=4F7C7E0E.1000105@linux.intel.com \
    --to=sgw@linux.intel.com \
    --cc=openembedded-core@lists.openembedded.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.