ofproto-dpif: Keep subfacet "used" times more up-to-date.
authorBen Pfaff <blp@nicira.com>
Fri, 13 Apr 2012 21:15:15 +0000 (14:15 -0700)
committerBen Pfaff <blp@nicira.com>
Thu, 19 Apr 2012 03:28:09 +0000 (20:28 -0700)
handle_flow_miss() didn't update subfacet "used" times for packets
processed by userspace.  This commit fixes the problem.

Found by inspection.  I didn't verify the bug in testing.

Signed-off-by: Ben Pfaff <blp@nicira.com>
lib/dpif.c

index febe8b0223cbe484459720e5a8157b3fb5239736..73696e4d8b9e08fab5b29fbf2930c5bc69b69245 100644 (file)
@@ -678,10 +678,10 @@ void
 dpif_flow_stats_extract(const struct flow *flow, const struct ofpbuf *packet,
                         struct dpif_flow_stats *stats)
 {
-    memset(stats, 0, sizeof(*stats));
     stats->tcp_flags = packet_get_tcp_flags(packet, flow);
     stats->n_bytes = packet->size;
     stats->n_packets = 1;
+    stats->used = time_msec();
 }
 
 /* Appends a human-readable representation of 'stats' to 's'. */