On 05/02/2021 18:46, Ian Jackson wrote:
> Igor Druzhinin writes ("[PATCH v2 2/2] tools/libxl: only set viridian flags
> on new domains"):
>> Domains migrating or restoring should have viridian HVM param key in
>> the migration stream already and setting that twice results in Xen
>> returing -EEXIST on the second attempt later (during migration stream
>> parsing)
>> in case the values don't match. That causes migration/restore operation
>> to fail at destination side.
>>
>> That issue is now resurfaced by the latest commits (983524671 and 7e5cffcd1e)
>> extending default viridian feature set making the values from the previous
>> migration streams and those set at domain construction different.
>>
>> Suggested-by: Andrew Cooper <[email protected]>
>> Signed-off-by: Igor Druzhinin <[email protected]>
> Thanks for splitting this up.
>
> Release-Acked-by: Ian Jackson <[email protected]>
>
> Andy, I think from irc that this meets with your approval but can I
> have a formal R-b ? If so please put my tools maintainer ack on it
> too.
Reviewed-by: Andrew Cooper <[email protected]>
