Patrick Tasse [Tue, 5 Apr 2016 22:24:24 +0000 (18:24 -0400)]
tmf: Fix leaking static Histogram instances
The two Histogram instances in HistogramView are static and therefore
are retained by the class in the ClassLoader with references to the last
histogram data model, even after the view is closed.
Change-Id: I7853628e99a7bd8f3815253060bf891eed217cae
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/70058
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Thu, 7 Apr 2016 20:19:12 +0000 (16:19 -0400)]
linux: make TID analysis handle no TID more gracefully - Bug 491275
This stops AttributNotFoundExceptions from being thrown in the beginning
of a trace.
Change-Id: I917fa81b3882ce3c98f23f782a87a6f7f8947b10
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/70170
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Geneviève Bastien [Thu, 31 Mar 2016 18:45:47 +0000 (14:45 -0400)]
analysis.io: Provide information from the IO model
This extracts a base Disk class from the DiskWriteModel, that can be used
by outputs and views to get data from the analysis.
It also adds an Information Provider for the input output analysis module
It comes with its units tests
Change-Id: Iffb8428eb161bbd5cfcb834deebfa55d812f4d00
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/69669
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Thu, 7 Apr 2016 18:54:36 +0000 (14:54 -0400)]
linux: use DefaultEventLayout.getInstance() in TidAnalysis
Make the newly merged TidAnalsysis use the layout in the proper way.
Change-Id: Iff7d882946de3783dc23586c0aa1a468c61c17f5
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/70166
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-by: Hudson CI
Matthew Khouzam [Wed, 6 Apr 2016 00:20:11 +0000 (20:20 -0400)]
linux.core: Introduce TID analysis
This analysis calculates a subset of what the kernel
state system has: which tid is running on which cpu.
This will allow these aspects to be available much faster
during state construction and in the reading after.
The new state system is typically 1% of the size of the kernel
state system. Therefore the seeks, even though they are log(n)
will accelerate. As getTid is something MANY analyses do often
this patch should improve performance accross the board.
In one corner case example, (System call analysis), where each state
required a TID, performance passed from 3000 to 140000 events per seconds.
On average, searches on the TID column should reduce in time by 5-10%
depending on the trace configuration (how many contexts etc...).
Change-Id: Icf7921c49232481129b9c021170556c9d06d5ca4
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/68878
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Hudson CI
Alexandre Montplaisir [Mon, 4 Apr 2016 22:14:29 +0000 (18:14 -0400)]
os.linux: Do not define the default kernel layout in the interface
Instead point users directry to DefaultEventLayout.
This avoids a circular reference from IKernelEventLayout layout
to one of its own implementations.
Change-Id: I3c87767df4285c18b9216209942b069fef4d34cf
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/70080
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Wed, 6 Apr 2016 22:14:00 +0000 (18:14 -0400)]
os.linux: Do not rely on static initialisation order for layouts
Expose DefaultLayout#getInstance() instead of DefaultLayout#INSTANCE
This solves a weird problem where the layout could be null.
To reproduce this problem, define and run the following test suite:
@RunWith(Suite.class)
@Suite.SuiteClasses({
VirtualMachineAnalysisTest.class,
LttngKernelAnalysisTest.class,
PartialStateSystemTest.class,
StateSystemInMemoryTest.class,
StateSystemFullHistoryTest.class
})
public class TestSuite { }
Change-Id: I772ee0944334283d4b36118001a858322921d98a
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/69864
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-by: Hudson CI
Alexandre Montplaisir [Mon, 4 Apr 2016 23:05:13 +0000 (19:05 -0400)]
lttng: Use long values for integer fields in the Graph test trace
Since state providers expect all integer fields to be Long objects
(as ctf.tmf does), test traces should also supply long values.
"cpu" should remain an int however.
Change-Id: Icee091a56710cd4a3c16d4f00ab00ca792b13d1e
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/69866
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Sun, 20 Mar 2016 21:12:01 +0000 (17:12 -0400)]
graph: add messages to IllegalStateExceptions
Change-Id: If8efc90b6e55395448abbf3ced6695678dc8b496
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/68876
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Geneviève Bastien [Mon, 21 Mar 2016 13:42:42 +0000 (09:42 -0400)]
analysis.io: Unit tests for the I/O analysis
It uses a few XML kernel stub traces to cover the various cases and events
of the IO analysis, then the content of the state system is verified at certain
times and for certain attributes.
Change-Id: I33f8deb62d9df6958fe88012ada3db5f077d0c09
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/69668
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Houssem Daoud [Mon, 21 Mar 2016 13:42:01 +0000 (09:42 -0400)]
analysis.io: Introduce the input/output linux analysis
This analysis uses the block_* events to track the requests to the disks
and the number of sectors read and written. It also stores the reads and
writes by process from the syscalls.
Change-Id: I6e1b00d53eb3a217b01a906d4d1fd566e53481ef
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Signed-off-by: Houssem Daoud <houssemmh@gmail.com>
Reviewed-on: https://git.eclipse.org/r/69667
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Wed, 6 Apr 2016 01:23:13 +0000 (21:23 -0400)]
linux.core: Simplify KernelThreadInformationProvider
This makes KernelthreadInformationProvider only check pertinent state
values. The logic should be 100% equivalent, but the code should be
much more legible.
Change-Id: I96a962c6f2fa80b2a57d753a324c60c394dae349
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/69975
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Matthew Khouzam [Thu, 28 Jan 2016 20:40:29 +0000 (15:40 -0500)]
ctf: mini bitbuffer optimisation
Change-Id: I10806685649ae1f46d679189e6d4f022a060060e
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/65415
Reviewed-by: Hudson CI
Reviewed-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Jean-Christian Kouame [Wed, 9 Mar 2016 19:22:43 +0000 (14:22 -0500)]
tmf : Add parameters to XML core methods
Add ScenarioInfo as parameters to XML core module methods. This will
help to get the active scenario details (quark, scenarioName,
scenarioState, ...) and use them when necessary during a pattern
analysis. If there is no scenario, the value is set to null.
Change-Id: I17f65b73cb236e81221c355a3cb2fce9800d4fe5
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/68090
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Jean-Christian Kouame [Wed, 23 Mar 2016 15:29:32 +0000 (11:29 -0400)]
tmf : Add latency statistics view for the pattern analysis
Change-Id: Ie4fe7e7e68e8e3b074ddb5f0b3ec97a554652166
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/67842
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Fri, 25 Mar 2016 21:34:28 +0000 (17:34 -0400)]
tmf: Move the symbol provider messages to their own package
Message classes should be package-specific as much as possible.
Change-Id: I7eeae05c357799e4794af908fc53d376ab7cc28a
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/69559
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Mon, 4 Apr 2016 23:03:14 +0000 (19:03 -0400)]
ctf: Explicitly create a Long object for Integer fields
Just in case, and to be more clear, explicitly box the primitive
into its expected Long object.
Change-Id: I8cd2cbee034479ca7e6361b02e7ef1b74b51e178
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/69865
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Thu, 24 Mar 2016 13:52:13 +0000 (09:52 -0400)]
tmf: Bug 490372: Hide PieCharts when trace has only 1 event type
Change-Id: Ie5a2587fd45ccc004910421fd2ca0e23911eac7f
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/69247
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Geneviève Bastien [Wed, 24 Feb 2016 16:24:41 +0000 (11:24 -0500)]
tmf: Add an attribute pool for state system analyses
This new class allows to reuse attribute quarks in a state system. It can
typically be used by analyses who want to save data in the state system
for short intervals of time. Instead of creating a number of attributes with
lots of empty space before and after, they can reuse the same. It makes for
more compact time graph views as well.
Change-Id: Ie7f32eccb02096ef62df4c479a8e2bcfcd043af6
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/67263
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Matthew Khouzam [Tue, 9 Feb 2016 02:34:57 +0000 (21:34 -0500)]
ctf: Replace StructDeclaration map with an array
This will cause a 10% performance improvement while reading a trace
The LinkedHashMap of fields in a struct declaration is much slower
to iterate through than a regular array. This patch replaces the map
with an array. This yields a performance gain of approx 10%.
The patch also changes some methods behavior.
* getMaximumSize() clamps to Integer#MAX_VALUE instead of overflowing
* addField no longer overwrites a value already in the declaration.
As addField has been modified, extra attention has to be put on
the parser in the case of degenerate test cases to maintain the
current behavior.
Change-Id: Id76b3432b2c973a1e2cbecba5a9b22ad76a68162
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/66168
Reviewed-by: Hudson CI
Matthew Khouzam [Thu, 31 Mar 2016 22:04:39 +0000 (18:04 -0400)]
xml.core: test double valued xml fields
This test also tests the forcedType tag
Change-Id: I84249aa8a1c06bad80def9aba959fd368a06121c
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/69664
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Mon, 28 Mar 2016 11:41:44 +0000 (07:41 -0400)]
tmf: Support type double in XML analysis and XML XY views
Change-Id: Ic15969624191fba35ec4b5b7b90394e3253a7e61
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/69372
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Matthew Khouzam [Wed, 23 Mar 2016 21:39:01 +0000 (17:39 -0400)]
common.core: annotate String#substring
String.substring always returns a new String or "this".
Change-Id: Id37b8a20489f07b6bfe8a225064d5be1cd4fc7aa
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/69196
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Geneviève Bastien [Mon, 21 Mar 2016 13:35:15 +0000 (09:35 -0400)]
analysis.io: Add I/O events and fields to the event layout
And update it for LTTng event layout
Change-Id: Idabcd30fb321e03ef29e65b936b2832f96bc42a5
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/69146
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Julien Daoust [Wed, 16 Mar 2016 19:15:15 +0000 (15:15 -0400)]
common: Add documentation to the view KernelMemoryUsage
I add the description and the functionalities of the new
kernel Memory Usage view.
Change-Id: I457372fd1d99c262ca35d768f83e2d53b130c4cb
Signed-off-by: Julien Daoust <julien_daoust@hotmail.com>
Reviewed-on: https://git.eclipse.org/r/69335
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Najib Arbaoui [Thu, 31 Mar 2016 22:56:19 +0000 (18:56 -0400)]
analysis.os: New kernel memory usage view with Unit tests
This view keeps track of page allocation/deallocation events in the kernel.
It consists of a plot and a menu for thread selection. For a given time range,
the menu only lists threads in which there was activity. Values in the plot
are relative to the beginning of the selected trace.
Change-Id: Ie5605b7211828fd01accb920a5a735500fcae4b2
Signed-off-by: Najib Arbaoui <arbaouinajib@gmail.com>
Signed-off-by: Samuel Gagnon <samuel.gagnon92@gmail.com>
Reviewed-on: https://git.eclipse.org/r/65957
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Geneviève Bastien [Fri, 1 Apr 2016 18:22:18 +0000 (14:22 -0400)]
perf: Fix scenario names that should not contain %
The Scenario constructor in the o.e.test.performance contains this line of
code:
Assert.assertFalse(scenario.indexOf('%') >= 0);
This constructor does not appear to be called at insert, or it would fail, but
it is when reading the scenario data, which explains the performance tests job
failures when retrieving the data to display on
http://istmffastyet.dorsal.polymtl.ca.
Also changes the short names, that are limited to 40 characters in the database
Change-Id: If2a259e92595377a9a1872b008576a4564390038
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/69750
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Geneviève Bastien [Mon, 21 Mar 2016 16:15:50 +0000 (12:15 -0400)]
lttng: Add a common plugin for LTTng
This plugin will contain common stuff for both kernel and UST, such as
contexts, requirements, etc.
Change-Id: Ie1546e17436621e7473e0bfccad34897039779c5
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/68937
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Geneviève Bastien [Mon, 21 Mar 2016 17:14:52 +0000 (13:14 -0400)]
tmf.core.tests: Allow TmfXmlTraceStub to have additional aspects
These aspects can be added individually for each unit test as required.
Change-Id: Ib8a6894239958593f88d91c4d8bcd91d3b7b845a
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/68947
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Marc-Andre Laperle [Sat, 19 Mar 2016 22:02:02 +0000 (18:02 -0400)]
Add timeout exception when waiting for jobs
It can happen that jobs never finish and tests calling waitForJobs
will run forever (or time out if a Jenkins/Hudson job is set to
timeout). Normally it would be possible to set a timeout on individual
JUnit tests but this doesn't seem to play well with SWTBot (to be
investigated).
To reproduce the issue before this patch, add this line to the
beginning of SWTBotUtils.waitForJobs method:
new Job("Foo") {@Override protected IStatus run(IProgressMonitor monitor) {while(true){}}}.schedule();
This simulates a never finishing job. Then run OpenTraceStressTest.
The test should never time out.
With this patch, waitForJobs will timeout after 5 mins (300000ms) by
default. When it does timeout, it will print each job, its state and
stack trace.
To reproduce the timeout added with this patch, apply the line as
mentioned above, reduce MAX_JOBS_WAIT_TIME to a smaller value (5000ms)
then run OpenTraceStressTest.
Change-Id: I0cdbaa10c87b93aa2ac6c7f146f7a6e008983fbd
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/68850
Reviewed-by: Hudson CI
Bernd Hufmann [Thu, 31 Mar 2016 13:58:38 +0000 (09:58 -0400)]
rcp: add o.e.remote.jsch.core/ui plugins to the RCP feature
Change-Id: Ice402afafdd96d2a4061a47774a34af44bbddcb0
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/69622
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Marc-Andre Laperle [Thu, 31 Mar 2016 15:20:57 +0000 (11:20 -0400)]
lttng: Fix use of deprecated SWTBot API
Also bump the target sequence numbers to that people get the latest
SWTBot.
Change-Id: Ia615d6642849a97c3e18f49a29ec134b0e11151e
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/69638
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Patrick Tasse [Wed, 16 Mar 2016 20:51:18 +0000 (16:51 -0400)]
swtbot: Add SWTBot tests for collapsible event table header bar
Change-Id: I53e6d536b8995ed02190c22a4b9fa7d1eb59a32b
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/68583
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Geneviève Bastien [Mon, 21 Mar 2016 13:40:17 +0000 (09:40 -0400)]
tmf: Update javadoc for TmfTestHelper#executeAnalysis
Change-Id: I015e918b56a8c4607ba0606ba3c768fecc7f433f
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/69092
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Alexandre Montplaisir [Fri, 18 Mar 2016 22:38:22 +0000 (18:38 -0400)]
Remove "Analysis" from localized analysis names
As discussed in the March 18th project meeting, localized
analysis names should be aligned and not end with "Analysis".
Change-Id: I726dba212ef8bc50be49c5c5d52ad3ede580369c
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/69070
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Alexandre Montplaisir [Fri, 18 Mar 2016 06:42:16 +0000 (02:42 -0400)]
ss: Mark ITmfStateSystem#queryOngoingState as @NonNull
Just like querySingleState, an ongoing state cannot be null
if the passed attribute is valid. At worst the value can be a
NullStateValue.
Change-Id: I138ce6d0d62a0cb45706117fd4d0e4bdf8e8fc58
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/68815
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Alexandre Montplaisir [Tue, 15 Mar 2016 23:46:45 +0000 (19:46 -0400)]
common: Add a new StreamUtils class
We often use Iterable in our APIs, due its minimal behavior
exposition. Unfortunately, there is no Iterable.stream() method,
the official way is done using StreamSupport and the corresponding
spliterator.
Since this is a very commonly used pattern, we can introduce a
small utility method that will make it less verbose.
At the same time, we can move the StreamFlattener into the new
utils class.
Change-Id: Ia4fed4f8a5da56f08ac133eb10efae970004e511
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/68563
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Alexandre Montplaisir [Tue, 15 Mar 2016 21:31:00 +0000 (17:31 -0400)]
os.linux: Move ThreadPriorityAspect to the event.aspect package
Also move the related logic from KernelThreadInformationProvider
to the aspect itself.
Change-Id: I9dbf5854e3b8057fc5cb5dd147040e4b1a63cf68
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/68561
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Marc-Andre Laperle [Thu, 24 Mar 2016 21:46:42 +0000 (17:46 -0400)]
Disable live tracing feature
The Live Tracing feature is quite limited and error prone. Some
of the shortcomings are documented in bug 486728. Since there are no
short term plans to improve this in Neon, perhaps it is better to
disable this for now.
Change-Id: Iaf6cab29ce55fde58296aab9935d220eee29a297
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/69285
Reviewed-by: Hudson CI
Reviewed-by: Jonathan Rajotte Julien <jonathan.rajotte-julien@efficios.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Patrick Tasse [Tue, 22 Mar 2016 19:53:15 +0000 (15:53 -0400)]
swtbot: Stabilize FilterColorEditorTest
A condition is added that waits for the image at the specified bounds
to have changed when compared to a specified current image. This ensures
that the cell has painted with required changes following user input.
ImageHelper now implements hashCode() and equals().
Change-Id: I444ab78f074ebd5c28f8b4aea4927b1c01558486
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/69076
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Matthew Khouzam [Tue, 29 Mar 2016 17:04:31 +0000 (13:04 -0400)]
tmf.ui: Make TmfEventsTable#PackColumns return a value
This allows extending classes to know whether the pack actually happened
or if it was fast returned. The quick return shows that the columns have
already been packed. If the method does pack the columns, we know that it
was done the first time (on a new trace open) or after. This can allow
extending classes to have a hook to a location where the columns have
been initialized as they are not yet in the consutructor or in "init".
Change-Id: Ia4f3ab76e144d937bb5a8eb5e7597c9e99b3114f
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/69469
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Michael Jeanson [Tue, 22 Mar 2016 22:53:28 +0000 (18:53 -0400)]
os.linux: Add per cpu thread 0 modeling
The current model handle thread 0 as a normal thread which is incorrect
since it is the idle or swapper thread and can be active concurently on
multiple CPUs.
This commit adds this concept to the state system and the control flow
view by using a special thread attribute name of "0_X" where X is the
CPU number.
Change-Id: I476927be378482751ee523fac77bbf4d260da54a
Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Reviewed-on: https://git.eclipse.org/r/69089
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Fri, 11 Mar 2016 19:16:09 +0000 (14:16 -0500)]
os.linux: Fix CPU state when Softirq is interrupted
This patch makes the cpu the aggregate of the PROCESS, all SOFTIRQs
and all IRQs. The process now does a rigourous check of every state
in the CPU. This can be later replaced with a virtual or aggregate entry
if deemed necessary.
This patch fixes bug 481855.
Change-Id: I1fd368362daf269c12cc4f12f459c5a1e404e420
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/68002
Reviewed-by: Hudson CI
Marc-Andre Laperle [Tue, 29 Mar 2016 15:13:56 +0000 (11:13 -0400)]
Add Trace Compass 1.2.1 API baseline
Change-Id: I62d2c5471ffdae199013ea016d0d10d79bfe2152
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/69455
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Marc-Andre Laperle [Tue, 29 Mar 2016 06:09:09 +0000 (02:09 -0400)]
tmf: Skip CopyToClipboardTest tests when running GTK3 and Eclipse 4.5
These tests will never work in Eclipse 4.5.x because there was a bug
in SWT that only got fixed in 4.6.0. See bug 486302.
Change-Id: I78d9b95970bf3b99b07108e98d046f8d6dd58b94
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/69395
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Patrick Tasse [Mon, 21 Mar 2016 20:50:14 +0000 (16:50 -0400)]
swtbot: Stabilize use of SWTBotUtils.maximizeTable()
When Ctrl+M is pressed, the resize and layout of the workbench part does
not necessarily occur immediately. For tests that depend on the part
being maximized, the method should block until the resize has occurred.
So far, this issue was only seen when SWTKeyboardStrategy is used.
SWT strategy posts events to SWT and this differs from the AWT strategy.
But the solution in this patch offers an additional safety that is
beneficial to all keyboard strategies.
Change-Id: I3afac9314a82c319064accca1fce00c1dbeb893e
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/69075
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Patrick Tasse [Thu, 24 Mar 2016 21:18:07 +0000 (17:18 -0400)]
tmf: Fix SWTException in TimeGraphMarkerAxis
The zoom thread can call setMarkers() while the view containing the time
graph has been closed in the meantime, causing a SWTException: Widget is
disposed.
Change-Id: I586b3835fdee958e5e7c558f000954a3de5db616
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/69282
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Matthew Khouzam [Thu, 24 Mar 2016 13:33:07 +0000 (09:33 -0400)]
Fix task state unknown with Linux >= 4.1
The control flow view displays the task state as WAIT_UNKNOWN with
newer kernels. This is caused by a new task state value. The mask is
adjusted to fit this new value.
The fix is backward compatible.
Change-Id: I568dd51cd597ab64819cc6a83a3c6f06ec859489
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/67377
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Alexandre Montplaisir [Fri, 11 Mar 2016 05:09:44 +0000 (00:09 -0500)]
Fix new null warnings
Recent 4.6 JDT emits null warnings in a few more places.
Some un-annotated generic types are now implied to be nullable.
Change-Id: I37349ba28ce6d88a60cb44c5df7d4f8f5d7c439c
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/68227
Reviewed-by: Hudson CI
Alexandre Montplaisir [Fri, 25 Mar 2016 00:43:51 +0000 (20:43 -0400)]
releng: Use org.eclipse.remote plugins instead of the feature
The top-level feature now includes the doc.isv plugin, which
in turn depends on all org.eclipse.remote features (serial,
telnet, etc.) which we don't need.
Depend directly on the plugins instead.
Change-Id: Iac7eb44d75b0db46549cdc6d3cfe027e504a95bc
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/69298
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Hudson CI
Marc-Andre Laperle [Thu, 24 Mar 2016 21:59:36 +0000 (17:59 -0400)]
Add missing eclipse.ui.trace dependency
We used to get this dependency through o.e.remote.ui but
they don't depend on it anymore so we don't get it.
Change-Id: I9b2f4b7df5b571871b6757fb8afdd8855082ab65
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/69284
Michael Jeanson [Wed, 23 Mar 2016 18:48:45 +0000 (14:48 -0400)]
Add git-review configuration to repository
Add .gitreview to enable the use of the git-review plugin to interact
with Gerrit. Documentation on this plugin can be found at:
https://github.com/openstack-infra/git-review
Change-Id: I8240a5baf7e1171b932896a6615131453b55adad
Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Reviewed-on: https://git.eclipse.org/r/69173
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Jonathan Rajotte Julien <jonathan.rajotte-julien@efficios.com>
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Marc-Andre Laperle [Thu, 24 Mar 2016 04:47:18 +0000 (00:47 -0400)]
Update version numbers to at least the versions in stable-1.2
Change-Id: Ibd2898b5f67bcddb0ecafd44e43344ad7884c4a7
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/69215
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Marc-Andre Laperle [Sun, 20 Mar 2016 16:33:16 +0000 (12:33 -0400)]
lttng: Delete project at the end of ControlViewTest
So that the test can run twice in a row.
Change-Id: I4382c991c3add7f8fe53fae14963f65ce446e15a
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/68860
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Jean-Christian Kouame [Fri, 4 Mar 2016 21:23:34 +0000 (16:23 -0500)]
analysis : Abstract the latency statistics analysis
Change-Id: I452cf2f165ea74522dbd3cb98547ad2784324a73
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/67841
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Jean-Christian Kouame [Fri, 11 Mar 2016 17:40:09 +0000 (12:40 -0500)]
analysis : Remove checkNull() in Segment store statistic viewer
Previously, if the view was called and the trace is null, an exception
was raised. Now, if the trace is null, the view will do nothing. This situation
happens when using this viewer with XML views, the trace may still be null
when the viewer is updated.
Change-Id: I5f3e7dac7ef93e8b0cf795d7219278aad3fa9c90
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/68228
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Matthew Khouzam [Tue, 15 Mar 2016 00:28:02 +0000 (20:28 -0400)]
ss: no longer have a strings section in the HTNodes
The state systems nodes had a string section to allow for faster seeks
on the state system file if the entire interval was not loaded to memory.
This is never the case and it adds an extra integer to each non-integer state
as well as extra (but well tested) logic.
This patch makes the state sytem store each interval sequentially on a
given node.
Expect minimal performance difference, however, the size of the state system
may shrink a bit (approx 5%).
Change-Id: I01bc8594b7944fad97b6c9b715b848b6afa39913
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/68404
Reviewed-by: Hudson CI
Geneviève Bastien [Mon, 7 Mar 2016 14:47:45 +0000 (09:47 -0500)]
ss: Add unit tests for state values
Change-Id: Ibc7da569fcb3cd2920170a1d28bb48dc6497fe7f
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/67908
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Geneviève Bastien [Mon, 21 Mar 2016 13:43:56 +0000 (09:43 -0400)]
lttng: Remove dependency to version of testtraces
Change-Id: I920211c5453452f2b7c49b60732b7c82f055f20a
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/69025
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Marc-Andre Laperle [Sat, 19 Mar 2016 20:41:35 +0000 (16:41 -0400)]
releng: Build for Neon (4.6) by default
As we are approaching the Neon release, we should compile and test
against Neon as our primary focus.
Change-Id: Ifdf3a1a8cdb700e0a2b98f3d519480face9b61b8
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/68848
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Alexandre Montplaisir [Fri, 18 Mar 2016 06:31:00 +0000 (02:31 -0400)]
common: Annotate Integer.valueOf() as @NonNull
It always returns a new, or existing, Integer object.
Change-Id: Ia37b0fb93780131ce2e7369fb69add515f698e49
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/68814
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Thu, 17 Mar 2016 10:40:45 +0000 (06:40 -0400)]
common: Annotate Stream.map() and .filter()
Those common methods return (continue) a non-null stream.
Change-Id: Ic570291f15ce1be3274973ab45d6f65a93f244ef
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/68665
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Tue, 15 Mar 2016 23:42:07 +0000 (19:42 -0400)]
common: Annotate StreamSupport
All methods in StreamSupport return a new Stream, so they can
all be @NonNull.
Change-Id: I734beaf39c5373858cabc93e3947dd22e9c0529e
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/68562
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Patrick Tasse [Fri, 18 Mar 2016 20:40:34 +0000 (16:40 -0400)]
tmf: Bug 489971: Premature processing of payload in custom parser
Change-Id: Ifc49df1f2a40171bf285d600244350fb5da4e1a5
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/68817
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Bernd Hufmann [Thu, 17 Mar 2016 15:27:22 +0000 (11:27 -0400)]
control: Bug 489857: Fix live mode support for LTTng 2.4
Live streaming was added in 2.4.x. Since we can't retrieve whether
a session is live or not from a 2.4.x node, we copy the information
over from the session information at session creation time.
Note that this information gets lost when the user presses the refresh
button or restarts or Trace Compass.
For LTTng 2.5.0 and later it's possible to retrieve this information.
So we don't need copy this at creation time. This will make sure that
the session is recognized as live after a Trace Compass restart or a
refresh.
Change-Id: I4f07e0502de9c858a614946640305fbbcc8c27f6
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/68685
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Alexandre Montplaisir [Sun, 20 Mar 2016 06:12:16 +0000 (02:12 -0400)]
ss: Fix some tests's copyright header
I did not write any of this code.
Change-Id: Ic688309ee0eff9dfcb6aaed05ab01ada32b0019e
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/68854
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Geneviève Bastien [Mon, 7 Mar 2016 17:20:15 +0000 (12:20 -0500)]
ss: add unit tests for state value types in backends
Change-Id: I3b33478e62331cc6b58d551db3597ae8cdbc892c
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/68097
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-by: Hudson CI
Geneviève Bastien [Mon, 7 Mar 2016 17:19:20 +0000 (12:19 -0500)]
ss: Add a package-info file to backend unit tests
Change-Id: I29b2b13cb32468fa4c4484c89360eb0eaf08076f
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/68096
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-by: Hudson CI
Marc-Andre Laperle [Sat, 19 Mar 2016 19:39:47 +0000 (15:39 -0400)]
releng: Targets updates for Mars.2 and 4.6M6
- Update 4.5 target with final Mars.2 update sites, use
non-composite where possible as they should not change (speeds up
dependency resolution)
- Update staging (4.6) target to not use the simrel update site, as
per recommendation by simrel team. Update update sites to latest
miletones; this required to fix the product dependencies. Luckily
those work with the 4.5 target too.
Change-Id: I39db87c315e7ee94a33c8b6c99b45ab86c81b1ad
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/68843
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Marc-Andre Laperle [Sat, 19 Mar 2016 05:20:50 +0000 (01:20 -0400)]
releng: Update outdated maven plugins
Change-Id: I1fc1f569bfc0a124f76fcdd5703e26be1d9a1295
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/68828
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Alexandre Montplaisir [Tue, 15 Mar 2016 21:09:55 +0000 (17:09 -0400)]
os.linux: Rename the "kernelanalysis" package to just "kernel"
Long due change that was previously withheld to not break the API.
This aligns the naming with the other internal packages, which
were already renamed.
Change-Id: I9a0b3354e7018fa67f5baf7eb8a8522e1553f881
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/68559
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Alexandre Montplaisir [Tue, 15 Mar 2016 23:16:28 +0000 (19:16 -0400)]
Add missing package exports
Change-Id: I145fbf655c015a9ca46ee1d4e2c963f5a734deeb
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/68558
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Patrick Tasse [Tue, 15 Mar 2016 22:50:14 +0000 (18:50 -0400)]
doc: Update user guide for collapsible event table header bar
Change-Id: I45dfb1828398728bc85cdb0befa2bdf51b4b14db
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/68496
Reviewed-by: Hudson CI
Patrick Tasse [Wed, 2 Mar 2016 00:18:54 +0000 (19:18 -0500)]
tmf: Fix Select Next/Previous Marker navigation
Because the markers are sorted by time, when navigating to the next or
previous marker, the marker start time should always be visible.
The action should not rely on the 'ensureVisible' implementation of the
method setSelectionRangeNotify(), as this method will ensure that the
end time of the selection will be made visible when the start time has
not changed. This causes problems if there are two markers with the same
start time but the end time of the second is out of visible range. Since
the window range is modified, the resulting marker list may no longer
include the marker that should have been next in the navigation order.
Change-Id: I434d454d7411f8a00bf58406ad7acc3128472de7
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/67643
Reviewed-by: Hudson CI
Patrick Tasse [Tue, 15 Mar 2016 17:38:15 +0000 (13:38 -0400)]
tmf: Support horizontal scroll with mouse wheel in time graph
Horizontal scroll can now be performed using Shift+MouseWheel while over
the time graph control state area.
MouseWheel over the time scale, marker axis and horizontal scroll bar
will now perform horizontal scroll, while Ctrl+MouseWheel over those
same controls will now perform horizontal zoom.
Table of MouseWheel actions:
modifier: None Shift Ctrl Shift+Ctrl
time scale hScroll hScroll hZoom hZoom
time graph vScroll hScroll hZoom vZoom
marker axis hScroll hScroll hZoom hZoom
h.scrollbar hScroll hScroll hZoom hZoom
v.scrollbar vScroll vScroll vScroll vScroll
outside (Win) vScroll vScroll vScroll vScroll
Change-Id: I0e48dd37b121b934ef872509aa721506876ca199
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/68464
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Patrick Tasse [Mon, 7 Mar 2016 22:12:18 +0000 (17:12 -0500)]
tmf: Add collapsible event table header bar with applied filter labels
- The manual toggling of the event table header mode (search/filter) is
removed.
- When a search filter is applied, the header row icon is changed to a
"Add as Filter" icon. Tool tip support is added for this icon. Clicking
the icon will apply the search as a filter, and clear the search filter.
- Ctrl+Enter shortcut can be used to trigger the 'Add as Filter' action.
It can also be used instead of Enter while in a header cell 'search'
text editor to apply the regex immediately as a filter instead of as a
search filter.
- The 'Show Filter Bar' and 'Show Search Bar' actions in the header row
context menu are removed, and are replaced with the 'Add as Filter'
action when there is a search filter applied.
- If search filters are currently applied on multiple columns, they will
each be applied as separate filters.
- The 'Apply Preset Filter...' and 'Collapse Events' actions now add a
filter to the currently applied filters instead of replacing them.
- Implement TmfFilterObjectNode so that any external filter object can
be included in the filter tree model.
- The collapse filter is handled separately from other filters. It must
be invoked only on events which first match every other applied filter.
This allows for filtered events to be collapsed even if they are not
contiguous as unfiltered events.
- Implement TmfCollapseFilter.toString().
- Implement TmfEventsTableHeader. This is a header bar composite
displayed above the event table when any filter is applied.
- The header bar has one label for every filter that is currently
applied. The label name describes the filter. If the filter was created
from the table search/filter row, then clicking the label will set the
table highlighting to this label's regex (if no search filter is
currently applied).
- Clicking the 'remove' icon on any label will remove this particular
filter. If no filter remains, the header bar will be hidden.
- Pressing the Delete key will clear all filter highlighting (if no
search filter is currently applied).
- When the header bar is visible, clicking its background or collapse
icon will toggle its collapsed/expanded state to save space. Each filter
label name will be set as a tool tip for the smaller label.
- A NullPointerException is fixed when cancelling a search before its
event request has been created.
Change-Id: I40cc3e3389f1ce43145e2a6025ed1c2a99d029e8
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/67928
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Geneviève Bastien [Mon, 7 Mar 2016 17:24:00 +0000 (12:24 -0500)]
ss: Move the HistoryTreeTest class to a new package
The o.e.t.s.c.t.backend package only contains classes for backend tests
Change-Id: I838b242102a1d5fc2ea3ce5271499cba191fa455
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/68095
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-by: Hudson CI
Jean-Christian Kouame [Fri, 4 Mar 2016 22:02:00 +0000 (17:02 -0500)]
tmf : Add latency density view for the pattern analysis
Change-Id: Iedc781b8d8d5ae6a22eb699270e540f4826be38b
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/67840
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Jean-Christian Kouame [Fri, 4 Mar 2016 21:56:48 +0000 (16:56 -0500)]
tmf : Add latency scatter graph view for the pattern analysis
Change-Id: I44ccdf56c3b769e2f0d74dc26e7fe2f1eab568ea
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/67839
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Jean-Christian Kouame [Fri, 4 Mar 2016 21:52:51 +0000 (16:52 -0500)]
tmf : Add latency table view for the pattern analysis
Change-Id: I3e652767dc8e53dc2eb95c27e159f8350e939eaf
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/67478
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Patrick Tasse [Tue, 15 Mar 2016 18:15:28 +0000 (14:15 -0400)]
doc: Update user guide for time graph marker axis
Change-Id: Ic3ca210994cf1d690f566d0703fc51e57e55371f
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/68465
Reviewed-by: Hudson CI
Jean-Christian Kouame [Tue, 1 Mar 2016 19:38:03 +0000 (14:38 -0500)]
tmf : Introduce the modules for the XML pattern analysis
This patch introduces the XmlPatternAnalysis and its sub-modules
Change-Id: If27ed6816316752c405f14a31b9b91dcfad877c7
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/65750
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Matthew Khouzam [Wed, 10 Feb 2016 03:39:42 +0000 (22:39 -0500)]
os.linux: make SoftIrqs support being raised while executing.
In order to do this, statevalues are now bitmasks. Many more checks
at SoftIrq Raise, Entry and Exit handlers now take into account if
there are other SoftIrqs running and will retain their state.
Fixes part 2 of bug 381497
Change-Id: I2f109d1e4e3a227c10dcdf8f54df7b42c7776d4a
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/66259
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Jean-Christian Kouame [Tue, 23 Feb 2016 23:58:19 +0000 (18:58 -0500)]
analysis : Introduce ISegmentListener
This patch add a listener for segments creation
Change-Id: I1da2cf3f6f91599ef4fc1a92f05306e90db0dc03
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/67195
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Alexandre Montplaisir [Wed, 16 Mar 2016 22:09:00 +0000 (18:09 -0400)]
ctf: Increase ReadAllTraces test timeout to 1 minute
This test often times out on Hudson, we can increase it
to a more permissive amount.
Change-Id: Ib0c09bc157803b1b19ff213af2a804ee32458247
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/68595
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Jean-Christian Kouame [Fri, 15 Jan 2016 16:29:05 +0000 (11:29 -0500)]
analysis : Introduce buildAnalysisSegments() in Segment Store analysis
This abstract method helps the analysis decide how the segment store
will be built.
For instance, the segment store could be built using a request or using
segments generated by a prerequisite analysis.
Change-Id: I68c6b5d35ff71d8c06c5b4881bcbce25404b5581
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64592
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Bernd Hufmann [Fri, 11 Mar 2016 18:42:42 +0000 (13:42 -0500)]
lttng: Fix ust memory analysis help text display
When the analysis cannot be executed due to missing requirements, the
user can request the help text to get the reason why. However, this
was not working and caused an exception because the requirements are
only instantiated when the trace is open.
Change-Id: I43ca26b964467711bea10bf7ed52eba2988e3fd9
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/68236
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Fri, 11 Mar 2016 18:27:49 +0000 (13:27 -0500)]
lttng: Introduce a default lttng ust event layout
Change-Id: Ibd7d1813b920b08f7a003850247ab088cd9276fc
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/68234
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Geneviève Bastien [Wed, 2 Mar 2016 16:33:09 +0000 (11:33 -0500)]
tmf: Add the ID to the analysis property
This property is useful when developing data-driven views for existing
analyses.
Change-Id: I72fc0287c719326fe3a4de73cccafbfcffc490e9
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/67823
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Hudson CI
Geneviève Bastien [Mon, 14 Mar 2016 14:10:10 +0000 (10:10 -0400)]
common: Add annotations to HashMultimap#create methods
The returned HashMultimap is always @NonNull.
Change-Id: Ic6be32944771b175101c202d0d16780ba3372d0d
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/68360
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Matthew Khouzam [Wed, 9 Mar 2016 22:03:43 +0000 (17:03 -0500)]
lttng: make Critical path listen to thread selected signal instead of selection listener
This patch greatly simplifies the critical path parameter provider.
Now it will listen to a TmfThreadSelectedSignal instead of listenning to an
action on the control flow view. This will decouple the critical path from the
control flow view, instead allowing anything generating the signal the opportunity
to run a critical path analysis on a given thread and trace.
Change-Id: Ifb7f1fe2d5f81a8127ceab93b262cc9c0bbdd0e5
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/68099
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Matthew Khouzam [Tue, 16 Feb 2016 03:42:04 +0000 (22:42 -0500)]
linux.ui: Add context menu to Control Flow View to follow a thread
The command to follow a thread generates a TmfThreadSelectedSignal. To
see the context menu, one must right click on a thread entry in the control
flow view. More follow action locations can be:
* the events table
* the resources view, on an event
* the filters view
* the critical flow view
Change-Id: Ib74eb67f98c514f29fde689c47c2cb42c90868db
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/66665
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Matthew Khouzam [Tue, 16 Feb 2016 03:51:22 +0000 (22:51 -0500)]
tmf.core: Add cpu and thread to trace context
This patch allows to see what the last selected cpu and thread were
for a given trace. The cpu and thread are stores in a LinuxTraceContext.
To do this, a default method is introduced to ITmfTrace (createTraceContext).
This method is overridden by IKernelTrace to make LinuxTraceContext instead of
the regular TmfTraceContexts. The LinuxTrace Context has the CPU and Thread ID
information. An important thing to note is that if two kernel traces have the same
hostID they both get the signal. This is by design.
Change-Id: I25f044ec05fbd69acf93009861b464506327d616
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/66667
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Geneviève Bastien [Fri, 19 Feb 2016 03:38:36 +0000 (22:38 -0500)]
ss: Add benchmarks for the threaded history tree
This adds a new benchmark for the threaded history tree backend. It benchmarks
the insertion of intervals in the state system, single, full queries and
history range queries.
Change-Id: I4ac8eea4a6979c4f70b6d584b26ad3fc92ae4900
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/66891
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Tested-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Alexandre Montplaisir [Wed, 9 Mar 2016 22:49:15 +0000 (17:49 -0500)]
Remove unneeded checkNotNull() calls
With the introduction of Java 8 type annotations and external
@NonNull annotations, some of our checkNotNull() checks are no
longer needed, we can simply remove them.
Change-Id: I00b39d7428e042ae2d93b15d37dda5cbd50f52c2
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/68101
Reviewed-by: Hudson CI
Patrick Tasse [Fri, 11 Mar 2016 22:13:06 +0000 (17:13 -0500)]
tmf: Add unit tests for PeriodicMarkerEventSource
Change-Id: I38e8fb35f6a9f27c75a69e641819feb8c097a176
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/68254
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Patrick Tasse [Tue, 1 Mar 2016 22:52:23 +0000 (17:52 -0500)]
tmf: Bug 488806: Mouse scroll with count of zero not handled correctly
Change-Id: I11e9b5261b687b0095e29e2f7cb35a3507a18591
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/67639
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Tue, 8 Mar 2016 20:31:42 +0000 (15:31 -0500)]
doc: update user guide for importing traces as experiment from control
Change-Id: I8040ae89b4c1d5c9b5119ec316518386f26ed707
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/67995
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Marc-Andre Laperle [Mon, 14 Mar 2016 01:16:11 +0000 (21:16 -0400)]
lttng: Fix intermittent MemoryUsageViewTest failure
It is possible that a thread gets added to the memory usage chart
before the process name is determined. Once the process name is
determined, a new series is added to the chart but the old one is not
deleted. Not only this fails the test because the chart contains too
many series, but it shows the wrong information to the user.
To reproduce the issue, you can add a Thread.sleep(5000) in
UstMemoryStateProvider around line 222, before the process name gets
added to the state system.
Change-Id: I95631758e0fdffbc094a3adb923b7019841a4a60
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/68312
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Tue, 8 Mar 2016 20:30:26 +0000 (15:30 -0500)]
doc: update user guide for importing traces as experiment
Change-Id: Ie3efa0d4c2cf73fc4ef9b8cf07933af95eabf430
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/67994
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
This page took 0.048528 seconds and 5 git commands to generate.