lttng-tools.git
11 years agoDocument metadata_socket_lock nesting
Mathieu Desnoyers [Tue, 16 Jul 2013 01:19:19 +0000 (21:19 -0400)] 
Document metadata_socket_lock nesting

Reviewed-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agolttng_ustconsumer_recv_metadata does not need all those locks
Mathieu Desnoyers [Tue, 16 Jul 2013 01:17:06 +0000 (21:17 -0400)] 
lttng_ustconsumer_recv_metadata does not need all those locks

Since lttng_ustconsumer_recv_metadata() does not push data into the
stream anymore, it only needs the metadata cache lock.

Reviewed-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agodocument metadata_switch_timer() deadlock
Mathieu Desnoyers [Tue, 16 Jul 2013 01:15:05 +0000 (21:15 -0400)] 
document metadata_switch_timer() deadlock

Reviewed-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoFix: add missing metadata socket lock
Mathieu Desnoyers [Tue, 16 Jul 2013 01:06:07 +0000 (21:06 -0400)] 
Fix: add missing metadata socket lock

Protect metadata socket, which can be accessed concurrently by the
metadata timer and by snapshot.

Reviewed-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agodocument metadata_switch_timer() locking constraints
Mathieu Desnoyers [Tue, 16 Jul 2013 00:56:51 +0000 (20:56 -0400)] 
document metadata_switch_timer() locking constraints

Reviewed-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoconsumer: remove timeout for UST metadata
Mathieu Desnoyers [Tue, 16 Jul 2013 00:53:04 +0000 (20:53 -0400)] 
consumer: remove timeout for UST metadata

Remove time out for UST metadata by generating metadata packets whenever
they are about to be read from the ring buffer rather than filling them
when the metadata cache is updated, which requires a time out, and
therefore may fail.

Reviewed-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoIntroduce pipe for UST metadata cache and stream
Mathieu Desnoyers [Tue, 16 Jul 2013 00:22:48 +0000 (20:22 -0400)] 
Introduce pipe for UST metadata cache and stream

Reviewed-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoconsumer: replace DBG2() instances by DBG()
Mathieu Desnoyers [Tue, 16 Jul 2013 00:18:13 +0000 (20:18 -0400)] 
consumer: replace DBG2() instances by DBG()

The consumer daemon does not currently support DBG2() printout.

Reviewed-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoIntroduce utils_create_pipe_cloexec_nonblock()
Mathieu Desnoyers [Tue, 16 Jul 2013 00:16:07 +0000 (20:16 -0400)] 
Introduce utils_create_pipe_cloexec_nonblock()

Reviewed-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoust consumer: data_pending check is endpoint active
Mathieu Desnoyers [Tue, 16 Jul 2013 00:14:07 +0000 (20:14 -0400)] 
ust consumer: data_pending check is endpoint active

Reviewed-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoFix: kernel consumer: data_pending check if endpoint active
Mathieu Desnoyers [Tue, 16 Jul 2013 00:12:30 +0000 (20:12 -0400)] 
Fix: kernel consumer: data_pending check if endpoint active

Reviewed-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoconsumer: explicitly set endpoint status to active at init
Mathieu Desnoyers [Tue, 16 Jul 2013 00:07:36 +0000 (20:07 -0400)] 
consumer: explicitly set endpoint status to active at init

Ensures we explicitly set active at init for channels that don't
interact with a relayd.

Reviewed-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agodocument consumer_metadata_cache_flushed use of consumer_data.lock
Mathieu Desnoyers [Tue, 16 Jul 2013 00:03:15 +0000 (20:03 -0400)] 
document consumer_metadata_cache_flushed use of consumer_data.lock

Reviewed-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoconsumer: introduce channel lock
Mathieu Desnoyers [Mon, 15 Jul 2013 23:45:24 +0000 (19:45 -0400)] 
consumer: introduce channel lock

Reviewed-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoMerge branch 'master' of git://git.lttng.org/lttng-tools
David Goulet [Sun, 14 Jul 2013 19:29:41 +0000 (15:29 -0400)] 
Merge branch 'master' of git://git.lttng.org/lttng-tools

