From: Andrew Cooper <andrew.cooper3@citrix.com>
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: George Dunlap <george.dunlap@eu.citrix.com>,
xen-devel@lists.xen.org, ian.jackson@eu.citrix.com,
Rob Hoes <rob.hoes@citrix.com>,
dave.scott@eu.citrix.com
Subject: Re: [PATCH v6 00/11] libxl: ocaml: improve the bindings
Date: Tue, 10 Dec 2013 13:25:31 +0000 [thread overview]
Message-ID: <52A7164B.3040501@citrix.com> (raw)
In-Reply-To: <1386681620.30568.3.camel@kazak.uk.xensource.com>
On 10/12/13 13:20, Ian Campbell wrote:
> On Mon, 2013-12-09 at 15:17 +0000, Rob Hoes wrote:
>> This series contains version 6 of the remaining patches to fix the OCaml
>> bindings to libxl.
>>
>> The main change compared to version 5 is that we now properly register the
>> "user" values (OCaml values that are given to the libxl event system, and
>> returned to OCaml in callbacks) with the OCaml GC.
> So the release process has moved on sufficiently that I think we need to
> consider whether the previous release-ack still stands:
> http://thread.gmane.org/gmane.comp.emulators.xen.devel/180254/focus=180383
>
> I think the arguments made there still stand, in short it would be
> awesome if xapi could move to using libxl on top of 4.4 and the risks
> are almost entirely contained within this use case, which cannot be
> satisfied by the code as it stands today.
Irrespective of the (re)decision about the release ack (which I believe
should still stand),
Patches 10 and 11 are Coverity fixes from issues caused by the bits of
the series committed so far, so should be considered differently if the
release ack is rescinded.
~Andrew
next prev parent reply other threads:[~2013-12-10 13:25 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-09 15:17 [PATCH v6 00/11] libxl: ocaml: improve the bindings Rob Hoes
2013-12-09 15:17 ` [PATCH v6 01/11] libxl: ocaml: add simple test case for xentoollog Rob Hoes
2013-12-09 15:17 ` [PATCH v6 02/11] libxl: ocaml: implement some simple tests Rob Hoes
2013-12-09 15:17 ` [PATCH v6 03/11] libxl: ocaml: event management Rob Hoes
2013-12-10 13:48 ` Ian Campbell
2013-12-10 15:48 ` Rob Hoes
2013-12-10 16:00 ` Ian Campbell
2013-12-10 16:46 ` Rob Hoes
2013-12-10 16:12 ` David Scott
2013-12-09 15:17 ` [PATCH v6 04/11] libxl: ocaml: allow device operations to be called asynchronously Rob Hoes
2013-12-10 13:25 ` Ian Campbell
2013-12-10 13:28 ` Ian Campbell
2013-12-10 14:47 ` Rob Hoes
2013-12-09 15:17 ` [PATCH v6 05/11] libxl: ocaml: add disk and cdrom helper functions Rob Hoes
2013-12-09 15:17 ` [PATCH v6 06/11] libxl: ocaml: add VM lifecycle operations Rob Hoes
2013-12-10 13:29 ` Ian Campbell
2013-12-10 16:13 ` David Scott
2013-12-09 15:17 ` [PATCH v6 07/11] libxl: ocaml: add console reader functions Rob Hoes
2013-12-09 15:17 ` [PATCH v6 08/11] libxl: ocaml: drop the ocaml heap lock before calling into libxl Rob Hoes
2013-12-10 13:34 ` Ian Campbell
2013-12-10 14:51 ` Rob Hoes
2013-12-10 16:01 ` Ian Campbell
2013-12-10 16:13 ` Rob Hoes
2013-12-09 15:17 ` [PATCH v6 09/11] libxl: ocaml: add some missing CAML macros Rob Hoes
2013-12-09 15:17 ` [PATCH v6 10/11] libxl: ocaml: fix memory corruption when converting string and key/values lists Rob Hoes
2013-12-09 15:17 ` [PATCH v6 11/11] libxl: ocaml: remove dead code in xentoollog bindings Rob Hoes
2013-12-10 11:24 ` [PATCH v6 00/11] libxl: ocaml: improve the bindings Rob Hoes
2013-12-10 13:20 ` Ian Campbell
2013-12-10 13:25 ` Andrew Cooper [this message]
2013-12-10 13:42 ` Ian Campbell
2013-12-10 14:10 ` George Dunlap
2013-12-10 14:24 ` George Dunlap
2013-12-10 14:34 ` David Scott
2013-12-10 15:42 ` George Dunlap
2013-12-10 15:48 ` David Scott
2013-12-10 15:48 ` Ian Campbell
2013-12-10 15:53 ` Ian Jackson
2013-12-10 16:00 ` George Dunlap
2013-12-10 16:57 ` George Dunlap
2013-12-10 17:01 ` Rob Hoes
2013-12-10 15:50 ` Ian Jackson
2013-12-10 15:57 ` George Dunlap
2013-12-10 16:04 ` Ian Campbell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=52A7164B.3040501@citrix.com \
--to=andrew.cooper3@citrix.com \
--cc=Ian.Campbell@citrix.com \
--cc=dave.scott@eu.citrix.com \
--cc=george.dunlap@eu.citrix.com \
--cc=ian.jackson@eu.citrix.com \
--cc=rob.hoes@citrix.com \
--cc=xen-devel@lists.xen.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.