[Xen-devel] [PATCH 5/6] docs: Migration v2 is now no longer draft

[Andrew Cooper <andrew.cooper3@xxxxxxxxxx>]

Add further instructions to the libxc "Future Extensions" section, and
provide such a section for libxl.

In addition, drop the "In experimental __func__" IPRINTF()s from the
libxc implementations.

Finally, a correction to libxl's "Not Yet Included" section which
should have been amended in c/s 7eaec00 when libxl Remus support was
introduced into the protocol.

Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
CC: Ian Campbell <Ian.Campbell@xxxxxxxxxx>
CC: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
CC: Wei Liu <wei.liu2@xxxxxxxxxx>
---
 docs/specs/libxc-migration-stream.pandoc |    5 ++++-
 docs/specs/libxl-migration-stream.pandoc |   19 ++++++++++++++++---
 tools/libxc/xc_sr_restore.c              |    1 -
 tools/libxc/xc_sr_save.c                 |    1 -
 4 files changed, 20 insertions(+), 6 deletions(-)

diff --git a/docs/specs/libxc-migration-stream.pandoc 
b/docs/specs/libxc-migration-stream.pandoc
index 9d8f17b..8cd678f 100644
--- a/docs/specs/libxc-migration-stream.pandoc
+++ b/docs/specs/libxc-migration-stream.pandoc
@@ -1,7 +1,7 @@
 % LibXenCtrl Domain Image Format
 % David Vrabel <<david.vrabel@xxxxxxxxxx>>
   Andrew Cooper <<andrew.cooper3@xxxxxxxxxx>>
-% Draft G
+% Revision 1
 
 Introduction
 ============
@@ -679,6 +679,9 @@ introduced in Xen 3.0.
 Future Extensions
 =================
 
+All changes to this specification should bump the revision number in
+the title block.
+
 All changes to the image or domain headers require the image version
 to be increased.
 
diff --git a/docs/specs/libxl-migration-stream.pandoc 
b/docs/specs/libxl-migration-stream.pandoc
index c24a434..cdec168 100644
--- a/docs/specs/libxl-migration-stream.pandoc
+++ b/docs/specs/libxl-migration-stream.pandoc
@@ -1,6 +1,6 @@
 % LibXenLight Domain Image Format
 % Andrew Cooper <<andrew.cooper3@xxxxxxxxxx>>
-% Draft B
+% Revision 1
 
 Introduction
 ============
@@ -41,8 +41,6 @@ Not Yet Included
 The following features are not yet fully specified and will be
 included in a future draft.
 
-* Remus
-
 * ARM
 
 
@@ -215,3 +213,18 @@ A checkpoint end record marks the end of a checkpoint in 
the image.
     +-------------------------------------------------+
 
 The end record contains no fields; its body_length is 0.
+
+
+Future Extensions
+=================
+
+All changes to this specification should bump the revision number in
+the title block.
+
+All changes to the header require the header version to be increased.
+
+The format may be extended by adding additional record types.
+
+Extending an existing record type must be done by adding a new record
+type.  This allows old images with the old record to still be
+restored.
diff --git a/tools/libxc/xc_sr_restore.c b/tools/libxc/xc_sr_restore.c
index ebc8f2f..df885b6 100644
--- a/tools/libxc/xc_sr_restore.c
+++ b/tools/libxc/xc_sr_restore.c
@@ -740,7 +740,6 @@ int xc_domain_restore(xc_interface *xch, int io_fd, 
uint32_t dom,
     if ( checkpointed_stream )
         assert(callbacks->checkpoint);
 
-    IPRINTF("In experimental %s", __func__);
     DPRINTF("fd %d, dom %u, hvm %u, pae %u, superpages %d"
             ", checkpointed_stream %d", io_fd, dom, hvm, pae,
             superpages, checkpointed_stream);
diff --git a/tools/libxc/xc_sr_save.c b/tools/libxc/xc_sr_save.c
index 7b38051..46700a1 100644
--- a/tools/libxc/xc_sr_save.c
+++ b/tools/libxc/xc_sr_save.c
@@ -850,7 +850,6 @@ int xc_domain_save(xc_interface *xch, int io_fd, uint32_t 
dom,
     if ( ctx.save.checkpointed )
         assert(callbacks->checkpoint && callbacks->postcopy);
 
-    IPRINTF("In experimental %s", __func__);
     DPRINTF("fd %d, dom %u, max_iters %u, max_factor %u, flags %u, hvm %d",
             io_fd, dom, max_iters, max_factor, flags, hvm);
 
-- 
1.7.10.4


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