11 years agoFix: update lttng snapshot help output
David Goulet [Sun, 14 Jul 2013 19:27:45 +0000 (15:27 -0400)] 
Fix: update lttng snapshot help output

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoMan: fix part of snapshot documentation
David Goulet [Sun, 14 Jul 2013 19:24:15 +0000 (15:24 -0400)] 
Man: fix part of snapshot documentation

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: set tracefile size test with PID buffers
David Goulet [Fri, 12 Jul 2013 21:16:41 +0000 (17:16 -0400)] 
Fix: set tracefile size test with PID buffers

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: Babelstats fail to parse output with no process name or pid
Christian Babeux [Fri, 12 Jul 2013 20:39:02 +0000 (16:39 -0400)] 
Fix: Babelstats fail to parse output with no process name or pid

The process name and pid are optionnal components in the babeltrace
text output.

Signed-off-by: Christian Babeux <christian.babeux@efficios.com>
11 years agoMissing NULL pointer init in tap.c
Mathieu Desnoyers [Fri, 12 Jul 2013 15:41:08 +0000 (11:41 -0400)] 
Missing NULL pointer init in tap.c

Could lead to free() of unitialized pointer.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: Unchecked asprintf/vasprintf return values
Jérémie Galarneau [Fri, 12 Jul 2013 16:38:56 +0000 (12:38 -0400)] 
Fix: Unchecked asprintf/vasprintf return values

strp may be modified even in case of failure according to the
ASPRINTF(3) man page.

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoAdd snapshots test to fast regression
David Goulet [Fri, 12 Jul 2013 17:57:55 +0000 (13:57 -0400)] 
Add snapshots test to fast regression

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: kernel data unit test
David Goulet [Fri, 12 Jul 2013 17:47:12 +0000 (13:47 -0400)] 
Fix: kernel data unit test

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: snapshot returned valid LTTNG_ERR code
David Goulet [Fri, 12 Jul 2013 15:01:09 +0000 (11:01 -0400)] 
Fix: snapshot returned valid LTTNG_ERR code

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoAdd the number of snapshot taken to the output path
David Goulet [Fri, 12 Jul 2013 14:53:59 +0000 (10:53 -0400)] 
Add the number of snapshot taken to the output path

This number of snapshots is per session.

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: RCU read side lock unbalanced
David Goulet [Fri, 12 Jul 2013 14:23:28 +0000 (10:23 -0400)] 
Fix: RCU read side lock unbalanced

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: zeroed snapshot output at init
David Goulet [Fri, 12 Jul 2013 14:16:26 +0000 (10:16 -0400)] 
Fix: zeroed snapshot output at init

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoSupport del-output with an output name
David Goulet [Thu, 11 Jul 2013 20:57:53 +0000 (16:57 -0400)] 
Support del-output with an output name

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoUpdate man page with snapshot command
David Goulet [Thu, 11 Jul 2013 20:46:14 +0000 (16:46 -0400)] 
Update man page with snapshot command

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoNew UST default buffers is now per UID
David Goulet [Thu, 11 Jul 2013 19:30:49 +0000 (15:30 -0400)] 
New UST default buffers is now per UID

Acked-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoBump UST ABI major version for 2.3 release
David Goulet [Thu, 11 Jul 2013 19:20:21 +0000 (15:20 -0400)] 
Bump UST ABI major version for 2.3 release

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoAdd snapshot mode to lttng list session
David Goulet [Thu, 11 Jul 2013 19:13:23 +0000 (15:13 -0400)] 
Add snapshot mode to lttng list session

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: support temporary snapshot max size and name
David Goulet [Thu, 11 Jul 2013 18:51:35 +0000 (14:51 -0400)] 
Fix: support temporary snapshot max size and name

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoSupport snapshot max-size limitation
Julien Desfossez [Wed, 10 Jul 2013 02:42:13 +0000 (22:42 -0400)] 
Support snapshot max-size limitation

This features allows the user to specify a maximum size for the snapshot
collected. The maximum stream size is split equally by the total number
of streams active at the time of the snapshot record across all the
domains. The metadata is not limited and its size is not taken into
account in the maximum stream size computation.

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoTests: per-UID UST snapshot
Julien Desfossez [Mon, 8 Jul 2013 15:46:37 +0000 (11:46 -0400)] 
Tests: per-UID UST snapshot

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: snapshot support for UST and kernel in same session
David Goulet [Tue, 9 Jul 2013 14:50:25 +0000 (10:50 -0400)] 
Fix: snapshot support for UST and kernel in same session

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoImplement lttng create --snapshot command
David Goulet [Mon, 8 Jul 2013 19:19:28 +0000 (15:19 -0400)] 
Implement lttng create --snapshot command

This is to ease the life of the user creating a session in snapshot
mode using the new API call.

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoAdd create session snapshot API in lttng-sessiond
David Goulet [Mon, 8 Jul 2013 19:19:12 +0000 (15:19 -0400)] 
Add create session snapshot API in lttng-sessiond

This commit adds the "lttng_create_session_snapshot()" call to the API
which will create on the session daemon a session in snapshot mode
meaning that all channel will be in overwrite mode and mmap output.

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoAdd snapshot output init call that uses URIs
David Goulet [Mon, 8 Jul 2013 16:15:19 +0000 (12:15 -0400)] 
Add snapshot output init call that uses URIs

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: consumer err_sock cloexec
Mathieu Desnoyers [Wed, 10 Jul 2013 19:39:54 +0000 (15:39 -0400)] 
Fix: consumer err_sock cloexec

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoCallsite: add "ip" context
Woegerer, Paul [Wed, 10 Jul 2013 09:06:45 +0000 (11:06 +0200)] 
Callsite: add "ip" context

Support ip context in lttng add-context -t.

Acked-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: possible consumer sockets double close on cleanup
David Goulet [Tue, 9 Jul 2013 16:28:41 +0000 (12:28 -0400)] 
Fix: possible consumer sockets double close on cleanup

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoAutomatically load kvm-x86 and kvm-x86-mmu probes.
Mohamad Gebai [Mon, 8 Jul 2013 19:26:27 +0000 (15:26 -0400)] 
Automatically load kvm-x86 and kvm-x86-mmu probes.

Acked-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: Mohamad Gebai <mohamad.gebai@polymtl.ca>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: consumer: use uint64_t for all sessiond_id
Mathieu Desnoyers [Fri, 28 Jun 2013 16:18:35 +0000 (12:18 -0400)] 
Fix: consumer: use uint64_t for all sessiond_id

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoFix: add gpl and lgpl files to tarball
David Goulet [Mon, 8 Jul 2013 15:25:52 +0000 (11:25 -0400)] 
Fix: add gpl and lgpl files to tarball

Fixes #581

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: don't install libtap system wide
David Goulet [Mon, 8 Jul 2013 15:23:44 +0000 (11:23 -0400)] 
Fix: don't install libtap system wide

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: close consumer sockets in sessiond cleanup
David Goulet [Fri, 5 Jul 2013 20:03:50 +0000 (16:03 -0400)] 
Fix: close consumer sockets in sessiond cleanup

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: set globally visible flag to kernel stream
David Goulet [Fri, 5 Jul 2013 19:39:21 +0000 (15:39 -0400)] 
Fix: set globally visible flag to kernel stream

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: lttng: memory leak in snapshot record command
Mathieu Desnoyers [Thu, 4 Jul 2013 22:14:33 +0000 (18:14 -0400)] 
Fix: lttng: memory leak in snapshot record command

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoFix: kernel-consumer: double-close
Mathieu Desnoyers [Thu, 4 Jul 2013 22:06:44 +0000 (18:06 -0400)] 
Fix: kernel-consumer: double-close

check for -1 out_fd

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoFix: consumer: incorrect size zmalloc
Mathieu Desnoyers [Thu, 4 Jul 2013 21:53:50 +0000 (17:53 -0400)] 
Fix: consumer: incorrect size zmalloc

Also, check return value.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoFix: don't try to send stream to relayd if not in streaming
David Goulet [Fri, 5 Jul 2013 18:41:58 +0000 (14:41 -0400)] 
Fix: don't try to send stream to relayd if not in streaming

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: relayd refcount updates for stream
David Goulet [Thu, 4 Jul 2013 20:25:27 +0000 (16:25 -0400)] 
Fix: relayd refcount updates for stream

Increment refcount only when the stream was successfully sent to the
relayd and set the new stream's flag "sent_to_relayd" which is used
before the refcount update when closing the relayd. A stream that was
unable to be sent, the close relayd code path does not decrement the
refcount anymore.

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: don't send error to sessiond on orderly shutdown
David Goulet [Thu, 4 Jul 2013 20:31:27 +0000 (16:31 -0400)] 
Fix: don't send error to sessiond on orderly shutdown

