[lkml]   [2006]   [Sep]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    SubjectRe: [2.6 patch] re-add -ffreestanding
    On Thu, Sep 07, 2006 at 12:37:48AM +0200, Adrian Bunk wrote:
    > On Wed, Aug 30, 2006 at 07:39:05PM +0100, Russell King wrote:
    > > Looking at the effect of -ffreestanding on ARM, it appears that on one
    > > hand, the overall image size is reduced by 0.016% but we end up with worse
    > > code - eg, strlen() of the same string in the same function evaluated
    > > multiple times vs once without -ffreestanding.
    > >
    > > The difference probably comes down to the lack of __attribute__((pure))
    > > on our string functions in linux/string.h.
    > >
    > > If we are going to go for -ffreestanding, we need to fix linux/string.h
    > > in that respect _first_.
    > We are talking about reverting the patch that removed -ffreestanding,
    > and that broke at least two architectures although it wrongly claimed
    > it would have been a safe patch.

    Wrong. Your patch unconditionally adds it for _ALL_ architectures.
    Below is the extract which you posted which supports this fact.

    For the elimination of any doubt, I do _NOT_ want this patch merged as
    is. Take that as the _third_ architecture maintainer who has NACK'd
    your patch (as you should've taken my first objection as that and
    apparantly didn't.)

    ... and maybe you should copy linux-arch with architecture-wide changes
    so that all architecture maintainers are aware of what you're trying to

    --- linux-2.6.18-rc4-mm3/Makefile.old 2006-08-30 16:59:31.000000000 +0200
    +++ linux-2.6.18-rc4-mm3/Makefile 2006-08-30 17:02:42.000000000 +0200
    @@ -308,7 +308,7 @@

    CFLAGS := -Wall -Wundef -Wstrict-prototypes -Wno-trigraphs \
    - -fno-strict-aliasing -fno-common
    + -fno-strict-aliasing -fno-common -ffreestanding

    # Read KERNELRELEASE from include/config/kernel.release (if it exists)
    Russell King
    Linux kernel 2.6 ARM Linux -
    maintainer of: 2.6 Serial core
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2006-09-07 08:33    [W:0.020 / U:14.360 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site