[libvirt] [PATCH] hypervisor driver for Jailhouse

[Peter Krempa]

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20151111/09a3c25f/attachment-0001.sig>