From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Joelle van Dyne <j@getutm.app>
Cc: qemu-devel@nongnu.org
Subject: Re: [PATCH 04/10] meson: option to build as shared library
Date: Tue, 13 Oct 2020 16:57:57 +0100 [thread overview]
Message-ID: <20201013155757.GR70612@redhat.com> (raw)
In-Reply-To: <CA+E+eSBu_LOnQ=B+N7vm3H2zkBxey8z_T1V0JpSU8rZND6GRYQ@mail.gmail.com>
On Tue, Oct 13, 2020 at 08:16:46AM -0700, Joelle van Dyne wrote:
> I will start a separate conversation of UTM's license compatibility.
>
> Regarding the patch, would some sort of warning message in configure
> (if building as a shared library) regarding the license be wise? Or
> would it pollute the output logs?
I think there's also a more fundamental question of whether this is
a concept we actually want to support at all.
IIUC, this shared library just exposes a "qemu_main" method which
the external app has to jump into. IOW, the "char **argv" parameter
to qemu_main becomes the ELF library ABI. Declaring this a shared
library interface is a non-negligible commitment for QEMU maintainers,
especially given that there is alot about QMEU's command line parsing
that maintainers do not like and wish to change.
There is a further question about whether we want to commit to an
architectural model where fork() is not something we can use. A
significant area of development for QEMU right now is around the
modularization of the device model such that QEMU is no longer a
single process, but rather a collection of processes, each proc
responsible for some specific hardware emulation. Thus far all the
device modularization stuff is strictly optional, but we have the
freedom to make it mandatory and use it transparently behind the
scenes. If we declare this shared library model supported in order
to avoid use of fork, then we're restricting our future options
wrt device modularization.
So including the license issue, that's three major questionmarks
around desirability of supporting use of QEMU as a shared library.
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 :|
next prev parent reply other threads:[~2020-10-13 15:59 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-12 23:29 [PATCH 00/10] iOS and Apple Silicon host support Joelle van Dyne
2020-10-12 23:29 ` [PATCH 01/10] configure: option to disable host block devices Joelle van Dyne
2020-10-12 23:29 ` [PATCH 02/10] configure: cross-compiling without cross_prefix Joelle van Dyne
2020-10-12 23:29 ` [PATCH 03/10] qemu: add support for iOS host Joelle van Dyne
2020-10-12 23:29 ` [PATCH 04/10] meson: option to build as shared library Joelle van Dyne
2020-10-13 7:51 ` Daniel P. Berrangé
2020-10-13 14:41 ` BALATON Zoltan via
2020-10-13 14:46 ` Daniel P. Berrangé
2020-10-13 15:16 ` Joelle van Dyne
2020-10-13 15:57 ` Daniel P. Berrangé [this message]
2020-10-13 16:40 ` BALATON Zoltan via
2020-10-14 16:09 ` Joelle van Dyne
2020-10-13 15:23 ` BALATON Zoltan via
2020-10-12 23:29 ` [PATCH 05/10] slirp: update for iOS resolv fix Joelle van Dyne
2020-10-13 13:32 ` Philippe Mathieu-Daudé
2020-10-13 14:38 ` Marc-André Lureau
2020-10-13 14:59 ` Philippe Mathieu-Daudé
2020-10-14 15:54 ` Joelle van Dyne
2020-10-12 23:29 ` [PATCH 06/10] coroutine: add libucontext as external library Joelle van Dyne
2020-10-13 13:31 ` Stefan Hajnoczi
2020-10-13 14:49 ` BALATON Zoltan via
2020-10-13 15:25 ` Joelle van Dyne
2020-10-13 16:18 ` Daniel P. Berrangé
2020-10-12 23:29 ` [PATCH 07/10] tcg: implement bulletproof JIT Joelle van Dyne
2020-10-13 8:22 ` Philippe Mathieu-Daudé
2020-10-13 14:58 ` BALATON Zoltan via
2020-10-14 16:03 ` Joelle van Dyne
2020-10-14 20:35 ` Richard Henderson
2020-10-14 20:54 ` BALATON Zoltan via
2020-10-14 21:40 ` Richard Henderson
2020-10-14 20:58 ` Joelle van Dyne
2020-10-14 21:08 ` BALATON Zoltan via
2020-10-14 21:49 ` Richard Henderson
2020-10-14 22:54 ` Joelle van Dyne
2020-10-12 23:29 ` [PATCH 08/10] tcg: mirror mapping RWX pages for iOS optional Joelle van Dyne
2020-10-13 13:52 ` Paolo Bonzini
2020-10-13 15:10 ` Joelle van Dyne
2020-10-12 23:29 ` [PATCH 09/10] tcg: support JIT on Apple Silicon Joelle van Dyne
2020-10-13 13:55 ` Paolo Bonzini
2020-10-13 14:09 ` Peter Maydell
2020-10-13 15:13 ` Joelle van Dyne
2020-10-12 23:29 ` [PATCH 10/10] block: check availablity for preadv/pwritev on mac Joelle van Dyne
2020-10-13 1:21 ` [PATCH 00/10] iOS and Apple Silicon host support no-reply
2020-10-13 2:12 ` Joelle van Dyne
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=20201013155757.GR70612@redhat.com \
--to=berrange@redhat.com \
--cc=j@getutm.app \
--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).