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

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

Changlong Xie writes ("[PATCH v11 05/27] tools/libx{l,c}: add back channel to 
> [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>


Xen-devel mailing list



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