Re: [Xen-devel] [PATCH 03/12] libxc/migration: Rationalise the 'checkpointed' field to 'stream_type'

[Ian Jackson <ian.jackson@xxxxxxxxxx>]

Andrew Cooper writes ("[PATCH 03/12] libxc/migration: Rationalise the 
'checkpointed' field to 'stream_type'"):
> Originally, 'checkpointed' was a boolean signalling the difference between a
> plain and a Remus stream.  COLO was added later, but several bits of code
> retained boolean-style logic.  While correct, it is confusing to follow.
> 
> Additionally, XC_MIG_STREAM_NONE means "no checkpoints" but reads as "no
> stream".
> 
> Consolidate all the logic on the term 'stream_type', and rename STREAM_NONE
> to STREAM_PLAIN.  Re-position the stream_type variable so it isn't
> duplicated in both the save and restore unions.

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

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel