RE: [PATCH v9 4/8] docs/specs: add missing definitions to libxc-migration-stream

2020-10-05 Thread Durrant, Paul 
> -Original Message-
> From: Andrew Cooper 
> Sent: 02 October 2020 23:42
> To: Paul Durrant ; xen-devel@lists.xenproject.org
> Cc: Durrant, Paul ; George Dunlap 
> ; Ian Jackson
> ; Jan Beulich ; Julien Grall 
> ; Stefano
> Stabellini ; Wei Liu 
> Subject: RE: [EXTERNAL] [PATCH v9 4/8] docs/specs: add missing definitions to 
> libxc-migration-stream
> 
> CAUTION: This email originated from outside of the organization. Do not click 
> links or open
> attachments unless you can confirm the sender and know the content is safe.
> 
> 
> 
> On 24/09/2020 14:10, Paul Durrant wrote:
> > From: Paul Durrant 
> >
> > The STATIC_DATA_END, X86_CPUID_POLICY and X86_MSR_POLICY record types have
> > sections explaining what they are but their values are not defined. Indeed
> > their values are defined as "Reserved for future mandatory records."
> >
> > Also, the spec revision is adjusted to match the migration stream version
> > and an END record is added to the description of a 'typical save record for
> > and x86 HVM guest.'
> >
> > Signed-off-by: Paul Durrant 
> > Fixes: 6f71b5b1506 ("docs/migration Specify migration v3 and 
> > STATIC_DATA_END")
> > Fixes: ddd273d8863 ("docs/migration: Specify X86_{CPUID,MSR}_POLICY 
> > records")
> 
> Oops sorry.  I swear I had these at one point.  I can only presume it
> got swallowed by a rebase at some point.
> 

Can I take that as an R-b?

  Paul

> ~Andrew

Re: [PATCH v9 4/8] docs/specs: add missing definitions to libxc-migration-stream

2020-10-02 Thread Andrew Cooper 
On 24/09/2020 14:10, Paul Durrant wrote:
> From: Paul Durrant 
>
> The STATIC_DATA_END, X86_CPUID_POLICY and X86_MSR_POLICY record types have
> sections explaining what they are but their values are not defined. Indeed
> their values are defined as "Reserved for future mandatory records."
>
> Also, the spec revision is adjusted to match the migration stream version
> and an END record is added to the description of a 'typical save record for
> and x86 HVM guest.'
>
> Signed-off-by: Paul Durrant 
> Fixes: 6f71b5b1506 ("docs/migration Specify migration v3 and STATIC_DATA_END")
> Fixes: ddd273d8863 ("docs/migration: Specify X86_{CPUID,MSR}_POLICY records")

Oops sorry.  I swear I had these at one point.  I can only presume it
got swallowed by a rebase at some point.

~Andrew

Re: [PATCH v9 4/8] docs/specs: add missing definitions to libxc-migration-stream

2020-09-30 Thread Wei Liu 
On Thu, Sep 24, 2020 at 02:10:26PM +0100, Paul Durrant wrote:
> From: Paul Durrant 
> 
> The STATIC_DATA_END, X86_CPUID_POLICY and X86_MSR_POLICY record types have
> sections explaining what they are but their values are not defined. Indeed
> their values are defined as "Reserved for future mandatory records."
> 
> Also, the spec revision is adjusted to match the migration stream version
> and an END record is added to the description of a 'typical save record for
> and x86 HVM guest.'
> 
> Signed-off-by: Paul Durrant 
> Fixes: 6f71b5b1506 ("docs/migration Specify migration v3 and STATIC_DATA_END")
> Fixes: ddd273d8863 ("docs/migration: Specify X86_{CPUID,MSR}_POLICY records")

Acked-by: Wei Liu