From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerin Jacob Subject: Re: [PATCH] examples/l3fwd: em path hash offload to machine Date: Wed, 5 Oct 2016 17:22:45 +0530 Message-ID: <20161005115243.GA7027@localhost.localdomain> References: <1471964080-29958-1-git-send-email-hemant.agrawal@nxp.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Cc: To: Hemant Agrawal Return-path: Received: from NAM02-BL2-obe.outbound.protection.outlook.com (mail-bl2nam02on0084.outbound.protection.outlook.com [104.47.38.84]) by dpdk.org (Postfix) with ESMTP id 480058D3A for ; Wed, 5 Oct 2016 13:53:07 +0200 (CEST) Content-Disposition: inline In-Reply-To: <1471964080-29958-1-git-send-email-hemant.agrawal@nxp.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Tue, Aug 23, 2016 at 08:24:39PM +0530, Hemant Agrawal wrote: Maybe you can change the subject line to: examples/l3fwd: em: use hw accelerated crc hash function for arm64 instead of: examples/l3fwd: em path hash offload to machine > if machine level CRC extension are available, offload the > hash to machine provided functions e.g. armv8-a CRC extensions > support it > > Signed-off-by: Hemant Agrawal > --- > examples/l3fwd/l3fwd_em.c | 20 ++++++++++---------- > 1 file changed, 10 insertions(+), 10 deletions(-) > > diff --git a/examples/l3fwd/l3fwd_em.c b/examples/l3fwd/l3fwd_em.c > index def5a02..a889c67 100644 > --- a/examples/l3fwd/l3fwd_em.c > +++ b/examples/l3fwd/l3fwd_em.c > @@ -58,13 +58,13 @@ > > #include "l3fwd.h" > > -#ifdef RTE_MACHINE_CPUFLAG_SSE4_2 > +#if defined(RTE_MACHINE_CPUFLAG_SSE4_2) || defined(RTE_MACHINE_CPUFLAG_CRC32) Rather than adding new compilation flag everywhere, Maybe you can add #if defined(RTE_MACHINE_CPUFLAG_SSE4_2) && defined(RTE_MACHINE_CPUFLAG_CRC32) #define EM_HASH_CRC 1 #endif something like above to reduce the change for future platforms with crc support. Other than that, you can add: Reviewed-by: Jerin Jacob > #include > #define DEFAULT_HASH_FUNC rte_hash_crc > #else > #include > #define DEFAULT_HASH_FUNC rte_jhash > -#endif /* RTE_MACHINE_CPUFLAG_SSE4_2 */ > +#endif > > #define IPV6_ADDR_LEN 16 > > @@ -169,17 +169,17 @@ ipv4_hash_crc(const void *data, __rte_unused uint32_t data_len, > t = k->proto; > p = (const uint32_t *)&k->port_src; > > -#ifdef RTE_MACHINE_CPUFLAG_SSE4_2 > +#if defined(RTE_MACHINE_CPUFLAG_SSE4_2) || defined(RTE_MACHINE_CPUFLAG_CRC32) > init_val = rte_hash_crc_4byte(t, init_val); > init_val = rte_hash_crc_4byte(k->ip_src, init_val); > init_val = rte_hash_crc_4byte(k->ip_dst, init_val); > init_val = rte_hash_crc_4byte(*p, init_val); > -#else /* RTE_MACHINE_CPUFLAG_SSE4_2 */ > +#else > init_val = rte_jhash_1word(t, init_val); > init_val = rte_jhash_1word(k->ip_src, init_val); > init_val = rte_jhash_1word(k->ip_dst, init_val); > init_val = rte_jhash_1word(*p, init_val); > -#endif /* RTE_MACHINE_CPUFLAG_SSE4_2 */ > +#endif > > return init_val; > } > @@ -191,16 +191,16 @@ ipv6_hash_crc(const void *data, __rte_unused uint32_t data_len, > const union ipv6_5tuple_host *k; > uint32_t t; > const uint32_t *p; > -#ifdef RTE_MACHINE_CPUFLAG_SSE4_2 > +#if defined(RTE_MACHINE_CPUFLAG_SSE4_2) || defined(RTE_MACHINE_CPUFLAG_CRC32) > const uint32_t *ip_src0, *ip_src1, *ip_src2, *ip_src3; > const uint32_t *ip_dst0, *ip_dst1, *ip_dst2, *ip_dst3; > -#endif /* RTE_MACHINE_CPUFLAG_SSE4_2 */ > +#endif > > k = data; > t = k->proto; > p = (const uint32_t *)&k->port_src; > > -#ifdef RTE_MACHINE_CPUFLAG_SSE4_2 > +#if defined(RTE_MACHINE_CPUFLAG_SSE4_2) || defined(RTE_MACHINE_CPUFLAG_CRC32) > ip_src0 = (const uint32_t *) k->ip_src; > ip_src1 = (const uint32_t *)(k->ip_src+4); > ip_src2 = (const uint32_t *)(k->ip_src+8); > @@ -219,14 +219,14 @@ ipv6_hash_crc(const void *data, __rte_unused uint32_t data_len, > init_val = rte_hash_crc_4byte(*ip_dst2, init_val); > init_val = rte_hash_crc_4byte(*ip_dst3, init_val); > init_val = rte_hash_crc_4byte(*p, init_val); > -#else /* RTE_MACHINE_CPUFLAG_SSE4_2 */ > +#else > init_val = rte_jhash_1word(t, init_val); > init_val = rte_jhash(k->ip_src, > sizeof(uint8_t) * IPV6_ADDR_LEN, init_val); > init_val = rte_jhash(k->ip_dst, > sizeof(uint8_t) * IPV6_ADDR_LEN, init_val); > init_val = rte_jhash_1word(*p, init_val); > -#endif /* RTE_MACHINE_CPUFLAG_SSE4_2 */ > +#endif > return init_val; > } > > -- > 1.9.1 >