[libvirt] [PATCH] qemu_conf: Properly mark driver->config as self-locking APIs

Michal Privoznik mprivozn at redhat.com
Tue Jul 16 18:25:25 UTC 2013


Since a9e97e0c we don't require driver->config, or
virQEMUDriverGetConfig() to be called with locked QEMU driver. In fact,
calling it with the driver locked would cause deadlock. Hence, the
annotation to the struct field is not right.
---
 src/qemu/qemu_conf.h | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/qemu/qemu_conf.h b/src/qemu/qemu_conf.h
index 8229cfc..fd5cc57 100644
--- a/src/qemu/qemu_conf.h
+++ b/src/qemu/qemu_conf.h
@@ -160,8 +160,7 @@ struct _virQEMUDriverConfig {
 struct _virQEMUDriver {
     virMutex lock;
 
-    /* Require lock to get reference on 'config',
-     * then lockless thereafter */
+    /* Immutable pointer, self-locking APIs */
     virQEMUDriverConfigPtr config;
 
     /* Immutable pointer, self-locking APIs */
-- 
1.8.1.5




More information about the libvir-list mailing list