From: Maciej Gajdzica <maciejx.t.gajdz...@intel.com>

Added statistics for hash key32 table.

Signed-off-by: Maciej Gajdzica <maciejx.t.gajdzica at intel.com>
---
 lib/librte_table/rte_table_hash_key32.c |   41 +++++++++++++++++++++++++++++++
 1 file changed, 41 insertions(+)

diff --git a/lib/librte_table/rte_table_hash_key32.c 
b/lib/librte_table/rte_table_hash_key32.c
index 6790594..c230629 100644
--- a/lib/librte_table/rte_table_hash_key32.c
+++ b/lib/librte_table/rte_table_hash_key32.c
@@ -46,6 +46,20 @@

 #define RTE_BUCKET_ENTRY_VALID                                         0x1LLU

+#ifdef RTE_TABLE_STATS_COLLECT
+
+#define RTE_TABLE_HASH_KEY32_STATS_PKTS_IN_ADD(table, val) \
+       table->stats.n_pkts_in += val
+#define RTE_TABLE_HASH_KEY32_STATS_PKTS_LOOKUP_MISS(table, val) \
+       table->stats.n_pkts_lookup_miss += val
+
+#else
+
+#define RTE_TABLE_HASH_KEY32_STATS_PKTS_IN_ADD(table, val)
+#define RTE_TABLE_HASH_KEY32_STATS_PKTS_LOOKUP_MISS(table, val)
+
+#endif
+
 struct rte_bucket_4_32 {
        /* Cache line 0 */
        uint64_t signature[4 + 1];
@@ -61,6 +75,8 @@ struct rte_bucket_4_32 {
 };

 struct rte_table_hash {
+       struct rte_table_stats stats;
+
        /* Input parameters */
        uint32_t n_buckets;
        uint32_t n_entries_per_bucket;
@@ -850,6 +866,9 @@ rte_table_hash_lookup_key32_lru(
        uint32_t pkt11_index, pkt20_index, pkt21_index;
        uint64_t pkts_mask_out = 0;

+       __rte_unused uint32_t n_pkts_in = __builtin_popcountll(pkts_mask);
+       RTE_TABLE_HASH_KEY32_STATS_PKTS_IN_ADD(f, n_pkts_in);
+
        /* Cannot run the pipeline with less than 5 packets */
        if (__builtin_popcountll(pkts_mask) < 5) {
                for ( ; pkts_mask; ) {
@@ -864,6 +883,7 @@ rte_table_hash_lookup_key32_lru(
                }

                *lookup_hit_mask = pkts_mask_out;
+               RTE_TABLE_HASH_KEY32_STATS_PKTS_LOOKUP_MISS(f, n_pkts_in - 
__builtin_popcountll(pkts_mask_out));
                return 0;
        }

@@ -954,6 +974,7 @@ rte_table_hash_lookup_key32_lru(
                mbuf20, mbuf21, bucket20, bucket21, pkts_mask_out, entries, f);

        *lookup_hit_mask = pkts_mask_out;
+       RTE_TABLE_HASH_KEY32_STATS_PKTS_LOOKUP_MISS(f, n_pkts_in - 
__builtin_popcountll(pkts_mask_out));
        return 0;
 } /* rte_table_hash_lookup_key32_lru() */

@@ -974,6 +995,9 @@ rte_table_hash_lookup_key32_ext(
        struct rte_bucket_4_32 *buckets[RTE_PORT_IN_BURST_SIZE_MAX];
        uint64_t *keys[RTE_PORT_IN_BURST_SIZE_MAX];

+       __rte_unused uint32_t n_pkts_in = __builtin_popcountll(pkts_mask);
+       RTE_TABLE_HASH_KEY32_STATS_PKTS_IN_ADD(f, n_pkts_in);
+
        /* Cannot run the pipeline with less than 5 packets */
        if (__builtin_popcountll(pkts_mask) < 5) {
                for ( ; pkts_mask; ) {
@@ -1100,15 +1124,31 @@ grind_next_buckets:
        }

        *lookup_hit_mask = pkts_mask_out;
+       RTE_TABLE_HASH_KEY32_STATS_PKTS_LOOKUP_MISS(f, n_pkts_in - 
__builtin_popcountll(pkts_mask_out));
        return 0;
 } /* rte_table_hash_lookup_key32_ext() */

+static int
+rte_table_hash_key32_stats_read(void *table, struct rte_table_stats *stats, 
int clear)
+{
+       struct rte_table_hash *t = (struct rte_table_hash *) table;
+
+       if (stats != NULL)
+               memcpy(stats, &t->stats, sizeof(t->stats));
+
+       if (clear)
+               memset(&t->stats, 0, sizeof(t->stats));
+
+       return 0;
+}
+
 struct rte_table_ops rte_table_hash_key32_lru_ops = {
        .f_create = rte_table_hash_create_key32_lru,
        .f_free = rte_table_hash_free_key32_lru,
        .f_add = rte_table_hash_entry_add_key32_lru,
        .f_delete = rte_table_hash_entry_delete_key32_lru,
        .f_lookup = rte_table_hash_lookup_key32_lru,
+       .f_stats = rte_table_hash_key32_stats_read,
 };

 struct rte_table_ops rte_table_hash_key32_ext_ops = {
@@ -1117,4 +1157,5 @@ struct rte_table_ops rte_table_hash_key32_ext_ops = {
        .f_add = rte_table_hash_entry_add_key32_ext,
        .f_delete = rte_table_hash_entry_delete_key32_ext,
        .f_lookup = rte_table_hash_lookup_key32_ext,
+       .f_stats =rte_table_hash_key32_stats_read,
 };
-- 
1.7.9.5

Reply via email to