Yaniv Bronhaim has submitted this change and it was merged. Change subject: oop: Use a single instance of IOProcess per SD ......................................................................
oop: Use a single instance of IOProcess per SD Next step after switching from RFH to IOPROC is to use a single IOProcess instance for each SD. Currently only a single IOProcess instance is used for all SDs. To avoid load on the system by holding many unused IOProcesses we will maintain dictionary: KEY - domainID, VALUE - (timestamp, IOPROC) When IOProc for domainID is used we will renew the timestamp, or delete the item from dictionary We will also maintain another dictionary: KEY - domainID, VALUE - IOPROC weakref, that will allow us to check if IOPROC obj is still alive or needs to be recreated. Change-Id: I383fe617ee4ce22de368ba54f980887d70ff37c5 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1180899 Signed-off-by: Yeela Kaplan <[email protected]> Reviewed-on: http://gerrit.ovirt.org/31501 Reviewed-by: Dima Kuznetsov <[email protected]> Reviewed-by: Saggi Mizrahi <[email protected]> Reviewed-on: http://gerrit.ovirt.org/36953 Reviewed-by: Yaniv Bronhaim <[email protected]> --- M lib/vdsm/config.py.in M vdsm/storage/outOfProcess.py 2 files changed, 51 insertions(+), 15 deletions(-) Approvals: Yeela Kaplan: Verified Yaniv Bronhaim: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36953 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I383fe617ee4ce22de368ba54f980887d70ff37c5 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Yeela Kaplan <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Dima Kuznetsov <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Yeela Kaplan <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
