diff options
| author | Yazen Ghannam <Yazen.Ghannam@amd.com> | 2017-01-27 11:24:23 -0600 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-11-15 15:53:12 +0100 |
| commit | 7422c5813a9d8b78d95091ac69c9a51111c17a43 (patch) | |
| tree | e7e20079d23f970d139a6b070fc70081511a0e2b /drivers/edac/amd64_edac.c | |
| parent | 2f945e749ee21aa9a48c8c9bb4d9eeb43135d805 (diff) | |
| download | linux-stable-7422c5813a9d8b78d95091ac69c9a51111c17a43.tar.gz linux-stable-7422c5813a9d8b78d95091ac69c9a51111c17a43.tar.bz2 linux-stable-7422c5813a9d8b78d95091ac69c9a51111c17a43.zip | |
EDAC, amd64: Add x86cpuid sanity check during init
[ Upstream commit 1bd9900b8301fc505f032c90ea487824cf824e99 ]
Match one of the devices in amd64_cpuids[] before loading the module.
This is an additional sanity check against users trying to load
amd64_edac_mod on unsupported systems.
Signed-off-by: Yazen Ghannam <Yazen.Ghannam@amd.com>
Cc: linux-edac <linux-edac@vger.kernel.org>
Link: http://lkml.kernel.org/r/1485537863-2707-9-git-send-email-Yazen.Ghannam@amd.com
[ Get rid of err_ret label, make it a bit more readable this way. ]
Signed-off-by: Borislav Petkov <bp@suse.de>
Signed-off-by: Sasha Levin <alexander.levin@verizon.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/edac/amd64_edac.c')
| -rw-r--r-- | drivers/edac/amd64_edac.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/edac/amd64_edac.c b/drivers/edac/amd64_edac.c index ee181c53626f..431e5d7de215 100644 --- a/drivers/edac/amd64_edac.c +++ b/drivers/edac/amd64_edac.c @@ -2984,8 +2984,11 @@ static int __init amd64_edac_init(void) int err = -ENODEV; int i; + if (!x86_match_cpu(amd64_cpuids)) + return -ENODEV; + if (amd_cache_northbridges() < 0) - goto err_ret; + return -ENODEV; opstate_init(); @@ -3025,7 +3028,6 @@ err_free: kfree(ecc_stngs); ecc_stngs = NULL; -err_ret: return err; } |