All of lore.kernel.org
 help / color / mirror / Atom feed
From: Scott Wood <scottwood@freescale.com>
To: Kumar Gala <galak@kernel.crashing.org>
Cc: linuxppc-dev@ozlabs.org
Subject: Re: [PATCH] [POWERPC] Fix kernel builds with newer gcc versions and -Os
Date: Fri, 02 May 2008 16:40:14 -0500	[thread overview]
Message-ID: <481B8A3E.3080200@freescale.com> (raw)
In-Reply-To: <BFB7B7A0-6EA9-467D-B23E-B0D46EF6BA54@kernel.crashing.org>

Kumar Gala wrote:
> 
> On May 2, 2008, at 12:34 PM, Segher Boessenkool wrote:
> 
>>>> <brokenrecord>
>>>> Why don't we just link with libgcc?
>>>> </brokenrecord>
>>>
>>> Its something of a PITA to do that in the kernel at this point since 
>>> we've duplicated libgcc functionality in it.  I'm sure there are some 
>>> historical reasons this wasn't done to start with.
>>
>> That's the same as saying that it would be a nice cleanup to remove all
>> that duplicated code now...
> 
> We'll hopefully this thread might spark either an explanation for why we 
> aren't just linking libgcc in a statement that says we should and we can 
> remove the code that implements libgcc functionality.
> 
> How would libgcc linking intermix with modules?  Would we have to 
> EXPORT_SYMBOL() all functions that libgcc implements? 

Yes, unfortunately.

A quick way to generate such a list would be to build a non-modular 
kernel and leave out the libgcc link (after removing the reimplemented 
functions), and see what linker errors you get.

 > I'm guessing that's varies w/different gcc versions.

Yeah, but so does the set of functions that Linux needs to reimplement.

-Scott

  reply	other threads:[~2008-05-02 21:39 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-02 14:21 [PATCH] [POWERPC] Fix kernel builds with newer gcc versions and -Os Kumar Gala
2008-05-02 15:07 ` Scott Wood
2008-05-02 15:26   ` Kumar Gala
2008-05-02 17:34     ` Segher Boessenkool
2008-05-02 21:34       ` Kumar Gala
2008-05-02 21:40         ` Scott Wood [this message]
2008-05-02 21:42         ` David Miller
2008-05-02 21:45           ` Scott Wood
2008-05-02 22:04             ` David Miller
2008-05-02 22:16               ` Scott Wood
2008-05-02 22:30                 ` David Miller
2008-05-02 22:38                   ` Scott Wood
2008-05-02 22:39                     ` David Miller
2008-05-03  0:15               ` Segher Boessenkool
2008-05-02 23:24           ` Benjamin Herrenschmidt
2008-05-02 23:23         ` Benjamin Herrenschmidt
2008-05-08  6:26         ` Paul Mackerras
2008-05-02 17:33 ` Segher Boessenkool
2008-05-02 21:31   ` Kumar Gala

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=481B8A3E.3080200@freescale.com \
    --to=scottwood@freescale.com \
    --cc=galak@kernel.crashing.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.