From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Garzik Subject: Re: [PATCH 0/2] atl1: Introduce CONFIG_ATL1_EXPERIMENTAL Date: Fri, 07 Sep 2007 19:54:37 -0400 Message-ID: <46E1E4BD.4000004@garzik.org> References: <46E1E1AC.5030801@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: atl1-devel@lists.sourceforge.net, Jay Cliburn , netdev@vger.kernel.org To: Chris Snook Return-path: Received: from srv5.dvmed.net ([207.36.208.214]:40577 "EHLO mail.dvmed.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751715AbXIGXyl (ORCPT ); Fri, 7 Sep 2007 19:54:41 -0400 In-Reply-To: <46E1E1AC.5030801@redhat.com> Sender: netdev-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Chris Snook wrote: > The atl1 driver is currently marked EXPERIMENTAL, because a few > supposedly performance-enhancing features still have problems. When > these features are disabled, the driver is completely stable, fully > functional, and performs well. > > Patch 1/2 Creates the kconfig option CONFIG_ATL1_EXPERIMENTAL, and > removes the EXPERIMENTAL designation from CONFIG_ATL1 > > Patch 2/2 Wraps some currently-disabled features in #ifdef > CONFIG_ATL1_EXPERIMENTAL, so developers and testers can play with these > features more easily, and distributions will still get a fast, stable > driver with existing .config files. > > We'll also be using this to wrap around various new features we'll be > experimenting with in coming months. Instead of using a half dozen > different kconfig options for each of them, like some drivers do, we'll > just use this, and make sure things are safe for everyone before we take > them out of the experimental wrapper. Well, I haven't received patch #2 yet, but in general a runtime switch (module option?) is greatly preferred. Jeff