Merge branch 'master' into lttng-kepler

Remove LTTng specific sonar and JaCoCo settings

Change-Id: I7e9b6523b16b0342ba427f60fb265c23996e7343
Reviewed-on: https://git.eclipse.org/r/8766
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Bernd Hufmann <bhufmann@gmail.com>
IP-Clean: Bernd Hufmann <bhufmann@gmail.com>

Merge branch 'master' into lttng-kepler

Conflicts:
lttng/org.eclipse.linuxtools.ctf.core.tests/pom.xml
lttng/org.eclipse.linuxtools.ctf.core/src/org/eclipse/linuxtools/ctf/core/trace/StreamInputPacketReader.java
lttng/org.eclipse.linuxtools.lttng.releng-site/pom.xml
lttng/org.eclipse.linuxtools.lttng/feature.xml
lttng/org.eclipse.linuxtools.lttng/pom.xml
lttng/org.eclipse.linuxtools.lttng2.kernel/feature.xml
lttng/org.eclipse.linuxtools.lttng2.kernel/pom.xml
lttng/org.eclipse.linuxtools.lttng2/feature.xml
lttng/org.eclipse.linuxtools.lttng2/pom.xml
lttng/org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/tmf/core/ctfadaptor/CtfTmfEvent.java
lttng/org.eclipse.linuxtools.tmf/feature.xml
lttng/org.eclipse.linuxtools.tmf/pom.xml

Change-Id: I5f85d26a7d5d4008cae9a5ed3ed6ebce7f1bd527

ensure bytes returned as int are never negative

Change-Id: I173304573594c66d1d4eccaa9827ce6ac8154b94
Reviewed-on: https://git.eclipse.org/r/8203
Tested-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
IP-Clean: Patrick Tasse <patrick.tasse@gmail.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>

Fix time range synchronization issues

- ResourcesView is unnecessarily refreshing with initial window offset
on setFocus(). This is removed.

- TimeGraphViewer method resetStartFinishTime() causes time range
notification callback. The method is changed to call
setStartFinishTime() instead of setStartFinishTimeNotify().

- TimeGraphScale handling of mouse double-click is calling
resetStartFinishTime() expecting the notification. The notification is
now triggered explicitly by calling notifyStartFinishTime().

- HistogramView initial window span can be clipped if it is set early
when not enough events have been indexed in the trace. The full trace
histogram time range is now re-set on reception of TmfTraceUpdated
signal.

- The full trace histogram uses the zoom's duration in updateTimeRange()
on mouse up, and this duration could have been clipped. The method now
used the stored range duration, which does not get clipped.

Change-Id: Id480340933494855330fc6d4fae9429155efa88f
Reviewed-on: https://git.eclipse.org/r/8745
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bhufmann@gmail.com>
IP-Clean: Bernd Hufmann <bhufmann@gmail.com>
Tested-by: Bernd Hufmann <bhufmann@gmail.com>

Implement simultaneously opened traces in TMF

- Change all supporting views to use a generic ITmfTrace instead of a
TmfExperiment.
- TmfExperiment* signals removed, replaced by generic TmfTrace* signals.
- TmfTraceOpened signal is sent when opening a trace. Views should get
ready to receive TmfTraceRangeUpdated if they need to coalesce requests.
It is too early to access any state system.
- TmfTraceRangeUpdated signal is sent from the TmfTrace after receiving
TmfTraceOpened. Request coalescing can happen handling this signal.
- TmfTrace base class triggers its indexing on receiving
TmfTraceRangeUpdated.
- Removed indexing of custom traces in initTrace, now triggered by
TmfTraceOpened in TmfTrace base class.
- TmfExperiment initializes the streaming monitor on reception of
TmfTraceOpened.
- TmfTraceSelected signal is sent when the trace is initially brought to
top or any time it becomes the active trace. Any component can also
broadcast the signal to trigger bringing the trace to top.
- TmfTraceClosed signal is sent when the trace is closed. All
jobs/threads/requests related to this trace should be canceled. The
trace will be disposed after the signal has been broadcast.
- These signals can contain a TmfExperiment as the affected trace. Any
view which needs to check or handle individual trace should specially
handle an instance of TmfExperiment and examine its underlying traces.
- Building of state system and statistics moved from initTrace to
reception of TmfTraceOpenedSignal. This allows, in the case of a
reopened trace, for the previous trace to be disposed and its files
closed before the new instance initializes the state system.
- All supporting views cancel running jobs/threads/requests on
TmfTraceClosedSignal.
- ControlFlow/Resources view store entry list of all opened traces in
hash map.
- ControlFlow/Resources view use cancelable BuildThread.
- Cleanup the state system when disposing a CtfKernelTrace.
- Dispose of CtfTmfTrace clears reference to CTFTrace and this is used
to prevent NPE on getNext of disposed trace.
- Replace Events view with editor area in LTTng Kernel and Tracing
perspectives.
- Updated definition of traceType extension for defaultEditor element.
- OpenExperimentHandler opens the experiment in events editor. Default
editor is used unless all underlying traces share a common defaultEditor
in their respective traceType definition.
- OpenTraceHandler opens the trace in default or specified events
editor. It no longer uses a dummy experiment to wrap the trace.
- Removed methods and variables related to current experiment.
- Removed EventsViewEditor previously used as a workaround to open
bookmarks in Events view.
- Removed Clears Views command, made obsolete by closing all traces
(Close All on any tab).
- ITmfEventsFilterListener/Provider interfaces removed and replaces with
signal handling. This removes complexity of handling multiple listeners
and providers being added and removed.
- Fixed missing scaling to nanoseconds of timestamps in
AbstractStateChangeInput.
- Fixed synchronization issue with armRequest in TmfTrace when trace is
disposed (executor is shutdown).
- TmfEditorInput modified to prevent lingering reference in platform.
- TmfEventsEditor implements IPartListener to trigger broadcast of
TmfTraceSelectedSignal, instead of at setFocus which was sometimes
called twice by the platform.
- TmfEventsEditor no longer needs to use inner TmfExperiment subclass to
trigger indexing when experiment opened from bookmark.
- UnsortedPropertySheetPage class created instead of inner class in
TmfEventsEditor which was causing lingering reference to editor in
platform.
- Remove unused TmfTraceParserUpdatedSignal.
- TmfEnvironmentView refactored to update its table on
TmfTraceSelectedSignal.
- Use nanosecond timestamp scale constant where appropriate.
- Add interface to HistogramView to allow its controls to retrieve the
current trace.
- Implement synchronization of time range in TimeChartView.
- Fix lingering reference to completed requests in TmfUml2SDSyncLoader.
- Fix TmfVirtualTable setSelection allowing top rank to be too large
when selecting near end of table.
- Fix hanging wait cursor in sequence diagram view for traces without
any sequence events.

