[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20080930174117.2e2ef568.rdunlap@xenotime.net>
Date: Tue, 30 Sep 2008 17:41:17 -0700
From: Randy Dunlap <rdunlap@...otime.net>
To: "Yinghai Lu" <yinghai@...nel.org>
Cc: "H. Peter Anvin" <hpa@...or.com>,
"Randy Dunlap" <rdunlap@...otime.net>,
"Ingo Molnar" <mingo@...e.hu>,
"Thomas Gleixner" <tglx@...utronix.de>,
"Andrew Morton" <akpm@...ux-foundation.org>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/3] x86: mtrr_cleanup update command line
On Tue, 30 Sep 2008 16:59:49 -0700 Yinghai Lu wrote:
> On Tue, Sep 30, 2008 at 4:52 PM, Yinghai Lu <yinghai@...nel.org> wrote:
> > On Tue, Sep 30, 2008 at 4:41 PM, H. Peter Anvin <hpa@...or.com> wrote:
> >> Randy Dunlap wrote:
> >>>
> >>> E.g., the "apic" entries are not grouped together and these mtrr entries
> >>> should
> >>> not be grouped together unless they all begin with "mtrr", which is an
> >>> option here:
> >>> they could be renamed to "mtrr-cleanup" and "mtrr-nocleanup".
> >>> And "disable_mtrr_trim" could be renamed to "mtrr-notrim".
> >>>
> >>
> >> That does collide with the (not always kept) convention of prefixing "no" to
> >> disable a boolean option, though.
> >>
> >
> > it seems should group them and then provide one index section...
> >
> or
> 1. put all description in .c files
> 2. have one scripts to search early_param and __setup and create that
> kernel_parameter...
Here's a script that can be used as a starting point if someone wants to:
http://www.xenotime.net/linux/scripts/module-params
---
~Randy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists