Messages in this thread | | | Date | Tue, 30 Sep 2008 17:35:46 -0700 | From | Randy Dunlap <> | 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@kernel.org> wrote: > > On Tue, Sep 30, 2008 at 4:41 PM, H. Peter Anvin <hpa@zytor.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...
Until that's done, please keep them in alpha order, like they (mostly) are now.
--- ~Randy
| |