Skip to content
Snippets Groups Projects
Commit f928d509 authored by Guy Zana's avatar Guy Zana
Browse files

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.
parent 9da6f1bc
No related branches found
No related tags found
No related merge requests found
...@@ -206,7 +206,6 @@ namespace virtio { ...@@ -206,7 +206,6 @@ namespace virtio {
// Wait for rx queue (used elements) // Wait for rx queue (used elements)
virtio_driver::wait_for_queue(_rx_queue); virtio_driver::wait_for_queue(_rx_queue);
int i = 0;
u32 len; u32 len;
virtio_net_req * req; virtio_net_req * req;
...@@ -216,8 +215,6 @@ namespace virtio { ...@@ -216,8 +215,6 @@ namespace virtio {
ii++; ii++;
auto m = req->um.release(); 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; delete req;
m->m_pkthdr.len = len; m->m_pkthdr.len = len;
...@@ -327,12 +324,10 @@ namespace virtio { ...@@ -327,12 +324,10 @@ namespace virtio {
void virtio_net::tx_gc() void virtio_net::tx_gc()
{ {
int i = 0;
u32 len; u32 len;
virtio_net_req * req; virtio_net_req * req;
while((req = static_cast<virtio_net_req*>(_tx_queue->get_buf(&len))) != nullptr) { while((req = static_cast<virtio_net_req*>(_tx_queue->get_buf(&len))) != nullptr) {
virtio_net_d(fmt("%s: gc %d") % __FUNCTION__ % i++);
delete req; delete req;
} }
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment