[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH v13] support offline migration



On Fri, Nov 09, 2012 at 10:57:23 +0800, li guang wrote:
...
> > > @@ -2150,6 +2192,9 @@ qemuMigrationRun(struct qemud_driver *driver,
> > >          return -1;
> > >      }
> > >  
> > > +    if (flags & VIR_MIGRATE_OFFLINE)
> > > +        return 0;
> > > +
> > >      if (!(mig = qemuMigrationEatCookie(driver, vm, cookiein,
> > cookieinlen,
> > >
> > QEMU_MIGRATION_COOKIE_GRAPHICS)))
> > >          goto cleanup;
> > 
> > I still think we should not even get into qemuMigrationRun when doing
> > offline migration.
> 
> No, it will get into here for I did not touch migrationPerform path,
> initially I don't want to change any code at libvirt.c so I must keep
> offline migration walk through whole path like normal, or there're maybe
> lots of fixes, any will break up the migration path.

Well, and that's the thing that actually needs fixing. The migration code in
libvirt.c and the code doing peer-to-peer migration in qemu_migration.c are
very similar because they are doing almost the same job (only at different
levels). It makes sense to change them both at the same time. Why do you want
to avoid changing code in libvirt.c?

Jirka


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]