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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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

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

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>

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>

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>

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>

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>

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>

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>

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

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

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

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

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>

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>

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>

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>

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>

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>

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>

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

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>

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

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

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

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

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>

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>

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

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

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

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>

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>

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>

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>

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>

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>

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>

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

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

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>

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>

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

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

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

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>

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>

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

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

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>

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>

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>

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

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>

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>

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

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

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

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>

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>

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>

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>

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>

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>

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>

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

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>

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>

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>

lttng: Add utility methods to get CTF tracer versions

It's not uncommon to check for tracer names and versions of CTF
traces. We can offer convenience methods to do so, instead of
having to parse the environment metadata every time.

Change-Id: I9c4cbc7e36d24d972889f6c4bbc16d2db852f741
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/51661
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI

lttng: Use SWTBotJunit4ClassRunner in CallStackViewTest for screenshot on fail

Change-Id: I078f3c1ca4c18fd0f9d5a247f42ea71ae1f3823d
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57145
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf : Redefine CompareTo() in EventOccurrenceObject

Change-Id: I6251531e51ff398afdcc892d915092203f011dec
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57108
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>

tmf: Support markers in abstract time graph views

Change-Id: Iadbe8cd42681fec05ca295b095b60e333edb6865
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/56753
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf: Avoid duplicate literals in GraphNode

The string literals have been extracted into a constant variable

Change-Id: I4f48ca60faac42186bba26f9f9513993409d2bbe
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57085
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf.ui : Iterate in OffsetTraceHandler using the entryset instead of the keyset

Change-Id: I2c7c510feec20793570faa6723a7c8e1e0ed1ab6
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/56992
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

Use pde.junit runtime from target to allow e4.5 tests to run on Eclipse 4.6

Fixes trying to run tests with Eclipse 4.6 targetting Eclipse 4.5 using Java 7.
Whitout this change, the runtime from Eclipse 4.6 gets picked up and it
requires Java 8.

Change-Id: I8597ee30fd094bad5b004e21d3fa5b939ca49b7a
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/56563
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>