From: Paolo Bonzini <pbonzini@redhat.com>
To: qemu-devel@nongnu.org
Subject: [Qemu-devel] [PATCH 00/11] Trim dependencies of qemu-ga, vscclient, tests
Date: Mon, 29 Oct 2012 19:07:07 +0100 [thread overview]
Message-ID: <1351534038-15503-1-git-send-email-pbonzini@redhat.com> (raw)
qemu-ga and vscclient have a very long list of dependencies, but
this is mostly because of tools-obj-y and because of qemu-tool.c's
own dependencies.
These are complex to disentangle, and became worse with recent additions
(especially fdsets). Luckily, some dependencies (for example on
qemu-timer.c, patches 8-10) can be removed from qemu-tool.c. When not
possible or overly complex, stuff can be moved out of qemu-tool.c and
into the files that actually require the stubbed-out functions, using
weak references.
I would like to do more of these, but because of the currently-broken
state of Win32 and slightly different semantics of weak references
between ELF and Win32, this does just the minimum necessary.
Paolo
Paolo Bonzini (11):
janitor: move iovector functions out of cutils.c
build: move cutils.o and qemu-timer-common.o to oslib-obj-y
compiler: use weak aliases to provide default definitions
sockets: use weak aliases instead of qemu-tool.c
fdsets: use weak aliases instead of qemu-tool.c/qemu-user.c
iohandler: add weak alias in qemu-sockets.c, for qemu-ga
win32: add weak version of qemu_fd_register
qemu-timer: make initialization functions idempotent
main-loop: unify qemu_init_main_loop between QEMU and tools
qemu-tool: do not depend on qemu-timer.c
build: do not include main loop where it is not actually used
Makefile | 7 ++--
Makefile.objs | 13 ++++---
arch_init.h | 2 +-
compiler.h | 11 +++---
cutils.c | 108 ---------------------------------------------------------
iov.c | 103 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
main-loop.c | 5 ++-
main-loop.h | 10 ------
osdep.c | 30 ++++++++++++++++
oslib-win32.c | 5 +++
qemu-common.h | 1 -
qemu-sockets.c | 18 ++++++++++
qemu-timer.c | 12 +++++--
qemu-tool.c | 35 +------------------
qemu-user.c | 20 -----------
qmp.c | 3 +-
tests/Makefile | 4 +--
vl.c | 5 ---
18 file modificati, 188 inserzioni(+), 204 rimozioni(-)
--
1.7.12.1
next reply other threads:[~2012-10-29 18:07 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-29 18:07 Paolo Bonzini [this message]
2012-10-29 18:07 ` [Qemu-devel] [PATCH 01/11] janitor: move iovector functions out of cutils.c Paolo Bonzini
2012-10-29 18:07 ` [Qemu-devel] [PATCH 02/11] build: move cutils.o and qemu-timer-common.o to oslib-obj-y Paolo Bonzini
2012-10-29 18:07 ` [Qemu-devel] [PATCH 03/11] compiler: use weak aliases to provide default definitions Paolo Bonzini
2012-10-29 18:07 ` [Qemu-devel] [PATCH 04/11] sockets: use weak aliases instead of qemu-tool.c Paolo Bonzini
2012-10-29 18:07 ` [Qemu-devel] [PATCH 05/11] fdsets: use weak aliases instead of qemu-tool.c/qemu-user.c Paolo Bonzini
2012-10-29 18:07 ` [Qemu-devel] [PATCH 06/11] iohandler: add weak alias in qemu-sockets.c, for qemu-ga Paolo Bonzini
2012-10-29 18:07 ` [Qemu-devel] [PATCH 07/11] win32: add weak version of qemu_fd_register Paolo Bonzini
2012-10-29 18:07 ` [Qemu-devel] [PATCH 08/11] qemu-timer: make initialization functions idempotent Paolo Bonzini
2012-10-29 18:07 ` [Qemu-devel] [PATCH 09/11] main-loop: unify qemu_init_main_loop between QEMU and tools Paolo Bonzini
2012-10-29 18:07 ` [Qemu-devel] [PATCH 10/11] qemu-tool: do not depend on qemu-timer.c Paolo Bonzini
2012-10-29 18:07 ` [Qemu-devel] [PATCH 11/11] build: do not include main loop where it is not actually used Paolo Bonzini
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=1351534038-15503-1-git-send-email-pbonzini@redhat.com \
--to=pbonzini@redhat.com \
--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).