linux-kernel - Re: [tip:perfcounters/core] perf_counter: x86: Fix call-chain support to use NMI-safe methods

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:	Mon, 15 Jun 2009 20:45:36 +0200
From:	Ingo Molnar <mingo@...e.hu>
To:	"H. Peter Anvin" <hpa@...or.com>
Cc:	Mathieu Desnoyers <mathieu.desnoyers@...ymtl.ca>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	mingo@...hat.com, paulus@...ba.org, acme@...hat.com,
	linux-kernel@...r.kernel.org, a.p.zijlstra@...llo.nl,
	penberg@...helsinki.fi, vegard.nossum@...il.com, efault@....de,
	jeremy@...p.org, npiggin@...e.de, tglx@...utronix.de,
	linux-tip-commits@...r.kernel.org
Subject: Re: [tip:perfcounters/core] perf_counter: x86: Fix call-chain
	support to use NMI-safe methods


* H. Peter Anvin <hpa@...or.com> wrote:

> Ingo Molnar wrote:
> > * Mathieu Desnoyers <mathieu.desnoyers@...ymtl.ca> wrote:
> > 
> >> Hrm, would it be possible to save the c2 register upon nmi handler 
> >> entry and restore it before iret instead ? This would ensure a 
> >> nmi-interrupted page fault handler would continue what it was 
> >> doing with a non-corrupted cr2 register after returning from nmi.
> >>
> >> Plus, this involves no modification to the page fault handler fast 
> >> path.
> > 
> > I guess this kind of nesting would work too - assuming the cr2 can 
> > be written to robustly.
> > 
> > And i suspect CPU makers pull off a few tricks to stage the cr2 info 
> > away from the page fault entry execution asynchronously, so i'd not 
> > be surprised if writing to it uncovered unknown-so-far side-effects 
> > in CPU implementations.
> > 
> 
> I wouldn't actually expect that, *as long as* there is 
> serialization between the cr2 write and the cr2 read.

Well, is there any OS that heavily relies on cr2 writes and which 
uses them from NMI context, and which CPU makers care about? 
(Meaning: Windows, pretty much.)

If not then i agree that in theory it should work fine, but in 
practice we only know that we dont know the unknown risk here ;-)

	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/