From f928d509636b60abb3938781102aa2edb79f13c7 Mon Sep 17 00:00:00 2001
From: Guy Zana <guy@cloudius-systems.com>
Date: Thu, 6 Jun 2013 22:29:59 +0300
Subject: [PATCH] virtio: remove some extra verbose debug messages

next patch is changing the debug function to tprintf_d, which may be
implemented as do{}while(0) in case conf-logger_debug=0, in this case
compilation breaks complaining about unused variables.

these debug prints are not very useful today, so I remove them. Instead,
they may be implemented as tracepoints.
---
 drivers/virtio-net.cc | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/drivers/virtio-net.cc b/drivers/virtio-net.cc
index 0aed9aece..957fb97cc 100644
--- a/drivers/virtio-net.cc
+++ b/drivers/virtio-net.cc
@@ -206,7 +206,6 @@ namespace virtio {
             // Wait for rx queue (used elements)
             virtio_driver::wait_for_queue(_rx_queue);
 
-            int i = 0;
             u32 len;
             virtio_net_req * req;
 
@@ -216,8 +215,6 @@ namespace virtio {
                 ii++;
 
                 auto m = req->um.release();
-                u8* buf = mtod(m, u8*);
-                virtio_net_d(fmt("\t got hdr len:%d = %d, len= %d vaddr=%p") % i++ % (int)req->hdr.hdr_len % len % (void*)buf);
                 delete req;
 
                 m->m_pkthdr.len = len;
@@ -327,12 +324,10 @@ namespace virtio {
 
     void virtio_net::tx_gc()
     {
-        int i = 0;
         u32 len;
         virtio_net_req * req;
 
         while((req = static_cast<virtio_net_req*>(_tx_queue->get_buf(&len))) != nullptr) {
-            virtio_net_d(fmt("%s: gc %d") % __FUNCTION__ % i++);
             delete req;
         }
     }
-- 
GitLab