Re: [Xen-devel] [PATCH v1 0/5] Misc cleanups for libxl

[Yang Hongyang <yanghy@xxxxxxxxxxxxxx>]

Hi Ian,

  Thank you very much for taking time to review this!

On 06/02/2015 10:53 PM, Ian Campbell wrote:
> On Wed, 2015-05-20 at 18:01 +0800, Yang Hongyang wrote:
>
> I've just been through this, sorry for the delay.
>
> I also have written on my TODO list "Remus on Migration-v2 (v7)", but I
> have no such patches in my folders and I think I already applied
> everything to do with that, so I think that's just a stale entry. If
> there's something remus related I've dropped please point me at it.
Yeah, you have already applied all Remus on Migration-v2 stuff. Thanks again..

> "[PATCH v1 COLO Pre 00/12]" from earlier is on my TODO list separately
> already.
Thank you!

> > This patchset mainly focus on libxl save, most of the patches are
> > simply move codes out of libxl_dom.c, no functional changes, may conflict
> > with Andrew Cooper's libxl work for migration v2, so CCed Andrew for
> > comments.
> >
> > For convenience, my previous sent patch
> >    [PATCH v1] libxl/save: Refactor libxl__domain_suspend_state
> > is included in this series because all the other patches in this
> > series are based on that.
> >
> > Please see individual patch for detail.
> >
> > Can get the whole patchset from:
> >      https://github.com/macrosheep/xen/tree/misc-libxl-v1
> >
> > Yang Hongyang (5):
> >    libxl/save: Refactor libxl__domain_suspend_state
> >    tools/libxl: move domain suspend codes into a separate file
> >    tools/libxl: move remus codes into libxl_remus.c
> >    tools/libxl: move toolstack code into libxl_toolstack.c
> >    tools/libxl: move domain save codes into libxl_dom_save.c
> >
> >   tools/libxl/Makefile             |    5 +-
> >   tools/libxl/libxl.c              |   67 +--
> >   tools/libxl/libxl_dom.c          | 1202 --------------------------------------
> >   tools/libxl/libxl_dom_save.c     |  493 ++++++++++++++++
> >   tools/libxl/libxl_dom_suspend.c  |  400 +++++++++++++
> >   tools/libxl/libxl_internal.h     |   62 +-
> >   tools/libxl/libxl_netbuffer.c    |    2 +-
> >   tools/libxl/libxl_remus.c        |  306 ++++++++++
> >   tools/libxl/libxl_save_callout.c |    2 +-
> >   tools/libxl/libxl_toolstack.c    |  196 +++++++
> >   10 files changed, 1452 insertions(+), 1283 deletions(-)
> >   create mode 100644 tools/libxl/libxl_dom_save.c
> >   create mode 100644 tools/libxl/libxl_dom_suspend.c
> >   create mode 100644 tools/libxl/libxl_remus.c
> >   create mode 100644 tools/libxl/libxl_toolstack.c
>
> .
--
Thanks,
Yang.

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