On Thu, Jul 10, 2014 at 11:44 AM, Borislav Petkov <b...@alien8.de> wrote: > if (atomic_dec_and_test(&mce_banks[i].poll)) > m.status = mce_rdmsrl(MSR_IA32_MCx_STATUS(i)); > > atomic_add_unless(&mce_banks[i].poll, 1, 1); > > so that you have only one CPU read the status register of mce_banks[i]. > > For non-shared banks, this will always work because no other CPU will > dec that variable anyway.
You don't know if the bank is shared or not. If it is not shared, then the above code might skip reading the bank because some other cpu is busy reading that bank number - and seeing its own private bank. -Tony -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
