From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754549AbYI3XkY (ORCPT ); Tue, 30 Sep 2008 19:40:24 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754479AbYI3XjO (ORCPT ); Tue, 30 Sep 2008 19:39:14 -0400 Received: from xenotime.net ([66.160.160.81]:48413 "HELO xenotime.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1754472AbYI3XjN (ORCPT ); Tue, 30 Sep 2008 19:39:13 -0400 Date: Tue, 30 Sep 2008 16:57:39 -0700 From: Randy Dunlap To: Yinghai Lu Cc: Ingo Molnar , Thomas Gleixner , "H. Peter Anvin" , Andrew Morton , linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/3] x86: mtrr_cleanup update command line Message-Id: <20080930165739.7847be37.rdunlap@xenotime.net> In-Reply-To: <1222817380-29481-1-git-send-email-yhlu.kernel@gmail.com> References: <1222817380-29481-1-git-send-email-yhlu.kernel@gmail.com> Organization: YPO4 X-Mailer: Sylpheed 2.5.0 (GTK+ 2.12.0; x86_64-unknown-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 30 Sep 2008 16:29:38 -0700 Yinghai Lu wrote: > change enable_mtrr_cleanup to mtrr-cleanup, disable_mtrr_cleanup to nomtrr-cleanup. > > Signed-off-by: Yinghai Lu > > --- > Documentation/kernel-parameters.txt | 4 ++-- > arch/x86/Kconfig | 2 +- > arch/x86/kernel/cpu/mtrr/main.c | 4 ++-- > 3 files changed, 5 insertions(+), 5 deletions(-) > > Index: linux-2.6/Documentation/kernel-parameters.txt > =================================================================== > --- linux-2.6.orig/Documentation/kernel-parameters.txt > +++ linux-2.6/Documentation/kernel-parameters.txt > @@ -614,8 +614,8 @@ and is between 256 and 4096 characters. > See drivers/char/README.epca and > Documentation/digiepca.txt. > > - disable_mtrr_cleanup [X86] > - enable_mtrr_cleanup [X86] > + nomtrr-cleanup [X86] > + mtrr-cleanup [X86] > The kernel tries to adjust MTRR layout from continuous > to discrete, to make X server driver able to add WB > entry later. This parameter enables/disables that. Looks like Documentation/kernel-parameters.txt needs a comment that says that entries are supposed to be listed in alphabetical order, not grouped by . Please don't add them like this. 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". > Index: linux-2.6/arch/x86/Kconfig > =================================================================== > --- linux-2.6.orig/arch/x86/Kconfig > +++ linux-2.6/arch/x86/Kconfig > @@ -1250,7 +1250,7 @@ config MTRR_SANITIZER > Convert MTRR layout from continuous to discrete, so X drivers can > add writeback entries. > > - Can be disabled with disable_mtrr_cleanup on the kernel command line. > + Can be disabled with nomtrr-cleanup on the kernel command line. > The largest mtrr entry size for a continous block can be set with > mtrr_chunk_size. > > Index: linux-2.6/arch/x86/kernel/cpu/mtrr/main.c > =================================================================== > --- linux-2.6.orig/arch/x86/kernel/cpu/mtrr/main.c > +++ linux-2.6/arch/x86/kernel/cpu/mtrr/main.c > @@ -826,7 +826,7 @@ static int __init disable_mtrr_cleanup_s > enable_mtrr_cleanup = 0; > return 0; > } > -early_param("disable_mtrr_cleanup", disable_mtrr_cleanup_setup); > +early_param("nomtrr-cleanup", disable_mtrr_cleanup_setup); > > static int __init enable_mtrr_cleanup_setup(char *str) > { > @@ -834,7 +834,7 @@ static int __init enable_mtrr_cleanup_se > enable_mtrr_cleanup = 1; > return 0; > } > -early_param("enable_mtrr_cleanup", enable_mtrr_cleanup_setup); > +early_param("mtrr-cleanup", enable_mtrr_cleanup_setup); > > static int __init mtrr_cleanup_debug_setup(char *str) > { --- ~Randy