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 E74E3A034C; Wed, 27 Apr 2022 16:59:17 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A9EB942816; Wed, 27 Apr 2022 16:58:39 +0200 (CEST) Received: from relay.smtp-ext.broadcom.com (lpdvsmtp11.broadcom.com [192.19.166.231]) by mails.dpdk.org (Postfix) with ESMTP id 6565042801 for ; Wed, 27 Apr 2022 16:58:38 +0200 (CEST) Received: from dhcp-10-123-153-22.dhcp.broadcom.net (bgccx-dev-host-lnx2.bec.broadcom.net [10.123.153.22]) by relay.smtp-ext.broadcom.com (Postfix) with ESMTP id E7CB2C0000E8; Wed, 27 Apr 2022 07:58:36 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 relay.smtp-ext.broadcom.com E7CB2C0000E8 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=broadcom.com; s=dkimrelay; t=1651071517; bh=vriGM81puhrUcIojX289dRs8UmSZJgidawiYwipHkg4=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=nYDr/8fhhzLSm0kO9/Epov6UJkPNI1Qr5FK0wMZd3Bw6IrPkE0Rdy5JNg0cAV9S0n qmIgTbfnTOdiIisbxByFRd4uUVqW7RyFe/F5VrUzrpUIgdiShNQvTbGJ3kpjVukZ42 QhgfHa2rwR5eE/Q01gXHOCeD/iCsgSBWGWctcZ8g= From: Kalesh A P To: dev@dpdk.org Cc: ajit.khaparde@broadcom.com Subject: [PATCH 06/17] net/bnxt: remove support for COUNT action Date: Wed, 27 Apr 2022 20:28:10 +0530 Message-Id: <20220427145821.5987-7-kalesh-anakkur.purayil@broadcom.com> X-Mailer: git-send-email 2.10.1 In-Reply-To: <20220427145821.5987-1-kalesh-anakkur.purayil@broadcom.com> References: <20220427145821.5987-1-kalesh-anakkur.purayil@broadcom.com> 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: Somnath Kotur 'Count' action was never really implemented in the legacy/AFM model. But there was some place holder code, remove it so that the user will see a failure when a flow with 'count' action is being created. Signed-off-by: Somnath Kotur Reviewed-by: Kalesh AP --- drivers/net/bnxt/bnxt_flow.c | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/drivers/net/bnxt/bnxt_flow.c b/drivers/net/bnxt/bnxt_flow.c index f7c90c4..71a8edd 100644 --- a/drivers/net/bnxt/bnxt_flow.c +++ b/drivers/net/bnxt/bnxt_flow.c @@ -1405,23 +1405,6 @@ bnxt_validate_and_parse_flow(struct rte_eth_dev *dev, bnxt_update_filter_flags_en(filter, filter1, use_ntuple); break; - case RTE_FLOW_ACTION_TYPE_COUNT: - vnic0 = &bp->vnic_info[0]; - filter1 = bnxt_get_l2_filter(bp, filter, vnic0); - if (filter1 == NULL) { - rte_flow_error_set(error, - ENOSPC, - RTE_FLOW_ERROR_TYPE_ACTION, - act, - "New filter not available"); - rc = -rte_errno; - goto ret; - } - - filter->fw_l2_filter_id = filter1->fw_l2_filter_id; - filter->flow_id = filter1->flow_id; - filter->flags = HWRM_CFA_NTUPLE_FILTER_ALLOC_INPUT_FLAGS_METER; - break; case RTE_FLOW_ACTION_TYPE_VF: act_vf = (const struct rte_flow_action_vf *)act->conf; vf = act_vf->id; -- 2.10.1