git.efficios.com Git - deliverable/tracecompass.git/log

tmf: Fix zoom with mouse drag on time graph scale

The time graph scale no longer has a name space. It is now occupied by
the time graph viewer's tree header.

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

releng: Update staging target with Oxygen M4 sites

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

releng: Update 4.6 target with Neon.2 sites

Change-Id: I443893ab9f0c846486468360d354a6dee5f09a84
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/89248
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>

timing: Extract statistics to a generic class

These statistics receive any object in parameter and allow to specify a
function that returns the value of the object on which to do the stats.

Change-Id: I16fb1801dd8639fa8cd4da02c095163e53e6709b
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/88689
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI

ss: Remove the synchronized from the full query

The full queries do not need to be synchronized. Only the transient state
does and it is.

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

tmf: Fix some critical and major Sonar warnings

Security - Array is stored directly
Naming - Suspicious constant field name
Performance - Should be a static inner class

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

analysis: Fix concurrent data access violations

Fixes CID 160189, 160190

Change-Id: I7046182081d2f754821c4938874f81d6efef870e
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/88800
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>

tmf: Scale tree images in time graph

If the item height is smaller than the image height, scale the image.

Change-Id: I9393a7d0d7eebf28c30c4a51e271d7c6ea8e2216
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/88688
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Bug 509691: Changes to mutable trace context can be lost

The method TmfTraceManager.updateTraceContext() is now used to apply all
changes to a trace context under synchronization of the trace manager.

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

ss.test: Fix nullness warnings

Add a few @NonNull annotations to remove warnings on code that is never
null.

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

tmf: Update XML trace stub to use ns directly

Now that custom parsers support nanoseconds timestamps, the custom XML trace
stub does not need to convert the timestamps to nanoseconds anymore, it can
be defined directly in the XML trace definition.

Change-Id: I2791b350c303bb82df9f83f0cabbb663e1166a75
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/88513
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf: Deprecate TimeGraphCombo and support columns in TimeGraphViewer

The class TimeGraphCombo is deprecated and AbstractTimeGraphView method
getTimeGraphCombo() is deprecated and returns null.

The TimeGraphViewer supports columns and label provider with new methods
setTimeGraphLabelProvider(), setColumns(String[]), setWeights(int[]) and
getTree(). The tree contains no tree items and is only used to show
column headers and allow for column handling.

If no column names are specified, a single nameless column is used.

If no label provider is specified, the time graph entry's name and the
time graph presentation provider's item image is shown in the first
column.

The TimeGraphViewer is made to extend the Viewer class.

The TimeGraphControl supports auto resize of columns to grow columns at
the item text's preferred width. Auto resize turns off automatically the
first time that a user resizes a column to be narrower than its
preferred width.

The tree expand and collapse controls are changed to be triangle widgets
that are bound in size by the item's height.

The name space split line is made easier to hover with the mouse.
Double-clicking it snaps the name space to the right of the last visible
column text's preferred width. The split line can now be dragged even if
the viewer has no input model.

The TimeGraphControl methods getTree(), getLabelProvider() and
getItemBounds(ITimeGraphEntry) are added to help with UI testing.

Change-Id: I1061371b083499298da4f7c5640929e63ed651db
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/84135
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

releng: Update maven plugins version

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

common.core: clean up ProcessUtils a bit

While the code does work, it has some issues that could potentially be
problems with the code if not used in the way it should.

* Extract magic number to constants

This is simple, it makes modifying the behaviour easier.

* Explicitly fill code blocks for catch blocks

Improve code readability. It makes it clear that it is intentional.

* Remove redundant modifier on interfaces

An interface is always static, there is no need to add the keyword
"static" to it.

* Add charsets to inputstream

This explicitly expects the default encoding (UTF8). However, this is
not the ideal solution, it would be much better, later to have an
interface with the charset passed in it. It can be used if we want to
launch a process in another machine.

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

tmf: Bug 510144: IndexOutOfBoundsException in CallStackView

Get the quarks list before doing the full query, and when accessing the
full query result later, do a bounds check and redo the full query if
necessary.

