Re: [Qemu-devel] [PATCH v4 09/10] migration: merge enforce_config_section somewhat

[Juan Quintela]

Peter Xu <address@hidden> wrote:
> These two parameters:
>
> - MachineState::enforce_config_section
> - MigrationState::send_configuration
>
> are playing similar role here. This patch merges the first one into
> second, then we'll have a single place to reference whether we need to
> send the configuration section.
>
> I didn't remove the MachineState.enforce_config_section field since when
> applying that machine property (in machine_set_property()) we haven't
> yet initialized global properties and migration object. Then, it's
> still not easy to pass that boolean to MigrationState at such an early
> time.
>
> A natural benefit for current patch is that now we kept the meaning of
> "enforce-config-section" since it'll still have the highest
> priority (that's what "enforce" mean I guess).
>
> Signed-off-by: Peter Xu <address@hidden>

Nice!!!

Reviewed-by: Juan Quintela <address@hidden>