From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id C9C8A12A8 for ; Wed, 6 May 2015 02:36:01 +0200 (CEST) Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga103.jf.intel.com with ESMTP; 05 May 2015 17:36:02 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,376,1427785200"; d="scan'208";a="566872563" Received: from irsmsx104.ger.corp.intel.com ([163.33.3.159]) by orsmga003.jf.intel.com with ESMTP; 05 May 2015 17:36:00 -0700 Received: from irsmsx105.ger.corp.intel.com ([169.254.7.178]) by IRSMSX104.ger.corp.intel.com ([169.254.3.49]) with mapi id 14.03.0224.002; Wed, 6 May 2015 01:35:59 +0100 From: "Ananyev, Konstantin" To: "De Lara Guarch, Pablo" , "dev@dpdk.org" Thread-Topic: [dpdk-dev] [PATCH v3 3/6] hash: update jhash function with the latest available Thread-Index: AQHQh0Ifa916QN+t0kuol3MAxqDHRp1uDeow Date: Wed, 6 May 2015 00:35:58 +0000 Message-ID: <2601191342CEEE43887BDE71AB97725821424ED1@irsmsx105.ger.corp.intel.com> References: <1429874587-17939-1-git-send-email-pablo.de.lara.guarch@intel.com> <1430837034-21031-1-git-send-email-pablo.de.lara.guarch@intel.com> <1430837034-21031-4-git-send-email-pablo.de.lara.guarch@intel.com> In-Reply-To: <1430837034-21031-4-git-send-email-pablo.de.lara.guarch@intel.com> Accept-Language: en-IE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [163.33.239.180] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v3 3/6] hash: update jhash function with the latest available X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 06 May 2015 00:36:03 -0000 Hi Pablo, > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Pablo de Lara > Sent: Tuesday, May 05, 2015 3:44 PM > To: dev@dpdk.org > Subject: [dpdk-dev] [PATCH v3 3/6] hash: update jhash function with the l= atest available >=20 > Jenkins hash function was developed originally in 1996, > and was integrated in first versions of DPDK. > The function has been improved in 2006, > achieving up to 60% better performance, compared to the original one. >=20 > This patch integrates that code into the rte_jhash library. >=20 > Signed-off-by: Pablo de Lara > --- > lib/librte_hash/rte_jhash.h | 261 +++++++++++++++++++++++++++++++------= ------ > 1 files changed, 188 insertions(+), 73 deletions(-) >=20 > diff --git a/lib/librte_hash/rte_jhash.h b/lib/librte_hash/rte_jhash.h > index a4bf5a1..0e96b7c 100644 > --- a/lib/librte_hash/rte_jhash.h > +++ b/lib/librte_hash/rte_jhash.h > @@ -1,7 +1,7 @@ > /*- > * BSD LICENSE > * > - * Copyright(c) 2010-2014 Intel Corporation. All rights reserved. > + * Copyright(c) 2010-2015 Intel Corporation. All rights reserved. > * All rights reserved. > * > * Redistribution and use in source and binary forms, with or without > @@ -45,38 +45,68 @@ extern "C" { > #endif >=20 > #include > +#include > +#include >=20 > /* jhash.h: Jenkins hash support. > * > - * Copyright (C) 1996 Bob Jenkins (bob_jenkins@burtleburtle.net) > + * Copyright (C) 2006 Bob Jenkins (bob_jenkins@burtleburtle.net) > * > * http://burtleburtle.net/bob/hash/ > * > * These are the credits from Bob's sources: > * > - * lookup2.c, by Bob Jenkins, December 1996, Public Domain. > - * hash(), hash2(), hash3, and mix() are externally useful functions. > - * Routines to test the hash are included if SELF_TEST is defined. > - * You can use this free for any purpose. It has no warranty. > + * lookup3.c, by Bob Jenkins, May 2006, Public Domain. > + * > + * These are functions for producing 32-bit hashes for hash table lookup= . > + * hashword(), hashlittle(), hashlittle2(), hashbig(), mix(), and final(= ) > + * are externally useful functions. Routines to test the hash are inclu= ded > + * if SELF_TEST is defined. You can use this free for any purpose. It'= s in > + * the public domain. It has no warranty. > * > * $FreeBSD$ > */ >=20 > +#define rot(x, k) (((x) << (k)) | ((x) >> (32-(k)))) > + > /** @internal Internal function. NOTE: Arguments are modified. */ > #define __rte_jhash_mix(a, b, c) do { \ > - a -=3D b; a -=3D c; a ^=3D (c>>13); \ > - b -=3D c; b -=3D a; b ^=3D (a<<8); \ > - c -=3D a; c -=3D b; c ^=3D (b>>13); \ > - a -=3D b; a -=3D c; a ^=3D (c>>12); \ > - b -=3D c; b -=3D a; b ^=3D (a<<16); \ > - c -=3D a; c -=3D b; c ^=3D (b>>5); \ > - a -=3D b; a -=3D c; a ^=3D (c>>3); \ > - b -=3D c; b -=3D a; b ^=3D (a<<10); \ > - c -=3D a; c -=3D b; c ^=3D (b>>15); \ > + a -=3D c; a ^=3D rot(c, 4); c +=3D b; \ > + b -=3D a; b ^=3D rot(a, 6); a +=3D c; \ > + c -=3D b; c ^=3D rot(b, 8); b +=3D a; \ > + a -=3D c; a ^=3D rot(c, 16); c +=3D b; \ > + b -=3D a; b ^=3D rot(a, 19); a +=3D c; \ > + c -=3D b; c ^=3D rot(b, 4); b +=3D a; \ > +} while (0) > + > +#define __rte_jhash_final(a, b, c) do { \ > + c ^=3D b; c -=3D rot(b, 14); \ > + a ^=3D c; a -=3D rot(c, 11); \ > + b ^=3D a; b -=3D rot(a, 25); \ > + c ^=3D b; c -=3D rot(b, 16); \ > + a ^=3D c; a -=3D rot(c, 4); \ > + b ^=3D a; b -=3D rot(a, 14); \ > + c ^=3D b; c -=3D rot(b, 24); \ > } while (0) >=20 > /** The golden ratio: an arbitrary value. */ > -#define RTE_JHASH_GOLDEN_RATIO 0x9e3779b9 > +#define RTE_JHASH_GOLDEN_RATIO 0xdeadbeef > + > +#if RTE_BYTE_ORDER =3D=3D RTE_LITTLE_ENDIAN > +#define RTE_JHASH_BYTE0_SHIFT 0 > +#define RTE_JHASH_BYTE1_SHIFT 8 > +#define RTE_JHASH_BYTE2_SHIFT 16 > +#define RTE_JHASH_BYTE3_SHIFT 24 > +#else > +#define RTE_JHASH_BYTE0_SHIFT 24 > +#define RTE_JHASH_BYTE1_SHIFT 16 > +#define RTE_JHASH_BYTE2_SHIFT 8 > +#define RTE_JHASH_BYTE3_SHIFT 0 > +#endif > + > +#define LOWER8b_MASK rte_le_to_cpu_32(0xff) > +#define LOWER16b_MASK rte_le_to_cpu_32(0xffff) > +#define LOWER24b_MASK rte_le_to_cpu_32(0xffffff) >=20 > /** > * The most generic version, hashes an arbitrary sequence > @@ -95,42 +125,119 @@ extern "C" { > static inline uint32_t > rte_jhash(const void *key, uint32_t length, uint32_t initval) > { > - uint32_t a, b, c, len; > - const uint8_t *k =3D (const uint8_t *)key; > - const uint32_t *k32 =3D (const uint32_t *)key; > + uint32_t a, b, c; > + union { > + const void *ptr; > + size_t i; > + } u; >=20 > - len =3D length; > - a =3D b =3D RTE_JHASH_GOLDEN_RATIO; > - c =3D initval; > + /* Set up the internal state */ > + a =3D b =3D c =3D RTE_JHASH_GOLDEN_RATIO + ((uint32_t)length) + initval= ; >=20 > - while (len >=3D 12) { > - a +=3D k32[0]; > - b +=3D k32[1]; > - c +=3D k32[2]; > + u.ptr =3D key; >=20 > - __rte_jhash_mix(a,b,c); > + /* Check key alignment. For x86 architecture, first case is always opti= mal */ > + if (!strcmp(RTE_ARCH,"x86_64") || !strcmp(RTE_ARCH,"i686") || (u.i & 0x= 3) =3D=3D 0) { Wonder why strcmp(), why not something like: 'if defined(RTE_ARCH_I686) || = defined(RTE_ARCH_X86_64)' as in all other places? Another question what would be in case of RTE_ARCH=3D"x86_x32"? Konstantin > + const uint32_t *k =3D (const uint32_t *)key; >=20 > - k +=3D (3 * sizeof(uint32_t)), k32 +=3D 3; > - len -=3D (3 * sizeof(uint32_t)); > - } > + while (length > 12) { > + a +=3D k[0]; > + b +=3D k[1]; > + c +=3D k[2]; >=20 > - c +=3D length; > - switch (len) { > - case 11: c +=3D ((uint32_t)k[10] << 24); > - case 10: c +=3D ((uint32_t)k[9] << 16); > - case 9 : c +=3D ((uint32_t)k[8] << 8); > - case 8 : b +=3D ((uint32_t)k[7] << 24); > - case 7 : b +=3D ((uint32_t)k[6] << 16); > - case 6 : b +=3D ((uint32_t)k[5] << 8); > - case 5 : b +=3D k[4]; > - case 4 : a +=3D ((uint32_t)k[3] << 24); > - case 3 : a +=3D ((uint32_t)k[2] << 16); > - case 2 : a +=3D ((uint32_t)k[1] << 8); > - case 1 : a +=3D k[0]; > - default: break; > - }; > + __rte_jhash_mix(a, b, c); > + > + k +=3D 3; > + length -=3D 12; > + } > + > + switch (length) { > + case 12: > + c +=3D k[2]; b +=3D k[1]; a +=3D k[0]; break; > + case 11: > + c +=3D k[2] & LOWER24b_MASK; b +=3D k[1]; a +=3D k[0]; break; > + case 10: > + c +=3D k[2] & LOWER16b_MASK; b +=3D k[1]; a +=3D k[0]; break; > + case 9: > + c +=3D k[2] & LOWER8b_MASK; b +=3D k[1]; a +=3D k[0]; break; > + case 8: > + b +=3D k[1]; a +=3D k[0]; break; > + case 7: > + b +=3D k[1] & LOWER24b_MASK; a +=3D k[0]; break; > + case 6: > + b +=3D k[1] & LOWER16b_MASK; a +=3D k[0]; break; > + case 5: > + b +=3D k[1] & LOWER8b_MASK; a +=3D k[0]; break; > + case 4: > + a +=3D k[0]; break; > + case 3: > + a +=3D k[0] & LOWER24b_MASK; break; > + case 2: > + a +=3D k[0] & LOWER16b_MASK; break; > + case 1: > + a +=3D k[0] & LOWER8b_MASK; break; > + /* zero length strings require no mixing */ > + case 0: > + return c; > + }; > + } else { > + const uint8_t *k =3D (const uint8_t *)key; > + > + /* all but the last block: affect some 32 bits of (a, b, c) */ > + while (length > 12) { > + a +=3D ((uint32_t)k[0]) << RTE_JHASH_BYTE0_SHIFT; > + a +=3D ((uint32_t)k[1]) << RTE_JHASH_BYTE1_SHIFT; > + a +=3D ((uint32_t)k[2]) << RTE_JHASH_BYTE2_SHIFT; > + a +=3D ((uint32_t)k[3]) << RTE_JHASH_BYTE3_SHIFT; > + b +=3D ((uint32_t)k[4]) << RTE_JHASH_BYTE0_SHIFT; > + b +=3D ((uint32_t)k[5]) << RTE_JHASH_BYTE1_SHIFT; > + b +=3D ((uint32_t)k[6]) << RTE_JHASH_BYTE2_SHIFT; > + b +=3D ((uint32_t)k[7]) << RTE_JHASH_BYTE3_SHIFT; > + c +=3D ((uint32_t)k[8]) << RTE_JHASH_BYTE0_SHIFT; > + c +=3D ((uint32_t)k[9]) << RTE_JHASH_BYTE1_SHIFT; > + c +=3D ((uint32_t)k[10]) << RTE_JHASH_BYTE2_SHIFT; > + c +=3D ((uint32_t)k[11]) << RTE_JHASH_BYTE3_SHIFT; > + > + __rte_jhash_mix(a, b, c); > + > + k +=3D 12; > + length -=3D 12; > + } > + > + /* last block: affect all 32 bits of (c) */ > + /* all the case statements fall through */ > + switch (length) { > + case 12: > + c +=3D ((uint32_t)k[11]) << RTE_JHASH_BYTE3_SHIFT; > + case 11: > + c +=3D ((uint32_t)k[10]) << RTE_JHASH_BYTE2_SHIFT; > + case 10: > + c +=3D ((uint32_t)k[9]) << RTE_JHASH_BYTE1_SHIFT; > + case 9: > + c +=3D ((uint32_t)k[8]) << RTE_JHASH_BYTE0_SHIFT; > + case 8: > + b +=3D ((uint32_t)k[7]) << RTE_JHASH_BYTE3_SHIFT; > + case 7: > + b +=3D ((uint32_t)k[6]) << RTE_JHASH_BYTE2_SHIFT; > + case 6: > + b +=3D ((uint32_t)k[5]) << RTE_JHASH_BYTE1_SHIFT; > + case 5: > + b +=3D ((uint32_t)k[4]) << RTE_JHASH_BYTE0_SHIFT; > + case 4: > + a +=3D ((uint32_t)k[3]) << RTE_JHASH_BYTE3_SHIFT; > + case 3: > + a +=3D ((uint32_t)k[2]) << RTE_JHASH_BYTE2_SHIFT; > + case 2: > + a +=3D ((uint32_t)k[1]) << RTE_JHASH_BYTE1_SHIFT; > + case 1: > + a +=3D ((uint32_t)k[0]) << RTE_JHASH_BYTE0_SHIFT; > + break; > + case 0: > + return c; > + } > + } >=20 > - __rte_jhash_mix(a,b,c); > + __rte_jhash_final(a, b, c); >=20 > return c; > } > @@ -151,33 +258,51 @@ rte_jhash(const void *key, uint32_t length, uint32_= t initval) > static inline uint32_t > rte_jhash2(const uint32_t *k, uint32_t length, uint32_t initval) > { > - uint32_t a, b, c, len; > + uint32_t a, b, c; >=20 > - a =3D b =3D RTE_JHASH_GOLDEN_RATIO; > - c =3D initval; > - len =3D length; > + /* Set up the internal state */ > + a =3D b =3D c =3D RTE_JHASH_GOLDEN_RATIO + (((uint32_t)length) << 2) + = initval; >=20 > - while (len >=3D 3) { > + /* Handle most of the key */ > + while (length > 3) { > a +=3D k[0]; > b +=3D k[1]; > c +=3D k[2]; > + > __rte_jhash_mix(a, b, c); > - k +=3D 3; len -=3D 3; > - } >=20 > - c +=3D length * 4; > + k +=3D 3; > + length -=3D 3; > + } >=20 > - switch (len) { > - case 2 : b +=3D k[1]; > - case 1 : a +=3D k[0]; > - default: break; > + /* Handle the last 3 uint32_t's */ > + switch (length) { > + case 3: > + c +=3D k[2]; > + case 2: > + b +=3D k[1]; > + case 1: > + a +=3D k[0]; > + __rte_jhash_final(a, b, c); > + /* case 0: nothing left to add */ > + case 0: > + break; > }; >=20 > - __rte_jhash_mix(a,b,c); > - > return c; > } >=20 > +static inline uint32_t > +__rte_jhash_3words(uint32_t a, uint32_t b, uint32_t c, uint32_t initval) > +{ > + a +=3D RTE_JHASH_GOLDEN_RATIO + initval; > + b +=3D RTE_JHASH_GOLDEN_RATIO + initval; > + c +=3D RTE_JHASH_GOLDEN_RATIO + initval; > + > + __rte_jhash_final(a, b, c); > + > + return c; > +} >=20 > /** > * A special ultra-optimized versions that knows it is hashing exactly > @@ -197,17 +322,7 @@ rte_jhash2(const uint32_t *k, uint32_t length, uint3= 2_t initval) > static inline uint32_t > rte_jhash_3words(uint32_t a, uint32_t b, uint32_t c, uint32_t initval) > { > - a +=3D RTE_JHASH_GOLDEN_RATIO; > - b +=3D RTE_JHASH_GOLDEN_RATIO; > - c +=3D initval; > - > - __rte_jhash_mix(a, b, c); > - > - /* > - * NOTE: In particular the "c +=3D length; __rte_jhash_mix(a,b,c);" > - * normally done at the end is not done here. > - */ > - return c; > + return __rte_jhash_3words(a + 12, b + 12, c + 12, initval); > } >=20 > /** > @@ -226,7 +341,7 @@ rte_jhash_3words(uint32_t a, uint32_t b, uint32_t c, = uint32_t initval) > static inline uint32_t > rte_jhash_2words(uint32_t a, uint32_t b, uint32_t initval) > { > - return rte_jhash_3words(a, b, 0, initval); > + return __rte_jhash_3words(a + 8, b + 8, 8, initval); > } >=20 > /** > @@ -243,7 +358,7 @@ rte_jhash_2words(uint32_t a, uint32_t b, uint32_t ini= tval) > static inline uint32_t > rte_jhash_1word(uint32_t a, uint32_t initval) > { > - return rte_jhash_3words(a, 0, 0, initval); > + return __rte_jhash_3words(a + 4, 4, 4, initval); > } >=20 > #ifdef __cplusplus > -- > 1.7.4.1