[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] tools/libxc: Document checkpoint compression in xg_save_restore.h



On Thu, 2011-06-23 at 02:04 +0100, Shriram Rajagopalan wrote:
> # HG changeset patch
> # User Shriram Rajagopalan <rshriram@xxxxxxxxx>
> # Date 1308790913 25200
> # Node ID 8bef913a3c4d14d2246086ef30a8e80f45ad1beb
> # Parent  9eed27800ff6a2e6d73f138f20af072c1b41925e
> tools/libxc: Document checkpoint compression in xg_save_restore.h
> 
> Add comments to xg_save_restore.h explaining changes in Remus
> wire protocol when checkpoint compression is enabled.

Thanks!

> Signed-off-by: Shriram Rajagopalan <rshriram@xxxxxxxxx>
> 
> diff -r 9eed27800ff6 -r 8bef913a3c4d tools/libxc/xg_save_restore.h
> --- a/tools/libxc/xg_save_restore.h   Wed Jun 22 06:34:55 2011 -0700
> +++ b/tools/libxc/xg_save_restore.h   Wed Jun 22 18:01:53 2011 -0700
> @@ -82,9 +82,24 @@
>   *     page data        : PAGE_SIZE bytes for each page marked present in PFN
>   *                        array
>   *
> + * In case of Remus with checkpoint compression, since the compressed page 
> data
> + * can be of variable size, only the pfn array is sent with a +ve chunk type.
> + *

Is this true on every round or only for second and subsequent rounds?

How do we know on the receiving end whether or not we are using
checkpoint compression? Is there an XC_SAVE_ID which is sent to trigger
this mode?

>   * If the chunk type is -ve then chunk consists of one of a number of
>   * metadata types.  See definitions of XC_SAVE_ID_* below.
>   *
> + * If the chunk type is -ve and equals XC_SAVE_ID_COMPRESSED_DATA, then the
> + * chunk consists of compressed page data, in the following format:
> + *
> + *     unsigned long        : Size of the compressed chunk to follow
> + *     compressed page data : variable length data of size indicated above.
> + *                            This chunk consists of compressed page data. 
> The
> + *                            number of pages in one chunk varies with 
> respect
> + *                            to amount of space available in the sender's
> + *                            output buffer.
> + *
> + * There can be one or more chunks with type XC_SAVE_ID_COMPRESSED_DATA.

Having seen an XC_SAVE_ID_COMPRESSED_DATA can you ever see a +ve chunk
type at that point? IOW is XC_SAVE_ID_COMPRESSED_DATA also the trigger I
mentioned above?

Part of me thinks this bit belongs below with the #define and the other
part thinks its too big to fit in nicely there so it is better here.

Ian.

>   * If chunk type is 0 then body phase is complete.
>   *
>   * TAIL PHASE
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.