Allon Mureinik has posted comments on this change. Change subject: hsm: unify vm ovf management lock ......................................................................
Patch Set 1: (1 comment) http://gerrit.ovirt.org/#/c/28068/1//COMMIT_MSG Commit Message: Line 5: CommitDate: 2014-05-23 11:09:53 -0400 Line 6: Line 7: hsm: unify vm ovf management lock Line 8: Line 9: Taking one lock per VM definition could be estremely slow (especially s/estremely/extremely/ Line 10: when the engine sends 100 ovf). In one case it took more than 3 minutes Line 11: to take 76 locks. This patch unifies the VM locks into a generic one. Line 12: There was no real advantage in the current implementation, in fact if Line 13: by any chance updateVM was stuck trying to persist a VM definition it -- To view, visit http://gerrit.ovirt.org/28068 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia857d4afa25e7eebd59fb10ef31e98c25caa9928 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Liron Ar <[email protected]> Gerrit-Reviewer: Marina Kalinin <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
