qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Fam Zheng <famz@redhat.com>
To: qemu-devel@nongnu.org
Cc: kwolf@redhat.com, peter.maydell@linaro.org, stefanha@redhat.com,
	mjt@tls.msk.ru, xiawenc@linux.vnet.ibm.com, alex@alex.org.uk,
	pbonzini@redhat.com, vilanova@ac.upc.edu, rth@twiddle.net
Subject: [Qemu-devel] [PATCH v13 0/8] Shared Library Module Support
Date: Fri, 11 Oct 2013 09:16:42 +0800	[thread overview]
Message-ID: <1381454210-30978-1-git-send-email-famz@redhat.com> (raw)

This series implements feature of shared object building as described in:

http://wiki.qemu.org/Features/Modules

The main idea behind modules is to isolate dependencies on third party
libraries from qemu executables, such as libglusterfs or librbd, so that the
end users can install core qemu package with fewer dependencies.  And only for
those who want to use particular modules, need they install qemu-foo
sub-package, which in turn requires libbar and libbiz packages.

It's implemented in three steps:

1. The first patches fix current build system to correctly handle nested
   variables and object specific options:

    [01/08] ui/Makefile.objs: delete unnecessary cocoa.o dependency
    [02/08] make.rule: fix $(obj) to a real relative path
    [03/08] rule.mak: allow per object cflags and libs

2. The Makefile changes adds necessary options and rules to build DSO objects:

    [04/08] build-sys: introduce common-obj-m and block-obj-m for DSO

3. The next patch adds code to load modules from installed directory:

    [05/08] module: implement module loading

A few more changes are following to complete it:

    [06/08] Makefile: install modules with "make install"
    [07/08] .gitignore: ignore module related files (dll, so, mo)

In the end of series, the block drivers are converted:

    [08/08] block: convert block drivers linked with libs to modules

v13: Drop --moddir option in configure. (Paolo)

v12: Rebase to current master, no conflict.
     Drop "-Wl,--enable-new-tags -Wl,-rpath,'$$ORIGIN'". (Paolo)

v11:
    [04] Link DSO with  -Wl,--enable-new-dtags -Wl,-rpath,'$$ORIGIN' (Richard)
    [05] Reuse module_init_type in module_load, no separate load type enums.
         Separate list of modules by type. It's simply list of built modules
         now. No whitelist option in configure.
         Support multiple module_init() in single module.

v10:
    All modules in a single directory (moddir), with module type prefixed:
        /usr/lib/qemu/block-{curl,iscsi,...}.so
    The module names for user to list in module whitelist is consequently:
        block-curl, block-iscsi, ui-*, etc.
    In Makfile, the installed module filename is simply generated by:
        $(subst /,-,%.so)
    Which is also the rule for module names.

    [05] Add #undef CONFIG_MODULE_WHITELIST in config-host.h.
         Use static array for whitelist. (Richard)


Fam Zheng (7):
  make.rule: fix $(obj) to a real relative path
  rule.mak: allow per object cflags and libs
  build-sys: introduce common-obj-m and block-obj-m for DSO
  module: implement module loading
  Makefile: install modules with "make install"
  .gitignore: ignore module related files (dll, so, mo)
  block: convert block drivers linked with libs to modules

Peter Maydell (1):
  ui/Makefile.objs: delete unnecessary cocoa.o dependency

 .gitignore            |   3 ++
 Makefile              |  30 +++++++++++++-
 Makefile.objs         |  19 ++-------
 Makefile.target       |  21 ++++++++--
 block/Makefile.objs   |  11 +++++-
 configure             |  73 ++++++++++++++++++++++------------
 include/qemu/module.h |  12 ++++++
 module-common.c       |  10 +++++
 rules.mak             |  80 +++++++++++++++++++++++++++++++------
 scripts/create_config |  14 +++++++
 ui/Makefile.objs      |   2 -
 util/module.c         | 107 +++++++++++++++++++++++++++++++++++++++++++++++++-
 12 files changed, 317 insertions(+), 65 deletions(-)
 create mode 100644 module-common.c

-- 
1.8.3.1

             reply	other threads:[~2013-10-11  1:17 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-11  1:16 Fam Zheng [this message]
2013-10-11  1:16 ` [Qemu-devel] [PATCH v13 1/8] ui/Makefile.objs: delete unnecessary cocoa.o dependency Fam Zheng
2013-10-11  1:16 ` [Qemu-devel] [PATCH v13 2/8] make.rule: fix $(obj) to a real relative path Fam Zheng
2013-10-11  1:16 ` [Qemu-devel] [PATCH v13 3/8] rule.mak: allow per object cflags and libs Fam Zheng
2013-10-11  1:16 ` [Qemu-devel] [PATCH v13 4/8] build-sys: introduce common-obj-m and block-obj-m for DSO Fam Zheng
2013-10-11  1:16 ` [Qemu-devel] [PATCH v13 5/8] module: implement module loading Fam Zheng
2013-10-14 16:38   ` Paolo Bonzini
2013-10-11  1:16 ` [Qemu-devel] [PATCH v13 6/8] Makefile: install modules with "make install" Fam Zheng
2013-10-11  1:16 ` [Qemu-devel] [PATCH v13 7/8] .gitignore: ignore module related files (dll, so, mo) Fam Zheng
2013-10-11  1:16 ` [Qemu-devel] [PATCH v13 8/8] block: convert block drivers linked with libs to modules Fam Zheng
2013-10-15 15:57 ` [Qemu-devel] [PATCH v13 0/8] Shared Library Module Support 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=1381454210-30978-1-git-send-email-famz@redhat.com \
    --to=famz@redhat.com \
    --cc=alex@alex.org.uk \
    --cc=kwolf@redhat.com \
    --cc=mjt@tls.msk.ru \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@twiddle.net \
    --cc=stefanha@redhat.com \
    --cc=vilanova@ac.upc.edu \
    --cc=xiawenc@linux.vnet.ibm.com \
    /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).