Openembedded Core Discussions
 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox