WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

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

To: Shriram Rajagopalan <rshriram@xxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] tools/libxc: Document checkpoint compression in xg_save_restore.h
From: Ian Campbell <Ian.Campbell@xxxxxxxxxx>
Date: Thu, 23 Jun 2011 15:56:32 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Delivery-date: Thu, 23 Jun 2011 07:57:14 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <8bef913a3c4d14d22460.1308791079@xxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Organization: Citrix Systems, Inc.
References: <8bef913a3c4d14d22460.1308791079@xxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
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