Re: [PATCH] amd64_edac: Add support for newer F16h models

2014-02-27 Thread Borislav Petkov
On Thu, Feb 20, 2014 at 10:28:46AM -0600, Aravind Gopalakrishnan wrote: > Extending ECC decoding support for F16h M30h. > > Tested on F16h M30h with ECC turned on using mce_amd_inj module and > the patch works fine. > > Signed-off-by: Aravind Gopalakrishnan > Tested-by: Arindam Nath Added hpa'

Re: [PATCH] amd64_edac: Add support for newer F16h models

2014-02-27 Thread Borislav Petkov
On Thu, Feb 20, 2014 at 10:28:46AM -0600, Aravind Gopalakrishnan wrote: > Extending ECC decoding support for F16h M30h. > > Tested on F16h M30h with ECC turned on using mce_amd_inj module and > the patch works fine. > > Signed-off-by: Aravind Gopalakrishnan > Tested-by: Arindam Nath Acked-by:

Re: [PATCH] amd64_edac: Add support for newer F16h models

2014-02-27 Thread Borislav Petkov
On Thu, Feb 27, 2014 at 10:22:27AM -0600, Aravind Gopalakrishnan wrote: > Ok, here's more info: we will need to add support in k10temp for this > model; and support is not required in fam15h_power as register bits > used by the driver code are unavailable for OS. But F{3,4} usage is > split: k10tem

Re: [PATCH] amd64_edac: Add support for newer F16h models

2014-02-27 Thread Aravind Gopalakrishnan
On 2/27/2014 10:31 AM, Borislav Petkov wrote: On Thu, Feb 27, 2014 at 10:22:27AM -0600, Aravind Gopalakrishnan wrote: Ok, here's more info: we will need to add support in k10temp for this model; and support is not required in fam15h_power as register bits used by the driver code are unavailable

Re: [PATCH] amd64_edac: Add support for newer F16h models

2014-02-27 Thread Aravind Gopalakrishnan
On 2/24/2014 8:09 AM, Borislav Petkov wrote: On Fri, Feb 21, 2014 at 03:04:47PM -0600, Aravind Gopalakrishnan wrote: F{3,4} are used by hwmon/k10temp.c and hwmon/fam15h_power.c and they will need definitions to be places here in pci_ids.h But I'm not sure if the hwmon drivers need to carry suppo

Re: [PATCH] amd64_edac: Add support for newer F16h models

2014-02-24 Thread Borislav Petkov
On Fri, Feb 21, 2014 at 03:04:47PM -0600, Aravind Gopalakrishnan wrote: > F{3,4} are used by hwmon/k10temp.c and hwmon/fam15h_power.c and they > will need definitions to be places here in pci_ids.h But I'm not sure > if the hwmon drivers need to carry support for this model yet (will > verify that)

Re: [PATCH] amd64_edac: Add support for newer F16h models

2014-02-21 Thread Aravind Gopalakrishnan
On 2/21/2014 8:27 AM, Borislav Petkov wrote: On Thu, Feb 20, 2014 at 10:28:46AM -0600, Aravind Gopalakrishnan wrote: diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h index 97fbecd..e91ad27 100644 --- a/include/linux/pci_ids.h +++ b/include/linux/pci_ids.h @@ -528,6 +528,8 @@ #def

Re: [PATCH] amd64_edac: Add support for newer F16h models

2014-02-21 Thread Borislav Petkov
On Thu, Feb 20, 2014 at 10:28:46AM -0600, Aravind Gopalakrishnan wrote: > Extending ECC decoding support for F16h M30h. > > Tested on F16h M30h with ECC turned on using mce_amd_inj module and > the patch works fine. > > Signed-off-by: Aravind Gopalakrishnan > Tested-by: Arindam Nath > --- > ar

[PATCH] amd64_edac: Add support for newer F16h models

2014-02-20 Thread Aravind Gopalakrishnan
Extending ECC decoding support for F16h M30h. Tested on F16h M30h with ECC turned on using mce_amd_inj module and the patch works fine. Signed-off-by: Aravind Gopalakrishnan Tested-by: Arindam Nath --- arch/x86/kernel/amd_nb.c | 2 ++ drivers/edac/amd64_edac.c | 24