Change-Id: Ica9ab6d414fb2286ad41deceff0b2be911be64d9
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/88304
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

ss: Move the safe byte buffer to datastore plugin

The unit tests also moved

Change-Id: I9118cb56bad032ce61065a5a81513c03a47fcc84
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/85799
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>

ss: Change datastore.tests to a fragment plug-in

This will allow us to use the same package names in the test
classes as the runtime ones. That way tests will have access
to package-visible (default visibility methods).

This will remove the need of having tons of public methods just
for testing. Default-visibility methods (or even protected, where
needed) are arguably much safer, and won't risk confusing future
clients.

Change-Id: I0393ac9103f94afcb9a818cbe2e7c8dfcadcac18
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/87163
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>

tmf: Bug 509613: Trace Manager loses Linux trace context data

The trace context classes are updated to provide a builder that can be
extended and that is used to create a duplicate of a trace context with
some data that can be modified.

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

doc: Add futex analysis user guide

Add the futex doc and update the name of associated timegraph views.

Change-Id: Ia94940935d8516a1ff5a8e35bb5c283cb26d5a69
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87448
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf: Fix concurrent data access violations

Fixes CID 157631, 157632, 157633, 157634, 157635, 157636.

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

tmf: Fix remaining TmfSignalManager leaking listeners

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

tmf.ui: replace size()==0 with isEmpty()

Changes CustomTxtParserInputWizardPage to better handle checks. Size is
not guaranteed to be O(1) whereas isEmpty() is.

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

tmf: Fix unresponsive UI when mouse wheel zooming in TimeGraphViewer

When developing the time graph marker axis, it was noticed under Windows
that there was a delay in painting the marker axis when compared to the
time graph state area. It seems that if a new redraw request occurs
while the time graph state area is being painted, the following control
in the draw order (the marker axis) is not painted before until the next
paint event. The visual effect is that the time graph state area follows
the user input while the marker axis does not update until the user
input stops.

The solution found was to call Control.update() on those parts, which
would block the UI thread until all parts have been painted.

However, under Linux this can cause pending user inputs to be queued.
While the time graph is painting, rapid mouse wheel scrolling queues
many MouseEvent that each individually trigger blocking paint events.
This also prevents many mouse wheel scroll events to eventually combine
into triggering a single paint event. After quickly scrolling the mouse
wheel many times then stopping, the user can see the queued events
triggering a multitude of paint events that can last a few seconds even
while there is no user input. Any new user input needs to wait for all
the queued paint events to be completed before being processed.

The blocking calls to update() are therefore removed. This can cause a
temporary visual misalignment under Windows, but this effect is much
less noticeable under Linux.

Change-Id: I12d3d17eff0d8a443822083e0aa2113c3b23fedc
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/87246
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf.xml: Fix xml timegraph resolution

When the window range is less than the display width, the resolution was
then equal to 0. The time event was not shown until the user change the
zoom level. Now we set the resolution value to 1 instead of 0.

Change-Id: I4f0ae39576c452f0bdfbaf23b836f3af758f0b91
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87339
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf.xml: Add SWTBot XML timegraph view test

This test the loading and and viewing of xml trace. An
Xml view is created then validated.

Change-Id: I908771685129942eb044a2dbb497e377180a202e
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87340
Reviewed-by: Hudson CI

linux.swtbot: add KernelMemoryUsage SWTBot test

This test will make sure range selection and the data chart
have sane values.

Change-Id: I69f348d49164991a1466a63ef4c2d536371171e4
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87193
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI

linux.core: fix memory allocation issue.

The order field in page alloc/free say how many pages are allocated.
It was ignored, which gave false totals.

It is artificially limited to 2^62 page size, as larger would garanty
an overflow.

With this patch, memory overflows can occur much faster, but only on
systems which trace or report over Long.MAX_VALUE memory.

Change-Id: I5c805115cb3aaa5c6bc6146a6dc0db20f11854ef
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87181
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>

tmf.ui: bug 503678 The ss explorer should not execute analyses

