Re: [Xen-devel] [PATCH v9 03/25] libxc/migration: Specification update for DIRTY_PFN_LIST records

2016-01-27 Thread Andrew Cooper
On 27/01/16 10:00, Ian Campbell wrote: > On Wed, 2016-01-27 at 15:12 +0800, Wen Congyang wrote: >> On 01/27/2016 04:44 AM, Konrad Rzeszutek Wilk wrote: + 0x000F: DIRTY_PFN_LIST + >>> Perhaps make it part of the optional and prefix it with CHECKPOINT? >> IIUC, optional

Re: [Xen-devel] [PATCH v9 03/25] libxc/migration: Specification update for DIRTY_PFN_LIST records

2016-01-27 Thread Ian Campbell
On Wed, 2016-01-27 at 15:12 +0800, Wen Congyang wrote: > On 01/27/2016 04:44 AM, Konrad Rzeszutek Wilk wrote: > > > + 0x000F: DIRTY_PFN_LIST > > > + > > > > Perhaps make it part of the optional and prefix it with CHECKPOINT? > > IIUC, optional record can be ignored, but this

Re: [Xen-devel] [PATCH v9 03/25] libxc/migration: Specification update for DIRTY_PFN_LIST records

2016-01-26 Thread Wen Congyang
On 01/27/2016 04:44 AM, Konrad Rzeszutek Wilk wrote: >> + 0x000F: DIRTY_PFN_LIST >> + > > Perhaps make it part of the optional and prefix it with CHECKPOINT? IIUC, optional record can be ignored, but this record cannot be ignored. To Andrew Cooper: Should I mark this record as

Re: [Xen-devel] [PATCH v9 03/25] libxc/migration: Specification update for DIRTY_PFN_LIST records

2016-01-26 Thread Wen Congyang
On 01/27/2016 04:44 AM, Konrad Rzeszutek Wilk wrote: >> + 0x000F: DIRTY_PFN_LIST >> + > > Perhaps make it part of the optional and prefix it with CHECKPOINT? Will be fixed in the next version. Thanks Wen Congyang > >> + 0x0010 - 0x7FFF: Reserved for future

Re: [Xen-devel] [PATCH v9 03/25] libxc/migration: Specification update for DIRTY_PFN_LIST records

2016-01-26 Thread Konrad Rzeszutek Wilk
> + 0x000F: DIRTY_PFN_LIST > + Perhaps make it part of the optional and prefix it with CHECKPOINT? > + 0x0010 - 0x7FFF: Reserved for future _mandatory_ > records. > > 0x8000 - 0x: Reserved for future _optional_

[Xen-devel] [PATCH v9 03/25] libxc/migration: Specification update for DIRTY_PFN_LIST records

2015-12-29 Thread Wen Congyang
Used by secondary to send it's dirty bitmap to primary under COLO. Signed-off-by: Yang Hongyang Signed-off-by: Wen Congyang --- docs/specs/libxc-migration-stream.pandoc | 24 +++- tools/libxc/xc_sr_common.c |