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 F00DCA0C41; Fri, 5 Nov 2021 22:54:49 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 818F941130; Fri, 5 Nov 2021 22:54:35 +0100 (CET) Received: from shelob.oktetlabs.ru (shelob.oktetlabs.ru [91.220.146.113]) by mails.dpdk.org (Postfix) with ESMTP id 849EC40DFD for ; Fri, 5 Nov 2021 22:54:31 +0100 (CET) Received: from bree.oktetlabs.ru (bree.oktetlabs.ru [192.168.34.5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by shelob.oktetlabs.ru (Postfix) with ESMTPS id 3A18F7F52E; Sat, 6 Nov 2021 00:54:31 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 shelob.oktetlabs.ru 3A18F7F52E Authentication-Results: shelob.oktetlabs.ru/3A18F7F52E; dkim=none; dkim-atps=neutral From: Ivan Malov To: dev@dpdk.org Cc: Andrew Rybchenko , Andy Moreton Date: Sat, 6 Nov 2021 00:54:07 +0300 Message-Id: <20211105215409.5706-4-ivan.malov@oktetlabs.ru> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20211105215409.5706-1-ivan.malov@oktetlabs.ru> References: <20211105215409.5706-1-ivan.malov@oktetlabs.ru> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH 3/5] common/sfc_efx/base: factor out no-op helper functions 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 Sender: "dev" When an action gets added to an action set, a special helper is used to handle its arguments. There are actions which have no arguments, and the corresponding helpers are duplicates in fact. Use a unified no-op helper instead of them. Signed-off-by: Ivan Malov Reviewed-by: Andrew Rybchenko Reviewed-by: Andy Moreton --- drivers/common/sfc_efx/base/efx_mae.c | 80 ++------------------------- 1 file changed, 4 insertions(+), 76 deletions(-) diff --git a/drivers/common/sfc_efx/base/efx_mae.c b/drivers/common/sfc_efx/base/efx_mae.c index c93fe9bdfc..41444e1926 100644 --- a/drivers/common/sfc_efx/base/efx_mae.c +++ b/drivers/common/sfc_efx/base/efx_mae.c @@ -1406,7 +1406,7 @@ efx_mae_action_set_spec_fini( } static __checkReturn efx_rc_t -efx_mae_action_set_add_decap( +efx_mae_action_set_no_op( __in efx_mae_actions_t *spec, __in size_t arg_size, __in_bcount(arg_size) const uint8_t *arg) @@ -1510,47 +1510,6 @@ efx_mae_action_set_add_vlan_push( return (rc); } -static __checkReturn efx_rc_t -efx_mae_action_set_add_encap( - __in efx_mae_actions_t *spec, - __in size_t arg_size, - __in_bcount(arg_size) const uint8_t *arg) -{ - efx_rc_t rc; - - /* - * Adding this specific action to an action set spec and setting encap. - * header ID in the spec are two individual steps. This design allows - * the client driver to avoid encap. header allocation when it simply - * needs to check the order of actions submitted by user ("validate"), - * without actually allocating an action set and inserting a rule. - * - * In order to fill in the encap. header ID, the caller is supposed to - * invoke efx_mae_action_set_fill_in_eh_id(). If they do not do that, - * efx_mae_action_set_alloc() invocation will throw an error. - * - * For now, no more work is supposed to be done. - */ - - if (arg_size != 0) { - rc = EINVAL; - goto fail1; - } - - if (arg != NULL) { - rc = EINVAL; - goto fail2; - } - - return (0); - -fail2: - EFSYS_PROBE(fail2); -fail1: - EFSYS_PROBE1(fail1, efx_rc_t, rc); - return (rc); -} - static __checkReturn efx_rc_t efx_mae_action_set_add_count( __in efx_mae_actions_t *spec, @@ -1596,37 +1555,6 @@ efx_mae_action_set_add_count( return (rc); } -static __checkReturn efx_rc_t -efx_mae_action_set_add_flag( - __in efx_mae_actions_t *spec, - __in size_t arg_size, - __in_bcount(arg_size) const uint8_t *arg) -{ - efx_rc_t rc; - - _NOTE(ARGUNUSED(spec)) - - if (arg_size != 0) { - rc = EINVAL; - goto fail1; - } - - if (arg != NULL) { - rc = EINVAL; - goto fail2; - } - - /* This action does not have any arguments, so do nothing here. */ - - return (0); - -fail2: - EFSYS_PROBE(fail2); -fail1: - EFSYS_PROBE1(fail1, efx_rc_t, rc); - return (rc); -} - static __checkReturn efx_rc_t efx_mae_action_set_add_mark( __in efx_mae_actions_t *spec, @@ -1693,7 +1621,7 @@ typedef struct efx_mae_action_desc_s { static const efx_mae_action_desc_t efx_mae_actions[EFX_MAE_NACTIONS] = { [EFX_MAE_ACTION_DECAP] = { - .emad_add = efx_mae_action_set_add_decap + .emad_add = efx_mae_action_set_no_op }, [EFX_MAE_ACTION_VLAN_POP] = { .emad_add = efx_mae_action_set_add_vlan_pop @@ -1702,13 +1630,13 @@ static const efx_mae_action_desc_t efx_mae_actions[EFX_MAE_NACTIONS] = { .emad_add = efx_mae_action_set_add_vlan_push }, [EFX_MAE_ACTION_ENCAP] = { - .emad_add = efx_mae_action_set_add_encap + .emad_add = efx_mae_action_set_no_op }, [EFX_MAE_ACTION_COUNT] = { .emad_add = efx_mae_action_set_add_count }, [EFX_MAE_ACTION_FLAG] = { - .emad_add = efx_mae_action_set_add_flag + .emad_add = efx_mae_action_set_no_op }, [EFX_MAE_ACTION_MARK] = { .emad_add = efx_mae_action_set_add_mark -- 2.30.2