Bugzilla – Full Text Bug Listing
| Summary: | [LTTng] Add a latency analysis compoment | ||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | Francois Chouinard <fchouinard> | ||||||||||||||||
| Component: | LinuxTools | Assignee: | Bernd Hufmann <bernd.hufmann> | ||||||||||||||||
| Status: | CLOSED FIXED | QA Contact: | Francois Chouinard <fchouinard> | ||||||||||||||||
| Severity: | enhancement | ||||||||||||||||||
| Priority: | P3 | CC: | abdelhaq.saoudi, ali.jawhar, bernd.hufmann, INF4990.A2010, mathieu.denis, so.mailloux | ||||||||||||||||
| Version: | unspecified | ||||||||||||||||||
| Target Milestone: | --- | ||||||||||||||||||
| Hardware: | PC | ||||||||||||||||||
| OS: | Linux | ||||||||||||||||||
| Whiteboard: | |||||||||||||||||||
| Bug Depends on: | 354272, 355084, 355141 | ||||||||||||||||||
| Bug Blocks: | 290046 | ||||||||||||||||||
| Attachments: |
|
||||||||||||||||||
|
Description
Francois Chouinard
Created attachment 184440 [details]
Patch adding a new TmfView where latency analysis charts are displayed.
Based on revision 27050.
Here is a patch adding a latency analysis component to LTTng. This patch adds a new TmfView named "Latency View", which displays two charts : * A scatter plot of latency times vs trace times * A histogram of the number of events sharing the same latency times. Also allows the user to increase/decrease the size of the histogram bars, display the list of latency events pairs and add/remove custom latency events pairs. The latency computation is implemented as a State pattern in an Analyser, allowing for further refinements or features in the future by adding new states. Developed by me (Philippe Sawicki), Ali Jawhar, Sophie Mailloux and Abdelhaq Saoudi of École Polytechnique de Montréal. Created attachment 185278 [details]
Latency View (updated)
Replaces the previous patch which is problematic.
Created attachment 185279 [details]
Latency View icons (zipped)
Separate attachment for the (binary) icons
Created attachment 201090 [details]
Update for toside 0.7.0
Must apply the patch from Francois before this one. Useful for those who already have the component in their packages.
Created attachment 201276 [details]
Latency Analysis Icons
Shorter list of icons since there are some from the original package of Francois that are already in the package lttng.ui.
These icons need to be placed into a new folder named latency/ under icons/ folder of the package org.eclipse.linuxtools.lttng.ui
Created attachment 201280 [details]
Update icons path
Correct the latency view icons by itself. No need to download the Latency View icons separately (tested in git perspective).
This patch must be applied at the base of the package org.eclipse.linuxtools.lttng.ui
I, Ali Jawhar, declare that I developed attached code from scratch, without referencing any 3rd party materials except material licensed under the EPL. I am authorized by École Polytechnique de Montréal to make this contribution under the EPL. I, Philippe Sawicki, declare that I developed attached code from scratch, without referencing any 3rd party materials except material licensed under the EPL. I am authorized by École Polytechnique de Montréal to make this contribution under the EPL. I, Abdelhaq Saoudi, declare that I developed attached code from scratch, without referencing any 3rd party materials except material licensed under the EPL. I am authorized by École Polytechnique de Montréal to make this contribution under the EPL. I, Sophie Mailloux, declare that I developed attached code from scratch, without referencing any 3rd party materials except material licensed under the EPL. I am authorized by École Polytechnique de Montréal to make this contribution under the EPL. Created attachment 209983 [details] New version of latency analysis This patch contains the implementation of a latency analysis feature. It is based on the contribution in attachment 184440 [details] done be the students. The new implementation has the following main changes: - Model-View-Control implementation to decouple model and views - Re-use of the Histogram model for the Latency Distribution graph - Improved view synchronization - Some bug fixes in widgets Additionally, this patch contains the refactored code for git that was done in attachment 201090 [details]. Has a CQ been opened for this/these contribution(s)? (In reply to comment #13) > Has a CQ been opened for this/these contribution(s)? Yes, I opened one today. http://dev.eclipse.org/ipzilla/show_bug.cgi?id=5996 /Bernd Latency Analysis Feature (as in attachment 209983 [details]) has been committed after approved CQ.
Delivered in 0.10.0 |