Re: [Xen-devel] [PATCH v6 COLO 06/15] libxc/save: support COLO save

2015-06-09 Thread Yang Hongyang
On 06/09/2015 05:10 PM, Andrew Cooper wrote: On 09/06/15 10:09, Yang Hongyang wrote: Shouldn't get_dirty_pfn be mandatory for COLO streams (even if it is a noop to start with) ? It should be mandatory, it shouldn't be noop under COLO. perhaps we should add sanity check at the beginnin

Re: [Xen-devel] [PATCH v6 COLO 06/15] libxc/save: support COLO save

2015-06-09 Thread Andrew Cooper
On 09/06/15 10:09, Yang Hongyang wrote: > > >> >>> >> Shouldn't get_dirty_pfn be mandatory for COLO streams (even if it >> is a >> noop to start with) ? > > It should be mandatory, it shouldn't be noop under COLO. perhaps we > should > add sanity check at the beginning.

Re: [Xen-devel] [PATCH v6 COLO 06/15] libxc/save: support COLO save

2015-06-09 Thread Yang Hongyang
On 06/09/2015 04:51 PM, Andrew Cooper wrote: On 09/06/15 09:45, Yang Hongyang wrote: Even if there are no dirty pages on secondary, pfn_list shouldn't be NULL, it's just that pfn_list[0] will be 0. if pfn_list is NULL, there might be unexpected error happened. get_dirty_pfn() should be dec

Re: [Xen-devel] [PATCH v6 COLO 06/15] libxc/save: support COLO save

2015-06-09 Thread Andrew Cooper
On 09/06/15 09:45, Yang Hongyang wrote: > >>> Even if there are no dirty pages on secondary, pfn_list shouldn't be >>> NULL, it's just that pfn_list[0] will be 0. if pfn_list is NULL, >>> there might be unexpected error happened. >> >> get_dirty_pfn() should be declared alongside a >> >> struct pfn

Re: [Xen-devel] [PATCH v6 COLO 06/15] libxc/save: support COLO save

2015-06-09 Thread Yang Hongyang
On 06/09/2015 03:20 PM, Andrew Cooper wrote: On 09/06/2015 04:15, Yang Hongyang wrote: On 06/08/2015 09:04 PM, Andrew Cooper wrote: On 08/06/15 04:45, Yang Hongyang wrote: call callbacks->get_dirty_pfn() after suspend primary vm to get dirty pages on secondary vm, and send pages both dirty

Re: [Xen-devel] [PATCH v6 COLO 06/15] libxc/save: support COLO save

2015-06-09 Thread Andrew Cooper
On 09/06/2015 04:15, Yang Hongyang wrote: > > > On 06/08/2015 09:04 PM, Andrew Cooper wrote: >> On 08/06/15 04:45, Yang Hongyang wrote: >>> call callbacks->get_dirty_pfn() after suspend primary vm to >>> get dirty pages on secondary vm, and send pages both dirty on >>> primary/secondary to secondar

Re: [Xen-devel] [PATCH v6 COLO 06/15] libxc/save: support COLO save

2015-06-08 Thread Yang Hongyang
On 06/08/2015 09:04 PM, Andrew Cooper wrote: On 08/06/15 04:45, Yang Hongyang wrote: call callbacks->get_dirty_pfn() after suspend primary vm to get dirty pages on secondary vm, and send pages both dirty on primary/secondary to secondary. Signed-off-by: Yang Hongyang Signed-off-by: Wen Congy

Re: [Xen-devel] [PATCH v6 COLO 06/15] libxc/save: support COLO save

2015-06-08 Thread Yang Hongyang
On 06/08/2015 09:04 PM, Andrew Cooper wrote: On 08/06/15 04:45, Yang Hongyang wrote: call callbacks->get_dirty_pfn() after suspend primary vm to get dirty pages on secondary vm, and send pages both dirty on primary/secondary to secondary. Signed-off-by: Yang Hongyang Signed-off-by: Wen Congy

Re: [Xen-devel] [PATCH v6 COLO 06/15] libxc/save: support COLO save

2015-06-08 Thread Andrew Cooper
On 08/06/15 04:45, Yang Hongyang wrote: > call callbacks->get_dirty_pfn() after suspend primary vm to > get dirty pages on secondary vm, and send pages both dirty on > primary/secondary to secondary. > > Signed-off-by: Yang Hongyang > Signed-off-by: Wen Congyang > CC: Andrew Cooper > --- > tool

[Xen-devel] [PATCH v6 COLO 06/15] libxc/save: support COLO save

2015-06-07 Thread Yang Hongyang
call callbacks->get_dirty_pfn() after suspend primary vm to get dirty pages on secondary vm, and send pages both dirty on primary/secondary to secondary. Signed-off-by: Yang Hongyang Signed-off-by: Wen Congyang CC: Andrew Cooper --- tools/libxc/xc_sr_save.c | 49 +++