deliverable/tracecompass.git
7 years agoos.linux & tmf.ui: introduce dynamic filter for cfv: Active threads and Threads on... dynamic-active-cpu-filters
Jonathan Rajotte [Fri, 18 Nov 2016 00:16:40 +0000 (19:16 -0500)] 
os.linux & tmf.ui: introduce dynamic filter for cfv: Active threads and Threads on CPUs

A dynamic filter is defined as a filter that is calculated on each
update of the visual time range.

Users can now apply dynamic filters on the Control Flow view. A new
sub menu of the view menu allows users to enable and configure those
dynamic filters. For now, one dynamic filter is available: "Active
Threads Only". This filter has two configurable options:

    - Show all threads considered active.
    - Show threads running on a selection of CPUs.

A thread is considered inactive if its state is one of the following:

    - non-existing
    - unknown
    - wait (blocked)
    - wait (unknown)

Otherwise the thread is considered active.

The "Show threads running on a selection of CPUs" filter shows
threads that were currently running or waiting on the run queue for the
ranges of CPUs supplied to the filter during the visible time range of
the Control Flow view.

Change-Id: Ib58fe09aabd070e4db9270955f86737c36f309ca
Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
7 years agoos.linux: use Current_cpu_rq for control flow view tool tip
Jonathan Rajotte [Fri, 25 Nov 2016 21:11:06 +0000 (16:11 -0500)] 
os.linux: use Current_cpu_rq for control flow view tool tip

Enable the displaying on hovering of the CPU run queue information for
thread in the WAIT_FOR_CPU state.

Using Current_cpu_rq is also valid for currently running thread.

Move the addition of the CPU tool tip attribute to
getEventHoverToolTipInfo so the presented information reflect the
reality at the hovering time.

Change-Id: Ib01cfd883e5ebc4b3f24143bebaf0ad5da74ff69
Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
7 years agoos.linux: Add some util methods to check thread state
Alexandre Montplaisir [Wed, 16 Nov 2016 20:52:35 +0000 (15:52 -0500)] 
os.linux: Add some util methods to check thread state

This can be used by view filters to show either only active
threads, or threads on a given subset of CPUs.

Change-Id: I3887868d3c6c97009f772c1088274eda2bfbdd65
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
7 years agoos.linux: Handle cpu field in statedump events
Alexandre Montplaisir [Fri, 25 Nov 2016 03:51:17 +0000 (22:51 -0500)] 
os.linux: Handle cpu field in statedump events

LTTng 2.10 will introduce a new "cpu" field in
statedump_process_state events, which will indicate
on which CPU's run queue the thread is currently
scheduled.

Change-Id: I521aea89d5802c5bad4881ab4172c8a7211285b4
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
7 years agoos.linux: Correctly model each CPU's run queue
Alexandre Montplaisir [Wed, 16 Nov 2016 20:50:11 +0000 (15:50 -0500)] 
os.linux: Correctly model each CPU's run queue

Add a new attribute to the kernel state system:

THREADS
 +--1000
     +--Current_cpu_rq

Which will track on which CPU's run queue this thread
is currently located. Unlike the cpu -> Current_thread
attribute, several threads can be present on the same
CPU's run queue at the same time.

This will allow for example tracking on which CPU a
thread is expecting to go when it is in the WAIT_FOR_CPU
state.

This requires tweaks to the sched_switch, sched_wakeup
and process_fork handlers, as well as adding a new handler
for the sched_migrate_task event (which simply moves a
non-running thread from one run queue to another).

Change-Id: I6e0386214df195eb6389cec87f2e6a42fdc0691d
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
7 years agolttng: Also apply the wakeup handler to sched_waking
Alexandre Montplaisir [Fri, 25 Nov 2016 20:43:14 +0000 (15:43 -0500)] 
lttng: Also apply the wakeup handler to sched_waking

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.

