EDAC/amd64: Tone down messages about missing PCI IDs
authorYazen Ghannam <yazen.ghannam@amd.com>
Tue, 15 Dec 2020 17:01:31 +0000 (17:01 +0000)
committerBorislav Petkov <bp@suse.de>
Mon, 28 Dec 2020 18:18:03 +0000 (19:18 +0100)
Give these messages a debug severity as they are really only useful to
the module developers.

Also, drop the "(broken BIOS?)" phrase, since this can cause churn for
BIOS folks. The PCI IDs needed by the module, at least on modern systems,
are fixed in hardware.

Signed-off-by: Yazen Ghannam <yazen.ghannam@amd.com>
Signed-off-by: Borislav Petkov <bp@suse.de>
Link: https://lkml.kernel.org/r/20201215170131.8496-1-Yazen.Ghannam@amd.com
drivers/edac/amd64_edac.c

index 5754f429a8d2d6dc56803fe04c9ee2564275a295..543221e3fe9b25d75e49766d72e2c0d6e6917aff 100644 (file)
@@ -2665,7 +2665,7 @@ reserve_mc_sibling_devs(struct amd64_pvt *pvt, u16 pci_id1, u16 pci_id2)
        if (pvt->umc) {
                pvt->F0 = pci_get_related_function(pvt->F3->vendor, pci_id1, pvt->F3);
                if (!pvt->F0) {
-                       amd64_err("F0 not found, device 0x%x (broken BIOS?)\n", pci_id1);
+                       edac_dbg(1, "F0 not found, device 0x%x\n", pci_id1);
                        return -ENODEV;
                }
 
@@ -2674,7 +2674,7 @@ reserve_mc_sibling_devs(struct amd64_pvt *pvt, u16 pci_id1, u16 pci_id2)
                        pci_dev_put(pvt->F0);
                        pvt->F0 = NULL;
 
-                       amd64_err("F6 not found: device 0x%x (broken BIOS?)\n", pci_id2);
+                       edac_dbg(1, "F6 not found: device 0x%x\n", pci_id2);
                        return -ENODEV;
                }
 
@@ -2691,7 +2691,7 @@ reserve_mc_sibling_devs(struct amd64_pvt *pvt, u16 pci_id1, u16 pci_id2)
        /* Reserve the ADDRESS MAP Device */
        pvt->F1 = pci_get_related_function(pvt->F3->vendor, pci_id1, pvt->F3);
        if (!pvt->F1) {
-               amd64_err("F1 not found: device 0x%x (broken BIOS?)\n", pci_id1);
+               edac_dbg(1, "F1 not found: device 0x%x\n", pci_id1);
                return -ENODEV;
        }
 
@@ -2701,7 +2701,7 @@ reserve_mc_sibling_devs(struct amd64_pvt *pvt, u16 pci_id1, u16 pci_id2)
                pci_dev_put(pvt->F1);
                pvt->F1 = NULL;
 
-               amd64_err("F2 not found: device 0x%x (broken BIOS?)\n", pci_id2);
+               edac_dbg(1, "F2 not found: device 0x%x\n", pci_id2);
                return -ENODEV;
        }