vgaarb: Downgrade userspace-triggerable messages to debug
authorBjorn Helgaas <bhelgaas@google.com>
Fri, 18 Nov 2016 14:11:58 +0000 (08:11 -0600)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 18 Nov 2016 15:29:52 +0000 (16:29 +0100)
To avoid userspace-triggerable dmesg spam, downgrade messages in the sysfs
write parsing code to debug level.

Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/20161118141158.32415.71438.stgit@bhelgaas-glaptop.roam.corp.google.com
drivers/gpu/vga/vgaarb.c

index 3625ad12c29fa3041131149a23481c9cd263ad66..b3d27182edd9e9d86182c6b1c9c92a4c7ae038e3 100644 (file)
@@ -1191,9 +1191,9 @@ static ssize_t vga_arb_write(struct file *file, const char __user *buf,
                        }
                        pdev = pci_get_domain_bus_and_slot(domain, bus, devfn);
                        if (!pdev) {
-                               pr_err("invalid PCI address %04x:%02x:%02x.%x\n",
-                                       domain, bus, PCI_SLOT(devfn),
-                                       PCI_FUNC(devfn));
+                               pr_debug("invalid PCI address %04x:%02x:%02x.%x\n",
+                                        domain, bus, PCI_SLOT(devfn),
+                                        PCI_FUNC(devfn));
                                ret_val = -ENODEV;
                                goto done;
                        }
@@ -1207,7 +1207,7 @@ static ssize_t vga_arb_write(struct file *file, const char __user *buf,
                pr_debug("vgadev %p\n", vgadev);
                if (vgadev == NULL) {
                        if (pdev) {
-                               vgaarb_err(&pdev->dev, "not a VGA device\n");
+                               vgaarb_dbg(&pdev->dev, "not a VGA device\n");
                                pci_dev_put(pdev);
                        }
 
@@ -1227,7 +1227,7 @@ static ssize_t vga_arb_write(struct file *file, const char __user *buf,
                        }
                }
                if (i == MAX_USER_CARDS) {
-                       vgaarb_err(&pdev->dev, "maximum user cards (%d) number reached, ignoring this one!\n",
+                       vgaarb_dbg(&pdev->dev, "maximum user cards (%d) number reached, ignoring this one!\n",
                                MAX_USER_CARDS);
                        pci_dev_put(pdev);
                        /* XXX: which value to return? */