[libvirt] [PATCH] Clean some comments about locking

Daniel P. Berrange berrange at redhat.com
Tue Feb 12 15:17:29 UTC 2013


On Tue, Feb 12, 2013 at 08:14:40AM -0700, Eric Blake wrote:
> On 02/12/2013 05:52 AM, Martin Kletzander wrote:
> > Since the BQDL has been broken, this patch tries to cleanup some
> > information about it in various comments.
> > ---
> > To be applied after Dan's patch [1] which removes unnecessary
> > qemuDriverLocks.
> > 
> > @@ -1734,7 +1733,7 @@ qemuDomainSnapshotDiscardAllMetadata(virQEMUDriverPtr driver,
> >  }
> > 
> >  /*
> > - * The caller must hold a lock on both driver and vm, and there must
> > + * The caller must hold a lock the vm and there must
> 
> s/lock the/lock on the/
> 
> ACK, whether Dan squashes this in directly or you push as a followup.

It really ought to be squashed into the main patch I think

Daniel
-- 
|: 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 :|




More information about the libvir-list mailing list