Alexandre Montplaisir [Fri, 30 Oct 2015 11:28:43 +0000 (07:28 -0400)]
Annotate Guava Immutable collections
The following methods in Immutable{List|Set|Map} are all used
to create new collections and never return null values:
builder()
Builder.build()
copyOf()
of()
They can all be annotated as @NonNull.
Change-Id: I7149f488d2905c5def7b7b39993e9584642a4b0a
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/59326
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Matthew Khouzam [Mon, 16 Nov 2015 19:15:08 +0000 (14:15 -0500)]
lttng: add NON-NLS annotations to Lttng27EventLayout
Change-Id: Ieef1cd4c22f5747748e7fc4e69c1bd633e934ce4
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/60547
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI
Jean-Christian Kouame [Mon, 16 Nov 2015 20:19:41 +0000 (15:19 -0500)]
gdbtrace : remove occurrences of literals
Change-Id: I337dc2efba794ec3a8b12adec65c76a6c2d1a1e4
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/60551
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Jean-Christian Kouame [Mon, 16 Nov 2015 17:50:30 +0000 (12:50 -0500)]
tmf : use of StringBuilder.append() instead of operator + for strings
Change-Id: Ia0156a18fb2e81db4a375d8c36d34e3ced09d55b
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/60536
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Marc-Andre Laperle [Thu, 5 Nov 2015 21:07:22 +0000 (16:07 -0500)]
statesystem: Remove ITmfStateInterval.getViewerEndTime
Nothing calls it and its semantics are not well defined.
Also, it makes little sense that the state system would have knowledge
about what views want.
Change-Id: Idff4d0bf24c4173c8cd8a67ac18753dcd6fb4ff4
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59785
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI
Marc-Andre Laperle [Fri, 13 Nov 2015 00:27:56 +0000 (19:27 -0500)]
tmf: Fix SWTBot test on Windows related to short temp path
On Windows, the system property for the temorary path looks like this:
java.io.tmpdir=C:\Users\MARC-A~1\AppData\Local\Temp\
Which is a "short path". When used in comparison with full, canonical
paths, it fails.
Change-Id: I3a90f792791e6983c2eab60d54b1e0694ed98dab
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/60273
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Patrick Tasse [Wed, 11 Nov 2015 15:21:50 +0000 (10:21 -0500)]
tmf: Fix trace markers with experiments in AbstractTimeGraphView
The key of the marker event sources map should always be the view trace.
Change-Id: Iecb458fd291c3ae1b62a5fed9f9954306f41a09a
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/60138
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Patrick Tasse [Wed, 11 Nov 2015 14:49:12 +0000 (09:49 -0500)]
ctf: Fix CtfTmfLostEvent time range does not use timestamp transform
When a timestamp transform or trace offset was set on the trace, the
lost event time range end time was not transformed, but the timestamp
and time range start time were.
Change-Id: Ie2e79b5e90e900bdaf2e7e8391868132bd632332
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/60136
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Alexandre Montplaisir [Mon, 2 Nov 2015 20:31:19 +0000 (15:31 -0500)]
Add workspace/ to the gitignore
A top-level workspace/ directory gets created by the Maven build now.
It should not be tracked in Git.
Change-Id: If8a7e0bce1c99d36f9727d642dbebf95bb11da90
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/59879
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Alexandre Montplaisir [Sun, 1 Nov 2015 07:47:09 +0000 (02:47 -0500)]
lttng: Rename orphaned relayd test to TestApplication
This file is a test application that is not intented to be tested as a Junit
test but it is written as such for convenience. We use this while developping
the Live feature.
Change-Id: I705f5a9fdfcb5b01b9478fe5402b48587a4c871c
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/59441
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Hudson CI
Patrick Tasse [Thu, 5 Nov 2015 17:50:26 +0000 (12:50 -0500)]
tmf: Add Lost Events marker event source and factory
A marker event source factory for Lost Events is enabled for all traces.
The marker event source can reuse a previously returned marker list if
the requested parameters are identical to the previous request.
The event types statistics state system is updated to add a new
attribute which is used to compute the time ranges that are covered by
at least one lost event time range.
Change-Id: I2479afc3f07deb363f5847f4816940010bae3e9e
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59389
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Patrick Tasse [Fri, 30 Oct 2015 20:15:04 +0000 (16:15 -0400)]
tmf: Add TmfTraceAdapterFactory
Base class for a trace adapter factory. The factory creates a single
instance of each adapter type per trace, and disposes the adapter when
the trace is closed, if it is an instance of IDisposableAdapter.
Change-Id: Ieec69d17d0cff037f2df0c8dded14eef91db6f97
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59388
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Patrick Tasse [Thu, 5 Nov 2015 15:43:41 +0000 (10:43 -0500)]
tmf: Support adjustment of time graph window range on range selection
When the selection range is modified and 'ensureVisible' is true, then
the window range will be centered on either the selection begin time (if
it has changed) otherwise on the selection end time, if that time is
outside of the current window.
This adjustment is now enabled in time graph views when the
TmfSelectionRangeUpdatedSignal is received.
Change-Id: I8b5ec885b00009be05222db02acd745218759131
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59758
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Patrick Tasse [Fri, 6 Nov 2015 15:38:14 +0000 (10:38 -0500)]
tmf: CallStackView sets wrong bounds
Apply the same changes that were done in patch
https://git.eclipse.org/r/#/c/55593 for the AbstractTimeGraphView.
Change-Id: I3f1bf9b8cc8dc017e60d92b079e74c892e3b8530
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59856
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Patrick Tasse [Mon, 2 Nov 2015 20:35:34 +0000 (15:35 -0500)]
tmf: Add keyboard shortcuts for Next/Previous Marker
Support '.' for Next Marker and ',' for Previous Marker.
Support Shift key for extending the selection, both with the keyboard
shortcut and with the action button.
Update the status line when Next/Previous Marker changes the selection.
Clarify the marker event source API to indicate that the previous and
next markers should be included in the marker list.
Change-Id: I1f435e9620030809f10f3788eca40521c8a745a9
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59512
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Patrick Tasse [Thu, 5 Nov 2015 20:08:23 +0000 (15:08 -0500)]
analysis: Fix missing setItemCount in AbstractSegmentStoreTableViewer
When the last trace is closed, the table viewer input is set to null.
Its item count should also be set to 0. Otherwise this can later cause a
NullPointerException in the TableViewer.
Change-Id: Ic1d482fd5e4ee64f7969375b1fe4d8b626fa31e0
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59781
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Jean-Christian Kouame [Wed, 30 Sep 2015 18:03:57 +0000 (14:03 -0400)]
tmf : Change the order of modifiers of reinitializeCharts() in TmfPieChartViewer
Change-Id: I122d2706472dd5a421ee3e2da1c7277f90959783
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57109
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Alexandre Montplaisir [Sat, 31 Oct 2015 19:57:55 +0000 (15:57 -0400)]
rcp: Add compile-time dependency to jdt.annotation
The rcp.ui plugin will indirectly depend on the null annotations
due to upcoming new external annotations.
Change-Id: Ief9a2635187cf3fe8ad998810a765b0bbc58176d
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/59419
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Hudson CI
Alexandre Montplaisir [Fri, 30 Oct 2015 09:42:35 +0000 (05:42 -0400)]
common: Rectify some annotations in NonNullUtils
In-line genereic type definitions should not have @Nullable/@NonNull
annotations. These should be only on the "real" parameter and/or return
types.
Change-Id: Iba75a9cf6eb9cbe4662eec25089fa91ffa079728
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/59324
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Marc-Andre Laperle [Thu, 29 Oct 2015 22:34:21 +0000 (18:34 -0400)]
lttng: Remove discriminator part when parsing addr2line output
Remove discriminator part, for example: /build/buildd/glibc-2.21/elf/dl-object.c:78 (discriminator 8)
This fixed a NumberFormatException
Change-Id: I92682bee4106d64dcf2964dc98823a80597df3fa
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59298
Reviewed-by: Hudson CI
Marc-Andre Laperle [Fri, 23 Oct 2015 14:28:07 +0000 (10:28 -0400)]
Upgrade to Tycho 0.24.0
See https://wiki.eclipse.org/Tycho/Release_Notes/0.24
The immediate improvement is that it uses JDT from Mars.1 to compile.
Change-Id: Id66030216ad242996c20dbc37c8409f617838fc2
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58819
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Bernd Hufmann [Tue, 27 Oct 2015 11:40:22 +0000 (07:40 -0400)]
lttng: update and add HR timer related events and fields
Change-Id: I87b3e1ac4a79059a2c70b1b3ef7e07be6baa4f10
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59011
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Tue, 27 Oct 2015 15:18:32 +0000 (11:18 -0400)]
analysis: Don't run latency analysis automatically
This reverts an unintentional change.
Change-Id: I5e4a9b1b8e8a2f54aac7924b78d1200f831497e6
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59017
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Alexandre Montplaisir [Fri, 30 Oct 2015 20:14:14 +0000 (16:14 -0400)]
ctf: Fix simple Javadoc warning
Change-Id: I2bd8c9de68a4b10de1e751da38d4ed061c1f3a18
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/59420
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Geneviève Bastien [Tue, 15 Sep 2015 18:54:02 +0000 (14:54 -0400)]
ss: Replace some asserts by IllegalStateExceptions
Change-Id: I692279143582c78b629d973107ec65e40f850793
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/56014
Reviewed-by: Hudson CI
Geneviève Bastien [Wed, 28 Oct 2015 03:56:45 +0000 (23:56 -0400)]
ss: Fix a bug where depth of history tree increases at each new node
When adding a sibling after a new root, there was one too many levels.
Change-Id: I19f460347cbba91c5c1f70d9ee4566a3cb164f37
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/59076
Reviewed-by: Hudson CI
Matthew Khouzam [Thu, 1 Oct 2015 00:59:02 +0000 (20:59 -0400)]
TMF: Fix behavior of XML time graph views with experiments
Expected behavior:
* Show the content of analyses when the trace contains those analyses.
* If experiment, show the content of analyses if any of the trace or
the experiment itself contains those analyses.
* Empty if no trace contains the required analyses.
Change-Id: Id04c2f9d4f1ecd9c8c466a9a0536e1b07277ea1f
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/35490
Reviewed-by: Hudson CI
Geneviève Bastien [Wed, 28 Oct 2015 03:42:53 +0000 (23:42 -0400)]
ss: Fix wrong calculation of string size in history tree
The strings size was calculated twice in the node size. The nodes were
therefore not used at full capacity.
Change-Id: Iea15dfd701859deaaea7857de83c36c3455eefb9
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/52755
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI
Patrick Tasse [Wed, 25 Mar 2015 15:27:50 +0000 (11:27 -0400)]
ss: Add AttributeTreeTest unit test
Change-Id: I7a67d33ad694370393fa2789e2508233ec570877
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/45686
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Bernd Hufmann [Tue, 13 Oct 2015 11:47:07 +0000 (07:47 -0400)]
lttng: pack columns in latency table
Change-Id: I18855e0e4d728aca681bcd39229ca156e45cedb8
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58062
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Bernd Hufmann [Wed, 28 Oct 2015 15:39:39 +0000 (11:39 -0400)]
analysis: rename latency to segment store for statistics
Change-Id: I1bd00e238843e9256d0ffcab38c1752ac0f35180
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59134
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Bernd Hufmann [Wed, 28 Oct 2015 15:40:39 +0000 (11:40 -0400)]
analysis: avoid division by 0 in latency statistics
Change-Id: Iaa30a5989517777b37aa4b7b21b5c4da025dcec9
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59133
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Fri, 30 Oct 2015 13:09:06 +0000 (09:09 -0400)]
analysis: Add null check for dependent analysis
This fixes a NPE and also avoids execution the statistics analysis on
other trace types than LTTng Kernel when the view is open.
Change-Id: I6583dcbb80044586f25067f32a5fbd9f28c7d4ed
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59045
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Mon, 26 Oct 2015 18:56:14 +0000 (14:56 -0400)]
analysis: Rename latency related classes to System Call Analysis
Change-Id: I16393568ca9453768f4d95f4e4bbef8edae85327
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58988
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Marc-Andre Laperle [Wed, 28 Oct 2015 18:10:05 +0000 (14:10 -0400)]
analysis: Allow segment table viewer/provider to accept ISegment[] input
This makes it easier to use a subset of a segmentstore as input.
Change-Id: I39b453805b9dc9f612bc0d72372ebde3c8cc4774
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59155
Reviewed-by: Hudson CI
Marc-Andre Laperle [Thu, 29 Oct 2015 22:22:15 +0000 (18:22 -0400)]
lttng: Fix AttributeNotFoundException when dealing with event before state dump
Change-Id: I6fcd21928d1430f1e450dede276eefa3820493fe
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59297
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI
Marc-Andre Laperle [Thu, 29 Oct 2015 22:24:17 +0000 (18:24 -0400)]
common: Add external annotations to NavigableSet
The following methods in NavigableSet:
ceiling()
floor()
higher()
lower()
can all return null if there is no corresponding element in the set,
independently of the container type declaration. They should be
marked @Nullable.
Change-Id: I66b9cd5ba8b6eafc69032fb0ae27a9d1ce9e464c
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/59296
Reviewed-by: Hudson CI
Matthew Khouzam [Wed, 28 Oct 2015 18:48:44 +0000 (14:48 -0400)]
common: Annotate some methods in String
All variants of:
copyValueOf()
format()
intern()
toString()*
valueOf()
return a non-null value, so can be marked as such.
* Do not confound with Object.toString()! That one has no nullness
guarantees.
Change-Id: Ia739dab7ab294e0c32156bd3b756f4241e97a4a1
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59163
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI
Alexandre Montplaisir [Sat, 24 Oct 2015 18:46:05 +0000 (14:46 -0400)]
lttng: Do not throw exception if callsite binary is missing
If a trace with debug information is opened on a different machine
than the one where it was taken, it's very possible for the referred
binary files to not be present. In such a case it should not throw
an exception, but simply return that no source callsite information
is available.
The (upcoming) "binary callsite" information should still be available
in such cases though.
Change-Id: I5c3c77d9086b1445b8a1b11e8bb312929321d7f3
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/58883
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Alexandre Montplaisir [Fri, 23 Oct 2015 23:06:12 +0000 (19:06 -0400)]
Replace instances of FluentIterable with Java 8 Streams
Guava's FluentIterable was already used in some places to make use of
functional idioms. Now that we have moved to Java 8 we can replace
those with base Java Streams.
Change-Id: I41bce430327020837b6fb3cc053405ab231801ab
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/58882
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Alexandre Montplaisir [Mon, 25 May 2015 16:38:29 +0000 (12:38 -0400)]
releng: Transition to jdt.annotation 2.0
The new o.e.jdt.annotation 2.0 annotations (based on Java 8
Type Annotations [1]) can now apply on generic type declarations,
allowing things like "List<@NonNull String>".
The @NonNullByDefault annotation, already in use, will now apply
to these locations. This required some code changes to eliminate
compilation errors, notably in places where null checks were
previously made and were not needed, or where some existing
declarations had to be specified as @Nullable.
Another relatively important change is the new syntax for []
arrays. Before, only "@NonNull String[]" was allowed, which meant
"a non-null reference to an array of unspecified Strings". Now, it
instead means "an unspecified reference to an array of non-null
Strings".
It is still possible to specify the array reference itself, but by
doing: "String @NonNull []". Both locations can be annotated
independently.
[1] See https://docs.oracle.com/javase/tutorial/java/annotations/type_annotations.html
Change-Id: I4cb0a3266928c24cfd7f8ed442c93556132f892c
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/57252
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Fri, 23 Oct 2015 22:24:40 +0000 (18:24 -0400)]
Add null checks for methods missing them
In preparation of annotation-enforced @Nullable Map.get() etc.
Change-Id: I81385aba7007baf8ad94fc1519689621f1945493
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58880
Reviewed-by: Hudson CI
Patrick Tasse [Wed, 28 Oct 2015 18:56:17 +0000 (14:56 -0400)]
tmf: Add drop-down to Next Marker action to set navigation enablement
The Next Marker action now has the AS_DROP_DOWN_MENU style. The menu
items allow to disable navigation of markers by category.
The Next/Previous Marker actions now skip markers of disabled
categories.
The marker list is cleared immediately when switching traces in the
abstract time graph view. If the trace was not a supported trace for
that specific view, the marker list would never get updated.
The marker actions now disabled if the time graph has no content (empty
time bounds).
The enablement/disablement of the Next/Previous Marker actions based on
current time selection that was requested and implemented in
https://git.eclipse.org/r/57135 is reverted for the following reasons:
- It would now require to go through the list of markers and comparing
each marker category against the set of disabled categories, just to set
the action enabled state, whenever the time selection changes.
- It would make the Next Marker drop down menu inaccessible if the
current time selection is after the last marker.
Change-Id: Idd8361ccc6a00c0bb19749019965e5389aa2692a
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59171
Reviewed-by: Hudson CI
Patrick Tasse [Tue, 27 Oct 2015 22:40:40 +0000 (18:40 -0400)]
tmf: Add Show Markers menu for time graph viewer
Support the dynamic Show Markers menu in time graph viewer that contains
a checked action for each of: Bookmarks category, view-specific marker
categories, and trace-specific marker categories for the current trace.
Add the menu to the abstract time graph view's view menu.
Hide markers of categories that are unchecked in the Show Markers menu.
This will also make them ignored in Next/Previous Marker navigation.
Change-Id: I31c0ee983c217a149070f8628a162fc7420222d6
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59072
Reviewed-by: Hudson CI
Patrick Tasse [Tue, 27 Oct 2015 19:23:26 +0000 (15:23 -0400)]
tmf: Support navigation for all markers in time graph viewer
The time graph viewer now stores both the bookmark list and the marker
list. The time graph control's marker list is the sorted union of both.
The Next/Previous Bookmark actions are renamed to Next/Previous Marker,
and now apply to all markers in the merged list.
Change-Id: Idf2b14394726885753a926c1424ce7e0d315610b
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59064
Reviewed-by: Hudson CI
Patrick Tasse [Tue, 27 Oct 2015 15:21:54 +0000 (11:21 -0400)]
tmf: Support marker event categories
Change-Id: Ic60590932a23e21d39b800a7e3893ea0661d16d9
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/59063
Reviewed-by: Hudson CI
Alexandre Montplaisir [Fri, 23 Oct 2015 22:42:13 +0000 (18:42 -0400)]
lttng: Remove outdated comment
The return type was changed already.
Change-Id: I3106689d272359af2c0ff0f0ab7fbda8656361e2
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/58881
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Alexandre Montplaisir [Wed, 20 May 2015 23:35:39 +0000 (19:35 -0400)]
Link all plugins to the external annotation location
Change-Id: I329f7bc22a8509534540b97ef3451fd21840094d
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/57251
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Wed, 28 Oct 2015 18:49:16 +0000 (14:49 -0400)]
common: Annotate some static methods in Collections and Arrays
The following methods and static fields can be marked as @NonNull:
Arrays.asList()
Collections.empty{Map|List|Set}()
Collections.EMPTY_{MAP|LIST|SET}
Change-Id: I6a36ac288441099219de19b1ec7793ed5cfd2a01
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59165
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI
Matthew Khouzam [Wed, 28 Oct 2015 18:49:38 +0000 (14:49 -0400)]
common: Annotate java.util.Formatter
Annotate both Formatter.format() methods to return a @NonNull formatter
(they return the formatter object itself).
Change-Id: I8f18f439d6db909f10c0f54fd4793665c4966a6b
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59164
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI
Matthew Khouzam [Wed, 28 Oct 2015 18:50:28 +0000 (14:50 -0400)]
common: Update external Map annotations
Make entrySet() return a @NonNull Set (if the map exists, the entrySet
necessarily does).
Change-Id: I08dc06455239701a45ad6f6d9c678f811af73c0f
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59162
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI
Alexandre Montplaisir [Thu, 22 Oct 2015 22:20:41 +0000 (18:20 -0400)]
common: Add external annotations for Guava's Table#get
Just like Map.get(), Table.get() can return a null value to
indicate that the key is not present in the table, independently
of the declaration of the type V.
Guava Tables are now used in the graph/critical path classes, so
this is also required.
Slightly harder to add annotations to Guava classes atm due to
https://bugs.eclipse.org/bugs/show_bug.cgi?id=480462
but one can still write the .eea files manually.
Change-Id: Id5cc1bdea73bafe62af60df0fdf7f3e9c5f40796
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/58776
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Tue, 9 Jun 2015 13:51:27 +0000 (15:51 +0200)]
common: Add external annotations for IAdaptable/IAdapterFactory
Change-Id: I0a35aa77392ae57602e7171d68b490fb9fe6411f
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/57250
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Tue, 19 May 2015 00:33:29 +0000 (20:33 -0400)]
common: Add external annotations for Map.get()/.remove()
Even if one declares a Map<Object, @NonNull Object>, Map.get()
can return a null to indicate "there is no value for this key".
Same applies to .remove() too.
Since the base methods are not annotated, we either have to
supply our own annotation, or make sure to always use @Nullable
keys. The former is less error-prone, so let's do that.
See http://stackoverflow.com/a/
25371836/
4227853 .
Change-Id: I3bb7395d7dca053d354f2f418d8529696edd0346
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/57249
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Thu, 27 Aug 2015 17:53:16 +0000 (13:53 -0400)]
control: Add LTTng profile save dialog
Change-Id: I42ab05d707369d5082f2503f0ef04993d4193cf3
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57685
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Bernd Hufmann [Wed, 12 Aug 2015 00:56:15 +0000 (20:56 -0400)]
control: Add support for downloading profiles after saving of sessions
Change-Id: Ie42d8d2d408f52fd093780193859015cb0907613
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57684
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Bernd Hufmann [Tue, 2 Jun 2015 20:06:20 +0000 (16:06 -0400)]
control: Add support for saving session
This commit adds support for saving one or more session in
default location and force overwrite
Change-Id: I6996415456c86b9bc82df6504af04ee1a13f1440
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57683
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Bernd Hufmann [Mon, 3 Aug 2015 13:04:43 +0000 (09:04 -0400)]
control: Add LTTng profile preference page
Change-Id: Iaf8f420eadf4747866a8eee2b3b9a9bc723d7396
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57682
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Bernd Hufmann [Wed, 12 Aug 2015 01:13:06 +0000 (21:13 -0400)]
control: Add enhanced support for loading of sessions
Change-Id: I68df4fba7eb53542cd9eca4cf15e161cc679d782
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57681
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Patrick-Jeffrey Pollo Guilbert [Fri, 3 Apr 2015 03:02:32 +0000 (23:02 -0400)]
control: Base code for profile dialog window
Modified ILttngControlService to add a loadSession command, also
implemented the methods in the appropriate children classes
(LTTngControlService and NullControlService).
Added ProfileDialog and ProfileHandler, as well as the appropriate
modifications to the plugin.xml to allow the execution of the handler as
well as the call to the Handler. As for now, pre-selected profiles must
be in
org.eclipse.tracecompass.internal.lttng2.control.ui/resources/folders.txt.
Appropriate .icons, strings and whatnot are externalized and at the
appropriate location > messages.properties.
Signed-off-by: Patrick-Jeffrey Pollo Guilbert <patrick.pollo.guilbert@gmail.com>
Signed-off-by: William Tri-Khiem Truong <tri-khiem-william.truong@polymtl.ca>
Signed-off-by: William Enright <william.enright@polymtl.ca>
Change-Id: Id312708b1907f3dd36fab65da7d96d5f917404a2
Reviewed-on: https://git.eclipse.org/r/45958
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Mon, 5 Oct 2015 16:47:42 +0000 (12:47 -0400)]
lttng: Rename the debug-info callsite aspect to "source callsite"
If not all debug information is available in the binary, it is possible
to only display the "binary file/symbol/offset" tuple. The latter will
be named "binary callsite", to differentiate it from the complete
source callsite.
Change-Id: Ie7e0368a6b1c02c63f759536728835f464ea463b
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/58775
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Sat, 5 Sep 2015 00:53:19 +0000 (20:53 -0400)]
tmf: Allow ITmfSourceLookup to open files outside the workspace
Bug: 476682
Change-Id: I663bec71c4b97ecc71dc8fcd0bbc620de15bc8a7
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/56193
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Matthew Khouzam [Sun, 25 Oct 2015 01:40:18 +0000 (21:40 -0400)]
analysis: Improve comments for IKernelAnalysis.java
Add javadoc to IKernelAnalysis. The tracepoints used by the kernel state
system now are described in a simpler language, as it is unfair to assume
that an eclipse developper working with trace compass is also a Linux
kernel developer.
This patch is part of an effort to make it easier to adapt new operating
system traces to trace compass. The javadoc is written based on the commit
messages for given patches and Linux Kernel Maling List discussions as well
as wikipedia, trace metadata and Linux Weekly News updates.
Change-Id: I33eb53d0b172c3b9d235eaa2e4681eaa410e4e76
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58886
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Bernd Hufmann [Thu, 22 Oct 2015 17:44:13 +0000 (13:44 -0400)]
btf: fix sonar warning about null safety
Change-Id: I197a0708f15a89f28c190e242abec53a1bf94c36
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58758
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Sat, 24 Oct 2015 02:23:10 +0000 (22:23 -0400)]
analysis: Clean up critical path algorithm a bit
Change-Id: I8c798fc612a28640b29292a6e51af046ebe50d97
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58872
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Jean-Christian Kouame [Wed, 30 Sep 2015 21:11:55 +0000 (17:11 -0400)]
tmf : extract duplicate code into sortNodes() in GraphNode
Change-Id: I6309676ab960c0791bb8b0ce39d44951a8ed3075
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57246
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Jean-Christian Kouame [Wed, 14 Oct 2015 17:12:28 +0000 (13:12 -0400)]
tmf : use of the map values instead of keyset
Change-Id: I80a02e117d0afe3b45b557903e185443c6f099ca
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58182
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>
Bernd Hufmann [Mon, 26 Oct 2015 13:46:37 +0000 (09:46 -0400)]
analysis: Register latency statistics view as analysis output
and remove action from context sensitive menu of latency table
Change-Id: I806b87ac2088adff18e69836f847f1353c5153e5
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58932
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Mon, 26 Oct 2015 13:32:23 +0000 (09:32 -0400)]
analysis: Clean-up latency statistics
Change-Id: I978c8c4cf268d96989f9c135301c6a323d1f7130
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58929
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Patrick Tasse [Fri, 23 Oct 2015 22:58:50 +0000 (18:58 -0400)]
tmf: Support trace-specific markers in AbstractTimeGraphView
Adds an IMarkerEventSource adapter interface.
Computes the list of adapters when a trace is loaded in the view.
Uses the adapters to get the trace-specific marker events for the
current window range when zooming.
Renames getMarkerList() to getViewMarkerList() to better differentiate
with getTraceMarkerList().
Change-Id: I97e00d5cf8eb0c39b78b04077322e26f7d2a85da
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/58864
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Patrick Tasse [Fri, 23 Oct 2015 22:55:05 +0000 (18:55 -0400)]
tmf: Add TmfTraceAdapterManager
This adapter manager can handle adapters that are specific to a
particular trace type id, to provide specific adapters even if the trace
instance is of the same class.
Unlike AdapterManager, this manager will return a list of all matching
adapters that adapt the given trace.
Change-Id: I4c8d6e6d725cf80a2e9ab865ea3217ae05f8cc73
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/58863
Reviewed-by: Hudson CI
Patrick Tasse [Fri, 23 Oct 2015 20:18:30 +0000 (16:18 -0400)]
tmf: Add previous full state in AbstractStateSystemTimeGraphView queries
Adding the previous full state allows the implementation to be able to
prevent duplicated elements at the boundaries of the query iterations.
Change-Id: Idd3e9718a0e4fa97419e5231713b6fe1711ad2e5
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/58862
Reviewed-by: Hudson CI
Patrick Tasse [Tue, 20 Oct 2015 16:51:05 +0000 (12:51 -0400)]
tmf: Improve bookmark and marker handling
- Support goto marker without rank (with time and duration) in event
table.
- Update goto marker handling to broadcast time/time range selection.
- Execute pending goto marker handling after trace opened signal sent.
- Add initial value to AddBookmarkDialog.
- Use AddBookmarkDialog in event table.
- Improve format of bookmark location string.
- Add interface for marker attribute strings.
- Fix handling of marker changes when trace is closed or deleted.
- Support ADDED resource change delta and consolidate handling of all
deltas from the same resource change event.
- Add methods to add or remove one or more markers in event table.
- Support location attribute as int (backward compatible) or String
value in trace package import operation.
Change-Id: I64fa20af7c7c3bc7854c92184df2b86d1831f850
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/58543
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Alexandre Montplaisir [Tue, 20 Oct 2015 19:59:25 +0000 (15:59 -0400)]
tmf: Also dispose the inner table of TmfSimpleTableViewer
Calling dispose() on a TmfSimpleTableViewer does not clear it from its
view, because the inner Table object also needs to be disposed.
Change-Id: I3b05643e93223e4a4a24fd0f1ef5399905e05c4c
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/58777
Reviewed-by: Hudson CI
Bernd Hufmann [Tue, 29 Sep 2015 14:34:31 +0000 (10:34 -0400)]
lttng: Add possibility to remove latency listener from latency view
Also, store the listener in a ListenerList instead of a HashSet.
Change-Id: Ia8a7d272db34f09bfc457f74c98b5b928b6a82d7
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/56962
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Thu, 9 Jul 2015 17:14:04 +0000 (13:14 -0400)]
Remove duplicate groupId and versions in pom.xml files
If a groupId or plugin version is defined in a parent pom, it
is not necessary to repeat it in its children.
Change-Id: Idcff9cd6cd8154cc06aebde0152f1e7c23665b1b
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/51680
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Bernd Hufmann [Mon, 26 Oct 2015 13:18:05 +0000 (09:18 -0400)]
analysis: Change visibility modifiers in segment store analysis module
Change-Id: I3c16dceccf12a7124898116bee7dd6653e701fd0
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58925
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Tue, 13 Oct 2015 11:31:04 +0000 (07:31 -0400)]
lttng: fix NPE when segment selection is null
Change-Id: I1ac9dd36c0465b3261805f362c41a595a55f5df5
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58061
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Thu, 22 Oct 2015 17:40:50 +0000 (13:40 -0400)]
pcap: fix sonar warnings for null references
Change-Id: I8f1834d47609e3a4aeb2792f0fafe609ae7326fb
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58757
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Thu, 22 Oct 2015 17:34:53 +0000 (13:34 -0400)]
control: Unlock lock in all cases
Change-Id: Id49c4e71254020df236be3e61cd32cabf029175e
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58756
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Fri, 16 Oct 2015 18:06:04 +0000 (14:06 -0400)]
lttng: Provide tooltip for latency scatter chart
Change-Id: I42b32fa02eeb2bcbd1c4edb65c2d37fadb0e0cbe
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58544
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>
Alexandre Montplaisir [Fri, 4 Sep 2015 00:09:21 +0000 (20:09 -0400)]
lttng: Create TmfCallsite's from the debug info analysis
Change-Id: I95e9257be38ddc3f27a26a07626bf02f4630dff2
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/56190
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Bernd Hufmann [Fri, 23 Oct 2015 14:30:02 +0000 (10:30 -0400)]
analysis: Dispose the segment store when disposing the segment analysis
Change-Id: If157a9cff7db60f14d686868524eb0101fbec2e6
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58820
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Fri, 4 Sep 2015 22:34:08 +0000 (18:34 -0400)]
lttng: Add a LttngUstEvent class for LttngUstTraces
This class will support ITmfSourceLookup, now done through the
UST debug info analysis.
Change-Id: I904f7e8eb1811ab1f74ba95165d3d9167548ca3d
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/56192
Reviewed-by: Hudson CI
Alexandre Montplaisir [Fri, 4 Sep 2015 21:32:13 +0000 (17:32 -0400)]
ctf: Assign a CtfTmfEventFactory to each trace
A singleton factory means we cannot possibly extend CtfTmfEvent,
because the sole factory can only generate CtfTmfEvent objects.
To rectify this, we can turn the factory into a standard non-static
class, and have each trace take ownership of its event factory. This
means that sub-classes of CtfTmfTrace will be able to provide their
own extension of CtfTmfEventFactory if they want to generate other
event types.
Bug: 476679
Change-Id: I66e9cd49044edd2a77276d779d0f563a2e20d520
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/56191
Reviewed-by: Hudson CI
Bernd Hufmann [Mon, 19 Oct 2015 20:00:34 +0000 (16:00 -0400)]
analysis: Add abstract latency scatter chart view and use it in LTTng
Change-Id: I55d1e9ea96281f6f059c0310a50b3481f1294660
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58360
Reviewed-by: Hudson CI
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Thu, 15 Oct 2015 12:55:28 +0000 (08:55 -0400)]
lttng: Add latency statistics
Change-Id: I49863b3046717b3bba81443fc05305d78a80b302
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58262
Reviewed-by: Hudson CI
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Wed, 14 Oct 2015 11:06:09 +0000 (07:06 -0400)]
tmf: add comparator getter to ISegmentAspect interface
Change-Id: Ida38b31726076eaa5b3b465933033cf6e22b0ac7
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58261
Reviewed-by: Hudson CI
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Tue, 13 Oct 2015 18:17:11 +0000 (14:17 -0400)]
analysis: introduce abstract latency analysis table
Change-Id: I29a893ad5ae3cca3bdf8450921467249a8bfd5a9
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58093
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>
Bernd Hufmann [Tue, 6 Oct 2015 12:59:55 +0000 (08:59 -0400)]
lttng: provide a system call column for in Latency Table
It uses the newly created ISegmentAspect interface
Change-Id: I6bcdf746b396a4a79f10a22233d5cd994d778ad6
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57515
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Tue, 6 Oct 2015 12:58:33 +0000 (08:58 -0400)]
tmf: Introduce ISegmentAspect interfaces
Similar to ITmfEventAspect the interface ISegmentAspect provides means
to describe and retrieve the content beyond the data that is provided
by ISegment.
Change-Id: I67facc1212ba222c2012389c25e84d861d1fe190
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57514
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Tue, 6 Oct 2015 10:50:12 +0000 (06:50 -0400)]
analysis: add abstract latency analysis module and use it in LTTng
Change-Id: If1431cc9b31837a0a546ad952526f0e7b47d61f5
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57513
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Francis Giraldeau [Thu, 22 Oct 2015 01:23:57 +0000 (21:23 -0400)]
analysis: basic checks for acyclic property of TmfGraph
The TmfGraph is Directed Acyclic Graph (DAG). This patch adds three
checks:
* verify that a node is not linked to itself (this was possible because
the timestamp is greater or equal to itself).
* assert that getHead() will terminate in the special case where the
tail points to the head indirectly.
* fix infinite loop bug with TmfGraph.scanLineTraverse() with a
specific graph.
Change-Id: I68a3eaed2c1098df1547d1fba34c35bc1d038404
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Reviewed-on: https://git.eclipse.org/r/58682
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Fri, 16 Oct 2015 01:50:21 +0000 (21:50 -0400)]
analysis: add critical path icon
Change-Id: Id4926070cd1c4a36f6288f6cd1b1beaba6e93233
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58292
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Francis Giraldeau [Mon, 9 Feb 2015 03:52:06 +0000 (22:52 -0500)]
lttng: Add the link between the control flow view and the critical path
A parameter provider listens to the selection of the Control Flow View
and passes that selection as a parameter to the critical path analysis
Change-Id: I2866ffe27d715d76db6d401d97273a0020f25ab4
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/41482
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Francis Giraldeau [Mon, 3 Aug 2015 14:03:49 +0000 (10:03 -0400)]
analysis: Add the critical path view
Added in a new org.eclipse.tracecompass.analysis.graph.ui plugin
Change-Id: Id1f9691273c35e3e91b3ef57c000a37ee8bc4ad1
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/41481
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Marc-Andre Laperle [Wed, 21 Oct 2015 22:09:34 +0000 (18:09 -0400)]
lttng: Fix a package dependency in control.core
Change-Id: Ieab9d01b146073d16ff92c35cb9f303eac196142
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58675
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Marc-Andre Laperle [Wed, 21 Oct 2015 21:25:47 +0000 (17:25 -0400)]
ctf: Fix ExperimentBenchmark use of tracecompass-test-traces
The way it was used, it was trying to parse the root of the experiment
as a CTF trace.
Change-Id: If575a1ef26a9de56c99bcea21fc5e12ae38e0131
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/58673
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Wed, 8 Apr 2015 14:12:17 +0000 (10:12 -0400)]
tmf.ui: Move TmfEventsTable listeners from anonymous to nested classes
The listeners are pretty big and make the code harder to read. This patch
puts them together at the top of the file so that the constructor is simpler.
Change-Id: Ib0ab2d86afec4a26eb725556eb52e046b6b7850f
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/45464
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Matthew Khouzam [Tue, 23 Dec 2014 21:17:36 +0000 (16:17 -0500)]
ctf: cache file name
Java File.getName() gets a path and does a substring on it,
this operation is called on a per-event level and should be cached.
Change-Id: Ib8bd18df3bc0ac307f65fde615d151c832918a64
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/38740
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
This page took 0.045628 seconds and 5 git commands to generate.