From: Adrian Bunk <adrian.bunk@movial.fi>
To: Tim Bird <tim.bird@am.sony.com>
Cc: linux-tiny <Linux-tiny@selenic.com>,
	linux-embedded <linux-embedded@vger.kernel.org>,
	linux kernel <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] console - Add configurable support for console charset translation
Date: Wed, 4 Jun 2008 13:33:53 +0300	[thread overview]
Message-ID: <20080604103353.GC27335@cs181133002.pp.htv.fi> (raw)
In-Reply-To: <48447615.5050806@am.sony.com>
On Mon, Jun 02, 2008 at 03:37:09PM -0700, Tim Bird wrote:
> With CONSOLE_TRANSLATIONS turned off, this saves about 6K
> on my kernel configured for an ARM development board (OMAP
> 5912 OSK).  In embedded products I'm familiar with,
> console translations are not needed.
>
> This was taken from the Linux-tiny project and updated slightly
> for 2.6.25.
>...
It seems to be always me who asks the controversial questions...:
Does the linux-tiny approach of adding a kconfig variable for each 5kB 
of code actually make sense? I'm asking since an exploding amount of 
kconfig variables and their interdependencies have a not so small 
maintainance impact in the long term.
And I'm wondering whether it's the best approach for reaching 
measurable results. E.g. my small patch that removed 
-maccumulate-outgoing-args on 32-bit x86 reduced the kernel size
there for the CONFIG_CC_OPTIMIZE_FOR_SIZE=y case by at about 2.5% (sic)
without adding any kconfig variables or #ifdef's.
Can you take a reasonable (best-case) .config for an existing device and 
get the following numbers:
- Ignoring patches that came from linux-tiny, by how many percent did 
  the size of the kernel increase or decrease between 2.6.16 and 2.6.26?
- By how many percent did the kernel size decrease due to patches from
  the linux-tiny project that added such kconfig options for a few kB 
  of code in the same timeframe?
My gut feeling is that the influence of this kind of linux-tiny patches 
is hardly noticably compared to the overall code size development, but 
if you have numbers that prove me wrong just point me to them and I'll 
stand corrected.
cu
Adrian
BTW: I'm not insisting on "between 2.6.16 and 2.6.26", that's just some
     timeframe big enough for showing general trends.
-- 
       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed
next prev parent reply	other threads:[~2008-06-04 10:33 UTC|newest]
Thread overview: 85+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-02 22:37 [PATCH] console - Add configurable support for console charset translation Tim Bird
2008-06-03  1:34 ` H. Peter Anvin
2008-06-03  7:00   ` Jamie Lokier
2008-06-03 15:46   ` Matt Mackall
2008-06-03 21:31     ` Rob Landley
2008-06-03 22:37       ` H. Peter Anvin
2008-06-04  2:56         ` Paul Mundt
2008-06-04 17:36           ` Rob Landley
2008-06-04 17:42             ` Bill Gatliff
2008-06-04 18:55               ` Rob Landley
2008-06-04 19:05                 ` Bill Gatliff
2008-06-05  0:46             ` Paul Mundt
2008-06-05 23:35               ` Rob Landley
2008-06-07  3:35                 ` H. Peter Anvin
2008-06-09 11:38                   ` Jamie Lokier
2008-06-09 11:51                     ` Geert Uytterhoeven
2008-06-04 17:34         ` Rob Landley
2008-06-03  8:36 ` David Woodhouse
2008-06-03 13:45 ` David Woodhouse
2008-06-03 14:06   ` Holger Schurig
2008-06-03 14:10     ` David Woodhouse
2008-06-04  0:01   ` Tim Bird
2008-06-04  0:16     ` David Woodhouse
2008-06-04  1:03       ` Josh Boyer
2008-06-04  1:05         ` David Woodhouse
2008-06-04  3:13           ` Josh Boyer
2008-06-04  9:16             ` David Woodhouse
2008-06-04 10:07               ` Adrian Bunk
2008-06-04 10:10                 ` David Woodhouse
2008-06-04 15:35       ` Tim Bird
2008-06-04 11:55     ` Alan Cox
2008-06-04 13:55       ` David Woodhouse
2008-06-04 14:07         ` Alan Cox
2008-06-04 14:27           ` David Woodhouse
2008-06-03 21:18 ` Rob Landley
2008-06-03 21:26   ` David Woodhouse
2008-06-03 21:30   ` Matt Mackall
2008-06-04  7:02     ` Holger Schurig
2008-06-04  7:04     ` Holger Schurig
2008-06-04  7:36       ` Dave Hylands
2008-06-04 15:24         ` Randy Dunlap
2008-06-04 17:19       ` Rob Landley
2008-06-04 17:26         ` linux-embedded archives [was Re: [PATCH] console - Add configurable support for console charset translation] T Ziomek
2008-06-04 18:08           ` Rob Landley
2008-06-04 10:33 ` Adrian Bunk [this message]
2008-06-04 17:51   ` [PATCH] console - Add configurable support for console charset translation Rob Landley
2008-06-04 18:34   ` Bernhard Fischer
2008-06-04 19:01     ` Adrian Bunk
2008-06-04 19:21       ` Bernhard Fischer
2008-06-04 19:20         ` Alan Cox
2008-06-04 18:51   ` Tim Bird
2008-06-04 19:15     ` Sam Ravnborg
2008-06-04 19:23       ` Tim Bird
2008-06-04 20:23         ` Adrian Bunk
2008-06-04 20:42           ` Tim Bird
2008-06-05  6:55             ` Jörn Engel
2008-06-05  7:18           ` Uwe Klein
2008-06-04 20:24       ` Jörn Engel
2008-06-04 19:40     ` Adrian Bunk
2008-06-06 23:47       ` Tim Bird
2008-06-07  4:29         ` Rob Landley
2008-06-10  1:37           ` mainlining min-configs Tim Bird
2008-06-10  3:14             ` Ben Nizette
2008-06-10  4:16               ` Paul Mundt
2008-06-10  8:36             ` Adrian Bunk
2008-06-10 18:18               ` Tim Bird
2008-06-10 18:30                 ` Adrian Bunk
2008-06-10 18:51                   ` Sam Ravnborg
2008-06-10 19:05                     ` Adrian Bunk
2008-06-11  5:09                     ` Rob Landley
2008-06-11  6:39                       ` Sam Ravnborg
2008-06-11 19:09                         ` Tim Bird
2008-06-11 19:22                           ` Sam Ravnborg
2008-06-11 19:36                           ` Adrian Bunk
2008-06-11 19:46                             ` Tim Bird
2008-06-12  1:42                               ` Rob Landley
2008-06-11 19:48                             ` Sam Ravnborg
2008-06-12  0:01                           ` Rob Landley
2008-06-11  5:17                   ` Rob Landley
2008-06-11  5:51                 ` Rob Landley
2008-06-11  3:48               ` Rob Landley
2008-06-11  3:32             ` Rob Landley
2008-06-11  8:59             ` Christian MICHON
2008-06-04 19:42     ` [PATCH] console - Add configurable support for console charset translation Bernhard Fischer
2008-06-11  7:08   ` Holger Schurig
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=20080604103353.GC27335@cs181133002.pp.htv.fi \
    --to=adrian.bunk@movial.fi \
    --cc=Linux-tiny@selenic.com \
    --cc=linux-embedded@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tim.bird@am.sony.com \
    /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;
as well as URLs for NNTP newsgroup(s).