Fix multiple enable events

The session name was not set after the first event enabled.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Update version to 2.0-pre5

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Print trace output patch on lttng create

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix adding multiple kernel context at once

The session name was not set when multiple add-context was asked.

Also fix error message and session daemon return error when one add
kernel context fails.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix auto session path duplication

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix bad strncmp length

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Update version to 2.0-pre4

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix channel creation when enabling an event

If the channel is not found when enabling an event, that channel will be
created.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add event count in channel structure

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix uninitialized kprobe attributes

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix bad address conversion

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Trace path creation made on the client side

The traces output path is set on the client side with the date and time
if no output path is specified.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix safe list iteration

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add reset of the session message

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix comments in main.c

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Increase possible verbose level for session daemon

Using -vv, the kernel consumer goes into verbose mode.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix set session command

The command was appending a new session name and not replacing it.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add kernel trace data structure test

Test all kernel data structure found in ltt-sessiond/trace.c/.h.

Also adds a utils.h having a pretty print for OK and FAIL. The session
test is modified to use those prints.

Better runall.sh which iterates over an array containing test binaries
name and terminates when one test fails.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix missing pointer initialisation

Channel and event context pointer was not set to NULL at creation.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add tracing session test

Tests the tracing session functions found in ltt-sessiond/session.h/.c

Creation, destruction and iteration is tested.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Update version to 2.0-pre3

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix perf hw cache counter config values

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Update version to 2.0-pre2

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add missing fprintf format

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

UI print cleanup

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Use "probe" rather than "kprobe" for UI

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Update quickstart for new perf UI

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Update handling of perf counters in add-context

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Remove assert from kconsumerd (multi-fd handling is ok)

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Cleanup header

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Add string options for add-context

List of strings can be used for context type, perf type and perf id.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix sending fd through sendmsg/recvmsg ancillary data

ltt-kconsumerd was only reading a "char" rather than an integer. It
worked fine on little endian for values below 256, and failed on big
endian machines (always reading "0").

Also cleanup msg initialization.
Also perform final msg.msg_controllen = cmsg->cmsg_len; after memcpy, as
done in cmsg(3) example (extra safety).

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Improve add-context command

Better multiple context type handling using a linked list at args
parsing.

Change some error message.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Update and add to the quickstart guide

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Merge branch 'master' of git://git.lttng.org/lttng-tools

Rename binary installation path define

INSTALL_PATH --> INSTALL_BIN_PATH

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix double-unlock in kconsumerd_find_session_fd()

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix error handling

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Remove format strings warnings when compiling on 32 bits

On 32 bits, %lu expects a 32 bits integer, so changed some %lu for PRIu64 when printing a 64 bits integer.

Signed-off-by: Simon Marchi <simon.marchi@polymtl.ca>

Use install path for exec()

Add INSTALL_PATH define for binary execution path.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix set but no use variables warnings

Shown with gcc >= 4.6.0

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Update version to 2.0-pre1

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Update authors

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Change bzip2 for dist tarball

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix session trace output message

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add usage section to README

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix README and add doc to tarball

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix build system for valid make dist

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Remove debug mode at kconsumerd exec

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix wrong spacing

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix invalid channel creation

Also add a message when all events are successfully enabled.

Reported-by: Simon Marchi <simon.marchi@ericsson.com>
Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Make the receiving thread non blocking

Avoids the risk of blocking indefinitely on the command socket when the
ltt-sessiond is not available and allow the signal handler to stop the
process when necessary.

Signed-off-by: Julien Desfossez <julien.desfossez@polymtl.ca>

Change licensing

Public API lttng.h and liblttngctl.c is licensed LGPLv2.1.

The rest of the code is licensing GPLv2 and NOT later.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Change lttng.h API char to const char

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

New public API for lttng control

Introduce the lttng domain that is used to identify the tracer.

Remove all 'kernel' from the functions name and now take the domain
struct as first parameter.

Add channel feature is removed and merged with enable-channel.

Move kernel specific data structure from lttng.h to lttng-kernel.h which
is copied from the LTTng kernel modules ABI. Genereic structures are
used to create a generic view of lttng event and attributes (agnostif of
the Kernel and user-space).

Default channel is NOT created unless it is needed.

A lot of comments has been added to lttng.h

Fix a bug in the liblttngctl. A return value was not set correctly.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Remove connect/disconnect sessiond from lttng API

Those actions are made internally when a command to the session daemon
is done so no need for them to be public.

Move code around to make things clearer and fit static declaration.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

quickstart guide review

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Makes kprobe attribute specific to kernel

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix tracing events data structure

Make lttng_kernel_output generic to user-space and kernel becoming
lttng_event_output.

Renaming enum to fit kernel ABI.

Add comments.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Remove libustctl and libustcomm

