public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Mike Frysinger <vapier@gentoo.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2] x86: Wrap small helper functions from libgcc to avoid an ABI mismatch
Date: Wed, 9 Nov 2011 00:34:41 -0500	[thread overview]
Message-ID: <201111090034.42368.vapier@gentoo.org> (raw)
In-Reply-To: <CALButCJ2AeTFW0+LpOBxcALwOU2x6AhYkuELQ+SZcfVzZMr2Jw@mail.gmail.com>

On Tuesday 08 November 2011 22:57:31 Graeme Russ wrote:
> On Wed, Nov 9, 2011 at 3:49 PM, Mike Frysinger wrote:
> > On Tuesday 08 November 2011 18:14:46 Graeme Russ wrote:
> >> Ah, yes you have - Can you give me a list? I think to be pragmatic we
> >> should either wrap the all or go down the private libgcc path like ARM
> >> has done. Private lib functions would elliminate the overhead, but is
> >> this really such a problem anyway?
> > 
> > it then becomes a sync issue ... updates to gcc's libgcc aren't reflected
> > in u- boot automatically ...
> 
> Are those updates needed?

you mean bug/math fixes and optimizations ?  i would think so.

> We already have a fair chunk of libc which is not automatically sync'd

we have very very little of glibc ... really only a few optimized 
string/memory funcs, and those aren't glibc specific.  the only other C lib 
type stuff is taken from Linux, or zlib, or dlmalloc, or other locations that 
aren't possible to link against.  libgcc is a bit unique in this regard.

> and going by what is in arch/arm/lib,
> there are very few libgcc functions (far less than libc) and each of
> those are relatively trivial and unlikely to require updating.

it depends on the architecture.  if the core ISA doesn't change, then the math 
funcs won't change much.  i'm not terribly familiar with the gcc x86 internals 
to say how much we actually rely on libgcc.a considering most of the fun stuff 
is real hardware insns.  unlike the normal embedded risc arches which need to 
implement more complicated funcs with many insns.

> Also, I already know of issues compiling U-Boot on an x64 OS because
> of the 32/64 bit incompatibility of libgcc. I never encountered this
> because I only have a 32-bit build machine

yes, this would be an issue, although most people on x86-64 have multilib 
toolchains, so it'd work anyways.  maybe the x86 config.mk should be 
automatically adding -m32 when available.

> So the handful of libgcc functions are starting to become a
> disproportionate headache

seems fairly low to me, but i'm not the x86 maintainer.  i'm not seeing these 
funcs in Linux' arch/x86/ tree though ... maybe there's a better solution ?
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20111109/21da441d/attachment.pgp 

  reply	other threads:[~2011-11-09  5:34 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-08  9:27 [U-Boot] [PATCH] [x86] Wrap small helper functions from libgcc to avoid an ABI mismatch Gabe Black
2011-11-08 13:33 ` Mike Frysinger
2011-11-08 22:27   ` Gabe Black
2011-11-08 22:34 ` [U-Boot] [PATCH v2] x86: " Gabe Black
2011-11-08 23:14   ` Graeme Russ
2011-11-09  4:49     ` Mike Frysinger
2011-11-09  3:57       ` Graeme Russ
2011-11-09  5:34         ` Mike Frysinger [this message]
2011-11-17  9:01           ` [U-Boot] [PATCH v3] " Gabe Black
2011-11-17  9:13             ` Gabe Black
2011-11-30 11:03             ` Graeme Russ
2011-11-09  3:05   ` [U-Boot] [PATCH v2] " Graeme Russ
2011-11-09 10:32   ` [U-Boot] [RFC] x86: Do no use reparm as it break libgcc linkage Graeme Russ
2011-11-09 17:12     ` Mike Frysinger
2011-11-09 21:42       ` Graeme Russ
2011-11-10  4:13         ` Mike Frysinger
2011-11-10  4:22           ` Graeme Russ
2011-11-10  5:10             ` Graeme Russ
2011-11-10 17:15             ` Mike Frysinger
2011-11-10 22:53               ` Graeme Russ
2011-11-11  0:23                 ` Mike Frysinger
2011-11-11  1:23                   ` Graeme Russ
2011-11-11  1:40                     ` Mike Frysinger
2011-11-11  1:51                       ` Graeme Russ
2011-11-11  1:55                         ` Mike Frysinger
2011-11-11  1:59                           ` Graeme Russ
2011-11-11  2:10                             ` Gabe Black
2011-11-11  2:22                               ` Graeme Russ
2011-11-11  2:41                                 ` Gabe Black
2011-11-11  4:49                                   ` Graeme Russ
2011-11-11  5:04                                     ` Mike Frysinger
2011-11-11  5:16                                       ` Graeme Russ
2011-11-11 16:24                                         ` Mike Frysinger
2011-11-11  2:44                             ` Mike Frysinger
2011-11-11 19:59                           ` Scott Wood
2011-11-16 23:00     ` Graeme Russ

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=201111090034.42368.vapier@gentoo.org \
    --to=vapier@gentoo.org \
    --cc=u-boot@lists.denx.de \
    /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