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

Re: [Xen-devel] [PATCH v5 00/12] libxl: ocaml: improve the bindings

Ian Campbell wrote:
> On Tue, 2013-11-26 at 17:52 +0000, Rob Hoes wrote:
> > This series contains version 5 of the remaining patches to fix the
> > OCaml bindings to libxl.
> Since we are now into the freeze period we (really, you ;-) need to make
> an argument to the release manager (George) for why this should go in now.
> His guidance is in
> http://article.gmane.org/gmane.comp.emulators.xen.devel/180209.
> The benefit of this series is that it allows for ocaml based xen
> toolstacks (specifically xapi) to start making the transition to using
> libxl. It would be awesome if 4.4 provided a decent baseline for that.

I would indeed be great if we can start using libxl from Xen 4.4 out of the box!

> The risk is that we break the bindings for some existing users, but I am
> 99.9% certain there can be none of those. Effectively Rob is the only user
> today. The bindings in 4.3 were not really usable, at least not in anger.

I don't think anyone would have been really able to use the bindings as they 
were, because they were incomplete and caused memory corruption. In a way, you 
could consider the whole series to be one large bug fix :)

About 80% of my original series has already been merged. This contained really 
just the ground work. This version adds some essential bindings to VM lifecycle 
operations, the event system, and some bug fixes. These patches only affect 
files in tools/ocaml (plus .git/.hgignore).


> Overall I think the benefits outweigh the risks.
> (sorry if we made this argument on a previous posting, I don't see it in
> my QUEUE if so)
> Ian.
> > The main changes are a fix for a deadlock issue identified by Ian
> > Jackson (patches 8-9), and some fixes for issues found by Coverity
> (patches 10-11).
> >
> > For convenience, the patches in this series may be pulled using:
> > git pull git://github.com/robhoes/xen.git hydrogen-upstream-v5
> >

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.