Messages in this thread | | | From | "Kani, Toshi" <> | Subject | RE: [PATCH v2 4/7] EDAC: Get chipset-specific edac drivers selected only when ghes_edac is not enabled | Date | Thu, 18 Aug 2022 23:56:50 +0000 |
| |
On Wednesday, August 17, 2022 8:35 AM, Jia He wrote: > When ghes_edac is loaded, a regular edac driver for the CPU type / platform > still attempts to register itself and fails in its module_init call. > > Suggested-by: Toshi Kani <toshi.kani@hpe.com> > Suggested-by: Borislav Petkov <bp@alien8.de> > Signed-off-by: Jia He <justin.he@arm.com> > --- > drivers/edac/amd64_edac.c | 3 +++ > drivers/edac/pnd2_edac.c | 3 +++ > drivers/edac/sb_edac.c | 3 +++ > drivers/edac/skx_base.c | 3 +++ > 4 files changed, 12 insertions(+)
Can you change i10nm_base.c and igen6_edac.c as well?
They are listed in your list below.
On Monday, August 15, 2022 8:20 PM, Justin He wrote: > I assume that all those edac drivers which used owner checking are > impacted, right? So the impacted list should be: > drivers/edac/pnd2_edac.c:1532: if (owner && strncmp(owner, > EDAC_MOD_STR, sizeof(EDAC_MOD_STR))) > drivers/edac/sb_edac.c:3513: if (owner && strncmp(owner, > EDAC_MOD_STR, sizeof(EDAC_MOD_STR))) > drivers/edac/amd64_edac.c:4333: if (owner && strncmp(owner, > EDAC_MOD_STR, sizeof(EDAC_MOD_STR))) > drivers/edac/i10nm_base.c:552: if (owner && strncmp(owner, > EDAC_MOD_STR, sizeof(EDAC_MOD_STR))) > drivers/edac/skx_base.c:657: if (owner && strncmp(owner, > EDAC_MOD_STR, sizeof(EDAC_MOD_STR))) > drivers/edac/igen6_edac.c:1275: if (owner && strncmp(owner, > EDAC_MOD_STR, sizeof(EDAC_MOD_STR)))
Thanks, Toshi
| |