Re: [PATCH 17/20] tools/libx[cl]: Plumb static_data_done() up into libxl

[Ian Jackson <ian.jackson@xxxxxxxxxx>]

Andrew Cooper writes ("Re: [PATCH 17/20] tools/libx[cl]: Plumb 
static_data_done() up into libxl"):
> There are several things going on here.
> 
> One is the control flow marker of "We reached the end of the static
> data".  A higher level toolstack needs to know this unconditionally,
> which is why the callback is mandatory.
> 
> For v2 compatibility, its callers cope with "this is where an end of
> static data would be in a v3 stream", but that abstracted away so the
> higher level toolstack doesn't know or need to care.
> 
> The missing parameter is "p.s. here are the things we were expecting but
> didn't get - you need to pick up the pieces".  For now, it is synonymous
> with "here is a v2 stream without CPUID data", but that won't be
> accurate in the future if/when new static data records get retrofitted.

Thanks for the explanation.

Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>