The send always fail if the sessiond has closed the socket so no point
of trying to send an error.

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: bad pathname used when sending kernel stream to relayd
David Goulet [Thu, 4 Jul 2013 20:14:46 +0000 (16:14 -0400)] 
Fix: bad pathname used when sending kernel stream to relayd

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: add globally visible flag in stream
David Goulet [Thu, 4 Jul 2013 19:35:21 +0000 (15:35 -0400)] 
Fix: add globally visible flag in stream

This flag is added so we can destroy it correctly with the
consumer-stream API.

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: destroy metadata stream on setup metadata error path
David Goulet [Thu, 4 Jul 2013 18:57:06 +0000 (14:57 -0400)] 
Fix: destroy metadata stream on setup metadata error path

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: send kernel stream to relayd only if needed
David Goulet [Thu, 4 Jul 2013 18:07:04 +0000 (14:07 -0400)] 
Fix: send kernel stream to relayd only if needed

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: destroy streams for kernel snapshot sessions as well
Julien Desfossez [Fri, 5 Jul 2013 17:37:01 +0000 (13:37 -0400)] 
Fix: destroy streams for kernel snapshot sessions as well

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: close and destroy metadata stream after a kernel snapshot
Julien Desfossez [Fri, 5 Jul 2013 17:37:00 +0000 (13:37 -0400)] 
Fix: close and destroy metadata stream after a kernel snapshot

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: print errno message on connect() error
David Goulet [Fri, 5 Jul 2013 16:01:10 +0000 (12:01 -0400)] 
Fix: print errno message on connect() error

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: possible double-close on stream out_fd
Mathieu Desnoyers [Thu, 4 Jul 2013 22:37:35 +0000 (18:37 -0400)] 
Fix: possible double-close on stream out_fd

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: session ID signess to uin64_t in sessiond
David Goulet [Thu, 4 Jul 2013 19:54:38 +0000 (15:54 -0400)] 
Fix: session ID signess to uin64_t in sessiond

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoTests: fix validation trace path in kernel snapshot
Julien Desfossez [Thu, 4 Jul 2013 15:56:47 +0000 (11:56 -0400)] 
Tests: fix validation trace path in kernel snapshot

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoTests: Add UST snapshot local and streaming
Julien Desfossez [Thu, 4 Jul 2013 02:29:41 +0000 (22:29 -0400)] 
Tests: Add UST snapshot local and streaming

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoAdd UST snapshot support
Julien Desfossez [Tue, 18 Jun 2013 19:25:53 +0000 (15:25 -0400)] 
Add UST snapshot support

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: consumer_add_relayd_socket() report errors to sessiond
Mathieu Desnoyers [Tue, 2 Jul 2013 19:14:02 +0000 (15:14 -0400)] 
Fix: consumer_add_relayd_socket() report errors to sessiond

Wait until the side-effect to relayd is actually performed before
replying that "all is fine" to sessiond.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoFix: add missing enum lttcomm_return_code entries
Mathieu Desnoyers [Tue, 2 Jul 2013 19:12:42 +0000 (15:12 -0400)] 
Fix: add missing enum lttcomm_return_code entries

Those should not appear as unknown.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoFix: UST per-UID channels persist across application teardown
Mathieu Desnoyers [Tue, 2 Jul 2013 20:20:33 +0000 (16:20 -0400)] 
Fix: UST per-UID channels persist across application teardown

Fixes #582

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoFix: kernel snapshot metadata handling and error paths
David Goulet [Fri, 28 Jun 2013 20:13:53 +0000 (16:13 -0400)] 
Fix: kernel snapshot metadata handling and error paths

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: coding style and debug statement
David Goulet [Fri, 28 Jun 2013 19:44:14 +0000 (15:44 -0400)] 
Fix: coding style and debug statement

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: put subbuffer back in kernel snapshot error path
David Goulet [Fri, 28 Jun 2013 19:30:25 +0000 (15:30 -0400)] 
Fix: put subbuffer back in kernel snapshot error path

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: overflow in uri_to_str_url
Julien Desfossez [Wed, 12 Jun 2013 18:59:05 +0000 (14:59 -0400)] 
Fix: overflow in uri_to_str_url

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: detect the correct version of LTTng-UST
David Goulet [Fri, 28 Jun 2013 18:35:09 +0000 (14:35 -0400)] 
Fix: detect the correct version of LTTng-UST

