Nir Soffer has posted comments on this change. Change subject: Add ResourceLocker ......................................................................
Patch Set 2: (1 comment) https://gerrit.ovirt.org/#/c/43554/2/vdsm/storage/resourceLocker.py File vdsm/storage/resourceLocker.py: Line 40: if len(self._resources) >= LOCK_CACHE_SIZE: Line 41: self._drop_cached_lock() Line 42: if res not in self._resources: Line 43: self._resources[res] = RWLock() Line 44: self._resources[res].acquire(exclusive) > To support timed acquire, we can use the new rwlock that support timeouts ( But - lets not work on fancy debugging here before we finish the entire feature. Line 45: Line 46: def releaseResource(self, res): Line 47: with self._lock: Line 48: self._resources[res].release() -- To view, visit https://gerrit.ovirt.org/43554 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I29589889efd39920d72979e89405ff6cf36b803b Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches