Re: Improved INCLUDE_CONFIG_FILE

From: Wojciech A. Koszek <wkoszek_at_FreeBSD.ORG>
Date: Mon, 26 Mar 2007 17:10:16 +0000
On Mon, Mar 26, 2007 at 03:47:06PM +0200, Oliver Fromme wrote:
> Wojciech A. Koszek wrote:
>  > Alex Kozlov wrote:
>  > > Wojciech A. Koszek wrote:
>  > > > Current implementation of INCLUDE_CONFIG_FILE option has number of
>  > > > issues. Including it in MAC or SMP configurations will bring only text
>  > > > of this single file into the kernel file. We're not able to see
>  > > > configuration of running ("live") kernel, which could be helpful while
>  > > > tracking users' reports. You can't get easy to use file format, ready
>  > > > for configuration process.
>  > > > 
>  > > > In my Perforce wkoszek_kconftxt branch:
>  > > > 
>  > > >   //depot/user/wkoszek/wkoszek_kconftxt/...
>  > > > 
>  > > > I brought some modifications to existing config(8) and added system
>  > > > interface that would let us to see configuration of running kernel
>  > > > (currently -- via kern.conftxt sysctl), as well as other kernel file
>  > > > through config(8)'s -k option.
>  > > 
>  > > By the way, any plan to include INCLUDE_CONFIG_FILE in GENERIC?
>  > 
>  > I'd like to have this enabled by default, and I know there should be no
>  > strong objections.
> 
> No objection from me, but please fix it so include files
> are also included in the kernel.
> 
> Many of my kernel config files have only few lines (e.g.
> "options SMP" and "include MYKERNEL").  Currently, the
> INCLUDE_CONFIG_FILE feature only includes those two lines,
> rendering that feature useless (and even potentially
> dangerous if someone relies on his config to be in the
> kernel).

You'll get full configuration file, no matter what you include. After
redirecting it to file, you'll be able to compile the kernel with no
additional modifications.

-- 
Wojciech A. Koszek
wkoszek_at_FreeBSD.org
http://FreeBSD.czest.pl/dunstan/
Received on Mon Mar 26 2007 - 14:39:43 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:39:07 UTC