lttng-tools.git
12 years agoAdd missing function references with no UST support
David Goulet [Mon, 7 Nov 2011 17:07:34 +0000 (12:07 -0500)] 
Add missing function references with no UST support

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
12 years agoFix incorrect parenthesis for poll test in consumer
Mathieu Desnoyers [Sat, 5 Nov 2011 17:57:01 +0000 (13:57 -0400)] 
Fix incorrect parenthesis for poll test in consumer

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
12 years agoUST consumer: implement missing ust-specific callbacks + fixes.
Mathieu Desnoyers [Sat, 5 Nov 2011 17:45:35 +0000 (13:45 -0400)] 
UST consumer: implement missing ust-specific callbacks + fixes.

Also fix consumer handling of poll fd: a mask should be used.
Also fix UST stream output passed to the consumer (the fix is a hack
currently, left a FIXME).

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
12 years agoAdd missing rcu_read_unlock on error paths
Mathieu Desnoyers [Sat, 5 Nov 2011 15:31:24 +0000 (11:31 -0400)] 
Add missing rcu_read_unlock on error paths

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
12 years agoFix order of streams
Mathieu Desnoyers [Sat, 5 Nov 2011 15:27:53 +0000 (11:27 -0400)] 
Fix order of streams

Use a list instead of hash table to send streams in correct order.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoUST consumer interaction fix: send all channel streams
Mathieu Desnoyers [Fri, 4 Nov 2011 12:31:56 +0000 (08:31 -0400)] 
UST consumer interaction fix: send all channel streams

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoLTTng-UST support: --disable-lttng-ust build option
Mathieu Desnoyers [Fri, 4 Nov 2011 11:32:26 +0000 (07:32 -0400)] 
LTTng-UST support: --disable-lttng-ust build option

From now on, configure fails if lttng-ust is not found.
--disable-lttng-ust must be explicitely specified to build without UST
support.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoust consumer: flag shm/wait fd as copy
Mathieu Desnoyers [Thu, 3 Nov 2011 23:33:24 +0000 (19:33 -0400)] 
ust consumer: flag shm/wait fd as copy

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoust consumer: fix double close
Mathieu Desnoyers [Thu, 3 Nov 2011 23:18:51 +0000 (19:18 -0400)] 
ust consumer: fix double close

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoconsumer handling fix: send metadata channel shm size
Mathieu Desnoyers [Thu, 3 Nov 2011 19:18:04 +0000 (15:18 -0400)] 
consumer handling fix: send metadata channel shm size

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoFix UST consumer FD send: channels only send one FD
Mathieu Desnoyers [Thu, 3 Nov 2011 19:01:43 +0000 (15:01 -0400)] 
Fix UST consumer FD send: channels only send one FD

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoFix debug message in send consumer fd.
Mathieu Desnoyers [Thu, 3 Nov 2011 18:52:01 +0000 (14:52 -0400)] 
Fix debug message in send consumer fd.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoRemove *bad* debug printf
David Goulet [Thu, 3 Nov 2011 18:26:26 +0000 (14:26 -0400)] 
Remove *bad* debug printf

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoChange linking libs using new UST names
David Goulet [Thu, 3 Nov 2011 17:19:06 +0000 (13:19 -0400)] 
Change linking libs using new UST names

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix configure.ac using new UST headers location
David Goulet [Thu, 3 Nov 2011 17:09:22 +0000 (13:09 -0400)] 
Fix configure.ac using new UST headers location

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoChange UST header location and update in tree headers
David Goulet [Thu, 3 Nov 2011 16:55:11 +0000 (12:55 -0400)] 
Change UST header location and update in tree headers

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoSupport urcu 0.6.6 with the likely/unlikely API change
David Goulet [Thu, 3 Nov 2011 15:59:02 +0000 (11:59 -0400)] 
Support urcu 0.6.6 with the likely/unlikely API change

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoRename ltt-sessiond to lttng-sessiond
David Goulet [Thu, 3 Nov 2011 15:34:59 +0000 (11:34 -0400)] 
Rename ltt-sessiond to lttng-sessiond

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix UST renaming and update ust headers
David Goulet [Thu, 3 Nov 2011 15:06:40 +0000 (11:06 -0400)] 
Fix UST renaming and update ust headers

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoUpdate README and TODO
David Goulet [Thu, 3 Nov 2011 15:00:50 +0000 (11:00 -0400)] 
Update README and TODO

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix build system with libustcomm removed
David Goulet [Thu, 3 Nov 2011 14:13:01 +0000 (10:13 -0400)] 
Fix build system with libustcomm removed

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoRemove unused and empty lib
David Goulet [Thu, 3 Nov 2011 14:01:17 +0000 (10:01 -0400)] 
Remove unused and empty lib

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoltt-sessiond: use short options for modprobe in order to support Busybox
Thomas Petazzoni [Thu, 3 Nov 2011 11:51:21 +0000 (12:51 +0100)] 
ltt-sessiond: use short options for modprobe in order to support Busybox

