ACK/cmt: [Artful/Bionic] [Config] enable EDAC_GHES for ARM64

Seth Forshee seth.forshee at canonical.com
Thu Feb 8 02:45:26 UTC 2018


On Wed, Feb 07, 2018 at 11:30:50AM -0600, Manoj Iyer wrote:
> Please consider this patch to enable CONFIG_EDAC_GHES option for ARM64 in 
> Artful and Bionic, so that the memory errors are processed and reported to 
> the user space. This fixes the bug 
> https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1747746
> 
> A test kernel is available in PPA: 
> https://launchpad.net/~centriq-team/+archive/ubuntu/lp1747746 
> and was tested by me on QTI ARM64 QDF2400 platform. I also tested this 
> on Power8 to check for any regressions. Test results are documented in the 
> bug report.

The reason this was disabled was due to LP #1714112. I think it's likely
that we can enable it for x86 too, provided that we also have
9a821fd210c0 ("EDAC, sb_edac: Don't create a second memory controller if
HA1 is not present"). The commit is already on
artful/master-next-backlog, all we really need is some testing to verify
that the commit does allow turning the option back on for the affected
users.

I don't object enabling it for arm64 in the mean time though, so
provided that we do the follow-up for x86:

Acked-by: Seth Forshee <seth.forshee at canonical.com>

For bionic I think we should just go ahead and enable it for all
architectures, so I've pushed a commit for those to do that and remove
the note for this option in the annotations file.




More information about the kernel-team mailing list