And because the state system does not execute the analyses, the state
systems may not exist when the tree is built. The hierarchy is thus
changed so that the analyses themselves appear as top-level items and
their children are the state system. This way, if the state system did
not exist at first and appears later, it will show when the view
refreshes (without having to refresh the whole view).

Change-Id: Icc750abcdfb27d74d486b9fdcf8b3e910310684f
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/83811
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf: Simplify boolean expression in handleEvent()

Change-Id: I59f992cf166050f80d8db50c2740596fc3135ff1
Signed-off-by: Sonia Farrah <sonia.farrah@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/72105
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>

linux.core: add Order to fields of Kernel Analysis Event layout

This will allow getting the number of pages allocated.

Change-Id: I16a0aa00cb4b32f83c964cecc9543d2e0add4e85
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87180
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI

tmf.ui: add FIXME to TmfSymbolProviderUpdatedSignal

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

lttng.swtbot: add state system explorer SWTBot test

This test runs three passes, opens the view, re-opens the view
then re-opens the trace.

Future tests can look at the content of the state system. This
test enumerates the state systems available.

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

tmf.ui: Avoid creating two Files for an archive import

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

tmf.ui: Gracefully handle invalid tar import operations

A tar.gz file may have an invalid tar file in the gzip. The
getNextEntry would return null in such a case. This performs
that null check in order to validate the tar and avoid a user-
facing NPE.

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

tmf: Replace Array Loops to copy arrays with System.ArrayCopy

This yields a performance improvement as well as an increase
in legibility.

Change-Id: Ia7be6a54476a8263d8b5e99f203d7fc3cf374a1d
Signed-off-by: Sonia Farrah <sonia.farrah@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/71939
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

ss: replace a for loop with a foreach operation

And extract a constant

Change-Id: If4e65223ed4e9ce617eb9da3023db0294ede151e
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/86172
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-by: Hudson CI
Tested-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>

tmf.ui: fix state system explorer ID

It now matches the plugin.xml's ID

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

ss: remove linear component from node search.

Before, nodes had their intervals only sorted by end times,
which meant finding the first interval which started later
than time t implied a binary search for any interval ending
at time t, then a linear iteration to the first interval
ending before (inclusively) t.
We remove this component by ordering intervals by end times
then by start times, so that the binary search can directly
return the first interval ending before (inclusively) t.

Change-Id: I3e4fd02880d56360fc082023c08b92f8a6e4350e
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/86865
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>

kernel.swtbot: test resources view filters

This tests the filter functionalities of the resources view.

Change-Id: Ifdf76ab29e843e1b8d91d6483373abf86e8763b7
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84849
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

ust: Add swtbot for callstack view responsiveness

This swtbot test navigates a trace with the callstack view opened. This
test is not meant to be run in the performance test suite, but enabling
JUL logging with it allows to have a reproducible series of UI operations
and analyze its output.

Change-Id: If4081a4581bb1c774c3643fc0bb0dd418c421379
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/80258
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf.ui: fix Bug 479996 infinite loop

Opening an empty trace called queryFullStates an infinite number
of times resulting in OOM. Checking the arguments will avoid
this.

Bug 479996

Change-Id: I0ac29da7a883d154fffb5c4d24aa50db32796a6f
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/86862
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

Update Oomph setup with latest API baseline

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

xml: support any aspects to be used in XML analysis

Some traces have aspects to show special data in a column. With this
commit it is possible to use any aspect in the XML anlysis.

Change-Id: I9dedc910e9465ac94f82912dc9095e9d315b0882
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/86002
Reviewed-by: Hudson CI
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

Bump versions for 2.3.0 release

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

releng: Add 2.2.0 baseline and fix API errors

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

common: Add a ProcessUtils for external process launching

Consolidate the existing external-process-launching utility
methods into a new common class. Right now this includes the
FileOffsetMapper, which calls addr2line for debug-info symbol
resolution, as well as the LamiAnalysis class which calls
the LAMI analysis scripts.

Bug: 508406

Change-Id: I685fb461a93cd6726575b5df771233f37e423e5f
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/85973
Reviewed-by: Hudson CI

