From patchwork Sat Oct 10 04:11:52 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ajit Khaparde X-Patchwork-Id: 80266 X-Patchwork-Delegate: ajit.khaparde@broadcom.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 51AB3A04BC; Sat, 10 Oct 2020 06:16:34 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id D173D1D8D6; Sat, 10 Oct 2020 06:13:21 +0200 (CEST) Received: from mail-pf1-f228.google.com (mail-pf1-f228.google.com [209.85.210.228]) by dpdk.org (Postfix) with ESMTP id 8151A1D6D8 for ; Sat, 10 Oct 2020 06:13:16 +0200 (CEST) Received: by mail-pf1-f228.google.com with SMTP id n14so8542868pff.6 for ; Fri, 09 Oct 2020 21:13:16 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=DiwToIxbh+1ELIlUO1EQBDYlGl/1CGMgwO8BzvnzS4I=; b=GRqYbDs9HekoNo0jiE6Vh3enNyui7NdTr6pSDkWkjCceBsfABeu9ULeVfOiNs/SQny vIg/4hKBjcK2uw0cF3yl/hKLVNH/mMvTllIPTIOWTLEY6/JVqm5HgRkYeJp88tpFnrzI yB/KC2ks+r/zxiRVgHoReusf6NHyziMZ7Qlf4= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=DiwToIxbh+1ELIlUO1EQBDYlGl/1CGMgwO8BzvnzS4I=; b=kVvKJYNEQZX+pOq4+1TMbEA6AG0p0M1Xc11AycGNOYXK9V0PxZ/Vql7+WwWZrnp/3t vkSVyJNHDV6l5/PjuyXbogBCpHpyLwtEBoyNMZWs/aj5DTR15mmcFhXkYn915nd15xlu KbUiGEtkOeTnwxM6cIELrd+F18Cw63o1F3bfy6wCesFr+wVsF7CG4jpAL0ym/A9hyM9w MS+P30hSfUZif7TvvhDPSNR5NZ/l4eA/TuHOL/tKb5JkR85UgtD8FfDl6RYtu238rEFR A0tVchtxWWaRupKKy6pLiWzjywCI3z/f0g+Rli0eq8MAGT/dDOUfdLOKUwI7HH0PRVXL MbAw== X-Gm-Message-State: AOAM530sn3i3JllVckUl0UxUcjzv4g3cLOXclrwwKcxGbqfTVzhxIinN QMOcsN3oAuyx/AZ8EwqvNqyreJ+6tqelMs/1kAkpVHoQpvWhEjuG2ERKooiJKRZkMSRFG5JT1+j 447Udk4UuMgDXsX4l4pkeOAPPafLSC7dTeZ4HvqrnK34knET0gb54/ZEAbLUdq16Dv3I2o78O3i CpeQ== X-Google-Smtp-Source: ABdhPJwTn1bubceXozHI7XS1aPtpC/tjBCl3U9EeCMyArb3CQ46i1c0+vaSG6YqapKqCHd9UbKrMZ61sBaYi X-Received: by 2002:a62:fb09:0:b029:152:117a:facd with SMTP id x9-20020a62fb090000b0290152117afacdmr15162055pfm.22.1602303195565; Fri, 09 Oct 2020 21:13:15 -0700 (PDT) Received: from localhost.localdomain ([192.19.223.252]) by smtp-relay.gmail.com with ESMTPS id b4sm1597022pjn.3.2020.10.09.21.13.15 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Fri, 09 Oct 2020 21:13:15 -0700 (PDT) X-Relaying-Domain: broadcom.com From: Ajit Khaparde To: dev@dpdk.org Cc: Mike Baucom , Kishore Padmanabha Date: Fri, 9 Oct 2020 21:11:52 -0700 Message-Id: <20201010041153.63921-12-ajit.khaparde@broadcom.com> X-Mailer: git-send-email 2.21.1 (Apple Git-122.3) In-Reply-To: <20201010041153.63921-1-ajit.khaparde@broadcom.com> References: <20201009111130.10422-1-somnath.kotur@broadcom.com> <20201010041153.63921-1-ajit.khaparde@broadcom.com> MIME-Version: 1.0 Subject: [dpdk-dev] [PATCH v2 11/12] net/bnxt: remove flow db table type from templates X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" From: Mike Baucom FDB type is now driven by the caller, not the template. So remove it. Signed-off-by: Mike Baucom Reviewed-by: Ajit Khaparde Reviewed-by: Kishore Padmanabha --- .../tf_ulp/ulp_template_db_stingray_act.c | 18 ++--- .../tf_ulp/ulp_template_db_stingray_class.c | 69 +++++++------------ .../bnxt/tf_ulp/ulp_template_db_wh_plus_act.c | 18 ++--- .../tf_ulp/ulp_template_db_wh_plus_class.c | 69 +++++++------------ drivers/net/bnxt/tf_ulp/ulp_template_struct.h | 1 - 5 files changed, 58 insertions(+), 117 deletions(-) diff --git a/drivers/net/bnxt/tf_ulp/ulp_template_db_stingray_act.c b/drivers/net/bnxt/tf_ulp/ulp_template_db_stingray_act.c index 68e4d8e59..2237ffb94 100644 --- a/drivers/net/bnxt/tf_ulp/ulp_template_db_stingray_act.c +++ b/drivers/net/bnxt/tf_ulp/ulp_template_db_stingray_act.c @@ -12,38 +12,32 @@ struct bnxt_ulp_mapper_tbl_list_info ulp_stingray_act_tmpl_list[] = { [1] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 6, - .start_tbl_idx = 0, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 0 }, [2] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 3, - .start_tbl_idx = 6, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 6 }, [3] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 3, - .start_tbl_idx = 9, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 9 }, [4] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 6, - .start_tbl_idx = 12, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 12 }, [5] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 6, - .start_tbl_idx = 18, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 18 }, [6] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 5, - .start_tbl_idx = 24, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 24 } }; diff --git a/drivers/net/bnxt/tf_ulp/ulp_template_db_stingray_class.c b/drivers/net/bnxt/tf_ulp/ulp_template_db_stingray_class.c index 1fa364e29..62b940daa 100644 --- a/drivers/net/bnxt/tf_ulp/ulp_template_db_stingray_class.c +++ b/drivers/net/bnxt/tf_ulp/ulp_template_db_stingray_class.c @@ -12,140 +12,117 @@ struct bnxt_ulp_mapper_tbl_list_info ulp_stingray_class_tmpl_list[] = { [1] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 6, - .start_tbl_idx = 0, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_DEFAULT + .start_tbl_idx = 0 }, [2] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 7, - .start_tbl_idx = 6, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_DEFAULT + .start_tbl_idx = 6 }, [3] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 7, - .start_tbl_idx = 13, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_DEFAULT + .start_tbl_idx = 13 }, [4] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 7, - .start_tbl_idx = 20, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_DEFAULT + .start_tbl_idx = 20 }, [5] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 1, - .start_tbl_idx = 27, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_DEFAULT + .start_tbl_idx = 27 }, [6] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 5, - .start_tbl_idx = 28, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 28 }, [7] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 5, - .start_tbl_idx = 33, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 33 }, [8] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 6, - .start_tbl_idx = 38, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 38 }, [9] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 6, - .start_tbl_idx = 44, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 44 }, [10] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 6, - .start_tbl_idx = 50, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 50 }, [11] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 6, - .start_tbl_idx = 56, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 56 }, [12] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 5, - .start_tbl_idx = 62, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 62 }, [13] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 5, - .start_tbl_idx = 67, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 67 }, [14] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 5, - .start_tbl_idx = 72, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 72 }, [15] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 5, - .start_tbl_idx = 77, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 77 }, [16] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 5, - .start_tbl_idx = 82, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 82 }, [17] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 5, - .start_tbl_idx = 87, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 87 }, [18] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 6, - .start_tbl_idx = 92, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 92 }, [19] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 6, - .start_tbl_idx = 98, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 98 }, [20] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 6, - .start_tbl_idx = 104, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 104 }, [21] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 6, - .start_tbl_idx = 110, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 110 }, [22] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 5, - .start_tbl_idx = 116, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 116 }, [23] = { .device_name = BNXT_ULP_DEVICE_ID_STINGRAY, .num_tbls = 5, - .start_tbl_idx = 121, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 121 } }; diff --git a/drivers/net/bnxt/tf_ulp/ulp_template_db_wh_plus_act.c b/drivers/net/bnxt/tf_ulp/ulp_template_db_wh_plus_act.c index 381278ee9..39e8ec40b 100644 --- a/drivers/net/bnxt/tf_ulp/ulp_template_db_wh_plus_act.c +++ b/drivers/net/bnxt/tf_ulp/ulp_template_db_wh_plus_act.c @@ -12,38 +12,32 @@ struct bnxt_ulp_mapper_tbl_list_info ulp_wh_plus_act_tmpl_list[] = { [1] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 6, - .start_tbl_idx = 0, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 0 }, [2] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 3, - .start_tbl_idx = 6, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 6 }, [3] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 3, - .start_tbl_idx = 9, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 9 }, [4] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 6, - .start_tbl_idx = 12, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 12 }, [5] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 6, - .start_tbl_idx = 18, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 18 }, [6] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 5, - .start_tbl_idx = 24, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 24 } }; diff --git a/drivers/net/bnxt/tf_ulp/ulp_template_db_wh_plus_class.c b/drivers/net/bnxt/tf_ulp/ulp_template_db_wh_plus_class.c index 7024658ce..470d91ce2 100644 --- a/drivers/net/bnxt/tf_ulp/ulp_template_db_wh_plus_class.c +++ b/drivers/net/bnxt/tf_ulp/ulp_template_db_wh_plus_class.c @@ -12,140 +12,117 @@ struct bnxt_ulp_mapper_tbl_list_info ulp_wh_plus_class_tmpl_list[] = { [1] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 6, - .start_tbl_idx = 0, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_DEFAULT + .start_tbl_idx = 0 }, [2] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 7, - .start_tbl_idx = 6, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_DEFAULT + .start_tbl_idx = 6 }, [3] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 7, - .start_tbl_idx = 13, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_DEFAULT + .start_tbl_idx = 13 }, [4] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 7, - .start_tbl_idx = 20, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_DEFAULT + .start_tbl_idx = 20 }, [5] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 1, - .start_tbl_idx = 27, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_DEFAULT + .start_tbl_idx = 27 }, [6] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 5, - .start_tbl_idx = 28, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 28 }, [7] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 5, - .start_tbl_idx = 33, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 33 }, [8] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 6, - .start_tbl_idx = 38, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 38 }, [9] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 6, - .start_tbl_idx = 44, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 44 }, [10] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 6, - .start_tbl_idx = 50, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 50 }, [11] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 6, - .start_tbl_idx = 56, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 56 }, [12] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 5, - .start_tbl_idx = 62, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 62 }, [13] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 5, - .start_tbl_idx = 67, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 67 }, [14] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 5, - .start_tbl_idx = 72, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 72 }, [15] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 5, - .start_tbl_idx = 77, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 77 }, [16] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 5, - .start_tbl_idx = 82, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 82 }, [17] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 5, - .start_tbl_idx = 87, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 87 }, [18] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 6, - .start_tbl_idx = 92, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 92 }, [19] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 6, - .start_tbl_idx = 98, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 98 }, [20] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 6, - .start_tbl_idx = 104, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 104 }, [21] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 6, - .start_tbl_idx = 110, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 110 }, [22] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 5, - .start_tbl_idx = 116, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 116 }, [23] = { .device_name = BNXT_ULP_DEVICE_ID_WH_PLUS, .num_tbls = 5, - .start_tbl_idx = 121, - .flow_db_table_type = BNXT_ULP_FDB_TYPE_REGULAR + .start_tbl_idx = 121 } }; diff --git a/drivers/net/bnxt/tf_ulp/ulp_template_struct.h b/drivers/net/bnxt/tf_ulp/ulp_template_struct.h index 107a06a3d..69bb61e11 100644 --- a/drivers/net/bnxt/tf_ulp/ulp_template_struct.h +++ b/drivers/net/bnxt/tf_ulp/ulp_template_struct.h @@ -175,7 +175,6 @@ struct bnxt_ulp_mapper_tbl_list_info { uint32_t device_name; uint32_t start_tbl_idx; uint32_t num_tbls; - enum bnxt_ulp_fdb_type flow_db_table_type; }; struct bnxt_ulp_mapper_tbl_info {