[ovs-dev] [PATCH 1/3] dpif-netdev: properly maintain exact match cache hit counter

Andy Zhou azhou at ovn.org
Wed Jan 27 21:31:02 UTC 2016


Current logic counts dropped packet as cache hit which is not
correct. This patch removes dropped packet to improve accuracy.

Signed-off-by: Andy Zhou <azhou at ovn.org>
---
 lib/dpif-netdev.c | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c
index cd72e62..dec06fc 100644
--- a/lib/dpif-netdev.c
+++ b/lib/dpif-netdev.c
@@ -3211,13 +3211,14 @@ emc_processing(struct dp_netdev_pmd_thread *pmd, struct dp_packet **packets,
 {
     struct emc_cache *flow_cache = &pmd->flow_cache;
     struct netdev_flow_key key;
-    size_t i, notfound_cnt = 0;
+    size_t i, n_missed = 0, n_dropped = 0;
 
     for (i = 0; i < cnt; i++) {
         struct dp_netdev_flow *flow;
 
         if (OVS_UNLIKELY(dp_packet_size(packets[i]) < ETH_HEADER_LEN)) {
             dp_packet_delete(packets[i]);
+            n_dropped++;
             continue;
         }
 
@@ -3235,17 +3236,17 @@ emc_processing(struct dp_netdev_pmd_thread *pmd, struct dp_packet **packets,
             dp_netdev_queue_batches(packets[i], flow, &key.mf, batches,
                                     n_batches);
         } else {
-            if (i != notfound_cnt) {
-                dp_packet_swap(&packets[i], &packets[notfound_cnt]);
+            if (i != n_missed) {
+                dp_packet_swap(&packets[i], &packets[n_missed]);
             }
 
-            keys[notfound_cnt++] = key;
+            keys[n_missed++] = key;
         }
     }
 
-    dp_netdev_count_packet(pmd, DP_STAT_EXACT_HIT, cnt - notfound_cnt);
+    dp_netdev_count_packet(pmd, DP_STAT_EXACT_HIT, cnt - n_dropped - n_missed);
 
-    return notfound_cnt;
+    return n_missed;
 }
 
 static inline void
-- 
1.9.1




More information about the dev mailing list