Remove tmf.ui magic numbers

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

ctf: make CTFStreamInput no longer autoclosable

This does not hold any resources. A rule of thumb is "if and only if it ends in reader, it is autoclosable"

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

CTF: Remove autoclosable from CTFTrace

Metadata is now autoclosable so the trace doesn't have to be.

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

tracecompass: remove duplicate test case

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

lttng: Fix Resource View for traces without sched_switch events

The Resource View assumes that if the base quark of a resource is
present, the "Status" sub-attribute will always be there. This may
not be the case, for example if the trace does not have sched_switch
events enabled, but has shed_wakeup.

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

ctf: Further updates to the CTF-testsuite tests

The stress/metadata/pass directory is now populated with scripts
to generate traces, update the test case accordingly.

Also skip the new regression/stream/pass/integer-large-size test,
we fail it at the moment. Keeps the build working until we can
look into it.

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

Fix magic numbers in BTFTrace

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

ctf: Update ctf-testsuite test for the new stream stress tests

Won't run anything yet, because the test traces need to be generated
with a Python script. The temerary ones may generate them manually,
then the test suite will run them.

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

ctf: Port the ctf-testsuite test to java.nio.Files/Path

Allows using some fancy DirectoryStream filter stuff.

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

ctf: Use a ConcurrentHashMap in LexicalScope

The fChildren map in LexicalScope is normally written once, then
read over and over and over by many (possibly concurrent) threads.

For this use case, a ConcurrentHashMap is more appropriate than a
Collections.synchronizedMap(). A synchronized map will require readers
to obtain the lock, while a ConcurrentHashMap won't.

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

lttng/live: get last timestamp first

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

TMF: Synchronize an experiment instead of collection of traces

That will make it compatible with streaming since one full trace does not
have to be read before the others. It will also accelerate the time necessary
to get results, as matches will be found earlier. The overhead of reading the
experiment instead of the traces decreases performance though.

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

tmf: Bug 434646: Control Flow view not cleared when opening UST trace

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

tmf: fix IllegalArgumentException when doing auto-detection

with trace type that doesn't exist (anymore). Since it is determined
during runtime which trace types exists this should not throw
an exception.

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

tmf: add (internal) API for handling name clashes during import

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

tmf: Add missing NonNull import to tmf.ui.tests

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

tmf: Consolidate constructors in TmfEvent

The default constructor can be used by the ITmfEvent extension
point, so should remain available, but should not be used by
normal code.

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

XML state system: Add "eventName" to the current stateattribute types.

We already have the "eventName" as a type for statevalue, but sometimes we need
it as a valid stateattribute type as well, e.g., in counting the number of each
event type.

Change-Id: Ie699221d71cee1686c6865bcffc3b38a991a0731
Signed-off-by: Naser Ezzati <n.ezzati@polymtl.ca>
Reviewed-on: https://git.eclipse.org/r/36059
Tested-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>

tmf: Add some nonNull annotation to the tmf.core.analysis package

And some more of those where required.

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

ctf: Use new location for the test traces

Simply change them from .../test-traces/ to .../test-traces/ctf/
We will eventually put other trace types there (pcap for instance),
so it would make it a bit cleaner to have each in their own directory.

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

tmf: Move TmfExperiment to its own package

With many experiment-specific concepts, and more on the way, it's
making more sense to put TmfExperiment in its own package.

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

tmf: Consolidate constructors in TmfExperiment

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

Disable "warn for deprecated in deprecated code" warnings in test plugins

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

lttng: Experimental support for perf-CTF traces

Implement initial support for perf kernel traces in CTF format.
They can re-use the event layout capabilities introduced previously,
so that we can add this support without implementing a completely new
trace type and accompanying analyses.

Also rename the "LTTng Kernel" trace type and analysis to "Linux Kernel",
as they are now tracer-agnostic.

The perf-in-CTF format is not set in stone yet, so the settings used
here will most probably have to be updated in the future.

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

lttng: Fix system calls for LTTng 2.6 kernel traces

The "syscall entry" and "syscall exit" events were renamed in LTTng 2.6.
This patch adds a new trace layout, which extends the base LTTng definitions,
but updates the system call event names.

"syscall exit" also had to be changed to a prefix, because LTTng 2.6 now lists
the syscall name in exits too. We don't make use of it though, so it shouldn't
affect other tracers who don't.

Also required updating the CFV/RV to only display the non-prefix part of the
syscall name, no matter what layout is used underneat.

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

lttng: Add interface to abstract the event layout away

It would be possible to run the kernel analyses using events coming
from different tracers (or different versions of the same tracer).
This adds a IKernelAnalysisEventLayout, which specifies know concepts
in the Linux kernel, and the "support implementation" of each tracer
(or version) can extend it.

