diff options
| author | Ethan Jackson <ethan@nicira.com> | 2011-08-16 14:16:58 -0700 |
|---|---|---|
| committer | Ethan Jackson <ethan@nicira.com> | 2011-08-16 14:33:17 -0700 |
| commit | 9adea03b888b0087cdd5877f61ed6a50170ea8f7 (patch) | |
| tree | f05079d12de359232c34bfcc3df43d562c86e33c /ofproto | |
| parent | bbb5d219de99cc99f0b88d25313a03ecaef13f9c (diff) | |
| download | openvswitch-9adea03b888b0087cdd5877f61ed6a50170ea8f7.tar.gz | |
ofproto: Fix over accounting of byte counters.
The update_stats() function in ofproto was attributing more bytes
to facets than they had actually accrued. This could potentially
throw off bond load balancing.
Found by inspection.
Diffstat (limited to 'ofproto')
| -rw-r--r-- | ofproto/ofproto-dpif.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c index fdd205718..3aa0e7860 100644 --- a/ofproto/ofproto-dpif.c +++ b/ofproto/ofproto-dpif.c @@ -1849,7 +1849,7 @@ update_stats(struct ofproto_dpif *p) facet->dp_byte_count = stats->n_bytes; facet_update_time(p, facet, stats->used); - facet_account(p, facet, stats->n_bytes); + facet_account(p, facet, 0); facet_push_stats(facet); } else { /* There's a flow in the datapath that we know nothing about. |