Re: [libvirt] [PATCH 5/9] util: Introduce virObjectLockableRecursiveNew
On Tue, May 30, 2017 at 03:03:14PM +0200, Pavel Hrdina wrote:
On Tue, May 30, 2017 at 07:38:17AM -0400, John Ferlan wrote:
> Introduce a recursive option for the lockable object which calls
> virMutexInitRecursive instead of virMutexInit.
We should avoid using recursive lock because they are dangerous. I
would rather cleanup the nwfilter code than introducing more stuff for
recursive locks.
Last time I looked I came to the conclusion that it wasn't possible to
remove the recursive locking from nwfilter.
I agree about /not/ introducing new usage of recursive locking though.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|