diff options
| author |   Ido Barnea <ibarnea@cisco.com> | 2016-05-17 14:01:20 +0300 |
|---|---|---|
| committer | Ido Barnea <ibarnea@cisco.com> | 2016-05-18 19:23:49 +0300 |
| commit | d9cc63306748f0ef954667753441ebc3367708e0 (patch) | |
| tree | ffc1b2b3cf95bd620b33efe5d0b1d6fdce8c0bdc | |
| parent | 614582c85a6429088464de3154c68c0d8932e280 (diff) | |
small corrections to counters
| -rw-r--r-- | src/flow_stat.cpp | 4 | ||||
| -rw-r--r-- | src/stateless/rx/trex_stateless_rx_core.cpp | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/src/flow_stat.cpp b/src/flow_stat.cpp index 5fc3aadd..7f086ab3 100644 --- a/src/flow_stat.cpp +++ b/src/flow_stat.cpp @@ -1051,13 +1051,13 @@ bool CFlowStatRuleMgr::dump_json(std::string & s_json, std::string & l_json, boo user_id_info_p->set_ooo_cnt(rfc2544_info[hw_id].get_ooo_cnt()); l_data_section[str_user_id]["latency"] = lat_hist; l_data_section[str_user_id]["latency"]["last_max"] = rfc2544_info[hw_id].get_last_max_usec(); - l_data_section[str_user_id]["jitter"] = rfc2544_info[hw_id].get_jitter_usec(); + l_data_section[str_user_id]["latency"]["jitter"] = rfc2544_info[hw_id].get_jitter_usec(); } else { // Not mapped to hw_id. Get saved info. user_id_info_p->get_latency_json(lat_hist); l_data_section[str_user_id]["latency"]["histogram"] = lat_hist; l_data_section[str_user_id]["latency"]["last_max"] = 0; - l_data_section[str_user_id]["jitter"] = user_id_info_p->get_jitter_usec(); + l_data_section[str_user_id]["latency"]["jitter"] = user_id_info_p->get_jitter_usec(); } //todo: add last 10 samples l_data_section[str_user_id]["err_cntrs"]["dropped"] diff --git a/src/stateless/rx/trex_stateless_rx_core.cpp b/src/stateless/rx/trex_stateless_rx_core.cpp index 463fa7d3..39ecb4f2 100644 --- a/src/stateless/rx/trex_stateless_rx_core.cpp +++ b/src/stateless/rx/trex_stateless_rx_core.cpp @@ -154,8 +154,8 @@ void CRxCoreStateless::handle_rx_pkt(CLatencyManagerPerPortStl *lp, rte_mbuf_t * m_rfc2544[hw_id].out_of_order += 1; // We thought it was lost, but it was just out of order m_rfc2544[hw_id].seq_err -= 1; - m_rfc2544[hw_id].seq_err_events_too_low++; } + m_rfc2544[hw_id].seq_err_events_too_low++; } } else { if (unlikely (m_rfc2544[hw_id].seq - seq > 100000)) { @@ -166,8 +166,8 @@ void CRxCoreStateless::handle_rx_pkt(CLatencyManagerPerPortStl *lp, rte_mbuf_t * m_rfc2544[hw_id].out_of_order += 1; // We thought it was lost, but it was just out of order m_rfc2544[hw_id].seq_err -= 1; - m_rfc2544[hw_id].seq_err_events_too_low++; } + m_rfc2544[hw_id].seq_err_events_too_low++; } else { // seq > m_rfc2544[hw_id].seq. Assuming lost packets m_rfc2544[hw_id].seq_err += seq - m_rfc2544[hw_id].seq; |