[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20090615184536.GH11248@elte.hu>
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/
Powered by blists - more mailing lists