Many embedded systems are based on Busybox, and therefore use the
Busybox implementation of modprobe. This implementation does not
support long options such as --remove and --quiet, only short options
such as -r and -q are supported.

This patches changes ltt-sessiond to use the short options, which are
more widely available, and allows lttng to work easily on a
Busybox-based system.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
13 years agoPreliminary work for full UST support
David Goulet [Wed, 2 Nov 2011 19:44:27 +0000 (15:44 -0400)] 
Preliminary work for full UST support

Add hashtable support for all UST data structure. At this point, enable
channel and event works with the global UST domain. The interaction with
the ust consumer has problems for now thus not working at this stage.

NOTICE: This commit does NOT work with the user space tracer yet. Please
refer to next commit for working version.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoUpdate RCU hashtable from upstream urcu
David Goulet [Wed, 2 Nov 2011 19:43:50 +0000 (15:43 -0400)] 
Update RCU hashtable from upstream urcu

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoImport hashtable files from URCU upstream git
David Goulet [Thu, 27 Oct 2011 12:50:45 +0000 (14:50 +0200)] 
Import hashtable files from URCU upstream git

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoBasic hashtable support for UST
David Goulet [Thu, 27 Oct 2011 12:49:48 +0000 (14:49 +0200)] 
Basic hashtable support for UST

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoUpdate documentation following babeltrace UI change
Mathieu Desnoyers [Wed, 2 Nov 2011 14:06:49 +0000 (10:06 -0400)] 
Update documentation following babeltrace UI change

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoUST support: add UST control commands
Mathieu Desnoyers [Thu, 27 Oct 2011 12:33:28 +0000 (14:33 +0200)] 
UST support: add UST control commands

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoFix kconsumerd number of fd expected
Mathieu Desnoyers [Thu, 27 Oct 2011 12:02:40 +0000 (08:02 -0400)] 
Fix kconsumerd number of fd expected

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoAdd lttng-consumerd to gitignore
David Goulet [Wed, 26 Oct 2011 11:50:16 +0000 (07:50 -0400)] 
Add lttng-consumerd to gitignore

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoRemove unused variable
David Goulet [Wed, 26 Oct 2011 11:49:38 +0000 (07:49 -0400)] 
Remove unused variable

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoAPI change for lttng_get_readeable_code
David Goulet [Tue, 25 Oct 2011 19:29:50 +0000 (15:29 -0400)] 
API change for lttng_get_readeable_code

The liblttngctl lttng_get_readeable_code call is renamed to
lttng_strerror.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix array size in kconsumer add stream
David Goulet [Tue, 25 Oct 2011 19:26:09 +0000 (15:26 -0400)] 
Fix array size in kconsumer add stream

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix inline function bad definition
David Goulet [Tue, 25 Oct 2011 19:25:11 +0000 (15:25 -0400)] 
Fix inline function bad definition

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoUse UST comm lib
Mathieu Desnoyers [Sun, 23 Oct 2011 13:23:12 +0000 (09:23 -0400)] 
Use UST comm lib

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoUpdate UST messages
Mathieu Desnoyers [Sun, 23 Oct 2011 12:50:13 +0000 (08:50 -0400)] 
Update UST messages

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoUST 2.0 support
Mathieu Desnoyers [Sun, 23 Oct 2011 01:06:11 +0000 (21:06 -0400)] 
UST 2.0 support

Refactoring of the consumer/sessiond interaction so the consumer
supports applications instrumented with libust (UST 2.0).

At this point, more testing of interaction between libust and sessiond
is required.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoFix disable event
David Goulet [Wed, 5 Oct 2011 22:40:19 +0000 (18:40 -0400)] 
Fix disable event

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoAdd EEXIST support for kernel event on the tracer side
David Goulet [Wed, 5 Oct 2011 22:20:34 +0000 (18:20 -0400)] 
Add EEXIST support for kernel event on the tracer side

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix bad channel handling on enable-event
David Goulet [Wed, 5 Oct 2011 22:09:23 +0000 (18:09 -0400)] 
Fix bad channel handling on enable-event

