Re: [lttng-dev] [PATCH] Fix linker library order

2013-05-14 Thread Mathieu Desnoyers
* Alexandre Montplaisir (alexmon...@voxpopuli.im) wrote: On 13-05-11 10:59 PM, Mathieu Desnoyers wrote: * Francis Giraldeau (francis.girald...@gmail.com) wrote: Libraries must be specified after the binary target. merged, thanks! Mathieu This seems to have broken the daily PPA

Re: [lttng-dev] User-space tracing by the root lttng-sessiond

2013-05-14 Thread Mathieu Desnoyers
* Thibault, Daniel (daniel.thiba...@drdc-rddc.gc.ca) wrote: I ran a little experiment whose results surprised me. Using a non-tracing group user account, I first made sure I had a root lttng-sessiond running and a user-space lttng-sessiond. In the first I set up a trace for some kernel

Re: [lttng-dev] User-space tracing by the root lttng-sessiond

2013-05-14 Thread Thibault, Daniel
(Question: If there were several users active at once, would there be several root user-space lttng-consumerd daemons or still just one?) no Please clarify: no, there will be several root user-space lttng-consumerd daemons or no, there will be just one root user-space lttng-consumerd

Re: [lttng-dev] [PATCH lttng-tools] Tests: Remove unused utils.h header

2013-05-14 Thread David Goulet
Merged with a small modification that removes the include in one of the unit test. David Christian Babeux: Signed-off-by: Christian Babeux christian.bab...@efficios.com --- tests/utils/Makefile.am | 4 ++-- tests/utils/utils.h | 43 --- 2 files

[lttng-dev] [PATCH 3/3 lttng-tools] Fix: use lttng pipe to send new stream to thread

2013-05-14 Thread David Goulet
Signed-off-by: David Goulet dgou...@efficios.com --- src/common/kernel-consumer/kernel-consumer.c | 15 +++ src/common/ust-consumer/ust-consumer.c | 16 2 files changed, 15 insertions(+), 16 deletions(-) diff --git

Re: [lttng-dev] [PATCH] Fix linker library order

2013-05-14 Thread Francis Giraldeau
Le 2013-05-14 13:38, Simon Marchi a écrit : On 14 May 2013 08:37, Mathieu Desnoyers mathieu.desnoy...@efficios.com wrote: * Mathieu Desnoyers (mathieu.desnoy...@efficios.com) wrote: * Alexandre Montplaisir (alexmon...@voxpopuli.im) wrote: On 13-05-11 10:59 PM, Mathieu Desnoyers wrote: *

[lttng-dev] [PATCH] Remove -pie flag for shared library

2013-05-14 Thread Francis Giraldeau
Some build environment with hardening enabled adds -pie flags to LDFLAGS. While it's required for ELF binaries, it doesn't make sens for shared libraries. For this case, filter-out the erroneous option. Signed-off-by: Francis Giraldeau francis.girald...@gmail.com --- doc/examples/demo/Makefile

Re: [lttng-dev] [PATCH] Fix linker library order

2013-05-14 Thread Jérémie Galarneau
Francis has already submitted a patch but we could also add a --disable-examples switch to configure if hand-crafted Makefiles become a problem. On Tue, May 14, 2013 at 4:51 PM, Alexandre Montplaisir alexmon...@voxpopuli.im wrote: On 13-05-14 01:38 PM, Simon Marchi wrote: [...] If you read

Re: [lttng-dev] [PATCH] Remove -pie flag for shared library

2013-05-14 Thread Simon Marchi
Thank you Francis. I tested your patch locally and it works, so I included it in the packaging on Launchpad so that the package builds for now. However, it would be preferable if the problem could be addressed more gracefully upstream. Simon On 14 May 2013 16:15, Francis Giraldeau

[lttng-dev] [PATCH lttng-ust] Add warning about default prefix and library paths to README

2013-05-14 Thread Jérémie Galarneau
Fixes #467 Signed-off-by: Jérémie Galarneau jeremie.galarn...@efficios.com --- README | 5 + 1 file changed, 5 insertions(+) diff --git a/README b/README index 4b3053c..2a335ce 100644 --- a/README +++ b/README @@ -52,6 +52,11 @@ INSTALLATION INSTRUCTIONS: If compiling from the git

Re: [lttng-dev] [PATCH] Fix linker library order

2013-05-14 Thread Mathieu Desnoyers
* Alexandre Montplaisir (alexmon...@voxpopuli.im) wrote: On 13-05-14 01:38 PM, Simon Marchi wrote: [...] If you read carefully the log sent by Alexandre, you see that it is when building the shared libs in this directory (lttng-ust-provider-ust-tests-demo.so) that the build fails. I

Re: [lttng-dev] [PATCH] Fix linker library order

2013-05-14 Thread Mathieu Desnoyers
* Mathieu Desnoyers (mathieu.desnoy...@efficios.com) wrote: * Alexandre Montplaisir (alexmon...@voxpopuli.im) wrote: On 13-05-14 01:38 PM, Simon Marchi wrote: [...] If you read carefully the log sent by Alexandre, you see that it is when building the shared libs in this directory

[lttng-dev] [PATCH lttng-tools 1/3] Tests: Move the babelstats utility to tests/utils

2013-05-14 Thread Christian Babeux
This utility will prove useful in trace validation for other tests, so put it in a common location accessible by the tests. Signed-off-by: Christian Babeux christian.bab...@efficios.com --- tests/regression/tools/filtering/Makefile.am | 4 +-

[lttng-dev] [PATCH lttng-tools 2/3] Tests: Add the tracefile count test

2013-05-14 Thread Christian Babeux
This test validate the newly introduced tracefile count feature. In order to so, we enable a channel with the appropriate limits in overwrite mode. The later constraint is to ensure that at least the last few events are properly recorded and that we can validate that they are present in the

[lttng-dev] [PATCH lttng-tools 3/3] Tests: Add the tracefile size test

2013-05-14 Thread Christian Babeux
This test validate the newly introduced tracefile count feature. In order to so, we enable a channel with the appropriate file size limits. After running an instrumented application, we validate that each file in the trace folder is no larger than the limit set on the command line and also that

[lttng-dev] [PATCH babeltrace] Fix: Suppress a compiler warning (always-false condition)

2013-05-14 Thread Jérémie Galarneau
Clang reports an always-false condition (checking if an unsigned value 0) caused by a type mismatch in ctf-visitor-generate-io-struct.c. Remove trailing whitespaces. Signed-off-by: Jérémie Galarneau jeremie.galarn...@efficios.com --- formats/ctf/metadata/ctf-visitor-generate-io-struct.c | 13

[lttng-dev] [PATCH babeltrace] Remove unused array in bt_context_add_traces_recursive

2013-05-14 Thread Jérémie Galarneau
Signed-off-by: Jérémie Galarneau jeremie.galarn...@efficios.com --- converter/babeltrace.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/converter/babeltrace.c b/converter/babeltrace.c index 2fd371c..1008351 100644 --- a/converter/babeltrace.c +++