lkml.org 
[lkml]   [2003]   [Sep]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [2.6 patch] add a config option for -Os compilation
    On Sun, Sep 14, 2003 at 03:21:43PM +0200, Adrian Bunk wrote:
    > On Sun, Sep 14, 2003 at 01:33:49PM +0100, Russell King wrote:
    > > On Sun, Sep 14, 2003 at 02:16:56PM +0200, Adrian Bunk wrote:
    > > > The patch below adds a config option OPTIMIZE_FOR_SIZE for telling gcc
    > > > to use -Os instead of -O2. Besides this, it removes constructs on
    > > > architectures that had a -Os hardcoded in their Makefiles.
    > >
    > > I'd rather retain the -Os default for ARM please. (The init/Kconfig
    > > defaults it to 'n' for everything.)
    >
    > Below is the patch with the ARM part omitted.

    But it doesn't make sense - you include a generic configuration option
    which people will see, yet it makes no effect on ARM - seems to be rather
    silly putting it there in the first place.

    Also, do users particularly care what -Os and -O2 mean?

    Maybe you need to change init/Kconfig to be something like the following,
    and reinstate the change to the ARM makefile:

    config OPTIMIZE_FOR_SIZE
    bool "Optimize for size" if EXPERIMENTAL
    default n if !ARM
    default y if ARM
    help
    Enabling this option will cause the compiler to reduce the code
    size of the kernel by disabling certain optimisations. However,
    the resulting kernel may run faster due to more efficient
    cache utilisation.

    If unsure, say N.

    --
    Russell King (rmk@arm.linux.org.uk) http://www.arm.linux.org.uk/personal/
    Linux kernel maintainer of:
    2.6 ARM Linux - http://www.arm.linux.org.uk/
    2.6 PCMCIA - http://pcmcia.arm.linux.org.uk/
    2.6 Serial core
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2005-03-22 13:48    [W:4.232 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site