Add workspace/ to the gitignore

A top-level workspace/ directory gets created by the Maven build now.
It should not be tracked in Git.

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

lttng: Rename orphaned relayd test to TestApplication

This file is a test application that is not intented to be tested as a Junit
test but it is written as such for convenience. We use this while developping
the Live feature.

Change-Id: I705f5a9fdfcb5b01b9478fe5402b48587a4c871c
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/59441
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: Add Lost Events marker event source and factory

A marker event source factory for Lost Events is enabled for all traces.

The marker event source can reuse a previously returned marker list if
the requested parameters are identical to the previous request.

The event types statistics state system is updated to add a new
attribute which is used to compute the time ranges that are covered by
at least one lost event time range.

Change-Id: I2479afc3f07deb363f5847f4816940010bae3e9e
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59389
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: Add TmfTraceAdapterFactory

Base class for a trace adapter factory. The factory creates a single
instance of each adapter type per trace, and disposes the adapter when
the trace is closed, if it is an instance of IDisposableAdapter.

Change-Id: Ieec69d17d0cff037f2df0c8dded14eef91db6f97
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59388
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: Support adjustment of time graph window range on range selection

When the selection range is modified and 'ensureVisible' is true, then
the window range will be centered on either the selection begin time (if
it has changed) otherwise on the selection end time, if that time is
outside of the current window.

This adjustment is now enabled in time graph views when the
TmfSelectionRangeUpdatedSignal is received.

Change-Id: I8b5ec885b00009be05222db02acd745218759131
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59758
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: CallStackView sets wrong bounds

Apply the same changes that were done in patch
https://git.eclipse.org/r/#/c/55593 for the AbstractTimeGraphView.

Change-Id: I3f1bf9b8cc8dc017e60d92b079e74c892e3b8530
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59856
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: Add keyboard shortcuts for Next/Previous Marker

Support '.' for Next Marker and ',' for Previous Marker.

Support Shift key for extending the selection, both with the keyboard
shortcut and with the action button.

Update the status line when Next/Previous Marker changes the selection.

Clarify the marker event source API to indicate that the previous and
next markers should be included in the marker list.

Change-Id: I1f435e9620030809f10f3788eca40521c8a745a9
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59512
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

analysis: Fix missing setItemCount in AbstractSegmentStoreTableViewer

When the last trace is closed, the table viewer input is set to null.
Its item count should also be set to 0. Otherwise this can later cause a
NullPointerException in the TableViewer.

Change-Id: Ic1d482fd5e4ee64f7969375b1fe4d8b626fa31e0
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59781
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 : Change the order of modifiers of reinitializeCharts() in TmfPieChartViewer

Change-Id: I122d2706472dd5a421ee3e2da1c7277f90959783
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57109
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

rcp: Add compile-time dependency to jdt.annotation

The rcp.ui plugin will indirectly depend on the null annotations
due to upcoming new external annotations.

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

common: Rectify some annotations in NonNullUtils

In-line genereic type definitions should not have @Nullable/@NonNull
annotations. These should be only on the "real" parameter and/or return
types.

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

lttng: Remove discriminator part when parsing addr2line output

Remove discriminator part, for example: /build/buildd/glibc-2.21/elf/dl-object.c:78 (discriminator 8)
This fixed a NumberFormatException

Change-Id: I92682bee4106d64dcf2964dc98823a80597df3fa
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59298
Reviewed-by: Hudson CI

Upgrade to Tycho 0.24.0

See https://wiki.eclipse.org/Tycho/Release_Notes/0.24
The immediate improvement is that it uses JDT from Mars.1 to compile.

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

lttng: update and add HR timer related events and fields

Change-Id: I87b3e1ac4a79059a2c70b1b3ef7e07be6baa4f10
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59011
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

analysis: Don't run latency analysis automatically

This reverts an unintentional change.

Change-Id: I5e4a9b1b8e8a2f54aac7924b78d1200f831497e6
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59017
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: Fix simple Javadoc warning

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

ss: Replace some asserts by IllegalStateExceptions

Change-Id: I692279143582c78b629d973107ec65e40f850793
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/56014
Reviewed-by: Hudson CI

ss: Fix a bug where depth of history tree increases at each new node

When adding a sibling after a new root, there was one too many levels.

Change-Id: I19f460347cbba91c5c1f70d9ee4566a3cb164f37
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/59076
Reviewed-by: Hudson CI

TMF: Fix behavior of XML time graph views with experiments

Expected behavior:
* Show the content of analyses when the trace contains those analyses.
* If experiment, show the content of analyses if any of the trace or
the experiment itself contains those analyses.
* Empty if no trace contains the required analyses.

Change-Id: Id04c2f9d4f1ecd9c8c466a9a0536e1b07277ea1f
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/35490
Reviewed-by: Hudson CI

ss: Fix wrong calculation of string size in history tree

The strings size was calculated twice in the node size. The nodes were
therefore not used at full capacity.

