Re: [lttng-dev] [PATCH 2/2] liblttng-ust-comm/lttng-ust-com.c: remove unnecessary goto in ustcomm_accept_unix_sock()

2012-06-25 Thread Mathieu Desnoyers
gt; return new_fd; > - > -error: > - return -1; > } > > /* > -- > 1.7.9.5 > > > ___ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Desnoyers Operating

Re: [lttng-dev] [UST PATCH] Add a multi-thread version of the 'hello' test program

2012-06-25 Thread Mathieu Desnoyers
ment when running the program, this will > +indicate how many threads to use. For example > + > +./hello 10 > + > +will run the test with 10 threads. If no argument is passed, the default of > +1 thread is used, in which case the behaviour should be similar to the > original

Re: [lttng-dev] Some questions about Lttng

2012-06-25 Thread Mathieu Desnoyers
g-ust No ETA at the moment for this feature. It's a "wishlist" feature, but it is not planned for any specific point in time. Thanks, Mathieu -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com ___

Re: [lttng-dev] [PATCH] Expose kernel tracer to user-space

2012-06-25 Thread Mathieu Desnoyers
/probes/Makefile > +++ b/probes/Makefile > @@ -14,6 +14,8 @@ obj-m += lttng-probe-sched.o > obj-m += lttng-probe-irq.o > obj-m += lttng-probe-signal.o > obj-m += lttng-probe-timer.o > +obj-m += lttng-probe-uevent.o > +obj-m += lttng-uevent.o > > obj-m += lttng-probe-

Re: [lttng-dev] [UST PATCH] Add a multi-thread version of the 'hello' test program

2012-06-26 Thread Mathieu Desnoyers
* Alexandre Montplaisir (alexmon...@voxpopuli.im) wrote: > On 12-06-26 02:03 AM, Mathieu Desnoyers wrote: > > * Alexandre Montplaisir (alexmon...@voxpopuli.im) wrote: > >> [...] > >> diff --git a/tests/hello-mt/README b/tests/hello-mt/README > >> new file mod

Re: [lttng-dev] [UST PATCH] Add a multi-thread version of the 'hello' test program

2012-06-26 Thread Mathieu Desnoyers
* Brosseau, Yannick (yannick.bross...@gmail.com) wrote: > On Tue, Jun 26, 2012 at 8:02 AM, Mathieu Desnoyers > wrote: > > * Alexandre Montplaisir (alexmon...@voxpopuli.im) wrote: > >> On 12-06-26 02:03 AM, Mathieu Desnoyers wrote: > >> > * Alexandre Montplaisi

Re: [lttng-dev] Version of LTTng for kernel 2.6.25?

2012-06-26 Thread Mathieu Desnoyers
ld, MA 01880-2400 > > ___ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.e

Re: [lttng-dev] [PATCH] Expose kernel tracer to user-space

2012-06-26 Thread Mathieu Desnoyers
* Francis Giraldeau (francis.girald...@gmail.com) wrote: > Le 2012-06-26 08:40, Mathieu Desnoyers a écrit : > >> +/* > >> + * lttng_uevent_set_handler - set handler functions for uevent > >> + * > >> + * Access to handler code is protected with rwlock in

[lttng-dev] [RELEASE] LTTng modules 2.0.4

2012-06-26 Thread Mathieu Desnoyers
link: http://lttng.org/download (please refer to the README files for installation instructions and lttng-tools doc/quickstart.txt for usage information) -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios

Re: [lttng-dev] [RELEASE] Userspace RCU 0.7.3

2012-06-27 Thread Mathieu Desnoyers
ve this commit already: commit e51500edbd9919cee53bc85cbb4b22cd4786fc42 Author: Mathieu Desnoyers Date: Tue Jun 12 11:24:31 2012 -0400 Fix c99 compatibility: use __asm__ and __volatile__ in public headers Signed-off-by: Mathieu Desnoyers Does it fix it for you ? Thanks, Mathieu &g

Re: [lttng-dev] [RELEASE] Userspace RCU 0.7.3

2012-06-27 Thread Mathieu Desnoyers
ian''s 6.7.2 rather than the newer version. > > Gerhard > > > On Wed, 27 Jun 2012, Mathieu Desnoyers wrote: > > > Date: Wed, 27 Jun 2012 09:35:24 -0400 > > From: Mathieu Desnoyers > > To: Gerhard Mack > > Cc: Alexandre Montplais

Re: [lttng-dev] [PATCH] tracepoint event exclusion

2012-06-27 Thread Mathieu Desnoyers
struct ltt_event *event; > int ret = 0; > > + if (strlen(event_param->name) > 0 && event_param->name[0] == '!') > + { > + struct session_wildcard *sw = NULL; > + ltt_wildcard_create(chan, event_param, &sw ); >

Re: [lttng-dev] Support for large files

2012-06-30 Thread Mathieu Desnoyers
catch. I guess we did not test enough on 32-bit. It's fixed on the lttng-tools master branch by commit: commit c72b7d965eb65a9f49a5615cac731cec3082aa7f Author: Mathieu Desnoyers Date: Fri Jun 29 12:40:30 2012 +0200 Fix: support large files on 32-bit systems Signed-off-by: Mathieu Desnoyers it sh

Re: [lttng-dev] notrace missing in lttng-ust

2012-06-30 Thread Mathieu Desnoyers
f Google Inc. Use of this trademark is subject to > Google Permissions. > Linux is the registered trademark of Linus Torvalds in the U.S. and other > countries. > > > ___ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http

[lttng-dev] [babeltrace] FTS does not support large files on 32-bit

2012-06-30 Thread Mathieu Desnoyers
Hi Yannick, The babeltrace commit: commit 6cba487f031260536d6a77acde888c8b1a876fcf Author: Mathieu Desnoyers Date: Fri Feb 10 12:01:01 2012 -0500 babeltrace lib cleanup, folded with open/remove trace functions Folded patch from Yannick Brosseau , along with various updates

Re: [lttng-dev] [PATCH] Expose kernel tracer to user-space (version 5)

2012-07-02 Thread Mathieu Desnoyers
sched.o > obj-m += lttng-probe-irq.o > obj-m += lttng-probe-signal.o > obj-m += lttng-probe-timer.o > +obj-m += lttng-probe-uevent.o > +obj-m += lttng-uevent.o > > obj-m += lttng-probe-statedump.o > > diff --git a/probes/lttng-probe-uevent.c b/probes/lttng-probe-uevent.c >

Re: [lttng-dev] Adding New Tracepoints

2012-07-02 Thread Mathieu Desnoyers
Thanks, Mathieu > > > I am using. > Lttng modules 2.0.2 > Lttng tools:2.0.1 > Kernel:2.6.33.7 with RT-29 patch. > > Thanks In Advance, > Pavan > > -Original Message- > From: Pavan Anumula > Sent: Thursday, June 21, 2012 8:54 PM > To: 'Da

Re: [lttng-dev] [PATCH] Expose kernel tracer to user-space (version 5)

2012-07-02 Thread Mathieu Desnoyers
* Francis Giraldeau (francis.girald...@gmail.com) wrote: > Le 2012-07-02 16:13, Mathieu Desnoyers a écrit : > > * Francis Giraldeau (francis.girald...@gmail.com) wrote: > >> By writing to the file /proc/lttng, a user-space application creates a > >> kernel event. The ev

Re: [lttng-dev] [PATCH] Expose kernel tracer to user-space (version 6)

2012-07-02 Thread Mathieu Desnoyers
j-m += lttng-probe-sched.o > obj-m += lttng-probe-irq.o > obj-m += lttng-probe-signal.o > obj-m += lttng-probe-timer.o > +obj-m += lttng-uevent.o > > obj-m += lttng-probe-statedump.o > > diff --git a/probes/lttng-uevent.c b/probes/lttng-uevent.c > new file mode 1006

Re: [lttng-dev] [PATCH] Expose kernel tracer to user-space (version 6)

2012-07-03 Thread Mathieu Desnoyers
* Francis Giraldeau (francis.girald...@gmail.com) wrote: > Le 2012-07-03 00:23, Mathieu Desnoyers a écrit : > > * Francis Giraldeau (francis.girald...@gmail.com) wrote: > >> By writing to the file /proc/lttng_uevent, a user-space application > >> creates a > >&g

Re: [lttng-dev] [PATCH] Expose kernel tracer to user-space (version 6)

2012-07-03 Thread Mathieu Desnoyers
* Francis Giraldeau (francis.girald...@gmail.com) wrote: > Le 2012-07-03 13:39, Mathieu Desnoyers a écrit : > > * Francis Giraldeau (francis.girald...@gmail.com) wrote: > >> Le 2012-07-03 00:23, Mathieu Desnoyers a écrit : > >>> * Francis Giraldeau (francis.girald.

[lttng-dev] [RFC PATCH lttng-tools] Fix mmap for large subbuffers and handle EINTR

2012-07-04 Thread Mathieu Desnoyers
. Feedback is welcome. Signed-off-by: Mathieu Desnoyers --- diff --git a/src/common/kernel-consumer/kernel-consumer.c b/src/common/kernel-consumer/kernel-consumer.c index bbc31f8..eca710d 100644 --- a/src/common/kernel-consumer/kernel-consumer.c +++ b/src/common/kernel-consumer/kernel-consumer.c

Re: [lttng-dev] Questions about CTF format

2012-07-05 Thread Mathieu Desnoyers
t in CC, which I'm doing here. Thanks, Mathieu -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com ___ lttng-dev mailing list lttng-dev@lists.lttng.org http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Re: [lttng-dev] [RFC PATCH lttng-tools] Fix mmap for large subbuffers and handle EINTR

2012-07-06 Thread Mathieu Desnoyers
* David Goulet (dgou...@efficios.com) wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > Comments below. > > On 04/07/12 12:10 PM, Mathieu Desnoyers wrote: > > With large subbuffer (packet) size, if write() returns before copying the > > entire packet f

[lttng-dev] [PATCH lttng-tools] Fix mmap write() for large subbuffers and handle EINTR

2012-07-06 Thread Mathieu Desnoyers
return ret; + return written; } /* @@ -384,7 +394,7 @@ int lttng_ustconsumer_read_subbuffer(struct lttng_consumer_stream *stream, assert(err == 0); /* write the subbuffer to the tracefile */ ret = lttng_consumer_on_read_subbuffer_mmap(ctx, stream, len); - if (ret < 0) { +

[lttng-dev] [PATCH lttng-tools] Fix mmap write() for large subbuffers and handle EINTR (v2)

2012-07-06 Thread Mathieu Desnoyers
struct lttng_consumer_stream *stream, assert(err == 0); /* write the subbuffer to the tracefile */ ret = lttng_consumer_on_read_subbuffer_mmap(ctx, stream, len); - if (ret < 0) { + if (ret != len) {

[lttng-dev] [RFC PATCH lttng-tools] kernel consumer should empty the input splice pipe

2012-07-09 Thread Mathieu Desnoyers
currently encountered. Testing is welcome. Signed-off-by: Mathieu Desnoyers --- diff --git a/src/common/kernel-consumer/kernel-consumer.c b/src/common/kernel-consumer/kernel-consumer.c index bbc31f8..cc16f9f 100644 --- a/src/common/kernel-consumer/kernel-consumer.c +++ b/src/common/kernel-consumer

Re: [lttng-dev] Question: multiple cft_integer for one argument?

2012-07-10 Thread Mathieu Desnoyers
nding, it decides the output > format of each argument. > Why does the argument anint need three cft_integer with different types? > > Best Regards > Zheng > > ___ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-b

Re: [lttng-dev] Download links outdated

2012-07-10 Thread Mathieu Desnoyers
gt; lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com ___ lttng-dev mailing list lttng-dev@lists.lttng.org http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Re: [lttng-dev] [PATCH userspace-rcu] Add MIPS support

2012-07-10 Thread Mathieu Desnoyers
[ Edit by Mathieu Desnoyers: add explanations about supported MIPS architectures, extracted from conversation with Ralf Baechle: * Supported architectures Ralf Baechle (edited by Mathieu Desnoyers): This code works on all MIPS architecture variants. The memor

Re: [lttng-dev] Could I enable a particular system call tracing in LTTng 2.0

2012-07-12 Thread Mathieu Desnoyers
you very much. > > Regards, > Rui > ___ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com __

Re: [lttng-dev] Query about LTTng 2.0 Debian packages

2012-07-17 Thread Mathieu Desnoyers
* Jon Bernard (jbern...@debian.org) wrote: > * Jon Bernard wrote: > > * Mathieu Desnoyers wrote: > > > Hi Jon, > > > > > > I wanted to get in touch with you regarding LTTng 2.0 Debian packages. > > > Is there anything we can do to help you getting

Re: [lttng-dev] notrace missing in lttng-ust

2012-07-17 Thread Mathieu Desnoyers
\ > struct ltt_event *__event = __tp_data; \ > @@ -547,6 +554,8 @@ static struct lttng_probe_desc > _TP_COMBINE_TOKENS(__probe_desc___, TRACEPOINT_PR > /* Reset all macros within TRACEPOINT_EVENT */ > #include > static void __attribute__((

Re: [lttng-dev] UST in flight recorder mode

2012-07-17 Thread Mathieu Desnoyers
ffer snapshot. Hoping this helps, Thanks, Mathieu > > Cheers > > -- Sebastian > > ___ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Des

Re: [lttng-dev] notrace missing in lttng-ust

2012-07-18 Thread Mathieu Desnoyers
* Woegerer, Paul (paul_woege...@mentor.com) wrote: > On 07/17/2012 04:13 PM, Mathieu Desnoyers wrote: > >> I think we want to make the notrace always active. I don't see the point >> in letting UST lib be compiled with those tracing stubs in place. > > OK, I'

Re: [lttng-dev] notrace missing in lttng-ust

2012-07-18 Thread Mathieu Desnoyers
I ended up pushing your work as: commit a8909ba54718ed54cefb1b839a5d41a065df09f9 Author: Paul Woegerer Date: Wed Jul 18 15:28:44 2012 -0400 Make lttng-ust robust against -finstrument-functions. [ Edit by Mathieu Desnoyers: We need to declare the no_instrument_function

Re: [lttng-dev] Question about file descriptor of sockets and pipes

2012-07-19 Thread Mathieu Desnoyers
___ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com ___ lttng-dev mailing list lttng-dev@lists.lttng.org http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Re: [lttng-dev] LTTng

2012-07-20 Thread Mathieu Desnoyers
logging. > > Thanks, > > Luay Alawneh > Nuance Communications > > ___ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathi

Re: [lttng-dev] some questions on lttng

2012-07-20 Thread Mathieu Desnoyers
ffer is very modular, it should be straightforward to implement. Thanks! Mathieu > > Thank you, > - Bingfeng > > ___ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi

Re: [lttng-dev] [babeltrace] FTS does not support large files on 32-bit

2012-07-20 Thread Mathieu Desnoyers
ly he will make progress on this issue within the next week. Adding a bug entry would be helpful to track this issue, yes. Thanks, Mathieu > > > Cheers, > > Fredrik Östman > > > ___ > lttng-dev mailing list > lttng-dev

Re: [lttng-dev] LTTng-UST alternative to LD_PRELOAD

2012-07-20 Thread Mathieu Desnoyers
; Best Regards, > Lars Ekman > > > > ___________ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc.

Re: [lttng-dev] [RFC PATCH lttng-tools] relayd pipes and error handling

2012-07-20 Thread Mathieu Desnoyers
HYRc3aKk1+X5u > RMnIuqx4ZdkakeGkTtZMr3ujmy/vOj9NOMuZsigV9z7g/jL5uXzfL0eJcW9kJLQ3 > cNEtdO519q9hInoxpRBkRQtJya1LHoT66oc5AK8TWMBCLjHzvTvvqWNSEBEOfQqH > b7CulD3x6//DvnlVImA3zFNB2YXLvY6bL3XnU9I/DIEENjG1uAP+9/h+IzWRMNcS > F0bttSrK9faUqBedMB81hadx0uLjpQxnIB9oFR2NEW/W1J3HAwwVN5YntZFo+lw= > =BfP8 > -END PGP SIGNATURE- > > ___ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com ___ lttng-dev mailing list lttng-dev@lists.lttng.org http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Re: [lttng-dev] When is tracepoint registration finished?

2012-07-20 Thread Mathieu Desnoyers
_ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com __

Re: [lttng-dev] Need inputs on LTTng for ARM Linux-3.0.15

2012-07-20 Thread Mathieu Desnoyers
ot;The Good You Do, The Best You GET" > > Regards > Srikanth Krishnakar > ** -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com ___ lttng-dev mailing list lttng-dev@lists.lttng.org http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Re: [lttng-dev] some questions on lttng

2012-07-23 Thread Mathieu Desnoyers
* bingfeng.z...@emc.com (bingfeng.z...@emc.com) wrote: > > > > -Original Message- > > From: Mathieu Desnoyers [mailto:mathieu.desnoy...@efficios.com] > > Sent: Saturday, July 21, 2012 1:26 AM > > To: Zhao, Bingfeng; t...@linutronix.de > > Cc: lttng

[lttng-dev] [PATCH lttng-tools] Fix: health monitoring (various fixes)

2012-07-23 Thread Mathieu Desnoyers
ror, not on "normal" teardown condition (thread quit pipe being closed). Flagging threads as being in error condition upon all exit paths would lead to false "errors" sent to the client, which we want to avoid, since the client could then think it needs to kill a sessiond wh

[lttng-dev] [PATCH lttng-tools] Fix: health check and time

2012-07-23 Thread Mathieu Desnoyers
the "last" snapshot, so we can do a time delta between the current and last snapshot to figure out if we need to report the thread as stalled or not. Signed-off-by: Mathieu Desnoyers CC: David Goulet --- diff --git a/src/bin/lttng-sessiond/health.c b/src/bin/lttng-sessiond/health.c in

Re: [lttng-dev] [PATCH lttng-tools] Fix: health check and time

2012-07-24 Thread Mathieu Desnoyers
the following fix ? Fix: set health to bad state for good when error state detected Signed-off-by: Mathieu Desnoyers --- diff --git a/src/bin/lttng-sessiond/health.c b/src/bin/lttng-sessiond/health.c index 7bf41c8..3637225 100644 --- a/src/bin/lttng-sessiond/health.c +++ b/src/bin/lttng-sess

Re: [lttng-dev] Question about the performance of LTTng-UST

2012-07-25 Thread Mathieu Desnoyers
Having those showing up in strace tells me that you kernel might be quite old, or use use an architecture that implements those as standard system calls rather than vDSO. Thanks, Mathieu > > BR > Zheng > ___ > lttng-dev mailing list >

Re: [lttng-dev] lttng-ust, libtool and cross-compilation

2012-07-30 Thread Mathieu Desnoyers
ng that script. So typically it will require no interaction from users, except when they "know what they are doing" and want to explicitely disable our work-around. Thoughts ? Mathieu > > Other components of LTTng might be affected be the same issue. > > Any thoughts or add

Re: [lttng-dev] how to get the sockaddr struct from its pointer in kernel tracing

2012-07-31 Thread Mathieu Desnoyers
tp://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com ___ lttng-dev mailing list lttng-dev@lists.lttng.org http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Re: [lttng-dev] how to get the sockaddr struct from its pointer in kernel tracing

2012-07-31 Thread Mathieu Desnoyers
am be able to do it or not within > time constrain. > > Regards, > Rui > > On Tue, Jul 31, 2012 at 12:29 PM, Mathieu Desnoyers < > mathieu.desnoy...@efficios.com> wrote: > > > * Rui Han (r@umiami.edu) wrote: > > > Hi, > > > > > > In the

Re: [lttng-dev] [PATCH] Fix: unresolvable dependencies to liblttng-ust-tracepoint when cross-compiling.

2012-08-02 Thread Mathieu Desnoyers
H_LIBDL > -- > 1.7.0.4 > > > ___ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com ___ lttng-dev mailing list lttng-dev@lists.lttng.org http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Re: [lttng-dev] [PATCH lttng-ust] Fix: Libtool fails to find dependent libraries when cross-compiling lttng-ust

2012-08-02 Thread Mathieu Desnoyers
t; + AC_MSG_WARN([the detected libtool will not link all > dependencies, forcing link_all_deplibs = unknown]) > + sed -i "s/\($libtool_flag_pattern\)=no/\1=unknown/g" > $libtool_m4 Could you modify your patch to tabs and spaces around "=" wo

Re: [lttng-dev] LTTng / Android questions

2012-08-02 Thread Mathieu Desnoyers
n conjunction with an Android build (or NDK?) > area. Would you view one of these (or something else), a "preferred" > way? > > Thanks, > Kent > > -Original Message- > From: Mathieu Desnoyers [mailto:mathieu.desnoy...@efficios.com] > Sent: Tues

Re: [lttng-dev] [PATCH v2 lttng-ust] Fix: Libtool fails to find dependent libraries when cross-compiling lttng-ust

2012-08-02 Thread Mathieu Desnoyers
attern_count]) > + AC_MSG_WARN([the detected libtool will not link all > dependencies, forcing link_all_deplibs = unknown]) > + sed -i "s/\($libtool_flag_pattern\)\s*=\s*no/\1=unknown/g" > $libtool_m4 > + ], > + [ >

Re: [lttng-dev] Getting 64bit int from babeltrace library

2012-08-03 Thread Mathieu Desnoyers
_ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com

Re: [lttng-dev] ERROR: No buffer space available

2012-08-05 Thread Mathieu Desnoyers
nect macros to enable the logging of the > connected IP/port from the socket. Is this method right? what are other > possible solutions in order to logging the details of the socket > information? > > Thank you very much. > > Regards, > Rui -- Mathieu Desnoyers Operati

Re: [lttng-dev] Babeltrace API timestamp manipulation changes

2012-08-05 Thread Mathieu Desnoyers
ced by > > "bt_ctf_get_real_timestamp" > > - the function "bt_ctf_get_cycles_timestamp" is added > > > Why not simply calling them > bt_ctf_get_timestamp and bt_ctf_get_cycles ? Yes, it makes sense. Julien, any thoughts on this ? Thanks, Mathieu > >

Re: [lttng-dev] urcu rfc: privatize urcu/futex.h

2012-08-07 Thread Mathieu Desnoyers
ve on the wake up site too ? > > Thanks, > Lai. > > (today I found a old patchset in my private tree, tried to rebase it > and suddenly notice this urcu/futex.h but the patchset has nothing > related with urcu/futex.h). I look forward to see that work! Thanks! Mathieu --

[lttng-dev] [PATCH] Cleanup: Remove misplaced config.h include

2012-08-07 Thread Mathieu Desnoyers
lddir)/include -include config.h"), but let's remove this misleading line of code. Signed-off-by: Mathieu Desnoyers --- diff --git a/src/common/hashtable/rculfhash.c b/src/common/hashtable/rculfhash.c index 776f9f3..423d186 100644 --- a/src/common/hashtable/rculfhash.c +++ b/src/common/hashta

Re: [lttng-dev] [BABELTRACE PATCH] API cleanup name get_timestamp and get_cycles

2012-08-07 Thread Mathieu Desnoyers
event offsetted > * with the system clock source (in ns) or -1ULL on error > */ > -uint64_t bt_ctf_get_real_timestamp(const struct bt_ctf_event *event); > +uint64_t bt_ctf_get_timestamp(const struct bt_ctf_event *event); > > /* > * bt_ctf_get_field_list: set list point

Re: [lttng-dev] [RFC] LTTng address API proposal v2

2012-08-07 Thread Mathieu Desnoyers
UK/94MEPjeWd2cSNJxsaidYj9mD07ST9UWfputU= > =GYy5 > -END PGP SIGNATURE- > RFC - LTTng address API proposal > > Author: David Goulet > > Contributors: > * Mathieu Desnoyers > * Yannick Brosseau > > Version: > - v0.1: 31/07/2012 >

Re: [lttng-dev] [RFC] LTTng address API proposal v3

2012-08-07 Thread Mathieu Desnoyers
ikqWyRt/4wwiufaZNtlC85/w > C0pGJL/9TBTY4rFmyZ/e19lJRUoNEygrFRRoJb5qxKWH5nHvfRXdFGw78xjDgim7 > g9AvrWZ7g73KZaS7LEpmAcofgRicWafIMd0+jUXv0MMrj3NeT/xD66UbtcUusl8= > =jRvr > -END PGP SIGNATURE- > RFC - LTTng address API proposal > > Author: David Goulet > > Contributors: > * Mathieu De

Re: [lttng-dev] [BABELTRACE PATCH] Fix: wrong type in bt_ctf_get_uint64/int64

2012-08-07 Thread Mathieu Desnoyers
ct definition *field) > { > - int ret = 0; > + int64_t ret = 0; > > if (field && bt_ctf_field_type(field) == CTF_TYPE_INTEGER) > ret = get_signed_int(field); > -- > 1.7.10.4 > -- Mathieu Desnoyers Operating System Efficiency R&D Consu

Re: [lttng-dev] Question about Transformation to CTF file

2012-08-09 Thread Mathieu Desnoyers
}; > }; > stream { > packet.context := struct { > uint32_t content_size; > uint32_t packet_size; > }; > typealias integer { size = 64; align = 64; signed = false; } := > uint64_t; > event.header := st

Re: [lttng-dev] syscall options

2012-08-09 Thread Mathieu Desnoyers
gt; lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com ___ lttng-dev mailing list lttng-de

Re: [lttng-dev] [PATCH lttng-tools 1/2] Fix: Libtool fails to find dependent libraries when cross-compiling lttng-tools

2012-08-09 Thread Mathieu Desnoyers
_deplibs = no in >./config/libtool.m4... 3 > configure: WARNING: the detected libtool will not link all > dependencies, forcing link_all_deplibs = unknown > > Signed-off-by: Christian Babeux Acked-by: Mathieu Desnoyers > --- > configure.ac | 25 +++

Re: [lttng-dev] [PATCH lttng-tools 2/2] Fix: Missing libs dependencies in configure check for lttng-ust-ctl

2012-08-09 Thread Mathieu Desnoyers
> # check for dlopen > -- > 1.7.11.3 > > > ___ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com ___ lttng-dev mailing list lttng-dev@lists.lttng.org http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Re: [lttng-dev] ERROR: No buffer space available

2012-08-09 Thread Mathieu Desnoyers
the header. Could > you give me some hint that I can understand lttng and debug the code to get > what I wanted? > > Thank you very much. > > Regards, > Rui > > On Sun, Aug 5, 2012 at 1:03 PM, Mathieu Desnoyers < > mathieu.desnoy...@efficios.com> wrot

Re: [lttng-dev] Uprobe and extractor for coredump

2012-08-09 Thread Mathieu Desnoyers
breaking the code dump. Thoughts ? Thanks, Mathieu > > > > BR > > Zheng > > ___ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/l

Re: [lttng-dev] [PATCH 1/2] urcu: add hint to DEFINE_URCU_TLS() for compound types

2012-08-09 Thread Mathieu Desnoyers
LS(my_int_array_type, var_name); > + * > + * Another exmaple: > + * typedef void (*call_rcu_flavor)(struct rcu_head *, ); > + * DECLARE_URCU_TLS(call_rcu_flavor, p_call_rcu); > + */ > + > # define DECLARE_URCU_TLS(type, name)\ > CONFIG_RCU_TLS type name &

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

2012-08-09 Thread Mathieu Desnoyers
id) (in tls-compat.h) and make sure it is async-signal-safe I guess ? I would prefer that solution: it would make the code more robust for a rarely taken performance hit. Thoughts ? Thanks, Mathieu > + */ > + > # include > > struct urcu_tls { > -- > 1.7

Re: [lttng-dev] [PATCH] urcu: fix compat_futex_noasync()

2012-08-09 Thread Mathieu Desnoyers
c(int32_t *uaddr, int op, int32_t > val, > pthread_cond_wait(&compat_futex_cond, &compat_futex_lock); > break; > case FUTEX_WAKE: > - for (i = 0; i < val; i++) > - pthread_cond_signal(&compat_futex_cond); > +

Re: [lttng-dev] [PATCH 1/2] urcu: move exsit code and name it ___cds_wfq_node_sync_next()

2012-08-09 Thread Mathieu Desnoyers
(). Signed-off-by: Lai Jiangshan Signed-off-by: Mathieu Desnoyers Thanks! Mathieu > > Signed-off-by: Lai Jiangshan > --- > urcu/static/wfqueue.h | 36 +--- > 1 files changed, 25 insertions(+), 11 deletions(-) > > diff --g

Re: [lttng-dev] [PATCH lttng-tools 2/2] Fix: Missing libs dependencies in configure check for lttng-ust-ctl

2012-08-09 Thread Mathieu Desnoyers
ONDITIONAL([HAVE_LIBLTTNG_UST_CTL], [test "x$lttng_ust_ctl_found" = > >> xyes]) > > > > why are you changing this line ? > > > > I only changed the test flag to lttng_ust_ctl_found for clarity sake. > It might not be obvious to everyone th

Re: [lttng-dev] [babeltrace PATCH] Fix: correct name of bt_ctf_field_get_error in comments and typo in man page

2012-08-09 Thread Mathieu Desnoyers
e value otherwise. > */ > -- > 1.7.9.5 > > > ___ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com ___ lttng-dev mailing list lttng-dev@lists.lttng.org http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

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

2012-08-09 Thread Mathieu Desnoyers
gt; > On Thu, Aug 9, 2012 at 10:14 PM, Mathieu Desnoyers > wrote: > > * Lai Jiangshan (la...@cn.fujitsu.com) wrote: > >> Signed-off-by: Lai Jiangshan > >> --- > >> urcu/tls-compat.h | 14 ++ > >> 1 files changed, 14 insertions(+), 0

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

2012-08-09 Thread Mathieu Desnoyers
b/lib/iterator.c > index bcb77d8..5125afb 100644 > --- a/lib/iterator.c > +++ b/lib/iterator.c > @@ -3,7 +3,7 @@ > * > * Babeltrace Library > * > - * Copyright 2010-2011 EfficiOS Inc. and Linux Foundation > +/ * Copyright 2010-2011 EfficiOS Inc. and Linux Foundatio

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
-signal-safety of pthread_key_* and TLS. The code we refer to is: http://git.lttng.org/?p=userspace-rcu.git;a=blob;f=urcu/tls-compat.h;h=192a53609fb5f6bc445f98fdd6bc26918126687e;hb=HEAD) * Lai Jiangshan (la...@cn.fujitsu.com) wrote: > On 08/10/2012 04:02 AM, Mathieu Desnoyers wrote: > > L

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

2012-08-10 Thread Mathieu Desnoyers
_wfq_dequeue_all_blocking(&crdp->cbs, &tail); > + assert(head); > + > /* Create default call rcu data if need be */ > (void) get_default_call_rcu_data(); > - cbs_endprev = (struct cds_wfq_node **) > -

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

2012-08-10 Thread Mathieu Desnoyers
> babeltrace/types.h \ > babeltrace/ctf-ir/metadata.h \ > babeltrace/ctf/events-internal.h \ > -- > 1.7.9.5 > > > ___ > lttng-dev mailing list > lttng-dev@lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/list

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

2012-08-10 Thread Mathieu Desnoyers
- strncpy(handle->path, path, PATH_MAX); > - handle->path[PATH_MAX - 1] = '\0'; > + if (path) { > + strncpy(handle->path, path, PATH_MAX); > + handle->path[PATH_MAX - 1] = '\0'; > + } > > if (fmt->set_h

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

2012-08-10 Thread Mathieu Desnoyers
tc = iter->ctx->tc; > + > + ret = seek_last_ctf_file_stream(tc, &cfs); > + if (ret < 0) > + goto error; > + /* remove all stream from the heap*/ > + heap_free(iter->stream_heap); > +

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

2012-08-11 Thread Mathieu Desnoyers
lt call rcu data if need be */ > (void) get_default_call_rcu_data(); > - cbs_endprev = (struct cds_wfq_node **) > - uatomic_xchg(&default_call_rcu_data, cbs_tail); > - *cbs_endprev = cbs; > + > + __cds_w

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

2012-08-11 Thread Mathieu Desnoyers
* Mathieu Desnoyers (mathieu.desnoy...@efficios.com) 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: &

[lttng-dev] [RFC PATCH] wfqueue: expand API, simplify implementation, small performance boost

2012-08-12 Thread Mathieu Desnoyers
enqueue, 2% (performance improvement) for dequeue. Overall number of operations (dequeue+enqueue) increased with the patch. We can verify that: successful enqueues - successful dequeues = end_dequeues For all runs (ensures correctness: no lost node). CC: Lai Jiangshan CC: Paul McKenney Signed-of

Re: [lttng-dev] LTTng: problem with lttng-sessiond.

2012-08-13 Thread Mathieu Desnoyers
are set [in set_permissions() at main.c:5310] > DEBUG1: epoll set max size is 180337 [in compat_epoll_set_max_size() > at compat-epoll.c:224] > Illegal instruction > > What to do? :P > > -- > Henrik Hautakoski > hen...@fiktivkod.org -- Mathieu Desnoyers Operati

Re: [lttng-dev] [RFC PATCH] wfqueue: expand API, simplify implementation, small performance boost

2012-08-13 Thread Mathieu Desnoyers
Hi Lai, * Lai Jiangshan (eag0...@gmail.com) wrote: > On Sun, Aug 12, 2012 at 10:50 PM, Mathieu Desnoyers > wrote: > > This work is derived from the patch from Lai Jiangshan submitted as > > "urcu: new wfqueue implementation" > > (http://lists.lttng.org/pipermail

[lttng-dev] [RFC PATCH LTTng-UST] Fix UST SIGPIPE handling

2012-08-14 Thread Mathieu Desnoyers
= pthread_sigmask(SIG_SETMASK, &old_set, NULL); + assert(!ret); + } } } - } } } -- Mathieu Desnoyers Operating System Efficiency R&D Consultant Eff