Fixes #580

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: sessiond: use uint64_t for all session ids
Mathieu Desnoyers [Fri, 28 Jun 2013 16:19:51 +0000 (12:19 -0400)] 
Fix: sessiond: use uint64_t for all session ids

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoTests: add kernel snapshot streaming to root regression
David Goulet [Fri, 28 Jun 2013 18:23:23 +0000 (14:23 -0400)] 
Tests: add kernel snapshot streaming to root regression

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoTests: remove debug output from test
David Goulet [Fri, 28 Jun 2013 18:22:00 +0000 (14:22 -0400)] 
Tests: remove debug output from test

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoTests: Add kernel snapshot streaming
David Goulet [Fri, 28 Jun 2013 18:17:34 +0000 (14:17 -0400)] 
Tests: Add kernel snapshot streaming

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: use snapshot consumer output for kernel
David Goulet [Fri, 28 Jun 2013 18:16:15 +0000 (14:16 -0400)] 
Fix: use snapshot consumer output for kernel

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: periodical flush check trace before stop
David Goulet [Fri, 28 Jun 2013 16:07:17 +0000 (12:07 -0400)] 
Fix: periodical flush check trace before stop

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: consumer: 64-bit index for relayd rather than 32-bit (v2)
Mathieu Desnoyers [Fri, 28 Jun 2013 15:26:04 +0000 (11:26 -0400)] 
Fix: consumer: 64-bit index for relayd rather than 32-bit (v2)

Relayd "unique" ids wrap every 32-bit, and in some cases, negative
values are considered as error.

Change this to make the error value specifically -1ULL, use a direct
comparison (since we use an unsigned 64-bit integer, comparison with 0
becomes incorrect).

Since we now use a 64-bit ID, it is assumed to _never_ wrap-around
(remember, value -1ULL is an _error_). Therefore,
consumer_add_relayd_socket() can become much more strict than it was:
instead of accepting re-use of net_seq_idx, we can now assert that upon
LTTNG_STREAM_CONTROL socket, we have indeed allocated a relayd object,
and upon LTTNG_STREAM_DATA, we have found a relayd object.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix UST channel/stream output assignation
Julien Desfossez [Tue, 18 Jun 2013 22:07:21 +0000 (18:07 -0400)] 
Fix UST channel/stream output assignation

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: send per-pid session id in channel creation
Julien Desfossez [Thu, 27 Jun 2013 21:57:06 +0000 (17:57 -0400)] 
Fix: send per-pid session id in channel creation

The registry indexing for per-pid sessions is done with a per-pid
session id. So for per-pid buffers, we need to send the per-pid session
id as well as the global session id to the consumer in order to give it
enough information if it needs to request metadata later.

This patch adds the session_id_per_pid to the channel creation message
and to the consumer. When the sessiond receives a metadata_request,
depending on the buffer type (per-pid or per-uid), it selects the right
id to do the registry lookup.

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
Conflicts:
src/bin/lttng-sessiond/consumer.c
src/bin/lttng-sessiond/consumer.h
src/bin/lttng-sessiond/ust-consumer.c
src/common/consumer.c
src/common/consumer.h
src/common/kernel-consumer/kernel-consumer.c
src/common/sessiond-comm/sessiond-comm.h
src/common/ust-consumer/ust-consumer.c

11 years agoFix: consumer double-close on error
Mathieu Desnoyers [Thu, 27 Jun 2013 21:32:37 +0000 (17:32 -0400)] 
Fix: consumer double-close on error

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
11 years agoUpdate URCU detection to correctly check for a 0.7 version
Yannick Brosseau [Thu, 27 Jun 2013 21:03:29 +0000 (17:03 -0400)] 
Update URCU detection to correctly check for a 0.7 version

