Re: [Xen-devel] [PATCH v7 COLO 02/18] tools/libxl: handle colo_context records in a libxl migration v2 stream

[Yang Hongyang <yanghy@xxxxxxxxxxxxxx>]

Hi Ian,

 Thanks for the review.

On 07/14/2015 11:19 PM, Ian Campbell wrote:
> On Thu, 2015-06-25 at 14:30 +0800, Yang Hongyang wrote:
> > From: Wen Congyang <wency@xxxxxxxxxxxxxx>
> >
> > Signed-off-by: Wen Congyang <wency@xxxxxxxxxxxxxx>
> > ---
> >   tools/libxl/libxl_internal.h    |  3 +++
> >   tools/libxl/libxl_stream_read.c | 51 +++++++++++++++++++++++++++++++++++++++++
> >   2 files changed, 54 insertions(+)
> This patch is certainly not so trivial that it can get away with a
> subject only and no commit body.
>
> The same goes for patch #3.
>
> In particular both of them add some new stream state which is not
> described anywhere.
This patch is changed a lot due to the migration v2 change, I've added some
commit message in v8, I will add more desciption as you suggested.

> Ian.
>
> .
--
Thanks,
Yang.

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