Re: [lttng-dev] [RFC PATCH] wfqueue: expand API, simplify implementation, small performance boost

2012-08-14 Thread Mathieu Desnoyers
* Lai Jiangshan (eag0...@gmail.com) wrote: > On Tue, Aug 14, 2012 at 4:12 AM, Mathieu Desnoyers > wrote: [...] > >> > /* > >> > + * ___cds_wfq_first_blocking: get first node of a queue, without > >> > dequeuing. > >> > + * > >> >

Re: [lttng-dev] [RFC PATCH] wfqueue: expand API, simplify implementation, small performance boost

2012-08-14 Thread Mathieu Desnoyers
* Mathieu Desnoyers (mathieu.desnoy...@efficios.com) wrote: > * Lai Jiangshan (eag0...@gmail.com) wrote: > > On Tue, Aug 14, 2012 at 4:12 AM, Mathieu Desnoyers > > wrote: > [...] > > >> > /* > > >> > + * ___cds_wfq_first_blocking: get fi

Re: [lttng-dev] [RFC PATCH] wfqueue: expand API, simplify implementation, small performance boost

2012-08-14 Thread Mathieu Desnoyers
* Mathieu Desnoyers (mathieu.desnoy...@efficios.com) wrote: [...] > One more thing: I think we need to add cmm_smp_read_barrier_depends() > before returning the node pointers in dequeue, first, and next > operations. This memory barrier would match the implicit barrier in the > enqu

Re: [lttng-dev] [RFC PATCH] wfqueue: expand API, simplify implementation, small performance boost

2012-08-15 Thread Mathieu Desnoyers
* Lai Jiangshan (eag0...@gmail.com) wrote: > On Wed, Aug 15, 2012 at 6:27 AM, Mathieu Desnoyers > wrote: > > * Lai Jiangshan (eag0...@gmail.com) wrote: > >> On Tue, Aug 14, 2012 at 4:12 AM, Mathieu Desnoyers > >> wrote: > > [...] > >> >> >

[lttng-dev] [RFC URCU PATCH] Introduce wfqueue v0 ABI compatibility

2012-08-15 Thread Mathieu Desnoyers
Introduce wfqueue v0 ABI compatibility Preparing for v1 ABI. CC: Lai Jiangshan CC: Paul McKenney Signed-off-by: Mathieu Desnoyers --- diff --git a/wfqueue0.c b/wfqueue0.c new file mode 100644 index 000..e928c0a --- /dev/null +++ b/wfqueue0.c @@ -0,0 +1,191 @@ +/* + * wfqueue0.c

[lttng-dev] [RFC URCU PATCH] wfqueue: ABI v1, simplify implementation, 2.3x to 2.6x performance boost

2012-08-15 Thread Mathieu Desnoyers
only way to run into this corner-case would be to combine objects compiled with different versions of the urcu/wfqueue.h header. CC: Lai Jiangshan CC: Paul McKenney Signed-off-by: Mathieu Desnoyers --- diff --git a/Makefile.am b/Makefile.am index 2396fcf..31052ef 100644 --- a/Makefile.am

Re: [lttng-dev] [PATCH lttng-tools 1/2] Fix: Invalid free on session_name when destroying session

2012-08-16 Thread Mathieu Desnoyers
s gracefully. > > Signed-off-by: Christian Babeux Acked-by: Mathieu Desnoyers > --- > src/bin/lttng/commands/destroy.c | 27 --- > 1 file changed, 16 insertions(+), 11 deletions(-) > > diff --git a/src/bin/lttng/commands/destroy.c > b/src/bin/lttn

Re: [lttng-dev] [PATCH lttng-tools 2/2] Cleanup: Remove unused get_default_session_name() function

2012-08-16 Thread Mathieu Desnoyers
* Christian Babeux (christian.bab...@efficios.com) wrote: > > Signed-off-by: Christian Babeux Acked-by: Mathieu Desnoyers > --- > src/bin/lttng/commands/destroy.c | 15 --- > 1 file changed, 15 deletions(-) > > diff --git a/src/bin/lttng/commands/destroy

Re: [lttng-dev] [RFC PATCH] wfqueue: expand API, simplify implementation, small performance boost

2012-08-16 Thread Mathieu Desnoyers
n wfqueue without having to have legacy code in our way: I plan to move all the legacy ABI to wfqueue0.c, and have a fresh start in terms of ABI for the new wfqueue. That way, we can eventually remove the wfqueue v0 ABI when we decide to bump the library version number (and break compatibi

Re: [lttng-dev] [RFC PATCH] wfqueue: expand API, simplify implementation, small performance boost

2012-08-16 Thread Mathieu Desnoyers
ze, which would break API compatibility. Any idea on how to > >> best do this without causing incompatibility would be welcome. > >> > > > > choice 1) two set of APIs?(cache-line-opt and none-cache-line-opt), > > many users don't need the cache-line-opt

<    2   3   4   5   6   7   8   9   10   11   >