[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 0/3] Usb 20170929 patches
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PULL 0/3] Usb 20170929 patches |
Date: |
Wed, 4 Oct 2017 11:58:40 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 04/10/2017 11:37, Thomas Huth wrote:
> Ah, deja vu. I think you've now run into the problem that I had with my
> patch, too - see:
>
> https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg01875.html
> https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg02082.html
>
> I think this is a race condition when building with "make -j" in
> parallel. Just add a line like this and you should be fine:
>
> common-obj-$(CONFIG_ALL) += host-stub.o
That's correct! Maybe it should be documented in build-system.txt.
It's basically an optimization, CONFIG_ALL is used when some stubs are
not used in all subtargets but you still want to build them once-only.
Then the toplevel Makefile will typically see the configuration symbol
as defined (because it operates on the union of all targets'
configuration symbols), and will skip builting the "stubs" file. Using
CONFIG_ALL for the stubs file ensures that the object file is available
before recursing.
Paolo