drtracing.org Git - deliverable/tracecompass.git/log

TMF: TimeGraphPresentationProvider can notify listeners of color change

Bug 419282: If a presentation provider see its state table change and modify
colors, it fires a notification to listeners
The TimeGraphControl implements that listening interface.

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

tmf: Disable scheduler executionOrder test

That unit test has always been prone to multi-threaded uncertainty,
but recently it has been failing a lot more, which is very annoying
for the CI. I suspect it's because of the recent changes to the CTF
parser, some requests may end up finishing faster than they did before,
and before another request could be scheduled.

I suggest disabling it until we rewrite it to be more permissive (or
better, add testing mechanisms to the scheduler, and test the order
of requests being *sent*, not the order in which they finish).

Change-Id: I70db04feb9f86534f5d12623cf33f06d56b3a737
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/18942
Tested-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
IP-Clean: Matthew Khouzam <matthew.khouzam@ericsson.com>

TMF: add support of cut/copy/paste/dnd in FilterView

Signed-off-by: Xavier Raynaud <xavier.raynaud@kalray.eu>
Change-Id: I67f15cd9c1dcb046de8dc6eef196012be14d9265
Reviewed-on: https://git.eclipse.org/r/17019
Tested-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
IP-Clean: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

TMF: Add assert exists and dispose to traces in synchronization tests

Change-Id: Ida426ccb6abe6404aa6c443dc5ae38b8d175f745
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/18862
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Merge branch 'master' into lttng-luna

ctf: Fix multiple coding style and typo issues while reading code.

Change-Id: If63330f4c86e7cd87cf979e1709a81fb7c8958ab
Signed-off-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18541
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
IP-Clean: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Hudson CI

ctf: Plug the unsigned utils comparator into Stream comparator.

This patch-set fixes a pending TODO found randomly.
It modifies the code to use the unsigned comparator and avoid
bugs for large timestamp values.

The comparator no longer accept null values.

Change-Id: Idee0e56edfe85ba01a27fb79d1076736fee24874
Signed-off-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/18785
Reviewed-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Tested-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
IP-Clean: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

tmf: Fix core tests failing on Windows

2 tests did not dispose the index correctly. Solution: make sure
fCheckpointCollection gets assigned when recreating the collection so
that it gets disposed in the tearDown.

Change-Id: I6a79b62cdabf136790bcf76ca606e71c31cdd81b
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18481
Tested-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
IP-Clean: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

tmf: Add timeouts to potentially hanging tests

Change-Id: Ia429dad124844abbac44afa644250238c8a60f5a
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/18863
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Etienne Bergeron <etienne.bergeron@gmail.com>

tmf: Remove unused TmfSortedArray.

This data-structure seems unused.
It's implemented with an insertion sort, and won't scale.

Change-Id: I49ae518a384817c4fd537c989240454e00f6f38b
Signed-off-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18875
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

[ctf] Fix coding style issues in BitBuffer.

Fix potential issue with an undefined behavior in CTF.
- Reading a integer of zero bit at the end of the buffer may be valid.
- Change the behavior to accept it.

Change-Id: I22fccc4e2ffee3bbf796e52b516b00f7fc5e31ac
Signed-off-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18707
Reviewed-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

[ctf] Fix binary search for a long sequence of same timestamps.

The binary search algorithm does not need a third case
(when values are equals) if there is a guarantee to remove
one element in the search space at each iteration.
The actual binary search implementation performs a sequential
search on elements with the same timestamp (to find the first
one).

The ETW2CTF traces produce many debug events for the debugging
information at the module load timestamp. Which is the worse
case for the actual implementation.

We changed the algorithm to use the timestamp end of a packet
instead of the beginning. We changed the way to choose a
middle element to ease the recursion by using only two cases.

Change-Id: I4f16d43b9533f8f1449cdb3c4c213bcb9f962daf
Signed-off-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18654
Reviewed-by: Matthias Nick <Matthias.Nick@bsiag.com>
Reviewed-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Tested-by: Hudson CI
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

