From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH v2 4/6] libxc/save: Adjust stream-position callbacks for checkpointed streams Date: Mon, 11 May 2015 13:05:48 +0100 Message-ID: <1431345948.8263.40.camel@citrix.com> References: <1431119675-23847-1-git-send-email-andrew.cooper3@citrix.com> <1431119675-23847-5-git-send-email-andrew.cooper3@citrix.com> <1431344184.8263.33.camel@citrix.com> <555098D7.6090700@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <555098D7.6090700@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Andrew Cooper Cc: Wei Liu , Yang Hongyang , Ian Jackson , Xen-devel List-Id: xen-devel@lists.xenproject.org On Mon, 2015-05-11 at 12:56 +0100, Andrew Cooper wrote: > On 11/05/15 12:36, Ian Campbell wrote: > > On Fri, 2015-05-08 at 22:14 +0100, Andrew Cooper wrote: > >> There are some records which should only be sent once in the stream, and not > >> repeated for each checkpoint. {start,end}_of_stream() become per-checkpoint, > >> and a new start_of_stream() is introduced. > >> > >> There is no resulting change record order, but the X86_PV_INFO and > >> X86_PV_P2M_FRAMES records are positively identified as once per stream, rather > >> than once per checkpoint. > > In the case of the latter judging from the comments this is currently an > > implementation detail and in principal we want to be able to resend the > > p2m frames as well? > > Correct, and the spec regarding X86_PV_P2M_FRAMES was explicitly > intended to allow partial updates in the future. > > It is currently an implementation bug (or many) that the P2M can't > change during the live part of migration. In which case I'd suggest avoiding saying "positively identified as once per stream" since that gives the impression that this is per the spec rather than just the current implementation. Ian.