Change-Id: Idf9eaf1ecba0346b22283adfbe71ad56d52b045f
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
7 years agotmf.ui.swtbot.tests: Fix javadoc warning
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>
7 years agosegstore: use eclipse Performance for SegmentStore perf test
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>
7 years agocommon: annotate as @NonNull Apache StringUtils#EMPTY_STRING
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>
7 years agolttng.swtbot: Add test for closing and opening histogram
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>
7 years agograph.ui: Add Apache Commons Lang3 to use EmptyString
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>
7 years agoswtbot: Fix selection event for SWTBotSash with SWT.HORIZONTAL style
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>
7 years agoss: introduce getIteratorOverQuark
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
7 years agoDo not report API error when adding a default method
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>
7 years agotiming.core: Add more unit tests to ICalledFunctions
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>
7 years agolttng.swtbot: add test to open and re-open time graph views.
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>
7 years agolttng.swtbot: remove fViewBot from ResourcesViewTest
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>
7 years agoreleng: replace some "" with StringUtils.EMPTY
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>
7 years agolttng.swtbot: remove field fViewBot
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>
7 years agotmf: Add missing schema in build.properties
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>
7 years agoss: Add a new plugins for data stores and tests
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
7 years agoswtbot: Add SWTBotTimeGraph and update SWTBotSash
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>
7 years agoPut analysis.timing.swtbot.tests inside the relevant profile
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>
7 years agotmf: Fix rounding and range issues in periodic marker source
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
7 years agotmf: Use DecimalUnitFormat for time graph selection range less than 1s
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>
7 years agotmf.ui: Remove redundant annotations in ITimeGraphPersentationProvider2
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>
7 years agocommon: Add external annotations to Guava Cache and LoadingCache
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
7 years agotiming.core: Make statistics analysis support different analyses
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>
7 years agotmf.xml: Fix children in time graph view
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>
7 years agotiming.core/ui/tests: add function density test
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>
7 years agocommon.test: Fix Todo and warning in BBQ tests
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>
7 years agocommon.core: make BufferedBlockingQueue validate inputs
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>
7 years agotmf: Bug 507415: Fix some random file wrongly seen as Tar
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
7 years agoConsider time scale when creating lost events for CTF
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>
7 years agoctf: Fix clock-related Javadoc in CTFTrace
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
7 years agotmf: Bug 496504: Fix duplicate child entries in Control Flow view
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>
7 years agoreleng: Add missing test plugin to test feature
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
7 years agotmf: Fix pessimistic null analysis of wildcard parameter
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>
7 years agotmf: Fix time alignment not occurring when switching perspective
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>
7 years agotmf: Fix MenuDetect handling in TimeGraphControl
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>
7 years agotmf.ui: Introduce TmfSymbolMapUpdatedSignal
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>
7 years agotmf.swtbot: Add a failure message to 'isTableCellFilled'
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>
7 years agoos.linux: Remove latency table view and use the one from timing
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>
7 years agotiming.swtbot: add Generic SegmentTable tests
Matthew Khouzam [Thu, 27 Oct 2016 00:35:25 +0000 (20:35 -0400)] 
timing.swtbot: add Generic SegmentTable tests

This makes table tests truly extendible. In order to implement
custom tests, one must override getSegmentStoreProvider with the
analysis to test, createSegment to return the right type of
segment and the test validation they want.

The System Call table view is an example implementation.

Change-Id: Id77c8afce3a5296d066d94885e8f54ddcf719de0
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84446
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
7 years agoss: Move selectNextChildren to CoreNode and return sequenceNumber
Loïc Prieur-Drevon [Thu, 20 Oct 2016 18:55:05 +0000 (14:55 -0400)] 
ss: Move selectNextChildren to CoreNode and return sequenceNumber

SelectNextChildren was only called on CoreNodes to return their
children.
Returning the SequenceNumber will allow overlapping trees to read
the nodes from disk when necessary and limit the footprint of the
queue for large queries.

Change-Id: I14ac3909bf7fb90490479d7b79cfa011c2dcb56c
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/83647
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
7 years agoss: Add a JUL statement for HT backend file size
Geneviève Bastien [Thu, 6 Oct 2016 20:59:21 +0000 (16:59 -0400)] 
ss: Add a JUL statement for HT backend file size

