From owner-freebsd-current@FreeBSD.ORG Mon May 7 23:08:00 2007 Return-Path: X-Original-To: freebsd-current@freebsd.org Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id B118A16A402 for ; Mon, 7 May 2007 23:08:00 +0000 (UTC) (envelope-from dunstan@freebsd.czest.pl) Received: from freebsd.czest.pl (freebsd.czest.pl [80.48.250.4]) by mx1.freebsd.org (Postfix) with ESMTP id 1CB2E13C43E for ; Mon, 7 May 2007 23:07:59 +0000 (UTC) (envelope-from dunstan@freebsd.czest.pl) Received: from freebsd.czest.pl (freebsd.czest.pl [80.48.250.4]) by freebsd.czest.pl (8.13.4/8.12.9) with ESMTP id l47NgelS002436 for ; Mon, 7 May 2007 23:42:40 GMT (envelope-from dunstan@freebsd.czest.pl) Received: (from dunstan@localhost) by freebsd.czest.pl (8.13.4/8.12.9/Submit) id l47NgehH002435 for freebsd-current@freebsd.org; Mon, 7 May 2007 23:42:40 GMT (envelope-from dunstan) Date: Mon, 7 May 2007 23:42:39 +0000 From: "Wojciech A. Koszek" To: freebsd-current@freebsd.org Message-ID: <20070507234239.GA2343@FreeBSD.czest.pl> Mail-Followup-To: "Wojciech A. Koszek" , freebsd-current@freebsd.org References: <20070506164247.GA77786@FreeBSD.czest.pl> <200705072054.l47KsJwg044996@lurza.secnetix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-2 Content-Disposition: inline In-Reply-To: <200705072054.l47KsJwg044996@lurza.secnetix.de> User-Agent: Mutt/1.4.2.1i X-Greylist: Sender DNS name whitelisted, not delayed by milter-greylist-2.0.2 (freebsd.czest.pl [80.48.250.4]); Mon, 07 May 2007 23:42:40 +0000 (UTC) Subject: Re: INCLUDE_CONFIG_FILE patches X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 07 May 2007 23:08:00 -0000 On Mon, May 07, 2007 at 10:54:19PM +0200, Oliver Fromme wrote: > Wojciech A. Koszek wrote: > > We talked about improved INCLUDE_CONFIG_FILE work some time ago. I > > cleaned it up, and I have prepared a patch for the latest -CURRENT. > > > > It's here: > > > > http://people.freebsd.org/~wkoszek/patches/kernconf.patch > > > > If you have any suggestions, please let me know as I'd really like to > > see this patch into the tree before RELENG_7. > > I'm currently unable to test the patch (lack of a working > 7-current machine), but I've had a closer look at the patch > anyway. Basically it looks very good. Thanks, > I just have one problem with it. It seems that it doesn't > preserve comments and the exact formatting of the config > file, right? (Please correct me if I'm wrong.) If that's > true, that would be a regression. The current behaviour > (i.e. with "strings ... | sed ...") reproduces the exact > configuration file, with comments and everything. This patch addresses a problem: "I used to have a proper kernel configuration file, everything seemed to work just fine, but someone accidently deleted it from my server's disk" or "What did this strange guy put into that kernel image to make it work correctly" and similar cases people described earlier. Noone mentioned about a comments issue. We agreed it's a GENERIC-like feature, and even with optimized config(8), people complained about memory problems. Adding comments will make memory consumption even bigger. I'd like to stay away from that. How to handle comments in the included files? For "MAC" you get twice as much text, as every file has a rather long description in it. This problem isn't present in the current implemention, as we don't take the "include" statements upon consideration. Once you know what had been included into your configuration file before accident, you can find respective comment into src/sys//conf directory. I don't think comments are really all that important. -- Wojciech A. Koszek wkoszek@FreeBSD.org http://FreeBSD.czest.pl/dunstan/