From mboxrd@z Thu Jan 1 00:00:00 1970 From: "De Lara Guarch, Pablo" Subject: Re: [PATCH v2] hash: move rte_hash_set_cmp_func() to ver DPDK_2.2. Date: Thu, 3 Dec 2015 09:25:49 +0000 Message-ID: References: <1447895712-12159-1-git-send-email-yuwb_bjy@ctbri.com.cn> <1449120209-9990-1-git-send-email-yuwb_bjy@ctbri.com.cn> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable To: Yu Nemo Wenbin , "dev@dpdk.org" Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id 9F0495683 for ; Thu, 3 Dec 2015 10:25:53 +0100 (CET) In-Reply-To: <1449120209-9990-1-git-send-email-yuwb_bjy@ctbri.com.cn> Content-Language: en-US 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" Hi Nemo, > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Yu Nemo Wenbin > Sent: Thursday, December 03, 2015 5:23 AM > To: dev@dpdk.org > Subject: [dpdk-dev] [PATCH v2] hash: move rte_hash_set_cmp_func() to > ver DPDK_2.2. >=20 > Also modified the comments of rte_hash_set_cmp_func(). >=20 > Signed-off-by: Yu Nemo Wenbin > --- > lib/librte_hash/rte_hash.h | 4 ++-- > lib/librte_hash/rte_hash_version.map | 8 +++++++- > 2 files changed, 9 insertions(+), 3 deletions(-) >=20 > diff --git a/lib/librte_hash/rte_hash.h b/lib/librte_hash/rte_hash.h > index 8378a42..dfca5ef 100644 > --- a/lib/librte_hash/rte_hash.h > +++ b/lib/librte_hash/rte_hash.h > @@ -66,7 +66,7 @@ typedef uint32_t hash_sig_t; > typedef uint32_t (*rte_hash_function)(const void *key, uint32_t key_len, > uint32_t init_val); >=20 > -/** Type of function used to compare the key. It works like the memcmp() > */ > +/** Type of function used to compare the hash key. */ > typedef int (*rte_hash_cmp_eq_t)(const void *key1, const void *key2, > size_t key_len); >=20 > /** > @@ -108,7 +108,7 @@ rte_hash_create(const struct rte_hash_parameters > *params); >=20 > /** > * Set the rte_hash_set_cmp_func. > - * Set the new hash compare function if the default one is not suitable > enough. > + * Set a new hash compare function other than the default one. > * > * @note Function pointer does not work with multi-process, so don't use > it > * in multi-process mode. > diff --git a/lib/librte_hash/rte_hash_version.map > b/lib/librte_hash/rte_hash_version.map > index 3bc1e2c..1aa94f9 100644 > --- a/lib/librte_hash/rte_hash_version.map > +++ b/lib/librte_hash/rte_hash_version.map > @@ -30,6 +30,12 @@ DPDK_2.1 { > rte_hash_lookup_data; > rte_hash_lookup_with_hash_data; > rte_hash_reset; > - rte_hash_set_cmp_func; >=20 > } DPDK_2.0; > + > +DPDK_2.2 { > + global: > + > + rte_hash_set_cmp_func; > + > +} DPDK_2.1; > -- > 1.9.1 I see you are basing off the v1 of this patch. You need to send all the cha= nges in one go. So, include the missing code in rte_cuckoo_hash.c, and include rte_hash_set= _cmp_func in rte_hash_version.map, but don't remove it from DPDK_2.1, as it does not = exist now. In other words, create a patch from the current head of mainline, send it a= s a v3, include the changes you made from v1->v2 and v2->v3 and send it again (and = don't forget to use --in-reply-to). Plus, remove the "to ver DPDK_2.2" part from the tile of the patch. Thanks! Pablo