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 6787242D23; Fri, 23 Jun 2023 09:24:43 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id DF6AE40E09; Fri, 23 Jun 2023 09:24:42 +0200 (CEST) Received: from agw.arknetworks.am (agw.arknetworks.am [79.141.165.80]) by mails.dpdk.org (Postfix) with ESMTP id 9D2B4400EF for ; Fri, 23 Jun 2023 09:24:40 +0200 (CEST) Received: from debian (unknown [78.109.72.176]) (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 agw.arknetworks.am (Postfix) with ESMTPSA id B9B50E11CD; Fri, 23 Jun 2023 11:24:39 +0400 (+04) Date: Fri, 23 Jun 2023 11:24:38 +0400 (+04) From: Ivan Malov To: Artemii Morozov cc: dev@dpdk.org, Viacheslav Galaktionov , Andrew Rybchenko Subject: Re: [PATCH v8 2/4] common/sfc_efx/base: add API to get installed filters count In-Reply-To: <20230623054756.8896-3-artemii.morozov@arknetworks.am> Message-ID: <17e5b9a-53cb-ded-9950-bf27f9480df@arknetworks.am> References: <20230531134122.119508-1-artemii.morozov@arknetworks.am> <20230623054756.8896-1-artemii.morozov@arknetworks.am> <20230623054756.8896-3-artemii.morozov@arknetworks.am> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed 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 On Fri, 23 Jun 2023, Artemii Morozov wrote: > This API allows to get number of installed filters. This will > be used in the future patches. > > Signed-off-by: Artemii Morozov > Acked-by: Andrew Rybchenko Acked-by: Ivan Malov Thank you. > --- > drivers/common/sfc_efx/base/ef10_filter.c | 20 ++++++++++++++ > drivers/common/sfc_efx/base/ef10_impl.h | 6 +++++ > drivers/common/sfc_efx/base/efx_filter.c | 32 +++++++++++++++++++++++ > drivers/common/sfc_efx/base/efx_impl.h | 7 +++++ > 4 files changed, 65 insertions(+) > > diff --git a/drivers/common/sfc_efx/base/ef10_filter.c b/drivers/common/sfc_efx/base/ef10_filter.c > index d6940011c0..8ceaf98a5f 100644 > --- a/drivers/common/sfc_efx/base/ef10_filter.c > +++ b/drivers/common/sfc_efx/base/ef10_filter.c > @@ -2113,6 +2113,26 @@ ef10_filter_reconfigure( > return (rc); > } > > + __checkReturn efx_rc_t > +ef10_filter_get_count( > + __in efx_nic_t *enp, > + __out uint32_t *countp) > +{ > + ef10_filter_table_t *table = enp->en_filter.ef_ef10_filter_table; > + uint32_t filter_count; > + > + EFSYS_ASSERT(EFX_FAMILY_IS_EF100(enp) || EFX_FAMILY_IS_EF10(enp)); > + EFSYS_ASSERT(countp != NULL); > + > + filter_count = table->eft_unicst_filter_count + > + table->eft_mulcst_filter_count + > + table->eft_encap_filter_count; > + > + *countp = filter_count; > + > + return (0); > +} > + > void > ef10_filter_get_default_rxq( > __in efx_nic_t *enp, > diff --git a/drivers/common/sfc_efx/base/ef10_impl.h b/drivers/common/sfc_efx/base/ef10_impl.h > index 877aedad45..3476f274ce 100644 > --- a/drivers/common/sfc_efx/base/ef10_impl.h > +++ b/drivers/common/sfc_efx/base/ef10_impl.h > @@ -1347,6 +1347,12 @@ ef10_filter_reconfigure( > __in_ecount(6*count) uint8_t const *addrs, > __in uint32_t count); > > +LIBEFX_INTERNAL > +extern __checkReturn efx_rc_t > +ef10_filter_get_count( > + __in efx_nic_t *enp, > + __out uint32_t *countp); > + > LIBEFX_INTERNAL > extern void > ef10_filter_get_default_rxq( > diff --git a/drivers/common/sfc_efx/base/efx_filter.c b/drivers/common/sfc_efx/base/efx_filter.c > index 83c37ff859..2c8c7bdc33 100644 > --- a/drivers/common/sfc_efx/base/efx_filter.c > +++ b/drivers/common/sfc_efx/base/efx_filter.c > @@ -53,6 +53,7 @@ static const efx_filter_ops_t __efx_filter_siena_ops = { > siena_filter_delete, /* efo_delete */ > siena_filter_supported_filters, /* efo_supported_filters */ > NULL, /* efo_reconfigure */ > + NULL, /* efo_get_count */ > }; > #endif /* EFSYS_OPT_SIENA */ > > @@ -65,6 +66,7 @@ static const efx_filter_ops_t __efx_filter_ef10_ops = { > ef10_filter_delete, /* efo_delete */ > ef10_filter_supported_filters, /* efo_supported_filters */ > ef10_filter_reconfigure, /* efo_reconfigure */ > + ef10_filter_get_count, /* efo_get_count */ > }; > #endif /* EFX_OPTS_EF10() */ > > @@ -77,6 +79,7 @@ static const efx_filter_ops_t __efx_filter_rhead_ops = { > ef10_filter_delete, /* efo_delete */ > ef10_filter_supported_filters, /* efo_supported_filters */ > ef10_filter_reconfigure, /* efo_reconfigure */ > + ef10_filter_get_count, /* efo_get_count */ > }; > #endif /* EFSYS_OPT_RIVERHEAD */ > > @@ -309,6 +312,35 @@ efx_filter_reconfigure( > fail1: > EFSYS_PROBE1(fail1, efx_rc_t, rc); > > + return (rc); > +} > + > + __checkReturn efx_rc_t > +efx_filter_get_count( > + __in efx_nic_t *enp, > + __out uint32_t *countp) > +{ > + efx_rc_t rc; > + > + EFSYS_ASSERT3U(enp->en_magic, ==, EFX_NIC_MAGIC); > + EFSYS_ASSERT3U(enp->en_mod_flags, &, EFX_MOD_PROBE); > + EFSYS_ASSERT3U(enp->en_mod_flags, &, EFX_MOD_FILTER); > + > + if (enp->en_efop->efo_get_count == NULL) { > + rc = ENOTSUP; > + goto fail1; > + } > + > + if ((rc = enp->en_efop->efo_get_count(enp, countp)) != 0) > + goto fail2; > + > + return (0); > + > +fail2: > + EFSYS_PROBE(fail2); > +fail1: > + EFSYS_PROBE1(fail1, efx_rc_t, rc); > + > return (rc); > } > > diff --git a/drivers/common/sfc_efx/base/efx_impl.h b/drivers/common/sfc_efx/base/efx_impl.h > index 92a30c34ae..9a387a7efc 100644 > --- a/drivers/common/sfc_efx/base/efx_impl.h > +++ b/drivers/common/sfc_efx/base/efx_impl.h > @@ -288,6 +288,7 @@ typedef struct efx_filter_ops_s { > efx_rc_t (*efo_reconfigure)(efx_nic_t *, uint8_t const *, boolean_t, > boolean_t, boolean_t, boolean_t, > uint8_t const *, uint32_t); > + efx_rc_t (*efo_get_count)(efx_nic_t *, uint32_t *); > } efx_filter_ops_t; > > LIBEFX_INTERNAL > @@ -302,6 +303,12 @@ efx_filter_reconfigure( > __in_ecount(6*count) uint8_t const *addrs, > __in uint32_t count); > > +LIBEFX_INTERNAL > +extern __checkReturn efx_rc_t > +efx_filter_get_count( > + __in efx_nic_t *enp, > + __out uint32_t *countp); > + > #endif /* EFSYS_OPT_FILTER */ > > #if EFSYS_OPT_TUNNEL > -- > 2.34.1 > >