Add EDAC support on GHES
cover Add EDAC support on GHES
It turns that making ghes to talk with EDAC to not be a hard task.
This patch series makes ghes->edac integration.
This was compile-tested only, as its, for now, just a proof of concept.
There are lots of space for improvements, like:
- maybe split edac-ghes into a separate file;
- add a way to retrieve the DIMM label (not sure if APEI GHES interface supports it);
- detect the memory layout via ACPI;
Comments, sugestions etc are welcome.
Mauro Carvalho Chehab (3):
ghes: Be a good citzen with EDAC
edac: add support for raw error reports
ghes: add support for reporting errors via EDAC
drivers/acpi/apei/Kconfig | 1 + drivers/acpi/apei/Makefile | 2 + drivers/acpi/apei/ghes.c | 129 +++++++++++++++++++++++++++++++++++++++++++-- drivers/edac/edac_core.h | 17 ++++++ drivers/edac/edac_mc.c | 109 ++++++++++++++++++++++++++++---------- include/linux/edac.h | 5 ++ 6 files changed, 233 insertions(+), 30 deletions(-)
Signature · 1 line
1.7.11.7
RFC 1/3 ghes: Be a good citzen with EDAC
Register itself at EDAC MC core, in order to avoid other drivers to get it.
The edac core will warrant that just one driver will be used, so the first one to register will be the one that will be reporting the hardware errors.
--- drivers/acpi/apei/Kconfig | 1 + drivers/acpi/apei/Makefile | 2 ++ drivers/acpi/apei/ghes.c | 77 ++++++++++++++++++++++++++++++++++++++++++++++ include/linux/edac.h | 4 +++ 4 files changed, 84 insertions(+)
drivers/acpi/apei/Kconfig +1 -0
@@ -12,6 +12,7 @@ config ACPI_APEI config ACPI_APEI_GHES bool "APEI Generic Hardware Error Source" depends on ACPI_APEI && X86 + depends on !(EDAC_MM_EDAC=m) select ACPI_HED select IRQ_WORK select GENERIC_ALLOCATOR
drivers/acpi/apei/Makefile +2 -0
@@ -4,3 +4,5 @@ obj-$(CONFIG_ACPI_APEI_EINJ) += einj.o obj-$(CONFIG_ACPI_APEI_ERST_DEBUG) += erst-dbg.o apei-y := apei-base.o hest.o cper.o erst.o + +ccflags-y += -I$(srctree)/drivers/edac
drivers/acpi/apei/ghes.c +77 -0
107 lines not on this page · raw message
include/linux/edac.h +4 -0
@@ -375,6 +375,9 @@ enum scrub_type {* @EDAC_MC_LAYER_CHANNEL: memory layer is named "channel" * @EDAC_MC_LAYER_SLOT: memory layer is named "slot" * @EDAC_MC_LAYER_CHIP_SELECT: memory layer is named "chip select" +* @EDAC_MC_LAYER_UNKNOWN: memory layout is unknown. All memory is mapped +* as a single memory area. This is used when +* retrieving errors from apei/ghes driver. * * This enum is used by the drivers to tell edac_mc_sysfs what name should * be used when describing a memory stick location.
@@ -384,6 +387,7 @@ enum edac_mc_layer_type {EDAC_MC_LAYER_CHANNEL ,EDAC_MC_LAYER_SLOT ,EDAC_MC_LAYER_CHIP_SELECT , + EDAC_MC_LAYER_ALL_MEM, };/**
Signature · 1 line
1.7.11.7
RFC 2/3 edac: add support for raw error reports
In reply to Mauro Carvalho ChehabThat allows APEI GHES driver to report errors directly, using the EDAC error report API.
--- drivers/edac/edac_core.h | 17 ++++++++ drivers/edac/edac_mc.c | 109 +++++++++++++++++++++++++++++++++++------------ 2 files changed, 99 insertions(+), 27 deletions(-)
drivers/edac/edac_core.h +17 -0
@@ -453,6 +453,23 @@ extern struct mem_ctl_info *find_mci_by_dev(struct device *dev);extern struct mem_ctl_info *edac_mc_del_mc (struct device *dev);extern int edac_mc_find_csrow_by_page (struct mem_ctl_info *mci,unsigned long page); + +void edac_raw_mc_handle_error (const enum hw_event_mc_err_type type, +struct mem_ctl_info *mci, +long grain, +const u16 error_count, +const int top_layer, +const int mid_layer, +const int low_layer, +const unsigned long page_frame_number, +const unsigned long offset_in_page, +const unsigned long syndrome, +const char *msg, +const char *location, +const char *label, +const char *other_detail, +const bool enable_per_layer_report); + void edac_mc_handle_error(const enum hw_event_mc_err_type type, struct mem_ctl_info *mci, const u16 error_count,
drivers/edac/edac_mc.c +82 -27
132 lines not on this page · raw message
Signature · 1 line
1.7.11.7
RFC 3/3 ghes: add support for reporting errors via EDAC
In reply to Mauro Carvalho Chehab--- drivers/acpi/apei/ghes.c | 52 +++++++++++++++++++++++++++++++++++++++++++++--- include/linux/edac.h | 1 + 2 files changed, 50 insertions(+), 3 deletions(-)
drivers/acpi/apei/ghes.c +49 -3
@@ -115,6 +115,7 @@ struct ghes {struct ghes_estatus_node {struct llist_node llnode;struct acpi_hest_generic *generic; +struct ghes *ghes; };struct ghes_estatus_cache {
@@ -457,7 +458,49 @@ static void ghes_clear_estatus(struct ghes *ghes) ghes->flags &= ~GHES_TO_CLEAR ; } -static void ghes_do_proc (const struct acpi_hest_generic_status *estatus) +static void ghes_edac_report_mem_error (struct ghes *ghes,int sev, +struct cper_sec_mem_err *mem_err) +{ +#ifdef CONFIG_EDAC_MM_EDAC +enum hw_event_mc_err_type type; +unsigned long page =0 , offset =0 , grain =0 ; +char location[80 ]; +char *label ="unknown" ; + +if (mem_err->validation_bits &CPER_MEM_VALID_PHYSICAL_ADDRESS ) { + page = mem_err->physical_addr >>PAGE_SHIFT ; + offset = mem_err->physical_addr & ~PAGE_MASK ; + grain = ~(mem_err->physical_addr_mask & ~PAGE_MASK ); + } + +switch (sev) { +case GHES_SEV_CORRECTED : + type =HW_EVENT_ERR_CORRECTED ; +break ; +case GHES_SEV_RECOVERABLE : + type =HW_EVENT_ERR_UNCORRECTED ; +break ; +case GHES_SEV_PANIC : + type =HW_EVENT_ERR_FATAL ; +break ; +default : +case GHES_SEV_NO : + type =HW_EVENT_ERR_INFO ; + } + +sprintf (location,"node:%d card:%d module:%d bank:%d device:%d row: %d column:%d bit_pos:%d" , + mem_err->node, mem_err->card, mem_err->module, + mem_err->bank, mem_err->device, mem_err->row, mem_err->column, + mem_err->bit_pos); + +edac_raw_mc_handle_error (type, ghes->mci, grain,1 ,0 ,0 ,0 , + page, offset,0 , +"APEI" , location, label,"" ,0 ); +#endif +} + +static void ghes_do_proc (struct ghes *ghes, +const struct acpi_hest_generic_status *estatus) {int sev, sec_sev;struct acpi_hest_generic_data *gdata;
@@ -469,6 +512,8 @@ static void ghes_do_proc(const struct acpi_hest_generic_status *estatus) CPER_SEC_PLATFORM_MEM)) {struct cper_sec_mem_err *mem_err; mem_err = (struct cper_sec_mem_err *)(gdata+1 ); +ghes_edac_report_mem_error (ghes, sev, mem_err); +#ifdef CONFIG_X86_MCE apei_mce_report_mem_error (sev ==GHES_SEV_CORRECTED , mem_err);
@@ -687,7 +732,7 @@ static int ghes_proc(struct ghes *ghes)if (ghes_print_estatus (NULL , ghes->generic, ghes->estatus))ghes_estatus_cache_add (ghes->generic, ghes->estatus); } -ghes_do_proc (ghes->estatus); +ghes_do_proc (ghes, ghes->estatus); out:ghes_clear_estatus (ghes);return 0 ;
@@ -780,7 +825,7 @@ static void ghes_proc_in_irq(struct irq_work *irq_work) estatus =GHES_ESTATUS_FROM_NODE (estatus_node); len =apei_estatus_len (estatus); node_len =GHES_ESTATUS_NODE_LEN (len); -ghes_do_proc (estatus); +ghes_do_proc (estatus_node->ghes, estatus);if (!ghes_estatus_cached (estatus)) { generic = estatus_node->generic;if (ghes_print_estatus (NULL , generic, estatus))
@@ -869,6 +914,7 @@ static int ghes_notify_nmi(unsigned int cmd, struct pt_regs *regs) estatus_node = (void *)gen_pool_alloc (ghes_estatus_pool, node_len);if (estatus_node) { + estatus_node->ghes = ghes; estatus_node->generic = ghes->generic; estatus =GHES_ESTATUS_FROM_NODE (estatus_node);memcpy (estatus, ghes->estatus, len);
include/linux/edac.h +1 -0
@@ -100,6 +100,7 @@ enum hw_event_mc_err_type {HW_EVENT_ERR_CORRECTED ,HW_EVENT_ERR_UNCORRECTED ,HW_EVENT_ERR_FATAL , + HW_EVENT_ERR_INFO, };/**
Signature · 1 line
1.7.11.7