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 | |
| parent | 6cd18453b68942913fd3b1913b707646e544c2ac (diff) | |
| download | linux-7e5d6cf35329c8b232a1e97114545c1745d79083.tar.bz2 | |
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')
| -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) |