[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <86802c440804181507t4baf6c4dl1a68f89d3d7ce47c@mail.gmail.com>
Date: Fri, 18 Apr 2008 15:07:46 -0700
From: "Yinghai Lu" <yhlu.kernel@...il.com>
To: "Andrew Morton" <akpm@...ux-foundation.org>
Cc: "Yinghai Lu" <yhlu.kernel.send@...il.com>,
"Ingo Molnar" <mingo@...e.hu>,
"Linux Kernel Mailing List" <linux-kernel@...r.kernel.org>
Subject: Re: x86: memtest bootparam
On Fri, Apr 18, 2008 at 2:52 PM, Andrew Morton
<akpm@...ux-foundation.org> wrote:
> On Fri, 18 Apr 2008 17:17:44 GMT
> Linux Kernel Mailing List <linux-kernel@...r.kernel.org> wrote:
>
> > Gitweb: http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=c64df70793a9c344874eb4af19f85e0662d2d3ee
> > Commit: c64df70793a9c344874eb4af19f85e0662d2d3ee
> > Parent: 9b967106da0357ef8b08847dce35584a04134f20
> > Author: Yinghai Lu <yhlu.kernel.send@...il.com>
> > AuthorDate: Fri Mar 21 18:56:19 2008 -0700
> > Committer: Ingo Molnar <mingo@...e.hu>
> > CommitDate: Thu Apr 17 17:41:21 2008 +0200
> >
> > x86: memtest bootparam
> >
> > ...
> >
> >
> > + memtest= [KNL,X86_64] Enable memtest
> > + Format: <integer>
> > + range: 0,4 : pattern number
> > + default : 0 <disable>
>
> OK. Did the new memtest feature get documented anywhere, btw? I'm only
> vaguely aware of its existence.
need one text file in Documentation ?
>
> >
> > +config MEMTEST_BOOTPARAM
> > + bool "Memtest boot parameter"
> > + depends on X86_64
> > + default y
> > + help
> > + This option adds a kernel parameter 'memtest', which allows memtest
> > + to be disabled at boot. If this option is selected, memtest
> > + functionality can be disabled with memtest=0 on the kernel
> > + command line. The purpose of this option is to allow a single
> > + kernel image to be distributed with memtest built in, but not
> > + necessarily enabled.
> > +
> > + If you are unsure how to answer this question, answer Y.
> > +
> > +config MEMTEST_BOOTPARAM_VALUE
> > + int "Memtest boot parameter default value (0-4)"
> > + depends on MEMTEST_BOOTPARAM
> > + range 0 4
> > + default 0
> > + help
> > + This option sets the default value for the kernel parameter
> > + 'memtest', which allows memtest to be disabled at boot. If this
> > + option is set to 0 (zero), the memtest kernel parameter will
> > + default to 0, disabling memtest at bootup. If this option is
> > + set to 4, the memtest kernel parameter will default to 4,
> > + enabling memtest at bootup, and use that as pattern number.
> > +
> > + If you are unsure how to answer this question, answer 0.
>
> These seem pointless. Why not just unconditionally implement memtest=?
> All the code for that should be __init anyway so we're hardly saving
> anything here.
also
for big box, like 64g above, that will take a while.
YH
--
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