This will allow re-using the same modules and state providers, no matter
which actual tracer took the kernel trace. Also avoids having to define
separate trace types.

Also took the chance to rename some classes and packages to align them
better (i.e., the "CPU usage" and "kernel" analyses now use the same
naming scheme).

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

lttng.kernel: Rename analysis packages and classes

There are two analyses in the kernel plugin right now, "kernel" and
"cpu usage". Simply rename the packages and classes to align them
better.

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

TMF: Add possibility to set autoExpand level in time graph views

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

ctf: Remove deprecated datatypes

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

Use archive.eclipse.org for the test traces

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

lttng: Do not rebuild the SS in every single state system test

At some point we changed the state system tests to rebuild the whole
thing in the @Before, because at that point AssumeTrue() inside a
@BeforeClass did not work very well. Unfortunately this made running
the tests much longer.

Instead, really build the analysis/state system once per test class,
but only keep the AssumeTrue() in the main @Before that is ran every
test. That check will test a static boolean set by the @BeforeClass.

tl;dr Make the state system tests ~20 times faster.

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

Add some missing NonNull annotation "imports"

Now needed indirectly by some analysis modules in plugins that
didn't have it before.

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

tmf: Add element to its parent's children automatically and remove setChildren

Also, add a few useful setters.

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

Change versions to 0.1.0

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

tmf.ctf: Split the classes into proper packages

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

tmf.ctf: Move each CtfIteratorManager into its own trace object

Which allows removing the "iterator manager manager" completely.

Some extra synchronization had to be added to CTFTraceReader and
CtfIteratorManager, because we've now lost the all-synchronized
manager-manager.

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

tmf.ctf: Rename CtfIteratorManager classes

This patch simply renames existing constructs related to TMF-CTF iterators
to better represent what they do, to help future refactoring:

CtfIteratorManager -> CtfIteratorCEO .It is a "meta-manager" that manages
all the existing, now-called CtfIteratorManager's. Silly name, but don't
worry it's temporary ;)

CtfTraceManager -> CtfIteratorManager. This is the object that manages all
the iterators of one trace. It should eventually be moved to inside the trace.

Change-Id: Ie5483844856610907d3e332307eb0844209a93ef
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/35614
Tested-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

Update some feature names to include 'Trace Compass'

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

Add support for signing the jars when building the repo

Also update siteDestination to actual real path on the server.

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

XML state system: Add support of different comparison operators to the XML.

This will add 6 new comparison operators to the xml conditional statements.
These operators are: eq, ne, ge, gt, le, lt.

Change-Id: Ice9e87503eec6fea5a5d2281ffd8e42e59c8224a
Signed-off-by: Naser Ezzati <n.ezzati@polymtl.ca>
Reviewed-on: https://git.eclipse.org/r/35681
Tested-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>

tmf: Extract a method so that the manifest can be embedded in other xml files

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

tmf: Add package-info to internal.tmf.core.analysis package

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

tmf: Have ITmfTrace#getHostId return type be NonNull

This will avoid a lot of null checks in analysis

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

TMF: Set the name field of TmfComponent @NonNull

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

TMF: Do not send objects in TmfCoreTracer#traceComponent and #traceRequest

These tracing methods were sometimes called in constructors and called methods
from the object that it is not necessarily safe to use when the object is not
finished building. This patch fixes this.

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

TMF: Add support to more than one match event definition per trace

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

TMF: Remove some deprecated class from synchronization package

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

TMF: Improve the serialization of trace synchronization

Made some fields transient, so they are not serialized and initialize the
transient fields in the readObject method. Also added serialization unit tests.

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

TMF: Have ITmfEvent#getTrace() return NonNull

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

TMF: Update .options files to the tracecompass name

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

lttng: Fix a bug with lttng version when compiled from source

The output of the lttng version command is something like this:

lttng version heads/stable-2.5-0-g3910e60 - Fumisterie

The current code didn't recognize this as version.

Change-Id: Id480402ba6de0803e88d4561d1604ad464f80926
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/35495
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

Tmf: Fix a bug in CFV and Xml time graph views where entries have no parent

If a child entry starts before the parent entry, like it happens at the
beginning of a trace, then in time graph views, they are not associated with
the parent.

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

tmf: Fix legend image leak in Histogram view

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

lttng: add some stress test for opening and closing of traces

This will trigger concurrent starting and cancelling of analyses.
After fixing bug 447434 (commit e1c415) this test should run without
exceptions.

Change-Id: I9c7a35d29e0c4a406511ec1e8032b3d818498b94
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/35508
Tested-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

State system: Make ITmfStateValue Comparable

It is required to compare two state values based on different comparison
operators. It will be useful for the XML analysis.

