Patrick Tasse [Fri, 23 Dec 2016 16:04:13 +0000 (11:04 -0500)]
tmf: Bug 509691: Changes to mutable trace context can be lost
The method TmfTraceManager.updateTraceContext() is now used to apply all
changes to a trace context under synchronization of the trace manager.
Change-Id: Id8cecd92c7f7c4c7203d965ce7fb36a6fed74626
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/87711
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Fri, 13 Jan 2017 18:13:22 +0000 (13:13 -0500)]
ss.test: Fix nullness warnings
Add a few @NonNull annotations to remove warnings on code that is never
null.
Change-Id: I92bf7e77fea99e465b31932d5eb2b1c1711233f6
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/88674
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Geneviève Bastien [Wed, 11 Jan 2017 21:34:52 +0000 (16:34 -0500)]
tmf: Update XML trace stub to use ns directly
Now that custom parsers support nanoseconds timestamps, the custom XML trace
stub does not need to convert the timestamps to nanoseconds anymore, it can
be defined directly in the XML trace definition.
Change-Id: I2791b350c303bb82df9f83f0cabbb663e1166a75
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/88513
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Patrick Tasse [Mon, 24 Oct 2016 14:18:13 +0000 (10:18 -0400)]
tmf: Deprecate TimeGraphCombo and support columns in TimeGraphViewer
The class TimeGraphCombo is deprecated and AbstractTimeGraphView method
getTimeGraphCombo() is deprecated and returns null.
The TimeGraphViewer supports columns and label provider with new methods
setTimeGraphLabelProvider(), setColumns(String[]), setWeights(int[]) and
getTree(). The tree contains no tree items and is only used to show
column headers and allow for column handling.
If no column names are specified, a single nameless column is used.
If no label provider is specified, the time graph entry's name and the
time graph presentation provider's item image is shown in the first
column.
The TimeGraphViewer is made to extend the Viewer class.
The TimeGraphControl supports auto resize of columns to grow columns at
the item text's preferred width. Auto resize turns off automatically the
first time that a user resizes a column to be narrower than its
preferred width.
The tree expand and collapse controls are changed to be triangle widgets
that are bound in size by the item's height.
The name space split line is made easier to hover with the mouse.
Double-clicking it snaps the name space to the right of the last visible
column text's preferred width. The split line can now be dragged even if
the viewer has no input model.
The TimeGraphControl methods getTree(), getLabelProvider() and
getItemBounds(ITimeGraphEntry) are added to help with UI testing.
Change-Id: I1061371b083499298da4f7c5640929e63ed651db
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/84135
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Jean-Christian Kouame [Thu, 12 Jan 2017 18:57:02 +0000 (13:57 -0500)]
releng: Update maven plugins version
Change-Id: I4b3c4598490ddf508051fc3e7b55185e581c385e
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/88588
Reviewed-by: Hudson CI
Reviewed-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Matthew Khouzam [Tue, 10 Jan 2017 19:26:59 +0000 (14:26 -0500)]
common.core: clean up ProcessUtils a bit
While the code does work, it has some issues that could potentially be
problems with the code if not used in the way it should.
* Extract magic number to constants
This is simple, it makes modifying the behaviour easier.
* Explicitly fill code blocks for catch blocks
Improve code readability. It makes it clear that it is intentional.
* Remove redundant modifier on interfaces
An interface is always static, there is no need to add the keyword
"static" to it.
* Add charsets to inputstream
This explicitly expects the default encoding (UTF8). However, this is
not the ideal solution, it would be much better, later to have an
interface with the charset passed in it. It can be used if we want to
launch a process in another machine.
Change-Id: I5ef919e724dc28d9e73c5276811af88b4de12f67
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/88407
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Patrick Tasse [Mon, 9 Jan 2017 21:41:02 +0000 (16:41 -0500)]
tmf: Bug 510144: IndexOutOfBoundsException in CallStackView
Get the quarks list before doing the full query, and when accessing the
full query result later, do a bounds check and redo the full query if
necessary.
Change-Id: Ica9ab6d414fb2286ad41deceff0b2be911be64d9
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/88304
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Geneviève Bastien [Fri, 25 Nov 2016 20:07:54 +0000 (15:07 -0500)]
ss: Move the safe byte buffer to datastore plugin
The unit tests also moved
Change-Id: I9118cb56bad032ce61065a5a81513c03a47fcc84
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/85799
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Alexandre Montplaisir [Wed, 14 Dec 2016 19:24:57 +0000 (14:24 -0500)]
ss: Change datastore.tests to a fragment plug-in
This will allow us to use the same package names in the test
classes as the runtime ones. That way tests will have access
to package-visible (default visibility methods).
This will remove the need of having tons of public methods just
for testing. Default-visibility methods (or even protected, where
needed) are arguably much safer, and won't risk confusing future
clients.
Change-Id: I0393ac9103f94afcb9a818cbe2e7c8dfcadcac18
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/87163
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Patrick Tasse [Wed, 21 Dec 2016 20:40:24 +0000 (15:40 -0500)]
tmf: Bug 509613: Trace Manager loses Linux trace context data
The trace context classes are updated to provide a builder that can be
extended and that is used to create a duplicate of a trace context with
some data that can be modified.
Change-Id: Icd3de5891cb37976795f485c837ac2e0e66f5501
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/87588
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Jean-Christian Kouame [Mon, 19 Dec 2016 16:10:48 +0000 (11:10 -0500)]
doc: Add futex analysis user guide
Add the futex doc and update the name of associated timegraph views.
Change-Id: Ia94940935d8516a1ff5a8e35bb5c283cb26d5a69
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87448
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Patrick Tasse [Mon, 12 Dec 2016 22:10:03 +0000 (17:10 -0500)]
tmf: Fix concurrent data access violations
Fixes CID 157631, 157632, 157633, 157634, 157635, 157636.
Change-Id: I04ef3f396ce8fcf68b18e384cbde296434640daa
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/86996
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Patrick Tasse [Thu, 15 Dec 2016 18:00:15 +0000 (13:00 -0500)]
tmf: Fix remaining TmfSignalManager leaking listeners
Change-Id: Ie85771e159ae0ff7bfc23047c5d76915dc3999f3
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/87263
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Mon, 19 Dec 2016 15:13:46 +0000 (10:13 -0500)]
tmf.ui: replace size()==0 with isEmpty()
Changes CustomTxtParserInputWizardPage to better handle checks. Size is
not guaranteed to be O(1) whereas isEmpty() is.
Change-Id: I5646fa6fd019bcfb14697d748170462c38fd099b
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87412
Reviewed-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Hudson CI
Patrick Tasse [Thu, 15 Dec 2016 15:30:15 +0000 (10:30 -0500)]
tmf: Fix unresponsive UI when mouse wheel zooming in TimeGraphViewer
When developing the time graph marker axis, it was noticed under Windows
that there was a delay in painting the marker axis when compared to the
time graph state area. It seems that if a new redraw request occurs
while the time graph state area is being painted, the following control
in the draw order (the marker axis) is not painted before until the next
paint event. The visual effect is that the time graph state area follows
the user input while the marker axis does not update until the user
input stops.
The solution found was to call Control.update() on those parts, which
would block the UI thread until all parts have been painted.
However, under Linux this can cause pending user inputs to be queued.
While the time graph is painting, rapid mouse wheel scrolling queues
many MouseEvent that each individually trigger blocking paint events.
This also prevents many mouse wheel scroll events to eventually combine
into triggering a single paint event. After quickly scrolling the mouse
wheel many times then stopping, the user can see the queued events
triggering a multitude of paint events that can last a few seconds even
while there is no user input. Any new user input needs to wait for all
the queued paint events to be completed before being processed.
The blocking calls to update() are therefore removed. This can cause a
temporary visual misalignment under Windows, but this effect is much
less noticeable under Linux.
Change-Id: I12d3d17eff0d8a443822083e0aa2113c3b23fedc
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/87246
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Jean-Christian Kouame [Fri, 16 Dec 2016 21:03:26 +0000 (16:03 -0500)]
tmf.xml: Fix xml timegraph resolution
When the window range is less than the display width, the resolution was
then equal to 0. The time event was not shown until the user change the
zoom level. Now we set the resolution value to 1 instead of 0.
Change-Id: I4f0ae39576c452f0bdfbaf23b836f3af758f0b91
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87339
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Jean-Christian Kouame [Fri, 16 Dec 2016 21:04:59 +0000 (16:04 -0500)]
tmf.xml: Add SWTBot XML timegraph view test
This test the loading and and viewing of xml trace. An
Xml view is created then validated.
Change-Id: I908771685129942eb044a2dbb497e377180a202e
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87340
Reviewed-by: Hudson CI
Matthew Khouzam [Thu, 15 Dec 2016 04:07:32 +0000 (23:07 -0500)]
linux.swtbot: add KernelMemoryUsage SWTBot test
This test will make sure range selection and the data chart
have sane values.
Change-Id: I69f348d49164991a1466a63ef4c2d536371171e4
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87193
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Matthew Khouzam [Wed, 14 Dec 2016 22:07:03 +0000 (17:07 -0500)]
linux.core: fix memory allocation issue.
The order field in page alloc/free say how many pages are allocated.
It was ignored, which gave false totals.
It is artificially limited to 2^62 page size, as larger would garanty
an overflow.
With this patch, memory overflows can occur much faster, but only on
systems which trace or report over Long.MAX_VALUE memory.
Change-Id: I5c805115cb3aaa5c6bc6146a6dc0db20f11854ef
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87181
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Geneviève Bastien [Tue, 4 Oct 2016 19:46:53 +0000 (15:46 -0400)]
tmf.ui: bug 503678 The ss explorer should not execute analyses
And because the state system does not execute the analyses, the state
systems may not exist when the tree is built. The hierarchy is thus
changed so that the analyses themselves appear as top-level items and
their children are the state system. This way, if the state system did
not exist at first and appears later, it will show when the view
refreshes (without having to refresh the whole view).
Change-Id: Icc750abcdfb27d74d486b9fdcf8b3e910310684f
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/83811
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Sonia Farrah [Wed, 4 May 2016 21:37:49 +0000 (17:37 -0400)]
tmf: Simplify boolean expression in handleEvent()
Change-Id: I59f992cf166050f80d8db50c2740596fc3135ff1
Signed-off-by: Sonia Farrah <sonia.farrah@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/72105
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Matthew Khouzam [Wed, 14 Dec 2016 21:30:06 +0000 (16:30 -0500)]
linux.core: add Order to fields of Kernel Analysis Event layout
This will allow getting the number of pages allocated.
Change-Id: I16a0aa00cb4b32f83c964cecc9543d2e0add4e85
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87180
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Matthew Khouzam [Tue, 8 Nov 2016 21:26:45 +0000 (16:26 -0500)]
tmf.ui: add FIXME to TmfSymbolProviderUpdatedSignal
Change-Id: I97672005feee87627f5613526176169eb7231b6b
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84703
Reviewed-by: Hudson CI
Reviewed-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Matthew Khouzam [Wed, 9 Nov 2016 04:29:06 +0000 (23:29 -0500)]
lttng.swtbot: add state system explorer SWTBot test
This test runs three passes, opens the view, re-opens the view
then re-opens the trace.
Future tests can look at the content of the state system. This
test enumerates the state systems available.
Change-Id: I3136bb4bfc802d958d63981d7ca358f44248f0be
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84711
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Matthew Khouzam [Thu, 15 Dec 2016 15:50:20 +0000 (10:50 -0500)]
tmf.ui: Avoid creating two Files for an archive import
Change-Id: I2f74d283d07da528101b3e44715afe6b2dc35971
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87248
Reviewed-by: Hudson CI
Reviewed-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Matthew Khouzam [Thu, 15 Dec 2016 15:46:31 +0000 (10:46 -0500)]
tmf.ui: Gracefully handle invalid tar import operations
A tar.gz file may have an invalid tar file in the gzip. The
getNextEntry would return null in such a case. This performs
that null check in order to validate the tar and avoid a user-
facing NPE.
Change-Id: I28ae7c787d77a0f1c9136264cfc678d7e52cda1a
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87247
Reviewed-by: Hudson CI
Reviewed-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Sonia Farrah [Tue, 3 May 2016 18:53:42 +0000 (14:53 -0400)]
tmf: Replace Array Loops to copy arrays with System.ArrayCopy
This yields a performance improvement as well as an increase
in legibility.
Change-Id: Ia7be6a54476a8263d8b5e99f203d7fc3cf374a1d
Signed-off-by: Sonia Farrah <sonia.farrah@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/71939
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Thu, 1 Dec 2016 18:24:40 +0000 (13:24 -0500)]
ss: replace a for loop with a foreach operation
And extract a constant
Change-Id: If4e65223ed4e9ce617eb9da3023db0294ede151e
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/86172
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-by: Hudson CI
Tested-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Matthew Khouzam [Wed, 14 Dec 2016 21:11:44 +0000 (16:11 -0500)]
tmf.ui: fix state system explorer ID
It now matches the plugin.xml's ID
Change-Id: I56d99447c9bd41ed10cfdf41aa285f33647f1236
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87179
Reviewed-by: Hudson CI
Reviewed-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Loïc Prieur-Drevon [Fri, 9 Dec 2016 18:34:04 +0000 (13:34 -0500)]
ss: remove linear component from node search.
Before, nodes had their intervals only sorted by end times,
which meant finding the first interval which started later
than time t implied a binary search for any interval ending
at time t, then a linear iteration to the first interval
ending before (inclusively) t.
We remove this component by ordering intervals by end times
then by start times, so that the binary search can directly
return the first interval ending before (inclusively) t.
Change-Id: I3e4fd02880d56360fc082023c08b92f8a6e4350e
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/86865
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Matthew Khouzam [Fri, 11 Nov 2016 02:26:22 +0000 (21:26 -0500)]
kernel.swtbot: test resources view filters
This tests the filter functionalities of the resources view.
Change-Id: Ifdf76ab29e843e1b8d91d6483373abf86e8763b7
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84849
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 [Thu, 1 Sep 2016 20:12:31 +0000 (16:12 -0400)]
ust: Add swtbot for callstack view responsiveness
This swtbot test navigates a trace with the callstack view opened. This
test is not meant to be run in the performance test suite, but enabling
JUL logging with it allows to have a reproducible series of UI operations
and analyze its output.
Change-Id: If4081a4581bb1c774c3643fc0bb0dd418c421379
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/80258
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Loïc Prieur-Drevon [Wed, 7 Dec 2016 17:38:22 +0000 (12:38 -0500)]
tmf.ui: fix Bug 479996 infinite loop
Opening an empty trace called queryFullStates an infinite number
of times resulting in OOM. Checking the arguments will avoid
this.
Bug 479996
Change-Id: I0ac29da7a883d154fffb5c4d24aa50db32796a6f
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/86862
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Marc-Andre Laperle [Tue, 13 Dec 2016 16:07:22 +0000 (11:07 -0500)]
Update Oomph setup with latest API baseline
Change-Id: Ic9e42bb1730464a7367636310b46beabdcfc0be9
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/87063
Bernd Hufmann [Tue, 29 Nov 2016 22:36:11 +0000 (23:36 +0100)]
xml: support any aspects to be used in XML analysis
Some traces have aspects to show special data in a column. With this
commit it is possible to use any aspect in the XML anlysis.
Change-Id: I9dedc910e9465ac94f82912dc9095e9d315b0882
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/86002
Reviewed-by: Hudson CI
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Marc-Andre Laperle [Thu, 8 Dec 2016 15:34:52 +0000 (10:34 -0500)]
Bump versions for 2.3.0 release
Change-Id: I78996a7b4dd3b1fd8fea308163f3ac1fb008e153
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/86733
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Marc-Andre Laperle [Thu, 8 Dec 2016 15:33:36 +0000 (10:33 -0500)]
releng: Add 2.2.0 baseline and fix API errors
Change-Id: Ibd9762699e2bd4616c34d6641d678f89eafafb65
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/86730
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Alexandre Montplaisir [Tue, 25 Oct 2016 00:32:36 +0000 (20:32 -0400)]
common: Add a ProcessUtils for external process launching
Consolidate the existing external-process-launching utility
methods into a new common class. Right now this includes the
FileOffsetMapper, which calls addr2line for debug-info symbol
resolution, as well as the LamiAnalysis class which calls
the LAMI analysis scripts.
Bug: 508406
Change-Id: I685fb461a93cd6726575b5df771233f37e423e5f
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/85973
Reviewed-by: Hudson CI
Patrick Tasse [Fri, 2 Dec 2016 22:40:53 +0000 (17:40 -0500)]
test: Make CtfTmfTrace stubs of the right class
Kernel and UST traces created with CtfTestTraceUtils were not of the
right type, which prevented the corresponding analyses to execute.
Add LttngKernelTestTraceUtils and LttngUstTestTraceUtils which create
similar stubs of the right type (LttngKernelTrace and LttngUstTrace).
Change-Id: I8a873b5ff7c558f68ffbc3141c66b8a35e6053c1
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/86297
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Patrick Tasse [Wed, 30 Nov 2016 22:57:23 +0000 (17:57 -0500)]
tmf: Fix TmfSignalManager leaking listeners
Change-Id: I28c23d2cf88a4280d2b81fd463a4c5fac6af5143
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/86107
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Thu, 27 Oct 2016 15:36:50 +0000 (11:36 -0400)]
tmf.core: add resource leak detector
If the signal manager is disposed, it should be empty. Anything still
there is a resource leak.
The output looks like this:
Resource leak: org.eclipse.tracecompass(...) was not deregistered.
org.eclipse.tracecompass.tmf.c(...)TmfSignalManager.java:77)
(...)
Change-Id: Ie3950604b743423d2f7e136def5e7ef7077f629d
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84030
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Geneviève Bastien [Thu, 24 Nov 2016 20:53:26 +0000 (15:53 -0500)]
segstore: Add assert on item count for iteration benchmarks
This patch adds asserts on the item count when iterating the segment
store in the benchmark. It makes sure that all iterations actually
iterate over all the elements of the segment and does not give a false
impression of performance if there is a bug in the iterator.
Also remove the System.gc() that are not useful to the test
Change-Id: I87acbb264703f06a166239655f3911697c17126e
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/85724
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Marc-Andre Laperle [Wed, 7 Dec 2016 04:47:07 +0000 (23:47 -0500)]
tmf: Add missing icons in XML analysis
Change-Id: I6fd068cb5214a4f482f78e934de007dd334fcd2f
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/86561
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Alexandre Montplaisir [Thu, 1 Dec 2016 21:28:42 +0000 (16:28 -0500)]
os.linux: Manage sched_waking events
LTTng 2.8 (and kernels around that time) introduced the
sched_waking event. This event has the same fields as the
sched_wakeup one, and should cause the same state changes.
Since the linux-generic KernelStateProvider has no knowledge
of specific LTTng event layouts, we have to bring this
definition up into the base interface. Older traces simply
won't have any of these events.
Change-Id: Idf9eaf1ecba0346b22283adfbe71ad56d52b045f
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/85804
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Loïc Prieur-Drevon [Wed, 30 Nov 2016 07:21:32 +0000 (02:21 -0500)]
tmf.ui: use binary search when adding child to TimeGraphEntry
This brings the search time complexity to O(log(n)) down from
O(n).
Change-Id: I5b97c0f341bfe712d1bb8f9fd95cffc9ccd8686c
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/86000
Reviewed-by: Hudson CI
Reviewed-by: Jonathan Rajotte Julien <jonathan.rajotte-julien@efficios.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Matthew Khouzam [Thu, 17 Nov 2016 03:26:25 +0000 (22:26 -0500)]
tmf.swtbot: Stabilize and enhance TestImportExportPackageWizard
The test would fail if ran 1000 times in a row. File#createTempFile
appears to be the culprit.
Remove a useless "waitForJobs()" should accelerate by 500ms.
Add tests of the output size, removes two manual tests from Trace
Compass test spec.
Change-Id: Ia8bc3a517752a121874cd559d192fa0378221716
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/85191
Reviewed-by: Hudson CI
Patrick Tasse [Sat, 3 Dec 2016 13:40:05 +0000 (08:40 -0500)]
common: Fix @since tags in DecimalUnitFormat and DataSizeWithUnitFormat
Change-Id: I7bb99c6f966cdd4e66138c71e0806f8caf9e68ea
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/86303
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Patrick Tasse [Thu, 1 Dec 2016 22:45:23 +0000 (17:45 -0500)]
common: Support parsing in DataSizeWithUnitFormat
This allows parsing strings which represent a number with an optional
unit and binary prefix, such as "1 kB" (1024), "8 B" (8), "1.5M"
(
1572864), "10 Gb" (
10737418240), etc.
Change-Id: Iaaa222fb57b930f54ca9f0961b8240fe7b8e02ef
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/86185
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Patrick Tasse [Wed, 30 Nov 2016 20:27:51 +0000 (15:27 -0500)]
common: Support parsing in DecimalUnitFormat
This allows parsing strings which represent a number with an optional
unit and SI-prefix, such as "1.2 kb" (1200), "1.2" (1.2), "1.2 m"
(0.0012), "12 ns" (0.
000000012), etc.
Change-Id: Icd00870903bd22da6d322237ab1302276072217a
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/86097
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Tue, 29 Nov 2016 21:55:46 +0000 (16:55 -0500)]
common: Add StreamUtils method to wrap an Iterator into a Stream
Change-Id: Ib3027589bdfe4b328596dabdd572e77ca434dd68
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/85990
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Marc-Andre Laperle [Sun, 27 Nov 2016 00:54:51 +0000 (01:54 +0100)]
tmf: Add extension point to add custom trace types to plugins
This new extension point is used to contribute custom Text or XML
trace types so that they are available automatically without the need
to be imported by the user manually.
Change-Id: Iff481d659626a8ed8d756d333ec6a240b4316b49
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85831
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Loïc Prieur-Drevon [Wed, 30 Nov 2016 18:53:24 +0000 (13:53 -0500)]
ss: Change getIteratorOverQuark to a public class
This patch allows iteration to go backwards and forwards,
introduces a unit test and uses the new iterator for:
KernelThreadInformationProvider.getExecutableName
Change-Id: I41f35e30bd1984662f55b8809e2497a6c19d5468
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/85798
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Jonathan Rajotte [Wed, 30 Nov 2016 21:03:58 +0000 (16:03 -0500)]
analysis.lami: Fix internal signaling with several views on the same report
Use the parent LAMI tab page instance as the key for internal signaling.
The signal key is designed to restrict the signal to interested objects.
It was based on the hash of the immutable fResultTable field. Since for multiple
view of the same report the fResultTable is the same object, signals were
processed by objects that were not concerned. Using the result table
instance/hash as a key is not sufficient.
Bug: 495322
Change-Id: Ifba2056b10cd254d1a8c6c030c4b7263463ac834
Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/86103
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Wed, 23 Nov 2016 20:23:49 +0000 (15:23 -0500)]
common: Annotate java.util.Iterator
Just add an annotation to forEachRemaining(). The goal is to have
_some_ annotation in there so that the next() method makes use
of the declared type, and we don't get the "this type doesn't
seem designed with null annotations in mind" warning.
Change-Id: Idbd8b9e6e62ddd876f390280edb3d2b46d264602
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/85622
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Geneviève Bastien [Thu, 27 Oct 2016 16:28:02 +0000 (12:28 -0400)]
tmf.core: Add unit tests for state system re-read
This adds a unit test for a state system with full backend where the
results are re-read from a previously existing file.
Change-Id: Iab5fab6c6282f2411d2e4a0d3b0da3824653cb36
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/84036
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Geneviève Bastien [Thu, 27 Oct 2016 16:26:23 +0000 (12:26 -0400)]
tmf.core: Throw exception when no state system assigned
Instead of just logging an error that will happen for each event and
willl be very verbose, it should throw an exception.
Change-Id: Iaedcf461ffa5e8d9ac8abd4f11e98e9f70f6259e
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/84035
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Geneviève Bastien [Thu, 27 Oct 2016 16:24:27 +0000 (12:24 -0400)]
tmf.ctf: Delete supplementary files after tests
This way, each test has a reproducible state: without supplementary
file, otherwise some tests were using the supplementary files from
previous runs. It found bugs in other patches, but they were not related
to this.
Change-Id: I92cabd22d2193e1cd0fe43ce1f32be0e05b9fd20
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/84034
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Tue, 22 Nov 2016 20:57:43 +0000 (15:57 -0500)]
linux.core: replace ""; //NLS... with StringUtils.EMPTY
Change-Id: If632867af96536b756d9e0c845b663b2ad9ffe0a
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85551
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Alexandre Montplaisir [Wed, 23 Nov 2016 20:25:03 +0000 (15:25 -0500)]
common: Fix compilation error with recent JDT
An error pops up with the latest integration build. The type
inference seems weird, but we can specify the expected type
manually.
Change-Id: I83da0970e87991138ca8afbb776ce22ab5d46b67
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/85623
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Thu, 17 Nov 2016 03:22:36 +0000 (22:22 -0500)]
tmf.swtbot: make openTrace fail on failures not just exceptions
Check TmfOpenTraceHelper#openTraceFromPath for a failure. This fails
faster.
Change-Id: I8221249fd802cb2284c78610499f1303e0477811
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85190
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Jean-Christian Kouame [Wed, 9 Nov 2016 21:18:40 +0000 (16:18 -0500)]
lttng.xml: Add xml file for futex analysis
Change-Id: Ia9579f1db51333fed6f688a879770b1f792f3ae3
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84481
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Matthew Khouzam [Thu, 24 Nov 2016 20:14:19 +0000 (15:14 -0500)]
common.core: make TraceCompassLog call the same path on all getters
This allows better following of Loggers.
Change-Id: Ifd4660d41e9f76730590662900a0e691b95d1d9f
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85720
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Marc-Andre Laperle [Sun, 27 Nov 2016 00:04:11 +0000 (01:04 +0100)]
tmf: Deprecate and remove a few things not used in Trace type code
Some methods are never called and are achievable through other means
anyway.
Change-Id: Ifdc248f47b277cec6c00f0bc32a4df4ae862512e
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85830
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Jonathan Rajotte [Wed, 23 Nov 2016 22:31:57 +0000 (17:31 -0500)]
tmf.ui.swtbot.tests: Fix javadoc warning
Warning introduced by
2d594808e2906222ad806151e020c1bd6527f9e7
Change-Id: Iee04992bfa4bf7f95324e37c39bb24f8a73a045a
Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
Reviewed-on: https://git.eclipse.org/r/85638
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Loïc Prieur-Drevon [Fri, 4 Nov 2016 18:47:48 +0000 (14:47 -0400)]
segstore: use eclipse Performance for SegmentStore perf test
Before this patch, the segment store performance would only
print the benchmark results in the stdout after one run.
This patch uses the eclipse Performance plugin to run the
tests 10 times and provide more detailed output.
Change-Id: I98738c52f5c7f1bb81f152b38b97595056d8e60c
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/84496
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Tue, 22 Nov 2016 20:52:24 +0000 (15:52 -0500)]
common: annotate as @NonNull Apache StringUtils#EMPTY_STRING
Change-Id: If34ee46e029ad37775786f8c3ea395ab9b7fc7cc
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85549
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Matthew Khouzam [Wed, 9 Nov 2016 03:14:52 +0000 (22:14 -0500)]
lttng.swtbot: Add test for closing and opening histogram
Change-Id: I928e7a8f0eb6ad8fbc81d0f756079836031bce16
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84710
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 [Tue, 22 Nov 2016 20:46:46 +0000 (15:46 -0500)]
graph.ui: Add Apache Commons Lang3 to use EmptyString
This removes some ""; //NLS...
Change-Id: I3217d3d9bacc8123507ebd25c1095e3bb14a32e6
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85548
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Patrick Tasse [Thu, 24 Nov 2016 17:09:47 +0000 (12:09 -0500)]
swtbot: Fix selection event for SWTBotSash with SWT.HORIZONTAL style
Change-Id: Ie3d625881842864c9c7797fc6e06e236bce634ae
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/85712
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Loïc Prieur-Drevon [Mon, 18 Jul 2016 17:39:37 +0000 (13:39 -0400)]
ss: introduce getIteratorOverQuark
This patch adds a method which returns an iterator over the
intervals of an attribute and a basic unit test.
Change-Id: I8d9d59b1cdf1a80477f24e2849c777c109afbc10
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/77546
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Marc-Andre Laperle [Mon, 17 Oct 2016 21:57:24 +0000 (17:57 -0400)]
Do not report API error when adding a default method
This is a new setting in PDE that allows us to add default methods
without having an error.
See bug 501745
Change-Id: I777ad7336e6f559bec91854a9e84bc904b624433
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83398
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Matthew Khouzam [Thu, 3 Nov 2016 14:17:56 +0000 (10:17 -0400)]
timing.core: Add more unit tests to ICalledFunctions
Change-Id: I7603482bef8b03b7326a2a74ce1f2a4e4c7e790b
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84415
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, 21 Nov 2016 16:36:07 +0000 (11:36 -0500)]
lttng.swtbot: add test to open and re-open time graph views.
This removes 4 manual tests. ControlFlowView 1.3 and 1.4 and
ResourcesView 1.3 and 1.4.
Change-Id: I28c4b0bae9d5e6a846ac619c22e97d1df8ad47a8
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85429
Reviewed-by: Jonathan Rajotte Julien <jonathan.rajotte-julien@efficios.com>
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Matthew Khouzam [Mon, 21 Nov 2016 18:39:11 +0000 (13:39 -0500)]
lttng.swtbot: remove fViewBot from ResourcesViewTest
This allows getViewBot to work even if the view was closed
and re-opened.
Change-Id: I7b999f83dfea74785244e282748ed7975ef67d34
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85428
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, 17 Nov 2016 19:32:39 +0000 (14:32 -0500)]
releng: replace some "" with StringUtils.EMPTY
Change-Id: Ifcd84861dec27ee515f4a216cbdb22e09f6f3e9b
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85233
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Matthew Khouzam [Mon, 21 Nov 2016 16:34:03 +0000 (11:34 -0500)]
lttng.swtbot: remove field fViewBot
This makes the test more likely to fail in a controlled way but also
removes the cache which can need to be updated.
Change-Id: Ifd80b151edc4022100d2998426466ed1780ff3d9
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85421
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Marc-Andre Laperle [Thu, 17 Nov 2016 21:54:26 +0000 (16:54 -0500)]
tmf: Add missing schema in build.properties
Otherwise, adding the extension without full source code throws an
exception because of the missing schema.
Change-Id: I325f0ca070d73c87b1a789c7691a9bd244cf51c3
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85240
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Geneviève Bastien [Mon, 31 Oct 2016 17:53:27 +0000 (13:53 -0400)]
ss: Add a new plugins for data stores and tests
This plugin will contain among other things the base history tree structure
Change-Id: I5919ab01567525b4833cfafbf4c23423749b2de7
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/84257
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Patrick Tasse [Wed, 16 Nov 2016 22:48:48 +0000 (17:48 -0500)]
swtbot: Add SWTBotTimeGraph and update SWTBotSash
The new classes SWTBotTimeGraph and SWTBotTimeGraphEntry are added to
better support UI testing of the time graph widget.
The class SWTBotSash is updated to simulate drag using SWT events.
SWTBot tests are updated accordingly.
Change-Id: Ic0800e73b85c624864825ab3f63e1bfadbacf945
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/85178
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Wed, 16 Nov 2016 23:30:54 +0000 (18:30 -0500)]
Put analysis.timing.swtbot.tests inside the relevant profile
Build fails at the moment with the -Dskip-automated-ui-tests flag.
Change-Id: I68907d142b48149651612b61bbc498394f9d7966
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/85185
Reviewed-by: Hudson CI
Reviewed-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Patrick Tasse [Wed, 19 Oct 2016 15:21:34 +0000 (11:21 -0400)]
tmf: Fix rounding and range issues in periodic marker source
When the requested time range and reference are far apart, the floating
point calculations lose precision and can produce marker times that are
not multiples of the period but instead clamped to the nearest multiple
of a power of 2.
The calculations are modified to adjust the reference to a point that is
closer to the request time range, by converting the period to a fraction
representation, if possible, and using exact integer calculations.
Some rounding calculations are corrected to apply to smaller numbers.
When using alternate shading periodic markers, the first marker returned
could overlap with the start time. If this partially visible marker was
selected by the user, the 'previous marker' action would not find any
previous marker. The code is adjusted to make sure that the first marker
returned is the first fully non-visible previous marker.
Change-Id: I1a8ad36441bf44a735621ffe4361e73ee6faaa60
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/83545
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Patrick Tasse [Thu, 10 Nov 2016 20:38:52 +0000 (15:38 -0500)]
tmf: Use DecimalUnitFormat for time graph selection range less than 1s
Change-Id: Ia71c26f42f89f09621b57295768a38565f0f9247
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/84838
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Tue, 15 Nov 2016 20:51:09 +0000 (15:51 -0500)]
tmf.ui: Remove redundant annotations in ITimeGraphPersentationProvider2
Change-Id: Idb7c863765fd7038818da9fe08ca2ac541e19a5b
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/85093
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Alexandre Montplaisir [Tue, 15 Nov 2016 20:07:18 +0000 (15:07 -0500)]
common: Add external annotations to Guava Cache and LoadingCache
getIfPresent() should return a @Nullable V. Elsewhere it should
follow the declaration type.
Some other methods (asMap() etc.) are known to really return a
possibly empty map, so those can be marked @NonNull.
Change-Id: I5466151977ee639535b1f818ef9e3c14ba122f7e
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/85082
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Matthew Khouzam [Mon, 7 Nov 2016 04:10:39 +0000 (23:10 -0500)]
timing.core: Make statistics analysis support different analyses
* Make the "getIntersectingElements" not blind cast into Collection
they are Iterables
* Store the SegmentStoreProvider as a SegmentStoreProvider instead of
an IAnalysisModule. It is only used as an AnalysisModule at one point
in the code.
Change-Id: Ie0e81ecd69ceb4634f8e7a4236310efe02f74bea
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84544
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, 15 Nov 2016 15:33:10 +0000 (10:33 -0500)]
tmf.xml: Fix children in time graph view
Commit
7c094b3e0710bef7c69040d2d544f4e8781eaa4a changed the behavior of
the addChild(int, entry) of TimeGraphEntry. The parent should not be set
before calling this method anymore.
Change-Id: Ie716e9a5502f445dd6ef755adce42688ed000bbd
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/85065
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, 24 Oct 2016 15:39:25 +0000 (11:39 -0400)]
timing.core/ui/tests: add function density test
This is a unit test for the function density view.
Change-Id: I296d6f33fd4b81b99363cf3de5a280fb9ef04c68
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83794
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Matthew Khouzam [Tue, 8 Nov 2016 01:58:20 +0000 (20:58 -0500)]
common.test: Fix Todo and warning in BBQ tests
Now that we're in Java8, we can fix the todo to support
List<@NonNull...> and also properly annotate the output
of the stream.
Change-Id: I1584164748beed859a4b2883435c537aa97aa1ff
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84633
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Matthew Khouzam [Tue, 8 Nov 2016 01:55:36 +0000 (20:55 -0500)]
common.core: make BufferedBlockingQueue validate inputs
Also tests that inputs are validated
Change-Id: I6cc39a0851e0ccf8ada0dd473586c613387ffc68
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/47398
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Marc-Andre Laperle [Fri, 11 Nov 2016 05:56:43 +0000 (00:56 -0500)]
tmf: Bug 507415: Fix some random file wrongly seen as Tar
Change-Id: Iff3ee2a5cad352f6b1f658e4da462dfe45e21418
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84852
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Hudson CI
Bernd Hufmann [Fri, 11 Nov 2016 03:02:11 +0000 (22:02 -0500)]
Consider time scale when creating lost events for CTF
Change-Id: Ifd1523e6b23652c609e9774e2c9b7ce1ef67bac1
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84850
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Patrick Tasse [Thu, 10 Nov 2016 14:57:48 +0000 (09:57 -0500)]
ctf: Fix clock-related Javadoc in CTFTrace
Change-Id: Ibde44bbf044424dcf32981eb9e4dc7557f6f1925
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/84821
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Patrick Tasse [Wed, 26 Oct 2016 18:20:33 +0000 (14:20 -0400)]
tmf: Bug 496504: Fix duplicate child entries in Control Flow view
This could happen while building the state system if a child process is
first discovered in an early iteration and its parent process is found
in a second iteration. Removing the child from its old parent trace
entry would break the link to the new parent that was just set. On a
third iteration it would then add the child to the same parent again.
The TimeGraphEntry is changed to be more robust in its handling of the
parent-child relations.
Change-Id: I99743d80484d335180459df144067fc4494b8052
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/83961
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Marc-Andre Laperle [Thu, 10 Nov 2016 17:46:08 +0000 (12:46 -0500)]
releng: Add missing test plugin to test feature
Otherwise, if you use the feature, it will complain about a missing
dependency.
Change-Id: Ia60493453a7a17bcf209b1a90609f663ec68b39f
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84828
Reviewed-by: Hudson CI
Patrick Tasse [Wed, 9 Nov 2016 21:47:28 +0000 (16:47 -0500)]
tmf: Fix pessimistic null analysis of wildcard parameter
Unconstrained generic parameter is assumed to be nullable and its
unchecked access is reported as an error with Eclipse 4.7.
Change-Id: Ib7ac039a2d407fbfef1acfb0802043353668deb9
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/84768
Reviewed-by: Hudson CI
Reviewed-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-André Laperle <marc-andre.laperle@ericsson.com>
Patrick Tasse [Fri, 4 Nov 2016 14:50:54 +0000 (10:50 -0400)]
tmf: Fix time alignment not occurring when switching perspective
When switching perspective, the view can be resized just before it is
made visible. Queue the time alignment to ensure it occurs when the
parent composite is visible.
Change-Id: I6f4779bf0be59de703fadaacf549b28e823bd7c7
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/84474
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Patrick Tasse [Thu, 3 Nov 2016 18:47:49 +0000 (14:47 -0400)]
tmf: Fix MenuDetect handling in TimeGraphControl
- If the menu is invoked with the mouse over the name space, then there
is no need to delay the menu detect handling until mouseUp, since the
right mouse button isn't being used to zoom
- If it is a pending menu detect callback on mouseUp, and another drag
operation is in progress or it is a drag zoom outside of the margin,
then make sure the menu is not made visible by setting e.doit to false.
- If the menu is invoked with the mouse over the name space, or with the
keyboard, if there is any drag operation in progress make sure the menu
is not made visible by setting e.doit to false.
- If the menu detect is handled immediately (not after a pending menu
detect on mouseUp) then there is no need to set the menu visible
programmatically since the platform will do it after the MenuDetect
event handling when e.doit is true.
Change-Id: I0d76b9908782fc853a065567a6e907da5e2d9c86
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/84434
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Sun, 16 Oct 2016 10:47:12 +0000 (06:47 -0400)]
tmf.ui: Introduce TmfSymbolMapUpdatedSignal
This will allow all views depending on the symbol map to update properly.
Change-Id: I48dcb4d16cd14cf6a6572385f95ce699aeb9fd09
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83319
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Geneviève Bastien [Mon, 7 Nov 2016 18:41:40 +0000 (13:41 -0500)]
tmf.swtbot: Add a failure message to 'isTableCellFilled'
This patch adds a failure message for the isTableCellFilled SWTbot
condition to tell what text was in the cell, if any and what was
expected
Change-Id: I8a3a8290af9a5a37c08ecf48cece6d67b1c96f44
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/84609
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Geneviève Bastien [Thu, 22 Sep 2016 01:45:28 +0000 (21:45 -0400)]
os.linux: Remove latency table view and use the one from timing
The latency table is replaced by the timing's concrete table view with
secondary ID.
The SWTbot tests needed to be udpated as well. A utils method was added
to open views with secondary ids.
Change-Id: Ia71ece1f14f7f0032cef9bd219dba45d7cd1e242
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/80963
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
This page took 0.045789 seconds and 5 git commands to generate.