Re: [lttng-dev] [PATCH 2/2] urcu: new wfqueue implementation

2012-08-10 Thread Lai Jiangshan
On Sat, Aug 11, 2012 at 2:28 AM, Mathieu Desnoyers wrote: > * Lai Jiangshan (la...@cn.fujitsu.com) wrote: >> Some guys would be surprised by this fact: >> There are already TWO implementations of wfqueue in urcu. >> >> The first one is in urcu/static/wfqueue.h: >> 1) enqueue: exchange the tail and

Re: [lttng-dev] [Babeltrace PATCH] Add BT_SEEK_LAST type to bt_iter_pos.

2012-08-10 Thread Mathieu Desnoyers
* Francis Deslauriers (francis.deslauri...@polymtl.ca) wrote: > Signed-off-by: Francis Deslauriers > --- > include/babeltrace/iterator.h |1 + > lib/iterator.c| 189 > + > 2 files changed, 190 insertions(+) > > diff --git a/include/ba

Re: [lttng-dev] [BABELTRACE PATCH] Fix: safety checks for opening mmap traces

2012-08-10 Thread Mathieu Desnoyers
* Julien Desfossez (jdesfos...@efficios.com) wrote: > When adding a mmap trace in the context, we don't have any tracefile and > index, some safety checks were missing in the path handling and > timestamp manipulation. > > Signed-off-by: Julien Desfossez merged, thanks! Mathieu > --- > format

Re: [lttng-dev] [babeltrace PATCH] Remove trace-collection.h from include_headers

2012-08-10 Thread Mathieu Desnoyers
merged, thanks! Mathieu * Danny Serres (danny.ser...@efficios.com) wrote: > Signed-off-by: Danny Serres > --- > include/Makefile.am |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/Makefile.am b/include/Makefile.am > index f7488da..79247a0 100644 > --- a/incl

[lttng-dev] [BABELTRACE PATCH] Fix: safety checks for opening mmap traces

2012-08-10 Thread Julien Desfossez
When adding a mmap trace in the context, we don't have any tracefile and index, some safety checks were missing in the path handling and timestamp manipulation. Signed-off-by: Julien Desfossez --- formats/ctf/ctf.c |9 + lib/context.c |6 -- 2 files changed, 13 insertions

Re: [lttng-dev] [LTTNG-TOOLS PATCH] Fix: protect visibility of filter-parser functions

2012-08-10 Thread David Goulet
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Merged! On 10/08/12 05:19 PM, Julien Desfossez wrote: > From: Mathieu Desnoyers > > Some functions share the same name between ctf-parser/lexer and > filter-parser/lexer. > > This patch protects the visibility of the functions with the same name t

[lttng-dev] [LTTNG-TOOLS PATCH] Fix: protect visibility of filter-parser functions

2012-08-10 Thread Julien Desfossez
From: Mathieu Desnoyers Some functions share the same name between ctf-parser/lexer and filter-parser/lexer. This patch protects the visibility of the functions with the same name to avoid linkage confusions. Signed-off-by: Mathieu Desnoyers Signed-off-by: Julien Desfossez --- src/lib/lttng-

Re: [lttng-dev] [PATCH lttng-tools 0/2] relayd IPv6 fixes

2012-08-10 Thread David Goulet
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Merged! Thanks Christian Babeux: > Hi all, > > Here are a few fixes to IPv6 support in relayd. > > When trying to start the relayd using an IPv6 address, the relayd > simply segfaulted. The first patch fix this issue. > > The second patch fix a

Re: [lttng-dev] [PATCH 2/2] urcu: new wfqueue implementation

2012-08-10 Thread Mathieu Desnoyers
* Lai Jiangshan (la...@cn.fujitsu.com) wrote: > Some guys would be surprised by this fact: > There are already TWO implementations of wfqueue in urcu. > > The first one is in urcu/static/wfqueue.h: > 1) enqueue: exchange the tail and then update previous->next > 2) dequeue: wait for first node's n

Re: [lttng-dev] [PATCH 2/2] urcu: add notice to URCU_TLS() for it is not async-signal-safe

