Messages in this thread | | | From | "Ghannam, Yazen" <> | Subject | RE: [PATCH 1/2] Revert "x86/mce/AMD: Collect error info even if valid bits are not set" | Date | Tue, 27 Mar 2018 15:59:37 +0000 |
| |
> -----Original Message----- > From: linux-edac-owner@vger.kernel.org <linux-edac- > owner@vger.kernel.org> On Behalf Of Ghannam, Yazen > Sent: Tuesday, March 27, 2018 10:02 AM > To: Borislav Petkov <bp@alien8.de> > Cc: linux-edac@vger.kernel.org; linux-kernel@vger.kernel.org; > tony.luck@intel.com; x86@kernel.org > Subject: RE: [PATCH 1/2] Revert "x86/mce/AMD: Collect error info even if > valid bits are not set" > > > -----Original Message----- > > From: Borislav Petkov <bp@alien8.de> > > Sent: Monday, March 26, 2018 4:08 PM > > To: Ghannam, Yazen <Yazen.Ghannam@amd.com> > > Cc: linux-edac@vger.kernel.org; linux-kernel@vger.kernel.org; > > tony.luck@intel.com; x86@kernel.org > > Subject: Re: [PATCH 1/2] Revert "x86/mce/AMD: Collect error info even if > > valid bits are not set" > > > > On Mon, Mar 26, 2018 at 07:58:51PM +0000, Ghannam, Yazen wrote: > > > So at a minimum, we should always save and report as much as we can. > > > > Only on Zen or all AMD families? > > > > I'll confirm with the HW folks. I understand it as a change in philosophy > rather than a change in hardware. >
So this recommendation could apply to all families, but it's okay if we just apply this behavior to SMCA systems. That way we don't need to worry about changing things on legacy systems.
I'll write a new patch that abstracts the register reads and applies the different behaviors.
In any case, this patch should be reverted since faking the valid bits will cause the downstream code in the notifier blocks to process errors they shouldn't.
Thanks, Yazen
| |