Those libs are part of the soon to be deprecated user-space tracer. The
code is removed but not the directories and Makefile.am.

lttng-tools will only support UST 2.0 which is not yet released at this
point in time. Thus, this is not useful anymore and will be readded
later on to support the new user-space tracer.

Also disable a small portion of code in the session daemon that uses
libustctl. This code however remains relevant for future UST
integration.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Remove not working test

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add quick start tracing guide

First draft of a quick start guide. For now, there is only the kernel
tracing guide since ust 2.0 is not released at this point.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add session daemon model RFC

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add short option for no sessiond feature

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add error message if event not found

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix return code to 0 on success and 1 on error

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix session list locking

The session list lock was not used correctly when a client requested the
session list. The session counter and iteration over the list is
protected by a single critical section now.

Move get_lttng_sessions to the main.c in order to remove lttng.h
dependency to session.c/.h and to make easier for the session list
locking.

Remove the get_session_count which is useless since the counter is now
in the session list structure. Access the counter by locking the list
and reading 'count'.

This adds two function to lock and unlock the session list.

Reported-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix bad cleanup of context structure

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix bad exit conditions

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Clean exit cleanup of the session daemon

Remove the pthread_cancel and introduce the *thread_quit_pipe* that is
used to tell a thread that it should exit cleanly. This pipe is added to
the pollfd set of the thread and has to be close in order to trigger the
clean exit of the threads.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add version command

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Force to specify the type with add-context

-t, --type must be used or the help shows up. Also allowed multiple -t
to enable multiple context with one command.

Fix a memory leak.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add comment for the session list lock

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix double mutex unlock

Reported-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: David Goulet <david.goulet@polymtl.ca>

lttng command line UI: fix allocation/free

Freeing non-dynamically allocated memory is never a good idea.
Also fixing 2 memory leaks.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix missing value

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add locking for all session data structure

The session daemon is single threaded when processing client command
however the CPU hotplug thread was introduced and uses a lot of session
data structure that are now protected by pthread mutex with this commit.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add CPU hotplug support

This introduce a new thread to the session daemon that poll() on all
channel's fd for an event which indicate a new CPU.

If an even is catched, new streams are created for all channels of all
tracing sessions and kernel consumers are updated.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Remove useless kernel session counter

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Remove dependency on libuuid

The session name is unique across all sessions so no need to use UUIDs
from libuuid anymore.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix locking dependency problem

Acked-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: Julien Desfossez <julien.desfossez@polymtl.ca>

kconsumerd: signals should only set flag and write into pipe (part 1)

Simplify signal reception. Left todo.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Locking fix.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix mutex unprotected list

Reported-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add a check for duplicate session fd

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Parse kprobe options with enable-event

Address or offset can be specified in base 8 (beginning with 0), base 10
and base 16 (beginning with 0x).

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Add new kernel consumer name to gitignore

Signed-off-by: David Goulet <dgoulet@ev0ke.net>

Rename kconsumerd to ltt-kconsumerd

Avoid the confusion with a kernel thread in the process list.

Signed-off-by: Julien Desfossez <julien.desfossez@polymtl.ca>
Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Create a library for the kernel consumer

Kconsumerd functionnalities has been extracted to a library to allow
external tools to implement the kernel consumer functions.
Kconsumerd is now a daemon using the library, everything works as
before for this particular case.

Signed-off-by: Julien Desfossez <julien.desfossez@polymtl.ca>
Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix segfault on free() pointer

Reported-by: Julien Desfossez <julien.desfossez@polymtl.ca>
Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add list-commands and list-options to lttng ui

Those commands are mostly for bash auto completion and does not require
heavy parsing for that matter.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add detection of debugfs and kernel module loading

The session daemon now detects the path of the debugfs mount point or
create and mount it at a default location (/mnt/debugfs).

Then, the kernel module are loaded automatically, if present, and the
kernel tracer fd is opened at the previous detected debugfs path.

The kernel module list is hardcoded for now.

NOTE: There is no module unload at the session daemon cleanup. This is
to avoid removing tracing capabilities when a trace is ongoing and
written to disk.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Remove unused header file

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Disable user-space tracer control

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix missing tracing group when UID=0

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Change configuration file and trace output dir

The configuration file containing the session name information is now at
the HOME location and named ".lttngrc".

The trace output directory is now at the HOME location and named
"lttng-traces" containing the traces of each sessions identified by the
top level directory called "session_name-date-time".

Also fix the metadata's path that was pointing to the old default path
in /tmp/lttng.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add set session command to lttng cli

This command change the session name in the config file of the config
directory (.lttng).

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Add session name option to all command

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Overwrite the config file on create session

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Don't remove the config directory on destroy

Signed-off-by: David Goulet <david.goulet@polymtl.ca>