Geneviève Bastien [Wed, 22 Mar 2017 19:37:26 +0000 (15:37 -0400)] 
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
Geneviève Bastien [Thu, 22 Sep 2016 20:59:37 +0000 (16:59 -0400)]
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>
Matthew Khouzam [Mon, 17 Apr 2017 03:22:07 +0000 (23:22 -0400)] 
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>
Matthew Khouzam [Fri, 21 Apr 2017 18:39:58 +0000 (14:39 -0400)]
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>
Patrick Tasse [Thu, 20 Apr 2017 19:08:36 +0000 (15:08 -0400)] 
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>
Geneviève Bastien [Fri, 3 Mar 2017 17:51:48 +0000 (12:51 -0500)]
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>
Geneviève Bastien [Wed, 19 Apr 2017 19:42:18 +0000 (15:42 -0400)]
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
Geneviève Bastien [Fri, 3 Mar 2017 17:00:01 +0000 (12:00 -0500)]
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
Matthew Khouzam [Wed, 7 Dec 2016 01:43:28 +0000 (20:43 -0500)]
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>
Matthew Khouzam [Mon, 17 Apr 2017 02:10:34 +0000 (22:10 -0400)]
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
Bernd Hufmann [Thu, 13 Apr 2017 13:55:18 +0000 (09:55 -0400)] 
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>
Matthew Khouzam [Thu, 23 Mar 2017 01:55:28 +0000 (21:55 -0400)]
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>
Bernd Hufmann [Thu, 13 Apr 2017 19:48:56 +0000 (15:48 -0400)]
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>
Matthew Khouzam [Wed, 22 Mar 2017 02:57:59 +0000 (22:57 -0400)]
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
Loïc Prieur-Drevon [Tue, 13 Dec 2016 14:51:37 +0000 (09:51 -0500)] 
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>
Patrick Tasse [Thu, 13 Apr 2017 13:38:47 +0000 (09:38 -0400)]
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>
Matthew Khouzam [Mon, 5 Dec 2016 17:37:41 +0000 (12:37 -0500)]
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
Matthew Khouzam [Tue, 29 Nov 2016 21:55:18 +0000 (16:55 -0500)]
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
Geneviève Bastien [Mon, 27 Feb 2017 17:03:48 +0000 (12:03 -0500)]
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>
Patrick Tasse [Mon, 10 Apr 2017 18:14:41 +0000 (14:14 -0400)]
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>
Loïc Prieur-Drevon [Sun, 4 Dec 2016 17:46:06 +0000 (12:46 -0500)]
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>
Patrick Tasse [Wed, 5 Apr 2017 20:17:42 +0000 (16:17 -0400)]
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
Geneviève Bastien [Wed, 1 Mar 2017 17:03:42 +0000 (12:03 -0500)]
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>
Geneviève Bastien [Tue, 28 Feb 2017 16:20:21 +0000 (11:20 -0500)]
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>
Patrick Tasse [Thu, 6 Apr 2017 19:25:26 +0000 (15:25 -0400)]
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>
Simon Delisle [Tue, 4 Apr 2017 18:21:50 +0000 (14:21 -0400)] 
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>
Patrick Tasse [Thu, 6 Apr 2017 16:45:22 +0000 (12:45 -0400)]
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>
Geneviève Bastien [Mon, 3 Apr 2017 20:13:31 +0000 (16:13 -0400)]
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>
Loic Prieur-Drevon [Thu, 1 Dec 2016 01:38:19 +0000 (20:38 -0500)]
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>
Geneviève Bastien [Thu, 19 Jan 2017 15:18:02 +0000 (10:18 -0500)]
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
Geneviève Bastien [Mon, 27 Feb 2017 16:59:15 +0000 (11:59 -0500)]
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>
Geneviève Bastien [Fri, 3 Mar 2017 20:26:30 +0000 (15:26 -0500)]
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>
Loïc Prieur-Drevon [Fri, 31 Mar 2017 19:07:26 +0000 (15:07 -0400)]
tmf.core: Build a TmfAnalysisException from any type of Throwable.
Change-Id: I6b92592ff5092b2941259dcf8958a20a79fd974c
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/94242
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Patrick Tasse [Fri, 24 Mar 2017 22:08:21 +0000 (18:08 -0400)]
tmf: Fix BufferedRandomAccessFile
The method read(byte[]) was not overloaded and therefore reading
natively at the real file pointer, which could be different than the
buffered file pointer set at the last seek or read. The method now reads
from the buffer.
The write methods were not overloaded, and had the same issues. The real
file pointer is now synched with the buffered file pointer before the
write, and the buffered file pointer is synched with the real file
pointer after the write.
When repeatedly seeking backwards and reading, the buffer was
invalidated and filled at every iteration. The file will now position
itself to read ahead by half of the buffer size when seeking backwards.
This will improve the chance to benefit from the read buffer.
Unit tests are added.
Change-Id: If9662844818f1500f8d403b1120083892e788188
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/93840
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Jean-Christian Kouame [Mon, 3 Apr 2017 14:24:29 +0000 (10:24 -0400)] 
settings: Remove unused problem filters
Change-Id: I6f5e0c80472eef411f445c7de687ede00b66bd0e
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/94303
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Jean-Christian Kouame [Mon, 3 Apr 2017 19:30:25 +0000 (15:30 -0400)]
releng: Update tracecompass features version to 3.0.0
Change-Id: Ib5a8735f2b280ef164654ecd8166b52f8e9328a9
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/94176
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Matthew Khouzam [Tue, 15 Nov 2016 19:53:31 +0000 (14:53 -0500)]
linux.ui: add a centralized registry of event styles.
This ensures that a state in the Control Flow view and the Resources
view will always be the same color. Also allows localizable help text.
Change-Id: I17cf2ad0e3bda0e254a1ee6d12df0aa96177647a
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85091
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Patrick Tasse [Wed, 22 Mar 2017 18:24:24 +0000 (14:24 -0400)]
tmf: Fix abstract time graph view end bound
Some abstract time graph view implementations set the end time bound of
the view to the state system end time + 1. This was to be able to see
the last state in the case that the last event in the trace resulted in
a state change (the time graph event begins at end time and has a
duration of 1).
However the TmfTraceManager does not allow the window range to exceed
the trace bounds. So when the view window range was changed to show the
view end bound, it immediately snapped back to the trace end time. This
can be seen by double-clicking on the time graph scale to show the full
range, or by zooming in and scrolling all the way to the end.
This effect also prevents the view from zooming out of a 1 ns window
range at the trace end time.
The end time bound can be set to the state system end time directly. The
final state can still be seen, selected and hovered for its tool tip,
although it has a 1 pixel width.
Change-Id: I6e65fc2f960c54af0c53e4725dba334ba30fe827
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/93649
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Patrick Tasse [Wed, 22 Mar 2017 14:20:01 +0000 (10:20 -0400)]
tmf: Fix time graph scale ticks and grid lines
The time graph control depends on the time graph scale to get the tick
list of x-coordinates to use them to draw vertical grid lines. The tick
list is computed when the time graph scale is painted.
It appears that the order in which these two controls are painted is not
guaranteed. Therefore time graph control can sometimes paint grid lines
using the previous time range's tick list. With AbstractTimeGraphView
this usually gets fixed after a short delay when the zoom thread
triggers another repaint at the same window range, but not always.
This tick list in the time graph scale is no longer cached, and is
always recalculated from the current window range when requested by the
time graph control.
Another issue is that the last grid line disappears when its
corresponding tick label does not have room to draw completely. It is
now always drawn, and the tick label is simply truncated.
The calculation of the number of tick label digits for the CALENDAR time
format is fixed. It was too large and inconsistent, which could cause
grid lines to be unnecessarily far apart.
The digit width is cached so that it can be used outside of the time
graph scale's paint event.
Change-Id: Iaaa1caae8328d469700a7ad8aee726795b356930
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/93620
Reviewed-by: Hudson CI
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Marc-Andre Laperle [Sat, 1 Apr 2017 20:45:06 +0000 (16:45 -0400)] 
Fix some tests with Eclipse 4.7
In Preferences, the OK label changed to "Apply and Close"
Change-Id: Icfddee53a45a895a6df82edd650a5d31c2124e5f
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/94261
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Jean-Christian Kouame [Thu, 30 Mar 2017 20:52:26 +0000 (16:52 -0400)]
releng: Update target file for Neon.3
Change-Id: Ie34424bafeac522d71619cbd2d46100bcedfc121
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/94175
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Jean-Christian Kouame [Thu, 30 Mar 2017 19:52:31 +0000 (15:52 -0400)]
releng: Fix invalid since tag in several files
Change-Id: I2c61ca0faa7676861f9b1bb6419ec1a418a3ed7d
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/94171
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Fri, 24 Mar 2017 10:49:02 +0000 (06:49 -0400)]
releng: Add baseline target definition for release 2.3.0
Also update the oomph setup for that.
Change-Id: I3251ec01d3daf1b3e19e899fa0134c8ca11f207e
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/93796
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Geneviève Bastien [Tue, 20 Dec 2016 02:17:12 +0000 (21:17 -0500)]
lttng: Remove procname from callstack requirements
The analysis does not need the procname and it adds some overhead in a trace
with a lot of function calls. Though procname makes it prettier, it should not
prevent a user from seeing the callstack of a perfectly workable trace.
Change-Id: I3603b52f8559c40b86193d954ab94d9e65dc9bce
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/87449
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Jean-Christian Kouame [Tue, 28 Mar 2017 13:43:41 +0000 (09:43 -0400)]
tmf.ui: handle enabled/disabled trace types in the Filters View
Change-Id: Ia3b1849ba5e45b1ef6fcf2b7225b604dea2c1b44
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/93989
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Jean-Christian Kouame [Thu, 16 Mar 2017 19:41:09 +0000 (15:41 -0400)]
doc: Add documentation for trace type preference page
Change-Id: Iedd2a354117b64e4843f5f774ddc44d46069f24d
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/93244
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Jean-Christian Kouame [Thu, 16 Mar 2017 18:18:23 +0000 (14:18 -0400)]
tmf.ui: Add SWTBot test for trace type preferences page
Change-Id: I66c293de7c9c02fa90b637048c5e452048f43fc4
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/93237
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Jean-Christian Kouame [Mon, 27 Feb 2017 20:26:29 +0000 (15:26 -0500)]
tmf.ui: Add trace type preference page for trace type filtering
This patch adds a preference page to activate selected trace types. User
can only see those activated trace types when he will use the 'Select
Trace Type...' command as well as for trace import and only activated
trace types will be considered for auto-detection. Experiment types are
not taken into account.
Change-Id: Iecd6c481d6cca3adc1d7e510580c55803d9413ee
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/92508
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Jean-Christian Kouame [Thu, 23 Mar 2017 14:49:33 +0000 (10:49 -0400)]
tmf.ui: Add waitUntil#Predicate to WaitUtils
This patch adds a waitUntil() method with a predicate for SWTBot test
Change-Id: I98b33e9a84b309eb251783b2678006866be48c76
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/93738
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Jean-Christian Kouame [Tue, 28 Mar 2017 17:23:13 +0000 (13:23 -0400)]
tmf.ui: Fix import of ctf dependency into tmf.ui
dependency was introduced in https://git.eclipse.org/r/92386
Change-Id: I0015b1dd19d99fdd57bc161ed7c756303131837b
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/94005
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Loïc Prieur-Drevon [Wed, 22 Mar 2017 15:00:27 +0000 (11:00 -0400)] 
tmf.core: implement readEnd for CustomXmlTrace
Change-Id: Ic4f6f7f829c503beeb0201e4be3dd4b189cc8169
Signed-off-by: Loïc Prieur-Drevon <loic.prieur.drevon@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/93623
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>
Loïc Prieur-Drevon [Wed, 22 Mar 2017 14:49:33 +0000 (10:49 -0400)]
tmf.core: implement readEnd for CustomTxtTrace
Change-Id: Ieaf32d6dc66e0ae9ad0e293687a1e2882c0506ce
Signed-off-by: Loïc Prieur-Drevon <loic.prieur.drevon@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/93622
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>
Patrick Tasse [Tue, 14 Mar 2017 20:57:10 +0000 (16:57 -0400)]
tmf: Bug 506660: Fix zoom out from 1 ns window range in time graph
Make sure that zooming out from 1 ns window range is possible with the
Zoom Out tool bar button.
Fix double to long rounding error when zooming out.
Make the 'focused' time (selection if visible, center of previous window
range otherwise) centered in the new window range when zooming out from
1 ns window range.
Keep the selection as 'focused' time when it is exactly one of the
previous window range bounds.
Snap the window range to the time bounds when zooming out of range.
Change-Id: I4501d178acf1c1383caecc95d5bf2b3ec534e602
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/93066
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Geneviève Bastien [Thu, 23 Mar 2017 15:06:50 +0000 (11:06 -0400)]
tmf.ui: Update JUL log points in common X line charts
This will allow to analyze the command flow from the user interaction to
the views being updated with the new TraceCompassLogUtils API.
Change-Id: I640061e6a8eb3c19b17ee4a7025acd52aab47cd7
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/93743
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Geneviève Bastien [Fri, 24 Mar 2017 13:58:57 +0000 (09:58 -0400)]
common: Fix NPE in log arguments
Loggers should NEVER throw exceptions
Change-Id: I39f7ffd18735c6ee2919b76afaec1e1616702c30
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/93809
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Loic Prieur-Drevon [Wed, 1 Mar 2017 14:43:57 +0000 (09:43 -0500)]
tmf.ui: Show a trace's time range in the project explorer
This helps when working on experiments with several traces
to know the trace's time range without needing to display
it completely
Change-Id: I38cb9d2cac545d0b9062eda3301244996ee1feb1
Signed-off-by: Loic Prieur-Drevon <loic.prieur.drevon@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/92386
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Marc-Andre Laperle [Thu, 23 Mar 2017 19:19:33 +0000 (15:19 -0400)]
common: Fix LoggerTest on Windows
\r was making it in the stored messages. It can be ignored.
Change-Id: I1e79a876e8cc5b610b8bccee234648dbe16bbf19
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/93761
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Matthew Khouzam [Tue, 29 Nov 2016 21:47:27 +0000 (16:47 -0500)]
tmf.core: add StateSystemAnalysisModule log point
Change-Id: I5ba890533d6c6149c5bd3c4fbec72493abcaa0a3
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/86080
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Geneviève Bastien [Wed, 22 Mar 2017 19:06:47 +0000 (15:06 -0400)]
common: Use a builder for FlowScopeLog
The 3 available constructors for the FlowScopeLog class are hard to
differentiate because of the Object... parameter. It is error-prone, so
we use a builder instead to make clear which constructor is being
called.
Change-Id: I377f77353b3c0df2e7cbc2c0b8f326e894171f10
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/93650
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Sat, 18 Mar 2017 03:17:35 +0000 (23:17 -0400)]
tmf.ctf: Add unscaled time getter
This allows getting the unscaled timestamp without
rounding errors.
Change-Id: I5b09dd2211169e553a37997c9f5b0cf8581667af
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/93339
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Matthew Khouzam [Wed, 23 Nov 2016 21:57:40 +0000 (16:57 -0500)]
common.core: Introduce logger utils
This patch begins to bring notions or a defined trace format.
It provides a "blessed" way of formatting data to have analyses work
out of the box. It also provides an easy and robust way of
instrumenting without having to worry about details like
TIDs and time stamps.
The usage is reminiscent of LTTng's tracepoint API, in that
the user must supply a list of arguments in the end alternating
between key and value. It is easier to list them this way rather
than forcing a Map, however, this may throw some illegal argument
exceptions.
There are four reference implementations available:
* Scoped logging for durations
This is a try-with-resources logger that makes call stack and
call graph analysis much easier to perform.
* Flow logs for durations and inter-threads relations
This is a try-with resources logger that makes call stack and
call graph analysis much easier to perform, but also shows
inter-thread dependencies.
* Object lifecycle tracking
This allows the tracking of Objects that have notions of "disposing"
* Asynchronous message support
This allows a standardized way to track message passing as well as
asynchronouse operations.
Features missing:
* Marker events.
* Counter events
* Sampling Events
This changes the loglevel of the state system logs, as more information
is stored, it needs more bandwitdth.
Change-Id: I8320fa4246b33442fe44e3ce3935a824cd28feac
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85633
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Patrick Tasse [Mon, 20 Mar 2017 21:37:28 +0000 (17:37 -0400)]
tmf: Fix Marker Set action handling
When changing the active marker set, it didn't update the marker sources
when the opened trace is an experiment. The action only updated the
marker sources from the opened traces, but the marker sets were created
on each trace in the experiment's trace set, for the default
implementation of AbstractTimeGraphView.getTracesToBuild().
It wouldn't be good enough to update the traces using
TmfTraceManager.getTraceSet(), since we don't know the implementation of
getTracesToBuild() for each view. Also, calling
TmfTraceAdapterManager.getAdapters() on each opened trace could actually
create new adapter instances that weren't yet needed.
Instead the ConfigurableMarkerEventSource instances should handle the
TmfMarkerEventSourceUpdatedSignal, and reconfigure themselves.
Also, since the Marker Set menu items are radio, selecting a new marker
set first executes the action on the menu item that is being unchecked.
The action now only executes for the menu item that is being checked.
Change-Id: Ic8fd21b930ec29e8a619cbcedea712d4b7ee4d86
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/93463
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Patrick Tasse [Mon, 6 Mar 2017 19:23:12 +0000 (14:23 -0500)]
tmf: Support X11 colors in marker configuration file
Change-Id: I72bf7a3492e9325dfb914cac56cc6f4cd3d88972
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/92210
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Patrick Tasse [Mon, 6 Mar 2017 19:22:26 +0000 (14:22 -0500)]
tmf: Add X11 color registry
This adds a map of color name strings to RGB color values. It will allow
users to identify colors by name in dialogs, configuration files, etc.
Change-Id: I492292acfd864ebad418049e584c617c525f6106
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/92209
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Marc-Andre Laperle [Tue, 21 Mar 2017 19:17:02 +0000 (15:17 -0400)]
tmf: Fix a few exceptions that could happen when closing
Change-Id: Ibe66165b06a9687a1f4044585bc87db3e2afd8a7
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/93569
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Matthew Khouzam [Tue, 21 Mar 2017 16:54:30 +0000 (12:54 -0400)]
tmf.core: Allow analyses to be used directly from a class
SetTrace no longer throws an NPE if there is no extension registry.
Change-Id: If82c3e1e0ba880e1a9007aa5b22572eec5440df8
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/93554
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Bernd Hufmann [Wed, 15 Mar 2017 14:27:03 +0000 (10:27 -0400)]
releng: update staging target for oxygen M6 builds
Change-Id: I6e91674c063d52c8d730d5fca43829cb1555fdeb
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/93124
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Patrick Tasse [Mon, 20 Mar 2017 17:51:22 +0000 (13:51 -0400)]
tmf: Simplify SelectElementTypeContributionItem
The class TmfTraceType already manages the list of available trace
types, including experiment types and custom trace types. Its list of
trace type helpers can be used to build the menu items.
The unused command parameters BUNDLE and ICON are removed.
Note that menu items will now be grouped by category name instead of
category id. If two different category id used the same name, they will
now be merged instead of showing two sub-menus with the same name.
The items in a category sub-menu will be sorted alphabetically.
Change-Id: I411e88ad34db66d4ec3d32d8e4e6390ca66b6c6a
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/93446
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Matthew Khouzam [Sat, 18 Mar 2017 01:53:55 +0000 (21:53 -0400)]
tmf: fix @since annotations
Fix some issues in CtfTmfTrace and TmfTrace
Change-Id: I7d2c87e060bbaeb59ff7ff2fb0d1bff8b3e04f99
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/93337
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Bernd Hufmann [Wed, 15 Mar 2017 17:59:46 +0000 (13:59 -0400)]
releng: update 4.6/4.5 targets to use neon/mars EMF version
Change-Id: I57eda6b2a685027095b475bc87950a778097c031
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/93145
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Matthew Khouzam [Thu, 27 Oct 2016 20:30:23 +0000 (16:30 -0400)]
timing.swtbot: add flamegraph doubleclick on row test
Change-Id: I1a9cfb265b0f84a717b48197b6b5848d85ba0333
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84044
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Marc-Andre Laperle [Tue, 14 Mar 2017 15:16:15 +0000 (11:16 -0400)]
lttng: Fix a test on Windows with file not being able to be deleted
The synthetic trace was being regenerated while it was already opened.
So it was being used in a "nested way" in the test. The test was just
trying to reopen the trace using the path so pass the path instead which
does not try to delete/regenerate the trace.
Change-Id: If510d366f08a9f8895e667efe370bf9c70edcc5e
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/93033
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Patrick Tasse [Thu, 9 Mar 2017 21:34:24 +0000 (16:34 -0500)]
tmf: Bug 506095: Fix NullPointerException in XML views
When the XML Time Graph View or XML XY Chart View is opened from the
Window > Show View > Other... menu, it does not have a secondaryId. This
should not cause an exception, and the view should remain blank.
Additionally, the view should then not wait for the viewInfo name to be
set, it will never happen and the build job or line chart update thread
remains stuck forever. As a consequence, the viewInfo should be
initialized before creating the view controls.
Change-Id: I0d21cbcc02505bd5d83b7fb4335bf780943fe136
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/92736
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Matthew Khouzam [Tue, 25 Oct 2016 20:07:51 +0000 (16:07 -0400)]
Revert "ctf: Remove the callsite support"
This reverts commit
73511e6797ae5d1ef5669997e21788033d1b2eee.
The keyword callsite is part of the CTF specification. Dropping support
for it was a misstep.
Change-Id: I9b4c4842c1b2333acabac072d4a35fe224cfec5a
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83898
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Geneviève Bastien [Thu, 2 Feb 2017 20:09:31 +0000 (15:09 -0500)]
lami.ui: Remove previous lami charts
Only the custom charts are now used for lami
Change-Id: I6d5cda15916b32dc89e116965da63445fb2ceaa3
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/90217
Reviewed-by: Hudson CI
Gabriel-Andrew Pollo-Guilbert [Wed, 27 Jul 2016 17:43:42 +0000 (13:43 -0400)] 
lami.ui: Replace LAMI charts with custom charts
This patch makes the LAMI analyses use the new chart
plugin instead of their own charts.
Change-Id: Ic0bf607a68b7057b365ae2f40bb40f96895cfbaf
Signed-off-by: Gabriel-Andrew Pollo-Guilbert <gabrielpolloguilbert@gmail.com>
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/78012
Reviewed-by: Hudson CI
Gabriel-Andrew Pollo-Guilbert [Thu, 7 Jul 2016 19:11:20 +0000 (15:11 -0400)]
lami: Implement a custom chart data provider
This patch adds a data model for a LAMI analysis. It also implements a data
provider into the result table.
This allows the creation of custom charts using the tmf.chart plugin
Change-Id: I1375c81c6e99538d4b2e86afb517c4014b0a8a98
Signed-off-by: Gabriel-Andrew Pollo-Guilbert <gabrielpolloguilbert@gmail.com>
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/76899
Reviewed-by: Hudson CI
Geneviève Bastien [Fri, 3 Mar 2017 17:39:22 +0000 (12:39 -0500)]
xml: bug 513074 fix time graph view for events of size 1
This changes the boundaries used to get event data for XML time graph
views. Events that are the last ones and of size 1 are now displayed.
Change-Id: I04d3ead5a723ddaf03282f9f438a418662ad3404
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/92277
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Jean-Christian Kouame [Wed, 22 Feb 2017 18:12:59 +0000 (13:12 -0500)]
tmf.ui: Test the markers set
SWTBot test for Markers Set
Change-Id: Ic8f595d83336ce88b9c915500d7b68c409d92957
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/91736
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Patrick Tasse [Fri, 3 Feb 2017 23:05:58 +0000 (18:05 -0500)]
tmf: Update user guide for configurable marker sets
Change-Id: I5a56a94dff211627db8d4e7f0ac4bd2783cdf9c2
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/90318
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Patrick Tasse [Fri, 3 Feb 2017 17:16:56 +0000 (12:16 -0500)]
tmf: Add unit tests for configurable marker event source
Change-Id: Ibed108fb6b625163c7a2181d43fa9ad28add9781
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/90317
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Patrick Tasse [Mon, 30 Jan 2017 20:34:51 +0000 (15:34 -0500)]
tmf: Add XSD validation for marker configuration file
Change-Id: Id673d01fc65919cb8776dfaec2d8a15ceb2ae214
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/90316
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Patrick Tasse [Thu, 10 Nov 2016 15:46:12 +0000 (10:46 -0500)]
tmf: Support traces providing cycle conversion
The interface ICyclesConverter is added, and the configurable marker
event source will use it to calculate periods and offsets when the unit
is 'cycles' if the trace provides an adapter for it.
CtfTmfTrace is updated to implement this new interface.
Change-Id: I6cda33a880365f0bcc2d1d808f2164781e4f788c
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/84822
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Patrick Tasse [Wed, 9 Nov 2016 22:11:45 +0000 (17:11 -0500)]
tmf: Support traces providing periodic marker reference
The interface IMarkerReferenceProvider is added, and the configurable
marker event source will use it to determine the reference if the trace
provides an adapter for it.
The periodic marker configuration supports a 'referenceid' attribute
which is provided to the trace adapter.
Change-Id: Ib6feee645c58d9d6f63fa07df9a65fdce8c5597a
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/84820
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Patrick Tasse [Wed, 9 Nov 2016 21:27:38 +0000 (16:27 -0500)]
tmf: Make TmfTrace implement IAdaptable
This is useful to be able to use adapter interface for traces, either by
direct implementation or by registering an adapter factory.
Change-Id: Iec23a2e3025771966f3467c56843711698b03331
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/84819
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Patrick Tasse [Thu, 2 Mar 2017 20:45:39 +0000 (15:45 -0500)]
tmf: Add configurable marker event source
Change-Id: I1c8f9b2ebc833063d8fa7b9b4a469b722b8bb41c
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/83645
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Patrick Tasse [Wed, 19 Oct 2016 18:45:27 +0000 (14:45 -0400)]
tmf: Add method to filter-out periodic markers by index
The method isApplicable(long index) can be overridden to filter-out
markers by index, by returning false. The default implementation returns
true, filtering-out nothing.
Change-Id: Ib37043facf10aef4369f1a3701d94b975c68c9e5
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/83644
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Patrick Tasse [Fri, 14 Oct 2016 18:57:48 +0000 (14:57 -0400)]
tmf: Add method to get marker list for all categories at once
This can be useful for marker event sources that provide multiple
categories and for which it is easier to compute all the markers at the
same time.
Change-Id: I5efcfe4444f70ef879fb6a33aa4e28b844d08095
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/83643
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Marc-Andre Laperle [Mon, 6 Mar 2017 16:20:20 +0000 (11:20 -0500)]
tmf: Fix Mars build
In Eclipse 4.5, StructuredViewer had
public void setFilters(ViewerFilter[] filters)
It became in 4.6
public void setFilters(ViewerFilter... filters)
So passing an array of ViewerFilter will work for both.
Change-Id: I1f3e188a78f1b738f9d41bbc05f692141f02b1bb
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/92392
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Hudson CI
Geneviève Bastien [Wed, 1 Mar 2017 17:40:27 +0000 (12:40 -0500)]
xml: Use java.nio.Path instead or o.e.core.runtime
This is what is was meant to be from the start, but a wrong import went
unnoticed.
Change-Id: Iecf4794c9ed9e59a83ebd5e92af9b7ef7ef72b12
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/92112
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Bernd Hufmann [Thu, 2 Mar 2017 20:55:15 +0000 (15:55 -0500)]
lttng: bug 513013: fix invalid thread access
Change-Id: I019497d3cc776b8872c6600bd7c3aa4a5d331c3e
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/92204
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-by: Hudson CI
Marc-Andre Laperle [Mon, 27 Feb 2017 21:47:22 +0000 (16:47 -0500)]
releng: Use baseline target file instead of zip for Oomph
Instead of having to create a zip and pointing to it, use the
existing .target file. This requires to resolve the target first
then set it, as suggested here:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=500097#c7
Change-Id: I4e7b7d9b8ea8b013e24ba8c0ef760066c86251a2
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/77201
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Geneviève Bastien [Thu, 23 Feb 2017 20:27:18 +0000 (15:27 -0500)]
datastore: Fast return to for classic node children
When selecting next children of classic nodes, once the node's
end time is past the maximum of the range, it is no use searching
the rest of the children.
Change-Id: I7a1aba410718ea33a84b6a0b109121b0512fd284
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/91758
Reviewed-by: Hudson CI
Reviewed-by: Loic Prieur-Drevon <loic.prieur.drevon@ericsson.com>
Geneviève Bastien [Thu, 23 Feb 2017 22:07:47 +0000 (17:07 -0500)]
datastore: Rewrite some stream and list manipulations
This patch rewrites some list and stream manipulations to plain
java. It has a non-negligeable impact on performance.
Change-Id: I11881780d30f27592765140edda1af04b1a2c1e5
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/91757
Reviewed-by: Hudson CI
Reviewed-by: Loic Prieur-Drevon <loic.prieur.drevon@ericsson.com>
Geneviève Bastien [Thu, 16 Feb 2017 20:56:28 +0000 (15:56 -0500)]
datastore: Add intervals without check for node read
When nodes are read from disk, we know the intervals fit into it, there
is no need to go through the code path that checks if the interval can
go in the node.
This change alone greatly reduces the performance impact of both single
and full queries for large state system, when nodes are often read from
disk.
Change-Id: If653611b00d56b604d23ddd1f6e62bb22f822da0
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/91756
Reviewed-by: Hudson CI
Reviewed-by: Loic Prieur-Drevon <loic.prieur.drevon@ericsson.com>
Geneviève Bastien [Tue, 21 Feb 2017 19:35:45 +0000 (14:35 -0500)]
datastore: Keep a snapshot of the latest branch
This avoids making immutable copies of the same branch over and over.
Also use the latest branch directly in the node insertion instead of
the getter.
This change alone removes all performance impact of the datastore
when building.
Change-Id: If85ae9bded364cfd6f1243416d025df6307cf909
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/91755
Reviewed-by: Hudson CI
Reviewed-by: Loic Prieur-Drevon <loic.prieur.drevon@ericsson.com>
Geneviève Bastien [Tue, 21 Feb 2017 15:39:05 +0000 (10:39 -0500)]
datastore: Use time conditions with long
Using primitive type long instead of a generic Long for range conditions
reduces the need for boxing and unboxing time values.
This change alone reduces the performance impact of the datastore by ~2
in use cases of small size for both single and full queries. Coupled with
the patch with single queries API, it reduces the performance impact of
single queries to < 100%
Change-Id: I63ebb20b2441aff1345b733f1ffc25314e63ffd2
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/91754
Reviewed-by: Hudson CI
Reviewed-by: Loic Prieur-Drevon <loic.prieur.drevon@ericsson.com>
Geneviève Bastien [Wed, 15 Feb 2017 17:23:20 +0000 (12:23 -0500)]
datastore: Add API for single queries
When what we want is only one interval, there is no need to get them
all, we can just fast return when the one interval is found.
This change alones reduces the performance impact of the datastore by
~2x for single queries in the case of smallish traces.
This patch also adds a binary search to find the first interval to
look at.
Change-Id: I77d0ac9818f272f7d160f92666445ce2d8b3200a
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/91753
Reviewed-by: Hudson CI
Reviewed-by: Loic Prieur-Drevon <loic.prieur.drevon@ericsson.com>
Gabriel-Andrew Pollo-Guilbert [Tue, 2 Aug 2016 16:36:07 +0000 (12:36 -0400)]
charts: Add interactivity and signals
This patch adds a basic signal that is sent when the
selection inside a chart is updated.
Change-Id: I31e66701dfb2cf495fea4d948fe90e545485d30c
Signed-off-by: Gabriel-Andrew Pollo-Guilbert <gabrielpolloguilbert@gmail.com>
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/78348
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Gabriel-Andrew Pollo-Guilbert [Fri, 15 Jul 2016 19:49:12 +0000 (15:49 -0400)]
charts: Add dialog for making a custom chart
This patch adds a dialog that allow to configure a chart
based on the possible data descriptors of a data
provider.
Change-Id: Ia790b11083ef07b49953cf9f3578eeaea7785495
Signed-off-by: Gabriel-Andrew Pollo-Guilbert <gabrielpolloguilbert@gmail.com>
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/77162
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
This page took 0.047823 seconds and 5 git commands to generate.
projects / deliverable / tracecompass.git / log
