From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 4865945C9F; Thu, 7 Nov 2024 14:45:55 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id BA6B242F61; Thu, 7 Nov 2024 14:45:42 +0100 (CET) Received: from mail-pl1-f179.google.com (mail-pl1-f179.google.com [209.85.214.179]) by mails.dpdk.org (Postfix) with ESMTP id 1F03C42F51 for ; Thu, 7 Nov 2024 14:45:41 +0100 (CET) Received: by mail-pl1-f179.google.com with SMTP id d9443c01a7336-20c714cd9c8so10666125ad.0 for ; Thu, 07 Nov 2024 05:45:41 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; t=1730987140; x=1731591940; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=LLWrWLj22zB36ba6r0gEbvJ2DVRdRhtqcibYlAo5hLw=; b=CLgiugH6jVc7Jl7PRicsdlepFEdCxUm9gxXicwj6rZNwt6MVQCbNzGHe1XRMgJ3rNa W8ZzJwGJzsgCGvvlHrJtBbrFXTlvcYkKcbKwFFW5nNhWugsvBZMsV0+GSp2M6WLrIrWS MBHCYShpuYDjEQ74tVhu5H7M4xxmtZPk4smkc= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1730987140; x=1731591940; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=LLWrWLj22zB36ba6r0gEbvJ2DVRdRhtqcibYlAo5hLw=; b=TGreeTVcaLGRNQH3EXBsIoFEt+6ATccIKxQV50qUgiAHyVCUxZsUKekUyFFVQ+jsag pR/uNvdTSLmRUzAQutPJt5/Coem0CQhoLIxUH3kCPL+o1IeQsygFVjrAtbrikNSuljaE udX+BMzV0WKVKPIvQGxjXEpZqRd5+/7MPRw82iR6jlHcRHn1aPIYiE1ZCEa15M3vO9TE IvoKPENcaeXeyKRcR9zCfKCv6AOfxsF9QccDAd1EDy4fUbi1Y/i+QYLK/fdtDvMMwwom kp1p21Bb67duEfLgdFubQPiKUUzFIqsMd+8Ukl3JqYGu8Tg2UJTIGqV9RslJHfXYyP6C 1/bw== X-Gm-Message-State: AOJu0Yw0/+9hNH0n1Y27ZCZ12fFRarNFc8nW8G4qSrgUJMw5/yeLbc/Z kf2k6V+JhplVCwobPKXZuUkt9rafzJRe/mx6OB70yH7DsbCDD8aVxZzPhdDghWGJ9+9J3HjBk+Y mTD6zM6ceTO5ZM9Tu0T/fnLMV+GmNkNZ7LSnWiQNOcPvkL31elXqkhZMFaHEbcc+ZdRLIyWt49Z kUaj+RPnPmAdPWJ4Dq9GsTbXuhpH4fXjBifPx0neBMoA== X-Google-Smtp-Source: AGHT+IHDWczQIIYgyY927GtxP/IL16lgerxbL8zksqYmiWFwekxFRhFSnZ9GrQUjoSJKfynzTjbGgA== X-Received: by 2002:a17:90b:3886:b0:2e2:b281:536e with SMTP id 98e67ed59e1d1-2e94c2bddd7mr31151812a91.15.1730987139583; Thu, 07 Nov 2024 05:45:39 -0800 (PST) Received: from dhcp-10-123-154-23.dhcp.broadcom.net ([192.19.234.250]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-2e9a5f52969sm1409878a91.2.2024.11.07.05.45.37 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 07 Nov 2024 05:45:39 -0800 (PST) From: Sriharsha Basavapatna To: dev@dpdk.org Cc: Kishore Padmanabha , Shuanglin Wang , Sriharsha Basavapatna , Mike Baucom , Ajit Khaparde Subject: [PATCH v8 27/47] net/bnxt: tf_ulp: fixed parent child db counters Date: Thu, 7 Nov 2024 19:22:34 +0530 Message-Id: <20241107135254.1611676-28-sriharsha.basavapatna@broadcom.com> X-Mailer: git-send-email 2.39.0.189.g4dbebc36b0 In-Reply-To: <20241107135254.1611676-1-sriharsha.basavapatna@broadcom.com> References: <20241107135254.1611676-1-sriharsha.basavapatna@broadcom.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org From: Kishore Padmanabha The locking for the parent child counters need to be done till the stats are retrieved. Also the OVS is creating multiple F1 flows for same tunnel hence reference count needs to be maintined for the F1 flows. Fix name conflicts for class and action tables. Matcher allocates hash tables for class and action entries. These tables should have different names for each port. Fixes: bdf4a3c6316b ("net/bnxt: support tunnel offload") Signed-off-by: Kishore Padmanabha Signed-off-by: Shuanglin Wang Signed-off-by: Sriharsha Basavapatna Reviewed-by: Mike Baucom Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_ulp/ulp_fc_mgr.c | 37 +++++++++++++++++---------- drivers/net/bnxt/tf_ulp/ulp_flow_db.c | 22 ++++++++++------ drivers/net/bnxt/tf_ulp/ulp_flow_db.h | 2 ++ drivers/net/bnxt/tf_ulp/ulp_matcher.c | 12 +++++++-- 4 files changed, 50 insertions(+), 23 deletions(-) diff --git a/drivers/net/bnxt/tf_ulp/ulp_fc_mgr.c b/drivers/net/bnxt/tf_ulp/ulp_fc_mgr.c index b880b545da..0c46c7d4c9 100644 --- a/drivers/net/bnxt/tf_ulp/ulp_fc_mgr.c +++ b/drivers/net/bnxt/tf_ulp/ulp_fc_mgr.c @@ -563,23 +563,29 @@ int ulp_fc_mgr_query_count_get(struct bnxt_ulp_context *ctxt, } while (!rc && nxt_resource_index); - bnxt_ulp_cntxt_release_fdb_lock(ctxt); - - if (rc || !found_cntr_resource) + if (rc || !found_cntr_resource) { + bnxt_ulp_cntxt_release_fdb_lock(ctxt); return rc; + } dir = params.direction; - if (!(ulp_fc_info->flags & ULP_FLAG_FC_SW_AGG_EN)) - return fc_ops->ulp_flow_stat_get(ctxt, dir, - params.resource_hndl, count); + if (!(ulp_fc_info->flags & ULP_FLAG_FC_SW_AGG_EN)) { + rc = fc_ops->ulp_flow_stat_get(ctxt, dir, + params.resource_hndl, count); + bnxt_ulp_cntxt_release_fdb_lock(ctxt); + return rc; + } if (!found_parent_flow && params.resource_sub_type == BNXT_ULP_RESOURCE_SUB_TYPE_INDEX_TABLE_INT_COUNT) { hw_cntr_id = params.resource_hndl; - if (!ulp_fc_info->num_counters) - return fc_ops->ulp_flow_stat_get(ctxt, dir, - hw_cntr_id, count); + if (!ulp_fc_info->num_counters) { + rc = fc_ops->ulp_flow_stat_get(ctxt, dir, + hw_cntr_id, count); + bnxt_ulp_cntxt_release_fdb_lock(ctxt); + return rc; + } /* TODO: * Think about optimizing with try_lock later @@ -603,9 +609,14 @@ int ulp_fc_mgr_query_count_get(struct bnxt_ulp_context *ctxt, params.resource_sub_type == BNXT_ULP_RESOURCE_SUB_TYPE_INDEX_TABLE_INT_COUNT) { /* Get stats from the parent child table */ - ulp_flow_db_parent_flow_count_get(ctxt, pc_idx, - &count->hits, &count->bytes, - count->reset); + if (ulp_flow_db_parent_flow_count_get(ctxt, flow_id, + pc_idx, + &count->hits, + &count->bytes, + count->reset)) { + bnxt_ulp_cntxt_release_fdb_lock(ctxt); + return -EIO; + } if (count->hits) count->hits_set = 1; if (count->bytes) @@ -613,7 +624,7 @@ int ulp_fc_mgr_query_count_get(struct bnxt_ulp_context *ctxt, } else { rc = -EINVAL; } - + bnxt_ulp_cntxt_release_fdb_lock(ctxt); return rc; } diff --git a/drivers/net/bnxt/tf_ulp/ulp_flow_db.c b/drivers/net/bnxt/tf_ulp/ulp_flow_db.c index 2ff5227774..27f1115990 100644 --- a/drivers/net/bnxt/tf_ulp/ulp_flow_db.c +++ b/drivers/net/bnxt/tf_ulp/ulp_flow_db.c @@ -838,7 +838,6 @@ ulp_flow_db_fid_free(struct bnxt_ulp_context *ulp_ctxt, #ifdef RTE_LIBRTE_BNXT_TRUFLOW_DEBUG BNXT_DRV_DBG(DEBUG, "flow_id = %u:%u freed\n", flow_type, fid); #endif - /* all good, return success */ return 0; } @@ -1385,13 +1384,12 @@ ulp_flow_db_pc_db_parent_flow_set(struct bnxt_ulp_context *ulp_ctxt, if (set_flag) { pc_entry->parent_fid = parent_fid; + pc_entry->parent_ref_cnt++; } else { - if (pc_entry->parent_fid != parent_fid) - BNXT_DRV_DBG(ERR, "Panic: invalid parent id\n"); - pc_entry->parent_fid = 0; - + if (pc_entry->parent_ref_cnt > 0) + pc_entry->parent_ref_cnt--; /* Free the parent child db entry if no user present */ - if (!pc_entry->f2_cnt) + if (!pc_entry->parent_ref_cnt && !pc_entry->f2_cnt) ulp_flow_db_pc_db_entry_free(ulp_ctxt, pc_entry); } return 0; @@ -1446,7 +1444,7 @@ ulp_flow_db_pc_db_child_flow_set(struct bnxt_ulp_context *ulp_ctxt, ULP_INDEX_BITMAP_RESET(t[a_idx], child_fid); if (pc_entry->f2_cnt) pc_entry->f2_cnt--; - if (!pc_entry->f2_cnt && !pc_entry->parent_fid) + if (!pc_entry->f2_cnt && !pc_entry->parent_ref_cnt) ulp_flow_db_pc_db_entry_free(ulp_ctxt, pc_entry); } return 0; @@ -1538,7 +1536,7 @@ ulp_flow_db_parent_flow_count_accum_set(struct bnxt_ulp_context *ulp_ctxt, /* check for parent idx validity */ p_pdb = &flow_db->parent_child_db; if (pc_idx >= p_pdb->entries_count || - !p_pdb->parent_flow_tbl[pc_idx].parent_fid) { + !p_pdb->parent_flow_tbl[pc_idx].parent_ref_cnt) { BNXT_DRV_DBG(ERR, "Invalid parent child index %x\n", pc_idx); return -EINVAL; } @@ -1786,6 +1784,7 @@ ulp_flow_db_parent_flow_count_update(struct bnxt_ulp_context *ulp_ctxt, */ int32_t ulp_flow_db_parent_flow_count_get(struct bnxt_ulp_context *ulp_ctxt, + uint32_t flow_id, uint32_t pc_idx, uint64_t *packet_count, uint64_t *byte_count, uint8_t count_reset) { @@ -1798,6 +1797,13 @@ ulp_flow_db_parent_flow_count_get(struct bnxt_ulp_context *ulp_ctxt, return -EINVAL; } + /* stale parent fid */ + if (flow_id != pc_entry->parent_fid) { + *packet_count = 0; + *byte_count = 0; + return 0; + } + if (pc_entry->counter_acc) { *packet_count = pc_entry->pkt_count; *byte_count = pc_entry->byte_count; diff --git a/drivers/net/bnxt/tf_ulp/ulp_flow_db.h b/drivers/net/bnxt/tf_ulp/ulp_flow_db.h index 39810c81c4..5a4b5a1ebf 100644 --- a/drivers/net/bnxt/tf_ulp/ulp_flow_db.h +++ b/drivers/net/bnxt/tf_ulp/ulp_flow_db.h @@ -63,6 +63,7 @@ struct bnxt_ulp_flow_tbl { struct ulp_fdb_parent_info { uint32_t valid; uint32_t parent_fid; + uint32_t parent_ref_cnt; uint32_t counter_acc; uint64_t pkt_count; uint64_t byte_count; @@ -389,6 +390,7 @@ ulp_flow_db_parent_flow_count_update(struct bnxt_ulp_context *ulp_ctxt, */ int32_t ulp_flow_db_parent_flow_count_get(struct bnxt_ulp_context *ulp_ctxt, + uint32_t flow_id, uint32_t pc_idx, uint64_t *packet_count, uint64_t *byte_count, diff --git a/drivers/net/bnxt/tf_ulp/ulp_matcher.c b/drivers/net/bnxt/tf_ulp/ulp_matcher.c index b508bd2df6..9615fc7919 100644 --- a/drivers/net/bnxt/tf_ulp/ulp_matcher.c +++ b/drivers/net/bnxt/tf_ulp/ulp_matcher.c @@ -327,9 +327,17 @@ ulp_matcher_action_match(struct ulp_rte_parser_params *params, int32_t ulp_matcher_init(struct bnxt_ulp_context *ulp_ctx) { struct rte_hash_parameters hash_tbl_params = {0}; - char hash_class_tbl_name[64] = "bnxt_ulp_class_matcher"; - char hash_act_tbl_name[64] = "bnxt_ulp_act_matcher"; + char hash_class_tbl_name[64] = {0}; + char hash_act_tbl_name[64] = {0}; struct bnxt_ulp_matcher_data *data; + uint16_t port_id; + + /* append port_id to the buffer name */ + port_id = ulp_ctx->bp->eth_dev->data->port_id; + snprintf(hash_class_tbl_name, sizeof(hash_class_tbl_name), + "bnxt_ulp_class_matcher_%d", port_id); + snprintf(hash_act_tbl_name, sizeof(hash_act_tbl_name), + "bnxt_ulp_act_matcher_%d", port_id); data = rte_zmalloc("bnxt_ulp_matcher_data", sizeof(struct bnxt_ulp_matcher_data), 0); -- 2.39.3