Change-Id: Ib8965540a7456e8c303791df783733080760cb15
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/83822
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
7 years agoreleng: ProjectExplorer tests stability improvement
Marc-Andre Laperle [Wed, 26 Oct 2016 15:01:45 +0000 (11:01 -0400)] 
releng: ProjectExplorer tests stability improvement

Resuse code for testing events editor. It contains a workaround for
loss of focus that was not applied in one of the classes.

Change-Id: I68b70013060c23daad88cb13abf20c511032f057
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83944
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
7 years agoBump versions for 2.2.0 release
Marc-Andre Laperle [Fri, 21 Oct 2016 15:03:31 +0000 (11:03 -0400)] 
Bump versions for 2.2.0 release

Change-Id: I1ac6abb99addf04aca2eb2eeaed919807e5e4b70
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83700
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
7 years agoUpdate tycho-pomless to 0.26.0
Marc-Andre Laperle [Fri, 21 Oct 2016 15:34:35 +0000 (11:34 -0400)] 
Update tycho-pomless to 0.26.0

We forgot to update it the last few times. There's nothing really new
but the version should be consistent with the rest of Tycho that we
are using.

Change-Id: I50ee002d9b78426d3efbe6b656583c7738777da7
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83705
Reviewed-by: Hudson CI
7 years agoOutput workspace log to console during tests
Marc-Andre Laperle [Mon, 24 Oct 2016 17:47:26 +0000 (13:47 -0400)] 
Output workspace log to console during tests

This patch adds -consoleLog so that we see easily the workspace logs
in the console output. It's very easy to miss problems that were
outputted in the workspace log because you have to find the log in
target/work/data/.log.

One disadvantage is that it always outputs:
"Could not resolve module: org.eclipse.compare.win32"

Change-Id: I021f30713485fc152465bd81f6ac9baf23ef5f16
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83812
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
7 years agoreleng: Update staging target with Oxygen M3 sites
Marc-Andre Laperle [Wed, 2 Nov 2016 15:59:12 +0000 (11:59 -0400)] 
releng: Update staging target with Oxygen M3 sites

Change-Id: I1e8e9ff19928910735aeabbe5a546a601fee7059
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84357
Reviewed-by: Hudson CI
7 years agolttng: Fix test when in the presence of long workspace path
Marc-Andre Laperle [Sat, 22 Oct 2016 05:08:13 +0000 (01:08 -0400)] 
lttng: Fix test when in the presence of long workspace path

LTTngServiceMiTest validated against the old MI XSD which doesn't
allow sessions paths longer than 255 characters. When the workspace
of the test is too long, this fails. The test doesn't really need
a variable and can be hard-coded to any path. So this patch removes
the use of the workspace variable for this case.

See also bug 490557.

Change-Id: I382da2449c485e828c62d4ef73e3baf75e438374
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83737
Reviewed-by: Hudson CI
Reviewed-by: Jonathan Rajotte Julien <jonathan.rajotte-julien@efficios.com>
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
7 years agotiming.core: simplify hashCode, equals and toString of AbstractCalledFunction
Matthew Khouzam [Sat, 1 Oct 2016 05:25:15 +0000 (01:25 -0400)] 
timing.core: simplify hashCode, equals and toString of AbstractCalledFunction

* hashcode now uses Objects.hash
* equals new uses and Object.equals
* toString now avoids a useless new String()

Change-Id: Ib409d1eee45a17b917f334d93e52a6845bcd2954
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/82308
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
7 years agotmf: Fill label background with marker color in time graph marker axis
Patrick Tasse [Thu, 22 Sep 2016 21:23:31 +0000 (17:23 -0400)] 
tmf: Fill label background with marker color in time graph marker axis

This makes the marker labels visually nicer and easier to read.

Change-Id: I5cddafa7485142526121629e47b7670ccb2a7c11
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/83257
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
7 years agotmf: Bug 500542: Missing zoomed events in time graph entry
Patrick Tasse [Tue, 30 Aug 2016 21:43:21 +0000 (17:43 -0400)] 
tmf: Bug 500542: Missing zoomed events in time graph entry