The channel attributes and name were not passed to the create function
when a channel is requested and did not exist on enable-event.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix comments in lttng-sessiond-comm code
David Goulet [Tue, 4 Oct 2011 21:53:16 +0000 (17:53 -0400)] 
Fix comments in lttng-sessiond-comm code

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoAdd recv fds function to lttng-comm API
David Goulet [Tue, 4 Oct 2011 21:49:27 +0000 (17:49 -0400)] 
Add recv fds function to lttng-comm API

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoAdd missing mutex lock for kconsumerd pid
David Goulet [Tue, 4 Oct 2011 21:27:50 +0000 (17:27 -0400)] 
Add missing mutex lock for kconsumerd pid

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoAdd sem_wait timeout on the kconsumerd thread
David Goulet [Tue, 4 Oct 2011 21:15:32 +0000 (17:15 -0400)] 
Add sem_wait timeout on the kconsumerd thread

It's unlikely to happen but is the kconsumerd was not able to send us
the sock ready command, at least the session daemon will not stall
forever. A 30 seconds timeout is added and error is handle after that.

Also adds the handling of the sem_init ret value and adds mutex lock
around the kconsumerd pid in the same function.

There was also a problem on error code handling where LTTCOMM error was
returned by spawn kconsumerd function but handled as if it might be
negative at the end of the call chain. This was preventing to return the
right error code to the client when the kconsumerd failed.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoRemove session list lock acquisition of the search
David Goulet [Tue, 4 Oct 2011 20:24:10 +0000 (16:24 -0400)] 
Remove session list lock acquisition of the search

The lock/unlock of the session list in the find session by name function
is removed thus the caller must held the lock before calling that search
function.

Also, the path and name are now static size on the stack and not
allocated strings at session creation.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix possible use of uninitialized variable
David Goulet [Tue, 4 Oct 2011 14:54:05 +0000 (10:54 -0400)] 
Fix possible use of uninitialized variable

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix session list deadlock
David Goulet [Mon, 3 Oct 2011 19:23:16 +0000 (15:23 -0400)] 
Fix session list deadlock

On session destroy, the session list lock is acquired but this call
could be made inside a section where this lock is already acquired. The
call has been changed to using the session pointer and not the name thus
removing a useless list walk.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix enable syscall and bad value of poll set size
David Goulet [Mon, 3 Oct 2011 18:46:40 +0000 (14:46 -0400)] 
Fix enable syscall and bad value of poll set size

First, when enabling syscall tracing, on success 0 is return so we don't
add this value to the event list or else we were polling on fd 0 which
is not a valid event fd. A normal kernel create event, the fd of that
event is returned from the ioctl which is not the case for syscalls.

Second, the poll size was not reset when updating the kernel poll set so
at each kernel poll update, the value was not coherent with the real
poll size and thus creating unstable behavior (unknown).

This commit MIGHT fix a blocking problem happening when using a custom
channel and enabling syscall on it. This issue was reported by Julien
Desfossez <julien.desfossez@polymtl.ca>.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoDon't report an error when the last section of the path exist in mkdir_recursive
Yannick Brosseau [Mon, 19 Sep 2011 21:17:56 +0000 (17:17 -0400)] 
Don't report an error when the last section of the path exist in mkdir_recursive

Minor adjustment for perror call changed to PERROR using the lttngerr.h
interface.

Signed-off-by: Yannick Brosseau <yannick.brosseau@gmail.com>
Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoVersion 2.0-pre14 v2.0-pre14
Mathieu Desnoyers [Fri, 30 Sep 2011 15:32:22 +0000 (11:32 -0400)] 
Version 2.0-pre14

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoImplement enable all vs enable tracepoints vs enable syscalls
Mathieu Desnoyers [Fri, 30 Sep 2011 15:23:32 +0000 (11:23 -0400)] 
Implement enable all vs enable tracepoints vs enable syscalls

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoUpdate documentation for enable all events
Mathieu Desnoyers [Fri, 30 Sep 2011 13:58:50 +0000 (09:58 -0400)] 
Update documentation for enable all events

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoEnable all events command enable also syscalls
David Goulet [Fri, 30 Sep 2011 13:42:10 +0000 (09:42 -0400)] 
Enable all events command enable also syscalls

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoAdd liburcu function check to build system
David Goulet [Thu, 29 Sep 2011 20:24:16 +0000 (16:24 -0400)] 
Add liburcu function check to build system

