Peter Xu <pet...@redhat.com> writes:

> Before this patch, the blocktime context can be created very early, because
> postcopy_ram_supported_by_host() <- migrate_caps_check() can happen during
> migration object init.
>
> The trick here is the blocktime context needs system vCPU information,
> which seems to be possible to change after that point.  I didn't verify it,
> but it doesn't sound right.
>
> Now move it out and initialize the context only when postcopy listen
> starts.  That is already during a migration so it should be guaranteed the
> vCPU topology can never change on both sides.
>
> While at it, assert that the ctx isn't created instead this time; the old
> "if" trick isn't needed when we're sure it will only happen once now.
>
> Signed-off-by: Peter Xu <pet...@redhat.com>

Reviewed-by: Fabiano Rosas <faro...@suse.de>

Reply via email to