When adding an event that starts before the current zoomed event list's
first event, the list is deemed to be incomplete and missing events at
the beginning of the zoom range. The list is cleared and the new event
is added. The remaining events for the full zoom range should follow.

Change-Id: I7057fbaac6ca3a57b2923c786c08142b2ebf6fc3
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/80076
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
7 years agoanalysis: Add additional assertion to troubleshoot test failure
Marc-Andre Laperle [Tue, 1 Nov 2016 13:23:12 +0000 (09:23 -0400)] 
analysis: Add additional assertion to troubleshoot test failure

The axis range in the tested SWTChart doesn't have the correct range.
It would be interesting to know if the window range of the trace is
correct.

Change-Id: I001787016a5b3d78b83b526175d8a1e9656e2790
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84277
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
7 years agoctf: bug 506892 Fix experiment size request benchmark
Geneviève Bastien [Wed, 2 Nov 2016 12:41:39 +0000 (08:41 -0400)] 
ctf: bug 506892 Fix experiment size request benchmark

The experiment should be constructed only when all the traces are
initialized.

Change-Id: I1f5478013621ec3f135e409f4a0380688d7597f8
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/84341
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
7 years agosegstore: add default unhandled methods to ISegmentStore
Matthew Khouzam [Sat, 15 Oct 2016 20:06:55 +0000 (16:06 -0400)] 
segstore: add default unhandled methods to ISegmentStore

This handles common functionalities for all segment stores.
This can be seen as part of the segment store contract.

Anything removing items from a segment store asside from
clear should throw an unsupportedOperationException.

If an implementation decides to support this, that is fine, but
not the expected way the object should behave.

Change-Id: I9d00960ae8826cecc63092245ca7fc447a28defb
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83305
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
7 years agotiming.core: Simplify CalledFunctionFactory
Matthew Khouzam [Sat, 1 Oct 2016 12:31:27 +0000 (08:31 -0400)] 
timing.core: Simplify CalledFunctionFactory

Remove duplicated lines.
Make factory implementation final.

Change-Id: Iee670eb29e514107ec6d0a18fc8e1ed83edccca5
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/82315
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
7 years agotmf.ui: add export to tsv events table test
Matthew Khouzam [Thu, 6 Oct 2016 03:04:47 +0000 (23:04 -0400)] 
tmf.ui: add export to tsv events table test

This removes manual test case: 8.2, 8.1 could be
considered redundant too.

Change-Id: I854e48431bedf187d4e7e625778ae58371804d65
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/82571
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
7 years agosegStore: allow benchmarks to specify the size of the store
Geneviève Bastien [Thu, 20 Oct 2016 20:43:01 +0000 (16:43 -0400)] 
segStore: allow benchmarks to specify the size of the store

This will allow other benchmarks to extend this class but with a
different number of segments

Change-Id: I328c38831cbbb9bb23375d4367aec7a07dc5101e
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/83636
Reviewed-by: Hudson CI
7 years agotiming: Make scatter viewer update only once
Geneviève Bastien [Wed, 19 Oct 2016 13:55:08 +0000 (09:55 -0400)] 
timing: Make scatter viewer update only once

Before, it was updated when the parent's windowRangeUpdated requested an
updateContent and when the viewer itself requested and updateRange, also
in the windowRangeUpdated signal. Now the updateRange method overrides
the updateContent, so the update is done once only.

Change-Id: I8b92367f441c0db62e70e7cc7f2a7c7251b2e598
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/83526
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
7 years agotmf: Update data in common XY chart outside the UI thread
Genevieve Bastien [Sun, 9 Oct 2016 09:45:38 +0000 (05:45 -0400)] 
tmf: Update data in common XY chart outside the UI thread

Change-Id: Id7e991dab24bd0ba6dce062a9e477c50664587c0
Signed-off-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/82796
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
7 years agotiming.ui: Add dirty conditions for SWTbot to scatter graph viewer
Geneviève Bastien [Wed, 26 Oct 2016 19:00:11 +0000 (15:00 -0400)] 
timing.ui: Add dirty conditions for SWTbot to scatter graph viewer

