Re: [PATCH] btrfs-progs: Add dependencies explicitly to fix a parallel build issue

2013-09-23 Thread David Sterba
On Sun, Sep 22, 2013 at 09:06:39AM +0800, Rongqing Li wrote: > I want to know how many cores your cpu has? I can not reproduce it on > my 2 cores cpu, but it always happens when run on a server which is > a 16 cores cpu and "make -j20" Depends on what you call a core, I've tested this on a box wit

Re: [PATCH] btrfs-progs: Add dependencies explicitly to fix a parallel build issue

2013-09-23 Thread Rongqing Li
map-logical.o cmds-check.o:version.h +btrfs-debug-tree.o:version.h + %.static.o: %.c @echo "[CC] $@" $(Q)$(CC) $(DEPFLAGS) $(AM_CFLAGS) $(STATIC_CFLAGS) -c $< -o $@ I think this can be done more cleanly, I'll send a patch. -Eric If this one is not

Re: [PATCH] btrfs-progs: Add dependencies explicitly to fix a parallel build issue

2013-09-21 Thread Rongqing Li
On 09/18/2013 10:51 PM, David Sterba wrote: On Wed, Sep 18, 2013 at 09:11:01AM +0800, rongqing...@windriver.com wrote: From: Roy Li The dependencies of "all: version.h" or other similar ones can not fix the parallel build failure, only reduce the times; In fact, many *.o files require versio

Re: [PATCH] btrfs-progs: Add dependencies explicitly to fix a parallel build issue

2013-09-18 Thread David Sterba
On Wed, Sep 18, 2013 at 09:11:01AM +0800, rongqing...@windriver.com wrote: > From: Roy Li > > The dependencies of "all: version.h" or other similar ones can not > fix the parallel build failure, only reduce the times; In fact, > many *.o files require version.h file. > > #grep '#include "v

Re: [PATCH] btrfs-progs: Add dependencies explicitly to fix a parallel build issue

2013-09-17 Thread Rongqing Li
On 09/18/2013 09:51 AM, Eric Sandeen wrote: @echo "[CC] $@" >$(Q)$(CC) $(DEPFLAGS) $(AM_CFLAGS) $(STATIC_CFLAGS) -c $< -o $@ > I think this can be done more cleanly, I'll send a patch. -Eric Thanks, expect your patch -Roy -- Best Reagrds, Roy | RongQing Li -- To uns

[PATCH] btrfs-progs: Add dependencies explicitly to fix a parallel build issue

2013-09-17 Thread Eric Sandeen
On 9/17/13 8:11 PM, rongqing...@windriver.com wrote: > From: Roy Li > > The dependencies of "all: version.h" or other similar ones can not > fix the parallel build failure, only reduce the times; In fact, > many *.o files require version.h file. > > #grep '#include "version.h"' ./ -r >

[PATCH] btrfs-progs: Add dependencies explicitly to fix a parallel build issue

2013-09-17 Thread rongqing.li
From: Roy Li The dependencies of "all: version.h" or other similar ones can not fix the parallel build failure, only reduce the times; In fact, many *.o files require version.h file. #grep '#include "version.h"' ./ -r ./btrfs-corrupt-block.c:#include "version.h" ./btrfs.c