Change-Id: I357accc8fc7ea96544d32db758abff4760682304
Reviewed-on: https://git.eclipse.org/r/8554
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bhufmann@gmail.com>
IP-Clean: Bernd Hufmann <bhufmann@gmail.com>
Tested-by: Bernd Hufmann <bhufmann@gmail.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>

tmf: Add the Histogram query API to ITmfStatistics

See the added Javadoc for the complete description. This will
allow the Histogram view to easily plug into the trace's
ITmfStatistics object to get its information (so it can be
optimized for each trace type, etc.)

Change-Id: Ib94aeaa369a8bf40da5246825152e26326a45e92
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/8675
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>

Fix bug when contexts are enabled but there are no contexts

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

Fix benchmark to use ctftmflightweightcontexts

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

IOStructGen cleanup

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

Add support for older Eclipse version (e.g. 3.6) than 3.8

Change-Id: I67ec91e26e608ddb4b243eb9f21261b070ae6d03
Reviewed-on: https://git.eclipse.org/r/8654
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>

Fix unintended black background in HistogramTextControls

This problem is caused by a style bit which is not applicable
for the Group class. The problem is only shown in Eclipse 4.2.0

Change-Id: I11416e8a9642c0d9637d696b2a72c04b1150a5f4
Reviewed-on: https://git.eclipse.org/r/8652
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>

Fix initial trace time range.

The initial start time of TmfTrace is set to BIG_BANG so that the
initial time range is equal to NULL_RANGE. This allows to prevent
unnecessary histogram requests when a trace is opened.

Since its base class defaults to NULL_RANGE, it is no longer necessary
to explicitly set it in the TmfExperiment constructor.

Unit tests updated accordingly.

A fix is implemented in custom traces for possible NullPointerException
that uncovered this issue.

Change-Id: Ib1e49f372e1c5b1eaa3aecb972cc2b3830b3fbe3
Reviewed-on: https://git.eclipse.org/r/8603
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: Bernd Hufmann <bhufmann@gmail.com>
IP-Clean: Bernd Hufmann <bhufmann@gmail.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
IP-Clean: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

Remove seek(time) from CtfTmfTrace and use parent implementation

The implementation of CtfTmfTrace of seek by timestamp worked
correctly to seek within the CTF trace. However, the returned
context contains an incorrect rank of always 0.

This incorrect rank will cause a wrong selected event in the
TmfEventsTable if the trace is used directly and not through an
experiment.

The solution is to use the TmfTrace implementation of seek(time)
instead which is handling the rank correctly.

Change-Id: Ie28c88c8e3263cbcb42349bb033cc61f69634a72
Reviewed-on: https://git.eclipse.org/r/8719
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
IP-Clean: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

tmf: Replace ITmfTimestamp by long in ITmfStatistics API

Some methods in the statistics imlementations can call each
other, and having to create new wrapper objects every event
is not very interesting... The normalization already happens
in the Statistics view anyway.

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

Implement MultiLineInputDialog for Add Bookmark action.

Supports multiple line description.
Tool tip position calculation is corrected.

Change-Id: I1bf7fc4b32a507b90475f321a054ee4ab7ced823
Reviewed-on: https://git.eclipse.org/r/8615
Tested-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
IP-Clean: Patrick Tasse <patrick.tasse@gmail.com>

Fix default Content column in events table.

The default Content column in TmfEventsTable was being populated using
the event content value's toString if available, or the event content's
toString otherwise.

