netdev - Re: [patch 12/12] Configure out ethtool support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20080730.145727.07367670.davem@davemloft.net>
Date:	Wed, 30 Jul 2008 14:57:27 -0700 (PDT)
From:	David Miller <davem@...emloft.net>
To:	akpm@...ux-foundation.org
Cc:	thomas.petazzoni@...e-electrons.com, shemminger@...tta.com,
	jeff@...zik.org, netdev@...r.kernel.org, mpm@...enic.com
Subject: Re: [patch 12/12] Configure out ethtool support

From: Andrew Morton <akpm@...ux-foundation.org>
Date: Wed, 30 Jul 2008 14:48:12 -0700

> But I do think we should see some evidence that people are actually
> using CONFIG_ETHTOOL=n in real setups.

Heaven help them if they want to debug any real problems that
might involve checksumming, TSO, and other offload features.
Heaven help them if they want to probe the driver or firmware
version of the card they are using.

I'm sure you can find someone who runs with CONFIG_STAND_ON_MY_HEAD=y
but that doesn't mean that supporting such an option upstream makes
any sense.

We spent years getting rid of the per-driver module parameters to
configure these things.  And now folks want to remove the common
infrastructure we created to do it.

6K is peanuts compared to the per-driver code that implements these
facilities.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html