Re: [libvirt] [PATCH 2/7] Add a virObjectLockable class holding a mutex

From: "Daniel P. Berrange" <berrange(a)redhat.com&gt; A great many virObject instances require a mutex, so introduce a convenient class for this which provides a mutex. This avoids repeating the tedious init/destroy code --- src/libvirt_private.syms | 4 +++ src/util/virobject.c | 87 ++++++++++++++++++++++++++++++++++++++++++++++-- src/util/virobject.h | 20 +++++++++++ 3 files changed, 109 insertions(+), 2 deletions(-)

+void *virObjectLockableNew(virClassPtr klass) +{ + virObjectLockablePtr obj; + + if (!virClassIsDerivedFrom(klass, virClassForObjectLockable())) { + virReportInvalidArg(klass, + _("Class %s must derive from virObjectLockable"), + virClassName(klass)); + return NULL; + } + + if (!(obj = virObjectNew(klass))) + return NULL; + + if (virMutexInit(&obj->lock) < 0) { + virReportSystemError(VIR_ERR_INTERNAL_ERROR, "%s", + _("Unable to initialize mutex")); + virObjectUnref(obj); + return NULL; + }

+ + return obj; +} + + +static void virObjectLockableDispose(void *anyobj) +{ + virObjectLockablePtr obj = anyobj; + + virMutexDestroy(&obj->lock);

+ * virObjectLock: + * @anyobj: any instance of virObjectLockablePtr + * + * Acquire a lock on @anyobj. The lock must be + * released by virObjectUnlock. + */ +void virObjectLock(void *anyobj) +{ + virObjectLockablePtr obj = anyobj;

+void virObjectUnlock(void *anyobj) +{ + virObjectLockablePtr obj = anyobj; +