On Thu, Feb 23, 2006 at 11:50:41AM +0100, J?rn Engel wrote:
> 
> For my kernel, there would be 92 removals if the condition at the
> price of 135 bytes of extra object code.  Some of the removals would
> be in modules, so the numbers are not exactly fair.

IMHO source saving is cheap while binary bloat isn't.

> Another interesting question is: Why is kfree_skb inline in the first
> place?  After uninlining it, my patch would debloat both source and
> object code by a bit:
> 
> -rwxr-xr-x  1 joern src   4824435 Feb 23 11:46 vmlinux
> 
> 12157 bytes gained.  Plus a bit more when the 92 conditionals are
> removed.

Now there's a good idea.  After all, the great majority of callers
of kfree_skb expect to free the skb.  Dave, what do you think?

Cheers,
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <[EMAIL PROTECTED]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to