[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH 0/5] VirtualBox version 4.2 support for libvirt vbox driver



Peter Krempa <pkrempa <at> redhat.com> writes:

> 
> On 10/02/12 11:00, ryan woodsmall wrote:
> > This patch set adds VirtualBox 4.2 initial support for the libvirt vbox 
driver.
> > I've tested enough to check capabilities, create a VM, destroy it, etc. 
Five
> > patches total:
> >
> > - Patch 1 is the C API header file from Oracle, cleaned up for libvirt.
> > - Patch 2 is the version specific source file for version dependency.
> > - Patch 3 is the src/Makefile.am change to pick up the two new files.
> > - Patch 4 is the vbox driver support for the new VirtualBox API/version.
> > - Patch 5 is the vbox_tmpl.c template support for the new version.
> >
> > A few things have changed in the VirtualBox API - some small 
(capitalizations
> > of things in function names like Ip to IP and Dhcp to DHCP) and some 
much larger
> > (FindMedium is superceded by OpenMedium). The biggest change for the 
sake of this
> > patch set is the signature of CreateMachine is quite a bit different. 
Using the
> > Oracle source as a guide, to spin up a VM with a given UUID, it looks 
like a text
> > flag has to be passed in a new argument to CreateMachine. This flag is 
built in the
> > VirtualBox 4.2 specific ifdefs and is kind of ugly but works. 
Additionally, there
> > is now (unused) VM groups support in CreateMachine and the previous 
'osTypeId' arg
> > is currently set to nsnull as in the Oracle code.
> >
> > The FindMedium to OpenMedium changes were more straightforward and are 
pretty clear.
> > The rest of the vbox template changes are basically 
spelling/capitalization changes
> > from the looks of things.
> >
> > This probably isn't perfect, but it works on git and patched against 
0.10.2 for a
> > few quick tests. Not currently on the list, so ping me directly if you 
need any
> > other info on these, or if anything could use additional work. Thanks! -
r
> >
> >
> > ryan woodsmall (5):
> >    vbox C API header for VirtualBox v4.2
> >    vbox version-specific C file for VirtualBox v4.2
> >    Makefile.am additions for VirtualBox v4.2
> >    vbox driver support for VirtualBox v4.2
> >    vbox template support for VirtualBox v4.2
> >
> >   src/Makefile.am           |    3 +-
> >   src/vbox/vbox_CAPI_v4_2.h | 8855 
+++++++++++++++++++++++++++++++++++++++++++++
> >   src/vbox/vbox_V4_2.c      |   13 +
> >   src/vbox/vbox_driver.c    |    8 +
> >   src/vbox/vbox_tmpl.c      |   90 +-
> >   5 files changed, 8958 insertions(+), 11 deletions(-)
> >   create mode 100644 src/vbox/vbox_CAPI_v4_2.h
> >   create mode 100644 src/vbox/vbox_V4_2.c
> 
> As the patches can still be cleanly applied on the current master I'll 
> try to do a test/review and hopefully get it merged.
> 
> Peter
> 
> 

The 4 first patches can still be applied on the current master and I could 
apply the fifth one by rebasing Peter Krempa's repo on current master and 
then regenerating the patch and apply it. 
They work flawlessly with VirtualBox 4.2.12 r84980.
Could anyone review them for inclusion? If necessary I'm willing to make 
some changes to those patches and/or to provide the rebased fifth patch.
Regards, 
Manuel



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]