Message ID | 20150924094328.5a2ea3c8@miho (mailing list archive) |
---|---|
State | Accepted, archived |
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id 503318E65; Thu, 24 Sep 2015 09:43:38 +0200 (CEST) Received: from ernst.netinsight.se (ernst.netinsight.se [194.16.221.21]) by dpdk.org (Postfix) with SMTP id A741712A8 for <dev@dpdk.org>; Thu, 24 Sep 2015 09:43:36 +0200 (CEST) Received: from miho (unverified [10.100.1.152]) by ernst.netinsight.se (EMWAC SMTPRS 0.83) with SMTP id <B0031073144@ernst.netinsight.se>; Thu, 24 Sep 2015 09:43:27 +0200 Date: Thu, 24 Sep 2015 09:43:28 +0200 From: Simon Kagstrom <simon.kagstrom@netinsight.net> To: dev@dpdk.org, nelio.laranjeiro@6wind.com, stephen@networkplumber.org, thomas.monjalon@6wind.com, bruce.richardson@intel.com Message-ID: <20150924094328.5a2ea3c8@miho> X-Mailer: Claws Mail 3.9.3 (GTK+ 2.24.27; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: [dpdk-dev] [PATCH] mk: Quote $(KERNELCC) to allow ccache builds X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK <dev.dpdk.org> List-Unsubscribe: <http://dpdk.org/ml/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://dpdk.org/ml/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <http://dpdk.org/ml/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Commit Message
Simon Kagstrom
Sept. 24, 2015, 7:43 a.m. UTC
Otherwise building with KERNELCC="ccache gcc" will fail:
== Build lib/librte_eal/linuxapp/igb_uio
/usr/src/linux-headers-3.13.0-63-generic/arch/x86/Makefile:98: stack protector enabled but no compiler support
/usr/src/linux-headers-3.13.0-63-generic/arch/x86/Makefile:113: CONFIG_X86_X32 enabled but no binutils support
ccache: invalid option -- 'p'
Usage:
ccache [options]
ccache compiler [compiler options]
compiler [compiler options] (via symbolic link)
Options:
-c, --cleanup delete old files and recalculate size counters
(normally not needed as this is done automatically)
-C, --clear clear the cache completely
-F, --max-files=N set maximum number of files in cache to N (use 0 for
no limit)
-M, --max-size=SIZE set maximum size of cache to SIZE (use 0 for no
limit; available suffixes: G, M and K; default
suffix: G)
-s, --show-stats show statistics summary
-z, --zero-stats zero statistics counters
-h, --help print this help text
-V, --version print version and copyright information
Signed-off-by: Simon Kagstrom <simon.kagstrom@netinsight.net>
---
mk/rte.module.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
Ping? This is one of three outstanding DPDK patches I have which hasn't seen any activitiy in a while. Is there a list of pending applies somewhere to monitor activity? // Simon On Thu, 24 Sep 2015 09:43:28 +0200 Simon Kagstrom <simon.kagstrom@netinsight.net> wrote: > Otherwise building with KERNELCC="ccache gcc" will fail: > > == Build lib/librte_eal/linuxapp/igb_uio > /usr/src/linux-headers-3.13.0-63-generic/arch/x86/Makefile:98: stack protector enabled but no compiler support > /usr/src/linux-headers-3.13.0-63-generic/arch/x86/Makefile:113: CONFIG_X86_X32 enabled but no binutils support > ccache: invalid option -- 'p' > Usage: > ccache [options] > ccache compiler [compiler options] > compiler [compiler options] (via symbolic link) > > Options: > -c, --cleanup delete old files and recalculate size counters > (normally not needed as this is done automatically) > -C, --clear clear the cache completely > -F, --max-files=N set maximum number of files in cache to N (use 0 for > no limit) > -M, --max-size=SIZE set maximum size of cache to SIZE (use 0 for no > limit; available suffixes: G, M and K; default > suffix: G) > -s, --show-stats show statistics summary > -z, --zero-stats zero statistics counters > > -h, --help print this help text > -V, --version print version and copyright information > > Signed-off-by: Simon Kagstrom <simon.kagstrom@netinsight.net> > --- > mk/rte.module.mk | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mk/rte.module.mk b/mk/rte.module.mk > index 7bf77c1..53ed4fe 100644 > --- a/mk/rte.module.mk > +++ b/mk/rte.module.mk > @@ -78,7 +78,7 @@ build: _postbuild > $(MODULE).ko: $(SRCS_LINKS) > @if [ ! -f $(notdir Makefile) ]; then ln -nfs $(SRCDIR)/Makefile . ; fi > @$(MAKE) -C $(RTE_KERNELDIR) M=$(CURDIR) O=$(RTE_KERNELDIR) \ > - CC=$(KERNELCC) CROSS_COMPILE=$(CROSS) V=$(if $V,1,0) > + CC="$(KERNELCC)" CROSS_COMPILE=$(CROSS) V=$(if $V,1,0) > > # install module in $(RTE_OUTPUT)/kmod > $(RTE_OUTPUT)/kmod/$(MODULE).ko: $(MODULE).ko
Hi Simon, Sorry for not having answered before. On 10/13/2015 02:10 PM, Simon Kagstrom wrote: > Ping? > > This is one of three outstanding DPDK patches I have which hasn't seen > any activitiy in a while. Is there a list of pending applies somewhere > to monitor activity? There is the patchwork tool: http://dpdk.org/dev/patchwork/project/dpdk/list/ >> Otherwise building with KERNELCC="ccache gcc" will fail: >> >> == Build lib/librte_eal/linuxapp/igb_uio >> /usr/src/linux-headers-3.13.0-63-generic/arch/x86/Makefile:98: stack protector enabled but no compiler support >> /usr/src/linux-headers-3.13.0-63-generic/arch/x86/Makefile:113: CONFIG_X86_X32 enabled but no binutils support >> ccache: invalid option -- 'p' >> Usage: >> ccache [options] >> ccache compiler [compiler options] >> compiler [compiler options] (via symbolic link) >> >> Options: >> -c, --cleanup delete old files and recalculate size counters >> (normally not needed as this is done automatically) >> -C, --clear clear the cache completely >> -F, --max-files=N set maximum number of files in cache to N (use 0 for >> no limit) >> -M, --max-size=SIZE set maximum size of cache to SIZE (use 0 for no >> limit; available suffixes: G, M and K; default >> suffix: G) >> -s, --show-stats show statistics summary >> -z, --zero-stats zero statistics counters >> >> -h, --help print this help text >> -V, --version print version and copyright information >> >> Signed-off-by: Simon Kagstrom <simon.kagstrom@netinsight.net> Acked-by: Olivier Matz <olivier.matz@6wind.com>
On 2015-10-13 14:26, Olivier MATZ wrote: > Sorry for not having answered before. Thanks for looking at it now though! >> This is one of three outstanding DPDK patches I have which hasn't seen >> any activitiy in a while. Is there a list of pending applies somewhere >> to monitor activity? > > There is the patchwork tool: > http://dpdk.org/dev/patchwork/project/dpdk/list/ Thanks! I knew about it, but forgot to look there. It would be nice to have tags to signify e.g., for-v2.2, candidate-v2.2 etc. like you can have on github to easier see where patches are going, but perhaps patchwork doesn't work that way. Is the process to ping old patches like this on the mailing list? Two of the patches (this one included) I have outstanding are build fixes for use in our build environment, so it would be nice to them upstreamed. // Simon
2015-10-13 14:39, Simon Kågström: > On 2015-10-13 14:26, Olivier MATZ wrote: > > There is the patchwork tool: > > http://dpdk.org/dev/patchwork/project/dpdk/list/ > > Thanks! I knew about it, but forgot to look there. It would be nice to > have tags to signify e.g., for-v2.2, candidate-v2.2 etc. like you can > have on github to easier see where patches are going, but perhaps > patchwork doesn't work that way. No it's not needed currently because every patches in this mailing-list target an integration in the main branch for the next release. Exceptions must be notified. > Is the process to ping old patches like this on the mailing list? Yes it is the responsibility of the developer and the maintainer to get reviews. Please check in the MAINTAINERS file who to contact. > Two of the patches (this one included) I have outstanding are build fixes > for use in our build environment, so it would be nice to them upstreamed. Waiting for integration of your patches, maybe you have some free time to help other developers by making reviews ;) Thanks
On 2015-10-13 14:45, Thomas Monjalon wrote: > 2015-10-13 14:39, Simon Kågström: >> Two of the patches (this one included) I have outstanding are build fixes >> for use in our build environment, so it would be nice to them upstreamed. > > Waiting for integration of your patches, maybe you have some free time to > help other developers by making reviews ;) Waiting for integration is not my only work-task :-) Anyway, I have too superficial knowledge about DPDK to chime in with relevant comments in most cases, but I'll comment if I feel I can contribute. // Simon
> >> Signed-off-by: Simon Kagstrom <simon.kagstrom@netinsight.net> > > Acked-by: Olivier Matz <olivier.matz@6wind.com> Applied, thanks
diff --git a/mk/rte.module.mk b/mk/rte.module.mk index 7bf77c1..53ed4fe 100644 --- a/mk/rte.module.mk +++ b/mk/rte.module.mk @@ -78,7 +78,7 @@ build: _postbuild $(MODULE).ko: $(SRCS_LINKS) @if [ ! -f $(notdir Makefile) ]; then ln -nfs $(SRCDIR)/Makefile . ; fi @$(MAKE) -C $(RTE_KERNELDIR) M=$(CURDIR) O=$(RTE_KERNELDIR) \ - CC=$(KERNELCC) CROSS_COMPILE=$(CROSS) V=$(if $V,1,0) + CC="$(KERNELCC)" CROSS_COMPILE=$(CROSS) V=$(if $V,1,0) # install module in $(RTE_OUTPUT)/kmod $(RTE_OUTPUT)/kmod/$(MODULE).ko: $(MODULE).ko