rcu_thread_online and rcu_thread_offline are needed and only available
on urcu version >= 0.6.5

Reported-by: Yannick Brosseau <yannick.brosseau@polymtl.ca>
Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoWrite debug message on stderr
David Goulet [Thu, 29 Sep 2011 17:16:55 +0000 (13:16 -0400)] 
Write debug message on stderr

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix epoll not handling signal interruption
David Goulet [Thu, 29 Sep 2011 16:10:36 +0000 (12:10 -0400)] 
Fix epoll not handling signal interruption

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoRename traceable-app and change ust app call
David Goulet [Tue, 27 Sep 2011 14:06:42 +0000 (10:06 -0400)] 
Rename traceable-app and change ust app call

Rename traceable-app to ust-app.c/.h and all calls to ust_app_* to be
consistent with the code base.

Add a channel_ust_copy() to copy UST channel and event list. Mostly used
for channel copy between ust session and ust app.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoUpdate quickstart, describing syscall events
Mathieu Desnoyers [Tue, 27 Sep 2011 14:07:42 +0000 (10:07 -0400)] 
Update quickstart, describing syscall events

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoltt-kconsumerd: Fix incorrect typing
Mathieu Desnoyers [Tue, 27 Sep 2011 13:30:49 +0000 (09:30 -0400)] 
ltt-kconsumerd: Fix incorrect typing

ltt-kconsumerd.c: In function ‘on_recv_fd’:
ltt-kconsumerd.c:298: warning: passing argument 2 of
‘kernctl_get_mmap_len’ from incompatible pointer type
../include/lttng-kernel-ctl.h:47: note: expected ‘long unsigned int *’
but argument is of type ‘size_t *’

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoAdd enable channel support for UST
David Goulet [Mon, 26 Sep 2011 22:33:16 +0000 (18:33 -0400)] 
Add enable channel support for UST

Also improve data structure for UST session control to fit new model
where we will have a list of lttng domain to indentify the ust session.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix UST define in global headers
David Goulet [Mon, 26 Sep 2011 22:32:45 +0000 (18:32 -0400)] 
Fix UST define in global headers

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoUpdate UST ABI and add UST some error message
David Goulet [Mon, 26 Sep 2011 22:30:54 +0000 (18:30 -0400)] 
Update UST ABI and add UST some error message

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix channel name in all kernel event test
David Goulet [Mon, 26 Sep 2011 22:30:28 +0000 (18:30 -0400)] 
Fix channel name in all kernel event test

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoAdd default ust channel attributes
David Goulet [Fri, 23 Sep 2011 16:54:49 +0000 (12:54 -0400)] 
Add default ust channel attributes

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agokconsumerd: fix strict aliasing
Mathieu Desnoyers [Mon, 26 Sep 2011 18:59:49 +0000 (14:59 -0400)] 
kconsumerd: fix strict aliasing

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoAdd missing rculfhash.h header to make dist
Mathieu Desnoyers [Mon, 26 Sep 2011 18:52:13 +0000 (14:52 -0400)] 
Add missing rculfhash.h header to make dist

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoMerge rculfhash into lttng-tools
Mathieu Desnoyers [Mon, 26 Sep 2011 18:48:41 +0000 (14:48 -0400)] 
Merge rculfhash into lttng-tools

rculfhash is ready enough for what is needed by lttng-tools. Merge code
as from commit 4dd4a47355d9d870d482248786b4adf8b1317bfe of
urcu/ht-shrink branch.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoImprove debugging and add PERROR call
David Goulet [Thu, 22 Sep 2011 22:17:19 +0000 (18:17 -0400)] 
Improve debugging and add PERROR call

Add three level of verbosity with DBG, DBG2 and DBG3.

Each perror() call from now on should use PERROR().

Migration to these new debug facilities will come later on and
progressively.

To increase debug level use -v, -vv and -vvv respectively.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoMissing negative sign to interpret error code
David Goulet [Thu, 22 Sep 2011 21:14:58 +0000 (17:14 -0400)] 
Missing negative sign to interpret error code

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix enable all syscalls
David Goulet [Thu, 22 Sep 2011 20:22:02 +0000 (16:22 -0400)] 
Fix enable all syscalls

The -a -k --syscall was enabling all syscalls and all tracepoints.

Fix this issues by adding the enabel_kernel_event_syscalls.