Also remove a calls to update

Change-Id: Ifbf37b3dda4f1ebcea065adfba08f2f2ab14f6c1
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/83964
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
7 years agolttng.ui: Add test in symbol map test to verify names
Matthew Khouzam [Fri, 28 Oct 2016 15:26:29 +0000 (11:26 -0400)] 
lttng.ui: Add test in symbol map test to verify names

The names are now "main" ... due to a missing seek.

Change-Id: I8b8b35a87ccc73cfe4876026eb862200098bc7b6
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/84124
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
7 years agotmf.core: Make getParamater synchronized
Matthew Khouzam [Sat, 22 Oct 2016 12:10:16 +0000 (08:10 -0400)] 
tmf.core: Make getParamater synchronized

The setter (setParameter) is synchronized, since getParameter is not
atomic, it needs to to also be synchronized to avoid racy situations.

Change-Id: I1c7eb0a2fde364efd7e879a82411f91fae0deab2
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83739
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
7 years agolttng.swtbot: Add resources view swtbot tests
Matthew Khouzam [Tue, 18 Oct 2016 18:16:38 +0000 (14:16 -0400)] 
lttng.swtbot: Add resources view swtbot tests

This removes two tests from the manual tests.

Also, it adds the notion of a shared test base with
control flow view.

Change-Id: Ieea4539eafcd00d62d56347b6cab55654d137d53
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83468
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Jonathan Rajotte Julien <jonathan.rajotte-julien@efficios.com>
7 years agograph/timing: Fix a typo in Manifest.MF file
Geneviève Bastien [Mon, 31 Oct 2016 15:56:14 +0000 (11:56 -0400)] 
graph/timing: Fix a typo in Manifest.MF file

Sounds like somebody tried running an lttng command from the manifest file.
Otherwise, it shows that the Manifest-Version line doesn't have much use.

Change-Id: If6ccfa8d601d42ce4cd88ebf72068336d0f7cf36
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/84248
Reviewed-by: Hudson CI
Reviewed-by: Jonathan Rajotte Julien <jonathan.rajotte-julien@efficios.com>
7 years agolttng.swtbot: Add cfv toolbar order test
Matthew Khouzam [Tue, 18 Oct 2016 17:41:15 +0000 (13:41 -0400)] 
lttng.swtbot: Add cfv toolbar order test

This is a very simple test to make sure that the toolbar items
are all displayed/enabled.

Change-Id: If6639280ebd1c31a30ed02784e2dc3a3f0db2a3b
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83467
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Reviewed-by: Jonathan Rajotte Julien <jonathan.rajotte-julien@efficios.com>
7 years agolttng.ui.test: Enable SymbolMap test
Matthew Khouzam [Sun, 16 Oct 2016 11:34:15 +0000 (07:34 -0400)] 
lttng.ui.test: Enable SymbolMap test

Change-Id: I869219981337d6a2d7daf0ac4f79763803585f11
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83320
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
7 years agotiming.swtbot: add SWTBot tests to timing analysis
Matthew Khouzam [Wed, 19 Oct 2016 01:05:36 +0000 (21:05 -0400)] 
timing.swtbot: add SWTBot tests to timing analysis

Adds unit tests for flamegraph.

Change-Id: I63bee2198692dd5ca1a240f9a148fbed0ed98c69
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83481
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
7 years agosegStore: Add benchmarks for sorted iterations
Geneviève Bastien [Thu, 20 Oct 2016 19:12:40 +0000 (15:12 -0400)] 
segStore: Add benchmarks for sorted iterations

Change-Id: Ia9c5d679ed87c0f03c1890f2477f376c35a874e4
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/83632
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
7 years agoos.linux: Ignore testWriteToTsv swtbot test
Geneviève Bastien [Wed, 26 Oct 2016 19:27:59 +0000 (15:27 -0400)] 
os.linux: Ignore testWriteToTsv swtbot test

