On 2013/06/17 09:06AM, Borislav Petkov wrote: > On Mon, Jun 17, 2013 at 12:30:05PM +0530, Naveen N. Rao wrote: > > >Hmm, so if CMCI is not supported, you just disabled polling of this bank > > >and returned here. Not good. > > > > This is on purpose. If the bank doesn't support CMCI and we were > > polling it earlier, we want to disable that. The firmware should be > > notifying us of error events in either case. > > Not for corrected errors - for those we poll since the hardware doesn't > generate an exception for them. Yes, we used to poll since we do not get notified via MCE/CMCI. However, with firmware first set in CMC structure, the firmware is now controlling all corrected error reporting for these banks. So, we should not be looking at these MCA banks at all. > > Basically, the absence of CMCI support shouldn't change the old polling > behavior. Again, this changes with firmware first mode. The firmware should now notify us of all corrected errors through GHES. Thanks, Naveen > > Thanks. > > -- > Regards/Gruss, > Boris. > > Sent from a fat crate under my desk. Formatting is fine. > -- > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
