From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] librte_hash: Fix compile errors on IBM POWER Date: Wed, 09 Dec 2015 21:22:15 +0100 Message-ID: <1836160.8n9UzrWNog@xps13> References: <1449634307-24075-1-git-send-email-chaozhu@linux.vnet.ibm.com> <20151209044000.GA5769@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Cc: dev@dpdk.org To: Chao Zhu Return-path: Received: from mail-wm0-f53.google.com (mail-wm0-f53.google.com [74.125.82.53]) by dpdk.org (Postfix) with ESMTP id 1430C3195 for ; Wed, 9 Dec 2015 21:23:32 +0100 (CET) Received: by mail-wm0-f53.google.com with SMTP id v187so2346214wmv.1 for ; Wed, 09 Dec 2015 12:23:32 -0800 (PST) In-Reply-To: <20151209044000.GA5769@localhost.localdomain> 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" 2015-12-09 10:10, Jerin Jacob: > On Wed, Dec 09, 2015 at 12:11:47PM +0800, Chao Zhu wrote: > > This patch fixes the compile errors caused by lacking of "size_t" d= efinition in rte_hash.h. The > > compile error exists on IBM POWER and ARM (see jerin.jacob@caviumne= tworks.com's message). >=20 > remove relative references from git log, >=20 > - compile error exists on IBM POWER and ARM (see jerin.jacob@caviumne= tworks.com's message). > + compile error exists on IBM POWER and ARM64 GCC 5.2 >=20 > >=20 > > The errors are like: > > In file included from /tmp/dpdk/app/test/test_hash_scaling.c:35:0: > > /tmp/dpdk/build/include/rte_hash.h:70:70: error: unknown type name = =E2=80=98size_t=E2=80=99 > > typedef int (*rte_hash_cmp_eq_t)(const void *key1, const void *key= 2, size_t key_len); > > = ^ > > /tmp/dpdk/build/include/rte_hash.h:120:48: error: unknown type name= =E2=80=98rte_hash_cmp_eq_t=E2=80=99 > > void rte_hash_set_cmp_func(struct rte_hash *h, rte_hash_cmp_eq_t f= unc); >=20 > add fixes: >=20 > Fixes: 95da2f8e9c61 ("hash: customize compare function") >=20 > with above changes, >=20 > Acked-by: Jerin Jacob Applied, thanks