[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <200804081221.42465.vda.linux@googlemail.com>
Date: Tue, 8 Apr 2008 12:21:42 +0200
From: Denys Vlasenko <vda.linux@...glemail.com>
To: Chas Williams <chas@....nrl.navy.mil>,
David Miller <davem@...emloft.net>
Cc: kernel-janitors@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: [PATCH] drivers/atm/horizon.c: stop inlining largish static functions
Hi David,
Can you also take this patch?
drivers/atm/horizon.c has unusually large number
of static inline functions - 36.
I looked through them. Most of them seems to be small enough,
but a few are big, others are using udelay or busy loop,
and as such are better not be inlined.
This patch removes "inline" from these static functions
(regardless of number of callsites - gcc nowadays auto-inlines
statics with one callsite).
Size difference for 32bit x86:
text data bss dec hex filename
8201 180 6 8387 20c3 linux-2.6-ALLYES/drivers/atm/horizon.o
7840 180 6 8026 1f5a linux-2.6.inline-ALLYES/drivers/atm/horizon.o
Signed-off-by: Denys Vlasenko <vda.linux@...glemail.com>
--
vda
View attachment "deinline_horizon.diff" of type "text/x-diff" (7748 bytes)
Powered by blists - more mailing lists