From: Claudio Fontana <cfontana@suse.de>
To: Gerd Hoffmann <kraxel@redhat.com>, qemu-devel@nongnu.org
Cc: "Marc-André Lureau" <marcandre.lureau@redhat.com>,
"Daniel P. Berrangé" <berrange@redhat.com>,
"Eduardo Habkost" <ehabkost@redhat.com>,
"Paolo Bonzini" <pbonzini@redhat.com>
Subject: Re: [PULL 00/10] Modules 20200702 patches
Date: Fri, 3 Jul 2020 11:14:27 +0200 [thread overview]
Message-ID: <81e43374-25c7-5361-0ca3-04b26c0f4826@suse.de> (raw)
In-Reply-To: <20200702122048.27798-1-kraxel@redhat.com>
Hello Gerd,
I think in general the idea to make it easier to modularize things is great,
is this thought for devices only, or could I rework my changes to support modularizing per-target AccelClass types and all the related code on top of your design?
Thanks,
Claudio
On 7/2/20 2:20 PM, Gerd Hoffmann wrote:
> The following changes since commit fc1bff958998910ec8d25db86cd2f53ff125f7ab:
>
> hw/misc/pca9552: Add missing TypeInfo::class_size field (2020-06-29 21:16:10 +0100)
>
> are available in the Git repository at:
>
> git://git.kraxel.org/qemu tags/modules-20200702-pull-request
>
> for you to fetch changes up to 474a5d66036d18ee5ccaa88364660d05bf32127b:
>
> chardev: enable modules, use for braille (2020-07-01 21:08:11 +0200)
>
> ----------------------------------------------------------------
> qom: add support for qom objects in modules.
> build some devices (qxl, virtio-gpu, ccid, usb-redir) as modules.
> build braille chardev as module.
>
> note: qemu doesn't rebuild objects on cflags changes (specifically
> -fPIC being added when code is switched from builtin to module).
> Workaround for resulting build errors: "make clean", rebuild.
>
> ----------------------------------------------------------------
>
> Gerd Hoffmann (10):
> module: qom module support
> object: qom module support
> qdev: device module support
> build: fix device module builds
> ccid: build smartcard as module
> usb: build usb-redir as module
> vga: build qxl as module
> vga: build virtio-gpu only once
> vga: build virtio-gpu as module
> chardev: enable modules, use for braille
>
> Makefile.objs | 2 ++
> Makefile.target | 7 +++++
> include/qemu/module.h | 2 ++
> include/qom/object.h | 12 +++++++
> chardev/char.c | 2 +-
> hw/core/qdev.c | 6 ++--
> qdev-monitor.c | 5 +--
> qom/object.c | 14 +++++++++
> qom/qom-qmp-cmds.c | 3 +-
> softmmu/vl.c | 4 +--
> util/module.c | 67 ++++++++++++++++++++++++++++++++++++++++
> chardev/Makefile.objs | 5 ++-
> hw/Makefile.objs | 2 ++
> hw/display/Makefile.objs | 28 ++++++++++-------
> hw/usb/Makefile.objs | 13 +++++---
> 15 files changed, 148 insertions(+), 24 deletions(-)
>
next prev parent reply other threads:[~2020-07-03 9:31 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-02 12:20 [PULL 00/10] Modules 20200702 patches Gerd Hoffmann
2020-07-02 12:20 ` [PULL 01/10] module: qom module support Gerd Hoffmann
2020-07-02 12:20 ` [PULL 02/10] object: " Gerd Hoffmann
2020-07-02 12:20 ` [PULL 03/10] qdev: device " Gerd Hoffmann
2020-07-02 12:20 ` [PULL 04/10] build: fix device module builds Gerd Hoffmann
2020-07-03 9:01 ` Claudio Fontana
2020-07-03 10:25 ` Gerd Hoffmann
2020-07-03 13:00 ` Claudio Fontana
2020-07-06 13:14 ` Gerd Hoffmann
2020-07-02 12:20 ` [PULL 05/10] ccid: build smartcard as module Gerd Hoffmann
2020-07-02 12:20 ` [PULL 06/10] usb: build usb-redir " Gerd Hoffmann
2020-07-02 12:20 ` [PULL 07/10] vga: build qxl " Gerd Hoffmann
2020-07-02 12:20 ` [PULL 08/10] vga: build virtio-gpu only once Gerd Hoffmann
2020-07-02 12:20 ` [PULL 09/10] vga: build virtio-gpu as module Gerd Hoffmann
2020-07-02 12:20 ` [PULL 10/10] chardev: enable modules, use for braille Gerd Hoffmann
2020-07-03 8:54 ` [PULL 00/10] Modules 20200702 patches Peter Maydell
2020-07-03 10:39 ` Gerd Hoffmann
2020-07-03 12:05 ` Paolo Bonzini
2020-07-06 13:20 ` Gerd Hoffmann
2020-07-06 14:30 ` Claudio Fontana
2020-07-03 9:14 ` Claudio Fontana [this message]
2020-07-03 10:29 ` Gerd Hoffmann
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=81e43374-25c7-5361-0ca3-04b26c0f4826@suse.de \
--to=cfontana@suse.de \
--cc=berrange@redhat.com \
--cc=ehabkost@redhat.com \
--cc=kraxel@redhat.com \
--cc=marcandre.lureau@redhat.com \
--cc=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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.