On Monday 14 April 2008 06:54, David Miller wrote: > > 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@xxxxxxxxxxxxxx> > > Applied to net-2.6.26, thanks Denys. Thanks Dave. -- vda -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
