Re: [PATCH 3/5] x86/mcheck/AMD: Reduce number of blocks scanned per bank

From: Aravind Gopalakrishnan
Date: Thu Jan 14 2016 - 17:48:37 EST


On 1/14/2016 4:37 PM, Borislav Petkov wrote:
On Thu, Jan 14, 2016 at 04:05:38PM -0600, Aravind Gopalakrishnan wrote:
-#define NR_BLOCKS 9
+#define NR_BLOCKS 5
This doesn't look necessary to me. We do check MCi_MISC[BlkPtr] before
accessing that MSR.


True. But that BlkPtr logic also will undergo changes as it's interpretation for future processors is different.

We are guaranteed to have all the MISC registers (all 5 of them) going forward.
But we shouldn't be accessing MSRs beyond the 5th extended MISC register for each bank as that is the architectural boundary.
Hence the change here.

Thanks,
-Aravind.