From: "H. Peter Anvin" <hpa@zytor.com>
To: Yinghai Lu <yinghai@kernel.org>
Cc: "Ingo Molnar" <mingo@elte.hu>,
"\"J.A. Magallón\"" <jamagallon@ono.com>,
Linux-Kernel <linux-kernel@vger.kernel.org>
Subject: Re: Typo bug [Re: Linux 2.6.27-rc8]
Date: Tue, 30 Sep 2008 14:37:02 -0700 [thread overview]
Message-ID: <48E29BFE.4040309@zytor.com> (raw)
In-Reply-To: <86802c440809301435r79fb2553p65130b6451e83fa4@mail.gmail.com>
Yinghai Lu wrote:
> On Tue, Sep 30, 2008 at 12:59 PM, H. Peter Anvin <hpa@zytor.com> wrote:
>> Yinghai Lu wrote:
>>> On Tue, Sep 30, 2008 at 11:47 AM, H. Peter Anvin <hpa@zytor.com> 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.
-hpa
next prev parent reply other threads:[~2008-09-30 21:37 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-09-29 22:39 Linux 2.6.27-rc8 Linus Torvalds
2008-09-29 23:09 ` david
2008-09-29 23:33 ` Jiri Kosina
2008-09-30 1:56 ` Linus Torvalds
2008-09-30 1:59 ` Dave Airlie
2008-09-30 2:06 ` Arjan van de Ven
2008-09-30 2:23 ` Linus Torvalds
2008-09-30 2:24 ` Linus Torvalds
2008-09-30 12:06 ` Alan Cox
2008-09-30 3:42 ` Brandeburg, Jesse
2008-09-30 12:05 ` Alan Cox
2008-09-30 2:21 ` Linus Torvalds
2008-09-30 2:39 ` Dave Airlie
2008-09-30 3:19 ` Arjan van de Ven
2008-09-30 7:11 ` Jiri Kosina
2008-09-30 7:58 ` Eric Piel
2008-09-30 16:28 ` Luiz Fernando N. Capitulino
2008-09-30 18:27 ` Herton Ronaldo Krzesinski
2008-09-30 2:30 ` Brandeburg, Jesse
2008-09-30 22:07 ` Thomas Gleixner
2008-09-30 7:06 ` Jiri Kosina
2008-09-30 14:09 ` Krzysztof Halasa
2008-09-30 14:11 ` Jiri Kosina
2008-09-30 15:48 ` Allan, Bruce W
2008-10-01 15:37 ` Jiri Kosina
2008-09-30 7:43 ` Typo bug [Re: Linux 2.6.27-rc8] J.A. Magallón
2008-09-30 7:55 ` Ingo Molnar
2008-09-30 8:02 ` J.A. Magallón
2008-09-30 8:05 ` Ingo Molnar
2008-10-01 21:33 ` Domenico Andreoli
2008-10-02 5:27 ` Willy Tarreau
2008-10-02 9:26 ` Ingo Molnar
2008-10-02 9:45 ` Domenico Andreoli
2008-09-30 18:47 ` H. Peter Anvin
2008-09-30 19:30 ` Yinghai Lu
2008-09-30 19:59 ` H. Peter Anvin
2008-09-30 21:35 ` Yinghai Lu
2008-09-30 21:37 ` H. Peter Anvin [this message]
2008-09-30 21:42 ` Yinghai Lu
2008-09-30 22:02 ` H. Peter Anvin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=48E29BFE.4040309@zytor.com \
--to=hpa@zytor.com \
--cc=jamagallon@ono.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=yinghai@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.