diff options
| author | Qiuxu Zhuo <[email protected]> | 2025-06-18 16:23:06 +0000 |
|---|---|---|
| committer | Borislav Petkov (AMD) <[email protected]> | 2025-06-18 18:19:45 +0000 |
| commit | 88efa0de3285be66969b71ec137d9dab1ee19e52 (patch) | |
| tree | a903885f049984cedc92ef07667efec931bad0aa /lib/crypto/mpi/mpi-cmp.c | |
| parent | EDAC/amd64: Correct number of UMCs for family 19h models 70h-7fh (diff) | |
| download | kernel-88efa0de3285be66969b71ec137d9dab1ee19e52.tar.gz kernel-88efa0de3285be66969b71ec137d9dab1ee19e52.zip | |
EDAC/igen6: Fix NULL pointer dereference
A kernel panic was reported with the following kernel log:
EDAC igen6: Expected 2 mcs, but only 1 detected.
BUG: unable to handle page fault for address: 000000000000d570
...
Hardware name: Notebook V54x_6x_TU/V54x_6x_TU, BIOS Dasharo (coreboot+UEFI) v0.9.0 07/17/2024
RIP: e030:ecclog_handler+0x7e/0xf0 [igen6_edac]
...
igen6_probe+0x2a0/0x343 [igen6_edac]
...
igen6_init+0xc5/0xff0 [igen6_edac]
...
This issue occurred because one memory controller was disabled by
the BIOS but the igen6_edac driver still checked all the memory
controllers, including this absent one, to identify the source of
the error. Accessing the null MMIO for the absent memory controller
resulted in the oops above.
Fix this issue by reverting the configuration structure to non-const
and updating the field 'res_cfg->num_imc' to reflect the number of
detected memory controllers.
Fixes: 20e190b1c1fd ("EDAC/igen6: Skip absent memory controllers")
Reported-by: Marek Marczykowski-Górecki <[email protected]>
Closes: https://lore.kernel.org/all/aFFN7RlXkaK_loQb@mail-itl/
Suggested-by: Borislav Petkov <[email protected]>
Signed-off-by: Qiuxu Zhuo <[email protected]>
Signed-off-by: Tony Luck <[email protected]>
Signed-off-by: Borislav Petkov (AMD) <[email protected]>
Tested-by: Marek Marczykowski-Górecki <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Diffstat (limited to 'lib/crypto/mpi/mpi-cmp.c')
0 files changed, 0 insertions, 0 deletions
