libxc/migration: Rationalise the 'checkpointed' field to 'stream_type'
authorAndrew Cooper <andrew.cooper3@citrix.com>
Fri, 20 Dec 2019 16:34:16 +0000 (16:34 +0000)
committerAndrew Cooper <andrew.cooper3@citrix.com>
Wed, 15 Jan 2020 15:22:28 +0000 (15:22 +0000)
commit86cf92f5053339ca5f3c71be858f3d55dd05a950
tree5cfca54960c79598ec8f75a79fb8dd8982a58bb1
parent59e1f6d897102dd53abd77d84a308a4f308ddf06
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.

No functional change.

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
tools/libxc/include/xenguest.h
tools/libxc/xc_nomigrate.c
tools/libxc/xc_sr_common.h
tools/libxc/xc_sr_restore.c
tools/libxc/xc_sr_save.c
tools/libxl/libxl_save_helper.c