diff options
author | Robert Richter <rrichter@marvell.com> | 2020-01-23 09:03:00 +0000 |
---|---|---|
committer | Borislav Petkov <bp@suse.de> | 2020-02-17 13:10:14 +0100 |
commit | 65bb4d1af92cf007adc0a0c59dadcc393c5cada6 (patch) | |
tree | ee8826d490d3e435c3b0ff6790830b64f20cc79a /drivers/edac/ghes_edac.c | |
parent | 6334dc4e3ff53031a2d522b826c4fab92cfdea93 (diff) |
EDAC/mc: Report "unknown memory" on too many DIMM labels found
There is a limitation to report only EDAC_MAX_LABELS in e->label of
the error descriptor. This is to prevent a potential string overflow.
The current implementation falls back to "any memory" in this case and
also stops all further processing to find a unique row and channel of
the possible error location.
Reporting "any memory" is wrong as the memory controller reported an
error location for one of the layers. Instead, report "unknown memory"
and also do not break early in the loop to further check row and channel
for uniqueness.
[ bp: Massage commit message. ]
Signed-off-by: Robert Richter <rrichter@marvell.com>
Signed-off-by: Borislav Petkov <bp@suse.de>
Acked-by: Aristeu Rozanski <aris@redhat.com>
Link: https://lkml.kernel.org/r/20200123090210.26933-7-rrichter@marvell.com
Diffstat (limited to 'drivers/edac/ghes_edac.c')
0 files changed, 0 insertions, 0 deletions