From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v10 0/3] deduplicate EAL common functions Date: Thu, 30 Jul 2015 15:43:21 +0200 Message-ID: <1502678.UGGgkMtiuc@xps13> References: <1437852868-6031-1-git-send-email-rkerur@gmail.com> <3055506.vZLM2IGfDr@xps13> <55B9DC6F.7040506@6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: Olivier MATZ Return-path: Received: from mail-wi0-f181.google.com (mail-wi0-f181.google.com [209.85.212.181]) by dpdk.org (Postfix) with ESMTP id 8F2BDC630 for ; Thu, 30 Jul 2015 15:44:36 +0200 (CEST) Received: by wibxm9 with SMTP id xm9so69515811wib.1 for ; Thu, 30 Jul 2015 06:44:36 -0700 (PDT) In-Reply-To: <55B9DC6F.7040506@6wind.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" 2015-07-30 10:12, Olivier MATZ: > Hi Thomas & Ravi, > > On 07/27/2015 02:59 AM, Thomas Monjalon wrote: > > 2015-07-27 02:56, Thomas Monjalon: > >> v9 was a subset of previous deduplications by Ravi Kerur. > >> This v10 address the comments I've done on v9. > >> > >> Ravi Kerur (3): > >> eal: deduplicate lcore initialization > >> eal: deduplicate timer functions > >> eal: deduplicate memory initialization > > > > Applied shortly to integrate this old pending cleanup in RC2. > > > > When I try to compile the dpdk for x86_x32-native-linuxapp-gcc , I > get the following compilation error: > > CC eal_common_timer.o > In file included from /usr/include/sys/sysctl.h:63:0, > from > /home/matz/dpdk-pkg-cron/dpdk.org/lib/librte_eal/common/eal_common_timer.c:39: > /usr/include/bits/sysctl.h:19:3: error: #error "sysctl system call is > unsupported in x32 kernel" > # error "sysctl system call is unsupported in x32 kernel" > ^ > > Removing the "#include " line fixes the issue without > impacting the compilation. I think this include is not needed and > could be removed. > I can provide a patch if it's ok for you. After fixing another build issue on FreeBSD (patch sent), it builds well without sys/sysctl.h. So it seems to be an useless inclusion.