Re: [Xen-devel] [PATCH v2 0/6] Misc cleanups for libxl

[Yang Hongyang <yanghy@xxxxxxxxxxxxxx>]

On 06/03/2015 06:51 PM, Andrew Cooper wrote:
> On 03/06/15 09:01, Yang Hongyang wrote:
> > This patchset mainly focus on libxl save, most of the patches are
> > simply move codes out of libxl_dom.c, except a refactor patch.
> >
> > Please see individual patch for detail.
> >
> > Can get the whole patchset from:
> >      https://github.com/macrosheep/xen/tree/misc-libxl-v2
> Overall, these look like good changes, although I have not reviewed them
> in detail.
Thank you! hope it won't affect migration v2 too much.

> ~Andrew
>
> > v1->v2:
> >    - use dsps for suspend_state and dss for save_state.
> >    - move resume code to libxl_dom_suspend.c
> >    - move toolstatck save/restore code to libxl_dom_save.c
> >    - move refactor pacth to the end so that rebase of the patchset easier.
> >
> > Yang Hongyang (6):
> >    tools/libxl: rename libxl__domain_suspend to libxl__domain_save
> >    tools/libxl: move domain suspend code into libxl_dom_suspend.c
> >    tools/libxl: move domain resume code into libxl_dom_suspend.c
> >    tools/libxl: move remus code into libxl_remus.c
> >    tools/libxl: move save/restore code into libxl_dom_save.c
> >    libxl/save: Refactor libxl__domain_suspend_state
> >
> >   tools/libxl/Makefile             |    5 +-
> >   tools/libxl/libxl.c              |  126 +---
> >   tools/libxl/libxl_dom.c          | 1202 --------------------------------------
> >   tools/libxl/libxl_dom_save.c     |  672 +++++++++++++++++++++
> >   tools/libxl/libxl_dom_suspend.c  |  465 +++++++++++++++
> >   tools/libxl/libxl_internal.h     |   65 ++-
> >   tools/libxl/libxl_netbuffer.c    |    2 +-
> >   tools/libxl/libxl_remus.c        |  307 ++++++++++
> >   tools/libxl/libxl_save_callout.c |    2 +-
> >   9 files changed, 1503 insertions(+), 1343 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
> .
--
Thanks,
Yang.

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