Patrick Tasse [Tue, 27 Oct 2015 19:23:26 +0000 (15:23 -0400)]
tmf: Support navigation for all markers in time graph viewer
The time graph viewer now stores both the bookmark list and the marker
list. The time graph control's marker list is the sorted union of both.
The Next/Previous Bookmark actions are renamed to Next/Previous Marker,
and now apply to all markers in the merged list.
Change-Id: Idf2b14394726885753a926c1424ce7e0d315610b
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59064
Reviewed-by: Hudson CI
Patrick Tasse [Tue, 27 Oct 2015 15:21:54 +0000 (11:21 -0400)]
tmf: Support marker event categories
Change-Id: Ic60590932a23e21d39b800a7e3893ea0661d16d9
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59063
Reviewed-by: Hudson CI
Alexandre Montplaisir [Fri, 23 Oct 2015 22:42:13 +0000 (18:42 -0400)]
lttng: Remove outdated comment
The return type was changed already.
Change-Id: I3106689d272359af2c0ff0f0ab7fbda8656361e2
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/58881
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Alexandre Montplaisir [Wed, 20 May 2015 23:35:39 +0000 (19:35 -0400)]
Link all plugins to the external annotation location
Change-Id: I329f7bc22a8509534540b97ef3451fd21840094d
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/57251
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Wed, 28 Oct 2015 18:49:16 +0000 (14:49 -0400)]
common: Annotate some static methods in Collections and Arrays
The following methods and static fields can be marked as @NonNull:
Arrays.asList()
Collections.empty{Map|List|Set}()
Collections.EMPTY_{MAP|LIST|SET}
Change-Id: I6a36ac288441099219de19b1ec7793ed5cfd2a01
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59165
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI
Matthew Khouzam [Wed, 28 Oct 2015 18:49:38 +0000 (14:49 -0400)]
common: Annotate java.util.Formatter
Annotate both Formatter.format() methods to return a @NonNull formatter
(they return the formatter object itself).
Change-Id: I8f18f439d6db909f10c0f54fd4793665c4966a6b
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59164
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI
Matthew Khouzam [Wed, 28 Oct 2015 18:50:28 +0000 (14:50 -0400)]
common: Update external Map annotations
Make entrySet() return a @NonNull Set (if the map exists, the entrySet
necessarily does).
Change-Id: I08dc06455239701a45ad6f6d9c678f811af73c0f
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59162
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI
Alexandre Montplaisir [Thu, 22 Oct 2015 22:20:41 +0000 (18:20 -0400)]
common: Add external annotations for Guava's Table#get
Just like Map.get(), Table.get() can return a null value to
indicate that the key is not present in the table, independently
of the declaration of the type V.
Guava Tables are now used in the graph/critical path classes, so
this is also required.
Slightly harder to add annotations to Guava classes atm due to
https://bugs.eclipse.org/bugs/show_bug.cgi?id=480462
but one can still write the .eea files manually.
Change-Id: Id5cc1bdea73bafe62af60df0fdf7f3e9c5f40796
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/58776
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Tue, 9 Jun 2015 13:51:27 +0000 (15:51 +0200)]
common: Add external annotations for IAdaptable/IAdapterFactory
Change-Id: I0a35aa77392ae57602e7171d68b490fb9fe6411f
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/57250
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Tue, 19 May 2015 00:33:29 +0000 (20:33 -0400)]
common: Add external annotations for Map.get()/.remove()
Even if one declares a Map<Object, @NonNull Object>, Map.get()
can return a null to indicate "there is no value for this key".
Same applies to .remove() too.
Since the base methods are not annotated, we either have to
supply our own annotation, or make sure to always use @Nullable
keys. The former is less error-prone, so let's do that.
See http://stackoverflow.com/a/
25371836/
4227853 .
Change-Id: I3bb7395d7dca053d354f2f418d8529696edd0346
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/57249
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Thu, 27 Aug 2015 17:53:16 +0000 (13:53 -0400)]
control: Add LTTng profile save dialog
Change-Id: I42ab05d707369d5082f2503f0ef04993d4193cf3
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57685
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Bernd Hufmann [Wed, 12 Aug 2015 00:56:15 +0000 (20:56 -0400)]
control: Add support for downloading profiles after saving of sessions
Change-Id: Ie42d8d2d408f52fd093780193859015cb0907613
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57684
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Bernd Hufmann [Tue, 2 Jun 2015 20:06:20 +0000 (16:06 -0400)]
control: Add support for saving session
This commit adds support for saving one or more session in
default location and force overwrite
Change-Id: I6996415456c86b9bc82df6504af04ee1a13f1440
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57683
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Bernd Hufmann [Mon, 3 Aug 2015 13:04:43 +0000 (09:04 -0400)]
control: Add LTTng profile preference page
Change-Id: Iaf8f420eadf4747866a8eee2b3b9a9bc723d7396
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57682
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Bernd Hufmann [Wed, 12 Aug 2015 01:13:06 +0000 (21:13 -0400)]
control: Add enhanced support for loading of sessions
Change-Id: I68df4fba7eb53542cd9eca4cf15e161cc679d782
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57681
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Patrick-Jeffrey Pollo Guilbert [Fri, 3 Apr 2015 03:02:32 +0000 (23:02 -0400)]
control: Base code for profile dialog window
Modified ILttngControlService to add a loadSession command, also
implemented the methods in the appropriate children classes
(LTTngControlService and NullControlService).
Added ProfileDialog and ProfileHandler, as well as the appropriate
modifications to the plugin.xml to allow the execution of the handler as
well as the call to the Handler. As for now, pre-selected profiles must
be in
org.eclipse.tracecompass.internal.lttng2.control.ui/resources/folders.txt.
Appropriate .icons, strings and whatnot are externalized and at the
appropriate location > messages.properties.
Signed-off-by: Patrick-Jeffrey Pollo Guilbert <patrick.pollo.guilbert@gmail.com>
Signed-off-by: William Tri-Khiem Truong <tri-khiem-william.truong@polymtl.ca>
Signed-off-by: William Enright <william.enright@polymtl.ca>
Change-Id: Id312708b1907f3dd36fab65da7d96d5f917404a2
Reviewed-on: https://git.eclipse.org/r/45958
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Mon, 5 Oct 2015 16:47:42 +0000 (12:47 -0400)]
lttng: Rename the debug-info callsite aspect to "source callsite"
If not all debug information is available in the binary, it is possible
to only display the "binary file/symbol/offset" tuple. The latter will
be named "binary callsite", to differentiate it from the complete
source callsite.
Change-Id: Ie7e0368a6b1c02c63f759536728835f464ea463b
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/58775
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Sat, 5 Sep 2015 00:53:19 +0000 (20:53 -0400)]
tmf: Allow ITmfSourceLookup to open files outside the workspace
Bug: 476682
Change-Id: I663bec71c4b97ecc71dc8fcd0bbc620de15bc8a7
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/56193
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Matthew Khouzam [Sun, 25 Oct 2015 01:40:18 +0000 (21:40 -0400)]
analysis: Improve comments for IKernelAnalysis.java
Add javadoc to IKernelAnalysis. The tracepoints used by the kernel state
system now are described in a simpler language, as it is unfair to assume
that an eclipse developper working with trace compass is also a Linux
kernel developer.
This patch is part of an effort to make it easier to adapt new operating
system traces to trace compass. The javadoc is written based on the commit
messages for given patches and Linux Kernel Maling List discussions as well
as wikipedia, trace metadata and Linux Weekly News updates.
Change-Id: I33eb53d0b172c3b9d235eaa2e4681eaa410e4e76
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58886
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Bernd Hufmann [Thu, 22 Oct 2015 17:44:13 +0000 (13:44 -0400)]
btf: fix sonar warning about null safety
Change-Id: I197a0708f15a89f28c190e242abec53a1bf94c36
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58758
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Sat, 24 Oct 2015 02:23:10 +0000 (22:23 -0400)]
analysis: Clean up critical path algorithm a bit
Change-Id: I8c798fc612a28640b29292a6e51af046ebe50d97
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58872
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Jean-Christian Kouame [Wed, 30 Sep 2015 21:11:55 +0000 (17:11 -0400)]
tmf : extract duplicate code into sortNodes() in GraphNode
Change-Id: I6309676ab960c0791bb8b0ce39d44951a8ed3075
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57246
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Jean-Christian Kouame [Wed, 14 Oct 2015 17:12:28 +0000 (13:12 -0400)]
tmf : use of the map values instead of keyset
Change-Id: I80a02e117d0afe3b45b557903e185443c6f099ca
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58182
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Mon, 26 Oct 2015 13:46:37 +0000 (09:46 -0400)]
analysis: Register latency statistics view as analysis output
and remove action from context sensitive menu of latency table
Change-Id: I806b87ac2088adff18e69836f847f1353c5153e5
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58932
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Mon, 26 Oct 2015 13:32:23 +0000 (09:32 -0400)]
analysis: Clean-up latency statistics
Change-Id: I978c8c4cf268d96989f9c135301c6a323d1f7130
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58929
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Patrick Tasse [Fri, 23 Oct 2015 22:58:50 +0000 (18:58 -0400)]
tmf: Support trace-specific markers in AbstractTimeGraphView
Adds an IMarkerEventSource adapter interface.
Computes the list of adapters when a trace is loaded in the view.
Uses the adapters to get the trace-specific marker events for the
current window range when zooming.
Renames getMarkerList() to getViewMarkerList() to better differentiate
with getTraceMarkerList().
Change-Id: I97e00d5cf8eb0c39b78b04077322e26f7d2a85da
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/58864
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Patrick Tasse [Fri, 23 Oct 2015 22:55:05 +0000 (18:55 -0400)]
tmf: Add TmfTraceAdapterManager
This adapter manager can handle adapters that are specific to a
particular trace type id, to provide specific adapters even if the trace
instance is of the same class.
Unlike AdapterManager, this manager will return a list of all matching
adapters that adapt the given trace.
Change-Id: I4c8d6e6d725cf80a2e9ab865ea3217ae05f8cc73
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/58863
Reviewed-by: Hudson CI
Patrick Tasse [Fri, 23 Oct 2015 20:18:30 +0000 (16:18 -0400)]
tmf: Add previous full state in AbstractStateSystemTimeGraphView queries
Adding the previous full state allows the implementation to be able to
prevent duplicated elements at the boundaries of the query iterations.
Change-Id: Idd3e9718a0e4fa97419e5231713b6fe1711ad2e5
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/58862
Reviewed-by: Hudson CI
Patrick Tasse [Tue, 20 Oct 2015 16:51:05 +0000 (12:51 -0400)]
tmf: Improve bookmark and marker handling
- Support goto marker without rank (with time and duration) in event
table.
- Update goto marker handling to broadcast time/time range selection.
- Execute pending goto marker handling after trace opened signal sent.
- Add initial value to AddBookmarkDialog.
- Use AddBookmarkDialog in event table.
- Improve format of bookmark location string.
- Add interface for marker attribute strings.
- Fix handling of marker changes when trace is closed or deleted.
- Support ADDED resource change delta and consolidate handling of all
deltas from the same resource change event.
- Add methods to add or remove one or more markers in event table.
- Support location attribute as int (backward compatible) or String
value in trace package import operation.
Change-Id: I64fa20af7c7c3bc7854c92184df2b86d1831f850
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/58543
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Alexandre Montplaisir [Tue, 20 Oct 2015 19:59:25 +0000 (15:59 -0400)]
tmf: Also dispose the inner table of TmfSimpleTableViewer
Calling dispose() on a TmfSimpleTableViewer does not clear it from its
view, because the inner Table object also needs to be disposed.
Change-Id: I3b05643e93223e4a4a24fd0f1ef5399905e05c4c
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/58777
Reviewed-by: Hudson CI
Bernd Hufmann [Tue, 29 Sep 2015 14:34:31 +0000 (10:34 -0400)]
lttng: Add possibility to remove latency listener from latency view
Also, store the listener in a ListenerList instead of a HashSet.
Change-Id: Ia8a7d272db34f09bfc457f74c98b5b928b6a82d7
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/56962
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Thu, 9 Jul 2015 17:14:04 +0000 (13:14 -0400)]
Remove duplicate groupId and versions in pom.xml files
If a groupId or plugin version is defined in a parent pom, it
is not necessary to repeat it in its children.
Change-Id: Idcff9cd6cd8154cc06aebde0152f1e7c23665b1b
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/51680
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Bernd Hufmann [Mon, 26 Oct 2015 13:18:05 +0000 (09:18 -0400)]
analysis: Change visibility modifiers in segment store analysis module
Change-Id: I3c16dceccf12a7124898116bee7dd6653e701fd0
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58925
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Tue, 13 Oct 2015 11:31:04 +0000 (07:31 -0400)]
lttng: fix NPE when segment selection is null
Change-Id: I1ac9dd36c0465b3261805f362c41a595a55f5df5
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58061
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Thu, 22 Oct 2015 17:40:50 +0000 (13:40 -0400)]
pcap: fix sonar warnings for null references
Change-Id: I8f1834d47609e3a4aeb2792f0fafe609ae7326fb
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58757
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Thu, 22 Oct 2015 17:34:53 +0000 (13:34 -0400)]
control: Unlock lock in all cases
Change-Id: Id49c4e71254020df236be3e61cd32cabf029175e
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58756
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Fri, 16 Oct 2015 18:06:04 +0000 (14:06 -0400)]
lttng: Provide tooltip for latency scatter chart
Change-Id: I42b32fa02eeb2bcbd1c4edb65c2d37fadb0e0cbe
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58544
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Alexandre Montplaisir [Fri, 4 Sep 2015 00:09:21 +0000 (20:09 -0400)]
lttng: Create TmfCallsite's from the debug info analysis
Change-Id: I95e9257be38ddc3f27a26a07626bf02f4630dff2
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/56190
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Bernd Hufmann [Fri, 23 Oct 2015 14:30:02 +0000 (10:30 -0400)]
analysis: Dispose the segment store when disposing the segment analysis
Change-Id: If157a9cff7db60f14d686868524eb0101fbec2e6
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58820
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Fri, 4 Sep 2015 22:34:08 +0000 (18:34 -0400)]
lttng: Add a LttngUstEvent class for LttngUstTraces
This class will support ITmfSourceLookup, now done through the
UST debug info analysis.
Change-Id: I904f7e8eb1811ab1f74ba95165d3d9167548ca3d
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/56192
Reviewed-by: Hudson CI
Alexandre Montplaisir [Fri, 4 Sep 2015 21:32:13 +0000 (17:32 -0400)]
ctf: Assign a CtfTmfEventFactory to each trace
A singleton factory means we cannot possibly extend CtfTmfEvent,
because the sole factory can only generate CtfTmfEvent objects.
To rectify this, we can turn the factory into a standard non-static
class, and have each trace take ownership of its event factory. This
means that sub-classes of CtfTmfTrace will be able to provide their
own extension of CtfTmfEventFactory if they want to generate other
event types.
Bug: 476679
Change-Id: I66e9cd49044edd2a77276d779d0f563a2e20d520
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/56191
Reviewed-by: Hudson CI
Bernd Hufmann [Mon, 19 Oct 2015 20:00:34 +0000 (16:00 -0400)]
analysis: Add abstract latency scatter chart view and use it in LTTng
Change-Id: I55d1e9ea96281f6f059c0310a50b3481f1294660
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58360
Reviewed-by: Hudson CI
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Thu, 15 Oct 2015 12:55:28 +0000 (08:55 -0400)]
lttng: Add latency statistics
Change-Id: I49863b3046717b3bba81443fc05305d78a80b302
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58262
Reviewed-by: Hudson CI
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Wed, 14 Oct 2015 11:06:09 +0000 (07:06 -0400)]
tmf: add comparator getter to ISegmentAspect interface
Change-Id: Ida38b31726076eaa5b3b465933033cf6e22b0ac7
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58261
Reviewed-by: Hudson CI
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Tue, 13 Oct 2015 18:17:11 +0000 (14:17 -0400)]
analysis: introduce abstract latency analysis table
Change-Id: I29a893ad5ae3cca3bdf8450921467249a8bfd5a9
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58093
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Tue, 6 Oct 2015 12:59:55 +0000 (08:59 -0400)]
lttng: provide a system call column for in Latency Table
It uses the newly created ISegmentAspect interface
Change-Id: I6bcdf746b396a4a79f10a22233d5cd994d778ad6
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57515
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Tue, 6 Oct 2015 12:58:33 +0000 (08:58 -0400)]
tmf: Introduce ISegmentAspect interfaces
Similar to ITmfEventAspect the interface ISegmentAspect provides means
to describe and retrieve the content beyond the data that is provided
by ISegment.
Change-Id: I67facc1212ba222c2012389c25e84d861d1fe190
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57514
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Tue, 6 Oct 2015 10:50:12 +0000 (06:50 -0400)]
analysis: add abstract latency analysis module and use it in LTTng
Change-Id: If1431cc9b31837a0a546ad952526f0e7b47d61f5
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57513
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Francis Giraldeau [Thu, 22 Oct 2015 01:23:57 +0000 (21:23 -0400)]
analysis: basic checks for acyclic property of TmfGraph
The TmfGraph is Directed Acyclic Graph (DAG). This patch adds three
checks:
* verify that a node is not linked to itself (this was possible because
the timestamp is greater or equal to itself).
* assert that getHead() will terminate in the special case where the
tail points to the head indirectly.
* fix infinite loop bug with TmfGraph.scanLineTraverse() with a
specific graph.
Change-Id: I68a3eaed2c1098df1547d1fba34c35bc1d038404
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Reviewed-on: https://git.eclipse.org/r/58682
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Fri, 16 Oct 2015 01:50:21 +0000 (21:50 -0400)]
analysis: add critical path icon
Change-Id: Id4926070cd1c4a36f6288f6cd1b1beaba6e93233
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58292
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Francis Giraldeau [Mon, 9 Feb 2015 03:52:06 +0000 (22:52 -0500)]
lttng: Add the link between the control flow view and the critical path
A parameter provider listens to the selection of the Control Flow View
and passes that selection as a parameter to the critical path analysis
Change-Id: I2866ffe27d715d76db6d401d97273a0020f25ab4
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/41482
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Francis Giraldeau [Mon, 3 Aug 2015 14:03:49 +0000 (10:03 -0400)]
analysis: Add the critical path view
Added in a new org.eclipse.tracecompass.analysis.graph.ui plugin
Change-Id: Id1f9691273c35e3e91b3ef57c000a37ee8bc4ad1
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/41481
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Marc-Andre Laperle [Wed, 21 Oct 2015 22:09:34 +0000 (18:09 -0400)]
lttng: Fix a package dependency in control.core
Change-Id: Ieab9d01b146073d16ff92c35cb9f303eac196142
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58675
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Marc-Andre Laperle [Wed, 21 Oct 2015 21:25:47 +0000 (17:25 -0400)]
ctf: Fix ExperimentBenchmark use of tracecompass-test-traces
The way it was used, it was trying to parse the root of the experiment
as a CTF trace.
Change-Id: If575a1ef26a9de56c99bcea21fc5e12ae38e0131
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58673
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Wed, 8 Apr 2015 14:12:17 +0000 (10:12 -0400)]
tmf.ui: Move TmfEventsTable listeners from anonymous to nested classes
The listeners are pretty big and make the code harder to read. This patch
puts them together at the top of the file so that the constructor is simpler.
Change-Id: Ib0ab2d86afec4a26eb725556eb52e046b6b7850f
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/45464
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Matthew Khouzam [Tue, 23 Dec 2014 21:17:36 +0000 (16:17 -0500)]
ctf: cache file name
Java File.getName() gets a path and does a substring on it,
this operation is called on a per-event level and should be cached.
Change-Id: Ib8bd18df3bc0ac307f65fde615d151c832918a64
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/38740
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Alexandre Montplaisir [Wed, 13 May 2015 20:31:47 +0000 (16:31 -0400)]
Build whole project with Java 8
Change-Id: If94b76f857f73c91a8dd22aa1d04c7168f1f3c51
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/57248
Reviewed-by: Hudson CI
Bernd Hufmann [Mon, 19 Oct 2015 16:48:58 +0000 (12:48 -0400)]
analysis: add plug-ins for timing related analyses (e.g. latency)
Change-Id: Id5e4e73e7f128ed47bd293dacf13365f66c42695
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58480
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Francis Giraldeau [Wed, 4 Feb 2015 04:28:57 +0000 (23:28 -0500)]
TMF: Add extension point for parameter providers
It associates a parameter provider to an analysis. Previous way was to
register the parameter provider in the plugin's Activator, but in case
of lazy
loading of the plugin, it sometimes was not registered.
Change-Id: I409593d23971b41b0efc777b69a4aaa469da9339
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Reviewed-on: https://git.eclipse.org/r/41120
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Francis Giraldeau [Mon, 9 Feb 2015 03:49:33 +0000 (22:49 -0500)]
lttng: Add unit test for the dependency graph
Change-Id: I9233317d3feb240ecf5d98be3c11aeebd68c448f
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/41480
Reviewed-by: Hudson CI
Francis Giraldeau [Mon, 9 Feb 2015 03:46:04 +0000 (22:46 -0500)]
lttng: Add the dependency graph model and handlers for an LTTng kernel
trace
Change-Id: I0ec0ee9e141e13117ef2367059ad709afb74a130
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/41479
Reviewed-by: Hudson CI
Matthew Khouzam [Mon, 19 Oct 2015 03:15:56 +0000 (23:15 -0400)]
Analysis: Bug 453362 - Display Soft IRQ names in Resources View
Softirqs have pre-defined names in the Linux kernel, this patch displays
a description based on them in a the Resources View.
Change-Id: I0f8d301967f34d327e7a8e487619d10c3152b3ba
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58411
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Bernd Hufmann [Tue, 20 Oct 2015 13:50:53 +0000 (09:50 -0400)]
control: Bug 480217: Update MI XSD file for LTTng 2.7 targets
With the new XSD it is possible to use the Control view with LTTng 2.7
targets. All existing features that are supported in the Control view
right now will work again.
A basic test case was added to verify the XSD validation a basic lttng
session created with the Control view for a 2.7 target.
Change-Id: I72e4e89b6d461f0bfa45a13fb8dbf26a6a162223
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58514
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Sat, 17 Oct 2015 02:38:48 +0000 (22:38 -0400)]
analysis: remove useless //$NON-NLSes
Change-Id: I8c2042e98c7672e3b738f4e4d7e00020135114d2
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58390
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Matthew Khouzam [Sat, 17 Oct 2015 02:35:45 +0000 (22:35 -0400)]
analysis: add javadoc to LinuxValues.java
Sources LKML and LWN to find human readable explanations of all the flags.
Change-Id: Icd0f44afeb3bf443f16d984f034d088aa2ba8a74
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58389
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Matthew Khouzam [Fri, 16 Oct 2015 01:35:36 +0000 (21:35 -0400)]
analysis: fix typo in javadoc
Change-Id: I5cf3a98cc6cc8ddf1400c09eaff5bd9fd508c1b1
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58291
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Francis Giraldeau [Thu, 15 Oct 2015 16:07:05 +0000 (12:07 -0400)]
analysis: Add definition for extended events and fields
Change-Id: Ibf340e974aa9eef007facf7b29717fc125683667
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Reviewed-on: https://git.eclipse.org/r/58260
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Patrick Tasse [Fri, 16 Oct 2015 17:39:57 +0000 (13:39 -0400)]
tmf: Bug 479994: TmfTraceSelectedSignal not sent when editor reused
This ensures that TmfTraceSelectedSignal is sent when a trace editor is
reused with a TmfEditorInput after being initially opened with a
FileEditorInput, for example by clicking on a bookmark.
Change-Id: I9cf4d5f002856ff0f3320c4c2dd958b3a25964a4
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/58356
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Wed, 8 Jul 2015 19:26:20 +0000 (15:26 -0400)]
lttng: Add unit tests for the debug-info analysis
Change-Id: Ib1e302945f0c231d509e8c5a12d3528a33e4119e
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/51664
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Mon, 21 Sep 2015 19:04:26 +0000 (15:04 -0400)]
ctf: Depend on the tracecompass-test-traces project
Instead of a manually downloading the test traces, we can have the tests
depend on the new tracecompass-test-traces project, which ships all test
traces used by Trace Compass tests in jars.
The test-traces's p2 repo is now part of the target definitions.
Change-Id: I7a1ef80263c2a1e49202d889754e04b2ba44ff3f
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/56466
Reviewed-by: Hudson CI
Francis Giraldeau [Tue, 30 Jun 2015 02:16:12 +0000 (22:16 -0400)]
Analysis: Add unit tests for the critical path module
Change-Id: If3c2cc85675314b5278c2dfedf4dd4dec554eaab
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/51082
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Francis Giraldeau [Tue, 30 Jun 2015 01:52:14 +0000 (21:52 -0400)]
Analysis: Add the active path module
This module computes the active path from a execution graph. The active
path of a given task has all the blockings replaced recursively by the
edges representing the cause of the wait at the system level.
Change-Id: Ie19b461ebe5ad595bca43b55f380ce28db416445
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Reviewed-on: https://git.eclipse.org/r/51081
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Wed, 27 May 2015 21:49:39 +0000 (17:49 -0400)]
lttng: Introduce a debug-info analysis for UST traces
This UST-specific analysis will track the loading and unloading
of shared libraries, and will allow to match an instruction pointer
from an event context to the corresponding binary or library.
Once we have the correct binary (executable or .so) identified,
we can match it to an actual binary file with debug information,
and retrive the corresponding address/filename/line number, just
like is currently done with the Callstack view.
For now, the analysis will only map an instruction pointer to
the corresponding binary file. A future step will use CDT
binary parsing facilities to get the function name and line
number corresponding to the instruction pointer.
Change-Id: I6127e89cc5d3d8c7abff01610b8a6dc9a612f149
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/51662
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Francis Giraldeau [Thu, 15 Oct 2015 14:51:27 +0000 (10:51 -0400)]
Add linux softirq ids
The SoftIRQs are pretty much cast in stone on linux. This patch adds the
definition, as they are found in the softirq_entry event and vec field.
Change-Id: If4b3931d777eda540e3f2090939cd59ecd8d1643
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Reviewed-on: https://git.eclipse.org/r/58256
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Matthew Khouzam [Wed, 14 Oct 2015 01:56:39 +0000 (21:56 -0400)]
lttng: Add time alignment work to scatter chart
Removes adjustRange() on the X-axis because it can change the desired
range when no data point exists at the range bounds.
Changes TmfCommonXLineChartViewer to use X-axis range for time alignment
instead of first and last data points.
Change-Id: Ia15eff9a258da0b20d0204c25f003768adc6b5e0
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57011
Reviewed-by: Hudson CI
Francis Giraldeau [Thu, 15 Oct 2015 14:34:46 +0000 (10:34 -0400)]
Make the file formattable and adjust header
Trivial code cleanup patch.
Change-Id: I6fe8b28fb885c34d3a9dd7d7bd52bf02de78f723
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Reviewed-on: https://git.eclipse.org/r/58255
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Matthew Khouzam [Mon, 21 Sep 2015 20:18:01 +0000 (16:18 -0400)]
Introduce Latency scattergraph
Change-Id: I0adfe61c72dc5769f96921f10f810d457aa16133
Signed-off-by: France Lapointe Nguyen <francelap@gmail.com>
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/54310
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Cédric Biancheri [Fri, 25 Sep 2015 15:31:43 +0000 (11:31 -0400)]
tmf: Add the possibility to choose the root node for synchronization
The root node in the spanning tree used for synchronization was
always the first hostId in lexicographic order.
Now the user can choose the trace that will be used as reference
for the synchronization.
When the user wants to synchronize an experiment a window similar
as the one used to add a trace to an experiment is shown.
The first trace is selected by default.
Change-Id: Iaaeb4c7905dc6171d9f8a40c4dbf4e8ba65d5ebd
Signed-off-by: Cédric Biancheri <cdc.biancheri@gmail.com>
Reviewed-on: https://git.eclipse.org/r/56769
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Wed, 14 Oct 2015 02:41:42 +0000 (22:41 -0400)]
analysis: add icons for analyses
Change-Id: I494ca354404aa85e9af240567b7981e2921bb069
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58121
Reviewed-by: Hudson CI
Matthew Khouzam [Wed, 14 Oct 2015 00:12:10 +0000 (20:12 -0400)]
analysis: Fix TreeMapStore's serialization
Change-Id: Ia6c3299d8e0e940cae52118c14ca04c16aedd049
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58118
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Hudson CI
Matthew Khouzam [Wed, 14 Oct 2015 03:03:04 +0000 (23:03 -0400)]
analysis: add latency table icon
Change-Id: I2565d266495f6db2a555cfa1840ff66d3fa8e924
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58122
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Matthew Khouzam [Wed, 14 Oct 2015 03:06:54 +0000 (23:06 -0400)]
ust: add memory analysis icon
Change-Id: I3affe2d06c1ac80cf99408d37b55d6525c4e4294
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58124
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Alexandre Montplaisir [Mon, 25 May 2015 16:31:13 +0000 (12:31 -0400)]
Add null-checks for Map.get()
Map.get() can return null, and this can mean two different things:
1) The value associated with this key is 'null'.
2) There is no such key in this map.
Java 8 type annotations (Map<K, @NonNull V> for example) can cover
for the first case, but not the second! Map.get() can *always*
return null, no matter the map declaration. The enhanced null analysis
reveals that.
Many places in the code call Map.get() without null-checking, we can
fix these cases in advance. In many cases, this also exposed places
where iterating on map entries should have been done instead of
iterating on keys and doing a lookup.
Change-Id: I8b94a7ed7193f41479b3b0bc9b5990213b9718a3
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/52015
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Alexandre Montplaisir [Thu, 9 Jul 2015 18:30:50 +0000 (14:30 -0400)]
Add a script to check for outdated Maven dependencies
We already have a Hudson job that runs this, we could commit
the script in the git tree so that it can be run locally.
Change-Id: Iefcae17dad4639e917e3440bce6929811773bd9c
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/51682
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Hudson CI
Marc-Andre Laperle [Tue, 6 Oct 2015 21:55:14 +0000 (17:55 -0400)]
tmf: Allow refreshing manually all ITmfTraceEditors and traces under experiment
This is a follow-up to the comments in https://git.eclipse.org/r/#/c/57465
Change-Id: Ia6e8ecc056a5c662255c35f029f72bbe1cc2785a
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57559
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Marc-Andre Laperle [Tue, 6 Oct 2015 22:21:41 +0000 (18:21 -0400)]
Update Oomph setup file with Trace Compass 1.1.0 baseline
Change-Id: Ibf9ccc8474731010dc59e5a3a3d2d1d562c088b2
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57562
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Marc-Andre Laperle [Tue, 6 Oct 2015 22:00:11 +0000 (18:00 -0400)]
tmf: Fix minor typo in test
Change-Id: I50db090d9eeaccbbad5fc57d753bbc33770f4bf2
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57560
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Marc-Andre Laperle [Tue, 6 Oct 2015 20:21:15 +0000 (16:21 -0400)]
tmf: Fix time-axis alignment for cases where the series is not sorted
The time-axis alignment code for XY chart viewers (SWTCharts) assumes
that the series in the chart will be sorted and that the last item in
the series will be the one with the highest value. Since SWTChart
allows specifying series values in no specific order, this should be
handled properly in the time-axis alignment code.
We do not have any views right now that shows this behavior but if
you want to see this bug, you can try this patch in combination with:
https://git.eclipse.org/r/#/c/57011/
Change-Id: I8a850f9a12f945fe5616b0b82176d98ea45c33ae
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57552
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Marc-Andre Laperle [Tue, 6 Oct 2015 21:21:01 +0000 (17:21 -0400)]
Remove tracecompass-baseline-1.0
Change-Id: Ia04f2a5b1a3cbe3576579712e458f39efc9bbd35
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57557
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI
Marc-Andre Laperle [Sun, 20 Sep 2015 18:06:21 +0000 (14:06 -0400)]
releng: Upgrade jboss repository-utils to 0.23.2
This is the latest stable version available for the plugin.
We need to use skipBuildInfo otherwise some extra json files
are generated.
Change-Id: I437afd1483705311cdf36961f23d6c8df105c054
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/56317
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Marc-Andre Laperle [Mon, 5 Oct 2015 21:45:49 +0000 (17:45 -0400)]
Change some @since to 1.1 for things that were really added in 1.1
Change-Id: I57e900622e1e31984233391beebe3ce5a0cb6e07
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57456
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Marc-Andre Laperle [Mon, 5 Oct 2015 20:20:49 +0000 (16:20 -0400)]
Add Trace Compass 1.1 baseline, fix some API errors
Change-Id: I356706eae6fd2afb85f0f32fae91291c7e11dcad
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57447
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Matthew Khouzam [Thu, 1 Oct 2015 02:03:24 +0000 (22:03 -0400)]
tmf.core: make ITmfCallsite getFileName @NonNull
Change-Id: Ie93ca879cb2954bb86eaf68f3dea40fb056b6c2e
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/42916
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Matthew Khouzam [Tue, 15 Sep 2015 14:40:18 +0000 (10:40 -0400)]
tracecompass: update pom sonar plugin
This should add support for sonar 5.1+ if eclipse.org upgrades to it.
Change-Id: I02d8887d1f8b95a0b5696564d447ee341f643b21
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/55990
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Patrick Tasse [Thu, 1 Oct 2015 22:31:24 +0000 (18:31 -0400)]
tmf: Link bookmarks with resource markers in abstract time graph view
Change-Id: I5a0c8f03df5606c2bb42517bae10b7f52e539fcc
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/57254
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Patrick Tasse [Wed, 30 Sep 2015 22:32:15 +0000 (18:32 -0400)]
tmf: Support user bookmarks in time graph
Adds the Toggle Bookmark, Select Previous Bookmark and Select Next
Bookmark actions to the time graph viewer and the abstract time graph
view.
Adds an Add Bookmark dialog.
Adds an ITimeGraphBookmarkListener interface.
Uses the old Previous/Next Event icons for the Previous/Next Bookmark
actions, and modifies the Previous/Next Event icons.
Change-Id: Ic7e51c4cf7aab4f78b49ec7bc9e2b87faff9f325
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/57135
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Marc-Andre Laperle [Tue, 29 Sep 2015 21:01:27 +0000 (17:01 -0400)]
tmf: Add SWTBot tests for manually refresh of traces
Change-Id: I6e52066382b958e2c50ebdd9982e8651bdc87889
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57138
Reviewed-by: Hudson CI
Patrick Tasse [Tue, 15 Sep 2015 19:16:05 +0000 (15:16 -0400)]
swtbot: Add some failure messages to test conditions
Change-Id: I725b75e7f66447532b3eacd5bd86518ebfee8c8c
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/56017
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Marc-Andre Laperle [Fri, 25 Sep 2015 23:52:27 +0000 (19:52 -0400)]
tmf: Add command to manually refresh traces
At the moment, only the events table will be refreshed.
Change-Id: I7b9933d8e52e7e7fc821a3b41f1e688bf4c81d09
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57137
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Jean-Christian Kouame [Wed, 30 Sep 2015 14:30:46 +0000 (10:30 -0400)]
tmf: Delete unused code in GraphNode
Change-Id: Iab9a0769235f3b05ed25100e3c821d283a6e0422
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57087
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
This page took 0.046455 seconds and 5 git commands to generate.