Michael Kublin has uploaded a new change for review. Change subject: core: Deadlock on reconstruct master flow (#795105) ......................................................................
core: Deadlock on reconstruct master flow (#795105) https://bugzilla.redhat.com/795105 The following fix should solve a deadlock which was caused because of during reconstract flow were taken IrsProxyData.syncObj lock and we tried to take VdsManger._lockObj lock. The second thread started domain recovery and take a VdsManger._lockObj and tried to take IrsProxyData.syncObj These happend because of reconstruct already had succeeded and almost finished, that's why domain failover run with out any problem. Solution we don't need to run reconstruct inside critical section protected by IrsProxyData.syncObj Change-Id: I1e077a0722c4ae5a277fd85f11b8d0a353182c15 Signed-off-by: Michael Kublin <mkub...@redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java 1 file changed, 38 insertions(+), 32 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/82/5482/1 -- To view, visit http://gerrit.ovirt.org/5482 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I1e077a0722c4ae5a277fd85f11b8d0a353182c15 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkub...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
