RE: [PATCH] x86/mce: Always save severity in machine_check_poll

2017-06-21 Thread Ghannam, Yazen
> -Original Message- > From: Borislav Petkov [mailto:b...@alien8.de] > Sent: Monday, June 19, 2017 12:49 PM > To: Ghannam, Yazen > Cc: linux-e...@vger.kernel.org; Tony Luck ; > x...@kernel.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH] x86/mce: Al

Re: [PATCH] x86/mce: Always save severity in machine_check_poll

2017-06-19 Thread Borislav Petkov
On Fri, Jun 16, 2017 at 02:49:58PM +, Ghannam, Yazen wrote: > The code block being removed here was added in the following commit to decide > whether or not to schedule work. > > fa92c58 x86, mce: Support memory error recovery for both UCNA and Deferred > error in machine_check_poll Are you

RE: [PATCH] x86/mce: Always save severity in machine_check_poll

2017-06-16 Thread Ghannam, Yazen
nel.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH] x86/mce: Always save severity in machine_check_poll > > On Mon, Jun 12, 2017 at 11:54:06AM -0500, Yazen Ghannam wrote: > > From: Yazen Ghannam > > > > Remove code that was used to decide whether to schedule work. The &

Re: [PATCH] x86/mce: Always save severity in machine_check_poll

2017-06-14 Thread Borislav Petkov
On Mon, Jun 12, 2017 at 11:54:06AM -0500, Yazen Ghannam wrote: > From: Yazen Ghannam > > Remove code that was used to decide whether to schedule work. The decision ??? I'm missing a *lot* of background in order to understand what that sentence means. -- Regards/Gruss, Boris. Good mailing

RE: [PATCH] x86/mce: Always save severity in machine_check_poll

2017-06-12 Thread Ghannam, Yazen
ernel@vger.kernel.org > Subject: Re: [PATCH] x86/mce: Always save severity in machine_check_poll > > On Mon, Jun 12, 2017 at 11:54:06AM -0500, Yazen Ghannam wrote: > > - severity = mce_severity(&m, mca_cfg.tolerant, NULL, false); > > - > &g

Re: [PATCH] x86/mce: Always save severity in machine_check_poll

2017-06-12 Thread Luck, Tony
On Mon, Jun 12, 2017 at 11:54:06AM -0500, Yazen Ghannam wrote: > - severity = mce_severity(&m, mca_cfg.tolerant, NULL, false); > - > - if (severity == MCE_DEFERRED_SEVERITY && > mce_is_memory_error(&m)) > - if (m.status & MCI_STATUS_ADDRV) > -