This was preventing the possibility to show different information in the
Content column and in the raw viewer. The raw viewer is populated using
the event content value's toString.

Therefore if an event's root content had a value defined for use in the
raw viewer, the root content's toString was ignored for use in the table
column, and the column always showed the same data as the raw viewer.

The Content column is now always populated using the event content's
toString. The (optional) event content value's toString is only used by
the raw viewer.

The filter base class implementation is modified to use the same data as
is displayed by the table when filtering on the default Content
column.

Change-Id: Ic976104743eb8850d8176ce232ca84e0574badd3
Reviewed-on: https://git.eclipse.org/r/8570
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bhufmann@gmail.com>
IP-Clean: Bernd Hufmann <bhufmann@gmail.com>
Tested-by: Bernd Hufmann <bhufmann@gmail.com>

Update Grammar and Add support for callsites

A new feature in CTF v1.9
Add test cases for callsites
Improve other test cases

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

Fix incorrect seek by timestamp in CtfIterator.

Change-Id: I7902dead3d15dda15e692738bed227669d49b7e4
Reviewed-on: https://git.eclipse.org/r/8541
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: Hudson CI

tmf/lttng: Fix newly-introduced Javadoc warnings

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

Update project settings for CTF/LTTng/TMF

Remove the automatic line wrapping and unwrapping for code
lines. It had the tendency to absolutely destroy the formatting
if used on a complete file. The auto-wrapping of comments
remains at 80 columns.

Also enabled Javadoc warnings for protected fields and methods.
Since protected members in exported packages are part of the API,
those should be Javadoc-umented too.

Change-Id: Iab722d674e1f209f1e2b937ead9f058b9b09cab1
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/8473
Tested-by: Hudson CI
Reviewed-by: Francois Chouinard <fchouinard@gmail.com>
IP-Clean: Francois Chouinard <fchouinard@gmail.com>
Tested-by: Francois Chouinard <fchouinard@gmail.com>

tmf: Handle race between state system queries and diposal

It's possible that a state system gets disposed while a query
is ongoing (happens between two successive .getNextChild()
calls, for example).

Instead of implementing a convoluted reader-writer-lock, we can
just catch the ClosedChannelException and rethrow a
StateSystemDisposed exception.

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

tmf: Add a StateSystemDisposedException

Once a state system is diposed, no future queries should be run on
it. If such a thing happens, the new StateSystemDisposedException
will be thrown. This can help views know that they should stop
doing queries.

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

tmf: Add dispose methods to the state and stats backends

When a trace/experiment is closed while a state history is
being built (either for storing kernel states or statistics),
the build should be cancelled as part of the trace's dispose().
This means the threads should be stopped cleanly, the files
should be closed, and any ongoing requests should be cancelled.

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

tmf: Add some unit tests to the ITmfStatistics

Also fixed some bugs that those tests exposed:
- Time ranges are now inclusive (both at the beginning and end)
  for all implementations.
- All methods in TmfStateStatistics will now wait for the history
  construction to be complete, so hopefully no more hundreds of
  AttributeNotFoundException's.
- State system queries at the end time should now work correctly
  (the work-around was removed).
- A time range beginning at the trace's start time should now
  return the correct values. I couldn't test with the edge case
  where you have more than 1 event exactly at the start time, but
  I assume it should work correctly too.

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

tmf: Drop the Type in CtfTmfTimestamp

Also removed all redundant methods in the class and the test.
This brings CtfTmfTimestamp much closer to TmfTimestamp, now
it's simply a wrapper that ensures the scale is set to nanoseconds.

Change-Id: I8242434af7221ae1d6dd964d5c8d80cd0f4e60bc
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/8047
Tested-by: Hudson CI
Reviewed-by: Francois Chouinard <fchouinard@gmail.com>
IP-Clean: Francois Chouinard <fchouinard@gmail.com>
Tested-by: Francois Chouinard <fchouinard@gmail.com>

Fix for LTTng/TMF version IDs for R1.2

Change-Id: I933e12a6d0eaef2d8e64baa690d13598b7ebd398
Signed-off-by: Francois Chouinard <fchouinard@gmail.com>
Reviewed-on: https://git.eclipse.org/r/8471

tmf: Make TmfTimestamp and TmfTimeRange immutable

Mark the fields in TmfTimestamp/TmfTimeRange final, to ensure they
do not get modified after the object is created. The setter methods
didn't seem to be used anywhere so the impacts are minimal.

Change-Id: I693ffef78588e9e68fa3b22516cf93b3f98c6613
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/7278
Tested-by: Hudson CI

Update minor version number for plug-in and relevant features.

Change-Id: Id51f03f37d767b26260986f928217613758a3823
Signed-off-by: Bernd Hufmann <bhufmann@gmail.com>
Reviewed-on: https://git.eclipse.org/r/7255

Conflicts:
lttng/org.eclipse.linuxtools.lttng.releng-site/category.xml
lttng/org.eclipse.linuxtools.tmf.core/META-INF/MANIFEST.MF
lttng/org.eclipse.linuxtools.tmf.core/pom.xml

