Bernd Hufmann [Tue, 12 Jan 2016 17:53:59 +0000 (12:53 -0500)]
analysis: support for sorting of control flow view columns
Each column in the control flow view can be now sorted by clicking on
the column header. By default, it's sorted by birth time. For an
experiment the traces are sorted independently. The tree hierarchy is
kept, i.e. tree level is sorted separately.
An API has been added to support sorting of time graph entries
(ITimeGraphEntry) generically.
Change-Id: I05631ca69ea5cd57cdd97b3815cc263e7de9e43e
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64403
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Geneviève Bastien [Wed, 13 Jan 2016 03:42:57 +0000 (22:42 -0500)]
lttng: Fix thread ordering in virtual machine view
Threads in the virtual machine view are now ordered by their numeric thread IDs
instead of the corresponding string which makes it more intuitive.
Change-Id: I681e3423dcea9e946b74bebe4e2d0416680ccfa9
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/64205
Reviewed-by: Hudson CI
Marc-Andre Laperle [Wed, 20 Jan 2016 16:14:39 +0000 (11:14 -0500)]
rcp: Add profile to build only one rcp (one environment)
When running validation builds (Gerrit trigger), we don't need
to build all environments. A failure to assemble the product on
only one platform is highly unlikely and would be caught by the
nightly build, at worse.
This patch adds a profile so that only one environment is built
(Linux 64).
Change-Id: Icba6d6a1b01da605db6fe19cceb2e3a2b3694315
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64784
Reviewed-by: Hudson CI
Geneviève Bastien [Mon, 18 Jan 2016 03:59:26 +0000 (22:59 -0500)]
lttng: VirtualMachineView: extract some methods to simplify the class
This fixes some sonar warnings that methods were too complex. Some
repeated code was also extracted to a method so that both thread and CPU
entries can make use of it.
Change-Id: I18af32bfb2d16d63ee80e0bb1aba0664728ca633
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/64617
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Jean-Christian Kouame [Wed, 20 Jan 2016 18:50:17 +0000 (13:50 -0500)]
tmf : Improve toString() in TmfXmlStateValue and TmfXmlCondition
Change-Id: I25dc5961d71c84a6c3af82d6669a45fedcaa1504
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64804
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Thu, 14 Jan 2016 22:02:29 +0000 (17:02 -0500)]
ctf: Introduce IEventDefinition
This interface is extracted from EventDefinition. It allows all of
IEventDeclaration to return only the interface and leave the internals
in the internal folder.
This should improve the maintainability of the CTF plugin without impacting
performance or testability.
Change-Id: I398c4440c67738eaa80a2e63e6d562352d0d64a8
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64412
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 [Wed, 20 Jan 2016 01:36:36 +0000 (20:36 -0500)]
tmf.ui: Fix dead store in TimeAlignmentSynchronizer
The algorithm in getReferenceView should pick the view with left-most
time-axis position (to maximize viewing area). It was written like
that originally but a break statement snuck in.
Change-Id: I177c512d9b6553fb553d8eda6e2ab833aa3867c9
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64716
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Hudson CI
Geneviève Bastien [Mon, 18 Jan 2016 04:18:10 +0000 (23:18 -0500)]
lttng: Fix sonar warnings in VirtualMachinePresentationProvider
Change-Id: I27d475f49da87af2e81337e4ad8ea856b6722d4f
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/64616
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Geneviève Bastien [Mon, 18 Jan 2016 04:24:31 +0000 (23:24 -0500)]
lttng: Fix sonar warnings in VirtualMachineViewEntry
Change-Id: I85b1bd21b93aaa5680af56bc80468ae70242438e
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/64615
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Wed, 20 Jan 2016 01:25:47 +0000 (20:25 -0500)]
tmf.ui: ensure that the Histogram's text label is layouted
Change-Id: I399c98892b96953c7a310cc0c90ffea53103014e
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64714
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Marc-Andre Laperle [Wed, 20 Jan 2016 05:43:34 +0000 (00:43 -0500)]
Print some environment information at the start of SWTBot
This will help us investigate failing tests in the case where the
environment differs from what is known to work.
Change-Id: I4b6cf42355050bf57d420a5a95df4e1b35618fe2
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64723
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Matthew Khouzam [Wed, 20 Jan 2016 01:31:52 +0000 (20:31 -0500)]
tmf.ui: avoid dead store in deleteHandler
Change-Id: I5918ccd5408502fceda12c447d0d40d27c9e218c
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64715
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 [Mon, 18 Jan 2016 22:35:16 +0000 (17:35 -0500)]
Fix typo in System call density vs time title
Change-Id: I587a9af2bc01d6b5a74e69c42c53fbd4d8f88bb1
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64608
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Jean-Christian Kouame [Wed, 13 Jan 2016 22:47:09 +0000 (17:47 -0500)]
tmf : fix the 'Not' condition in TmfXmlCondition
Now, a 'Not' condition "Not X" will create a subcondition using its
first children instead of itself; which was actually creating the
condition "X".
Also, this patch update TmfXmlConditionTest to validate the fix.
Change-Id: I5b14d345c652a8eb553728c56ba6933b87a56ddd
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64309
Reviewed-by: Hudson CI
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Thu, 14 Jan 2016 21:41:48 +0000 (16:41 -0500)]
ctf: no longer rely on default character encoding in Metadata.java
Change-Id: I0aac8cbcef9871352545101978c09916278e59c5
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64411
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Hudson CI
Matthew Khouzam [Thu, 14 Jan 2016 21:31:02 +0000 (16:31 -0500)]
ctf: no longer invoke new String() in Metadata.java
Change-Id: Iba97dcb55d36fbb2a641f6597dbf04fb4544a8fa
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64408
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, 14 Jan 2016 21:03:48 +0000 (16:03 -0500)]
ctf: fix Modifier order to always be static final
Change-Id: I8ec44295ad279240e41af6e3442d5b9aa3de7dfd
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64407
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, 14 Jan 2016 20:39:35 +0000 (15:39 -0500)]
ctf: make all parser implementations final classes
Change-Id: I9c7bff1388e02832c3bcee80ee891faa91a0f5ed
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64406
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 [Fri, 15 Jan 2016 20:21:35 +0000 (15:21 -0500)]
analysis: Add grid in density viewer
Change-Id: Ic497fc36386f2ad0daaddc5cb263f16cd21b7676
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64500
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Marc-Andre Laperle [Thu, 14 Jan 2016 23:57:43 +0000 (18:57 -0500)]
Exclude more Sonar rules that are not working with jdt.annotation
A recent update to the Sonarqube server introduced a few new rules
that are not working with jdt.annotation. This is similar to the
previous ones that were disabled. See
https://git.eclipse.org/r/#/c/61292/
and
http://sourceforge.net/p/findbugs/bugs/1425/
Change-Id: I983d1656d667f8c8fd48489273d33f49761f22f1
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64427
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Alexandre Montplaisir [Fri, 15 Jan 2016 04:49:58 +0000 (23:49 -0500)]
releng: Update test-traces to 1.2.0
This new version contains the perf-CTF trace from Bug 484854.
Change-Id: Iedcecf76d7dc3731e4a50726e2f749a2ea7e593b
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/64428
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, 14 Jan 2016 20:14:41 +0000 (15:14 -0500)]
tmf: Make columns in TimeGraphCombos moveable by default
An example implementation is the Control Flow view.
Change-Id: I9d801a8c30b6ae3c8a0b30e2bf0eb28d761d1833
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64404
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Marc-Andre Laperle [Mon, 11 Jan 2016 22:45:01 +0000 (17:45 -0500)]
Revert "tmf: Remove an annotation temporarily to work around Sonarqube bug"
This reverts commit
d0da59edb3bd54ed5d8e8d2f81649b67755efebd.
The bug was fixed in Sonarqube (sonar-java 3.9) so this commit can be reverted.
Change-Id: Ic6e003f957f72b8f1d705037da116d1895370c35
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64085
Reviewed-by: Hudson CI
Mohamad Gebai [Thu, 4 Dec 2014 02:52:33 +0000 (21:52 -0500)]
LTTng: Add a virtual CPU view for the virtual machine analysis
This view shows for each guest the status of its virtual CPUs (running or
preempted) as well as a modified view of the threads running on them to
highlight when it was thought to be running, but was actually preempted on the
host.
Change-Id: Id260abc6d43e868ba261714de64c5530596de7ff
Signed-off-by: Mohamad Gebai <mohamad.gebai@polymtl.ca>
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/37602
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Sat, 29 Aug 2015 19:53:47 +0000 (15:53 -0400)]
ctf: split up IOStructGen into 44 files
IOstructGen is a very large file, it is hard to test and maintain, this patch
splits it into 42 files.
This patch has several benefits. It does the following:
* decreases the complexity of the file IOStructGen.
* improves the modularity of the TSDL parser.
* is very friendly to code navigators as the links of who calls who
are more obvious.
* improve testability of the parser.
* improve navigability in code.
* use a single pattern so each section is drop-in replaceable.
* add null checks at strategic places.
* ridiculously improve javadoc.
* add null annotations
* improve validation of data
Performance is similar.
Change-Id: Ie13ab1673d40b36087e7ce78d81b5379063ea3a1
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/56740
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 [Wed, 6 Jan 2016 23:38:17 +0000 (18:38 -0500)]
ss: Bug 485304: Incorrect interval returned in history tree full query
Change-Id: Ic68f991a168d5156a2d7a22dbc62b63b307f432f
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/63687
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-by: Hudson CI
Marc-Andre Laperle [Tue, 12 Jan 2016 21:48:05 +0000 (16:48 -0500)]
Update sonar-maven-plugin to 2.7.1
This fixes a bug that I was experiencing on a different Hudson server
https://jira.sonarsource.com/browse/MSONAR-122
Change-Id: Iee0aa27b1d025bc25b843432892bbff428bc8d69
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64383
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Marc-Andre Laperle [Wed, 18 Nov 2015 06:06:30 +0000 (01:06 -0500)]
tmf: Try expanding a second time when tree expand fails
This will stabilize StandardImportAndReadSmokeTest.
Sometimes in a JFace TreeViewer, it expands to nothing.
It looks like a timing issue with an update job started by the viewer.
This is a work around until we can investigate some more.
Change-Id: I01ead54ec7ff7baaf64e1184c3a5cae8906b86a9
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/60677
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Wed, 13 Jan 2016 20:15:33 +0000 (15:15 -0500)]
analysis: Update unit formatter for segment store
Change-Id: I5bd654504e4fb91f9c5b66ffac9737dd69e1ad5a
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64302
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Geneviève Bastien [Tue, 12 Jan 2016 15:00:54 +0000 (10:00 -0500)]
xml: bug 484257: allow to define values anywhere in the file
Values and locations do not have to be defined at the beginning of the file.
They should however precede any eventHandler related to it.
Change-Id: I27ce894f03b093d180d651a00325608228308273
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/64139
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Thu, 7 Jan 2016 21:20:13 +0000 (16:20 -0500)]
analysis: make scatter charts y axis use a human readable time format
Change-Id: I450dbf1e931420288f66e7b1c3c08e3d2d772191
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63798
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Matthew Khouzam [Thu, 7 Jan 2016 21:05:36 +0000 (16:05 -0500)]
analysis: make statistics show standard deviation
Also format time in a readable manner. Warning, this changes the behavior
of HumanReadibleTimeFormat. It now shows its decimals all the time.
Change-Id: If810ed09462fdaa09924945a2e5930debd7990e5
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63788
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Matthew Khouzam [Thu, 7 Jan 2016 20:40:04 +0000 (15:40 -0500)]
analysis: make statistics view use a more legible format
statistics will now display times using units and 3 significant
decimals as opposed to the way they were displayed before (unitless
left-aligned nanoseconds)
Change-Id: I35cfeeb7d038ac2b9f52f3648c3e4337a6ec279c
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63787
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Matthew Khouzam [Thu, 7 Jan 2016 20:32:05 +0000 (15:32 -0500)]
analysis: rename DensityTimeFormat to SubSecondTimeWithUnitFormat
Make it public.
Change-Id: I6adf91b7b15ccc96105f35940641d65704b1ab3d
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63786
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Patrick Tasse [Fri, 8 Jan 2016 20:34:19 +0000 (15:34 -0500)]
ss: Bug 485463: Incorrect parent sequence number in HTNode
When creating a new 'latest branch' in HistoryTree.addNewRootNode(), the
new nodes' parent sequence number is incorrectly set to their parent's
parent sequence number instead of their parent's sequence number.
The toString() implementation of HTNode used for debugging purposes is
augmented to show the sequence number of parent and children nodes. This
helps in debugging problems such as this one.
Change-Id: Ie6ec689bb28c72eda612d4279b6d27ead7ecc42c
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/63898
Reviewed-by: Hudson CI
Bernd Hufmann [Fri, 8 Jan 2016 15:55:02 +0000 (10:55 -0500)]
control: Update MI parsing for configured event filters
The MI for filter element contains true if a filter has been configured
and false if not.
Change-Id: I146c7b1da4ecdd8aa3df66f443bd7fe9d9772813
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63870
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Fri, 8 Jan 2016 15:48:35 +0000 (10:48 -0500)]
control: Improve check for empty strings
Change-Id: I1d1c6d6e6bf33a3d2eb424683f702d170cde5746
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63861
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Mon, 11 Jan 2016 16:03:43 +0000 (11:03 -0500)]
analysis: Move timing analysis ui classes in own java packages
With this patch, the timing analysis base classes for table, scatter
density and statistics are in their own java packages.
Change-Id: I2a2892c4aec8f649dbec8fa159c0b2344fc5803e
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64010
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Bernd Hufmann [Mon, 11 Jan 2016 17:50:59 +0000 (12:50 -0500)]
control: remove calibrate command
This feature never really was fully implemented in LTTng runtime and
because of that should not be there in the Control view.
Change-Id: I7573bcacf0fcf05f86d688d9ef7b4aafec735a9a
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64029
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Bernd Hufmann [Thu, 7 Jan 2016 16:23:19 +0000 (11:23 -0500)]
control: Remove remaining AllTests.java from test plug-ins
Change-Id: I119996728bd32a6b9d689457d1d60952939107ac
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63756
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Patrick Tasse [Wed, 6 Jan 2016 18:01:47 +0000 (13:01 -0500)]
tmf: Fix IllegalArgumentException in TimeGraphCombo
Change-Id: I4e2dcba654d1da9b0fa41c7686b0a0095506f28c
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/63663
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Fri, 8 Jan 2016 20:24:52 +0000 (15:24 -0500)]
StateSystem: Aesthetic changes to HT_IO.java
Make CACHE_SIZE have a pre-cached MASK.
Add comments.
Rename variables.
Remove "this" references.
Change-Id: Ib3b7c6421b614008f61312b6e22073d2c5c51d94
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63896
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Bernd Hufmann [Fri, 8 Jan 2016 19:23:39 +0000 (14:23 -0500)]
Bug 483621: Move segment store statistics into timing analysis plug-ins
Also, o.e.t.analysis.timing.core.tests is introduced.
Keep packages internal for now since APIs not finalized.
Change-Id: I5a5d4abfd409ff8498d304f97f3c123e0b7a421b
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63891
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Patrick Tasse [Thu, 17 Dec 2015 23:18:19 +0000 (18:18 -0500)]
tmf: Make font depend on item height in time graph
The font used for drawing item names and state labels is now dependent
on each item's height.
The time graph combo no longer has to adjust the time graph control's
font when zooming vertically.
A map of fonts per height is used.
The item state rectangles are adjusted to use less margin height when
the item height is decreased.
Change-Id: I1838855d864e76646bc8e387682ca0cb3c28c418
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/62980
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Bernd Hufmann [Thu, 7 Jan 2016 17:53:12 +0000 (12:53 -0500)]
doc: Correct lttng session load description for conflicting profiles
Change-Id: Ic99b0e9503d00291072c5cac29feedd7fa3a1e13
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63760
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Alexandre Montplaisir [Tue, 8 Dec 2015 19:37:05 +0000 (14:37 -0500)]
Use pom-less plugins where we can
Tycho 0.24 introduced pom-less builds [1]. With this, we can
remove the pom.xml in any plugin, test or feature plugin that
uses the default configuration.
This bumps the minimum required version of Maven to 3.3.
Some plugins have extra specific configuration in their pom,
like ctf or pcap traces that call maven-antrun-plugin to
download test traces. These will still need their usual pom.xml
file.
For test plugins, the top-level pom defines the configuration
used for *core* tests. UI and SWTBot tests use slightly
different parameters. Instead of repeating this information in
every single plugin, we can define profiles that are only
activated in those plugins.
Using empty files to selectively enable these profiles looks a
bit clunky, but it is the simplest solution I have found.
Suggestions welcome!
[1] https://wiki.eclipse.org/Tycho/Release_Notes/0.24#POM-less_Tycho_builds
Change-Id: Ibd379905ca52cd1535b8f5a92e58e1f1fb311a19
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/63050
Reviewed-by: Hudson CI
Bernd Hufmann [Wed, 6 Jan 2016 15:20:23 +0000 (10:20 -0500)]
doc: update user guide for kernel event filtering in Control view
Change-Id: I05da02b91b5b9bf1d73a4ae8c038f559d1744c87
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63648
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Tue, 5 Jan 2016 21:14:03 +0000 (16:14 -0500)]
control: add SWTBot test to test kernel filtering for events
Change-Id: I5de303ba182942b42cccb94db3995f0877cee5bd
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63592
Reviewed-by: Hudson CI
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Bernd Hufmann [Thu, 10 Dec 2015 21:06:59 +0000 (16:06 -0500)]
control: Support for filtering of kernel events
This feature is supported since LTTng Tools v2.7.0.
Change-Id: I1a17d6cc3807050bc15c8dd0eb6a763bbe4fb699
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63591
Reviewed-by: Hudson CI
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Matthew Khouzam [Wed, 6 Jan 2016 21:28:00 +0000 (16:28 -0500)]
analysis.test: add basic swtbot test for scatter charts
Change-Id: I35c19e240c66628a892646ae19ec6706324f2400
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63682
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Matthew Khouzam [Wed, 6 Jan 2016 21:06:38 +0000 (16:06 -0500)]
analysis.test: add basic tests for SWTChart in DensityViewerTest
Change-Id: I0efc9c1ebe7ef32fad7e4a455f75c64ce9e07447
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63680
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Matthew Khouzam [Tue, 5 Jan 2016 23:20:20 +0000 (18:20 -0500)]
analysis.test: add SWTBot test to Density view
Change-Id: I1f0932ebabb40ee15b45ed0b363b7883d069bcc6
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63596
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Geneviève Bastien [Tue, 16 Jun 2015 19:30:21 +0000 (15:30 -0400)]
TMF: Bug 454283: Initialize all analysis modules before executing them
Analysis modules are first initialized, then automatic ones are executed when
the traceOpened signal is received. This makes sure any dependent module
exists for a trace before the execution takes place.
Change-Id: Ica4d77ca5805b37ec73a42af2b9bc5f15c9e21cd
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/37698
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Marc-Andre Laperle [Thu, 3 Dec 2015 04:28:06 +0000 (23:28 -0500)]
Remove API tooling from test plugins
We do not track the API of test plugins so it is not necessary to have
them configured for API tooling. This could lead to errors in the case
where the API baseline contained some test plugins. In Oomph, for
example, the API baseline is just a zip of the complete update site, so
it does contain the test plugins and this triggers errors.
Change-Id: I6bd735c2890ae5eaf329316d4f1bae43348c2281
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/61839
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Bernd Hufmann [Wed, 6 Jan 2016 18:37:51 +0000 (13:37 -0500)]
doc: Add user guide for LTTng session profiles
Change-Id: I42f7de09aa21d66e9e137ee89f15d275f7d3b47f
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63665
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Matthew Khouzam [Tue, 5 Jan 2016 21:49:03 +0000 (16:49 -0500)]
analsysis: Remove lingering alltest.java
Change-Id: I036ecf7c79a477cd75a149123af5d7e0a6fd1dbb
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63593
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Hudson CI
Matthew Khouzam [Wed, 6 Jan 2016 15:05:48 +0000 (10:05 -0500)]
analysis: fix Loose Coupling and int division to double conversion
AbstractSegmentStoreDensityViewer is fixed and made more portable.
Change-Id: Ic7ca60aefb308eaa4a00c914cb7dff6b274c4bb7
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63646
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Matthew Khouzam [Wed, 6 Jan 2016 15:35:07 +0000 (10:35 -0500)]
analysis: fix modifier order for AbstractSegmemtStoreDensityView
Change-Id: Ibd1dc5a4a55f1063c5fda0624597cab06be49ed4
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63651
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Jean-Christian Kouame [Tue, 6 Oct 2015 14:09:47 +0000 (10:09 -0400)]
tmf : add of a range selection from other sources for the tmfEventsTable
Only the events within the range will be selected. If there is no event
within the range of the selection, we select the next event outside of
the range. For a single selection, the event at or the next event after
the requested timestamp is selected.
Change-Id: I1294da7e5d653477502ca45f9430a5e5c8583187
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57526
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Matthew Khouzam [Mon, 4 Jan 2016 19:01:00 +0000 (14:01 -0500)]
analysis: simplify perSyscallStats iteration
Change-Id: Ibac78c3912ac647ec90add64f2a2f16ce90539ef
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63496
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Marc-Andre Laperle [Wed, 28 Oct 2015 22:45:42 +0000 (18:45 -0400)]
Latency: introduce latency density view
This patch introduces a view to view segment densities. The
view is based on SWTChart's bar graph. Its x axis is NOT
time aligned as it is not on the main timeline. It takes
the liberty to display time in a more "human readable"
form showing a scale and up to 3 decimals max. (1.234 ms)
There is a second part to this view: a selection table.
This table shows the selected segments from a drag operation.
Follow up tasks for this can be to introduce other filters and
improve the tooltips.
An example implementation is provided for the SystemCall analysis.
Change-Id: I238a61dc8eeeefdca0da107424686a64fb63d8c3
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/57573
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Marc-Andre Laperle [Tue, 24 Nov 2015 22:14:51 +0000 (17:14 -0500)]
Some cleanups related to investigating redundant null checks
Change-Id: I6c71c90c2d61fe4c4c3d30c8b1fb1cb0c9eac2f2
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/61197
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Patrick Tasse [Mon, 4 Jan 2016 23:11:35 +0000 (18:11 -0500)]
tmf: Avoid recursion in AbstractSegmentStoreTableViewer pack listener
In the SWT.SetData listener TableColumn.pack() is called. This requires
all visible items from being filled in order to pack the column. For
each visible item a SWT.SetData event is fired which triggers another
TableColumn.pack().
In Windows, after 12 such iterations in the stack, the table somehow
behaves badly and the OS incorrectly returns an item count of zero,
which causes ArrayIndexOutOfBoundsException and other problems.
Since the intention is to pack the columns only once, the SWT.SetData
listener is removed before the pack instead of after the pack, thus
avoiding the recursive calls.
Change-Id: Id44c54a474ed6e04aa2d79c051d863011f55a8aa
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/63513
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Matthew Khouzam [Mon, 4 Jan 2016 18:57:24 +0000 (13:57 -0500)]
analysis: fix @NonNull issues with SystemCallLatencyStatisticsViewer
Change-Id: I2d14203943774a9447f618a76d300e509a2e212d
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63495
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Marc-Andre Laperle [Mon, 21 Dec 2015 04:07:49 +0000 (23:07 -0500)]
tmf: Fix remote.ui test on Windows due to line separators
This can happen when core.autocrlf is set to false in the git config.
The file containing an expected string is then not converted to crlf and
since the generated string in the test contains crlf, the comparison
fails.
To make tests consistent everywhere, I suggest to disable the autocrlf
of git directly in the Git repository using .gitattributes, see
http://git-scm.com/docs/gitattributes
And to fix the test on Windows, we can always convert the generated
String to lf line endings.
Change-Id: Ife8912ace7895ebca4d19adb53c6ba169ca2d003
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63080
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Marc-Andre Laperle [Mon, 21 Dec 2015 06:52:51 +0000 (01:52 -0500)]
tmf: Fix failure in StandardImportGzipTraceTest related to test ordering
If TmfAlignTimeAxisTest ran before, and the window size was small
enough, it was possible for the test to be running in an "Align 1"
perspective with insufficient height for the events table. Because of
that, the expected content in the table could not be found. The tests in
TmfAlignTimeAxisTest switch to generated perspectives without returning
to a "normal" perspective. It makes sense that the tests "cleanup" by
returning to a more normal perspective like the Tracing perspective.
Also switch to Tracing perspective explicitly in
StandardImportGzipTraceTest which is more consistent with the other
tests and is somewhat less error prone.
Change-Id: I0d28af39266bdcdc81e765b5522c92a4a8085602
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63081
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Alexandre Montplaisir [Wed, 9 Dec 2015 21:40:59 +0000 (16:40 -0500)]
Clean up top-level pom.xml
Re-arrange entries, fix indentation and add spacing to make the
whole file more readable.
Also specify <project.reporting.outputEncoding> (removes a
warning in the build output).
Change-Id: Ia508f2b24de0813fbaf8d9db1e79e755e351ac5f
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/62348
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 Dec 2015 21:44:28 +0000 (16:44 -0500)]
rcp: Update RCP product plugin's ID
It was simply "tracing", typically our artifactIds include the
whole symbolic name of the plugin.
Change-Id: I46a0faf45f18c64bf8005ad90cf2744e8bcb53d9
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/63059
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 [Fri, 18 Dec 2015 21:24:14 +0000 (16:24 -0500)]
ctf: make computeAttributeMap return a Map instead of ImmutableMap
Change-Id: Iba609c83c20f11b26bd7b4db7deb2242ad41cdf6
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/63058
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-by: Hudson CI
Matthew Khouzam [Thu, 17 Dec 2015 17:24:19 +0000 (12:24 -0500)]
ctf: fix modifier order of @Nullable annotation
Change-Id: Ib5f40b243b6ecae25fda50dd09ef7c8df16e36f7
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/62951
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Marc-Andre Laperle [Thu, 17 Dec 2015 23:12:14 +0000 (18:12 -0500)]
Fix main shell not active using Xvfb
There seems to be problems on some system where the main shell is not in
focus initially. This was seen using Xvfb and Xephyr on some occasions.
Change-Id: Ib6dda93e856deb96cf865dce6253b48c99171184
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/62979
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Matthew Khouzam [Thu, 16 Jul 2015 01:31:57 +0000 (21:31 -0400)]
tmf.ctf: Accelerate ByteArrayDefintion parsing
Make BADs no longer need to generate definitions uselessly.
This patch yields a 15-50% acceleration in Synchronization benchmarks,
a 5% acceleration in the Kernel Analysis benchmark, and an average event
size reduction of 33%.
Change-Id: Ie581abaecf1d8b188de9f1acc3010fe3f7a65d7a
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/52024
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Matthew Khouzam [Mon, 13 Jul 2015 14:39:19 +0000 (10:39 -0400)]
ctf: Collapse if statement in StructDefinition
Change-Id: I2736507e0ebbee70a172930cd4315c89872eb001
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/51857
Reviewed-by: Hudson CI
Reviewed-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Alexandre Montplaisir [Tue, 24 Nov 2015 23:50:53 +0000 (18:50 -0500)]
Remove all "AllTests" suites
tycho-surefire-plugin's default configuration is to look for @Tests
in classes that start or end with "Test", or end with "TestCase".
We can make use of this and avoid all the nested AllTests.java suites
found in every single test package in the tree. This also avoids
the need for most custom configuration in each test plugin's pom.xml.
To run a complete test plugin from within Eclipse, one can still
right-click on the plugin itself, and choose
Run As -> Junit Plug-in Test.
(Doing this before this patch would actually run all tests twice!)
Note: this breaks the suites in the "alltests" plugin, but those could
be brought back by using something like ClasspathSuite, if desired.
Change-Id: Ibf960df37c5d3c98dc0e0dd722d7c9fe5c0b9d4f
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/59442
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 [Thu, 17 Dec 2015 17:25:06 +0000 (12:25 -0500)]
tmf: Fix GTK bug when setting top item after changing tree font
Change-Id: If2658c1bdcd98f6a34ce31a76af0dc57c959de2c
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/62953
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, 17 Dec 2015 19:12:44 +0000 (14:12 -0500)]
analysis: Make SystemCallLatencyTableAnalysisTest more stable
Even if the jobs are finished, the table is not necessarily filled yet.
The update could be pending as a event in the UI thread queue, etc. We
can wait until cells are filled instead.
Also export the package so that it can be used elsewhere, (alltests
stress test for example).
Change-Id: I8177c7e6b7e487d59b248e0279c6bcd8da43b515
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/62959
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Patrick Tasse [Fri, 11 Dec 2015 20:57:01 +0000 (15:57 -0500)]
tmf: Support vertical zoom in time graph with Shift+Ctrl+Mouse Wheel
Change-Id: Ie5486855cd29c95ee3b0d7caad0ceac6c184d725
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/62534
Reviewed-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Reviewed-by: Hudson CI
Patrick Tasse [Wed, 16 Dec 2015 21:56:56 +0000 (16:56 -0500)]
tmf: Bug 484422: Fix inconsistent vertical zoom
In GTK3, after changing the tree font, the tree item bounds are not
updated until the tree has been painted. The alignment of tree items
should be done after a paint event has occurred.
Change-Id: I47617b881d3f0f981bdacf3ddda572c296dffaf7
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/62876
Reviewed-by: Hudson CI
Reviewed-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Patrick Tasse [Fri, 11 Dec 2015 20:16:15 +0000 (15:16 -0500)]
doc: Update user guide for time graph markers and vertical zoom
Change-Id: Id4e16737c46492efebb643b75455855ee619e3e1
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Reviewed-on: https://git.eclipse.org/r/62531
Reviewed-by: Hudson CI
Matthew Khouzam [Wed, 2 Dec 2015 21:04:59 +0000 (16:04 -0500)]
ctf: make CtfTmfEvent expose packet context attributes
This patch allows acces to the packet context of an event from that
event.
Change-Id: I8409c8fbcb7766a2d7be4104159a7d7de6aa188d
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/61817
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Matthew Khouzam [Wed, 16 Dec 2015 03:16:11 +0000 (22:16 -0500)]
ctf: expose stream in CtfTmfEvent
This allows one to access the stream number. It may lead to improved UX later.
The stream ID can allow analyses to know which stream an event is coming
from. If a trace has a control channel and a data channel, this will
allow the events to be easily filtered into the appropriate views.
Change-Id: Iba9de10d9f1f633deed6df81c0d849acf5a3aceb
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/62779
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Matthew Khouzam [Wed, 2 Dec 2015 21:01:14 +0000 (16:01 -0500)]
ctf: Make packet descriptor information available to event
This allows the event to know what the packet context is for
its parent packet. This is especially useful if one puts non-
LTTng information in the packet such as the source gpu.
Change-Id: I21ff283c1032e5a3157408b304b2611d1113d4f7
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/61816
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Matthew Khouzam [Wed, 2 Dec 2015 21:07:40 +0000 (16:07 -0500)]
ctf: Expose map of attributes in Packet Descriptor
The motivation is to allow accessing custom fields in CTF
packet contexts. It decouples the CTF parser from LTTng a bit.
This breaks an unused api.
Change-Id: I2bead35a7e080d4e3ab325395629c44eea455cfc
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/61815
Reviewed-by: Hudson CI
Reviewed-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Matthew Khouzam [Wed, 2 Dec 2015 20:41:59 +0000 (15:41 -0500)]
ctf: make SIPIE enums store both key and value
By adding the getStringValue and getIntegerValue,
instead of getValue, we have more information available.
This is useful for enums so we can look up the string
name but also sort it easier.
Change-Id: Icfa095a0a8c2b8924809c389dbfe53055c3b63be
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/61814
Reviewed-by: Hudson CI
Reviewed-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Matthew Khouzam [Tue, 15 Dec 2015 20:12:00 +0000 (15:12 -0500)]
ctf: Introduce ICTFStream
This allows a CTFStream to be shared while not being modifyiable.
Change-Id: I4bf4985abd5221ebfa543d0a7ff0cfeda0874fce
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/62767
Reviewed-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Reviewed-by: Hudson CI
Marc-Andre Laperle [Wed, 16 Dec 2015 03:35:31 +0000 (22:35 -0500)]
Disable overlay scrollbars when running tests to prevent crash
When running tests, the Eclipse launcher is not used which means
that the code that usually disables the overlay scrollbars is not
executed. The overlay scrollbars under GTK2 are unstable and can
cause crash. Specifically, a crash was frequently seen in
tmf.ui.swtbot.tests on Ubuntu.
Change-Id: I130d7fd95affa4f7eb68d2c2d1e0b0134e84ee07
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/62780
Reviewed-by: Hudson CI
Reviewed-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Matthew Khouzam [Mon, 13 Jul 2015 14:30:37 +0000 (10:30 -0400)]
ctf: make HashMap child a Map and extract lookupStructScope
The map is self explanatory. The method extraction improves readibility
in my opinion
Change-Id: I9fb23c3223641d80e2f70a18142c3d4b968f0dab
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/51854
Reviewed-by: Hudson CI
Reviewed-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Marc-Andre Laperle [Tue, 15 Dec 2015 06:11:43 +0000 (01:11 -0500)]
tmf: Remove an annotation temporarily to work around Sonarqube bug
There is a Sonarqube bug with type annotations and arrays, see
https://groups.google.com/forum/#!topic/sonarqube/VTsAYxhGRiA
and
https://github.com/SonarSource/sonar-java/pull/537
By removing the @NonNull, it brings back a warning but makes the
Sonarqube build work again.
Change-Id: I6a03295432c7c452b4dca9018d4c17ba4b991980
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/62684
Reviewed-by: Hudson CI
Reviewed-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Patrick Tasse [Mon, 14 Dec 2015 15:55:14 +0000 (10:55 -0500)]
tmf: Keep usage of ViewerSorter in navigatorContent extension point
Reverting the change to TmfViewerSorter from commit
e707a55.
Platform UI doesn't support ViewerComparator in the
org.eclipse.ui.navigator.navigatorContent extension point in the
navigatorContent/commonSorter element (refer to Bug 484248).
This causes a ClassCastException and prevents the TMF project model
elements from being sorted properly.
Change-Id: I3d63032763dc435b594c3f9e8df04d0cb6b38609
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/62640
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 [Mon, 14 Dec 2015 03:34:10 +0000 (22:34 -0500)]
releng: Fix alltests failure on Mac
The alltests plugin was failing on Mac because it tried to run 0 tests
on the UI thread but it needs to start on the first thread
(-XstartOnFirstThread). We can just set it to not un on the UI thread.
There is no need for the UI thread when running 0 tests.
Change-Id: I1e260f96e1ec51e256e8280262bcbd3c83a92de3
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/62574
Reviewed-by: Hudson CI
Marc-Andre Laperle [Mon, 14 Dec 2015 03:03:48 +0000 (22:03 -0500)]
tmf: FilterColorEditorTest on Mac
On Mac, RGB values that are captured with ImageHelper are affected by
monitor color profiles. To account for this, we can draw the expected
color in a simple shell and use that color as expected value instead.
Also, switching to the Filter mode in the table is not instantaneous,
we need to wait until it's done before capturing the next image.
Change-Id: I930570027ba53f3dbdcecb5925a8e67c24c239b2
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/62573
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Jean-Christian Kouame [Thu, 10 Dec 2015 20:34:16 +0000 (15:34 -0500)]
tmf : Fix the xml condition multiple problem
This patch initiates the number of state values involved in a comparison
only when the xml element is <condition>. Without this fix, conditions
like OR or AND with more than two operands fail because we were
validating that the maximum number of state values was actually two.
Since the XSD validates that already the condition is removed in this
patch and the allocation of the state value list is done only when the
element is <condition>. In other cases, the list is instantiated to an
empty list.
This patch also add a test with a condition with three operands in
TmfXmlConditionTest to validate this case.
Change-Id: I6e99f770b35f3359a5617bc01b8a53bc77df637e
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/62425
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Marc-Andre Laperle [Fri, 11 Dec 2015 19:53:23 +0000 (14:53 -0500)]
tmf: Fix existing trace not being reopened when it as a short path (Win)
This fixes the TestTraceOffsetting test.
On Windows, the temporary path can look like this:
java.io.tmpdir=C:\Users\MARC-A~1\AppData\Local\Temp\ Which is a "short
path". When used in comparison with full, canonical paths, it fails.
In the context of this bug, because the comparison failed, it ended up
creating a second trace for the same location then the test could not
find the proper editor because the title contained a '(2)'.
Change-Id: I41def2180ff910e25489415c9de513212d2f2ea2
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/62529
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Marc-Andre Laperle [Fri, 11 Dec 2015 18:22:34 +0000 (13:22 -0500)]
ctf: Fix InvalidPathException in CtfTmfTraceValidateTest on Windows
The path beeing passed to Paths.get was not a format supported (/C:/)
Change-Id: I80eb54ae3198381d91ab19c55c991004721d06dc
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/62521
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Marc-Andre Laperle [Fri, 11 Dec 2015 17:37:09 +0000 (12:37 -0500)]
ctf: Fixed leaked file handle when calling CTFTraceReader.update
If an instance of CTFStreamInputReader already existed in a
CTFTraceReader, it was not getting closed after checking whether or not
it existed. This was causing a test failure on Windows in
CTFTraceGrowingTest.
Change-Id: Ia064d3270882515b32834456771a108970ce8f8e
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/62515
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Marc-Andre Laperle [Thu, 19 Nov 2015 23:44:35 +0000 (18:44 -0500)]
tmf: Bug 479675: Fix for index not being properly deleted on Windows
Change-Id: Idc97913cf34658452fef95f6e3ba502a57fc13ab
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/60858
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Patrick Tasse [Thu, 10 Dec 2015 22:17:08 +0000 (17:17 -0500)]
doc: Update developer guide for time graph markers
Change-Id: I9d8b97e397b94359b8bf2f881f253bf8b19bd1a6
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/62434
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, 8 Dec 2015 14:48:36 +0000 (09:48 -0500)]
tmf: Add vertical zooming support to time graph viewer and combo
Vertical zoom is triggered with Ctrl+"+" and Ctrl+"-". Reset is
triggered by Ctrl+"0".
Ctrl+"=" is equivalent to Ctrl+"+" for vertical zoom-in, and "=" is now
equivalent to "+" for horizontal zoom-in also.
The font of the time graph control is changed along with the zooming,
and the views with post-draw handling to draw labels on time graph items
are modified to use a new method which centers the text vertically.
Change-Id: Ic75c1ebe7e937c612b64346e2814894c500f964f
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/62359
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Alexandre Montplaisir [Tue, 8 Dec 2015 19:24:16 +0000 (14:24 -0500)]
Replace usage of ViewerSorter with ViewerComparator
The former has been deprecated in 4.6, it's now recommended to use
ViewerComparator and .setComparator() instead.
Change-Id: Ib7b447a74756b2b4524924f76b93ec0d64f71ed1
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/62262
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 Dec 2015 17:48:31 +0000 (12:48 -0500)]
lttng: Add SWTBot tests for Resources view and time graph markers
Change-Id: Ida3d7c7bed355f8032797d8074db3a9d8551eed2
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/62138
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
This page took 0.046943 seconds and 5 git commands to generate.