diff mbox series

[v2,42/51] net/bnxt: manage VF to VFR conduit

Message ID	20200701065212.41391-43-ajit.khaparde@broadcom.com (mailing list archive)
State	Changes Requested, archived
Delegated to:	Ajit Khaparde
Headers	show Return-Path: <dev-bounces@dpdk.org> From: Ajit Khaparde <ajit.khaparde@broadcom.com> To: dev@dpdk.org Cc: Venkat Duvvuru <venkatkumar.duvvuru@broadcom.com>, Somnath Kotur <somnath.kotur@broadcom.com> Date: Tue, 30 Jun 2020 23:52:03 -0700 Message-Id: <20200701065212.41391-43-ajit.khaparde@broadcom.com> In-Reply-To: <20200701065212.41391-1-ajit.khaparde@broadcom.com> References: <20200612132934.16488-1-somnath.kotur@broadcom.com> <20200701065212.41391-1-ajit.khaparde@broadcom.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH v2 42/51] net/bnxt: manage VF to VFR conduit Precedence: list Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org>
Series	add features for host-based flow management \| expand [v2,00/51] add features for host-based flow management [v2,01/51] net/bnxt: add basic infrastructure for VF representors [v2,02/51] net/bnxt: add support for VF-reps data path [v2,03/51] net/bnxt: get IDs for VF-Rep endpoint [v2,04/51] net/bnxt: initialize parent PF information [v2,05/51] net/bnxt: modify port db dev interface [v2,06/51] net/bnxt: get port and function info [v2,07/51] net/bnxt: add support for hwrm port phy qcaps [v2,08/51] net/bnxt: modify port db to handle more info [v2,09/51] net/bnxt: add support for exact match [v2,10/51] net/bnxt: modify EM insert and delete to use HWRM direct [v2,11/51] net/bnxt: add multi device support [v2,12/51] net/bnxt: support bulk table get and mirror [v2,13/51] net/bnxt: update multi device design support [v2,14/51] net/bnxt: support two-level priority for TCAMs [v2,15/51] net/bnxt: add HCAPI interface support [v2,16/51] net/bnxt: add core changes for EM and EEM lookups [v2,17/51] net/bnxt: implement support for TCAM access [v2,18/51] net/bnxt: multiple device implementation [v2,19/51] net/bnxt: update identifier with remap support [v2,20/51] net/bnxt: update RM with residual checker [v2,21/51] net/bnxt: support two level priority for TCAMs [v2,22/51] net/bnxt: support EM and TCAM lookup with table scope [v2,23/51] net/bnxt: update table get to use new design [v2,24/51] net/bnxt: update RM to support HCAPI only [v2,25/51] net/bnxt: remove table scope from session [v2,26/51] net/bnxt: add external action alloc and free [v2,27/51] net/bnxt: align CFA resources with RM [v2,28/51] net/bnxt: implement IF tables set and get [v2,29/51] net/bnxt: add TF register and unregister [v2,30/51] net/bnxt: add global config set and get APIs [v2,31/51] net/bnxt: add support for EEM System memory [v2,32/51] net/bnxt: integrate with the latest tf core changes [v2,33/51] net/bnxt: add support for internal encap records [v2,34/51] net/bnxt: add support for if table processing [v2,35/51] net/bnxt: disable Tx vector mode if truflow is enabled [v2,36/51] net/bnxt: add index opcode and operand to mapper table [v2,37/51] net/bnxt: add support for global resource templates [v2,38/51] net/bnxt: add support for internal exact match entries [v2,39/51] net/bnxt: add support for conditional execution of mapper tables [v2,40/51] net/bnxt: enable port MAC qcfg command for trusted VF [v2,41/51] net/bnxt: enhancements for port db [v2,42/51] net/bnxt: manage VF to VFR conduit [v2,43/51] net/bnxt: parse representor along with other dev-args [v2,44/51] net/bnxt: fill mapper parameters with default rules info [v2,45/51] net/bnxt: add VF-rep and stat templates [v2,46/51] net/bnxt: create default flow rules for the VF-rep conduit [v2,47/51] net/bnxt: add port default rules for ingress and egress [v2,48/51] net/bnxt: fill cfa action in the Tx descriptor [v2,49/51] net/bnxt: add ULP Flow counter Manager [v2,50/51] net/bnxt: add support for count action in flow query [v2,51/51] doc: update release notes

Checks

Context	Check	Description
ci/Intel-compilation	fail	Compilation issues
ci/checkpatch	success	coding style OK

Commit Message

Ajit Khaparde July 1, 2020, 6:52 a.m. UTC

From: Venkat Duvvuru <venkatkumar.duvvuru@broadcom.com>

When VF-VFR conduits are created, a mark is added to the mark database.
mark_flag indicates whether the mark is valid and has VFR information
(VFR_ID bit in mark_flag). Rx path was checking for this VFR_ID bit.
However, while adding the mark to the mark database, VFR_ID bit is not
set in mark_flag.

Signed-off-by: Venkat Duvvuru <venkatkumar.duvvuru@broadcom.com>
Reviewed-by: Somnath Kotur <somnath.kotur@broadcom.com>
Reviewed-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
---
 drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c | 5 +++++
 1 file changed, 5 insertions(+)

diff mbox series

Patch

diff --git a/drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c b/drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c
index b3527eccb..b2c8c349c 100644
--- a/drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c
+++ b/drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c
@@ -18,6 +18,8 @@ 
 						BNXT_ULP_MARK_VALID)
 #define ULP_MARK_DB_ENTRY_IS_INVALID(mark_info) (!((mark_info)->flags &\
 						   BNXT_ULP_MARK_VALID))
+#define ULP_MARK_DB_ENTRY_SET_VFR_ID(mark_info) ((mark_info)->flags |=\
+						 BNXT_ULP_MARK_VFR_ID)
 #define ULP_MARK_DB_ENTRY_IS_VFR_ID(mark_info) ((mark_info)->flags &\
 						BNXT_ULP_MARK_VFR_ID)
 #define ULP_MARK_DB_ENTRY_IS_GLOBAL_HW_FID(mark_info) ((mark_info)->flags &\
@@ -263,6 +265,9 @@  ulp_mark_db_mark_add(struct bnxt_ulp_context *ctxt,
 		BNXT_TF_DBG(DEBUG, "Set LFID[0x%0x] = 0x%0x\n", fid, mark);
 		mtbl->lfid_tbl[fid].mark_id = mark;
 		ULP_MARK_DB_ENTRY_SET_VALID(&mtbl->lfid_tbl[fid]);
+
+		if (mark_flag & BNXT_ULP_MARK_VFR_ID)
+			ULP_MARK_DB_ENTRY_SET_VFR_ID(&mtbl->lfid_tbl[fid]);
 	}
 
 	return 0;