test: Make CtfTmfTrace stubs of the right class

Kernel and UST traces created with CtfTestTraceUtils were not of the
right type, which prevented the corresponding analyses to execute.

Add LttngKernelTestTraceUtils and LttngUstTestTraceUtils which create
similar stubs of the right type (LttngKernelTrace and LttngUstTrace).

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

tmf: Fix TmfSignalManager leaking listeners

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

tmf.core: add resource leak detector

If the signal manager is disposed, it should be empty. Anything still
there is a resource leak.

The output looks like this:

Resource leak: org.eclipse.tracecompass(...) was not deregistered.
org.eclipse.tracecompass.tmf.c(...)TmfSignalManager.java:77)
(...)

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

segstore: Add assert on item count for iteration benchmarks

This patch adds asserts on the item count when iterating the segment
store in the benchmark. It makes sure that all iterations actually
iterate over all the elements of the segment and does not give a false
impression of performance if there is a bug in the iterator.

Also remove the System.gc() that are not useful to the test

Change-Id: I87acbb264703f06a166239655f3911697c17126e
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/85724
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>

tmf: Add missing icons in XML analysis

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

os.linux: Manage sched_waking events

LTTng 2.8 (and kernels around that time) introduced the
sched_waking event. This event has the same fields as the
sched_wakeup one, and should cause the same state changes.

Since the linux-generic KernelStateProvider has no knowledge
of specific LTTng event layouts, we have to bring this
definition up into the base interface. Older traces simply
won't have any of these events.

Change-Id: Idf9eaf1ecba0346b22283adfbe71ad56d52b045f
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/85804
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>

tmf.ui: use binary search when adding child to TimeGraphEntry

This brings the search time complexity to O(log(n)) down from
O(n).

Change-Id: I5b97c0f341bfe712d1bb8f9fd95cffc9ccd8686c
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/86000
Reviewed-by: Hudson CI
Reviewed-by: Jonathan Rajotte Julien <jonathan.rajotte-julien@efficios.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

tmf.swtbot: Stabilize and enhance TestImportExportPackageWizard

The test would fail if ran 1000 times in a row. File#createTempFile
appears to be the culprit.

Remove a useless "waitForJobs()" should accelerate by 500ms.

Add tests of the output size, removes two manual tests from Trace
Compass test spec.

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

common: Fix @since tags in DecimalUnitFormat and DataSizeWithUnitFormat

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

common: Support parsing in DataSizeWithUnitFormat

This allows parsing strings which represent a number with an optional
unit and binary prefix, such as "1 kB" (1024), "8 B" (8), "1.5M"
(1572864), "10 Gb" (10737418240), etc.

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

common: Support parsing in DecimalUnitFormat

This allows parsing strings which represent a number with an optional
unit and SI-prefix, such as "1.2 kb" (1200), "1.2" (1.2), "1.2 m"
(0.0012), "12 ns" (0.000000012), etc.

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

common: Add StreamUtils method to wrap an Iterator into a Stream

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

tmf: Add extension point to add custom trace types to plugins

This new extension point is used to contribute custom Text or XML
trace types so that they are available automatically without the need
to be imported by the user manually.

Change-Id: Iff481d659626a8ed8d756d333ec6a240b4316b49
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85831
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI

ss: Change getIteratorOverQuark to a public class

This patch allows iteration to go backwards and forwards,
introduces a unit test and uses the new iterator for:
KernelThreadInformationProvider.getExecutableName

Change-Id: I41f35e30bd1984662f55b8809e2497a6c19d5468
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/85798
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI

analysis.lami: Fix internal signaling with several views on the same report

Use the parent LAMI tab page instance as the key for internal signaling.

The signal key is designed to restrict the signal to interested objects.
It was based on the hash of the immutable fResultTable field. Since for multiple
view of the same report the fResultTable is the same object, signals were
processed by objects that were not concerned. Using the result table
instance/hash as a key is not sufficient.

Bug: 495322

Change-Id: Ifba2056b10cd254d1a8c6c030c4b7263463ac834
Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/86103
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

common: Annotate java.util.Iterator

