summaryrefslogtreecommitdiffstats
path: root/drivers/net
diff options
context:
space:
mode:
authorAnthony Liguori <aliguori@us.ibm.com>2008-04-07 15:33:16 -0500
committerLinus Torvalds <torvalds@linux-foundation.org>2008-04-07 13:56:33 -0700
commit6ea0a4679d6a11c66cfeb26d15244fb6f9b52d14 (patch)
tree1f6f8034ef903af7ed63cffd8b5d9821e3048843 /drivers/net
parent099c736a470c8080a166e7a089f1e48e15f9947c (diff)
downloadop-kernel-dev-6ea0a4679d6a11c66cfeb26d15244fb6f9b52d14.zip
op-kernel-dev-6ea0a4679d6a11c66cfeb26d15244fb6f9b52d14.tar.gz
virtio_net: remove overzealous printk
The 'disable_cb' is really just a hint and as such, it's possible for more work to get queued up while callbacks are disabled. Under stress with an SMP guest, this printk triggers very frequently. There is no race here, this is how things are designed to work so let's just remove the printk. Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> Acked-by: Rusty Russell <rusty@rustcorp.com.au> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/net')
-rw-r--r--drivers/net/virtio_net.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c
index b58472c..d1a200f 100644
--- a/drivers/net/virtio_net.c
+++ b/drivers/net/virtio_net.c
@@ -284,7 +284,6 @@ again:
/* Activate callback for using skbs: if this returns false it
* means some were used in the meantime. */
if (unlikely(!vi->svq->vq_ops->enable_cb(vi->svq))) {
- printk("Unlikely: restart svq race\n");
vi->svq->vq_ops->disable_cb(vi->svq);
netif_start_queue(dev);
goto again;
OpenPOWER on IntegriCloud