This test causes instability in the SWTbot because of a potential race
condition with the update that changes the header column titles that do
not correspond to the system call data. The test should be modified
either to use SystemCall segments, or moved to timing.swtbot

Change-Id: Idea1c43f377ca79efb4bf5a4751f4611e6336891
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/83967
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
7 years agotiming: fix typos in segstore statistics test
Geneviève Bastien [Mon, 24 Oct 2016 20:10:39 +0000 (16:10 -0400)] 
timing: fix typos in segstore statistics test

And update some javadoc for tests

Change-Id: I9e35865a37c4f2b62ec33b895ee2dc8ec8d0e2a1
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/83823
Reviewed-by: Hudson CI
7 years agotiming.ui: fix typo in id of CallGraphDensityView
Matthew Khouzam [Fri, 21 Oct 2016 20:02:20 +0000 (16:02 -0400)] 
timing.ui: fix typo in id of CallGraphDensityView

The ID was never used, now the view can now be opened
programatically as it matches the plugin.xml

Change-Id: I25ab1ecf9324ef43001fe3a488c7381ff59853a7
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83725
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
7 years agotmf.ui/timing: Update XY viewers even if there is no data
Genevieve Bastien [Sat, 15 Oct 2016 14:45:05 +0000 (10:45 -0400)] 
tmf.ui/timing: Update XY viewers even if there is no data

The range of the window still changed and the view should follow that
range, even though there is no data to display

Change-Id: Iaa6d180de39f6ee5b109ce5826999c096f170469
Signed-off-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/83355
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
7 years agotmf.ui: Add a SWTbot condition for XY charts ready
Geneviève Bastien [Thu, 20 Oct 2016 14:03:37 +0000 (10:03 -0400)] 
tmf.ui: Add a SWTbot condition for XY charts ready

bug 500770

Change-Id: If70437ab08871962415a2b954307b58f24460331
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/83595
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
7 years agoreleng: Do not build the testing feature when some tests are skipped
Alexandre Montplaisir [Mon, 24 Oct 2016 23:14:37 +0000 (19:14 -0400)] 
releng: Do not build the testing feature when some tests are skipped

The testing feature, as well as the "alltests" plugin, cannot be built
when -Dskip-automated-ui-tests is on, as they require all SWTBot tests.
This means the the releng-site cannot be built either. All of them
should only be run when the tests are active.

Change-Id: I9d89444818c2373239dbcd04e8749ab6e033038c
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/83830
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Hudson CI
7 years agotiming.core: add testing for less used functions in the segment store
Matthew Khouzam [Sat, 1 Oct 2016 05:42:39 +0000 (01:42 -0400)] 
timing.core: add testing for less used functions in the segment store

Increases code coverage to ~80%. More importantly, makes the contract
automatically tested for new segment stores.

Change-Id: Id7c476a8df3254b01732a44e46227da4280274f2
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/82310
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-by: Hudson CI
7 years agotmf: Mark TmfTraceManager @NonNullByDefault
Alexandre Montplaisir [Thu, 1 Sep 2016 00:34:07 +0000 (20:34 -0400)] 
tmf: Mark TmfTraceManager @NonNullByDefault

Change-Id: Ib7581d4d2b8cb2807882afc62ba0071e7d123f82
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/81721
Reviewed-by: Hudson CI
7 years agotiming: Add a generic table view for any segment provider
Geneviève Bastien [Thu, 22 Sep 2016 01:04:53 +0000 (21:04 -0400)] 
timing: Add a generic table view for any segment provider

It uses the analysis ID as the secondary ID of the view, it is thus not
necessary anymore to implement a concrete view for each segment store
analysis.

Change-Id: I9a596ac8217a1ba9233a345ec917d1488f170737
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/78803
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
7 years agotiming.ui: Remove dependency on trace with FlameGraphContentProvider
Matthew Khouzam [Wed, 19 Oct 2016 13:42:05 +0000 (09:42 -0400)] 
timing.ui: Remove dependency on trace with FlameGraphContentProvider

This decouples the flamegraph from the notion of traces.
Also removes dependencies on trace with FlameGraphView and adds
synchronization to refresh so tests will not query in the middle
of a draw.

