Re: [PATCH 2/4] x86/mce/amd: Introduce deferred error interrupt handler

[Aravind Gopalakrishnan]

On 5/5/2015 3:28 PM, Luck, Tony wrote:
> Should you check whether the address is valid before blindly reading the register?
>
> >              m.bank = bank;
> 	if (m.status & MCI_STATUS_ADDRV)
>           		rdmsrl(MSR_IA32_MCx_ADDR(bank), m.addr);
> >              mce_log(&m);

Yes, missed that when I sent the snippet.
Fixed it:)

Thanks,
-Aravind.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/