At this commit, there is a problem with the list event that does not
list correctly all syscalls.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoReplace sys_enter/exit from tests
David Goulet [Mon, 19 Sep 2011 20:49:41 +0000 (16:49 -0400)] 
Replace sys_enter/exit from tests

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix wrong logical operator
David Goulet [Mon, 19 Sep 2011 20:44:35 +0000 (16:44 -0400)] 
Fix wrong logical operator

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix uninitialized pointer
David Goulet [Mon, 19 Sep 2011 20:23:47 +0000 (16:23 -0400)] 
Fix uninitialized pointer

Causing the free() to fail on error.

Reported-by: Yannick Brosseau <yannick.brosseau@polymtl.ca>
Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoRename syscalls -> syscall in UI/API/ABI
Mathieu Desnoyers [Mon, 19 Sep 2011 14:12:08 +0000 (10:12 -0400)] 
Rename syscalls -> syscall in UI/API/ABI

For consistency with other tracing modes.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoAdd support for --syscalls
Mathieu Desnoyers [Fri, 16 Sep 2011 23:33:28 +0000 (19:33 -0400)] 
Add support for --syscalls

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoUpdate version to 2.0-pre13 v2.0-pre13
David Goulet [Fri, 16 Sep 2011 16:06:08 +0000 (12:06 -0400)] 
Update version to 2.0-pre13

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoAdd missing file to extra dist
David Goulet [Fri, 16 Sep 2011 16:01:01 +0000 (12:01 -0400)] 
Add missing file to extra dist

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoAdd TODO file
David Goulet [Fri, 16 Sep 2011 15:32:50 +0000 (11:32 -0400)] 
Add TODO file

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoMajor code cleanup
David Goulet [Thu, 15 Sep 2011 23:24:23 +0000 (19:24 -0400)] 
Major code cleanup

Cleanup main.c of the session daemon. The main switch case to handle
client command is made more modular and clearer. Each command has a
function associated with it beginning by 'cmd_*'. Also adds channel.c/.h
and event.c/.h containing helper function to execute client commands.

The reste of the modified code is simple cleanup.

Some minor bugs were fixed also by this cleanup. All and for all, this
makes easier to maintain the session daeamon code base.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoMinor fixes to header files
David Goulet [Thu, 15 Sep 2011 23:23:52 +0000 (19:23 -0400)] 
Minor fixes to header files

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoAdd poll/epoll compat layer for session daemon
David Goulet [Fri, 9 Sep 2011 18:43:13 +0000 (14:43 -0400)] 
Add poll/epoll compat layer for session daemon

This is a big modification of the session daemon code base. Every thread
is monitoring file descriptors using a poll set. This commit adds a
compat wrapper for the use of poll(2) or epoll(7) determined at compile
time on if the epoll API is available. Since epoll(7) is Linux specific,
the poll(2) fallback is necessary for portability.

By default, epoll(7) will be used having the --enable-epoll default to yes.
To use the poll(2) API, simply run with --disable-epoll when running
configure.

With this implementation, some fixes are also introduced.
  * Two missing notification to the consumer when the default channel is
created.
  * Memory and socket file descriptor leak when connect()/close() is
    done by the daemon alive check of the lttngctl API.
  * Root check (UID=0) on cleanup() before removing LTTNG_RUNDIR.
  * Comments here and there.

At this commit, tests were made on the CPU hot plug feature, multiple
UST registration at the same time and git tree tests. All passed for
poll(2) and epoll(7) implementation.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoAdd epoll check to autoconf and Makefile
David Goulet [Mon, 12 Sep 2011 14:53:26 +0000 (10:53 -0400)] 
Add epoll check to autoconf and Makefile

Adds two m4 macros to check for epoll(7) API and change the gitignore to
fit thoses changes.

Also change the Makefile to add COMPAT_EPOLL value in order to select
the right source file at compile time.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix enable-channel that did not handle correctly lttng domains
David Goulet [Tue, 13 Sep 2011 16:58:33 +0000 (12:58 -0400)] 
Fix enable-channel that did not handle correctly lttng domains

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoltt-sessiond: fix 2 double-unlock
Mathieu Desnoyers [Sat, 10 Sep 2011 19:17:56 +0000 (12:17 -0700)] 
ltt-sessiond: fix 2 double-unlock

After a list iteration, the pointer is not null, so the unlock would
trigger.
The second path modified does 2 unlock when taking the if () branch.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
13 years agoCleanup poll handling events
David Goulet [Thu, 8 Sep 2011 21:23:22 +0000 (17:23 -0400)] 
Cleanup poll handling events