Change-Id: Iea15dfd701859deaaea7857de83c36c3455eefb9
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/52755
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI

ss: Add AttributeTreeTest unit test

Change-Id: I7a67d33ad694370393fa2789e2508233ec570877
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/45686
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

lttng: pack columns in latency table

Change-Id: I18855e0e4d728aca681bcd39229ca156e45cedb8
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58062
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

analysis: rename latency to segment store for statistics

Change-Id: I1bd00e238843e9256d0ffcab38c1752ac0f35180
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59134
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI

analysis: avoid division by 0 in latency statistics

Change-Id: Iaa30a5989517777b37aa4b7b21b5c4da025dcec9
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59133
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

analysis: Add null check for dependent analysis

This fixes a NPE and also avoids execution the statistics analysis on
other trace types than LTTng Kernel when the view is open.

Change-Id: I6583dcbb80044586f25067f32a5fbd9f28c7d4ed
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59045
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

analysis: Rename latency related classes to System Call Analysis

Change-Id: I16393568ca9453768f4d95f4e4bbef8edae85327
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58988
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

analysis: Allow segment table viewer/provider to accept ISegment[] input

This makes it easier to use a subset of a segmentstore as input.

Change-Id: I39b453805b9dc9f612bc0d72372ebde3c8cc4774
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59155
Reviewed-by: Hudson CI

lttng: Fix AttributeNotFoundException when dealing with event before state dump

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

common: Add external annotations to NavigableSet

The following methods in NavigableSet:

ceiling()
floor()
higher()
lower()

can all return null if there is no corresponding element in the set,
independently of the container type declaration. They should be
marked @Nullable.

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

common: Annotate some methods in String

All variants of:

copyValueOf()
format()
intern()
toString()*
valueOf()

return a non-null value, so can be marked as such.

* Do not confound with Object.toString()! That one has no nullness
  guarantees.

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

lttng: Do not throw exception if callsite binary is missing

If a trace with debug information is opened on a different machine
than the one where it was taken, it's very possible for the referred
binary files to not be present. In such a case it should not throw
an exception, but simply return that no source callsite information
is available.

The (upcoming) "binary callsite" information should still be available
in such cases though.

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

Replace instances of FluentIterable with Java 8 Streams

Guava's FluentIterable was already used in some places to make use of
functional idioms. Now that we have moved to Java 8 we can replace
those with base Java Streams.

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

releng: Transition to jdt.annotation 2.0

The new o.e.jdt.annotation 2.0 annotations (based on Java 8
Type Annotations [1]) can now apply on generic type declarations,
allowing things like "List<@NonNull String>".

The @NonNullByDefault annotation, already in use, will now apply
to these locations. This required some code changes to eliminate
compilation errors, notably in places where null checks were
previously made and were not needed, or where some existing
declarations had to be specified as @Nullable.

Another relatively important change is the new syntax for []
arrays. Before, only "@NonNull String[]" was allowed, which meant
"a non-null reference to an array of unspecified Strings". Now, it
instead means "an unspecified reference to an array of non-null
Strings".

It is still possible to specify the array reference itself, but by
doing: "String @NonNull []". Both locations can be annotated
independently.

[1] See https://docs.oracle.com/javase/tutorial/java/annotations/type_annotations.html

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

Add null checks for methods missing them

In preparation of annotation-enforced @Nullable Map.get() etc.

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

tmf: Add drop-down to Next Marker action to set navigation enablement

The Next Marker action now has the AS_DROP_DOWN_MENU style. The menu
items allow to disable navigation of markers by category.

The Next/Previous Marker actions now skip markers of disabled
categories.

The marker list is cleared immediately when switching traces in the
abstract time graph view. If the trace was not a supported trace for
that specific view, the marker list would never get updated.

The marker actions now disabled if the time graph has no content (empty
time bounds).

The enablement/disablement of the Next/Previous Marker actions based on
current time selection that was requested and implemented in
https://git.eclipse.org/r/57135 is reverted for the following reasons:

- It would now require to go through the list of markers and comparing
each marker category against the set of disabled categories, just to set
the action enabled state, whenever the time selection changes.

- It would make the Next Marker drop down menu inaccessible if the
current time selection is after the last marker.

Change-Id: Idd8361ccc6a00c0bb19749019965e5389aa2692a
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59171
Reviewed-by: Hudson CI

tmf: Add Show Markers menu for time graph viewer

Support the dynamic Show Markers menu in time graph viewer that contains
a checked action for each of: Bookmarks category, view-specific marker
categories, and trace-specific marker categories for the current trace.

Add the menu to the abstract time graph view's view menu.

Hide markers of categories that are unchecked in the Show Markers menu.
This will also make them ignored in Next/Previous Marker navigation.

Change-Id: I31c0ee983c217a149070f8628a162fc7420222d6
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59072
Reviewed-by: Hudson CI

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

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

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>

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>

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

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

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

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>