From patchwork Fri Aug 30 14:00:16 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sriharsha Basavapatna X-Patchwork-Id: 143476 X-Patchwork-Delegate: ajit.khaparde@broadcom.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 7F55F458A8; Fri, 30 Aug 2024 15:52:55 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0082A42EB7; Fri, 30 Aug 2024 15:52:12 +0200 (CEST) Received: from mail-pg1-f175.google.com (mail-pg1-f175.google.com [209.85.215.175]) by mails.dpdk.org (Postfix) with ESMTP id EA48C42EB5 for ; Fri, 30 Aug 2024 15:52:10 +0200 (CEST) Received: by mail-pg1-f175.google.com with SMTP id 41be03b00d2f7-7cda0453766so642222a12.2 for ; Fri, 30 Aug 2024 06:52:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; t=1725025930; x=1725630730; 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=97dgrWql6dfwkv7s/Ts+KKe5Nwr6ACf/byEuoC6WeZQ=; b=BrjcPCBpGurZVrHfXatlkzMve7NanCni7MbSyKv7ioZCruvJvYEmycvnvd1HwYituV a+jLp6bTG0OHFoRn4IJkzjgFwyy2gwz9sh1TTXe+zeQ4gBLoXxpjiTUYwOrvIDUuegN3 R9fXrH0HxFxVuPX2elQUYvZVhS0SMTaKvX+OY= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1725025930; x=1725630730; 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=97dgrWql6dfwkv7s/Ts+KKe5Nwr6ACf/byEuoC6WeZQ=; b=Qu7Jup1w12skoxDSC2BWlzOQkomBZPdN6fd6YVudq4Q5aC9pui5ailUTh7r5tP13wf deQUKRMcYfQ6hs1+3OAs3EQxvbCrzDtGO+xyq+DRumVC1CNKJ1gk65fJzsnyabQIAvYB BDnqBvCWxoiDnfI5STncn26lgtB4MgVGJJWsgLvaDnNa3NJJUiuS5A5j+Q81HMTVIpIu f4NTa8+TkL1vca1vXgM0ACzRlhWedZMVYFQPeApXAjpjF29kUzRK1pdkz7fCHT0fzKID eJ8pxJ12Rbq+M6RG4SqBDrhHJDkRc9vIaV+8RnjZZNWJICFAsIxEc6mj0l7VlBMev7F6 3fgg== X-Gm-Message-State: AOJu0YxO8J6XXTOzv+G5BoQ+yFwztUFWczvgiPlySLQsFSus40hihdKl g8Gu2oWr8qN0S9CdQXv9rNxPwaJXXLIl51fNLdIm9UoLTkIfWURwvTkrrMuiQMwySabaYjamVMH DYdnUtXoc82Mco5DRc1e9EcdWaGlhHR9hBbCOIHiTg5DOLI5q6bJzTZGBHm0xitqPz5CfG7+/Dd N19e7uE6SkEhiHKaIp/bdy4r9i42eMwzZWCEdeqrmYYLS6 X-Google-Smtp-Source: AGHT+IHPgQgeJJDWraNzUd+oTL5N3XX/fodm1h2sN5hsPwoOSe/38AcBvDxo4wfHeYra+IGw78a8Lw== X-Received: by 2002:a17:903:22c8:b0:203:a0ea:6265 with SMTP id d9443c01a7336-2050c33e473mr82502485ad.11.1725025929534; Fri, 30 Aug 2024 06:52:09 -0700 (PDT) Received: from dhcp-10-123-154-23.dhcp.broadcom.net ([192.19.234.250]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-205155658dfsm27067145ad.297.2024.08.30.06.52.07 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 30 Aug 2024 06:52:09 -0700 (PDT) From: Sriharsha Basavapatna To: dev@dpdk.org Cc: Kishore Padmanabha , Shahaji Bhosle , Sriharsha Basavapatna Subject: [PATCH 14/47] net/bnxt: tf_ulp: add support for vf to vf flow offload Date: Fri, 30 Aug 2024 19:30:16 +0530 Message-Id: <20240830140049.1715230-15-sriharsha.basavapatna@broadcom.com> X-Mailer: git-send-email 2.39.0.189.g4dbebc36b0 In-Reply-To: <20240830140049.1715230-1-sriharsha.basavapatna@broadcom.com> References: <20240830140049.1715230-1-sriharsha.basavapatna@broadcom.com> MIME-Version: 1.0 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 Added support for the vf to vf flow offload for the whitney platform. It includes the change of the pipeline from using vlan tags to using custom L2 encap and decap of the packets. Signed-off-by: Kishore Padmanabha Reviewed-by: Shahaji Bhosle Signed-off-by: Sriharsha Basavapatna --- drivers/net/bnxt/tf_ulp/bnxt_ulp.c | 5 ++++ drivers/net/bnxt/tf_ulp/bnxt_ulp_flow.c | 32 +++++++++++++++++++++++++ drivers/net/bnxt/tf_ulp/ulp_def_rules.c | 26 ++++++++++++++++++++ drivers/net/bnxt/tf_ulp/ulp_mapper.c | 6 +++++ drivers/net/bnxt/tf_ulp/ulp_port_db.c | 32 +++++++++++++++++++++++++ drivers/net/bnxt/tf_ulp/ulp_port_db.h | 13 ++++++++++ 6 files changed, 114 insertions(+) diff --git a/drivers/net/bnxt/tf_ulp/bnxt_ulp.c b/drivers/net/bnxt/tf_ulp/bnxt_ulp.c index 912946303a..3f0a5f2300 100644 --- a/drivers/net/bnxt/tf_ulp/bnxt_ulp.c +++ b/drivers/net/bnxt/tf_ulp/bnxt_ulp.c @@ -1927,6 +1927,11 @@ bnxt_ulp_port_init(struct bnxt *bp) } } + /* setup the l2 etype tunnel for custom l2 encap/decap */ + rc = ulp_l2_etype_tunnel_alloc(bp); + if (rc) + goto jump_to_error; + /* Update bnxt driver flags */ rc = ulp_dparms_dev_port_intf_update(bp, bp->ulp_ctx); if (rc) { diff --git a/drivers/net/bnxt/tf_ulp/bnxt_ulp_flow.c b/drivers/net/bnxt/tf_ulp/bnxt_ulp_flow.c index ad04644db4..2f159ae486 100644 --- a/drivers/net/bnxt/tf_ulp/bnxt_ulp_flow.c +++ b/drivers/net/bnxt/tf_ulp/bnxt_ulp_flow.c @@ -606,6 +606,38 @@ bnxt_ulp_action_handle_create(struct rte_eth_dev *dev, BNXT_ULP_DIR_EGRESS); } + /* perform the conversion from dpdk port to bnxt ifindex */ + if (ulp_port_db_dev_port_to_ulp_index(ulp_ctx, + dev->data->port_id, + &ifindex)) { + BNXT_TF_DBG(ERR, "Port id is not valid\n"); + goto parse_error; + } + port_type = ulp_port_db_port_type_get(ulp_ctx, ifindex); + if (port_type == BNXT_ULP_INTF_TYPE_INVALID) { + BNXT_TF_DBG(ERR, "Port type is not valid\n"); + goto parse_error; + } + + bnxt_ulp_init_parser_cf_defaults(¶ms, dev->data->port_id); + + /* Emulating the match port for direction processing */ + ULP_COMP_FLD_IDX_WR(¶ms, BNXT_ULP_CF_IDX_MATCH_PORT_TYPE, + port_type); + + if ((params.dir_attr & BNXT_ULP_FLOW_ATTR_INGRESS) && + port_type == BNXT_ULP_INTF_TYPE_VF_REP) { + ULP_COMP_FLD_IDX_WR(¶ms, BNXT_ULP_CF_IDX_DIRECTION, + BNXT_ULP_DIR_EGRESS); + } else { + /* Assign the input direction */ + if (params.dir_attr & BNXT_ULP_FLOW_ATTR_INGRESS) + ULP_COMP_FLD_IDX_WR(¶ms, BNXT_ULP_CF_IDX_DIRECTION, + BNXT_ULP_DIR_INGRESS); + else + ULP_COMP_FLD_IDX_WR(¶ms, BNXT_ULP_CF_IDX_DIRECTION, + BNXT_ULP_DIR_EGRESS); + } /* Parse the shared action */ ret = bnxt_ulp_rte_parser_act_parse(actions, ¶ms); if (ret != BNXT_TF_RC_SUCCESS) diff --git a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c index 8237dbd294..e2d281130e 100644 --- a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c +++ b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c @@ -12,6 +12,10 @@ #include "ulp_flow_db.h" #include "ulp_mapper.h" +static void +ulp_l2_custom_tunnel_id_update(struct bnxt *bp, + struct bnxt_ulp_mapper_create_parms *params); + struct bnxt_ulp_def_param_handler { int32_t (*vfr_func)(struct bnxt_ulp_context *ulp_ctx, struct ulp_tlv_param *param, @@ -306,6 +310,7 @@ ulp_default_flow_create(struct rte_eth_dev *eth_dev, struct ulp_rte_act_bitmap act = { 0 }; struct bnxt_ulp_context *ulp_ctx; uint32_t type, ulp_flags = 0, fid; + struct bnxt *bp = eth_dev->data->dev_private; int rc = 0; memset(&mapper_params, 0, sizeof(mapper_params)); @@ -366,6 +371,9 @@ ulp_default_flow_create(struct rte_eth_dev *eth_dev, ULP_COMP_FLD_IDX_WR(&mapper_params, BNXT_ULP_CF_IDX_VF_META_FID, BNXT_ULP_META_VF_FLAG | mapper_params.func_id); + /* update the upar id */ + ulp_l2_custom_tunnel_id_update(bp, &mapper_params); + BNXT_TF_DBG(DEBUG, "Creating default flow with template id: %u\n", ulp_class_tid); @@ -641,3 +649,21 @@ bnxt_ulp_delete_vfr_default_rules(struct bnxt_representor *vfr) memset(info, 0, sizeof(struct bnxt_ulp_vfr_rule_info)); return 0; } + +static void +ulp_l2_custom_tunnel_id_update(struct bnxt *bp, + struct bnxt_ulp_mapper_create_parms *params) +{ + if (!bp->l2_etype_tunnel_cnt) + return; + + if (bp->l2_etype_upar_in_use & + HWRM_TUNNEL_DST_PORT_QUERY_OUTPUT_UPAR_IN_USE_UPAR0) { + ULP_COMP_FLD_IDX_WR(params, BNXT_ULP_CF_IDX_L2_CUSTOM_UPAR_ID, + ULP_WP_SYM_TUN_HDR_TYPE_UPAR1); + } else if (bp->l2_etype_upar_in_use & + HWRM_TUNNEL_DST_PORT_QUERY_OUTPUT_UPAR_IN_USE_UPAR1) { + ULP_COMP_FLD_IDX_WR(params, BNXT_ULP_CF_IDX_L2_CUSTOM_UPAR_ID, + ULP_WP_SYM_TUN_HDR_TYPE_UPAR2); + } +} diff --git a/drivers/net/bnxt/tf_ulp/ulp_mapper.c b/drivers/net/bnxt/tf_ulp/ulp_mapper.c index ea1311c0c4..9dda70d75b 100644 --- a/drivers/net/bnxt/tf_ulp/ulp_mapper.c +++ b/drivers/net/bnxt/tf_ulp/ulp_mapper.c @@ -1073,6 +1073,12 @@ ulp_mapper_field_port_db_process(struct bnxt_ulp_mapper_parms *parms, return -EINVAL; } break; + case BNXT_ULP_PORT_TABLE_VF_FUNC_FID: + if (ulp_port_db_port_vf_fid_get(parms->ulp_ctx, port_id, val)) { + BNXT_TF_DBG(ERR, "Invalid port id %u\n", port_id); + return -EINVAL; + } + break; default: BNXT_TF_DBG(ERR, "Invalid port_data %d\n", port_data); return -EINVAL; diff --git a/drivers/net/bnxt/tf_ulp/ulp_port_db.c b/drivers/net/bnxt/tf_ulp/ulp_port_db.c index ba1f966ec3..bc0a7a95e6 100644 --- a/drivers/net/bnxt/tf_ulp/ulp_port_db.c +++ b/drivers/net/bnxt/tf_ulp/ulp_port_db.c @@ -760,3 +760,35 @@ ulp_port_db_port_meta_data_get(struct bnxt_ulp_context *ulp_ctxt, } return -EINVAL; } + +/* Api to get the function id for a given port id + * + * ulp_ctxt [in] Ptr to ulp context + * port_id [in] dpdk port id + * fid_data [out] the function id of the given port + * + * Returns 0 on success or negative number on failure. + */ +int32_t +ulp_port_db_port_vf_fid_get(struct bnxt_ulp_context *ulp_ctxt, + uint16_t port_id, uint8_t **fid_data) +{ + struct bnxt_ulp_port_db *port_db; + uint32_t ifindex; + + port_db = bnxt_ulp_cntxt_ptr2_port_db_get(ulp_ctxt); + if (!port_db || port_id >= RTE_MAX_ETHPORTS) { + BNXT_TF_DBG(ERR, "Invalid Arguments\n"); + return -EINVAL; + } + ifindex = port_db->dev_port_list[port_id]; + if (!ifindex) + return -ENOENT; + + if (port_db->ulp_intf_list[ifindex].type != BNXT_ULP_INTF_TYPE_VF && + port_db->ulp_intf_list[ifindex].type != BNXT_ULP_INTF_TYPE_VF_REP) + return -EINVAL; + + *fid_data = (uint8_t *)&port_db->ulp_intf_list[ifindex].vf_func_id; + return 0; +} diff --git a/drivers/net/bnxt/tf_ulp/ulp_port_db.h b/drivers/net/bnxt/tf_ulp/ulp_port_db.h index d4efe0a3d5..9d7ab02ad7 100644 --- a/drivers/net/bnxt/tf_ulp/ulp_port_db.h +++ b/drivers/net/bnxt/tf_ulp/ulp_port_db.h @@ -354,4 +354,17 @@ ulp_port_db_port_is_pf_get(struct bnxt_ulp_context *ulp_ctxt, int32_t ulp_port_db_port_meta_data_get(struct bnxt_ulp_context *ulp_ctxt, uint16_t port_id, uint8_t **meta_data); + +/* Api to get the function id for a given port id + * + * ulp_ctxt [in] Ptr to ulp context + * port_id [in] dpdk port id + * fid_data [out] the function id of the given port + * + * Returns 0 on success or negative number on failure. + */ +int32_t +ulp_port_db_port_vf_fid_get(struct bnxt_ulp_context *ulp_ctxt, + uint16_t port_id, uint8_t **fid_data); + #endif /* _ULP_PORT_DB_H_ */