Make the accept in the thread client only if we have a POLLIN request.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoImprove poll management of registration thread
David Goulet [Thu, 8 Sep 2011 19:15:35 +0000 (15:15 -0400)] 
Improve poll management of registration thread

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoIncrease listen() connexion limit
David Goulet [Thu, 8 Sep 2011 16:27:19 +0000 (12:27 -0400)] 
Increase listen() connexion limit

To handle a large number of UST registration at the same time, this
limit needs to be raised.

See /proc/sys/net/core/somaxconn for the maximum possible connexions.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix missing events for all threads pollfd struct
David Goulet [Thu, 8 Sep 2011 14:35:11 +0000 (10:35 -0400)] 
Fix missing events for all threads pollfd struct

The thread_quit_pipe event field of the pollfd was not set nor zeroed.
At high rate UST registration and modifying the pollfd set frequently,
the session daemon hang on the poll() call even if the thread_quit_pipe
was closed and explain by poll() "undefined behavior".

Also add a MSG() and ERR() statement after some errors.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoAdd a safety check to nbfd of apps cmd pollfd
David Goulet [Wed, 7 Sep 2011 19:50:08 +0000 (15:50 -0400)] 
Add a safety check to nbfd of apps cmd pollfd

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoAdd benchmark directory to git ignore
David Goulet [Wed, 7 Sep 2011 16:26:13 +0000 (12:26 -0400)] 
Add benchmark directory to git ignore

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix pollfd update bug in thread_manage apps
David Goulet [Tue, 6 Sep 2011 21:10:42 +0000 (17:10 -0400)] 
Fix pollfd update bug in thread_manage apps

When updating the pollfd array, the wrong pollfd size was used so at
multiple UST registration at the same time the socket was not added to
the poll list.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoRemove fork() for the wait shm creation process
David Goulet [Thu, 1 Sep 2011 20:46:39 +0000 (16:46 -0400)] 
Remove fork() for the wait shm creation process

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoAdd the notify UST application scheme
David Goulet [Thu, 1 Sep 2011 17:58:40 +0000 (13:58 -0400)] 
Add the notify UST application scheme

This commit introduce the use of a wait shm futex that is used to notify
applications that the session daemon is running and ready to accept
registrations. See shm.c/.h for those calls and the new
futex_wait_update() in futex.c. This scheme is also found in lttng-ust
2.0 and mainly taken from there.

Also, enable-channel of the lttng client was modified to accept UST PID
domain. It's the first phase for testing UST features of the
liblttngctl.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoUpdate and fix UST register and session creation
David Goulet [Sun, 28 Aug 2011 21:29:40 +0000 (17:29 -0400)] 
Update and fix UST register and session creation

The handling of the apps command pollfd, in the registration thread and
the update function, was incorrect. The update process, when
unregistring, was overwritting connected socket thus not being able to
monitor them with poll().

This is the first step forward to fully support LTTng UST tracer.

Add create UST session to ustctl and the session daemon. At this
commit, this feature is not used yet.

Move the register done command to ustctl. Adds the ustcomm_send_command
which returns the reply structure from the UST application.

The ustctl and ustcomm are cleaned up.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoMerge branch 'master' of git://git.lttng.org/lttng-tools
David Goulet [Fri, 26 Aug 2011 20:20:03 +0000 (16:20 -0400)] 
Merge branch 'master' of git://git.lttng.org/lttng-tools

13 years agoBetter debug and fix an error handling
David Goulet [Fri, 26 Aug 2011 18:42:07 +0000 (14:42 -0400)] 
Better debug and fix an error handling

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoMerge branch 'master' of ssh://git.lttng.org/home/git/lttng-tools
Mathieu Desnoyers [Fri, 26 Aug 2011 17:52:32 +0000 (13:52 -0400)] 
Merge branch 'master' of ssh://git.lttng.org/home/git/lttng-tools

13 years agoFix missing headers
David Goulet [Fri, 26 Aug 2011 17:15:37 +0000 (13:15 -0400)] 
Fix missing headers

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
13 years agoFix memory leaks and modify kernel session creation
David Goulet [Fri, 26 Aug 2011 16:41:56 +0000 (12:41 -0400)] 
Fix memory leaks and modify kernel session creation

Set the path of the kernel session in trace-kernel.c in order to
centralize that setup at only one place.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
This page took 0.048286 seconds and 4 git commands to generate.