Signed-off-by: Francois Chouinard <fchouinard@gmail.com>
Change-Id: Ifce4d4a18d59ca785517bbf8ac64007175e68328
Reviewed-on: https://git.eclipse.org/r/8462
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

tmf: Statistics provider based on event requests

This patch adds a TmfEventsStatistics, which uses trace event
requests and counts the instances of each event types to
provide statistics information. This is useful for trace types
that cannot use the default implementation using the state
system (traces where events are not chronologically ordered,
for example).

The previous TmfStatistics has been renamed TmfStateStatistics,
to better differentiate between the two alternatives that are
now available.

To keep the request coalescing, the separate threading is now
done is a new ITmfStatistics.updateStats(). This allows the signal
handler code to send the queries, but return immediately and
spawn a thread that will wait for the requests to be done.

When the results are ready, they will be sent back to the view
using the new TmfStatsUpdatedSignal. This makes the handler for
the StateSystemBuildCompleted signal obsolete.

Change-Id: I5b6237aef7f5c7cb373c68e4624376146a6e5086
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/8376
Tested-by: Hudson CI
Reviewed-by: Francois Chouinard <fchouinard@gmail.com>
IP-Clean: Francois Chouinard <fchouinard@gmail.com>
Tested-by: Francois Chouinard <fchouinard@gmail.com>

Fix for supplementary comments on TmftimestampFormat implementation

Change-Id: I46acc468c064d76cf301618fa47377820d17262f
Signed-off-by: Francois Chouinard <fchouinard@gmail.com>
Reviewed-on: https://git.eclipse.org/r/8435
Tested-by: Hudson CI

tmf: Allow waiting for a state system's completion

