Subject: Re: [patch 12/12] Configure out ethtool support

From: Stephen Hemminger <[email-masked]>
Date: Wed, 30 Jul 2008 13:37:27 -0700

On Wed, 30 Jul 2008 12:39:04 -0700
[email-masked] wrote:

> From: Thomas Petazzoni <[email-masked]>
> 
> Add the CONFIG_ETHTOOL option which allows to remove support for ethtool,
> not necessarly used on embedded systems.  As this is a size-reduction
> option, it depends on CONFIG_EMBEDDED.  It allows to save ~6 kilobytes of
> kernel code:
> 
>    text	   data	    bss	    dec	    hex	filename
> 1258447	 123592	 212992	1595031	 185697	vmlinux
> 1252147	 123592	 212992	1588731	 183dfb	vmlinux.new
>   -6300       0       0   -6300   -189C +/-
> 
> Question: should we also remove ethtool-related functions from all network
> drivers ?
> 
> This patch has been originally written by Matt Mackall
> <[email-masked]>, and is part of the Linux Tiny project.
> 
> Signed-off-by: Thomas Petazzoni <[email-masked]>
> Cc: Matt Mackall <[email-masked]>
> Cc: "David S. Miller" <[email-masked]>
> Cc: Jeff Garzik <[email-masked]>
> Signed-off-by: Andrew Morton <[email-masked]>

This option needs to be forced on when bridging or bonding is selected.