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

2015-06-02 Thread Ian Campbell
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.

[PATCH v1 COLO Pre 00/12] from earlier is on my TODO list separately
already.

 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
 



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


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

2015-06-02 Thread Yang Hongyang

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@lists.xen.org
http://lists.xen.org/xen-devel


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

2015-05-27 Thread Yang Hongyang

ping...

On 05/20/2015 06:01 PM, Yang Hongyang wrote:

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@lists.xen.org
http://lists.xen.org/xen-devel