[dpdk-dev] [PATCH v3 16/22] net/bnxt: modify shadow tcam and tbl reference count logic

[Ajit Khaparde]

From: Mike Baucom <michael.bau...@broadcom.com>

Moved setting the refcnt for shadow tcam and table entries to the
allocation path only.  The insert can be called multiple times for
updates and was resetting the refcnt to 1 each time.  Now multiple
insertion/modifications will not change the reference count.

Signed-off-by: Mike Baucom <michael.bau...@broadcom.com>
Reviewed-by: Kishore Padmanabha <kishore.padmana...@broadcom.com>
---
 drivers/net/bnxt/tf_core/tf_shadow_tbl.c  | 2 --
 drivers/net/bnxt/tf_core/tf_shadow_tcam.c | 2 +-
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/net/bnxt/tf_core/tf_shadow_tbl.c 
b/drivers/net/bnxt/tf_core/tf_shadow_tbl.c
index 019a26eba..a4207eb3a 100644
--- a/drivers/net/bnxt/tf_core/tf_shadow_tbl.c
+++ b/drivers/net/bnxt/tf_core/tf_shadow_tbl.c
@@ -687,8 +687,6 @@ tf_shadow_tbl_insert(struct tf_shadow_tbl_insert_parms 
*parms)
        if (!TF_SHADOW_HB_HANDLE_IS_VALID(sr_entry->hb_handle))
                return 0;
 
-       sr_entry->refcnt = 1;
-
        return 0;
 }
 
diff --git a/drivers/net/bnxt/tf_core/tf_shadow_tcam.c 
b/drivers/net/bnxt/tf_core/tf_shadow_tcam.c
index a0130d6a8..e2c347a1e 100644
--- a/drivers/net/bnxt/tf_core/tf_shadow_tcam.c
+++ b/drivers/net/bnxt/tf_core/tf_shadow_tcam.c
@@ -472,6 +472,7 @@ tf_shadow_tcam_bind_index(struct 
tf_shadow_tcam_bind_index_parms *parms)
        /* Write the result table */
        sr_entry->key_size = parms->key_size;
        sr_entry->hb_handle = parms->hb_handle;
+       sr_entry->refcnt = 1;
 
        return 0;
 }
@@ -738,7 +739,6 @@ tf_shadow_tcam_insert(struct tf_shadow_tcam_insert_parms 
*parms)
 
        memcpy(sr_entry->result, sparms->result, sparms->result_size);
        sr_entry->result_size = sparms->result_size;
-       sr_entry->refcnt = 1;
 
        return 0;
 }
-- 
2.21.1 (Apple Git-122.3)