Change-Id: I58f072cba473a74641b8ca8c364c85bfc0ef5cee
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83598
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
7 years agolinux.swtbot: Add more tests to SystemCallLatencyTableAnalysisTest
Matthew Khouzam [Mon, 24 Oct 2016 19:04:01 +0000 (15:04 -0400)] 
linux.swtbot: Add more tests to SystemCallLatencyTableAnalysisTest

All the entries in the TSV to test.

Change-Id: I69e1e11b48ac34b5bc9fc170ac127ef8343d4d08
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83816
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
7 years agoreleng: Provide a build property to skip automated UI tests
Alexandre Montplaisir [Sat, 8 Oct 2016 00:02:47 +0000 (20:02 -0400)] 
releng: Provide a build property to skip automated UI tests

Allow passing a `-Dskip-automated-ui-tests` option to skip running
the automated UI tests (like SWTBot and RCPTT modules) during
the Maven build.

Some setups may want to run these longer tests only in certain
conditions, or at a different frequency than the "normal" unit
tests.

Change-Id: Ifb1d6307d8bf4cddaba4e8835ea5d5e7595bbf80
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/83403
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
7 years agoanalysis.ui: add Export to TSV action test
Matthew Khouzam [Wed, 5 Oct 2016 17:51:38 +0000 (13:51 -0400)] 
analysis.ui: add Export to TSV action test

Change-Id: I4852f270b5af4d1e957285690f271f3f94ad9d5e
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/82552
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
7 years agotmf: make latches synchronized in abstract analysis module [bug 485793]
Matthew Khouzam [Mon, 24 Oct 2016 13:47:55 +0000 (09:47 -0400)] 
tmf: make latches synchronized in abstract analysis module [bug 485793]

Fixes a build instability caused by schedule (by design) spawning a job
that runs asynchronously. This makes cancellation racy since it can occur
before said job starts.

Change-Id: I93617efb80f4eb196ef0aba5cbe72e67917c876c
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83786
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
7 years agosegstore: introduce sorted iterators
Loïc Prieur-Drevon [Tue, 27 Sep 2016 16:25:40 +0000 (12:25 -0400)] 
segstore: introduce sorted iterators

add a sorted iterator to the segment store interface.
this will be useful for external memory segment stores
for which the intersecting segments cannot be held and sorted
in main memory, and if external memory stores have optimized
sorted iteration.

Change-Id: I02076daf1721cdf8bdd66f5e892f5c5280e46a3b
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/82015
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
7 years agotmf: Automatically sync experiments set up with the same hosts
Alexandre Montplaisir [Tue, 4 Oct 2016 23:56:44 +0000 (19:56 -0400)] 
tmf: Automatically sync experiments set up with the same hosts

Automatically correct the clock offset of traces inside an experiment
who define a custom property "clock_offset", by calculating
the average of those offsets for all concerned traces.

This is useful for LTTng traces coming from different domains.

This behaviour is similar to how Babeltrace handles traces with
clocks with different offsets.

Bug: 484620

Change-Id: Ic30837fd6e9fc8b9a7592a1722f0240145c77d62
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Signed-off-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/65330
Reviewed-by: Hudson CI
7 years agodoc: Add documentation for the local statistics
Jean-Christian Kouame [Mon, 24 Oct 2016 15:49:02 +0000 (11:49 -0400)] 
doc: Add documentation for the local statistics

Change-Id: Ifac7e24e2dbaaf911aaa1258f60214f9826a691c
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83795
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
7 years agotmf.ui: bug 505695 fix time graph views with GTK
Geneviève Bastien [Tue, 18 Oct 2016 20:56:02 +0000 (16:56 -0400)] 
tmf.ui: bug 505695 fix time graph views with GTK

Change-Id: Ib7cab31753d19e0a1d5766235dcdf88750f69a96
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/83473
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
7 years agosegStore: Run benchmarks as a parameterized test
Geneviève Bastien [Thu, 20 Oct 2016 15:53:36 +0000 (11:53 -0400)] 
segStore: Run benchmarks as a parameterized test

