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

Re: [Xen-devel] [PATCH/RFC 00/14] libxl: disk configuration handling

On Thu, 2011-05-12 at 15:36 +0100, Ian Jackson wrote:
> This is the latest version of my disk config parsing patch series.
> 00-04  Hopefully uncontroversial cleanups and minor features.

For these:
Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>

Although I'm not sure about the benefit of #4, it is consistent with
e.g. free() accepting NULL so why not.

> 05-07  Preparatory work.
> 08-10  The new parser and its documentation.
> 11-12  Replace old parsers with calls to the new one.
> 13-14  Two features, one of them essential.
> The use of the pcre library is gone; everything is done with flex now.
> The result is reasonably clean.
> You can see how it improves the approach needed for addition of new
> features to the disk config syntax by looking at patch 13, which is
> precisely such a new feature.  In patch 13 everything but the change
> to libxlu_disk_l.l is autogenerated, so it is that subset that is the
> relevant change.
> I'd like to apply as many of these as I can get acks for :-).
> Signed-off-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel

Xen-devel mailing list



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