Signed-off-by: Yannick Brosseau <yannick.brosseau@gmail.com>
11 years agoFix: snapshot path
David Goulet [Fri, 28 Jun 2013 14:54:44 +0000 (10:54 -0400)] 
Fix: snapshot path

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoAdd utils function to format current time as a string
David Goulet [Fri, 28 Jun 2013 14:39:30 +0000 (10:39 -0400)] 
Add utils function to format current time as a string

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: set hidden attribute to utils_* calls
David Goulet [Fri, 28 Jun 2013 14:30:18 +0000 (10:30 -0400)] 
Fix: set hidden attribute to utils_* calls

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: consumer handling of metadata for relayd
Julien Desfossez [Fri, 14 Jun 2013 18:55:23 +0000 (14:55 -0400)] 
Fix: consumer handling of metadata for relayd

This patch sends the metadata stream to relayd and close it once done.

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoAdd kernel snapshot support
Julien Desfossez [Mon, 6 May 2013 19:57:24 +0000 (15:57 -0400)] 
Add kernel snapshot support

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoSupport flight recorder mode for a session
David Goulet [Wed, 1 May 2013 19:18:50 +0000 (15:18 -0400)] 
Support flight recorder mode for a session

This adds the --no-output option to the lttng UI which will flag a
session to not set an output destination meaning that tracing will be in
overwrite mode in the buffers.

This is also the only way for now to snapshot a session, it's to put it
in no output mode.

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoImplement snapshot commands in lttng-sessiond
David Goulet [Tue, 16 Apr 2013 17:44:09 +0000 (13:44 -0400)] 
Implement snapshot commands in lttng-sessiond

Basic functionalities are implemented in the session daemon so handle
snapshots and send the command to the consumer but note that at this
commit the consumer does not handle snapshot.

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoAdd snapshot command to lttng UI
David Goulet [Thu, 18 Apr 2013 19:41:16 +0000 (15:41 -0400)] 
Add snapshot command to lttng UI

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoInitial import of the snapshot ABI/API in lttng-ctl
David Goulet [Fri, 12 Apr 2013 19:47:13 +0000 (15:47 -0400)] 
Initial import of the snapshot ABI/API in lttng-ctl

Nothing on the session daemon side as been added except the command
values added to the process client command subsystem.

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoUse the consumer stream API in consumer_del_stream()
David Goulet [Mon, 27 May 2013 20:07:34 +0000 (16:07 -0400)] 
Use the consumer stream API in consumer_del_stream()

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoAdd consumer-stream.c/.h in libconsumer
David Goulet [Mon, 27 May 2013 19:17:19 +0000 (15:17 -0400)] 
Add consumer-stream.c/.h in libconsumer

These new files contain relevant calls to a lttng_consumer_stream object
and for now breaks down consumer_del_stream() from consumer.c that got
out of hand in terms of size and can not be used if the stream was not
previously successfully sent to a thread making difficult the deletion
of a stream in some error path or in no monitor mode (future mode for
streams with snapshots) outside of this use case.

It helps also modularize the consumer object and makes the code easier
to maintain and to better evolve.

At this commit, no components use this yet. Future work will
incrementaly move every consumer stream specific actions to that
seperate API.

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoMove multiple URLs parsing fct from lttng-ctl to uri.c
David Goulet [Wed, 24 Apr 2013 17:29:19 +0000 (13:29 -0400)] 
Move multiple URLs parsing fct from lttng-ctl to uri.c

Just move some functions around so we can use the multiple URLs parsing
to URIs function outside of liblttng-ctl.

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoAdd a lttng-ctl header to facilitate code separation
David Goulet [Mon, 15 Apr 2013 16:37:06 +0000 (12:37 -0400)] 
Add a lttng-ctl header to facilitate code separation

In order for lttng-ctl.c not to grow into an uncontrollable monster of
code, add this helper header and remove static declaration of some
functions to be exported out of the C file and set them hidden.

Future work on snapshot and trigger will use it in order to better split
the code into different files.

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoUpdate version to v2.2.0 v2.2.0
David Goulet [Tue, 25 Jun 2013 20:40:13 +0000 (16:40 -0400)] 
Update version to v2.2.0

Signed-off-by: David Goulet <dgoulet@efficios.com>
11 years agoFix: build path for unit test to build out of tree
David Goulet [Tue, 25 Jun 2013 21:25:04 +0000 (17:25 -0400)] 
Fix: build path for unit test to build out of tree

Signed-off-by: David Goulet <dgoulet@efficios.com>
This page took 0.0417 seconds and 4 git commands to generate.