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
Attachment:
signature.asc
Description: Digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list