Prev: lib: vsprintf: optimised put_dec() for 32-bit machines
Next: [PATCH RFC] [X86] Fix potential issue on memmove
From: Robert Richter on 13 Aug 2010 04:30
On 13.08.10 00:37:48, Frederic Weisbecker wrote:

> >
> > ... but this will not work. You have to mark the *absolute* nmi number
> > here. If you only raise a flag, the next unknown nmi will be dropped,
> > every.
>
>
>
> Isn't it what we want? Only the next unknown nmi gets dropped.
>
>
>
>
> > Because, in between there could have been other nmis that
> > stopped the chain and thus the 'unknown' path is not executed.
>
>
>
> I'm not sure what you mean here. Are you thinking about a third
> NMI source that triggers while we are still handling the first
> NMI in the back to back sequence?
>
>
>
> > The trick in my patch is that you *know*, which nmi you want to skip.
>
>
> Well with the flag you also know which nmi you want to skip.

We cannot assume that all cpus have the same behavior here. Imagine a
cpu that handles 2 counters and *does not* trigger a back-to-back
nmi. With flags only implemented, the next unknown nmi will be dropped
anyway, no matter when it fires. We have to check the nmi sequence.

The next thing you have to be aware is, a registered nmi handler is
not called with every nmi. If there was another nmi source and a
handler with higher priority was returning a stop, when all other
subsequent handlers are not called. Esp. 'unknown nmi' is called only
in rare cases. So, a handler might not get noticed of an nmi. This
means, if a handler gets called a 2nd time, it must not necessarily
the next (2nd) nmi.

-Robert

--
Advanced Micro Devices, Inc.
Operating System Research Center

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
First  |  Prev  | 
Pages: 1 2 3 4
Prev: lib: vsprintf: optimised put_dec() for 32-bit machines
Next: [PATCH RFC] [X86] Fix potential issue on memmove