Re: [libvirt] [PATCH] hypervisor driver for Jailhouse
On Tue, Nov 10, 2015 at 13:17:41 +0100, Christian Loehle wrote:
>From README:
The jailhouse hypervisor driver for the libvirt project aims to provide
rudimentary support for managing jailhouse with the libvirt library. The
main advantage of this is the possibility to use virt-manager as a GUI
to manage Jailhouse cells. Thus the driver is mainly built around the
API calls that virt-manager uses and needs.
Would you mind posting a link to 'Jailhouse' for the lazy ones?
Due to the concept of Jailhouse a lot of libvirt functions can't
be
realized, so this driver isn't as full-featured as upstream drivers of
the libvirt project.
That is actually fine, since many of our drivers have limited
functionality.
Currently the driver relies on the Jailhouse binary, which has to be
passed when connecting a libvirt client to it(e.g. virt-manager -c
jailhouse:///path/to/jailhouse/tools/jailhouse). This has the advantage
This might be a bit problematic, but we'll see later.
that remote support can be easily done by not passing the original
Jailhouse binary, but an executable that redirects its parameters
through ssh to the real Jailhouse binary and outputs that output. Be
aware though that the driver doesn't store any information about cells,
so most API calls use "jailhouse cell list" every time they're called to
get the current state.
I would like to get Jailhouse support upstream, any feedback is greatly
appreciated.
--
Christian Loehle
I'd like ask that you repost this using git send-email, or at least
using git format-patch, so that it can be properly applied, and ...
diff --git a/configure.ac b/configure.ac
index f481c50..8b68828 100644
--- a/configure.ac
+++ b/configure.ac
[...]
@@ -722,6 +726,16 @@ AM_CONDITIONAL([WITH_VMWARE], [test
"$with_vmware"
= "yes"])
dnl
+dnl Checks for the Jailhouse driver
+dnl
+
+if test "$with_jailhouse" = "yes"; then
+ AC_DEFINE_UNQUOTED([WITH_JAILHOUSE], 1, [whether Jailhouse driver
is enabled])
^^ please don't paste patches into your mail client since it wraps long
lines and makes this patch totaly unusable.
+fi
Peter
Attachments:
- signature.asc (application/pgp-signature — 819 bytes)