Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756592AbXLNXyh (ORCPT ); Fri, 14 Dec 2007 18:54:37 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752302AbXLNXy1 (ORCPT ); Fri, 14 Dec 2007 18:54:27 -0500 Received: from smtp2.linux-foundation.org ([207.189.120.14]:54866 "EHLO smtp2.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752151AbXLNXy0 (ORCPT ); Fri, 14 Dec 2007 18:54:26 -0500 Message-ID: <476317A6.3060302@linux-foundation.org> Date: Fri, 14 Dec 2007 15:54:14 -0800 From: Stephen Hemminger Organization: Linux Foundation User-Agent: Thunderbird 2.0.0.9 (Windows/20071031) MIME-Version: 1.0 To: Andrew Morton CC: Jeff Garzik , netdev@vger.kernel.org, randy.dunlap@oracle.com, auke-jan.h.kok@intel.com, linux-kernel@vger.kernel.org Subject: Re: [patch 01/10] e1000e: make E1000E default to the same kconfig setting as E1000 References: <200712140002.lBE02pUb025505@imap1.linux-foundation.org> <4762E9FE.1070707@garzik.org> <20071214152215.55ef46e8.akpm@linux-foundation.org> In-Reply-To: <20071214152215.55ef46e8.akpm@linux-foundation.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2627 Lines: 63 Andrew Morton wrote: > On Fri, 14 Dec 2007 15:39:26 -0500 > Jeff Garzik wrote: > > >> akpm@linux-foundation.org wrote: >> >>> From: Randy Dunlap >>> >>> Make E1000E default to the same kconfig setting as E1000. So people's >>> machiens don't stop working when they use oldconfig. >>> >>> Signed-off-by: Randy Dunlap >>> Cc: Jeff Garzik >>> Cc: Auke Kok >>> Signed-off-by: Andrew Morton >>> --- >>> >>> drivers/net/Kconfig | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff -puN drivers/net/Kconfig~e1000e-make-e1000e-default-to-the-same-kconfig-setting-as-e1000 drivers/net/Kconfig >>> --- a/drivers/net/Kconfig~e1000e-make-e1000e-default-to-the-same-kconfig-setting-as-e1000 >>> +++ a/drivers/net/Kconfig >>> @@ -1986,6 +1986,7 @@ config E1000_DISABLE_PACKET_SPLIT >>> config E1000E >>> tristate "Intel(R) PRO/1000 PCI-Express Gigabit Ethernet support" >>> depends on PCI >>> + default E1000 >>> >> I am not inclined to apply this one. This practice, applied over time, >> will tend to accumulate weird 'default' and 'select' statements. >> >> So I think the breakage that occurs is mitigated by two factors: >> 1) kernel hackers that do their own configs are expected to be able to >> figure this stuff. >> 2) kernel builders (read: distros, mainly) are expected to have put >> thought into the Kconfig selection and driver migration strategies. >> >> PCI IDs move across drivers from time, and we don't want to apply these >> sorts changes: Viewed in the long term, the suggested patch is merely a >> temporary change to allow kernel experts to more easily deal with the >> PCI ID migration across drivers. >> >> I would prefer simply to communicate to kernel experts and builders >> about a Kconfig issue that could potentially their booting/networking... >> because this patch is only needed if the kernel experts do not already >> know about a necessary config update. >> > > You can take it out again later on - most people's .configs will then have > E1000E set. People who still do `cp ancientconfig .config ; make oldconfig' > remain screwed. > Sounds like something build system should help with. Some more user friendly syntax for dealing with issues of driver conversion. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/