2012-08-10 Thread Paul E. McKenney
On Fri, Aug 10, 2012 at 10:16:15AM -0400, Mathieu Desnoyers wrote: > (for the records: this discussion is about userspace RCU tls-compat.h, > which uses TLS on systems supporting it, and fall back to > pthread_key_create/getspecific/setspecific if not. The culprit of the > issue is that we want to

[lttng-dev] [lttng-ust PATCH] Python module

2012-08-10 Thread Danny Serres
Proof of concept: A simple library to create tracepoints and allow tracing in Python programs. To build, configure with Python: ./configure --with-python To use, import the module in Python: import lttngust Signed-off-by: Danny Serres --- Makefile.am |4 + README

[lttng-dev] [babeltrace PATCH] API documentation file v2

2012-08-10 Thread Danny Serres
Signed-off-by: Danny Serres --- doc/API.txt | 257 +++ 1 file changed, 257 insertions(+) create mode 100644 doc/API.txt diff --git a/doc/API.txt b/doc/API.txt new file mode 100644 index 000..4688a75 --- /dev/null +++ b/doc/API.txt @@

[lttng-dev] [PATCH lttng-tools 2/2] Fix: Wrong domain used when initializing IPv6 sockets

2012-08-10 Thread Christian Babeux
Signed-off-by: Christian Babeux --- src/common/sessiond-comm/inet6.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/common/sessiond-comm/inet6.c b/src/common/sessiond-comm/inet6.c index 0d96c31..98aba04 100644 --- a/src/common/sessiond-comm/inet6.c +++ b/src/common/sessi

[lttng-dev] [PATCH lttng-tools 1/2] Fix: Off by one in lttcomm_sock_domain enum

2012-08-10 Thread Christian Babeux
The wrong value is used to lookup the socket creation function in net_families causing a segfault when using IPv6. Signed-off-by: Christian Babeux --- src/common/sessiond-comm/sessiond-comm.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/common/sessiond-co

[lttng-dev] [PATCH lttng-tools 0/2] relayd IPv6 fixes

2012-08-10 Thread Christian Babeux
Hi all, Here are a few fixes to IPv6 support in relayd. When trying to start the relayd using an IPv6 address, the relayd simply segfaulted. The first patch fix this issue. The second patch fix a wrong domain value used in the socket() call for the IPv6 socket causing a "Address family not supp

Re: [lttng-dev] [babeltrace PATCH] API documentation file‏

2012-08-10 Thread Julien Desfossez
Hi Danny, Thanks for the documentation, it is great ! I added some comments below. On 09/08/12 05:39 PM, Danny Serres wrote: > Signed-off-by: Danny Serres > --- > doc/API.txt | 237 > +++ > 1 file changed, 237 insertions(+) > create mo

[lttng-dev] [Babeltrace PATCH] Add BT_SEEK_LAST type to bt_iter_pos.

2012-08-10 Thread Francis Deslauriers
Signed-off-by: Francis Deslauriers --- include/babeltrace/iterator.h |1 + lib/iterator.c| 189 + 2 files changed, 190 insertions(+) diff --git a/include/babeltrace/iterator.h b/include/babeltrace/iterator.h index aa6470e..c13055d 1006

Re: [lttng-dev] [PATCH 2/2] urcu: new wfqueue implementation

2012-08-10 Thread Paul E. McKenney
On Thu, Aug 09, 2012 at 04:46:26PM +0800, Lai Jiangshan wrote: > Some guys would be surprised by this fact: > There are already TWO implementations of wfqueue in urcu. > > The first one is in urcu/static/wfqueue.h: > 1) enqueue: exchange the tail and then update previous->next > 2) dequeue: wait f

Re: [lttng-dev] qemuarm: Session dameon died (exit status 0)

2012-08-10 Thread David Goulet
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 That's interesting! Could you send us the gdb backtrace $ gdb lttng-sessiond gdb> r gdb> bt full And seems you use "-ww" instead of "-vvv" ... I doubt the segfault is in the option parsing but still. Thanks! David Paul Barrette: > > > On Fri,

Re: [lttng-dev] qemuarm: Session dameon died (exit status 0)

2012-08-10 Thread Paul Barrette
On Fri, Aug 10, 2012 at 10:31 AM, Yannick Brosseau < yannick.bross...@gmail.com> wrote: > Hi, > > Can you try to start the lttng-sessiond manually and see if there is error > message (with full logging enabled: lttng-sessiond -vvv) > > Hi, # lttng-sessiond -ww Segmentation fault. Pb > Yannick

Re: [lttng-dev] qemuarm: Session dameon died (exit status 0)

2012-08-10 Thread Yannick Brosseau
Hi, Can you try to start the lttng-sessiond manually and see if there is error message (with full logging enabled: lttng-sessiond -vvv) Yannick On 2012-08-10 10:02, Paul Barrette wrote: > Hi all, > lttng-modules 2.0.4 > lttng-tools 2.0.4 > linux kernel 3.4.x > > I am running qemuarm and seeing

Re: [lttng-dev] [PATCH 2/2] urcu: add notice to URCU_TLS() for it is not async-signal-safe

2012-08-10 Thread Mathieu Desnoyers
(for the records: this discussion is about userspace RCU tls-compat.h, which uses TLS on systems supporting it, and fall back to pthread_key_create/getspecific/setspecific if not. The culprit of the issue is that we want to allow reading the tls-compat variable from signal handlers, and thus async-

[lttng-dev] qemuarm: Session dameon died (exit status 0)

2012-08-10 Thread Paul Barrette
Hi all, lttng-modules 2.0.4 lttng-tools 2.0.4 linux kernel 3.4.x I am running qemuarm and seeing the following issue when running "lttng create": Error: Session daemon died (exit status 0) Is there a patch I am missing for arm? Thanks Pb -- -- Paul Barrette Linked In Profile