[PATCH] virnwfilterobj: Don't use virObjectLockGuard() with virNWFilterObj
Erik Skultety
eskultet at redhat.com
Fri Mar 18 12:54:08 UTC 2022
On Fri, Mar 18, 2022 at 11:43:34AM +0100, Michal Privoznik wrote:
> While its name would suggest that virNWFilterObj is an actual
> virObject it is not. It's a plain structure (with virMutex as its
> first member). Therefore, when locking the struct
> virObjectLockGuard() can' be used and virLockGuardLock() must be
> used instead.
>
> Spotted-by: Martin Kletzander <mkletzan at redhat.com>
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> ---
This fixes the crasher in the integration CI we saw yesterday:
Reviewed-by: Erik Skultety <eskultet at redhat.com>
After this patch, there's still a deadlock [1], but you should push this fix in
the meantime.
[1] https://paste.centos.org/view/6a7bfd66
More information about the libvir-list
mailing list