Multiple fixes for enable/disable UST support

Makes the enable/disable command inside the main.c uses the event.c
functions. For this, the disable tracepoint functions (single and all)
for UST are added to event.c

Also fix the goto error path in event.c for disable/enable functions
where the UST event was removed from the hashtable if the event already
exist on the tracer side.

Fix the wrong hashtable being used for events lookup in ust_app.c for
enable/disable event for UST global domain.

Adds the disable event per PID function call in ust_app.c

At this commit, the "lttng disable-event -a -u" command segfault
lttng-ust at commit 5cd29ace704432c7a421f7749ef8591e30e1894c.

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

Fix wrong lttng_channel name size

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

Fix wrong sizeof on channel name copy

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

Fix enable event -u -a

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

Update consumerd search paths comment

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

Document wildcards for userspace tracer

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

Prefer using the native consumer over 32-bit consumer for kernel

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

Change lttng command line options for UST domain

lttng command line option --all is removed and replace by using -u alone
which indicates that the command must be applied to the UST global
domain.

$ lttng enable-event -u --all
is changed to
$ lttng enable-event -u

Also, the long option for -a, --all-events, is replaced by --all.

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

API change for lttng_list_domains prototype

Now takes a session name and no lttng_handle anymore.

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

API change for lttng_start/stop_tracing prototype

Now takes a session name and no lttng_handle anymore.

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

API change for lttng_destroy_session prototype

Now takes a session name and no lttng_handle anymore.

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

Merge branch 'master'

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

Add support for UST enable all tracepoints

Support to enable all tracepoints for UST global domain

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

update loglevel printout

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

Cleanup lttng enable event command

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

listing and activation of loglevel by number

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

loglevels: allow enable/disable

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

Add loglevel to event list

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

ABIs now support 256 char event names

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

Update symbol name len size limit to 256

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

Use the consumerd_bin variables for kernel consumers too

Right now the sessiond will only look in its own directory for
the consumerd when starting a kernel trace. This patch makes it
check for consumerd32_bin and consumerd64_bin too.

Signed-off-by: Alexandre Montplaisir <alexandre.montplaisir@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Rename consumerd32/64_path options to _bin

The _path suffix was confusing as to whether it would include the
filename at the end or not. In fact it should (to allow greater
flexibility in moving/renaming the consumerd executable).

Rename all configure options, env vars and substition variables to
consumerd32/64_bin instead, and expect the user to pass a complete
basename + filename to those.

Signed-off-by: Alexandre Montplaisir <alexandre.montplaisir@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix ust app tracepoint list realloc

- memcpy data _after_ reallocation (corruption fix),
- fix off-by-one in size check (corruption fix),
- multiply size by 2 each time size increase is needed (optimisation).

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

Fix bad return value on enable syscalls making it freeze

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

Update libpopt dependency to 1.13

After careful review of popt cvs tree, after cross-correlation of CHANGE
file with source code POPT_ARGFLAG_OPTIONAL updates, identified that the
updates to this part of the code end at 1.13.

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

Document dependency on libpopt >= 1.8

ref: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=285666

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

Describe autotools/libtool/automake version dependency

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

Remove m4_ifdef for AC_PROG_LIBTOOL (deprecated)

This trick does not seem to work anyway.

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

Fix --disable-lttng-ust build after UST API update

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

Update following loglevel support UST API change

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

Add missing rcu read lock

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

Cleanup ust app delete functions

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

Fix missing declaration without UST support

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

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

Support for UST context

Adds the support to add context on channel and/or events for UST global
domain.

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

Remove function:entry option to discourage its use

Removed from kernel tracer.

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

Add context exist error code

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

Fix realloc invalid next size

Reported-by: Tan Dung Le Tran <tan.dung.le.tran@ericsson.com>
Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Support older versions of autotools

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

Add a --with-consumerd-only configure option

This will only build the consumer daemon (and its liblttng-consumer
library), and skip the lttng and lttng-sessiond binaries.

This will be useful when a user wants to support both 32-bit and
64-bit applications on the same system. Only one sessiond is
needed, but we need a consumer for each architecture.

Signed-off-by: Alexandre Montplaisir <alexandre.montplaisir@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Cleanup configure.ac

