Broken on 386

Posted Mar 15, 2012 2:00 UTC (Thu) by slashdot (guest, #22014)
In reply to: Broken on 386 by slashdot
Parent article: The x86 NMI iret problem

More precisely, the LATCHED -> EXECUTING transition (in addition to the EXECUTING -> NOT_RUNNING one) is what you actually want and is done by the next assignment, so local_dec_and_test allows to remove the assignment and is just conceptually better than the cmpxchg, in addition to working on 386s.

to post comments

Broken on 386

Posted Mar 16, 2012 15:07 UTC (Fri) by nevets (subscriber, #11875) [Link]

Hmm, I didn't realize that old versions of i386 didn't implement cmpxchg. Not sure there's many of these processors out there that will be using newer kernels that will need these features, but none-the-less, it should be fixed.

I actually like your solution. The assignment is still needed, but only for the first instance of the NMI, but the label could be moved.

It's great that you brought this up on LWN, but it would be much better if you posted a patch :-)