Change-Id: I1fcb9525a41b11a648922271b5c2e06d6bd309ac
Signed-off-by: Naser Ezzati <n.ezzati@polymtl.ca>
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/35414
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Hudson CI

Update top-level README file

The readme still had some references to linuxtools.

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

Update maven profile names

"tracingRcp" was used back in Linux Tools to differentiate between
the different RCPs in the tree. Now we can simply call the profile
"build rcp".

There doesn't seem to be any guidelines for profile names... but I
find names-with-dashes more readable than camelCaseNames.

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

tmf: Bump checkpoint index version due to incompatible timestamps

TmfTimestamp serialization was incompatible due to removal of precision.

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

ss: Cleanup TransientState a bit

- Remove assertions (see bug 409346)
- Add more null annotations
- Update comments

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

ss: Also search in the ThreadedHistoryBackend's interval queue

If a view, like a live-enabled time graph view, makes a lot of queries
close to the "current end time" while the history is being built, it
is very possible to miss some intervals due to them being in the
ThreadedHistoryBackend's interval queue. We should also search in that
queue for possible intervals.

Since this case is still relatively rare, it's usually faster to query
the history tree first, then go look through the queue, rather than
checking in the queue first every time (which can only be explored
in O(n)).

Fixes bug #443127.

Change-Id: I82e389cf95ce04002f61629d5a0d317a35a931ff
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/35504
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>

tmf: Allow reordering columns in event table

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

tmf: Add unit tests for time offset

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

tmf: Update user guide for time offsetting

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

tmf: Change Offset Dialog advanced mode layout

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

tmf: Remove the precision concept from ITmfTimestamp

Timestamp precision is a problem in tracers, not in a viewer.
We already have the notion of scale which kind-of does the same
thing (if a timestamp is specified up to milliseconds, we can
consider it precise up to milliseconds).

Not only this removes big chunks of unused code, it should speed
up timestamp comparisons a little bit.

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

ss: Remove checkValidTime from the backend interface

This is merely an internal method that implementations *should*
normally have, but it is not meant to be used externally.

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

tmf: Support for state-system based views in live mode

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

lttng: Support for existing live sessions

With this change, it is now possible to create a live session from the
command line and connect to it using the Control view as a live session.
We use the live timer interval as an indicator of whether or not a
session is live. The information about the host and the port is not
available with 'lttng list' but only local host is supported for now
anyway (127.0.0.1). For the port, we try use the default (5344). For
now, if the default port is incorrect, an error message "The connection
could not be established" is shown. As a future enhancement, there
should be a dialog to configure the relayd connection independently from
the session.

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

tmf: Remove AutoCloseable from IAnalysisModule

While it was very useful for tests (every test manages its own module with
try-with-resources), in reality analysis modules are stored in the trace
but need to be exposed to external components. And the trace has no knowledge
of which types of modules it contains, so it cannot provide access methods. For
this reason it was more annoying than useful to have them AutoCloseable.

Change-Id: I62a1c588155c59a6a0047a52e2965e1f3acd5744
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/35425
Tested-by: Hudson CI

tmf: Use the DebugSuite test runner for some tests

We've had the DebugSuite for a while, which allows printing the name
of every test class being started. This is extremely useful for
debugging hanging tests when run with Maven.

We could now use this by default, so that the Hudson builds also print
that output. It will help pinpointing hanging tests there too.

The class is in tmf.core.tests, so for now only enable it for plugins
that depend on tmf.core.tests already. This is a good start, since so far
most hanging tests were in tmf.core.tests. We should move the suite to
the "common" package once we get around to creating that...

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

tmf: enable SWTbot test for collapsing of repetitive events

Change-Id: I4bb26c59641964581ba6f4b3a2f7ac42a5d23826
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/35509
Tested-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

ss: Add a StateSystemUtils for advanced queries

We want to offer convenience and advance query methods for the
state system, but these should not be part of the ITmfStateSystem
interface. That one should only be for the "basic" types. Everything
else should go in a separate utility class.

Change-Id: I62936e77454112c67f6861a1f0ca7a60c9aef2d1
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/35341

tmf: Fix NLS bundle for the time offset dialog

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

tmf: Axe the TmfEventTypeManager

Track the available event types in each trace object instead. K.I.S.S.

Also removed the notion of "context" from TmfEventType, which is not
needed anymore.

Change-Id: I8dc1409abb08fd930dd1f8b01027ece488551ec1
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/35354
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>

pcap: Fix a test so that it works in any time zone

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

tmf: Make TmfTimePreferences completely static

Marking the class final shows us that all the methods can be marked
as static, since they do not use any internal fields. This means
the getInstance() itself is unneeded.

Change-Id: Iaed5876fd3f6151fceb90991d5deb452c6e70ef3
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/35334
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>

pcap: Fix tests always being skipped

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

