[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 08/10] build: Add SPICE_CFLAGS and SPICE_LIBS to relevant fil
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH 08/10] build: Add SPICE_CFLAGS and SPICE_LIBS to relevant files |
Date: |
Fri, 26 Jun 2020 18:26:13 +0100 |
User-agent: |
Mutt/1.14.0 (2020-05-02) |
On Fri, Jun 26, 2020 at 06:43:05PM +0200, Christophe de Dinechin wrote:
> Instead of adding the spice build flags to the top-level build
> options, add them where they are necessary. This is a step to move the
> burden of linking with spice libraries away from the top-level qemu.
>
> Signed-off-by: Christophe de Dinechin <dinechin@redhat.com>
> ---
> configure | 4 ++--
> hw/display/Makefile.objs | 1 +
> hw/i386/Makefile.objs | 1 +
> monitor/Makefile.objs | 3 +++
> softmmu/Makefile.objs | 2 +-
> stubs/Makefile.objs | 2 +-
> ui/Makefile.objs | 4 ++--
> util/module.c | 13 +++++++++++--
> 8 files changed, 22 insertions(+), 8 deletions(-)
> diff --git a/stubs/Makefile.objs b/stubs/Makefile.objs
> index f32b9e47a3..1df8bb3814 100644
> --- a/stubs/Makefile.objs
> +++ b/stubs/Makefile.objs
> @@ -19,10 +19,10 @@ stub-obj-y += replay.o
> stub-obj-y += runstate-check.o
> stub-obj-$(CONFIG_SOFTMMU) += semihost.o
> stub-obj-y += set-fd-handler.o
> -stub-obj-y += vmgenid.o
> stub-obj-y += sysbus.o
> stub-obj-y += tpm.o
> stub-obj-y += trace-control.o
> +stub-obj-y += vmgenid.o
> stub-obj-y += vmstate.o
> stub-obj-$(CONFIG_SOFTMMU) += win32-kbd-hook.o
>
This looks unrelated to this series.
> diff --git a/util/module.c b/util/module.c
> index 2fa93561fe..29b4806520 100644
> --- a/util/module.c
> +++ b/util/module.c
> @@ -22,11 +22,11 @@
> #ifdef CONFIG_MODULE_UPGRADES
> #include "qemu-version.h"
> #endif
> -#ifdef CONFIG_TRACE_RECORDER
> #include "trace/recorder.h"
> -#endif
>
>
> +RECORDER(modules, 16, "QEMU load modules");
> +
> typedef struct ModuleEntry
> {
> void (*init)(void);
> @@ -85,6 +85,15 @@ void register_dso_module_init(void (*fn)(void),
> module_init_type type)
> {
> ModuleEntry *e;
>
> +#ifdef CONFIG_TRACE_RECORDER
> + static const char *name[] = {
> + "MIGRATION", "BLOCK", "OPTS", "QOM",
> + "TRACE", "XEN_BACKEND", "LIBQOS", "FUZZ_TARGET",
> + "MAX"
> + };
> +#endif
> + record(modules, "Register DSO module init %p type %u %+s",
> + fn, type, name[type]);
> init_lists();
This looks unrelated too, but in general debugging should go via QEMU's
standard trace backends.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- Re: [PATCH 04/10] spice: Move all the spice-related code in spice-app.so, (continued)
Re: [PATCH 04/10] spice: Move all the spice-related code in spice-app.so, Gerd Hoffmann, 2020/06/29
[PATCH 05/10] build: Avoid build failure when building drivers as modules, Christophe de Dinechin, 2020/06/26
[PATCH 06/10] trivial: Remove extra trailing whitespace, Christophe de Dinechin, 2020/06/26
[PATCH 07/10] qxl - FIXME: Build as module, Christophe de Dinechin, 2020/06/26
[PATCH 08/10] build: Add SPICE_CFLAGS and SPICE_LIBS to relevant files, Christophe de Dinechin, 2020/06/26
[PATCH 09/10] spice: Put spice functions in a separate load module, Christophe de Dinechin, 2020/06/26
[PATCH 10/10] REMOVE: Instrumentation to show the module functions being replaced, Christophe de Dinechin, 2020/06/26