Dan Kenigsberg has submitted this change and it was merged. Change subject: profiling: Add an application wide profile ......................................................................
profiling: Add an application wide profile The built in Python profilers are good for single thread profile but given the excessive multi-threaded nature of vdsm, they are useless for understing where we spend our time. This patch uses yappi <http://code.google.com/p/yappi> to create an application wide profile. New configuration options added to enable profiling and selecting profile file format. The default file format is PSTAT, which can be used with the builtin pstats module. Another interesting format is CALLGRIND, used by KCachegrind visualization tool. Change-Id: I523f52c981f7bb34f3168d3117f00ed5eb6962f8 Relates-to: https://bugzilla.redhat.com/1074097 Signed-off-by: Nir Soffer <[email protected]> Reviewed-on: http://gerrit.ovirt.org/26113 Reviewed-by: Francesco Romani <[email protected]> Reviewed-by: Adam Litke <[email protected]> Reviewed-by: Douglas Schilling Landgraf <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M debian/vdsm-python.install M lib/vdsm/Makefile.am M lib/vdsm/config.py.in A lib/vdsm/profile.py M vdsm.spec.in M vdsm/vdsm 6 files changed, 71 insertions(+), 0 deletions(-) Approvals: Douglas Schilling Landgraf: Looks good to me, but someone else must approve Nir Soffer: Verified Adam Litke: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved Francesco Romani: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/26113 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I523f52c981f7bb34f3168d3117f00ed5eb6962f8 Gerrit-PatchSet: 10 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