Update some class paths in plugin.xml files

Probably due to some auto-resolved merge conflicts. Strange that they
didn't show up earlier.

Change-Id: I997f9182b288bd15ceac75204cbb7c9c9aede7ae
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/35333
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>

lttng: Mark a field private in the SWTBot test

This fixes a "missing Javadoc" warning.

Change-Id: Icfc5c1272453c9b246a9f7512b1d2639d4b3f09d
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/35332
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>

rcp: update branding plug-in for correct application reference

Change-Id: I71758aeb92c48297df898dfb20c3001a39f1e20e
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/35401
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: fix concurrency issue in analyses modules (bug 447434)

The access of member fTrace in class TmfAbstractAnalysesModule was
not thread-safe. The attribute was set to null when cancelling
the analyses (e.g. in the UI thread).

With this patch this attribute will be accessed (read and write)
using a synchronization lock and it will be checked for null
references before using it.

Additionally, this patch takes care of count down initialization
latches in case of cancellation.

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

ctf: Commit the generated ANTLR files to the source tree

This means users who just pulled the git tree won't need to run
a Maven build manually to generate them. Which in turn, completely
removes the need for installing m2e or such in their Eclipse.

They are now only built when the "ctf-grammar" profile is enabled.
Rebuilding always produces a diff: the first line of the generated
files contains a timestamp, and there is some randomness in the
generated comments. So it's better to only do it on-demand.

Also added a copyright header to these files, since they didn't
seem to have one.

Change-Id: I0fcf4cb32f111c7eecde3ab641205fb1d526fb28
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/35269

Fix cast warnings (errors) when targetting 4.5

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

Change linuxtools to tracecompass in generate-grammar script

Change-Id: Ida3bec0759fafeb88d7cae73ed3f2353535d0e90
Signed-off-by: Simon Marchi <simon.marchi@polymtl.ca>
Reviewed-on: https://git.eclipse.org/r/35352
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Remove the API Tooling builders and natures

Disable the API tooling, at least until we have an actual release
and can start tracking APIs again.

Change-Id: Ia0ac03ef4b2b181fa31795242c96ca400fbdef5f
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

tmf: Fix sequence diagram unit tests

The extension point location in the test facility was using
Activator.PLUGIN_ID, which was renamed to "org.eclipse.tracecompass",
but we left the extension point IDs to "org.eclipse.linuxtools" for
now. Fix the references accordingly and re-enable the tests.

Change-Id: I40e2d4d7021f55043f5f51b2472a07bec01eb5d5
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Rename remaining packages to org.eclipse.tracecompass.*

that being the packages in the "alltests" and "examples" plugins.

Change-Id: Ia9d28cefaed3d2974ba67e6902eb04763ba75413
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

btf: Rename packages to org.eclipse.tracecompass.*

Change-Id: I56194cc3c13c57c315aedd300efaf49bf0e57058
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

ctf: Rename packages to org.eclipse.tracecompass.*

Change-Id: Ia069f48869616df6661c12b616be124333758e1d
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

gdbtrace: Rename packages to org.eclipse.tracecompass.*

Change-Id: Iee74553bdb6ab5a5d1b50405f749d229575eea2c
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

lttng: Rename packages to org.eclipse.tracecompass.*

Change-Id: If67346f1e38c173ff74379e18738e606864d287f
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

pcap: Rename packages to org.eclipse.tracecompass.*

Change-Id: I8c1497ea665bcf7904408deec3fd7071e76be942
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

rcp: Rename plugins to org.eclipse.tracecompass

Change-Id: I29f43aa20166dd5a6af4cd5660eb0b430fd3a9c3
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

ss: Rename packages to org.eclipse.tracecompass.*

Change-Id: I2fcf70501c26510d0215852c99025c801051a715
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

tmf: Rename packages to org.eclipse.tracecompass.*

Change-Id: I7072126d88e93cf44b0a1169c5090edb626b73c5
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Move alltests plugin to the Trace Compass namespace

Change-Id: I9ab5a7ee50bae2b9871ba3f4cd9e5a4d7eafb78b
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Move misc plugins to Trace Compass namespace

"examples", "lttng.help" and "lttng.testing" plugins

Change-Id: I20b5bb985988444e7f8e1c72117d9f3cf36e4997
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Move RCP plugins to the Trace Compass namespace

Change-Id: I99b76ca059ea50ecb341b9928f5b4e992e9004e6
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

tmf: Move plugins to the Trace Compass namespace

We'll keep the name "TMF" for now to refer to the core framework.

Change-Id: Ifb221f2c8239feb0eb0e2e308cc24e094c6501e1
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

ss: Move plugins to Trace Compass namespace

Change-Id: I6cfee9874e9893d13374f05b139e73a6c8478fd7
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>