[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 25/27] build: some simplifications for "trace/Ma
From: |
Lluís Vilanova |
Subject: |
Re: [Qemu-devel] [PATCH 25/27] build: some simplifications for "trace/Makefile.objs" |
Date: |
Mon, 14 Jan 2013 16:19:38 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (gnu/linux) |
Paolo Bonzini writes:
> Signed-off-by: Lluís Vilanova <address@hidden>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> trace.h => include/trace.h | 0
> trace/Makefile.objs | 26 +++++++++++++-------------
> 2 files changed, 13 insertions(+), 13 deletions(-)
> rename trace.h => include/trace.h (100%)
> diff --git a/trace.h b/include/trace.h
> similarity index 100%
> rename from trace.h
> rename to include/trace.h
> diff --git a/trace/Makefile.objs b/trace/Makefile.objs
> index 40febce..ed2e30b 100644
> --- a/trace/Makefile.objs
> +++ b/trace/Makefile.objs
> @@ -1,12 +1,9 @@
> # -*- mode: makefile -*-
> ######################################################################
> -# Auto-generated tracing routines
> +# Auto-generated header for tracing routines
> -ifeq ($(TRACE_BACKEND),dtrace)
> -TRACE_H_EXTRA_DEPS=$(obj)/generated-tracers-dtrace.h
> -endif
> -$(obj)/generated-tracers.h: $(obj)/generated-tracers.h-timestamp
> $(TRACE_H_EXTRA_DEPS)
> +$(obj)/generated-tracers.h: $(obj)/generated-tracers.h-timestamp
> $(obj)/generated-tracers.h-timestamp: $(SRC_PATH)/trace-events
> $(BUILD_DIR)/config-host.mak
> $(call quiet-command,$(TRACETOOL) \
> --format=h \
> @@ -14,6 +11,10 @@ $(obj)/generated-tracers.h-timestamp:
> $(SRC_PATH)/trace-events $(BUILD_DIR)/conf
> < $< > $@," GEN $(patsubst %-timestamp,%,$@)")
> @cmp -s $@ $(patsubst %-timestamp,%,$@) || cp $@ $(patsubst
> %-timestamp,%,$@)
> +######################################################################
> +# Auto-generated tracing routines (non-DTrace)
> +
> +ifneq ($(TRACE_BACKEND),dtrace)
> $(obj)/generated-tracers.c: $(obj)/generated-tracers.c-timestamp
> $(obj)/generated-tracers.c-timestamp: $(SRC_PATH)/trace-events
> $(BUILD_DIR)/config-host.mak
> $(call quiet-command,$(TRACETOOL) \
> @@ -23,9 +24,6 @@ $(obj)/generated-tracers.c-timestamp:
> $(SRC_PATH)/trace-events $(BUILD_DIR)/conf
> @cmp -s $@ $(patsubst %-timestamp,%,$@) || cp $@ $(patsubst
> %-timestamp,%,$@)
> $(obj)/generated-tracers.o: $(obj)/generated-tracers.c
> $(obj)/generated-tracers.h
> -
> -ifneq ($(TRACE_BACKEND),dtrace)
> -trace-obj-y += generated-tracers.o
> endif
> @@ -35,19 +33,20 @@ endif
> # Normal practice is to name DTrace probe file with a '.d' extension
> # but that gets picked up by QEMU's Makefile as an external dependency
> # rule file. So we use '.dtrace' instead
> -$(obj)/generated-tracers-dtrace.dtrace:
> $(obj)/generated-tracers-dtrace.dtrace-timestamp
> -$(obj)/generated-tracers-dtrace.dtrace-timestamp: $(SRC_PATH)/trace-events
> $(BUILD_DIR)/config-host.mak
> +ifeq ($(TRACE_BACKEND),dtrace)
> +$(obj)/generated-tracers.dtrace: $(obj)/generated-tracers.dtrace-timestamp
> +$(obj)/generated-tracers.dtrace-timestamp: $(SRC_PATH)/trace-events
> $(BUILD_DIR)/config-host.mak
> $(call quiet-command,$(TRACETOOL) \
> --format=d \
> --backend=$(TRACE_BACKEND) \
> < $< > $@," GEN $(patsubst %-timestamp,%,$@)")
> @cmp -s $@ $(patsubst %-timestamp,%,$@) || cp $@ $(patsubst
> %-timestamp,%,$@)
> -$(obj)/generated-tracers-dtrace.h: trace/generated-tracers-dtrace.dtrace
> +$(obj)/generated-tracers-dtrace.h: $(obj)/generated-tracers.dtrace
> $(call quiet-command,dtrace -o $@ -h -s $<, " GEN $@")
> -trace-obj-$(CONFIG_TRACE_DTRACE) += generated-tracers-dtrace.o
> -
> +$(obj)/generated-tracers.o: $(obj)/generated-tracers.dtrace
> +endif
> ######################################################################
> # Backend code
This patch looks outdated. Last revision in master has an explicit line to
generate a .o that reads:
#v+
$(obj)/generated-tracers-dtrace.o: trace/generated-tracers-dtrace.dtrace
$(call quiet-command,dtrace -o $@ -G -s $<, " GEN $@")
#v-
> @@ -56,3 +55,4 @@ trace-obj-$(CONFIG_TRACE_DEFAULT) += default.o
> trace-obj-$(CONFIG_TRACE_SIMPLE) += simple.o
> trace-obj-$(CONFIG_TRACE_STDERR) += stderr.o
> trace-obj-y += control.o
> +trace-obj-y += generated-tracers.o
> --
> 1.8.1
Lluis
--
"And it's much the same thing with knowledge, for whenever you learn
something new, the whole world becomes that much richer."
-- The Princess of Pure Reason, as told by Norton Juster in The Phantom
Tollbooth
- [Qemu-devel] [PATCH 12/27] libcacard: rewrite Makefile in non-recursive style, (continued)
- [Qemu-devel] [PATCH 12/27] libcacard: rewrite Makefile in non-recursive style, Paolo Bonzini, 2013/01/12
- [Qemu-devel] [PATCH 13/27] libcacard: link vscclient to dynamic library, Paolo Bonzini, 2013/01/12
- [Qemu-devel] [PATCH 15/27] build: rename oslib-obj-y to util-obj-y, Paolo Bonzini, 2013/01/12
- [Qemu-devel] [PATCH 14/27] libcacard: list oslib-obj-y file explicitly, Paolo Bonzini, 2013/01/12
- [Qemu-devel] [PATCH 17/27] build: move files away from tools-obj-y, common-obj-y, user-obj-y, Paolo Bonzini, 2013/01/12
- [Qemu-devel] [PATCH 20/27] build: consolidate multiple variables into universal-obj-y, Paolo Bonzini, 2013/01/12
- [Qemu-devel] [PATCH 19/27] build: move qobject files to qobject/ and libqemuutil.a, Paolo Bonzini, 2013/01/12
- [Qemu-devel] [PATCH 21/27] build: move QAPI definitions for QEMU out of qapi-obj-y, Paolo Bonzini, 2013/01/12
- [Qemu-devel] [PATCH 18/27] build: move libqemuutil.a components to util/, Paolo Bonzini, 2013/01/12
- [Qemu-devel] [PATCH 25/27] build: some simplifications for "trace/Makefile.objs", Paolo Bonzini, 2013/01/12
- Re: [Qemu-devel] [PATCH 25/27] build: some simplifications for "trace/Makefile.objs",
Lluís Vilanova <=
- [Qemu-devel] [PATCH 26/27] build: fold trace-obj-y into libqemuutil.a, Paolo Bonzini, 2013/01/12
- [Qemu-devel] [PATCH 22/27] build: move base QAPI files to libqemuutil.a, Paolo Bonzini, 2013/01/12
- [Qemu-devel] [PATCH 23/27] build: move version-obj-y to the generic LINK rule, Paolo Bonzini, 2013/01/12
- [Qemu-devel] [PATCH 27/27] build: improve quiet output for .stp rules, Paolo Bonzini, 2013/01/12
- [Qemu-devel] [PATCH 24/27] build: remove coroutine-obj-y, Paolo Bonzini, 2013/01/12
- Re: [Qemu-devel] [PULL 00/27] More source reorganization and build system TLC, Stefan Hajnoczi, 2013/01/14
- Re: [Qemu-devel] [PULL 00/27] More source reorganization and build system TLC, Andreas Färber, 2013/01/18