linux-kernel - Re: [PATCH 4/4] x86, mce: Have MCE persistent event off by default for now

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 3 May 2011 09:23:02 +0200
From:	Borislav Petkov <bp@...64.org>
To:	Ingo Molnar <mingo@...e.hu>
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>,
	"Petkov, Borislav" <Borislav.Petkov@....com>
Subject: Re: [PATCH 4/4] x86, mce: Have MCE persistent event off by default
 for now

On Tue, May 03, 2011 at 02:45:05AM -0400, Ingo Molnar wrote:
> 
> * 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! :-)

Ok, the problem I see with it is that people without a RAS daemon
running will have the mechanism collecting MCEs in the background, using
up resources (4 pages per CPU is the buffer) and not doing anything (in
the best case that is, when we're not broken otherwise).

Do we want to have that on all x86 turned on by default? Or maybe invert
the flag semantics so that people can disable it on boot?

Thanks.

-- 
Regards/Gruss,
Boris.

Advanced Micro Devices GmbH
Einsteinring 24, 85609 Dornach
General Managers: Alberto Bozzo, Andrew Bowd
Registration: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632
--
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/