On Tue, Jan 15, 2013 at 05:08:59PM +0100, Peter V. Saveliev wrote:
> …
> Hello all.
> I'm working on improving libvirt usage in VDSM. Rationale is that
> libvirt driver has global lock, that holds us in any operation on
> many VMs simultaneously. Several small patches are prepared, so I'm
> starting to commit them.

FYI the global lock will be dead in either the next libvirt release,
or the one after. So I wouldn't jump through too many hoops in VDSM
for this IMHO.

> The first one [1] removes UUIDString check in VM startup. It was
> used to test local libvirt for the bug 603494 [2], which is closed
> so long time ago, so there is no need to support this case, fixed
> already all across the distributives, including RHEL 6.x.
> I wouldn't care, but in mass VM startup (dozens and hundreds of VMs)
> each extra call costs. So thanks for review/merge.

The UUIDString() method executes purely client side so does not
involve any libvirt locks.

|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|
vdsm-devel mailing list

Reply via email to