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

Add an integration tests plugin

This plugin will contain tests that possibly require many "leaf" dependencies
and that are not unit tests but instead fuller scenarios. For example,
it might test importing UST and Kernel traces together in one operation
and test for name clashes. Therefore it is not a unit test for UST or
Kernel.

The plugin is not added in the parent pom.xml for now as it would fail
because it does not have a test yet.

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

ctf: Make LttngKernelTraceGenerator also generate UST test traces

LttngKernelTraceGenerator can currently only generate Kernel traces
but with a few changes, it can generate a UST trace. The events do
not make much sense for a UST trace but this is good enough for trace
type validation, etc.

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

flamegraph: fix invalid window range

When loading a flamegraph we need to reset the start
and finish time so that the time range is fixed and the
whole flame graph range is displayed.

Otherwise, we end up showing a range of 1 ns after
zooming once and switching the trace.

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

tmf: Fix integer overflow in time graph calculations

An x-coordinate very far to the right could overflow to be very far to
the left. This could prevent states and/or their labels to be drawn when
zoomed-in very close.

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

lttng.ust: Remove some unused messages

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

tmf.xml: Add mapping group to XML description

The mapping groups act as maps. They are used to avoid large if-else
condition in the XML. For example, if the relevant event for an FSM have
an integer field 'mode' and this value could be map to string. We could
save the 'mode' value in the state system and add a mapping group in the
XML to be able to directly use the string instead of the 'mode' value
when creating segments.

example of mode could be ;
* 1 - read only
* 2 - read - write
* 3 - write - only
* ....

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

tmf.xml: Add support for consuming event in the fsm

This patch adds the possibility for an fsm to be consuming. A consuming
fsm can use an ongoing event for only one of its scenario.

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

common: Add saturated arithmetic for int values

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

releng: Bump micro versions of modified plugins

This change bumps the micro versions of plugins that had their code
modified or behavior changed.

The minor version of some plugins was also bumped because they were
deemed to have new features:

analysis/org.eclipse.tracecompass.analysis.os.linux.ui
lttng/org.eclipse.tracecompass.lttng2.control.core
lttng/org.eclipse.tracecompass.lttng2.control.ui
lttng/org.eclipse.tracecompass.lttng2.kernel.core
tmf/org.eclipse.tracecompass.tmf.analysis.xml.core
tmf/org.eclipse.tracecompass.tmf.analysis.xml.ui

This commit also adds a script that guides through the process of
bumping the micro versions.

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

tmf: Fix lingering zoom range markers on mouse release

When zooming with the right mouse button, when the button is released
the drag state should be reset before the change of window range redraws
so that the zoom drag selection lines and background are not drawn one
last time.

The redraw used to trigger the paint event after a short delay (in the
platform) at which time the drag state had been reset, but a recent
change in the time graph viewer has forced the paint event to happen
immediately by a call to Control.update().

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

releng: Move scripts to a subfolder

As scripts accumulate, it's better to have them under a folder.

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

lttng.control: Disable Java and Python logging for older LTTng versions

Only enable the Java logging for LTTng version 2.6 and above and 2.7 and
above for the Python logging.

Change-Id: Iafe8a36141238590ccfe2a1cb854841c9f42e1ee
Signed-off-by: Bruno Roy <bruno.roy@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/79365
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Hudson CI

lttng.control: Change the short name of the LogLevelType.ALL to "All"

Change the short name of the LogLevelType.ALL to "All" instead of an
empty string. This is because when all loglevel (LogLevelType.ALL) are
enabled for a Python logger, the loglevel is set to debug but we still
want to show the user "All".

Change-Id: I18e5800bc3261c11a74e124e76517ad316e2c4a3
Signed-off-by: Bruno Roy <bruno.roy@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/79215
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

tmf.xml: Move the ImmutableMap of action in the constructor

This patch moves the ImmutableMap of action in the constructor in the
event handler.
This change prevents a copy of the maps of test and action each time
they are needed. The maps are build in the constructor and are
unmodifiable.

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

tmf.xml: Add support for peek() stack operation

This implements the peek operation for a stack of state value. On a
condition, the peek operation read the value at the top without
removing. On a state change, the peek operation will do nothing. A test
for a stack state value is provided.

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

tmf.xml: replace exception with log error message in TmfXmlStateValue

This patch removes an exception that blocks the execution of Trace
Compass when hit. The exception is replaced by a log error message.

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

timing: support sorting by thread name or ID in Flame Graphs

This will add 2 buttons for switching the sort order between
thread name and thread ID.

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

timing: Add a threadId field for the flame graph

This creates a class for a thread that extends the
AggregatedCalledFunction class and having a field for the thread id.

This adds a new field for the thread id in the FlamegraphDepthEntry.

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

tmf: Remove usage of ICoreRunnable for time graph build job

ICoreRunnable is only in Eclipse 4.6. We can create the job the old
way to restore compatibility with Eclipse 4.5 and there is no loss
of functionality.

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

tmf: add minimum margin for right mouse drag zoom

This will prevent zooming when the right drag is less than 5 pixels.
For extensions that define a context sensitive menu for time events,
this will prevent zooming when the user accidentally moves the mouse
when wanting to get the context sensitive menu.

Change-Id: I198aaac1212f2f762e1dbe87d3fa3cb115a5d3a5
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/79370
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

tmf: fix right mouse click selection for context sensitive menu

This will allow context-sensitive menus to be created using the
selection provider when selecting events with right mouse click.

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

flamegraph: Add go to min/max context menus in UI

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

lttng.control: Fix: Allowing loglevel when enabling all loggers

The user can now select a loglevel when enabling all loggers. Before
this, if the user selected "All" loggers and a loglevel an error message
would pop up.

Change-Id: I78c81202a676b05dc63fd7e95095f52180081c8c
Signed-off-by: Bruno Roy <bruno.roy@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/79436
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
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: Bernd Hufmann <bernd.hufmann@ericsson.com>