[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH 01/22] Makefiles
From: |
Kirill Batuzov |
Subject: |
Re: [Qemu-devel] [RFC PATCH 01/22] Makefiles |
Date: |
Wed, 2 Jul 2014 14:46:35 +0400 (MSK) |
User-agent: |
Alpine 2.02 (DEB 1266 2009-07-14) |
On Tue, 1 Jul 2014, Pavel Dovgaluk wrote:
> Changes in main makefiles.
>
> Signed-off-by: Pavel Dovgalyuk <address@hidden>
> ---
>
> diff --git a/Makefile.target b/Makefile.target
> index 6089d29..357ab4d
> --- a/Makefile.target
> +++ b/Makefile.target
> @@ -76,6 +76,7 @@ all: $(PROGS) stap
> #########################################################
> # cpu emulator library
> obj-y = exec.o translate-all.o cpu-exec.o
> +obj-y += replay/
> obj-y += tcg/tcg.o tcg/optimize.o
> obj-$(CONFIG_TCG_INTERPRETER) += tci.o
> obj-$(CONFIG_TCG_INTERPRETER) += disas/tci.o
> @@ -85,6 +86,8 @@ obj-y += disas.o
> obj-$(call notempty,$(TARGET_XML_FILES)) += gdbstub-xml.o
> obj-$(call lnot,$(CONFIG_KVM)) += kvm-stub.o
>
> +replay/replay.o replay/replay-events.o replay/replay-internal.o: CFLAGS+=
> -Wno-unused-result
> +
> obj-$(CONFIG_LIBDECNUMBER) += libdecnumber/decContext.o
> obj-$(CONFIG_LIBDECNUMBER) += libdecnumber/decNumber.o
> obj-$(CONFIG_LIBDECNUMBER) += libdecnumber/dpd/decimal32.o
>
Is this change really needed? I think it is better to avoid disabling
compiler warnings without a good reason clearly stated in the commit
message.
> diff --git a/net/Makefile.objs b/net/Makefile.objs
> index a06ba59..cc6f32b
> --- a/net/Makefile.objs
> +++ b/net/Makefile.objs
> @@ -1,6 +1,6 @@
> common-obj-y = net.o queue.o checksum.o util.o hub.o
> common-obj-y += socket.o
> -common-obj-y += dump.o
> +common-obj-y += dump.o net-replay.o
> common-obj-y += eth.o
> common-obj-$(CONFIG_LINUX) += l2tpv3.o
> common-obj-$(CONFIG_POSIX) += tap.o vhost-user.o
>
>
This is not bisectable. You add net-replay.o to the makefile in
the patch 01/22 but net-replay.c is added only in the patch 18/22.
I've spotted similar problem with replay.h which is used before it is
added. You'll need to recombine/reorder your series.
--
Kirill