Just add an annotation to forEachRemaining(). The goal is to have
_some_ annotation in there so that the next() method makes use
of the declared type, and we don't get the "this type doesn't
seem designed with null annotations in mind" warning.

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

tmf.core: Add unit tests for state system re-read

This adds a unit test for a state system with full backend where the
results are re-read from a previously existing file.

Change-Id: Iab5fab6c6282f2411d2e4a0d3b0da3824653cb36
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/84036
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf.core: Throw exception when no state system assigned

Instead of just logging an error that will happen for each event and
willl be very verbose, it should throw an exception.

Change-Id: Iaedcf461ffa5e8d9ac8abd4f11e98e9f70f6259e
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/84035
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf.ctf: Delete supplementary files after tests

This way, each test has a reproducible state: without supplementary
file, otherwise some tests were using the supplementary files from
previous runs. It found bugs in other patches, but they were not related
to this.

Change-Id: I92cabd22d2193e1cd0fe43ce1f32be0e05b9fd20
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/84034
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

linux.core: replace ""; //NLS... with StringUtils.EMPTY

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

common: Fix compilation error with recent JDT

An error pops up with the latest integration build. The type
inference seems weird, but we can specify the expected type
manually.

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

tmf.swtbot: make openTrace fail on failures not just exceptions

Check TmfOpenTraceHelper#openTraceFromPath for a failure. This fails
faster.

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

lttng.xml: Add xml file for futex analysis

Change-Id: Ia9579f1db51333fed6f688a879770b1f792f3ae3
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84481
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>

common.core: make TraceCompassLog call the same path on all getters

This allows better following of Loggers.

Change-Id: Ifd4660d41e9f76730590662900a0e691b95d1d9f
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85720
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>

tmf: Deprecate and remove a few things not used in Trace type code

Some methods are never called and are achievable through other means
anyway.

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

tmf.ui.swtbot.tests: Fix javadoc warning

Warning introduced by 2d594808e2906222ad806151e020c1bd6527f9e7

Change-Id: Iee04992bfa4bf7f95324e37c39bb24f8a73a045a
Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
Reviewed-on: https://git.eclipse.org/r/85638
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>

segstore: use eclipse Performance for SegmentStore perf test

Before this patch, the segment store performance would only
print the benchmark results in the stdout after one run.
This patch uses the eclipse Performance plugin to run the
tests 10 times and provide more detailed output.

Change-Id: I98738c52f5c7f1bb81f152b38b97595056d8e60c
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/84496
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

common: annotate as @NonNull Apache StringUtils#EMPTY_STRING

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

lttng.swtbot: Add test for closing and opening histogram

Change-Id: I928e7a8f0eb6ad8fbc81d0f756079836031bce16
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84710
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

graph.ui: Add Apache Commons Lang3 to use EmptyString

This removes some ""; //NLS...

Change-Id: I3217d3d9bacc8123507ebd25c1095e3bb14a32e6
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85548
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

swtbot: Fix selection event for SWTBotSash with SWT.HORIZONTAL style

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

ss: introduce getIteratorOverQuark

This patch adds a method which returns an iterator over the
intervals of an attribute and a basic unit test.

Change-Id: I8d9d59b1cdf1a80477f24e2849c777c109afbc10
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/77546
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI

Do not report API error when adding a default method

This is a new setting in PDE that allows us to add default methods
without having an error.
See bug 501745

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

timing.core: Add more unit tests to ICalledFunctions

Change-Id: I7603482bef8b03b7326a2a74ce1f2a4e4c7e790b
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84415
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

lttng.swtbot: add test to open and re-open time graph views.

This removes 4 manual tests. ControlFlowView 1.3 and 1.4 and
ResourcesView 1.3 and 1.4.

Change-Id: I28c4b0bae9d5e6a846ac619c22e97d1df8ad47a8
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85429
Reviewed-by: Jonathan Rajotte Julien <jonathan.rajotte-julien@efficios.com>
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

lttng.swtbot: remove fViewBot from ResourcesViewTest

This allows getViewBot to work even if the view was closed
and re-opened.

