diff options
| author |   Borislav Petkov <bp@suse.de> | 2020-01-17 12:30:20 +0100 |
|---|---|---|
| committer | Borislav Petkov <bp@suse.de> | 2020-01-17 13:00:06 +0100 |
| commit | 7e5d6cf35329c8b232a1e97114545c1745d79083 (patch) | |
| tree | 9fd64a5b86c613e2f18dd80d1c6af9123baad921 /drivers/edac/amd64_edac.c | |
| parent | EDAC/sifive: Fix return value check in ecc_register() (diff) | |
| download | linux-dev-7e5d6cf35329c8b232a1e97114545c1745d79083.tar.xz linux-dev-7e5d6cf35329c8b232a1e97114545c1745d79083.zip | |
EDAC/amd64: Do not warn when removing instances
On machines which do not populate all nodes with DIMMs, the driver
doesn't initialize an instance there. However, the instance removal
remove_one_instance() path will warn unconditionally, which is wrong.
Remove the WARN_ON() even if the warning is innocent because it causes a
splat in dmesg.
Signed-off-by: Borislav Petkov <bp@suse.de>
Link: https://lkml.kernel.org/r/20200117115939.5524-1-bp@alien8.de
Diffstat (limited to 'drivers/edac/amd64_edac.c')
| -rw-r--r-- | drivers/edac/amd64_edac.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/edac/amd64_edac.c b/drivers/edac/amd64_edac.c index 428ce98f6776..d2a6d3319650 100644 --- a/drivers/edac/amd64_edac.c +++ b/drivers/edac/amd64_edac.c @@ -3573,9 +3573,6 @@ static void remove_one_instance(unsigned int nid) struct mem_ctl_info *mci; struct amd64_pvt *pvt; - mci = find_mci_by_dev(&F3->dev); - WARN_ON(!mci); - /* Remove from EDAC CORE tracking list */ mci = edac_mc_del_mc(&F3->dev); if (!mci) |