From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f172.google.com (mail-wr0-f172.google.com [209.85.128.172]) by dpdk.org (Postfix) with ESMTP id 9F1155F11 for ; Thu, 8 Mar 2018 11:54:47 +0100 (CET) Received: by mail-wr0-f172.google.com with SMTP id n7so5206056wrn.5 for ; Thu, 08 Mar 2018 02:54:47 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to :user-agent; bh=MNIkNEuXOZ37mTCKTnouLgePiOYbeKd+/nYhd2ZJ8vI=; b=khvdsPEmyG1BzM7xkWkw804qBYZAaGx11Mv2V7wto33tYhUD2ZATsy77v5HDuQNm2n y3ztg9JZwYoiibeQvLNASw40FbIlM3eXpfea/P4jxIH84vYBHC34E3gcn9JsSFLG3ofP tcjbh4crMd3RKbsL/FR0zAxNHyzrGOnhX12CtS7DoGCHMNcwcjA3L3K0K+znz92SpVdl 7/jyt0JPH1hUZWyvyc57Iee9fGlISg4++vjjxgeTUkhlZZ5y3lZ9wtqUDoSMETC5QosC 51LlT3MzV6T+GpmWa6FLWdqHzzCf5WPoplXj7HDg/y7XO1knPMpW4ti6pH7c0LXpTn3t MECg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to:user-agent; bh=MNIkNEuXOZ37mTCKTnouLgePiOYbeKd+/nYhd2ZJ8vI=; b=S6gYIfmnyOAH8f/GevPAnfe56xK4m8hcF2BoOi0ChyZ7OnaHU/M0awqiMVeXw66Oml Z3DtzSuc6GepubZ5jfLrqtqbQuvcbmjTHvvysZO8ogN+elVtbsPuiZZk8mDt8d/SjXiX ppp7xzQMAgHsj/98IItn6FG7mknCcB2GOo72UXFrySsCgHMnhhyOY/+Ss6m0c6WLTxzv fcshNXa1mNGtDxc/YYzLFj5y51+1++Y7K7mW4AzukRzjlUZ0t7Y26n7UnBZ+Vqvbi7mh For1I4nHO8eu9FckQlfqea0YCe/1rDqPDHEI+rMBpZL7r061gThOA3E9L0NWU5GJu3iS uxPw== X-Gm-Message-State: AElRT7Hpbj+XmfOuz1DeSozHP7zqXjUs0TpJ+ouHbbuQvK5IfYrHRI81 bkVVvvE/af4+1Ndd3ewG3Ole X-Google-Smtp-Source: AG47ELshQ0vqyzcvmcT/PUkau6NlJSrl5rdtisBDAUvEk4QDkKWwvurrrnAt3IeoI50qIBhv9EFCSw== X-Received: by 10.223.164.158 with SMTP id g30mr6127769wrb.62.1520506487331; Thu, 08 Mar 2018 02:54:47 -0800 (PST) Received: from laranjeiro-vm.dev.6wind.com (host.78.145.23.62.rev.coltfrance.com. [62.23.145.78]) by smtp.gmail.com with ESMTPSA id z78sm24385829wrc.53.2018.03.08.02.54.46 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 08 Mar 2018 02:54:46 -0800 (PST) Date: Thu, 8 Mar 2018 11:53:26 +0100 From: =?iso-8859-1?Q?N=E9lio?= Laranjeiro To: "Burakov, Anatoly" , thomas@monjalon.net Cc: dev@dpdk.org, keith.wiles@intel.com, jianfeng.tan@intel.com, andras.kovacs@ericsson.com, laszlo.vadkeri@ericsson.com, benjamin.walker@intel.com, bruce.richardson@intel.com, konstantin.ananyev@intel.com, kuralamudhan.ramakrishnan@intel.com, louise.m.daly@intel.com, yskoh@mellanox.com, pepperjo@japf.ch, jerin.jacob@caviumnetworks.com, hemant.agrawal@nxp.com, olivier.matz@6wind.com Message-ID: <20180308105326.c2knce7g3jump4tr@laranjeiro-vm.dev.6wind.com> References: <20180307152701.6gxx7z7xwdpvyy5g@laranjeiro-vm.dev.6wind.com> <806e0fd0-b16a-a8db-6069-5daa42003a38@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <806e0fd0-b16a-a8db-6069-5daa42003a38@intel.com> User-Agent: NeoMutt/20170113 (1.7.2) Subject: Re: [dpdk-dev] [PATCH 00/41] Memory Hotplug for DPDK X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 08 Mar 2018 10:54:47 -0000 On Thu, Mar 08, 2018 at 09:37:27AM +0000, Burakov, Anatoly wrote: > On 07-Mar-18 4:05 PM, Burakov, Anatoly wrote: > > On 07-Mar-18 3:27 PM, Nélio Laranjeiro wrote: > > > Hi Anatoly, > > > > > > I am trying to run some test with this series, but it seems to be based > > > on some other commits of yours. I have already identified the following > > > one [1] it seems I am missing some others. > > > > > > It is possible to have a list of commits to apply on the current master > > > branch [2] before this series? > > > > > > Thanks, > > > > > > [1] https://dpdk.org/patch/35043 > > > [2] https://dpdk.org/browse/dpdk/commit/?id=c06ddf9698e0c2a9653cfa971f9ddc205065662c > > > > > > > > > > Hi Nelio, > > > > Yes, my apologies. I'm aware of the apply issues. The issue is due to me > > missing a rebase on one of the dependent patchsets. I'm preparing a v2 > > that will fix the issue (pending some internal processes). > > > > Hi Nelio, > > The v2 is now up, with corrected rebase. You can see the list of dependent > patches in the cover letter [1]. Once again apologies for incorrect rebase > in v1. Looking forward to your feedback! > > [1] http://dpdk.org/ml/archives/dev/2018-March/092070.html Hi Anatoly, First feedbacks, I have some issue when compiling it on desktop/server machine with clang and GCC, maybe due some different configuration items depending on the machine compile it. Clang error ----------- dpdk.org/lib/librte_eal/linuxapp/eal/eal_memalloc.c:590:6: error: variable 'have_numa' is used uninitialized whenever 'if' condition is true [-Werror,-Wsometimes-uninitialized] if (!hi) { ^~~ CC eal_lcore.o dpdk.org/lib/librte_eal/linuxapp/eal/eal_memalloc.c:693:6: note: uninitialized use occurs here if (have_numa) ^~~~~~~~~ dpdk.org/lib/librte_eal/linuxapp/eal/eal_memalloc.c:590:2: note: remove the 'if' if its condition is always false if (!hi) { ^~~~~~~~~~ dpdk.org/lib/librte_eal/linuxapp/eal/eal_memalloc.c:580:6: error: variable 'have_numa' is used uninitialized whenever 'if' condition is true [-Werror,-Wsometimes-uninitialized] if (internal_config.legacy_mem) ^~~~~~~~~~~~~~~~~~~~~~~~~~ dpdk.org/lib/librte_eal/linuxapp/eal/eal_memalloc.c:693:6: note: uninitialized use occurs here if (have_numa) ^~~~~~~~~ dpdk.org/lib/librte_eal/linuxapp/eal/eal_memalloc.c:580:2: note: remove the 'if' if its condition is always false if (internal_config.legacy_mem) ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ dpdk.org/lib/librte_eal/linuxapp/eal/eal_memalloc.c:573:16: note: initialize the variable 'have_numa' to silence this warning bool have_numa; ^ = false GCC errors ---------- /root/dpdk/lib/librte_eal/common/eal_common_memzone.c: In function ‘rte_memzone_free’: /root/dpdk/lib/librte_eal/common/eal_common_memzone.c:355:2: error: ‘addr’ may be used uninitialized in this function [-Werror=maybe-uninitialized] rte_free(addr); ^~~~~~~~~~~~~~ /root/dpdk/lib/librte_eal/linuxapp/eal/eal_memalloc.c: In function ‘eal_memalloc_alloc_page_bulk’: /root/dpdk/lib/librte_eal/linuxapp/eal/eal_memalloc.c:693:5: error: ‘have_numa’ may be used uninitialized in this function [-Werror=maybe-uninitialized] if (have_numa) ^ cc1: all warnings being treated as errors /root/dpdk/mk/internal/rte.compile-pre.mk:114: recipe for target 'eal_common_memzone.o' failed make[5]: *** [eal_common_memzone.o] Error 1 make[5]: *** Waiting for unfinished jobs.... cc1: all warnings being treated as errors /root/dpdk/mk/internal/rte.compile-pre.mk:114: recipe for target 'eal_memalloc.o' failed make[5]: *** [eal_memalloc.o] Error 1 /root/dpdk/lib/librte_eal/linuxapp/eal/eal_memory.c: In function ‘rte_eal_hugepage_attach’: /root/dpdk/lib/librte_eal/linuxapp/eal/eal_memory.c:1556:7: error: ‘max_seg’ may be used uninitialized in this function [-Werror=maybe-uninitialized] if (cur_seg >= max_seg) ^ /root/dpdk/lib/librte_eal/linuxapp/eal/eal_memory.c:1391:24: note: ‘max_seg’ was declared here unsigned int cur_seg, max_seg; ^~~~~~~ /root/dpdk/lib/librte_eal/linuxapp/eal/eal_memory.c:1389:15: error: ‘i’ may be used uninitialized in this function [-Werror=maybe-uninitialized] unsigned int i; ^ It worse to ask Thomas to have a dedicated repository/branch on DPDK, otherwise it will be a nightmare for anyone who want to test if we need each time to apply 54 patches. Can you see it with him? Thanks, -- Nélio Laranjeiro 6WIND