[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 1/5] Makefile: Keep trace-events-subdirs ordered
From: |
Stefan Hajnoczi |
Subject: |
[PULL 1/5] Makefile: Keep trace-events-subdirs ordered |
Date: |
Thu, 30 Jan 2020 21:38:10 +0000 |
From: Philippe Mathieu-Daudé <address@hidden>
Adding the same directory multiple times to trace-events-subdirs
might trigger build failures, in particular when using the LTTng
Userspace Tracer library as backend.
For example when using two times the hw/core/ directory:
$ ./configure --enable-trace-backends=ust && make
[...]
CC trace-ust-all.o
In file included from trace-ust-all.h:13,
from trace-ust-all.c:13:
trace-ust-all.h:35151:1: error: redefinition of
‘__tracepoint_cb_qemu___loader_write_rom’
35151 | TRACEPOINT_EVENT(
| ^~~~~~~~~~~~~~~~
trace-ust-all.h:31791:1: note: previous definition of
‘__tracepoint_cb_qemu___loader_write_rom’ was here
31791 | TRACEPOINT_EVENT(
| ^~~~~~~~~~~~~~~~
To ease review and reduce likelihood of merge failures (see [*]),
keep trace-events-subdirs ordered when possible, following eb7ccb3c0.
[*] https://www.mail-archive.com/address@hidden/msg671007.html
Duplicate trace-events-subdirs entries generates duplicated
symbols when using the LTTng Userspace Tracer library.
Reviewed-by: Damien Hedde <address@hidden>
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Message-id: address@hidden
Message-Id: <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>
---
Makefile.objs | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Makefile.objs b/Makefile.objs
index f2da7241fc..26b9cff954 100644
--- a/Makefile.objs
+++ b/Makefile.objs
@@ -132,8 +132,8 @@ trace-events-subdirs += nbd
trace-events-subdirs += scsi
endif
ifeq ($(CONFIG_SOFTMMU),y)
-trace-events-subdirs += chardev
trace-events-subdirs += audio
+trace-events-subdirs += chardev
trace-events-subdirs += hw/9pfs
trace-events-subdirs += hw/acpi
trace-events-subdirs += hw/alpha
@@ -181,6 +181,7 @@ trace-events-subdirs += migration
trace-events-subdirs += net
trace-events-subdirs += ui
endif
+trace-events-subdirs += hw/core
trace-events-subdirs += hw/display
trace-events-subdirs += qapi
trace-events-subdirs += qom
@@ -193,7 +194,6 @@ trace-events-subdirs += target/riscv
trace-events-subdirs += target/s390x
trace-events-subdirs += target/sparc
trace-events-subdirs += util
-trace-events-subdirs += hw/core
trace-events-files = $(SRC_PATH)/trace-events
$(trace-events-subdirs:%=$(SRC_PATH)/%/trace-events)
--
2.24.1
- [PULL 0/5] Tracing patches, Stefan Hajnoczi, 2020/01/30
- [PULL 1/5] Makefile: Keep trace-events-subdirs ordered,
Stefan Hajnoczi <=
- [PULL 2/5] docs/devel/tracing.txt: Recommend only trace_event_get_state_backends(), Stefan Hajnoczi, 2020/01/30
- [PULL 4/5] hw/display/qxl.c: Use trace_event_get_state_backends(), Stefan Hajnoczi, 2020/01/30
- [PULL 3/5] memory.c: Use trace_event_get_state_backends(), Stefan Hajnoczi, 2020/01/30
- [PULL 5/5] qemu_set_log_filename: filename argument may be NULL, Stefan Hajnoczi, 2020/01/30
- Re: [PULL 0/5] Tracing patches, Peter Maydell, 2020/01/31
- Re: [PULL 0/5] Tracing patches, Peter Maydell, 2020/01/31