Use AS_IF() and AS_ECHO() macros whenever possible.
Print the configuration options at the end of the script
(so they don't get lost in the sea of the configure output).

Signed-off-by: Alexandre Montplaisir <alexandre.montplaisir@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix ust_app_clean_list segfault

The 4th parameter of cds_lfht_for_each_entry is the member of the
structure, not a pointer to a variable. Therefore, node was never
populated.

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

Check hashtable_del return values

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

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

Fix disable/enable channel when app registers

Upon registration, a disable channel in the global domain was created
but not disabled afterwords being enabled by default on the tracer. This
commit fix that for channels and events as well.

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

Rename ust app function for better semantic

Replace _all by _glb meaning global. This family of functions (ending
with _glb) apply an action on *all* registered applications.

Also fix the setting the enabled flag on disable-event.

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

Add UST event not found error code

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

Fix enable-event for UST events

When enabling an existing event, a new event creation occured which was
simply the bad thing to do. This patch fix that by adding the
enable_ust_event call for the case.

Also fixes a segfault where any UST object could be release with the
data pointer being set to NULL.

This patch introduce also assert() in the UST app code for
create/enable/disable event. The purpose is to catch, in dev, the bad
code execution flow and fix it.

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

Add UST event exist error code

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

Init point to NULL in case of free() for 64-bit too

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

Apply autoupdate to configure.ac

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

Fix --enable-lttng-ust configure option

Specifying either --enable-lttng-ust or --disable-lttng-ust
would DISABLE the option. This fix makes it listen to either we
want to enable or disable it (it remains enabled by default).

Signed-off-by: Alexandre Montplaisir <alexandre.montplaisir@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

AC_HELP_STRING is deprecated, use AS_HELP_STRING

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

Remove unused UST channel code

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

Rename kernel-ctl.c/.h to kernel.c/.h

Just to follow the function call standard and since kernelctl exist out
of the code tree.

Also remove ust-ctl.c which is not used and not compiled at all.

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

Init pointer to NULL in case of free()

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

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

Add missing declaration when UST is disabled

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

Rename consumerd32_bindir -> consumerd32_path

Fixes build error on 32-bit arches.

Signed-off-by: Alexandre Montplaisir <alexandre.montplaisir@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Setup LD_LIBRARY_PATH if 32/64 bit consumer libdir is set

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

Implement consumer bin/lib 32/64-bit configure/env/arg support

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

Introduce --with-consumer32-bindir and --with-consumer64-bindir

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

Change ust app clean list loop

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

lttng disable_events: ensure end of string is set to \0

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

streamline channel_new_default_attr switch statement with coding style

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

channel_new_default_attr: fix usage memory leaks

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

Add single disable UST event support

Support for disabling a single UST event for a specific channel.

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

NULL ptr validation across liblttngctl

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

Do not permit NULL pointer on enable API calls

Enable channel lttng API call now forbids the use of NULL pointer for
channel attributes.

Use lttng_channel_set_default_attr() before.

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

Add disable all UST event(s) command

Note that you can only disable all events for a specific channel (like
the kernel). So, using the UST global domain (-u -a), only the event(s)
in the specified channel will be disabled for all registered
applications.

Bottom line, it disables all events in a channel.

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

Add new API call lttng_channel_set_default_attr

This call takes a lttng domain and an allocated channel attributes
pointer and set the default attributes of the channel according to the
domain type.

This commit also fix a listing problem where the wrong output type of
the channel was returned for UST.

The enable-channel command is also changed to used that new API call.

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

Fix events listing using the channel name

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

Fix missing channel name copy for auto channel creation

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

Add disable-channel support for UST

The enable channel command (for UST) was not working for already created
channel so this commit fix it by adding an enable function call when the
UST channel already exist onto the session.

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

Add UST default channel creation support

When enabling event without a channel specified, a default channel
(channel0) will be created.

Also fix the enabled flag for channels when listing them.

Minor fix of code and comments.

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

Add debug statement to the teardown session

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

Cleanup and streamline kernel and ust teardown session

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

Fix null pointer dereference when destroying a UST session with no channels

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

Rename "comm" context into "procname"

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

Fix nb_domain signedness in list command

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

Fix multi-channel listing

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

edit printout in lttng list "all" channels

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

Report errors in write_config

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

Allocate session UID with count increment

Otherwise, the UST session UID gets assigned only when the first UST
channel is created for a session, causing IDs to be used for many
sessions (the count of the number of sessions allocated so far) if the
session creation / channel creation is not performed in a usual
sequence.

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

fix reversed enabled/disabled, active/inactive printout

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

Implement 32/64 bit consumer support

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

Fix consumer pid mutex double-unlock

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

Introduce build/env/argument override for 32-bit compat consumer

Also remove unnecessary use of strdup on optarg.

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

Add configure LTTNG_TOOLS_COMPAT_BIN_PREFIX env var support

Defines CONFIG_COMPAT_BIN_PREFIX in the build.

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

Simply close socket when application registration is refused

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

Update warning msg

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

Fix warning about hash type

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

Detect 32-bit compat applications + 32-bit warning fixes

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

Implement UST destroy session

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

Implement UST stop command

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

Ensure sockets are kept open across all their hash table life-span

Otherwise causes races where re-use of a socket is being incorrectly
removed.

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

Support older autotools versions lack of AM_SILENT_RULES

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

Add creation time to UST app path to ensure uniqueness

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

ust-app: fix bogus double-use of iterator

"yo dawg, I'm using my iterator in my iterator".

;-)

Fixes pretty much all of N event/N channel/N session UST problems.

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

UST consumer: fix read on hangup, and UST get subbuf error handling

This fixes multiple consecutive runs of the "demo" program, where
sometimes one channel would not be saved.

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