It is easier to benchmark only one type of segment store and add new segment
stores

Change-Id: I187c5396a143a3ed6321cfb55e7f13bf73670e09
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/83609
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
7 years agotmf.ui: Introduce TmfFileDialogFactory
Matthew Khouzam [Mon, 3 Oct 2016 20:20:46 +0000 (16:20 -0400)] 
tmf.ui: Introduce TmfFileDialogFactory

This factory can be used to stub out native FileDialogs in TraceCompass.
This is very useful when it comes to writing SWTBot tests.

Change-Id: I2fbcaa20818b806d6367031aa85f44883d0bc08b
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/82398
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
7 years agotiming.core: Add local statistics to the latency statistics
Jean-Christian Kouame [Tue, 6 Sep 2016 17:21:04 +0000 (13:21 -0400)] 
timing.core: Add local statistics to the latency statistics

Add the local statistics in the statistics view and update the general
item tree of the view. The SystemCallLatencyStatisticsTableAnalysisTest
is updated in consequence.

Change-Id: I5209f9c6430ee4658e3cf55ef50f4769b8e28772
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/80952
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
7 years agodoc: Update event matching developer documentation
Geneviève Bastien [Tue, 18 Oct 2016 13:52:22 +0000 (09:52 -0400)] 
doc: Update event matching developer documentation

It has been outdated for a while now.

Change-Id: I5f5de85e8048a81457fae95ed5198f04b65ddf87
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/83439
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
7 years agodoc: Modify 'Import mapping file' icon and title for callstack view
Geneviève Bastien [Tue, 20 Sep 2016 01:38:56 +0000 (21:38 -0400)] 
doc: Modify 'Import mapping file' icon and title for callstack view

It also adds a link to source lookup for LTTng-UST traces taken with LTTng 2.8+
(because it does not always work automatically when not in the same computer as
the one the trace was taken on)

Change-Id: If4f191d3e9fa650f852b1a2bed6857cd6e716a74
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/81404
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
7 years agosegstore: remove redundancies in deprecated class
Matthew Khouzam [Sat, 15 Oct 2016 19:59:12 +0000 (15:59 -0400)] 
segstore: remove redundancies in deprecated class

Make treemapstore extend the internal treemap store. It removes
a lot of duplication.

Change-Id: I89ccbbe506a3b93201342c16da6aef465034389d
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83304
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
7 years agoss: fix common node header size and check free space
Loïc Prieur-Drevon [Tue, 18 Oct 2016 16:14:30 +0000 (12:14 -0400)] 
ss: fix common node header size and check free space

COMMON_HEADER_SIZE had not been corrected after modifying strings
sections.
Check that node free space is consistent with what is written to
disk.

Change-Id: I9186f8b2e32159d9f2c771358c337aaf0ebb38c3
Signed-off-by: Loïc Prieur-Drevon <loic.prieurdrevon@gmail.com>
Reviewed-on: https://git.eclipse.org/r/83456
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
7 years agotiming.ui: make Scatter chart not update the UI in update data
Matthew Khouzam [Fri, 14 Oct 2016 16:46:52 +0000 (12:46 -0400)] 
timing.ui: make Scatter chart not update the UI in update data

Change-Id: Idaca90c31a1e3fdd32d4922816760c3eaa074e64
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83252
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
7 years agotmf.ui: make common line charts display the window range, not their model range
Matthew Khouzam [Fri, 14 Oct 2016 16:45:55 +0000 (12:45 -0400)] 
tmf.ui: make common line charts display the window range, not their model range

Change-Id: Ie6cea17dea1490af724adec1d0743d01cad884f6
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/83251
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
7 years agotiming: avoid a division by zero with scatter viewer
Genevieve Bastien [Sat, 15 Oct 2016 13:48:55 +0000 (09:48 -0400)] 
timing: avoid a division by zero with scatter viewer

Change-Id: Iee5bce80f07ae0231801b3be17d5cbad2d665e48
Signed-off-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/83354
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
This page took 0.061868 seconds and 5 git commands to generate.