Commit 2d6e9b91 authored by Tiago Vignatti's avatar Tiago Vignatti Committed by Dave Airlie

vgaarb: convert pr_devel() to pr_debug()

We want to be able to use CONFIG_DYNAMIC_DEBUG in arbiter code, switch
the few existing pr_devel() calls to pr_debug().

Also, add one more debug information regarding decoding count.
Signed-off-by: default avatarTiago Vignatti <tiago.vignatti@nokia.com>
Signed-off-by: default avatarDave Airlie <airlied@redhat.com>
parent ce04cc08
...@@ -155,8 +155,8 @@ static struct vga_device *__vga_tryget(struct vga_device *vgadev, ...@@ -155,8 +155,8 @@ static struct vga_device *__vga_tryget(struct vga_device *vgadev,
(vgadev->decodes & VGA_RSRC_LEGACY_MEM)) (vgadev->decodes & VGA_RSRC_LEGACY_MEM))
rsrc |= VGA_RSRC_LEGACY_MEM; rsrc |= VGA_RSRC_LEGACY_MEM;
pr_devel("%s: %d\n", __func__, rsrc); pr_debug("%s: %d\n", __func__, rsrc);
pr_devel("%s: owns: %d\n", __func__, vgadev->owns); pr_debug("%s: owns: %d\n", __func__, vgadev->owns);
/* Check what resources we need to acquire */ /* Check what resources we need to acquire */
wants = rsrc & ~vgadev->owns; wants = rsrc & ~vgadev->owns;
...@@ -268,7 +268,7 @@ static void __vga_put(struct vga_device *vgadev, unsigned int rsrc) ...@@ -268,7 +268,7 @@ static void __vga_put(struct vga_device *vgadev, unsigned int rsrc)
{ {
unsigned int old_locks = vgadev->locks; unsigned int old_locks = vgadev->locks;
pr_devel("%s\n", __func__); pr_debug("%s\n", __func__);
/* Update our counters, and account for equivalent legacy resources /* Update our counters, and account for equivalent legacy resources
* if we decode them * if we decode them
...@@ -575,6 +575,7 @@ static inline void vga_update_device_decodes(struct vga_device *vgadev, ...@@ -575,6 +575,7 @@ static inline void vga_update_device_decodes(struct vga_device *vgadev,
else else
vga_decode_count--; vga_decode_count--;
} }
pr_debug("vgaarb: decoding count now is: %d\n", vga_decode_count);
} }
void __vga_set_legacy_decoding(struct pci_dev *pdev, unsigned int decodes, bool userspace) void __vga_set_legacy_decoding(struct pci_dev *pdev, unsigned int decodes, bool userspace)
...@@ -831,7 +832,7 @@ static ssize_t vga_arb_write(struct file *file, const char __user * buf, ...@@ -831,7 +832,7 @@ static ssize_t vga_arb_write(struct file *file, const char __user * buf,
curr_pos += 5; curr_pos += 5;
remaining -= 5; remaining -= 5;
pr_devel("client 0x%p called 'lock'\n", priv); pr_debug("client 0x%p called 'lock'\n", priv);
if (!vga_str_to_iostate(curr_pos, remaining, &io_state)) { if (!vga_str_to_iostate(curr_pos, remaining, &io_state)) {
ret_val = -EPROTO; ret_val = -EPROTO;
...@@ -867,7 +868,7 @@ static ssize_t vga_arb_write(struct file *file, const char __user * buf, ...@@ -867,7 +868,7 @@ static ssize_t vga_arb_write(struct file *file, const char __user * buf,
curr_pos += 7; curr_pos += 7;
remaining -= 7; remaining -= 7;
pr_devel("client 0x%p called 'unlock'\n", priv); pr_debug("client 0x%p called 'unlock'\n", priv);
if (strncmp(curr_pos, "all", 3) == 0) if (strncmp(curr_pos, "all", 3) == 0)
io_state = VGA_RSRC_LEGACY_IO | VGA_RSRC_LEGACY_MEM; io_state = VGA_RSRC_LEGACY_IO | VGA_RSRC_LEGACY_MEM;
...@@ -917,7 +918,7 @@ static ssize_t vga_arb_write(struct file *file, const char __user * buf, ...@@ -917,7 +918,7 @@ static ssize_t vga_arb_write(struct file *file, const char __user * buf,
curr_pos += 8; curr_pos += 8;
remaining -= 8; remaining -= 8;
pr_devel("client 0x%p called 'trylock'\n", priv); pr_debug("client 0x%p called 'trylock'\n", priv);
if (!vga_str_to_iostate(curr_pos, remaining, &io_state)) { if (!vga_str_to_iostate(curr_pos, remaining, &io_state)) {
ret_val = -EPROTO; ret_val = -EPROTO;
...@@ -961,7 +962,7 @@ static ssize_t vga_arb_write(struct file *file, const char __user * buf, ...@@ -961,7 +962,7 @@ static ssize_t vga_arb_write(struct file *file, const char __user * buf,
curr_pos += 7; curr_pos += 7;
remaining -= 7; remaining -= 7;
pr_devel("client 0x%p called 'target'\n", priv); pr_debug("client 0x%p called 'target'\n", priv);
/* if target is default */ /* if target is default */
if (!strncmp(curr_pos, "default", 7)) if (!strncmp(curr_pos, "default", 7))
pdev = pci_dev_get(vga_default_device()); pdev = pci_dev_get(vga_default_device());
...@@ -971,11 +972,11 @@ static ssize_t vga_arb_write(struct file *file, const char __user * buf, ...@@ -971,11 +972,11 @@ static ssize_t vga_arb_write(struct file *file, const char __user * buf,
ret_val = -EPROTO; ret_val = -EPROTO;
goto done; goto done;
} }
pr_devel("vgaarb: %s ==> %x:%x:%x.%x\n", curr_pos, pr_debug("vgaarb: %s ==> %x:%x:%x.%x\n", curr_pos,
domain, bus, PCI_SLOT(devfn), PCI_FUNC(devfn)); domain, bus, PCI_SLOT(devfn), PCI_FUNC(devfn));
pbus = pci_find_bus(domain, bus); pbus = pci_find_bus(domain, bus);
pr_devel("vgaarb: pbus %p\n", pbus); pr_debug("vgaarb: pbus %p\n", pbus);
if (pbus == NULL) { if (pbus == NULL) {
pr_err("vgaarb: invalid PCI domain and/or bus address %x:%x\n", pr_err("vgaarb: invalid PCI domain and/or bus address %x:%x\n",
domain, bus); domain, bus);
...@@ -983,7 +984,7 @@ static ssize_t vga_arb_write(struct file *file, const char __user * buf, ...@@ -983,7 +984,7 @@ static ssize_t vga_arb_write(struct file *file, const char __user * buf,
goto done; goto done;
} }
pdev = pci_get_slot(pbus, devfn); pdev = pci_get_slot(pbus, devfn);
pr_devel("vgaarb: pdev %p\n", pdev); pr_debug("vgaarb: pdev %p\n", pdev);
if (!pdev) { if (!pdev) {
pr_err("vgaarb: invalid PCI address %x:%x\n", pr_err("vgaarb: invalid PCI address %x:%x\n",
bus, devfn); bus, devfn);
...@@ -993,7 +994,7 @@ static ssize_t vga_arb_write(struct file *file, const char __user * buf, ...@@ -993,7 +994,7 @@ static ssize_t vga_arb_write(struct file *file, const char __user * buf,
} }
vgadev = vgadev_find(pdev); vgadev = vgadev_find(pdev);
pr_devel("vgaarb: vgadev %p\n", vgadev); pr_debug("vgaarb: vgadev %p\n", vgadev);
if (vgadev == NULL) { if (vgadev == NULL) {
pr_err("vgaarb: this pci device is not a vga device\n"); pr_err("vgaarb: this pci device is not a vga device\n");
pci_dev_put(pdev); pci_dev_put(pdev);
...@@ -1029,7 +1030,7 @@ static ssize_t vga_arb_write(struct file *file, const char __user * buf, ...@@ -1029,7 +1030,7 @@ static ssize_t vga_arb_write(struct file *file, const char __user * buf,
} else if (strncmp(curr_pos, "decodes ", 8) == 0) { } else if (strncmp(curr_pos, "decodes ", 8) == 0) {
curr_pos += 8; curr_pos += 8;
remaining -= 8; remaining -= 8;
pr_devel("vgaarb: client 0x%p called 'decodes'\n", priv); pr_debug("vgaarb: client 0x%p called 'decodes'\n", priv);
if (!vga_str_to_iostate(curr_pos, remaining, &io_state)) { if (!vga_str_to_iostate(curr_pos, remaining, &io_state)) {
ret_val = -EPROTO; ret_val = -EPROTO;
...@@ -1058,7 +1059,7 @@ static unsigned int vga_arb_fpoll(struct file *file, poll_table * wait) ...@@ -1058,7 +1059,7 @@ static unsigned int vga_arb_fpoll(struct file *file, poll_table * wait)
{ {
struct vga_arb_private *priv = file->private_data; struct vga_arb_private *priv = file->private_data;
pr_devel("%s\n", __func__); pr_debug("%s\n", __func__);
if (priv == NULL) if (priv == NULL)
return -ENODEV; return -ENODEV;
...@@ -1071,7 +1072,7 @@ static int vga_arb_open(struct inode *inode, struct file *file) ...@@ -1071,7 +1072,7 @@ static int vga_arb_open(struct inode *inode, struct file *file)
struct vga_arb_private *priv; struct vga_arb_private *priv;
unsigned long flags; unsigned long flags;
pr_devel("%s\n", __func__); pr_debug("%s\n", __func__);
priv = kmalloc(sizeof(struct vga_arb_private), GFP_KERNEL); priv = kmalloc(sizeof(struct vga_arb_private), GFP_KERNEL);
if (priv == NULL) if (priv == NULL)
...@@ -1101,7 +1102,7 @@ static int vga_arb_release(struct inode *inode, struct file *file) ...@@ -1101,7 +1102,7 @@ static int vga_arb_release(struct inode *inode, struct file *file)
unsigned long flags; unsigned long flags;
int i; int i;
pr_devel("%s\n", __func__); pr_debug("%s\n", __func__);
if (priv == NULL) if (priv == NULL)
return -ENODEV; return -ENODEV;
...@@ -1112,7 +1113,7 @@ static int vga_arb_release(struct inode *inode, struct file *file) ...@@ -1112,7 +1113,7 @@ static int vga_arb_release(struct inode *inode, struct file *file)
uc = &priv->cards[i]; uc = &priv->cards[i];
if (uc->pdev == NULL) if (uc->pdev == NULL)
continue; continue;
pr_devel("uc->io_cnt == %d, uc->mem_cnt == %d\n", pr_debug("uc->io_cnt == %d, uc->mem_cnt == %d\n",
uc->io_cnt, uc->mem_cnt); uc->io_cnt, uc->mem_cnt);
while (uc->io_cnt--) while (uc->io_cnt--)
vga_put(uc->pdev, VGA_RSRC_LEGACY_IO); vga_put(uc->pdev, VGA_RSRC_LEGACY_IO);
...@@ -1165,7 +1166,7 @@ static int pci_notify(struct notifier_block *nb, unsigned long action, ...@@ -1165,7 +1166,7 @@ static int pci_notify(struct notifier_block *nb, unsigned long action,
struct pci_dev *pdev = to_pci_dev(dev); struct pci_dev *pdev = to_pci_dev(dev);
bool notify = false; bool notify = false;
pr_devel("%s\n", __func__); pr_debug("%s\n", __func__);
/* For now we're only intereted in devices added and removed. I didn't /* For now we're only intereted in devices added and removed. I didn't
* test this thing here, so someone needs to double check for the * test this thing here, so someone needs to double check for the
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment