[ovs-dev] [PATCH 07/10] netdev-dummy: Queue packets only to one rx queue.

[Ilya Maximets]

This is unused functionality and it will harm to
multiqueue implementation.

Signed-off-by: Ilya Maximets <i.maxim...@samsung.com>
---
 lib/netdev-dummy.c | 20 ++++++++------------
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/lib/netdev-dummy.c b/lib/netdev-dummy.c
index 0e536e8..4903e56 100644
--- a/lib/netdev-dummy.c
+++ b/lib/netdev-dummy.c
@@ -1338,26 +1338,22 @@ static void
 netdev_dummy_queue_packet(struct netdev_dummy *dummy, struct dp_packet *packet)
     OVS_REQUIRES(dummy->mutex)
 {
-    struct netdev_rxq_dummy *rx, *prev;
+    struct netdev_rxq_dummy *rx;
 
     if (dummy->rxq_pcap) {
         ovs_pcap_write(dummy->rxq_pcap, packet);
         fflush(dummy->rxq_pcap);
     }
-    prev = NULL;
-    LIST_FOR_EACH (rx, node, &dummy->rxes) {
+
+    if (list_size(&dummy->rxes)) {
+        ASSIGN_CONTAINER(rx, list_front(&dummy->rxes), node);
         if (rx->recv_queue_len < NETDEV_DUMMY_MAX_QUEUE) {
-            if (prev) {
-                netdev_dummy_queue_packet__(prev, dp_packet_clone(packet));
-            }
-            prev = rx;
+            netdev_dummy_queue_packet__(rx, packet);
+            return;
         }
     }
-    if (prev) {
-        netdev_dummy_queue_packet__(prev, packet);
-    } else {
-        dp_packet_delete(packet);
-    }
+
+     dp_packet_delete(packet);
 }
 
 static void
-- 
2.5.0

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev