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

segmentstore: fix incorrect iteration order in segment history

The initial values for minEnd and maxStart times would lead to
incorrect orders during tree build.
Also, the bounds for the node are now serialized.
Add a test to ensure the order is right.

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

tmf: Bug 516848: Open As Experiment doesn't set experiment type

The experiment type was only set if the experiment already existed (and
the experiment type had changed). It should always be set if it is a
newly created experiment.

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

timing.ui: Have SegmentStoreScatterGraphViewer use iterables

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

lttng.core: add Address handling to BinaryAspect

The address is identical to the context_.ip. This is a good fallback
if the user forgets to add the context.

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

tmf: Show all trace analysis and views under the experiment

This patch introduces an aggregate analysis element in the project
explorer that aggregates all common trace analysis elements for each
contained trace in the experiment

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

ss: remove deprecated code

Change-Id: Ic231c473db3720f3052ec72b8db077af63d28357
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/96686
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

lttng2.ust: remove deprecated code

Change-Id: I7f8e1e9e502a67cb77873e578c96d0c6d0e8e5c4
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/96685
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

analysis.timing: remove deprecated code

Change-Id: I82ed7944fba56386aa317f349da00ca3cdaf634c
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/96684
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

tmf: remove deprecated methods from tmf

Change-Id: Ie45fc1ab795e8226a09486ef1f6e95b2642a92e2
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/95843
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

ctf: Fix CtfTmfTrace.readEnd() to use createTimestamp()

The method should use createTimestamp(long) to create the timestamp, so
that any timestamp transformation configured for the trace is applied.

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

xml.ui: Fix pattern statistics viewer not refreshing data

The pattern statistics view was not refreshing the data when switching
the view for an other analysis of the same trace.

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

xml.ui: Add label prefix for XML Latency Views

This patch adds an element to the <header> of the xml analysis
description The new element is a prefix for the XML analyses latency
views. The new name of the latency table will be like "PREFIX_LABEL
Latency Table".
This could be easily extendable to all type of xml view.

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

timing: add statistics analysis to functions calls

This, with the flame graph, call stack, call graph and function densisty
can make a profiling perspective.

Additional tests coming in a separate patch.

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

releng: fix name in 4.7 target

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

releng: move master to build with Oxygen dependencies.

Right now it is a copy of the staging target.

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

critical path: Change relationship with graph builders

This adds the critical path module as a child of the graph builder
module. It also creates the view output under the graph builder and
the view will open with the right execution graph selected.

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

tmf: Make the TmfAbstractAnalysisModule#getTrace public

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

tmf.ui: Support secondary ID for TmfAnalysisViewOutput

This adds the possibility to add the analysis ID as secondary ID when
constructing the TmfAnalysisViewOutput objects.

This is useful for views that support a type of analysis instead of
only one analysis. Analyses of this type will associate the view and
the analysis ID to obtain a view output that will create a different
view for each analysis.

For example, each callstack analysis under a trace or experiment could
automatically have their own flamegraph associated.

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

releng: Update maven plugin versions

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

segStore: Merge ISegment and ISegment2

This breaks the API of ISegment, but makes all segments serializable in
an on disk segment store backend.

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

xml: Add the Host ID as event field

This is useful for analyses on experiment, to divide the resulting state
system per trace if necessary.

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

xml: Event field attributes retrieve aspects by name

instead of using the trace aspect directly. This allows to access global
aspects as well.

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

datastore: remove Generic RangeConditions

Specific RangeConditions, backed by primitives and arrays are up
to 33% faster.

Change-Id: I7828d777f6a534cf1219669c374b261ac829f828
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/94341
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>

datastore: Make the serialization package public

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

datastore: Make the interval package public

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

ust: Add trace constructor with event factory

Traces extending the UST trace may pass their own event factory to the
constructor to parse additional fields.

Example use case for this: A JUL trace of Trace Compass taken with LTTng
is a UST trace, but the CTF field 'msg' is a json string from which we
can extract extra fields. The event factory will do just that.

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

tmf: Retrieve aspects by name instead of class

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

tmf: Add possibility to add global aspects

Those aspects will be searched if no aspect in the trace gives results.

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

ust: add a default UST event layout

That layout can be used by LttngUstTraces that were not taken by the lttng tracer.

Change-Id: I8383813d1cae64417ce5630abc13a788f79712c4
Signed-off-by: Abder Benbachir <anis.benbachir@gmail.com>
Reviewed-on: https://git.eclipse.org/r/96683
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI

timing: Have the segment store view use lazy iterables

Views will make lazy-load the data using an iterator instead of a
collection of segments. This will avoid having to have all segments in
memory.

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

ust: Update callstack requirement message

The procname context is not mandatory anymore

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

common.core: add marker events

These will help create custom markers.

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

tmf: Add Open As Experiment... item to Project Explorer context menu

The user can select one or many traces and/or trace folders to create
and open an experiment of the selected experiment type. All selected
traces and traces recursively found in selected trace folders are added
to the experiment.

If an experiment with the same name and traces already exists, it is
reopened (or selected if it is already opened). Otherwise, a new
experiment is created and opened.

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

segmentstore: have ArrayListStore extend LazyArrayListStore to deduplicate code

Change-Id: Ia1e089945f3fc32ab1c4b0049dd1214165385c22
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/94727
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

segmentStore: remove deprecated APIs

Change-Id: Ic4176666d8aa0d3e0c9fa8cfd3597a084c8ad1c6
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/95720
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

tmf: Remove deprecated TimeGraphCombo

The class TimeGraphCombo is removed from the API. The tmf.ui plug-in
version is increased to 3.0 and affected @since tags are updated.

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

tmf.ui: Fix the destination group in the import wizard

The destination group was handle by the parent class and add a browse
button and the possibility to write the destination path. The text field
and the browse button does nothing in the current version of Trace
Compass and it is a bit confusing since these part are not deactivate. I
removed the browse button and set the text field to read-only.

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

timing: Listener on table viewer should not be mandatory

Race conditions can happen in views where there is a segment store table
and another viewer (like density view) where the data is updated twice
in the table, one by the table viewer itself and one by the density
viewer.

Now the density views can specify not to add a listener for the table
and let the density viewer update the data.

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

tmf.ui: make TimeEvents in ControlFlowView have thickness

This makes active threads much easier to spot.
Adds notion of thickness to the "StateItem". In order to
use it, override StateItem#getHeightFactor().

The patch also introduces the notion of a style map.
This is loosely based on CSS fill styles. The styles
in the map are suggestions and do not need to be implemented
by the UI, but could at a later date.

The styleMap is passes as follows where each step can override the
base map:

* LinuxStyle (Style on an analysis level)
* StateItem (Style on a per-view level)
* PresentationProvider (Style on a per-event level)

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

ss: add 2D iterator queries to the SS API

This patch adds 2D iterator queries to the ITmfStateSystem API,
which lazily returns intervals from a list of their quarks,
that intersect (a time range or a list of timestamps).

Change-Id: I52d510a1c6e5d24faa2d5d5466112c70d795b39a
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/85962
Reviewed-by: Hudson CI
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>

lttng.xml: Add xml analysis for irq

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

releng: update to Guava v21.0.0 for staging target

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

releng: deploy missing orbit dependencies on TC update sites

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

ctf: add os-events trace directory to .gitignore

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

common: package external annotations during build

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

alltests: remove dependencies to org.apache.commons.lang

and use org.apache.commons.lang3 instead

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

ctf.core: Support traces with empty packets before the end

This patch allows tracecompass to support CTF traces containing empty
packets.

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

ctf: make search return the first matching packet in a trace

The CTF parser would return a random packet in search when all packets
overlap. This patch makes it return the first. Performance impacts on
properly packetized traces should be negligeable.

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

lttng: fix memory usage view when ust trace is in an experiment

This fix avoids a TimeRangeException exception if experiment starts
before state system start time. Also, it clears the view if current
window range is outside the state system range of the memory usage.

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

analysis: make call graph analysis non-automatic

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

releng: use mylyn 3.21 to build help plug-ins

The current build process is aligned with the APIs of this
version. Trace Compass only depends on mylyn for the
generation of the help plug-ins and not at runtime.

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

swtbot: Make SWTBotTimeGraph/Entry.getEntry() wait for entry

These methods currently try only once to find the entry but output
"Timed out waiting for..." when it fails.

To match the failure message, and for increased test robustness, make
the methods actually use a wait condition.

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

tmf: Support child entries with different state systems in TimeGraphView

Decouple the roots of the AbstractTimeGraphView with the entries
associated with a state system in AbstractStateSystemTimeGraphView.

This allows, for example, a 'trace' entry to be root of the graph, that
has independent branches of children whose event lists, links and
markers are populated using their own, separate, state systems.

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

timing.core: use symbol aspect directly on call graph segment's getName()

getName() is a presentation level getter, it only makes sense to resolve
the symbol there.

Change-Id: Ifebaad310b79e7974dfbc1ce95298fab34721cf4
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/96062
Reviewed-by: Hudson CI
Reviewed-by: Loic Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

XML: Add multi-analyses views

This patch adds a new attribute to time graph view entry to be able to
use the state system from another analyses to populate some entries.

It also adds more regex support where path can contain place holders
like $1 to be replaced by any wildcard previously specified.

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

common.core: Annotate Objects#requiresNotNull

Change-Id: I2e80880b67042165c6374be3b54df479851c7b2c
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/95503
Reviewed-by: Hudson CI
Reviewed-by: Loic Prieur-Drevon <loic.prieurdrevon@gmail.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

releng: update mylyn update site since in staging target

because the previous one was removed from the remote server.

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

ctf: Add a benchmark trace enum

And add a first trace to be used for benchmarks. These traces are too
big to be added to the test traces repo, so they should be optional
resources to download to run some benchmarks on.

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

ctf.core: use TreeMap for EnumDeclaration

Change-Id: I670bfe871a34774df69e3bff60d0ab144817e617
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/95632
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

analysis: package call graph icon

Otherwise the icon is not available in RCP or upon installation
of the plug-in.

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

releng: Fix ProjectExplorerTracesFolderTest

The generated clashing LTTng traces in TestDirectoryStructureUtil are
changed to have a duration of 1001 ns (not 1000 ns) for 1001 events.

This prevents the time delta between events from flooring down to 0 ns,
which causes all events and all packet begin and end times to be equal.

There is probably another bug in the CTF reader that seeks to the
incorrect packet when there are multiple packets with the same begin and
end times.

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

segStore on disk: Fix boundaries for child lookup

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

Revert "tmf.ui.swtbot: disable RawTextEditorTest#readTrace"

This reverts commit 03ec84392e7bf66cadb1c15b52369bba19a43aa4.

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

Introduce an API to download and import traces from HTTP/HTTPS

This patch contains two parts:
- A class to download a file from HTTP links
- An import operation that use the download helper to import the
dowloaded traces in the workspace

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

tmf.ui.swtbot: Fix missing @Override in SWTBotSash.getBounds()

The parent now implements this method.

Temporarily disable failing checks in ProjectExplorerTracesFolderTest.

Change-Id: I4a60bfa5bafdc07eda2f840fdbda0c0b45a4661d
Signed-off-by: Simon Delisle <simon.delisle@ericsson.com>
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/95815
Reviewed-by: Hudson CI

tmf.ui: Update JUL log points in time graph view

This will allow to analyze the command flow from the user interaction to
the views being updated with the new TraceCompassLogUtils API.

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

timing: Add a generic statistics table view

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

tmf.core/ui: Move some logic of the symbol provider to core

This moves the Symbol provider to core. Special care must be taken to
ensure that the preference page is available if required.

Moves callstack symbol providers to core interfaces.

Makes flamegraph use symbol aspect.

Most changes are internal, so this does not affect the API much.
Also, now CallgraphAnalysis can be in Core.

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

tmf.ui.swtbot: disable RawTextEditorTest#readTrace

Until we can figure out what is breaking the build disable this test.

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

Fix @NonNull annotation errors

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

xml: Move XmlUtils#getElementInFile to TmfXmlUtils

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

critical path: Fix condition that caused infinite loop

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

xml: Move xml strings and schemas to public packages

Since the XML is now extendable, external plugins may need to make use
of those values, so they are now part of the API. As for the schema,
though not mandatory to run correctly, it makes it obvious that they
can be used by external plugins.

Also, a TmfXmlUtils class has been added to the API and the
getChildElements method from XmlUtils has been moved there.

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

tmf.core: instrument checkpoint indexer

This shows how many events were read.

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

tmf: Split ISymbolProvider into core and UI parts

The ISymbolProvider interface and related classes were all living in the UI
plugin only because of the preference page option, which is a UI-only
construct. This limitation was known since the inception of the symbol
provider, with the intent of eventually correctly splitting it into
core and UI components. This patch partially fullfills that.

Future work:

* Make symbol providers in core.

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

releng: Don't use Guava references that will be removed in Guava 21.0

By doing that, it will be possible to use the same master build on
Neon and Oxygen.

com.google.common.base.Objects.ToStringHelper will be removed and
is replaced by apache.commons.lang3.builder.ToStringBuilder.

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

common.core: Add Counter events

These events can be used to fill line charts.

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

tmf: fix strike-through of analyses that can't be executed

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

timing.core/ui: make flamegraphs aggregate threads by default

If you have a flow of data with several threads using a scatter/
gather pattern, the flamegraph aggregates it to show the global
picture of all the threads.

To switch from the per-thread (default) to aggregate (classic) mode
and back, click on the drop-down menu and go to "Content Presentation".

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

segStore: rename SegmentStore performance tests correctly

The package had not been renamed correctly after moving from
analysis.timing to segmentstore plug-in.

Change-Id: I82b8d6c1bf835cbc96c333f9c3330b19ece29a0c
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/87051
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>

tmf: Extract getDistinctColor() method to Utils

It can be now used by other classes.

Change-Id: I9e868799a4da131cd9b94bd89b480ffba37e85a4
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/95006
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.ui: Log analysis schedule/execute lifecycle.

This shows the use of flow scope event.

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

tmf.core: Add logging to Request executor.

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

charts: Add swtbot plugin and utility class

Creates the o.e.t.charts.ui.swtbot.tests plugin and add chart utils
class to be used by other swtbot tests who will create custom charts

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

tmf: Support nested experiments in core

This patch adds support for experiments within experiments in the core
plug-in. It is not yet possible to create nested experiments with the
Project Explorer UI, but this patch adds support for nested experiments
that can be created by plug-in extensions programmatically.

The method TmfTraceManager.getTraceSet() is now recursive and returns
only the leaf traces of a given experiment.

The method TmfTraceManager.getTraceSetWithExperiment() is now recursive
and returns all child experiments and traces of a given experiment,
including itself.

The method TmfTrace.traceOpened() now executes (e.g. analyses) when the
trace being opened is any parent of the trace instance.

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

segStore: getIntersectingElements performance tweaks.

Use binarySearch from (Lazy)ArrayList to correctly dimension
the returned ArrayList, resizing such large arrays happens
frequently during insertion and is costly.

Also stop copying elements into a new arraylist prior
to sorting if the returned Iterable is already a new
instance of arraylist.

Optimize TreeMapStore by reducing it to a single instance
of TreeMultiMap and using its ordered properties
efficiently to return intersecting Segments.

Benchmark results below, best if viewed in a spreadsheet:
SegmentStore Test Before (ms) After (ms) Gains (%)
LazyArrayList Fuzzy Iterate sorted by start 220 171 22.2727272727273
LazyArrayList Fuzzy Iterate sorted by end 45 27 40
LazyArrayList Fuzzy Iterate sorted by length 40 23 42.5
LazyArrayList Random Iterate sorted by start 382 289 24.3455497382199
LazyArrayList Random Iterate sorted by end 127 73 42.5196850393701
LazyArrayList Random Iterate sorted by length 119 71 40.3361344537815
Treemap store Ordered Insertion 1640 669 59.2073170731707
Treemap store Fuzzy Insertion 902 403 55.3215077605322
Treemap store Fuzzy Iterate sorted by start 1610 187 88.3850931677019
Treemap store Fuzzy Iterate sorted by end 1570 164 89.5541401273885
Treemap store Fuzzy Iterate sorted by length 1030 125 87.8640776699029
Treemap store Random Insertion 3960 898 77.3232323232323
Treemap store Random Iterate sorted by start 2890 299 89.6539792387543
Treemap store Random Iterate sorted by end 2460 299 87.8455284552846
Treemap store Random Iterate sorted by length 1550 275 82.258064516129
Treemap store Fuzzy First Insertion 923 196 78.7648970747562
Treemap store Fuzzy First Iteration 66 20 69.6969696969697
Treemap store Fuzzy Second Insertion 929 210 77.3950484391819
Treemap store Fuzzy Second Iteration 95 34 64.2105263157895
Treemap store Random First Insertion 3670 384 89.5367847411444
Treemap store Random First Iteration 265 25 90.5660377358491
Treemap store Random Second Insertion 4590 521 88.6492374727669
Treemap store Random Second Iteration 423 56 86.7612293144208

Change-Id: I3ae2191b74f4f2170f44f45a139b866753b2441a
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/86334
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>

swtbot: Add new utility methods

Add these SWTBotUtils methods:

- SWTBotUtils.getBoundsToDisplay()

- SWTBotUtils.pressShortcut()

- SWTBotUtils.applyTimeGraphFilter()

- ImageHelper.waitForNewImage()

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

xml: Add support for processing extended schemas

This adds an element to the XSD extension point to specify a class to parse
the extended schema elements. When creating helpers, those classes will
be called to provide the right analysis module helper that will create
the analysis module when necessary.

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

xml: Make the schema extendable

This patch adds an element to the tmfxml sequence that can be used to
add new element types to the XML.

It also adds an extension point for plugins to add additional XSD files
that can be used to validate XML with the new types.

The approach is validated by unit tests

Change-Id: I2c71d1b49affa1fa7df41cc9bd67db24af40e9a7
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/92113
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: Fix experiment trace time range in Project Explorer

When the supplementary files are deleted, the start and end times stored
in the corresponding TmfTraceElement are reset. However this does not
affect the trace element instances of the same trace that can be present
under an experiment element. These instances keep their own start and
end times that could be obsolete.

The navigator label provider should simply always use the corresponding
trace element under the traces folder for trace elements under
experiments.

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

releng: Add hamcrest to the target file

Add Hamcrest to the target file in order to get rid of the dependency
issue with mockito/eclipse.test

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

ctf: Fix NullPointerException in CtfTmfTrace.getCallsite()

The event declaration list can contain null elements.

Also, a callsite at exactly the same ip as the requested ip should be
considered valid.

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

segStore: Add unit test for history tree segment store

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

segStore: Add a history tree segment store

Change-Id: I9bf5cfffa7aa15aa6048b30f2e17446406aba891
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/75358
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

datastore: Add a Predicate for children node header

This adds the possibility to filter children nodes by their header data
from a Predicate on the intervals. There is a new method in the tree and
node to transform a Predicate on the interval to a Predicate on the core
node data. Each tree and node implementations are responsible for their
own data.

Change-Id: Ibe44b089a8d25e3f4b53718570372c96e150c1ed
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/89120
Reviewed-by: Loic Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-by: Hudson CI

lami.test: Add stub analyzes and share the package

This will allow other lami packages to use the same stub analyzes.

Also remove a few null warnings by adding assertNotNull before using the
value.

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

charts: Make IDataChartProvider public

It is quite straightforward and keeping it internal.provisional is more
trouble than it's worth since only internal classes in other plugins may
use it without API problems.

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