TMF: Check presence of fields for TCP event synchronization

Traces using an old version of the TCP addons module would get a NPE when
getting the key to match packets.

Change-Id: Ib0470834807b1933f5f46069c5377115764b9dda
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/18848
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

ctf: Replace Long.compare() with Java 6 compatible method

We still technically support Java 6, and Long.compare() was added in 7.

Change-Id: I565b40bbc11c1c1c3b05a4097cc0e619598d0553
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/18841
Reviewed-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Update histogram text controls

- 'Current Event' text control replaces with 'Selection Start' and
'Selection End' controls
- Text control layout updated
- Added link button to force single selection time
- Added selection span to histogram tool tip when mouse hovers over
selection range
- Fixed text control to support keypad Enter key
- Fixed to ignore invalid window span value
- Externalized tool tip messages

Change-Id: I2158dd579b2e508b3c2d0332aedc6b34626e6feb
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18634
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
IP-Clean: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Hudson CI
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

tmf: Update histogram layout

- Both histograms fill all available space with no minimum height
- The max nb event text control grows in width according to its maximum
(watermark) text extent but never shrinks, eliminating layout jitter
(bug 375990)

Change-Id: I1ca835434f3e0fca4196aa7f23eb29dead2f50db
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18355
Tested-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
IP-Clean: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

tmf: Support drag zoom in histogram

- Right button drag is used to select a new zoom window in both
histograms
- Shift-right button drag or click is used to modify the zoom window in
full range histogram
- Zoom is cancelled if the selected range duration is zero
- New time range is propagated when the mouse button is released
- Move draw time range window method to base class

Change-Id: If46c1edd5c8f67a1b48ebc2ebbe2527613ba927c
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18310
Tested-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
IP-Clean: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

tmf: Support drag move in time range histogram

- Add support for bucket display offset in histogram
- Middle button or Ctrl-left button are used to drag the window in time
range histogram
- New time range is propagated when the mouse button is released
- Remove restriction of selection buckets to visible range in data model
- Adjust histogram tool tip according to bucket display offset
- Prevent histogram tool tip when mouse is not over visible time range

Change-Id: I9ee3663511b57ac32a668cc25cf97e903a72e60f
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18309
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
IP-Clean: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

tmf: Support drag selection in histogram

- Left button drag makes a range selection in both histograms
- Middle button or Ctrl-left button are used to drag the window in full
trace histogram
- Middle-click or Shift-left-click on full trace histogram now centers
the window on the mouse position
- Selection is only propagated when the mouse button is released
- When moving the mouse only redraw instead of refreshing the whole
histogram
- Fix range selection start on Shift-click
- Limit selection range to visible time range
- Draw selection region background corrected when end time is before
start time
- Prevent mouse wheel zoom when dragging in progress

Change-Id: I393e0e72cd2f445be2bd75bb90090f27abc2a2da
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18308
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
IP-Clean: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

tmf: Fix time range histogram to show its full range

- Set the end time of the time range histogram to the time range end
time (instead of the last event time)
- Display the time range text controls even when model has no events
- Snap selection to the end time
- Draw the delimiter to the right of the end time instead of last event
time
- Fill the area to the right of the delimiter with the background color
- Draw selection range anywhere in the histogram range regardless of
last event time

Change-Id: I913d22a20d79a3ca138902acd0d9cc815918cbc9
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18307
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
IP-Clean: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

ctf: Remove unused StreamInputReaderComparator.

This is a clone of StreamInputReaderTimestampComparator.

Change-Id: Ib640cbafe8105bad2c18a3f5e35136e0a5693597
Signed-off-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18796
Reviewed-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
IP-Clean: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

[ctf] Simplify the logic of the unsigned long comparator.

To understand this modification, let assume the "char 8-bits" domain.

unsigned [0 .. 255]
signed [-128 .. 127]

By receiving the parameters encoded in a signed number, the method receives

left:  [0..127,-128..-1]  (which represents [0..127,128..255])
right: [0..127,-128..-1]  (which represents [0..127,128..255])

By definition (on an unconstrained domain), this assertion holds

left < right  <==>  left + k < right + k

Thus, the idea is to rotate the domain by k to allow a signed operator to
be able to compare to full domain.
In this case k must be -128.

By rotating the domain, left and right range become [-128..-1,0..127],
and are now comparable with the signed operator.

Change-Id: I92b27ab00e8f14102a04e085ef807e211e39a7f0
Signed-off-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18784
Tested-by: Hudson CI
Reviewed-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Reviewed-by: Xavier Raynaud <xavier.raynaud@kalray.eu>
IP-Clean: Xavier Raynaud <xavier.raynaud@kalray.eu>
Tested-by: Xavier Raynaud <xavier.raynaud@kalray.eu>

TMF: make BufferedRandomAccessFile fields private, and final if possible.

Change-Id: I213410e6e4cc5fe96e24c88d6eac61d6bb2b1b23
Signed-off-by: Xavier Raynaud <xavier.raynaud@kalray.eu>
Reviewed-on: https://git.eclipse.org/r/18711
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Etienne Bergeron <etienne.bergeron@gmail.com>

Avoid multiple seek to the same location.

There is multiple instance of seeking to the same location twice.
To solve this issue the seek method check if the current location is the same.
Here is the explanation:
[CtfTmfContext.java]
    public void setLocation(ITmfLocation location) {
        curLocation = (CtfLocation) location;
        if (curLocation != null) {
            getIterator().seek(curLocation.getLocationInfo());
        }
    }
