releng: Fix Mars build by adding missing gson dependency

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

lami: synchronize getter for currentReport

Fixes CID 112029

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

lami: getSymbolColors can return null

Fixes CID 112047

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

tmf: annotate TmfContext#location as nullable

This fixes a potential race condition if the context is changed
after a null check but before a dereference. In the current
implementation, it is extremely unlikely to happen, but this
may save later investigation time.

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

timing.ui: add Export to TSV to tables and statistics

This allows exporting to tsv files.

Testing is done with reflection to avoid native dialog issues.

Some issues are present:
* In Linux, an invisible column is added at the end,
while this seems ok, it makes the output inconsistent
between Windows and Linux.
* SWTBot cannot test native windows, therefore, the user actual
actions are untested. All possible logic has been moved
elsewhere to accomodate.
* Tree and Table have no common parents, this will incur code
duplication due to their similarities.
* In Linux, tooltips are not well handled in menu items.

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

Fix importing an archive containing colons (:) in the names on Windows

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

timing: Fix duration of flamegraph view

Instead of having a duration of the full length of the trace, it has the
duration of the longest element.

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

timing.ui: Make doubleclicking on a flame graph event zoom on it

This aims to bring feature parity between trace compass flame graphs
and the original flame graphs designed by Brendan Gregg.

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

linux.core: add Tux Icon to linux core analysis

According to the site : http://isc.tamu.edu/~lewing/linux/

Here are the results of an idea acquired from discussions
on the linux-kernel mailing list, and an initial suggestion
by Alan Cox.

Feel free to do whatever you see fit with the images, you
are encouraged to integrate them into other designs that
fit your need. Comments suggestions are also welcome, so
please tell me what you think of these. I suggest that you
look at some of the other images available with integrated
text.