Change-Id: I7b999f83dfea74785244e282748ed7975ef67d34
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85428
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

releng: replace some "" with StringUtils.EMPTY

Change-Id: Ifcd84861dec27ee515f4a216cbdb22e09f6f3e9b
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85233
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

lttng.swtbot: remove field fViewBot

This makes the test more likely to fail in a controlled way but also
removes the cache which can need to be updated.

Change-Id: Ifd80b151edc4022100d2998426466ed1780ff3d9
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85421
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

tmf: Add missing schema in build.properties

Otherwise, adding the extension without full source code throws an
exception because of the missing schema.

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

ss: Add a new plugins for data stores and tests

This plugin will contain among other things the base history tree structure

Change-Id: I5919ab01567525b4833cfafbf4c23423749b2de7
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/84257
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI

swtbot: Add SWTBotTimeGraph and update SWTBotSash

The new classes SWTBotTimeGraph and SWTBotTimeGraphEntry are added to
better support UI testing of the time graph widget.

The class SWTBotSash is updated to simulate drag using SWT events.

SWTBot tests are updated accordingly.

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

Put analysis.timing.swtbot.tests inside the relevant profile

Build fails at the moment with the -Dskip-automated-ui-tests flag.

Change-Id: I68907d142b48149651612b61bbc498394f9d7966
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/85185
Reviewed-by: Hudson CI
Reviewed-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>

tmf: Fix rounding and range issues in periodic marker source

When the requested time range and reference are far apart, the floating
point calculations lose precision and can produce marker times that are
not multiples of the period but instead clamped to the nearest multiple
of a power of 2.

The calculations are modified to adjust the reference to a point that is
closer to the request time range, by converting the period to a fraction
representation, if possible, and using exact integer calculations.

Some rounding calculations are corrected to apply to smaller numbers.

When using alternate shading periodic markers, the first marker returned
could overlap with the start time. If this partially visible marker was
selected by the user, the 'previous marker' action would not find any
previous marker. The code is adjusted to make sure that the first marker
returned is the first fully non-visible previous marker.

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

tmf: Use DecimalUnitFormat for time graph selection range less than 1s

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

tmf.ui: Remove redundant annotations in ITimeGraphPersentationProvider2

Change-Id: Idb7c863765fd7038818da9fe08ca2ac541e19a5b
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85093
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

common: Add external annotations to Guava Cache and LoadingCache

getIfPresent() should return a @Nullable V. Elsewhere it should
follow the declaration type.

Some other methods (asMap() etc.) are known to really return a
possibly empty map, so those can be marked @NonNull.

Change-Id: I5466151977ee639535b1f818ef9e3c14ba122f7e
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/85082
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI

timing.core: Make statistics analysis support different analyses

* Make the "getIntersectingElements" not blind cast into Collection
  they are Iterables
* Store the SegmentStoreProvider as a SegmentStoreProvider instead of
  an IAnalysisModule. It is only used as an AnalysisModule at one point
  in the code.

Change-Id: Ie0e81ecd69ceb4634f8e7a4236310efe02f74bea
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84544
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

tmf.xml: Fix children in time graph view

Commit 7c094b3e0710bef7c69040d2d544f4e8781eaa4a changed the behavior of
the addChild(int, entry) of TimeGraphEntry. The parent should not be set
before calling this method anymore.

Change-Id: Ie716e9a5502f445dd6ef755adce42688ed000bbd
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/85065
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

timing.core/ui/tests: add function density test

This is a unit test for the function density view.

Change-Id: I296d6f33fd4b81b99363cf3de5a280fb9ef04c68
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83794
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>

common.test: Fix Todo and warning in BBQ tests

Now that we're in Java8, we can fix the todo to support
List<@NonNull...> and also properly annotate the output
of the stream.

Change-Id: I1584164748beed859a4b2883435c537aa97aa1ff
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84633
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>

common.core: make BufferedBlockingQueue validate inputs

Also tests that inputs are validated

Change-Id: I6cc39a0851e0ccf8ada0dd473586c613387ffc68
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/47398
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>