GetIterator  --> will force a seek to curLocation
.seek        --> force an other seek to curLocation
Here are the called method:
    private CtfIterator getIterator() {
        return CtfIteratorManager.getIterator(fTrace, this);
    }
    public static synchronized CtfIterator getIterator(final CtfTmfTrace trace,
            final CtfTmfContext ctx) {
        return map.get(trace).getIterator(ctx);
    }
    public CtfIterator getIterator(final CtfTmfContext context) {
            [...]
            if (context.getLocation() != null) {
                final CtfLocationInfo location = (CtfLocationInfo) context.getLocation().getLocationInfo();
                retVal.seek(location);   /// FORCE SEEK!!!!!!!!!!!!
            }
Change-Id: Ib5ad6d05eb47045660c7ab5076930a30e829c7a0
Signed-off-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18461
Reviewed-by: François Doray <fdoray.eclipse@gmail.com>
Tested-by: Hudson CI
Reviewed-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
IP-Clean: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

tmf: Fix unsafe cast warning in TmfTrace

Now that TmfData/EventProvider are merged, .getType() will always
return an instance of an ÌTmfEvent. Simply fix the generic types
to take this in consideration.

Change-Id: I791b59c480311a34c7416564219f6d3fed3f5951
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/17054
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Hudson CI

tmf: Move TmfEventRequest.ALL_DATA to the interface

No reasons for static defines to not be in the interface. And it was
always confusing that ExcecutionType was in the interface, but
ALL_DATA was not.

Change-Id: I373f0d36e779e9194315ad5d1df8c1237b94be1f
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/17053
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Hudson CI

lttng rcp: Add Batch Import to File menu

Change-Id: I95e7c6f40aeb0c10c4ce92eabe9f8e3c3e39c0ac
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18697
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Deregister traces used for validation only from signal manager

Bug 421860

Change-Id: I885ab62960a8cfc8af50591e52c239d0ca9c2479
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18521
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Hudson CI

Merge branch 'master' into lttng-luna

tmf: Merge TmfDataProvider and TmfEventProvider together

Initially, there was a plan to support "traces" that had no timestamps,
and this would be handled by a DataProvider. Turns out the whole
framework requires some sort of timestamp nowadays, so this distinction
is no longer necessary.

(If a trace really does not have any timestamp, they could just use the
event rank as a sort of dummy timestamp, which is quite intuitive anyway).

This also merges Tmf{Data|Event}Request and TmfCoalesced{Data|Event}Request.

Change-Id: I734bb297d252d56902e1eb0534669b0d4088d911
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/17052
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

ctf: Update paths in the CTF-Testsuite tests

The upstream CTF-Testsuite was updated, let's follow 'suite'.

This also changes the test mechanisms to use parametrized tests. This
way, we get one JUnit test per test-trace being run, so we'll know the
exact amount of tests that are failing (instead of bailing out as soon
as we get one failure).

Those tests are disabled by default to avoid build failures for now,
but they should be fixed/reenabled as soon as possible.

Change-Id: Iadcd7c2d4352301cf60c132e0f2d8dc80f75a688
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/18479
Tested-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
IP-Clean: Matthew Khouzam <matthew.khouzam@ericsson.com>

Avoid flickering redraw of TmfEventsTable on open.

There is a 4 or 5 redraw of the Events table caused by the columns pack.
To avoid flickering, it's better to disable the redraw of the table.

I didn't try this fix on linux.

Change-Id: Idcfd1f5a79faee8ae0c31b268c67576ba1547760
Signed-off-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18648
Reviewed-by: François Doray <fdoray.eclipse@gmail.com>
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
IP-Clean: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

[ctf] Fixes multiple coding style issues while reading ctf Types.

Change-Id: I73a2e42b76090ed344a32da330cd0ab240b68099
Signed-off-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18706
Tested-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
IP-Clean: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

tmf: Fix NullPointerException TmfTraceElement

and fix timing issues in TMF UI tests. The NPE and timing issues
causes problems in the TMF UI tests. That's way they are committed in
one patch.

Change-Id: If80562c11f9daab5756adf982b35a9e6b308e03b
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18748
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Shortcut string creation in tracing functions.

This quick fix avoid useless creation of the message to be logged when tracing isn't enabled.

Change-Id: If1a667451c59afa67e25572f598e0826ce92f090
Signed-off-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18453
Reviewed-by: François Doray <fdoray.eclipse@gmail.com>
Tested-by: Hudson CI
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

tmf: Remember last selected directory in Batch Import wizard

Change-Id: I2a166af74ef8720ec672b56980493967bf7eb874
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18480
Tested-by: Hudson CI
IP-Clean: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

TMF: Add base class and interface for state system analysis

Also update the state system explorer to show state systems from modules
instead.

Change-Id: Iab6424be1a469c541d3e9b199d7377c8a4e70299
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/18098
Tested-by: Hudson CI

Merge branch 'master' into lttng-luna

Conflicts:
lttng/org.eclipse.linuxtools.tmf.ui/META-INF/MANIFEST.MF
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/project/handlers/DeleteTraceHandler.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/project/model/TmfTraceElement.java

Change-Id: I06c617e5e9ec6e691592971f9fc78812751623c8

Revert "ctf: Simplify position arithmetic in aligned reads"

This reverts commit 24aae8967b763e5e2c8d87acac9c1a10533ae598.

With the newest content of master, this patch fails some unit tests while it didn't before... This will have to be investigated further.

Change-Id: I08dae5325b2048d84a6492780a02132a485ca914
Reviewed-on: https://git.eclipse.org/r/18536
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Merge commit 'cea87c3123c41bd06cd2b816f885a27612dfc356' into lttng-luna

Conflicts:
lttng/org.eclipse.linuxtools.ctf.core.tests/pom.xml
lttng/org.eclipse.linuxtools.ctf.core.tests/shared/org/eclipse/linuxtools/ctf/core/tests/shared/CtfTestTrace.java
lttng/org.eclipse.linuxtools.ctf.core.tests/traces/.gitignore
lttng/org.eclipse.linuxtools.ctf.core.tests/traces/get-traces.xml
lttng/org.eclipse.linuxtools.tmf.core.tests/shared/org/eclipse/linuxtools/tmf/core/tests/shared/CtfTmfTestTrace.java

Change-Id: Id7e995409cfeace1407abea1bd7c3c81f758ef22

Merge commit '2c789b58062b5bf3bb16ca1654ccf9cf87f3c444' into lttng-luna

Conflicts:
lttng/org.eclipse.linuxtools.ctf.core/src/org/eclipse/linuxtools/ctf/core/event/types/EnumDefinition.java
lttng/org.eclipse.linuxtools.ctf.core/src/org/eclipse/linuxtools/ctf/core/event/types/FloatDefinition.java
lttng/org.eclipse.linuxtools.ctf.core/src/org/eclipse/linuxtools/ctf/core/event/types/IntegerDefinition.java
lttng/org.eclipse.linuxtools.ctf.core/src/org/eclipse/linuxtools/ctf/core/event/types/StringDefinition.java
lttng/org.eclipse.linuxtools.ctf.core/src/org/eclipse/linuxtools/ctf/core/event/types/StructDefinition.java

Change-Id: I1f0ffe6e8f8546aa1e0a6b850841558d57eb766b

ctf: Simplify position arithmetic in aligned reads

Thanks to Étienne Bergeron for this suggestion.

Change-Id: Ib8617651e797afa806b4997828f6ba7e392e8165
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/18466

ctf: Move CTF-testsuite tests to a separate file

Change-Id: I0ebbe99f577430c16388ee6619d7a4d3a98044ae
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/18478
Tested-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
IP-Clean: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf: Import and Export trace package user guide

Change-Id: Ib9b56867257434db0afcb8c53e8969a1d6729d19
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18110

tmf: Import and Export trace package

Import and export files, bookmarks, supplementary files for a
single trace as an archive (tar, tar.gz or zip)

Change-Id: I71f5f104b2c552a5dbb2721fbdde9b069dd1a306
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17666
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Hudson CI

Fix invalid cast which break trace files bigger than 2 gig.

The min operation received a value which can be negative after the cast.
(Values bigger then 2^31). Thus, the trace files has one chance on two to open.

Change-Id: Id97578f430af8336bab9d863f575f46ed7f54adc
Signed-off-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18456
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Hudson CI

Fix coding style in TmfVirtualTable.

Fixing some nits I've found while reading this file.
Mostly no behavior changed, except shortcut a loop and removing a useless test.

Change-Id: Iaa2db0f7fc6cf72a3b87b0f67763a4dbf073cf57
Signed-off-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18454
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

ctf: Add advanced trace tests

Using a more exotic test trace.

I'm not exactly sure why a string field ends up in-between the variant
selector and its value, but this is how they are printed by Babeltrace,
so I assume we're doing the right thing.

Change-Id: I4bd0f35d31a7c592fafd64b3321f182f9de71c79
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/17550

tmf: update batch import wizard to show file sizes

Change-Id: I970c569860f81a2bd33a3ab227aa7ff3db2679b8
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17690
Tested-by: Hudson CI

ctf: add synthetic LTTng kernel trace

This will allow us to have much more control over testing the parser
and it allows scalability testing too.

Change-Id: I433709dafab8fd5c8d412cb9b4b632ade03f18a9
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17050
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

lttng: Bump the target definition sequence number because of RSE change

The sequence number needs to be changed so that PDE reloads the target
definition properly in certain cases. In Eclipse 4.4 it can lead to
non-obvious
errors, see https://bugs.eclipse.org/bugs/show_bug.cgi?id=420322

Change-Id: I8fac34ba6c0edc3d3f243d70ec8b76bc6ddab9e9
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18483

Merge commit 'e1de8d2d152352eded708615a967021db7800709' into lttng-luna

Conflicts:
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/Messages.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/messages.properties

Change-Id: I542be9c92e142114af14796545097c44c8962360

tmf: Add a debugging test Suite

When a test suite is run with the new DebugSuite runner, the name
of every test being run will be printed to stdout. It's not used
by default, but it can be manually switched to. This can help debug
unit tests, especially when run from Maven.

Change-Id: I5b3ca8a0a0f7867977500e5b3a280c46d702672c
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/18464
Tested-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
IP-Clean: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

tmf: Add function name mapping to the Callstack View

Initial version of the "function name mapper" for the Callstack View.
It currently only supports a text file listing addresses followed by
names, like the output of "nm [binary] > out.txt".

The next steps will be:
- Add support for reading the binary directly, by tapping into the
relevant CDT components.
- Add a separate dialog that shows the currently imported file name(s).
- Add support for dynamically-loaded libraries. This will require some
work on the tracer side.

Change-Id: Id622cce487965b8c770fc40218685ec7605924ff
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/17978
Tested-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
IP-Clean: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

Avoid calling reverse on a list.

Change-Id: I3f3b17ea4372ef5db64b20f9c267d606ded8101a
Signed-off-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Reviewed-on: https://git.eclipse.org/r/18452
Reviewed-by: François Doray <fdoray.eclipse@gmail.com>
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

ctf: Move non-aligned reads to a unified method

* Clean up integer reading
* Clean up string reading
* Clean up enumeration reading
* Clean up float reading
* Clean up structure reading

Change-Id: I3627f0c1aaecbaa84d861cba05c0681a7b267388
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17465
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Hudson CI
Reviewed-by: Etienne Bergeron <etienne.bergeron@gmail.com>
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

lttng: Add icons to build.properties in UST UI plugin

Change-Id: Idfc348d1f281f609e12ae82118998c7b36fc858e
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18415
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

tmf/lttng: Add Export To Text feature to User Guide

Change-Id: Iff24f9af28fd0f97c0568bfb764d996ededa7b8b
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18428
Tested-by: Hudson CI
Reviewed-by: Xavier Raynaud <xavier.raynaud@kalray.eu>
IP-Clean: Xavier Raynaud <xavier.raynaud@kalray.eu>
Tested-by: Xavier Raynaud <xavier.raynaud@kalray.eu>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

lttng: Add o.e.l.lttng2.ust feature to Linux Tools and LTTng releng

Change-Id: I3e00018fc190fa0883c4b6674987f33a772618f2
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18312
Tested-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
IP-Clean: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

tmf: Delete experiment supp files when adding/removing trace

Bug: 421372

Change-Id: I9dde1c7988df47348e5c96ecda43df45dc062c4a
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18281
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>

TMF: Export trace to text

Add a command to export traces to text.
If a filter is applied, only filtered elements are exported.

Change-Id: Ie213eda715f68104572704cb3c08226aa5c52a85
Signed-off-by: Xavier Raynaud <xavier.raynaud@kalray.eu>
Reviewed-on: https://git.eclipse.org/r/17813
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

lttng-rcp: fix group-ids in pom.xml files

Change-Id: I13c40fce900473cee2a44051150e61f7353b04fa
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18384
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

lttng-rcp: build rcp plug-ins and features by default

This allows us to notice errors during every Gerrit contribution and
build. Note that the product is not build by default (yet).

Change-Id: Ic2e5e0aef7d43c80027b0fbc6a5829916bea2e5d
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18314
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

lttng: Use RSE SDK in target definitions so that sources are available

Change-Id: I73b42b577193da2b05ef7b6d14912bb9e08b466c
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18313
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>

lttng: Disallow adding contexts on events for v2.2 or later (419455)

Change-Id: Ib68015ed1fb4b3bccfa18f010763a7c82ddf5806
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17744
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

lttng/tmf/gdb: Remove not needed epl-v10 and license html files.

Since there is a shared license feature no need to have epl-v10.html
and license.html files in each feature project. All build.properties
file have been also updated for that.

Change-Id: I2e1265a95d73932672fa4fe9941541f7839523c9
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18243
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

lttng: Add possibility to select/deselect all in remote import dialog

Change-Id: I41c32e3d214b422795679ba1d2e425eb8af58e5b
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18099
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
IP-Clean: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Hudson CI

tmf: Fix display of title in the SelectSupplementaryResources dialog

Change-Id: I659333998e5a22da8aea35741675f6d12c666388
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18157
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

Merge branch 'master' into lttng-luna

tmf: add title to trace type selection dialog

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

tmf: Add open trace with trace type hint

Allows for better automation of trace opening if
you know your trace type in advance.

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

lttng: Provide detailed progress information for remote import

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

lttng: Fix display of titles of all dialogs of the LTTng Control

Change-Id: Ibcc2ad1d9cef3152bf24d632f79939ebdbedb341
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18102
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Hudson CI

tmf: Fix concurrency issue of TmfEventThread (bug 421016)

Change-Id: I4f8daf1fad5f24576e9b9db14a92edae2be8db70
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/18061
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Hudson CI

tmf: Update the suspend and resume handling of requests

The new implementation handles the event before suspending the thread
requested by the request executor. In the previous implementation the
thread is suspended before handleData() which causes unnecessary
handling of data if the request is canceled during the time the
thread is suspended.

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

tmf: Move default tracing project name to common constants

Currently, the default tracing project name "Tracing" is defined
in the RCP only. It makes sense to also create a default project in
other cases such as when importing a trace with no projects in the
workspace so the name needs to be accessible to other plug-ins.

Change-Id: I8739e651217f8b369a37d8b4784df5d9f8110e18
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17944
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Optimize the ETERNITY time range

TmfTimeRange.ETERNITY is often used in requests and the like, when
you do not want to filter on timestamps at all. Implement it as
a subclass which does not even bother checking the range borders,
since any other timestamp or time range is definitely included.

Change-Id: I45c80523b054c33bd96f859355b795be38f7e760
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/17981
Tested-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
IP-Clean: Patrick Tasse <patrick.tasse@gmail.com>

tmf: Fix zero timestamp being included in all time ranges

For some obscure unknown reason, if a timestamp's value was equal
to 0, the TmfTimeRange.contains() method would always consider that
timestamp to be included in the time range, which is obviously wrong.

Change-Id: I61afea2eb05ede61b02dbaaed388e1a49c041fa2
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/17980
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

TMF: Introduce a framework to hook trace analysis modules/plugins

* Add a new extension point to define trace analysis modules.
* Add new interface IAnalysisModule and abstract class
  TmfAbstractAnalysisModule to implement the analysis itself.
* TmfAnalysisManager is the main class to obtain which analysis are available
  to a trace.
* IAnalysisOutput interface describe the different outputs the analysis can
  provide.  Objects of classes implementing this interface can register to an
  analysis module.
* The IAnalysisParameterProvider allows other parts of the system to set an
  analysis' parameters.
* Analysis are executed as Eclipse jobs.
* On the UI side, analysis and their provided outputs (if any) are now children
  of the trace and can be directly opened.
* Unit tests in tmf.core and tmf.ui provide stub analysis.

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

TMF: Add open and select methods to trace stubs, and dispose method

bug 420406

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

TMF: Fix concurrency issue in TMF scheduler tests

Change-Id: I4b360b8d1e5e064e0a9470ed8f450f52b969d189
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17971
Tested-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
IP-Clean: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Fix ArrayIndexOutOfBoundsException in Batch import on Mac

Bug: 420693

Change-Id: Ifad8934a4c85bf7885a7c7c20cb99d179c6898a3
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17868
Tested-by: Hudson CI

tmf: Use a common method to get opened tmf projects

Change-Id: I2caf2ef68ab336c1263e2b2bd883648cde081fa5
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17867
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Fix SD unit tests with GTK3

Using a very recent Eclipse integration build (which now uses the GTK3 SWT
backend), two of the Sequence Diagram unit tests fail because of invalid
parameter values.

This patch fixes the test by clamping the height/width area values to 1.
Exceptions can then only be thrown for run-time problems (like getDisplay()
returning null), so we let those go as runtime exceptions.

Change-Id: Idccbb3490e6a0ed9dc83c1f7582c61251ec09142
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/17857
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
IP-Clean: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>

Merge branch 'master' into lttng-luna

Conflicts:
lttng/org.eclipse.linuxtools.ctf.core.tests/traces/get-traces.sh
lttng/org.eclipse.linuxtools.lttng2.ust.core/src/org/eclipse/linuxtools/lttng2/ust/core/trace/LttngUstTrace.java
lttng/org.eclipse.linuxtools.tmf.core.tests/src/org/eclipse/linuxtools/tmf/core/tests/trace/TmfCheckpointIndexTest.java
lttng/org.eclipse.linuxtools.tmf.core.tests/src/org/eclipse/linuxtools/tmf/core/tests/trace/indexer/checkpoint/TmfCheckpointIndexTest2.java
lttng/org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/tmf/core/ctfadaptor/CtfTmfEventField.java

Change-Id: Ia1967a2a53ffe8006e4f99e523e385a852e44b11

tmf: Bug 420220: TimerThread leak

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

lttng-rcp: remove unnecessary maven nature of branding project

Change-Id: If039938f8bc6fcefd31048f3958ede114a69cb2a
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17852
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Use long[] instead of List<Long> in CTF integer-array fields

Fixes a warning with Eclipse 4.4, as it doesn't like unchecked conversions
from non-generics types (here Object) to a type with generics.

Change-Id: I7645b61aebea8e76642572c8ad02d37abb1552d6
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/17549
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
IP-Clean: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Hudson CI

lttng: fix property display of function probe event Bug 419454

Change-Id: I4355b74e53c0241acf7f314477c7ea1695b91e21
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17745
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

[TMF] Don't enable OK button in delete supp files dialog unless something is selected

Bug: 420332

Change-Id: I66b4ab6e5b6529251db4657bf8ab21646e59de2d
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17748
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
IP-Clean: Patrick Tasse <patrick.tasse@gmail.com>

TMF: Add new ctf trace stub to identify ctf test traces

Change-Id: I77536c7723df04f95cb8c78f4aac8e2c02a22ae8
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/17734
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

[LTTng] Fix BufferOverflowException batch importing LTTng UST trace

Bug: 420203

Change-Id: Id0d793b39ad28c80fd04d61bcd2646b419656e9e
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17699
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

[TMF] NPE in BatchImportTraceWizard.recurse

Put the absolute path of the file in the map to make it consistent with
how we query the map.

Bug: 420132

Change-Id: Id995ab06e31e0dd9b01c2c31f4d929f6b75aa944
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17676

[TMF] Don't swallow exception in Batch import "startUpdateTask"

One of the tasks in Batch import swallows exceptions making it much more
difficult to figure out what went wrong. Another one was being printed
to standard output.

Change-Id: Ife9c61dee1e57db4af11986739b712ca43c1f7c9
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17675
Tested-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
IP-Clean: Matthew Khouzam <matthew.khouzam@ericsson.com>

ctf: Drop the get-traces.sh script

We have a get-traces.xml Ant script already, which is slightly more
portable. No need to keep two redundant scripts in sync.

Change-Id: Ic17334d1309e8e822b1c7a545cfda35b985fe5e4
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17548

Merge branch 'master' into lttng-luna

lttng-rcp: Use tar.gz on Mac, it is supported by default

Change-Id: Id7a224cf842d721e7804b34a627c3930bf986808
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17601
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>

Add ... to Configure min max because it's a dialog

Change-Id: I789ab46612905c6ddbd52196cc87b6095c640094
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/17320
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
IP-Clean: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
IP-Clean: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>