Re: [Xen-devel] [PATCH] Encapsulate several OCaml types within xenctrl

[Rok Strnisa <rok.strnisa@xxxxxxxxxx>]

Hi Ian,

> > Note that this is not a backward-compatible change. For example, code in
> xcp's
> > xen-api component needs to be modified accordingly.
> 
> Is someone (you?) also taking care of that side of things? Do we need to
> co-ordinate applying this patch on both sides?

Yes. The code has already been written:
https://github.com/xen-org/xen-api/pull/631

Internally, we'll most likely start using a slightly different (due to an extra
type that needed encapsulation: Xenctrl.runstateinfo) patch to Xen soon, 
together
with the changes to xen-api.

I have already spoken to Dave about this change. (Adding Dave and Jon to To.)

> Are you proposing this as a change for Xen 4.2? We are currently in
> feature freeze so an argument needs to be made for an exception. Are the
> xen-api developers happy with this change for 4.2?

I see no reason not to include it immediately, since it is only a namespace
and naming (albeit a backward incompatible) change, not also a semantic change,
and therefore cannot introduce regressions when the client code is only adapted
to use the new namespaces and names.

Regards,
Rok
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel