From: "Daniel P. Berrange" <berrange@redhat.com>
To: qemu-devel@nongnu.org
Cc: Peter Maydell <peter.maydell@linaro.org>
Subject: [Qemu-devel] [PATCH PULL v3 04/11] qom: allow QOM to be linked into tools binaries
Date: Tue, 15 Sep 2015 15:36:11 +0100 [thread overview]
Message-ID: <1442327778-6562-5-git-send-email-berrange@redhat.com> (raw)
In-Reply-To: <1442327778-6562-1-git-send-email-berrange@redhat.com>
The qom objects are currently added to common-obj-y
which is only linked into the system emulators. The
later crypto patches will depend on QOM infrastructure
and will also be used from tools binaries. Thus the QOM
objects are moved into a new qom-obj-y variable which
can be referenced when linking tools, system emulators
and tests.
Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
---
Makefile | 8 +++++---
Makefile.objs | 5 +++++
Makefile.target | 2 ++
qom/Makefile.objs | 7 ++++---
tests/Makefile | 4 +---
5 files changed, 17 insertions(+), 9 deletions(-)
diff --git a/Makefile b/Makefile
index 9c08b11..36943da 100644
--- a/Makefile
+++ b/Makefile
@@ -154,6 +154,7 @@ dummy := $(call unnest-vars,, \
block-obj-m \
crypto-obj-y \
crypto-aes-obj-y \
+ qom-obj-y \
common-obj-y \
common-obj-m)
@@ -176,6 +177,7 @@ SOFTMMU_SUBDIR_RULES=$(filter %-softmmu,$(SUBDIR_RULES))
$(SOFTMMU_SUBDIR_RULES): $(block-obj-y)
$(SOFTMMU_SUBDIR_RULES): $(crypto-obj-y)
+$(SOFTMMU_SUBDIR_RULES): $(qom-obj-y)
$(SOFTMMU_SUBDIR_RULES): config-all-devices.mak
subdir-%:
@@ -230,9 +232,9 @@ util/module.o-cflags = -D'CONFIG_BLOCK_MODULES=$(block-modules)'
qemu-img.o: qemu-img-cmds.h
-qemu-img$(EXESUF): qemu-img.o $(block-obj-y) $(crypto-obj-y) libqemuutil.a libqemustub.a
-qemu-nbd$(EXESUF): qemu-nbd.o $(block-obj-y) $(crypto-obj-y) libqemuutil.a libqemustub.a
-qemu-io$(EXESUF): qemu-io.o $(block-obj-y) $(crypto-obj-y) libqemuutil.a libqemustub.a
+qemu-img$(EXESUF): qemu-img.o $(block-obj-y) $(crypto-obj-y) $(qom-obj-y) libqemuutil.a libqemustub.a
+qemu-nbd$(EXESUF): qemu-nbd.o $(block-obj-y) $(crypto-obj-y) $(qom-obj-y) libqemuutil.a libqemustub.a
+qemu-io$(EXESUF): qemu-io.o $(block-obj-y) $(crypto-obj-y) $(qom-obj-y) libqemuutil.a libqemustub.a
qemu-bridge-helper$(EXESUF): qemu-bridge-helper.o
diff --git a/Makefile.objs b/Makefile.objs
index 655c1f0..3df2efc 100644
--- a/Makefile.objs
+++ b/Makefile.objs
@@ -26,6 +26,11 @@ block-obj-m = block/
crypto-obj-y = crypto/
crypto-aes-obj-y = crypto/
+#######################################################################
+# qom-obj-y is code used by both qemu system emulation and qemu-img
+
+qom-obj-y = qom/
+
######################################################################
# smartcard
diff --git a/Makefile.target b/Makefile.target
index 3d97966..edd136f 100644
--- a/Makefile.target
+++ b/Makefile.target
@@ -172,11 +172,13 @@ dummy := $(call unnest-vars,.., \
block-obj-m \
crypto-obj-y \
crypto-aes-obj-y \
+ qom-obj-y \
common-obj-y \
common-obj-m)
target-obj-y := $(target-obj-y-save)
all-obj-y += $(common-obj-y)
all-obj-y += $(target-obj-y)
+all-obj-y += $(qom-obj-y)
all-obj-$(CONFIG_SOFTMMU) += $(block-obj-y)
all-obj-$(CONFIG_USER_ONLY) += $(crypto-aes-obj-y)
all-obj-$(CONFIG_SOFTMMU) += $(crypto-obj-y)
diff --git a/qom/Makefile.objs b/qom/Makefile.objs
index 985003b..516349e 100644
--- a/qom/Makefile.objs
+++ b/qom/Makefile.objs
@@ -1,3 +1,4 @@
-common-obj-y = object.o container.o qom-qobject.o
-common-obj-y += cpu.o
-common-obj-y += object_interfaces.o
+qom-obj-y = object.o container.o qom-qobject.o
+qom-obj-y += object_interfaces.o
+
+common-obj-y = cpu.o
diff --git a/tests/Makefile b/tests/Makefile
index d77ccee..ddd5148 100644
--- a/tests/Makefile
+++ b/tests/Makefile
@@ -284,9 +284,7 @@ QEMU_CFLAGS += -I$(SRC_PATH)/tests
# Deps that are common to various different sets of tests below
test-util-obj-y = libqemuutil.a libqemustub.a
-test-qom-obj-y = qom/object.o qom/qom-qobject.o \
- qom/container.o qom/object_interfaces.o \
- $(test-util-obj-y)
+test-qom-obj-y = $(qom-obj-y) $(test-util-obj-y)
test-qapi-obj-y = tests/test-qapi-visit.o tests/test-qapi-types.o \
tests/test-qapi-event.o \
$(test-qom-obj-y)
--
2.4.3
next prev parent reply other threads:[~2015-09-15 14:36 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-15 14:36 [Qemu-devel] [PATCH PULL v3 00/11] Extract TLS handling code from VNC server Daniel P. Berrange
2015-09-15 14:36 ` [Qemu-devel] [PATCH PULL v3 01/11] qapi: allow override of default enum prefix naming Daniel P. Berrange
2015-09-15 14:36 ` [Qemu-devel] [PATCH PULL v3 02/11] tests: remove repetition in unit test object deps Daniel P. Berrange
2015-09-15 14:36 ` [Qemu-devel] [PATCH PULL v3 03/11] crypto: move crypto objects out of libqemuutil.la Daniel P. Berrange
2015-09-15 14:36 ` Daniel P. Berrange [this message]
2015-09-15 14:36 ` [Qemu-devel] [PATCH PULL v3 05/11] crypto: introduce new base module for TLS credentials Daniel P. Berrange
2015-09-15 14:36 ` [Qemu-devel] [PATCH PULL v3 06/11] crypto: introduce new module for TLS anonymous credentials Daniel P. Berrange
2015-09-15 14:36 ` [Qemu-devel] [PATCH PULL v3 07/11] crypto: introduce new module for TLS x509 credentials Daniel P. Berrange
2015-09-15 14:36 ` [Qemu-devel] [PATCH PULL v3 08/11] crypto: add sanity checking of " Daniel P. Berrange
2015-09-20 4:47 ` Peter Crosthwaite
2015-09-21 8:48 ` Daniel P. Berrange
2015-09-21 16:38 ` Daniel P. Berrange
2015-09-15 14:36 ` [Qemu-devel] [PATCH PULL v3 09/11] crypto: introduce new module for handling TLS sessions Daniel P. Berrange
2015-09-15 14:36 ` [Qemu-devel] [PATCH PULL v3 10/11] ui: fix return type for VNC I/O functions to be ssize_t Daniel P. Berrange
2015-09-15 14:36 ` [Qemu-devel] [PATCH PULL v3 11/11] ui: convert VNC server to use QCryptoTLSSession Daniel P. Berrange
2015-09-15 16:24 ` [Qemu-devel] [PATCH PULL v3 00/11] Extract TLS handling code from VNC server Peter Maydell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1442327778-6562-5-git-send-email-berrange@redhat.com \
--to=berrange@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).