[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110503064505.GF7751@elte.hu>
Date: Tue, 3 May 2011 08:45:05 +0200
From: Ingo Molnar <mingo@...e.hu>
To: Borislav Petkov <bp@...64.org>
Cc: Peter Zijlstra <peterz@...radead.org>,
Arnaldo Carvalho de Melo <acme@...radead.org>,
Steven Rostedt <rostedt@...dmis.org>,
Frederic Weisbecker <fweisbec@...il.com>,
Tony Luck <tony.luck@...el.com>,
Mauro Carvalho Chehab <mchehab@...hat.com>,
EDAC devel <linux-edac@...r.kernel.org>,
LKML <linux-kernel@...r.kernel.org>,
Borislav Petkov <borislav.petkov@....com>
Subject: Re: [PATCH 4/4] x86, mce: Have MCE persistent event off by default
for now
* Borislav Petkov <bp@...64.org> wrote:
> From: Borislav Petkov <borislav.petkov@....com>
>
> This is new functionality and it affects all of x86 so we want to be
> very conservative about it and have it off by default for now, in case
> something goes awry. You can always enable it by supplying "ras" on the
> kernel command line.
>
> Also, depending on whether it is enabled or not, we emit the tracepoint
> from a different place in the code to pick up additional decoded info.
>
> Signed-off-by: Borislav Petkov <borislav.petkov@....com>
> ---
> Documentation/kernel-parameters.txt | 2 ++
> arch/x86/include/asm/mce.h | 1 +
> arch/x86/kernel/cpu/mcheck/mce.c | 32 ++++++++++++++++++++++++++++++--
> drivers/edac/mce_amd.c | 5 +++++
> 4 files changed, 38 insertions(+), 2 deletions(-)
the boot flag is fine - but please keep it enabled by default if MCE support is
enabled in the .config, we do not chicken out when it comes to testing new
code! :-)
Thanks,
Ingo
--
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