[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20140506100217.GA27774@gmail.com>
Date: Tue, 6 May 2014 12:02:18 +0200
From: Ingo Molnar <mingo@...nel.org>
To: Jiri Kosina <jkosina@...e.cz>
Cc: Steven Rostedt <rostedt@...dmis.org>,
"H. Peter Anvin" <hpa@...ux.intel.com>,
Linus Torvalds <torvalds@...ux-foundation.org>,
linux-kernel@...r.kernel.org, x86@...nel.org,
Salman Qazi <sqazi@...gle.com>, Ingo Molnar <mingo@...e.hu>,
Michal Hocko <mhocko@...e.cz>, Borislav Petkov <bp@...en8.de>,
Vojtech Pavlik <vojtech@...e.cz>,
Petr Tesarik <ptesarik@...e.cz>, Petr Mladek <pmladek@...e.cz>
Subject: Re: 64bit x86: NMI nesting still buggy?
* Jiri Kosina <jkosina@...e.cz> wrote:
> On Tue, 29 Apr 2014, Steven Rostedt wrote:
>
> > > According to 38.4 of [1], when SMM mode is entered while the CPU is
> > > handling NMI, the end result might be that upon exit from SMM, NMIs will
> > > be re-enabled and latched NMI delivered as nested [2].
> >
> > Note, if this were true, then the x86_64 hardware would be extremely
> > buggy. That's because NMIs are not made to be nested. If SMM's come in
> > during an NMI and re-enables the NMI, then *all* software would break.
> > That would basically make NMIs useless.
> >
> > The only time I've ever witness problems (and I stress NMIs all the
> > time), is when the NMI itself does a fault. Which my patch set handles
> > properly.
>
> Yes, it indeed does.
>
> In the scenario I have outlined, the race window is extremely small,
> plus NMIs don't happen that often, plus SMIs don't happen that
> often, plus (hopefully) many BIOSes don't enable NMIs upon SMM exit.
Note, the "NMIs don't happen that often" condition is pretty rare on
x86 Linux systems. These days anyone doing a 'perf top', 'perf record'
or running a profiling tool like SysProf will generate tens of
thousands of NMIs, per second. Systems with profiling active are
literally bathed in NMIs, and that is how we found the page fault NMI
bug.
So I'd say any race condition hypothesis assuming "NMIs are rare" is
probably invalid on modern Linux systems.
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