[dpdk-dev] [PATCH 09/10] net/softnic: meter stats read

[Jasvinder Singh <jasvinder.singh@intel.com>]

Implement meter object stats read function.

Signed-off-by: Jasvinder Singh <jasvinder.singh@intel.com>
---
 drivers/net/softnic/rte_eth_softnic_meter.c | 99 ++++++++++++++++++++++++++++-
 1 file changed, 98 insertions(+), 1 deletion(-)

diff --git a/drivers/net/softnic/rte_eth_softnic_meter.c b/drivers/net/softnic/rte_eth_softnic_meter.c
index 41af9c7..2cea19f 100644
--- a/drivers/net/softnic/rte_eth_softnic_meter.c
+++ b/drivers/net/softnic/rte_eth_softnic_meter.c
@@ -629,6 +629,103 @@ pmd_mtr_policer_actions_update(struct rte_eth_dev *dev,
 	return 0;
 }
 
+/* MTR object stats read */
+static int
+pmd_mtr_stats_read(struct rte_eth_dev *dev,
+	uint32_t mtr_id,
+	struct rte_mtr_stats *stats,
+	uint64_t *stats_mask,
+	int clear,
+	struct rte_mtr_error *error)
+{
+	struct pmd_internals *p = dev->data->dev_private;
+	struct softnic_mtr *m;
+	uint32_t i;
+	int status;
+
+	/* MTR object id must be valid */
+	m = softnic_mtr_find(p, mtr_id);
+	if (m == NULL)
+		return -rte_mtr_error_set(error,
+			EEXIST,
+			RTE_MTR_ERROR_TYPE_MTR_ID,
+			NULL,
+			rte_strerror(EEXIST));
+
+	/* MTR meter object owner valid? */
+	if (m->flow == NULL)
+		return 0;
+
+	/* Meter stats */
+	struct pipeline *pipeline = m->flow->pipeline;
+	struct softnic_table *table = &pipeline->table[m->flow->table_id];
+	struct rte_table_action_mtr_counters counters;
+	uint32_t tc_mask = 1 << 0;
+
+	status = rte_table_action_meter_read(table->a,
+		m->flow->data,
+		tc_mask,
+		&counters,
+		clear);
+	if (status) {
+		rte_mtr_error_set(error,
+			EINVAL,
+			RTE_MTR_ERROR_TYPE_UNSPECIFIED,
+			NULL,
+			"Meter stats read failed");
+		return -1;
+	}
+
+	if (stats) {
+		struct rte_table_action_mtr_counters_tc tc_stats;
+
+		memcpy(&tc_stats, &counters.stats[tc_mask], sizeof(tc_stats));
+
+		for (i = 0; i < RTE_MTR_COLORS; i++) {
+			if (m->params.action[i] == MTR_POLICER_ACTION_COLOR_GREEN) {
+				stats->n_pkts[RTE_MTR_GREEN] += tc_stats.n_packets[i];
+				stats->n_bytes[RTE_MTR_GREEN] += tc_stats.n_bytes[i];
+			}
+
+			if (m->params.action[i] == MTR_POLICER_ACTION_COLOR_YELLOW) {
+				stats->n_pkts[RTE_MTR_YELLOW] += tc_stats.n_packets[i];
+				stats->n_bytes[RTE_MTR_YELLOW] += tc_stats.n_bytes[i];
+			}
+
+			if (m->params.action[i] == MTR_POLICER_ACTION_COLOR_RED) {
+				stats->n_pkts[RTE_MTR_RED] += tc_stats.n_packets[i];
+				stats->n_bytes[RTE_MTR_RED] += tc_stats.n_bytes[i];
+			}
+
+			if (m->params.action[i] == MTR_POLICER_ACTION_DROP) {
+				stats->n_pkts_dropped += tc_stats.n_packets[i];
+				stats->n_bytes_dropped += tc_stats.n_bytes[i];
+			}
+		}
+	}
+
+	if (stats_mask) {
+		if (stats->n_pkts[RTE_MTR_GREEN])
+			*stats_mask |= RTE_MTR_STATS_N_PKTS_GREEN;
+		if (stats->n_pkts[RTE_MTR_YELLOW])
+			*stats_mask |= RTE_MTR_STATS_N_PKTS_YELLOW;
+		if (stats->n_pkts[RTE_MTR_RED])
+			*stats_mask |= RTE_MTR_STATS_N_PKTS_RED;
+		if (stats->n_bytes[RTE_MTR_GREEN])
+			*stats_mask |= RTE_MTR_STATS_N_BYTES_GREEN;
+		if (stats->n_bytes[RTE_MTR_YELLOW])
+			*stats_mask |= RTE_MTR_STATS_N_BYTES_YELLOW;
+		if (stats->n_bytes[RTE_MTR_RED])
+			*stats_mask |= RTE_MTR_STATS_N_BYTES_RED;
+		if (stats->n_pkts_dropped)
+			*stats_mask |= RTE_MTR_STATS_N_PKTS_DROPPED;
+		if (stats->n_bytes_dropped)
+			*stats_mask |= RTE_MTR_STATS_N_BYTES_DROPPED;
+	}
+
+	return 0;
+}
+
 const struct rte_mtr_ops pmd_mtr_ops = {
 	.capabilities_get = NULL,
 
@@ -645,5 +742,5 @@ const struct rte_mtr_ops pmd_mtr_ops = {
 	.policer_actions_update = pmd_mtr_policer_actions_update,
 	.stats_update = NULL,
 
-	.stats_read = NULL,
+	.stats_read = pmd_mtr_stats_read,
 };
-- 
2.9.3