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>