public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: Detlef Vollmann <dv@vollmann.ch>
Cc: linux-arm-kernel@lists.infradead.org,
	Andrew Lunn <andrew@lunn.ch>,
	Russell King <linux@arm.linux.org.uk>,
	Jason Cooper <jason@lakedaemon.net>,
	Benjamin Cama <benoar@dolka.fr>,
	linux-kernel@vger.kernel.org,
	Gregory CLEMENT <gregory.clement@free-electrons.com>,
	"Evgeniy A. Dushistov" <dushistov@mail.ru>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
Subject: Re: [PATCH 0/5] ARM: orion5x/dove/mv78xx0 multiplatform
Date: Fri, 11 Dec 2015 00:10:55 +0100	[thread overview]
Message-ID: <14967875.ttNXUbSTvS@wuerfel> (raw)
In-Reply-To: <5669F5F8.5010005@vollmann.ch>

On Thursday 10 December 2015 23:00:24 Detlef Vollmann wrote:
> On 12/10/15 22:29, Arnd Bergmann wrote:
> > On Thursday 10 December 2015 22:14:25 Detlef Vollmann wrote:
> >> On 12/10/15 21:59, Arnd Bergmann wrote:
> > It may also be worth investigating what has made CONFIG_OF so costly,
> Probably because too much is done at runtime and too few things can
> be fixed at build time.
> 
> > maybe we can reduce this a bit again.
> Probably not without turning the wheel backward :-(
> 
> So for the test: yes it works, but I'm unhappy with it.

I'm not too happy about adding 80kb to the uncompressed kernel 
image either. I've spent some more time now trying to find where
we added the bloat. It's mainly in drivers, not in arch specific
code, a kilobyte here and there eventually adds up, but the largest
portion with a little over 50% of the total diff is drivers/of.

I don't expect to find a lot of low-hanging fruit there. For
reference, see the diff of 'size -A' below.

	Arnd

 
-orion5x_defconfig-dt/arch/arm/kernel/built-in.o  :
+orion5x_defconfig-nodt/arch/arm/kernel/built-in.o  :
--
-Total                                    48034
+Total                                    46863
 
 
-orion5x_defconfig-dt/arch/arm/mm/built-in.o  :
+orion5x_defconfig-nodt/arch/arm/mm/built-in.o  :
--
-Total                                       43680
+Total                                       43672
 
 
-orion5x_defconfig-dt/arch/arm/plat-orion/built-in.o  :
+orion5x_defconfig-nodt/arch/arm/plat-orion/built-in.o  :
--
-Total                                     22131
+Total                                     22103
 

-orion5x_defconfig-dt/drivers/ata/built-in.o  :
+orion5x_defconfig-nodt/drivers/ata/built-in.o  :


-Total                                                  160244
+Total                                                  159524
 
 
-orion5x_defconfig-dt/drivers/base/built-in.o  :
+orion5x_defconfig-nodt/drivers/base/built-in.o  :
--
-Total                                                      111366
+Total                                                      109090
 
 
-orion5x_defconfig-dt/drivers/bus/built-in.o  :
+orion5x_defconfig-nodt/drivers/bus/built-in.o  :
-Total                                        10151
+Total                                        8120
 
 
-orion5x_defconfig-dt/drivers/clk/built-in.o  :
+orion5x_defconfig-nodt/drivers/clk/built-in.o  :
--
-Total                                                  35514
+Total                                                  29469
 
 
-orion5x_defconfig-dt/drivers/crypto/built-in.o  :
+orion5x_defconfig-nodt/drivers/crypto/built-in.o  :
--
-Total             8712
+Total             8552
 
 
-orion5x_defconfig-dt/drivers/dma/built-in.o  :
+orion5x_defconfig-nodt/drivers/dma/built-in.o  :
--
-Total                                             18939
+Total                                             16186
 
 
-orion5x_defconfig-dt/drivers/gpio/built-in.o  :
+orion5x_defconfig-nodt/drivers/gpio/built-in.o  :
--
-Total                                               26663
+Total                                               19142
 
 
-orion5x_defconfig-dt/drivers/i2c/built-in.o  :
+orion5x_defconfig-nodt/drivers/i2c/built-in.o  :
--
-Total                                                  25772
+Total                                                  23778
 
 
-orion5x_defconfig-dt/drivers/input/built-in.o  :
+orion5x_defconfig-nodt/drivers/input/built-in.o  :
--
-Total                                          39847
+Total                                          38531
 
 
-orion5x_defconfig-dt/drivers/leds/built-in.o  :
+orion5x_defconfig-nodt/drivers/leds/built-in.o  :
--
-Total                                          10436
+Total                                          10372
 
 
-orion5x_defconfig-dt/drivers/misc/built-in.o  :
+orion5x_defconfig-nodt/drivers/misc/built-in.o  :
--
-Total             3163
+Total             1939
 
 
-orion5x_defconfig-dt/drivers/mtd/built-in.o  :
+orion5x_defconfig-nodt/drivers/mtd/built-in.o  :
--
-Total                                              162074
+Total                                              157383
 
 
-orion5x_defconfig-dt/drivers/net/built-in.o  :
+orion5x_defconfig-nodt/drivers/net/built-in.o  :
--
-Total                                    50038
+Total                                    46757
 
 
-orion5x_defconfig-dt/drivers/of/built-in.o  :
+orion5x_defconfig-nodt/drivers/of/built-in.o  :
--
-Total                                             46444
+Total                                             0
 
 
-orion5x_defconfig-dt/drivers/pci/built-in.o  :
+orion5x_defconfig-nodt/drivers/pci/built-in.o  :
--
-Total                                                   115926
+Total                                                   115352
 
 
-orion5x_defconfig-dt/drivers/perf/built-in.o  :
+orion5x_defconfig-nodt/drivers/perf/built-in.o  :
--
-Total                              4796
+Total                              4202
 
 
-orion5x_defconfig-dt/drivers/phy/built-in.o  :
+orion5x_defconfig-nodt/drivers/phy/built-in.o  :
--
-Total                                            5529
+Total                                            4909
 
 
-orion5x_defconfig-dt/drivers/rtc/built-in.o  :
+orion5x_defconfig-nodt/drivers/rtc/built-in.o  :
--
-Total                                       35840
+Total                                       35024
 
 
-orion5x_defconfig-dt/drivers/tty/built-in.o  :
+orion5x_defconfig-nodt/drivers/tty/built-in.o  :
--
-Total                                              125096
+Total                                              119738
 
 
-orion5x_defconfig-dt/drivers/usb/built-in.o  :
+orion5x_defconfig-nodt/drivers/usb/built-in.o  :
--
-Total                                                251268
+Total                                                250498
 


  parent reply	other threads:[~2015-12-10 23:11 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-25 15:49 [PATCH 0/5] ARM: orion5x/dove/mv78xx0 multiplatform Arnd Bergmann
2015-11-25 15:49 ` [PATCH 1/5] ARM: orion: move watchdog setup to mach-orion5x Arnd Bergmann
2015-11-25 15:49 ` [PATCH 2/5] ARM: orion: always use MULTI_IRQ_HANDLER Arnd Bergmann
2015-11-25 15:49 ` [PATCH 3/5] ARM: orion: use SPARSE_IRQ everywhere Arnd Bergmann
2015-11-25 15:49 ` [PATCH 4/5] ARM: orion: clean up mach/*.h headers Arnd Bergmann
2015-11-25 15:49 ` [PATCH 5/5] ARM: orion: multiplatform support Arnd Bergmann
2015-11-25 16:09 ` [PATCH 0/5] ARM: orion5x/dove/mv78xx0 multiplatform Andrew Lunn
2015-11-25 17:34   ` Gregory CLEMENT
2015-11-25 17:44     ` Andrew Lunn
2015-11-25 19:58       ` Arnd Bergmann
2015-11-26 14:50         ` Jason Cooper
2015-11-25 18:37   ` Russell King - ARM Linux
2015-11-25 19:16     ` Andrew Lunn
2015-11-25 19:29       ` Russell King - ARM Linux
2015-11-25 23:19       ` Russell King - ARM Linux
2015-11-25 20:05     ` Arnd Bergmann
2015-11-25 22:47       ` Russell King - ARM Linux
2015-11-25 20:14 ` Andrew Lunn
2015-11-30 13:37 ` Gregory CLEMENT
2015-12-01 22:51   ` Andrew Lunn
2015-12-01 22:55     ` Arnd Bergmann
2015-12-02 11:03     ` Gregory CLEMENT
2015-12-02 12:33   ` Russell King - ARM Linux
2015-12-02 15:49     ` Arnd Bergmann
2015-12-02 19:28       ` Russell King - ARM Linux
2015-12-02 19:37         ` Russell King - ARM Linux
2015-12-02 19:54           ` Arnd Bergmann
2015-12-02 19:51         ` Arnd Bergmann
2015-12-02 20:22           ` Russell King - ARM Linux
2015-12-02 20:38             ` Andrew Lunn
2015-12-02 21:03               ` Russell King - ARM Linux
2015-12-02 21:15               ` Arnd Bergmann
2015-12-02 23:59                 ` Andrew Lunn
     [not found]   ` <87wpsz61r7.fsf@free-electrons.com>
     [not found]     ` <20151201182226.ea7d860eb3a35086981cd18f@vollmann.ch>
     [not found]       ` <87fuzl2jm5.fsf@free-electrons.com>
2015-12-02 14:55         ` Andrew Lunn
     [not found]     ` <56697665.1070609@vollmann.ch>
2015-12-10 13:18       ` Arnd Bergmann
2015-12-10 13:43         ` Arnd Bergmann
     [not found]           ` <5669CC76.4030606@vollmann.ch>
2015-12-10 20:59             ` Arnd Bergmann
     [not found]               ` <5669EB31.4050700@vollmann.ch>
2015-12-10 21:29                 ` Arnd Bergmann
     [not found]                   ` <5669F5F8.5010005@vollmann.ch>
2015-12-10 23:10                     ` Arnd Bergmann [this message]
2015-12-11 13:36                       ` Jason Cooper
2015-12-11 14:46                         ` Arnd Bergmann
2015-12-11 14:55                           ` Jason Cooper
2015-12-11 18:27                         ` Josh Triplett
2015-12-11 18:46                           ` Jason Cooper
2015-12-11 19:08                             ` Josh Triplett
2015-12-11 19:14                               ` Jason Cooper
     [not found]         ` <5669808F.2050909@vollmann.ch>
2015-12-11 21:33           ` Arnd Bergmann

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=14967875.ttNXUbSTvS@wuerfel \
    --to=arnd@arndb.de \
    --cc=andrew@lunn.ch \
    --cc=benoar@dolka.fr \
    --cc=dushistov@mail.ru \
    --cc=dv@vollmann.ch \
    --cc=gregory.clement@free-electrons.com \
    --cc=jason@lakedaemon.net \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=sebastian.hesselbarth@gmail.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