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

Re: [libvirt] [PATCH 4/4] Remove qemuDriverLock from almost everywhere



On Thu, Feb 07, 2013 at 05:46:38PM +0000, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange" <berrange redhat com>
> 
> With the majority of fields in the virQEMUDriverPtr struct
> now immutable or self-locking, there is no need for practically
> any methods to be using the QEMU driver lock. Only a handful
> of helper APIs in qemu_conf.c now need it
> ---
>  src/qemu/THREADS.txt            | 194 +++-----------------
>  src/qemu/qemu_conf.c            |  50 ++++--
>  src/qemu/qemu_domain.c          | 213 +++++-----------------
>  src/qemu/qemu_domain.h          |  40 +----
>  src/qemu/qemu_driver.c          | 386 +++++++++-------------------------------
>  src/qemu/qemu_hotplug.c         | 118 ++++++------
>  src/qemu/qemu_migration.c       |  66 ++++---
>  src/qemu/qemu_process.c         | 223 ++++++++---------------
>  src/qemu/qemu_process.h         |   3 +-
>  src/security/security_selinux.c |  28 +--
>  10 files changed, 374 insertions(+), 947 deletions(-)

Don't bother reviewing this one yet, it is flawed.
I will be sending a new version

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


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