Make rate calculation more carefully by checking previous & current bytes/pkts counter.
Do calculation only if update time passed >= 1s. Signed-off-by: Vadim Kochan <[email protected]> --- flowtop.c | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/flowtop.c b/flowtop.c index b2b1336..dd6479d 100644 --- a/flowtop.c +++ b/flowtop.c @@ -304,6 +304,11 @@ static void flow_entry_update_time(struct flow_entry *n) bug_on(gettimeofday(&n->last_update, NULL)); } +#define CALC_RATE(fld) \ +do { \ + n->rate_##fld = (fld > (n->fld) ? (fld - n->fld) / sec : 0); \ +} while (0) + static void flow_entry_calc_rate(struct flow_entry *n, const struct nf_conntrack *ct) { uint64_t bytes_src = nfct_get_attr_u64(ct, ATTR_ORIG_COUNTER_BYTES); @@ -312,13 +317,13 @@ static void flow_entry_calc_rate(struct flow_entry *n, const struct nf_conntrack uint64_t pkts_dst = nfct_get_attr_u64(ct, ATTR_REPL_COUNTER_PACKETS); double sec = (double)time_after_us(&n->last_update) / USEC_PER_SEC; - if (sec <= 0) + if (sec < 1) return; - n->rate_bytes_src = (bytes_src - n->bytes_src) / sec; - n->rate_bytes_dst = (bytes_dst - n->bytes_dst) / sec; - n->rate_pkts_src = (pkts_src - n->pkts_src) / sec; - n->rate_pkts_dst = (pkts_dst - n->pkts_dst) / sec; + CALC_RATE(bytes_src); + CALC_RATE(bytes_dst); + CALC_RATE(pkts_src); + CALC_RATE(pkts_dst); } static inline struct flow_entry *flow_entry_xalloc(void) -- 2.6.1 -- You received this message because you are subscribed to the Google Groups "netsniff-ng" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