The backgrounds of these images are random colors (if your
viewer doesn't support transparent gifs). This is because
I want to be able to keep the outline clean (except when
blending into a scene or title bar). Each in-line image is
now also a link to the corresponding gif so that they are
more easily retrieved. The images I actually work from are
tifs which I'll make available if there is interest.

Neal Tucker was kind enough to provide a scalable vector
based postscript version of the black and white penguin.

Permission to use and/or modify this image is granted provided
you acknowledge me lewing@isc.tamu.edu and The GIMP if someone
asks.

This patch will still require a CQ, but it looks safe to push
to gerrit.

CQ: https://dev.eclipse.org/ipzilla/show_bug.cgi?id=12011

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

releng: Update targets with new Neon.1 update sites

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

tmf: bug 494698 Add per-event fields to custom parsers

Change-Id: I8e2d302ea53319a42f2d84828db41726af321b05
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/76907
Reviewed-by: Hudson CI

Fix javadoc warning

Change-Id: I94af2b7f6e7d6612b54e150fc17ce481be8e92e4
Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
Reviewed-on: https://git.eclipse.org/r/81921
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

lami: set the min value when clamping to zero.

Fixes CID 141171

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

releng: Add missing source for tracecompass.common.core

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

Remove unneeded API filters

Change-Id: I9d296b3e2ad74e468091b273b565136b0421ebab
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/81878
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: Add 2.1.0 baseline and fix API errors

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

Change a warning to an error in build.properties

A missing java source folder entry in those files result in build
failures if another plugin needs to use a class from this source
folder. So they should be reported as errors. We always want to put
all code in the jars anyway.

Also fix a missing one.

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

lttng: Rename LttngUiResponseTest to benchmark (it's not a test)

Otherwise, it runs during tests and closes all the views.

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

tmf: Allow any object to use TmfSignalThrottler

Some classes override broadcast(), so we can continue
calling that for subclasses of ITmfComponent.

Change-Id: I2b270fe58349e6206d00c874b84304581c56dce0
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/81723
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

analysis: let the soft irq raised state appear in the Resources view

When a soft irq was raised during an irq the view was swhowing nothing between
the end of the irq and the beginning of the soft irq for a cpu entry.

Change-Id: I22d5e3014efc531c7cf3ee9c48ac59af3a772a2e
Signed-off-by: Cédric Biancheri <cdc.biancheri@gmail.com>
Reviewed-on: https://git.eclipse.org/r/72230
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>

timing.core: speed up (Lazy)ArrayList queries with binary search

The getIntersectingElements queries on the (Lazy)ArrayList
SegmentStore would iterate through the entire store with a
stream().filter(), whereas the store is sorted by start time,
then by end time of the segments.

By using a binary search on the store, we can narrow down the
iteration to the relevant sublist thus speeding queries up to 50%
on querying from real analyses.

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

xml: add icon for analyses

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

linux.core: add Memory Usage Icon

This is based on the trope that cpus are LQFP, memory is DIP.

Change-Id: Ia6204ab868f549fbf32a490b526cb856d4aa2c36
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/81652
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: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>

linux.core: add IO Analysis icons

Made fore High-DPI screens.
Inspired by Hitachi Deskstar 80 gb drives. But
every rotary hard drive looks like this.

Change-Id: I3b6f1a18d950590e3a2e3c39267da1ff81598bab
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/81651
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: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>

linux.core: add CPU usage icons

Made with scaling in mind.
Inspired by resource monitor in many oses.

Change-Id: I5bb5f1b063ba0452fa476c06eeae6c6689b781b9
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/81650
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: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>

graph.core: add Critical Path Icon

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

linux.core: add Context Switch icon

This one was done with high-dpi screens in mind.
Inspiration comes from a real-estate based board game.

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

tmf.core: add statistics icon

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

ust: add icon for debug info analysis

Change-Id: Idf4a64f5650b4191cca65f3b1e55a6e8bb7fca9b
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/81632
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: Document Attribute Tree for UstMemoryStateProvider

Change-Id: I35594e499ddcdd7d2e4f4969ce83c76320a85600
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/81199
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Hudson CI

os.linux: Document Attribute Tree for StateProviders

Add documentation to headers for
KernelContextSwitch
KernelCpuUsage
KernelMemory
ActiveTid
InputOutput
Kernel

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

tmf: Replace link icon with png and add 2x version

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

tmf.xml: Remove useless parameters in TmfXmlPatternSegmentBuilder

Change-Id: I1814dfa8fb21b10ee0a6350dcf38ae6f7bb3d519
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/77168
Reviewed-by: Hudson CI
Reviewed-by: Bruno Roy <brusi_roy@hotmail.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

linux.core: Update Syscall analysis to use time#toNanos and event#getName

Using ITmfTimestamp#toNanos ensure that system calls will be in the same
units if mixed OS traces are used. Using event#getName is faster and simpler
to read than event#getType()#getName.

Change-Id: I70819a28dfdaa257fa6c484c5073f77610605427
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/80967
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: Update next input in custom parser wizard when removing input

When removing an input that has a previous input child to the same
parent input, the previous input's next input reference should be
updated to the next input of the removed input. Otherwise the removed
input may remain as a ghost next input and interfere with the previous
input's child inputs. This is only a problem when testing the parsing of
sample trace text from within the custom parser wizard, since the next
input relation is not stored in the XML definition, it is always rebuilt
from the parent-child relations when loading the definition from file
into memory.

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

lami: possible ressource leak on IntStream

Even if this is a fluke from Coverity using a for loop is simpler.

CID 141163 & 141165

Change-Id: I6087b22cd9ad9beb47f5546d54fd5167bdd66390
Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
Reviewed-on: https://git.eclipse.org/r/81638
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

analysis: Add missing icon for Critical Path view

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

os.linux: Create PPID and SYSTEM_CALL attributes on demand

This patch ensures that threads' PPID and SYSTEM_CALL attributes
are only created when the state is changed instead of on a
sched_switch. The methods which require said attributes are adapted
to handle correctly the absence of said attributes.
This results in a state system with 0 - 40% less attributes, thus
reducing the legacy SHT's space on disk by the same size.

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

releng: Update staging target to Oxygen

Start using Oxygen update sites for projects that have one (notably
the platform).

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

lttng.ust: Consider all statedump events to happen at SD start

Use the statedump:start event's timestamp as the start time of
library load mappings, instead of the individual bin_info
or related build_id/debug_link events.

This will allow trace events happening concurrently with a
statedump to display their debug-information correctly.

Bug: 497943

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

ss: Remove infinite loop in debugPrint() code

In practice extensions are not used, so the value would always
be "-1" and it would never go inside the loop. But if we had
started using extensions it would have infinite looped rather
quickly.

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

ss: Check return value of FileChannel.read() call

Fixes a Coverity warning.

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

releng: add missing plug-ins to testing feature

The dependency to o.e.tc.analysis.os.linux.ui.swtbot.test has been
recently added to the plug-in o.e.tc.lttng2.kernel.ui.swtbot.test.

Downstream projects that depend on o.e.tc.lttng2.kernel.ui.swtbot.test
in their test plug-ins won't build anymore.

The plug-in o.e.tc.analysis.os.linux.ui.swtbot.test needs to be added
to the feature.xml of o.e.tc.testing.

While being at it, add also all other missing test plug-ins to it.

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

Print GTK theme at the start of SWTBot

Since behavior can change greatly depending on the theme, this
might help us investigate failing tests in some cases.

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

ss: Have IHistoryTree#selectNextChild return a Collection instead

This is more generic for multiple implementations of IHistoryTree.

From running the state system benchmarks, there was no performance impact
with this patch by returning a Collection instead of a single element.

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

ss: Move all debugPrint() methods to the test stubs

Where they belong!

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

os.linux: remove unnecessary calls to getQuarkRelative

When the Status attribute was removed, some calls to
getAttributeQuarkRelative were not removed and that causes exceptions in
some cases.

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

tmf.ui: Add extra javadoc to the AbstractTimeGraphView

This adds some documentation to the AbstractTimeGraphView#buildEntryList
method.

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

ss: Extract an history tree interface

This is a second step towards supporting multiple types of SHTs

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

linux.core: Add icon for system call analysis

Add an icon drawn by hand in pinta. It's supposed to look like
a phone.

Change-Id: Icb99aa67015e84ff8ccc65bf4076a1a81a5be9bd
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/81269
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.ui: move logic of segment stats logic to abstract class

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

timing.core: add stream unit tests to segment store tests

Now test stream building segment statistics.

Also improve speed of other tests marginally by pre-allocating
arrays to right size.

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

KernelAnalysis: Use CPUs CoreAttributes to store "Status"

CPUs core attributes were not used to store interval data, this
leads to useless NullStateValue intervals ranging from tree start
to end. This patch uses the CPUs CoreNodes to store the "Status"
fields thus reducing the number of attributes and intervals.

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

KernelAnalysis: Use Threads CoreAttributes to store "Status"

Threads core attributes were not used to store interval data, this
leads to useless NullStateValue intervals ranging from tree start
to end. This patch uses the Threads CoreNodes to store the "Status"
fields thus reducing the number of attributes and intervals.

Change-Id: Ia24b8d5226b66738a8c14d3d652879169437b95a
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/70769
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>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>

tmf: Fix hang in TarInputStream.skipToEntry when importing large tar.gz

When importing a large tar.gz archive, it can hang in
TarInputStream.skipToEntry. Depending on the size of the entries in the
Tar and the order in which they are read, it is possible that bytesread
will overflow and become negative. Because of that, the code that skips
bytes fails to skip properly and enters in an infinite loop.

This patch fixes the issue by making the bytesread a 'long'. Because
this class is in the platform, several files had to be copied over in
order to get the fix right away. Those files will be removed once the
fix is in the platform and that the fix is released.

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

linux.ui: make Control flow view optimization work with mixed experiments

bug 501148

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

tmf: Add the view ID to some JUL log statements of time graph views

This patch adds a getViewId() method to AbstractTimeGraphView that returns the
view's primaryID:secondaryID string corresponding to the view. This view ID is
added to some JUL log statements.

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

linux.core: Remove throws runtime exceptions

AttributeNotFoundException is now a runtime exception. In practice, it
should never happen here as the quarks are always added before accessing
them.

This removes a few warnings.

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

timing.core.test: add perf to build.properties

This removes a warning

Change-Id: I68fbafc87712aefd29b80bf83088077ecaf8621e
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/80696
Reviewed-by: Hudson CI

tmf/ust: deprecate AbstractCallStackAnalysis and move the UST module to core

The only reason the CallStackAnalysisModule were in the UI was to register the
view output to it. A listener now does that part so the class in the tmf.ui
plugin can be deprecated and LttngUstCallStackAnalysis can also be moved to
the .core plugin and be made internal while at it.

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

linux/lttng: Add a swtbot test for UI responsiveness of kernel views

This adds abstract swtbot classes to open a trace, then navigate it
(full range, zooms, scroll, zoom back out) while some views are 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 the so-obtained traces.

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

tmf: Add a package-info file to the callstack package

Change-Id: Ife9bf3bd46401d9d89dd18b44611bba9e39938c9
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/80299
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

ust.core: add NON-NLS to toString of UstDebugInfoBinaryFile

This removes 4 warnings.

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

linux.ui: add kernel.swtbot as a friend

lttng.Kernel has the LTTng trace support needed to test these views.
That is why their unit tests are in the other package. Adding friend
allows access to the internal IDs instead of hard-coding them.

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

timing.ui: make Aggregated functions merge statistics

When calling merge, the statistics are now merged, not just
the max and min time.

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

timing.core: add merge function to segment store

This allows merging statistics node. This will make statistics
work in a more streamlined way.

Note: merging introduces a slight error to standard deviation,
This is due to the pooled variance algorithm used.

Potential use-cases for this:
* Map-reduce statistics on a segment store.
* Merging trees of statistics.

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

timing.core: add segment store benchmarks

this tests:
* filling
* filling a slightly random set
* filling a random set
* iterating

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

tmf: bug 496031 Get the synchronization formula from the original trace

The synchronize traces handler should get the formula from the original
trace's host ID, not the new one, since the host ID falls back to the trace
name and the new name is different than the original one.

Change-Id: Id4af62b64df4f0b6d042a19e12c04b7e3c060176
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/75179
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

doc: Document new previous/next event action in CFV

Document the "Go to next/previous event of the selected thread"
actions, and how they differ from the "Next/Previous State Change"
ones.

Change-Id: I27c1431933e1ac1e8ae9fe94090ab13e82b25a14
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/80605
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI

doc: update screenshot for system call latency statistics

to show the Total column

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

SegStore: fix iterator caching in segment store.

Fix bug 500607

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

lami.chart: use formatter on x axis for continuous data type

Swt bar chart use categories instead of tick simply format categories
label while constructing the map.

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

doc: update doc with OS Overview Perspective.

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

tmf: SWTBot integration tests for Project Explorer trace folders

This covers 20 test scenarios from the test specification dealing
with trace folders, mostly dealing with importing traces. More test
will be added later to cover completely this section of the test
specification.

The change extracts some common code with existing tests.
Add more tests are added, we could remove some tests from the
non-integration test suite because there will be too much overlap.

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

doc: Add developer documentation for JUL logging

This explains the guidelines to add logging statements to Trace Compass

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

tmf: Implement setFocus() in TmfStateSystemExplorer

Give focus to the viewer when clicking on the view tab.

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

tmf.ui: Callstack the entry selection done only on the first match.

Fixes issue which caused many requests to be queued before
the sync operation is completed.

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

doc: add and update documentation for latency views

This includes function density view, syscall views and latency views

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

releng: Update build-helper-maven-plugin to lastest version

Just keeping up with the latest.

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

timing: Bug 500592: fix symbols for with debug info of Lttng 2.8

The process ID of the trace application is propagated so that it can
be used for the Function Duration Density view and Flame Graph view.

For the Flame Graph it uses the the timestamp of the maximum segment to
get the symbol name from the LTTng UST symbol provider.

For the Function density view it takes the start time of the segment.

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

Remove extra item.select() calls

Those were necessary because of bug 498373 in SWTBot but it is
now fixed.

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

lttng.ust: Do not block in UstDebugInfoAnalysisModule

Query methods in the module should not call waitForCompletion()
on the analysis, but should return whatever data they have
(or null).

This is because this is called by the aspects, which are called
by the event table, which are called by the UI thread! The UI
should not block while the analysis is waiting construction.

If a specific caller wants to wait for the full data, they can
call waitForCompletion() manually, like the tests do.

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

Make 'missing export' a warning in test plugins, add missing ones

In general, plugins export all their packages. For test plugins,
in can be useful to reuse existing tests to create test suite.
By exporting all packages, it makes this use case easier and it's
also more consistent.

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

tmf: Use SWTBotUtils.selectTracesFolder to simplify in a few places

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

doc: Mention that all LTTng-Analyses are now supported

Trace Compass should now support the current latest
LTTng-Analyses scripts, so no need for users to remain
on the 0.4 branch.

While at it renamed the section titles to use present
progressive (Running, etc.) to match the rest of the
documentation.

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

lttng.ust: Do not skip unknown lines in addr2line output

addr2line will use "??" to indicate unknown function names or
source files. In these cases we should still report the other
information that may be available, instead of skipping the whole
callsite.

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

timing.core: make CallGraphAnalysis use LazyArrayListStore.

The callgraph analysis receive out of order segments when multi-threaded.
This accelerates the callgraph creation by an order of magnitude in that
case.

Bug 500591

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

timing: Introduce new segment store for slightly out of order datasets

The LazyArrayListStore is like an array list store but will only sort
when a read operation is received.

This structures are faster for handling segments that are out
of order.

Bug 500591

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

doc: add toolbar documentation for flame graph view

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

tmf.xml: Bug 500195. Modify initial state behavior

Fix Bug 500195 and add a new initialState element. This element acts as
a real state and support conditions and actions. Also, these conditions
are only verified  for the pending scenario and the pending scenario do
will ignore the preconditions.

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

tmf: Make order of files processing more natural during import

This changes the order of elements imported to be files first then
the folders, then by lexical order.

This makes it more predictable to the user when there can be name
clashes.

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

tmf: Bump target sequence numbers to pick up SWTBot update

SWTBotRootMenu.menuItems only exists in recent snapshot builds.
Bump the sequence numbers to that targets get reloaded automatically
when people pull the new code.

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

doc: Add view descriptions to introduction

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

doc : Add user guide for the Flame graph view

This shows what a cpu flame graph is, how to use the view and
an example workflow.

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

tmf: Add a way to clear the traces using SWTBot only

There is way to clear traces but it deletes trace elements
directly using TMF APIs. It would be good to have a way to
do it completely in the UI using SWT. This patch updates
ProjectExplorerTraceActionsTest to use the new way so that
it tests the UI more fully.

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

Move ProjectExplorerTraceActionsTest to integration tests

This class was written as an integration test. Now it has a proper
home. It also serves as a first example how TestDirectoryStructureUtil
can be used.

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

analysis.lami: Support LAMI 1.0 features (MI version, compatibility test)

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

lami: Bug 499953: invoke scripts always with end time >= start time

Change-Id: I6deea491880e8b02209ece2d7c2e87fce4671b6f
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/79434
Reviewed-by: Hudson CI
Reviewed-by: Jonathan Rajotte Julien <jonathan.rajotte-julien@efficios.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

analysis: make call graph analysis non automatic

Having automatic it will be executed for every trace since it is
available for each trace type at the moment. Moreover, since it
is dependent on the call stack analysis the call call stack
analysis is also triggered.

With this patch it is only done when one of the applicable call
graph views, e.g. Flame Graph view, is opened.

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

Add file util to create dummy structures

This will help with section 3 of the tests for the project view

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