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 <ykap...@redhat.com> Reviewed-on: http://gerrit.ovirt.org/31501 Reviewed-by: Dima Kuznetsov <dkuzn...@redhat.com> Reviewed-by: Saggi Mizrahi <smizr...@redhat.com> Reviewed-on: http://gerrit.ovirt.org/36953 Reviewed-by: Yaniv Bronhaim <ybron...@redhat.com> --- 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 <ykap...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Dima Kuznetsov <dkuzn...@redhat.com> Gerrit-Reviewer: Saggi Mizrahi <smizr...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Yeela Kaplan <ykap...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches