RE: [PATCH v7 1/9] xen/common: introduce a new framework for save/restore of 'domain' context

[Paul Durrant <xadimgnik@xxxxxxxxx>]

> -----Original Message-----
> From: Jan Beulich <jbeulich@xxxxxxxx>
> Sent: 26 August 2020 14:54
> To: Paul Durrant <paul@xxxxxxx>
> Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx; Paul Durrant <pdurrant@xxxxxxxxxx>; 
> Julien Grall <julien@xxxxxxx>;
> Andrew Cooper <andrew.cooper3@xxxxxxxxxx>; George Dunlap 
> <george.dunlap@xxxxxxxxxx>; Ian Jackson
> <ian.jackson@xxxxxxxxxxxxx>; Stefano Stabellini <sstabellini@xxxxxxxxxx>; Wei 
> Liu <wl@xxxxxxx>;
> Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>; Roger Pau Monné 
> <roger.pau@xxxxxxxxxx>
> Subject: Re: [PATCH v7 1/9] xen/common: introduce a new framework for 
> save/restore of 'domain' context
> 
> On 26.08.2020 15:32, Jan Beulich wrote:
> > On 18.08.2020 12:30, Paul Durrant wrote:
> >> v7:
> >>  - Add an option to domain_load_end() to ignore unconsumed data, which will
> >>    be needed by a subsequent patch
> >
> > May I suggest to name the parameter "ignore_tail" instead of
> > "ignore_data", as typically you don't mean to ignore all of
> > it?
> 
> Hmm, looking at patch 7 it's indeed all of the body which gets
> ignored. Not sure what the longer term expectations here are.
> 

I think it is useful to be able to consume as much or as little of a record as 
is needed.

  Paul