ITmfStateSystem.waitUntilBuilt() will now block the calling
thread until the state system underneath is finished building.
This will be immediately useful for statistics (intermediate
queries are useless, we need to wait until it's ready anyway).

It's implemented using a CountDownLatch, similar to the
.waitForCompletion() in event requests.

Change-Id: I98420acc3ca2903d76a315259f25dca7d88b8d42
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/8427
Tested-by: Hudson CI

tmf: Store totals separately in TmfStatistics

This allows the get*Total() methods to query the state directly.
With that, no need to get the map of events per type (which was
already longer by itself), and then add its contents at every
single query.

Change-Id: I2e6708a2708abcdae6cd68baab9b6c26a2cc2d30
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/8380
Tested-by: Hudson CI

Fix for the time range histogram

The time range histogram doesn't refresh very well when the new range
has no events either at the start or the end. This patch properly keeps
the range.

Change-Id: I414bb9288327dd9cf56ca8eb22ac2672f4e9e362
Signed-off-by: Francois Chouinard <fchouinard@gmail.com>
Reviewed-on: https://git.eclipse.org/r/8428
Tested-by: Hudson CI

Implement TmfTimestampFormat

Change-Id: I29d4f815679503c82a14ac74e8395f9aa3dc0cbf
Signed-off-by: Francois Chouinard <fchouinard@gmail.com>
Reviewed-on: https://git.eclipse.org/r/8419
Tested-by: Hudson CI
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

ctf: Add support for scoped event contexts

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

Add support for selected event in Properties view.

Change-Id: I41e8cc7a9d9e0c9f8870445741f9559f951eba0f

Temporary fix API version for Kepler

Change-Id: I303c9bbf3b023a4c551586fa04b90d7b8649f69a

Merge branch 'master' into lttng-kepler

Conflicts:
lttng/org.eclipse.linuxtools.tmf.ui/plugin.xml

Change-Id: Id994aa08fb1af64b99be885c30fbbeda4e67ce24

Workaround for bug 391232: ISharedImages constants not recognized.

Change-Id: I671fb92236d22c4829bd38a3a076e951595434a6

Temporary fix API version for Kepler

Change-Id: I276488d50b554f065d8060adcf0ec5133714dce5

Merge branch 'master' into lttng-kepler

Conflicts:
gcov/org.eclipse.linuxtools.gcov.core/src/org/eclipse/linuxtools/gcov/Activator.java
lttng/org.eclipse.linuxtools.tmf.core.tests/stubs/org/eclipse/linuxtools/tmf/tests/stubs/trace/TmfTraceStub.java
lttng/org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/tmf/core/ctfadaptor/CtfTmfTrace.java

Change-Id: Ifd5503a8abd3250f9c117f369de7fb9745507b8e

Add support for selected event in Properties view.

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

Fix for bug 392591: ArrayIndexOutOfBoundsException in virtual table.

Change-Id: Ie7f6e36765b5b0a111ba6a63e867f0ac99864fb8
Reviewed-on: https://git.eclipse.org/r/8329
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>

tmf: Move the stats tree structure logic to the nodes

Re-organize the TmfStatisticsTree so that the information
about node children are stored in the nodes themselves,
and not all at the top-level. This makes debugging, and
eventually extending the tree, much easier, imo.

This patch also drops the fKeys (which limited the types
of children each node can have) and the weird limitation
where only nodes whose path depths are mutiples of 2 could
have values. Now the tree is more generic, to give better
flexibility to potential sub-classes.

Change-Id: I30bc0f7f47256388b6c55e09f09b26286723c0b9
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/8298
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: Refactor TMF statistics

Merge TmfBaseStatisticsTree into AbsTmfStatisticsTree. The abstract
version required both total and per-event-type statistics values
anyway, so there was no need to have a separate class that basically
implemented the required stuff.

Also renamed TmfStatisticsTreeNodeFactory to TmfStatisticsManager,
since it's more a manager for the complete stats trees.

Change-Id: If9942beb14ee0d978722d97e6bdfeafb693c9b66
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/8265
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: Drop the use of TmfFixedArray for statistics

The same behaviour can be achieved by using straight [] arrays
and varargs methods. That way we don't need to instantiate new
wrapper objects all the time.

Unit tests were also updated.

Change-Id: I5f98ca2c08fd5bee5a1569c64466e63ef1e345d8
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/8264
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: Store the generic statistics timestamps in nanoseconds

Since the TmfStatistics can be used with any trace type, and not
just LTTng kernel traces, we can't know exactly what scale its
timestamps are in.

To be safe, convert all timestamps to nanoseconds before saving
them in the history database. When we want to display the
information, the query timestamps will also be converted to
nanoseconds.

Change-Id: Ibcd2fb309f01cdfeab6b480e41f740296694e4be
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/8242
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: Make TmfStatisticsViewer use the trace's stats provider

Rework the TmfStatisticsViewer to get its information from a
ITmfStatistics object (instead of doing direct event queries
on the trace). That way, the logic to compute the statistics
is now separate from the view, which I think is a good thing.

This removes the need for the ITmfExtraInfo completely. Unit
tests have also been updated to take account for the fact that
the view does not 'count' anymore, it just displays whatever
we send it.

Other changes:
 - Consolidated the setters in TmfStatisticsTree
 - Renamed TmfStatisticsNode.getValue() to .getValues(), to
   better indicate that it is the wrapper object around the
   values that is returned.

Fixes bug 387634.

Change-Id: I1e6b9931007c79cce8c1c93c6c0ddc955cbd0a93
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/6986
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>

Convert Windows line delimiters to Unix.

Change-Id: Ia737c3c468c739a7da20561e11d756036fc8d049

LTTng: Handle cancellation of login gracefully (bug 392197)

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

TMF: add support for popup menu in time chart widgets

Change-Id: I4c546a73f23ea596978cff3aa7920a481b81bdae
Reviewed-on: https://git.eclipse.org/r/8202
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>
Reviewed-by: Xavier Raynaud <xavier.raynaud@kalray.eu>

Fix removal of filter in SDView (Bug 391714)

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

Fix ArrayIndexOutOfBoundsException for sync signals (Bug 391716)

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

Initialize dialog member variable (bug 391589)

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

Fix for bug 391277: Rounding error in seekEvent by ratio.

Change-Id: I4fda9f56d5aaf199abcc59c3a9dacabeb681c9b8
Reviewed-on: https://git.eclipse.org/r/8069
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bhufmann@gmail.com>
IP-Clean: Bernd Hufmann <bhufmann@gmail.com>

tmf: Add an ID to each state system that gets built

When requesting to build a state system, we now need to specify
a String ID. This ID will be returned with the "state system build
completed" signal, so that a view can differentiate between many
state systems that could be associated to the same trace.

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

tmf: Introduce the ITmfStatistics interface

With this, any type of ITmfTrace can now define its own
statistics provider. This can then be used to populate views
like the Histogram or the Statistics view.

A base implementation is proposed in TmfStatistics (and used by
TmfTrace), and it uses a state system for its backend. That
state system is completely contained in the TmfStatistics object,
so that it's not confused with any "real" state system that could
be present in the trace.

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

tmf: Add an ID to the AbstractStateChangeInput

The Abstract version was still, wrongly, using "CTF Kernel"
as its identifier. Replace this with a configurable name,
provided as a constructor parameter.

It's only used for naming the thread that it starts, but it
makes it a bit more clear what is happening when debugging.

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

tmf: Expose getStateSystem() up to ITmfTrace

The default implementation in TmfTrace will not provide any
state system for now, sub-classes need to specify their own.

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

tmf: Add the Tmf- prefix to the state system interfaces

In preparation with them being exposed into ITmfTrace, the
interfaces to the state system should wear the proper Tmf-
prefix.

The read-only interface that most views and users will see has
been renamed to simply ITmfStateSystem. The builder interface
that is used by the trace for its construction is now called
ITmfStateSystemBuilder.

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

Fix a confusing activation condition for delete handlers

Change-Id: I12baa3a36fd4ad1a9238bf83a893c2754804eda1
Signed-off-by: Francois Chouinard <fchouinard@gmail.com>
Reviewed-on: https://git.eclipse.org/r/8059
Tested-by: Hudson CI

tmf: Make ITmfGraphEntry.getChildren() return a List

instead of a static array.

Because of the upcoming AbstractTimeGraphEntry and its subclasses,
we will now need to specify the exact class in each view's
.getChildren(). This is not possible to do with an array if we use
a generic type (e.g. E[]), we have to use a Collection like List<E>.

Change-Id: Icf858b471a95ea7d68e6b6b2169afdc35922545a
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/7747
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>

Fix for bug 390841: Experiment does not dispose trace contexts.

Change-Id: Ib2ac07efc2f4b4efbb15971bbaa48401997fa215
Reviewed-on: https://git.eclipse.org/r/8002
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>

Make the TmfLocation final and get rid of clone()

- Make fLocation final
- Make TmfLocation concrete classes 'final'
- Remove clone() - not needed anymore
- Fix test cases

Change-Id: Ie4b67907aa53f8b6d7ba513caa6a8eecdbb624b4
Signed-off-by: Francois Chouinard <fchouinard@gmail.com>
Reviewed-on: https://git.eclipse.org/r/7985

Merge branch 'master' into lttng-kepler

Conflicts:
lttng/org.eclipse.linuxtools.lttng2.kernel.core/src/org/eclipse/linuxtools/internal/lttng2/kernel/core/stateprovider/CtfKernelHandler.java
lttng/org.eclipse.linuxtools.lttng2.kernel.ui/src/org/eclipse/linuxtools/internal/lttng2/kernel/ui/Messages.java
lttng/org.eclipse.linuxtools.lttng2.kernel.ui/src/org/eclipse/linuxtools/internal/lttng2/kernel/ui/messages.properties
lttng/org.eclipse.linuxtools.lttng2.kernel.ui/src/org/eclipse/linuxtools/internal/lttng2/kernel/ui/views/controlflow/ControlFlowView.java
lttng/org.eclipse.linuxtools.lttng2.kernel.ui/src/org/eclipse/linuxtools/internal/lttng2/kernel/ui/views/resources/ResourcesPresentationProvider.java
lttng/org.eclipse.linuxtools.lttng2.kernel.ui/src/org/eclipse/linuxtools/internal/lttng2/kernel/ui/views/resources/ResourcesView.java
lttng/org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/internal/tmf/core/Tracer.java
lttng/org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/tmf/core/signal/TmfTraceSelectedSignal.java
lttng/org.eclipse.linuxtools.tmf.ui/plugin.xml
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/messages.properties
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/parsers/custom/CustomEvent.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/parsers/custom/CustomTxtEvent.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/parsers/custom/CustomTxtTrace.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/parsers/custom/CustomTxtTraceContext.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/parsers/custom/CustomXmlEvent.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/parsers/custom/CustomXmlTrace.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/parsers/custom/CustomXmlTraceContext.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/parsers/wizards/CustomTxtParserOutputWizardPage.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/parsers/wizards/CustomXmlParserOutputWizardPage.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/signal/TmfTraceClosedSignal.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/signal/TmfTraceOpenedSignal.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/editors/ITmfTraceEditor.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/editors/TmfEditorInput.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/editors/TmfEventsEditor.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/project/model/TmfExperimentFolder.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/signal/TmfTraceClosedSignal.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/signal/TmfTraceOpenedSignal.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/viewers/events/ITmfEventsFilterListener.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/views/timechart/TimeChartAnalysisEntry.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/views/timechart/TimeChartView.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/widgets/rawviewer/TmfRawEventViewer.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/widgets/timegraph/ITimeGraphPresentationProvider.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/widgets/timegraph/TimeGraphCombo.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/widgets/timegraph/TimeGraphPresentationProvider.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/widgets/timegraph/TimeGraphViewer.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/widgets/timegraph/dialogs/TimeGraphLegend.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/widgets/timegraph/widgets/Utils.java

Change-Id: Ie424277c584ffa52bd904649fae620d8c96ce1dc

Convert Windows line delimiters to Unix.

Change-Id: Idb33e8cc17d49436d2ed3edd49869cf965c86d13

Merge branch 'master' into lttng-kepler

Conflicts:
lttng/org.eclipse.linuxtools.lttng.releng-site/category.xml
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/viewers/events/TmfEventsCache.java

Change-Id: Ibcacc51ddaf09ece2433d279dfd6719e2b2c1d59

ctf: Handle traces with unknown event attributes

Instead of blowing up when it sees an attribute is doesn't know,
the parser should just ignore it. This is quite important,
since CTF allows for custom attributes and the parser can't
know in advance about all possible ones.

Ideally, we should also log whenever an unkown attribute is seen.
Unfortunately there is no logging in place in the ctf.core plugins
yet.

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

tmf: Rename the UI TmfStatistics class to TmfStatisticsValues

To avoid a clash with the new tmf.core.statistics.TmfStatistics.

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

Add a "Clear Tracing Views" command

Add the possibility to clear the views the tracing/LTTng perspective.
It does it by disposing of the experiment. Proper signal handlers were
added where necessary.

Change-Id: I33e974f26b6e66d55bcd0c34ac96b375bf3ccddb
Signed-off-by: Francois Chouinard <fchouinard@gmail.com>
Reviewed-on: https://git.eclipse.org/r/7962
Tested-by: Hudson CI

Fix for NPE in Sequence diagram when closing Eclipse (Bug 390552)

Change-Id: I132cb8085c44ff5e6af0bf7aa8d2e58c9ebb8310
Signed-off-by: Bernd Hufmann <bhufmann@gmail.com>
Reviewed-on: https://git.eclipse.org/r/7959
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>

Fix 2 actions that had old identifiers

Change-Id: I0fd1e8d933b2971906422859d33387321c3366ab
Signed-off-by: Francois Chouinard <fchouinard@gmail.com>

Fix for bug 390472: ArrayIndexOutOfBoundsException in TmfEventsCache.

Change-Id: I1025ec36868837a9e027399f13d02266c24a152e
Reviewed-on: https://git.eclipse.org/r/7936
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bhufmann@gmail.com>
IP-Clean: Bernd Hufmann <bhufmann@gmail.com>

Rationalize the TMF commands

- Simplify commands categories
- Re-use the platform command id:s where reasonable
- Adjust context menu order
- Adjust shortcuts and accelerators

Change-Id: I6f2e71239dd1bd8392452792bf3868aa6e6dbdf4
Signed-off-by: Francois Chouinard <fchouinard@gmail.com>
Reviewed-on: https://git.eclipse.org/r/7916
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bhufmann@gmail.com>
IP-Clean: Bernd Hufmann <bhufmann@gmail.com>
Tested-by: Bernd Hufmann <bhufmann@gmail.com>

Fix location cloning issue

- Provide a basic implementation of TmfLocation.clone()
- Add an abstract TmfLocation.cloneLocationInfo()
- Add implementations for Tmf<Type>Location
- Adjust Tmf<Type>Location.clone()
- Remove some leftover comments

Update: Make Tmf<Type>Location.clone() a one-liner where applicable.

Change-Id: I54cdb4c70846285c6d3fbd7b114a47b95592f2e7
Signed-off-by: Francois Chouinard <fchouinard@gmail.com>
Reviewed-on: https://git.eclipse.org/r/7901
Tested-by: Hudson CI

Fix location cloning issue

- Provide a basic implementation of TmfLocation.clone()
- Add an abstract TmfLocation.cloneLocationInfo()
- Add implementations for Tmf<Type>Location
- Adjust Tmf<Type>Location.clone()
- Remove some leftover comments

Change-Id: Ide1c1b0225251beff5cc527580207b8cc48a1fe7
Signed-off-by: Francois Chouinard <fchouinard@gmail.com>
Reviewed-on: https://git.eclipse.org/r/7893
Tested-by: Hudson CI

Remove Linux Tools URL from LTTng category.xml file

Change-Id: I7dc18cfcd14f6cec9c0b07b66528d9833d6a687c
Signed-off-by: Bernd Hufmann <bhufmann@gmail.com>
Reviewed-on: https://git.eclipse.org/r/7915
Tested-by: Hudson CI

Update remaining LTTng/TMF versions to 2.0.0

- LTTng parent pom.xml
- LTTng releng pom.xml / category.xml
- all references to LTTng parent pom.xml

Change-Id: I96973d7e33f5aabe813e5439432bb98ebf0e33b4
Signed-off-by: Bernd Hufmann <bhufmann@gmail.com>
Reviewed-on: https://git.eclipse.org/r/7912
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: Bump lttng2 and lttng2.kernel plugins to 2.0

Change-Id: I53cfe696d0ecd56b1c2099bb8299bb9a11df5d26
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/7803
Reviewed-by: Bernd Hufmann <bhufmann@gmail.com>
IP-Clean: Bernd Hufmann <bhufmann@gmail.com>

Removed a pile of unnecessary @SuppressWarning

Change-Id: Id6698243bb57be8a2fcb0b4bb1a933738c11a66a
Signed-off-by: Francois Chouinard <fchouinard@gmail.com>
Reviewed-on: https://git.eclipse.org/r/7842
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Rename 'locationData' to 'locationInfo' (for lack of a better name...)

Change-Id: I1980f6bd4f9a522804d6b782be1bb32261f917df
Signed-off-by: Francois Chouinard <fchouinard@gmail.com>
Reviewed-on: https://git.eclipse.org/r/7841
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Remove the generic location (replace by Comparable)

Change-Id: I05d4c813d0e2406f47e78beae992da6369f977ce
Signed-off-by: Francois Chouinard <fchouinard@gmail.com>
Reviewed-on: https://git.eclipse.org/r/7840
Tested-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
IP-Clean: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Make TmfLocation abstract and implement a couple of Tmf<type>Location

Change-Id: I1a8c3fdef01a9ec9c548592d1c71a248e6d9bd60
Signed-off-by: Francois Chouinard <fchouinard@gmail.com>
Reviewed-on: https://git.eclipse.org/r/7839
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>

Merge branch 'master' into lttng-kepler

Conflicts:
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/widgets/timegraph/widgets/TimeGraphControl.java
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/widgets/timegraph/widgets/TimeGraphTooltipHandler.java

Fix for bug 389907: Mouse drag zoom and other time graph mouse fixes.

Change-Id: Ib7ad84872ff680c1ea1cfdd27d8996a55a64b013
Reviewed-on: https://git.eclipse.org/r/7831
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bhufmann@gmail.com>
IP-Clean: Bernd Hufmann <bhufmann@gmail.com>
Tested-by: Bernd Hufmann <bhufmann@gmail.com>

tmf: Define the StateChangeInput's event type at the constructor

instead of using an abstract method just for that.

With this we can also check in the abstract class itself if the
event type is the right one before calling the method from the
sub-class.

Change-Id: I0cbce6c87e3ec51acfac4061c9db38f32bbcce9a
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/7807
Tested-by: Hudson CI

tmf: Add an abstract class for the StateChangeInput

The state-change-input plugin needed by the state system is
currently offered through a IStateChangeInput interface.
However much of the code is common between implementations.
Furthermore, the LTTng kernel plugin nicely splits the work
in two threads, so it would be great to be able to easily
re-use that feature.

This introduces a AbstractStateChangeInput class which will
provide a good ground base for building a state input plugin.
It will also wrap the multi-thread part of the code, so other
inputs can benefit from it.

The interface remains available for more exotic use cases.

Change-Id: I8c9210af62b63b48ffcdc54b3da35df7754e1583
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/7798
Tested-by: Hudson CI

tmf: Rename TmfLocation.fLocation to .fLocationData

and the corresponding methods (getLocationData(), etc.)

Simple rename, to make it more clear the contained value is
(usually) not a Location itself. This way we don't get things
like foo.getLocation().getLocation().

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

Re-factor and freshen up TmfCoreTracer (previously Tracer)

Summary of main changes:
- Renamed o.e.l.internal.tmf.core.Tracer to TmfCoreTracer
- Removed the error/warning/info tracing classes (already handled by
activator)

Change-Id: Ic75a05c6add22b6c29b699d928bb830124db6a73
Signed-off-by: Francois Chouinard <fchouinard@gmail.com>

tmf: CtfTmfEvent improvements

- Remove extraneous Javadoc
- Change fTimestamp from a singleton to a proper field, set at
  the constructor.
- Drop unneeded singletons (fReference and fSource)
- Remove getTimestampValue(), call getTimestamp.getValue() instead,
  it is more compatible with ITmfEvent/Timestamp interfaces.

Change-Id: I0522a40f3817bcc34bde48dc90358fb745f7bdbc
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/7648
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bhufmann@gmail.com>

Fix for bug 387934: Single event experiment not shown in events table.

Change-Id: Ic0984c905f2f7d5b7ca60734eda9e2b54f692243
Reviewed-on: https://git.eclipse.org/r/7386
Reviewed-by: Bernd Hufmann <bhufmann@gmail.com>
IP-Clean: Bernd Hufmann <bhufmann@gmail.com>
Tested-by: Bernd Hufmann <bhufmann@gmail.com>
Reviewed-by: Francois Chouinard <fchouinard@gmail.com>
IP-Clean: Francois Chouinard <fchouinard@gmail.com>
Tested-by: Francois Chouinard <fchouinard@gmail.com>

Merge branch 'master' into lttng-kepler

Conflicts:
lttng/org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/viewers/events/TmfEventsTable.java

Change-Id: I553c88f56037dc35b9f646887b12d83e623d3543

ctf: Correctly handle big-endian traces

Fixes the opening of big-endian traces, coming from PowerPC systems
for example.

Closes bug 389002.

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

ctf: Fix feature.xml to allow building with pde-build

When trying to build the CTF feature with pde-build, it currently
fails because of unsatisfied dependencies.

We need to:
- Remove the e.o.l.ctf.parser dep completely (the feature ships
  this plugin, it cannot depend on it before it's built!)
- Change the order of the plugins so .parser is built first, since
  the .core depends on it.

Side note, this .parser plugin is the only one (apart from the help
plugin) that does not get put into a jar when built. I really have
no idea why...

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

oups, revert last commit

Fix for bug 386075: Show the TID number in the Resource View's tooltip.

Change-Id: I83c30e7d33fed5184e0c430e34c9829f57a11283
Reviewed-on: https://git.eclipse.org/r/7714
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bhufmann@gmail.com>
IP-Clean: Bernd Hufmann <bhufmann@gmail.com>
Tested-by: Bernd Hufmann <bhufmann@gmail.com>

TMF: add support for popup menu in time chart widgets

Fix for bug 389328: Provide hover time for time graph tool tip info.

Change-Id: I2eaf701ff090fbac1a48f5bc159d27e64dd3938d
Reviewed-on: https://git.eclipse.org/r/7713
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bhufmann@gmail.com>
IP-Clean: Bernd Hufmann <bhufmann@gmail.com>
Tested-by: Bernd Hufmann <bhufmann@gmail.com>

Fix for bug 389300: Events table does not synchronize on search.

Change-Id: Ic403e97022f665278059f356b71ef33976242c66
Reviewed-on: https://git.eclipse.org/r/7715
Tested-by: Hudson CI
Reviewed-by: Bernd Hufmann <bhufmann@gmail.com>
IP-Clean: Bernd Hufmann <bhufmann@gmail.com>
Tested-by: Bernd Hufmann <bhufmann@gmail.com>