Re: [PATCH 3/3] qemu: Forbid migration with cache != none

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

 



On Wed, Feb 22, 2012 at 14:58:31 +0800, Shu Ming wrote:
> On 2012-2-22 0:17, Jiri Denemark wrote:
> > @@ -817,6 +817,27 @@ qemuMigrationIsAllowed(struct qemud_driver *driver, virDomainObjPtr vm,
> >       return true;
> >   }
> >
> > +static bool
> > +qemuMigrationIsSafe(virDomainDefPtr def)
> > +{
> > +    int i;
> > +
> > +    for (i = 0 ; i<  def->ndisks ; i++) {
> > +        virDomainDiskDefPtr disk = def->disks[i];
> > +
> > +        /* shared&&  !readonly implies cache=none */
> > +        if (disk->cachemode != VIR_DOMAIN_DISK_CACHE_DISABLE&&
> 
> how about disk write through flag here?  This flag should also imply 
> cache=none.
> 
> i.e, VIR_DOMAIN_DISK_CACHE_WRITETHRU

AFAIK, only VIR_DOMAIN_DISK_CACHE_DISABLE translates into cache=none unless
qemu binary is old enough to only support cache=on|off

Jirka

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]