From owner-cvs-all@FreeBSD.ORG Sun Oct 30 23:12:13 2005 Return-Path: X-Original-To: cvs-all@FreeBSD.org Delivered-To: cvs-all@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 1A65216A41F; Sun, 30 Oct 2005 23:12:13 +0000 (GMT) (envelope-from scottl@samsco.org) Received: from pooker.samsco.org (pooker.samsco.org [168.103.85.57]) by mx1.FreeBSD.org (Postfix) with ESMTP id 5EC6F43D48; Sun, 30 Oct 2005 23:12:09 +0000 (GMT) (envelope-from scottl@samsco.org) Received: from [192.168.254.11] (junior.samsco.home [192.168.254.11]) (authenticated bits=0) by pooker.samsco.org (8.13.4/8.13.4) with ESMTP id j9UNC7CX008901; Sun, 30 Oct 2005 16:12:08 -0700 (MST) (envelope-from scottl@samsco.org) Message-ID: <43655348.4000702@samsco.org> Date: Sun, 30 Oct 2005 16:12:08 -0700 From: Scott Long User-Agent: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.7.8) Gecko/20050615 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Nate Lawson References: <200510271734.j9RHYZAk015054@repoman.freebsd.org> <20051030062148.GA76667@dragon.NUXI.org> <20051030105647.GR99857@submonkey.net> <436509D5.3020401@root.org> In-Reply-To: <436509D5.3020401@root.org> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-1.4 required=3.8 tests=ALL_TRUSTED autolearn=failed version=3.1.0 X-Spam-Checker-Version: SpamAssassin 3.1.0 (2005-09-13) on pooker.samsco.org Cc: src-committers@FreeBSD.org, John Baldwin , cvs-src@FreeBSD.org, cvs-all@FreeBSD.org, David O'Brien , Ceri Davies Subject: Re: cvs commit: src/sys/i386/conf DEFAULTS GENERIC X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 30 Oct 2005 23:12:13 -0000 Nate Lawson wrote: > Ceri Davies wrote: > >> On Sat, Oct 29, 2005 at 11:21:48PM -0700, David O'Brien wrote: >> >>> On Thu, Oct 27, 2005 at 05:34:35PM +0000, John Baldwin wrote: >>> >>>> jhb 2005-10-27 17:34:35 UTC >>>> >>>> FreeBSD src repository >>>> >>>> Modified files: >>>> sys/i386/conf GENERIC Added files: >>>> sys/i386/conf DEFAULTS Log: >>>> Create a default kernel config for i386 and move 'device isa' and >>>> 'device npx' (both of which aren't really optional right now) and >>>> 'device io' and 'device mem' (to preserve POLA for 4.x users upgrading >>>> to 6.0) from GENERIC into DEFAULTS. >>> >>> >>> I may be missing something. I don't quite follow the benefit of the new >>> 'DEFAULTS' file. I'm also curious why we don't explicitly 'include' >>> DEFAULTS in GENERIC vs. the new automagic include feature. >> >> >> >> I was also confused by that. There are also no warnings if DEFAULTS is >> missing, which means that by removing a seemingly unrelated file I get a >> different (read: useless to me) kernel. Does INCLUDE_CONFIG_FILE show >> the contents of DEFAULTS as well? > > > A better way would be to make io and mem only compile time options > (default to on). There is no benefit to them being modules. Anyone who > wants to compile with NO_DEVIO or NO_DEVMEM will know why their kernel > stops working. > Using options was heavily debated, and I was the actually the proponent of it. Then it was pointed out that it would cause headaches for people moving their kernel configs from 5.x and 6-BETA, since effectively turning the code from a device to an option would mean that 'device io' would make config throw an error. There are things in a kernel config that are either non-optional or heavily recommended for novice users, so DEFAULTS provides an easy way to keep the foot shooting to a minimum. For the power users who are far too advanced for this, just delete the DEFAULTS file and go on your way. config will silently ignore it if it's not there. For those who are concerned about cvsup bringing it back automatically, we can look at adding a NODEFAULTS directive for your custom config files. Or, just use the 'nodevice' and 'nooption' directives to nullify what you don't like in DEFAULTS. Scott