
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