Re: [Xen-devel] [PATCH v11 05/27] tools/libx{l, c}: add back channel to libxc

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Changlong Xie writes ("[PATCH v11 05/27] tools/libx{l,c}: add back channel to 
libxc"):
> [stuff]

A good explanation, thanks.

> Note: it is different from the paper. We change the original design to
> the current one, according to our following concerns:
> 1. The original design needs extra memory on Secondary host. When there's
>    multiple backups on one host, the memory cost is high.
> 2. The memory cache code will be another 1k+, it will make the review
>    more time consuming.

Again, good.

> Note: this patch merely adds new parameters to various prototypes and
> functions. The new parameters are used in later patch called
> "libxc/restore: send dirty pfn list to primary when checkpoint under
> COLO".

Thanks for splitting this off like that; it makes review much easier.

Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>

Ian.

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