> So AFAINM, we want to do MCE work only after we've logged something to > the genpool. So we can do the much simplified thing below and kick the > workqueue from within mce_log() as everything that logs, calls that > function. > > Tony, any concerns?
@@ -156,7 +156,8 @@ void mce_log(struct mce *mce) /* Emit the trace record: */ trace_mce_record(mce); - atomic_notifier_call_chain(&x86_mce_decoder_chain, 0, mce); + mce_genpool_add(mce); + irq_work_queue(&mce_irq_work); Is it safe to call irq_work_queue() from MCE context? If that is OK, then I don't have any concerns. -Tony N�����r��y����b�X��ǧv�^�){.n�+����{����zX����ܨ}���Ơz�&j:+v�������zZ+��+zf���h���~����i���z��w���?�����&�)ߢf��^jǫy�m��@A�a��� 0��h���i