On Thu, Apr 13, 2017 at 02:12:16PM +0200, Borislav Petkov wrote: > On Thu, Apr 13, 2017 at 01:31:59PM +0200, Borislav Petkov wrote: > > @@ -321,18 +321,8 @@ static void __print_mce(struct mce *m) > > > > static void print_mce(struct mce *m) > > { > > - int ret = 0; > > - > > __print_mce(m); > > - > > - /* > > - * Print out human-readable details about the MCE error, > > - * (if the CPU has an implementation for that) > > - */ > > - ret = atomic_notifier_call_chain(&x86_mce_decoder_chain, 0, m); > > - if (ret == NOTIFY_STOP) > > - return; > > - > > + mce_log(m); > > Actually, we don't need that call here because do_machine_check() > already does it.
Yes. Don't add mce_log(m) here. We've already done it. With this change: Acked-by: Tony Luck <tony.l...@intel.com> -Tony