From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753845AbYI3Vmk (ORCPT ); Tue, 30 Sep 2008 17:42:40 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752599AbYI3Vmc (ORCPT ); Tue, 30 Sep 2008 17:42:32 -0400 Received: from rv-out-0506.google.com ([209.85.198.225]:33892 "EHLO rv-out-0506.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752552AbYI3Vmb (ORCPT ); Tue, 30 Sep 2008 17:42:31 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:sender:to:subject:cc:in-reply-to:mime-version :content-type:content-transfer-encoding:content-disposition :references:x-google-sender-auth; b=tp/qSVcmizAPaCkCSO88teIOQgZ3gOUrFAz2DDhDYpLx+zv4nQzT3vj21+3z/VeCHv drp//s2WzLkv8Tbj6N8xau0vmEgK6GUieovI02oFVq17Z+YPZbQj3fQ5Syeh6ki0yi7B V/STXRa+p+9wMDYOVMfDzhq3eJGtdmgC27BLc= Message-ID: <86802c440809301442l4d80ea8fqaa20db772cf14df5@mail.gmail.com> Date: Tue, 30 Sep 2008 14:42:30 -0700 From: "Yinghai Lu" To: "H. Peter Anvin" Subject: Re: Typo bug [Re: Linux 2.6.27-rc8] Cc: "Ingo Molnar" , "=?ISO-8859-1?Q?\"J.A._Magall=F3n\"?=" , Linux-Kernel In-Reply-To: <48E29BFE.4040309@zytor.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <20080930094346.307ef1e5@werewolf.home> <20080930075553.GB20241@elte.hu> <48E27425.7010705@zytor.com> <86802c440809301230s7dd33e69m83dab51f915580b1@mail.gmail.com> <48E28528.5070605@zytor.com> <86802c440809301435r79fb2553p65130b6451e83fa4@mail.gmail.com> <48E29BFE.4040309@zytor.com> X-Google-Sender-Auth: 1ac3027e40c5b1b9 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Sep 30, 2008 at 2:37 PM, H. Peter Anvin wrote: > Yinghai Lu wrote: >> >> On Tue, Sep 30, 2008 at 12:59 PM, H. Peter Anvin wrote: >>> >>> Yinghai Lu wrote: >>>> >>>> On Tue, Sep 30, 2008 at 11:47 AM, H. Peter Anvin wrote: >>>>> >>>>> Ingo Molnar wrote: >>>>>>> >>>>>>> early_param("enble_mtrr_cleanup", enable_mtrr_cleanup_setup); >>>>>>> ^^^^^^ >>>>>>> >>>>>>> Nice ;) >>>>>> >>>>>> heh. Could you send a patch with a changelog please? >>>>> >>>>> These options are also named inconsistently with all other options. >>>>> >>>>> The standard way to name an boolean option is "foo" versus "nofoo", in >>>>> this >>>>> case, "mtrrcleanup" vs "nomtrrcleanup". >>>>> >>>> ok, we could change it... >>> >>> If we're fixing a typo anyway I'd suggest so. We know we're not breaking >>> anyone's working setup... >> >> mtrr_cleanup and no_mtrr_cleanup? >> > > Dashes seem to be used more than underscores, so it probably should be > "mtrr-cleanup" and "nomtrr-cleanup" if you want a separator. > i need to document the mtrr_cleanup_debug too...change it to mtrrcleanup_debug ? just like initcall_debug? YH