Change in vdsm[master]: Synchronize libvirtconnection.get()

danken at redhat.com danken at redhat.com
Mon Sep 19 12:57:41 UTC 2011


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Synchronize libvirtconnection.get()
......................................................................


Synchronize libvirtconnection.get()

I'd like to consider making _ongoingCreation bigger than 1, and to
let all parts of the code a safe access to the libvirt connection.
libvirtconnection should never have trusted its user to take care of
the synchronization. Taking a lock would not slow down the seldom-used
libvirtconnection.get() noticeably.

Change-Id: I8d682d8cb1945f7d01008de84b20bf7893d68a1f
---
M vdsm/libvirtconnection.py
M vdsm/libvirtvm.py
2 files changed, 24 insertions(+), 18 deletions(-)

Approvals:
  Dan Kenigsberg: Verified; Looks good to me, approved


--
To view, visit http://gerrit.usersys.redhat.com/904
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8d682d8cb1945f7d01008de84b20bf7893d68a1f
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ayal Baron
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: David Naori <dnaori at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>


More information about the vdsm-patches mailing list