Add base contexts to event template

Add support for context in enable-event template

Add enable-events templates

Templates for the enable-events command can now be found in
$prefix/xml/lttng/templates/event.

Fix: use 'filter' in mi to comply with save format

Doc: add template-path option to enable-event man

Add --template-path support to enable-event

Rename loglevel_str_to_value to loglevel_ust_str_to_value

Add a new loglevel_str_to_value that return the loglevel value based on domain
type.

Temporary mesure: disable xsd validation config_document_get

Disable blank element parsing on get_document

This make sure that when dumping a raw element no blank elements are present.

Fix error handling for config_document/element

Add session-config api for array and element manipulation

Add utility function config_get_loglevel_type_string

Rename get_event_type to config_get_event_type

Add writer function for raw config_element

Move get_config_domain_str to session-config

rename it to config_get_domaain_str

Fix: use the config_event_type instead of opt_event_type

Support multi domain for enable-event

Enable multi domain per command

Make channel read timer interval optional in session.xsd

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Partial revert: channel name and default output

Since it loops on the attribute the second pass always triggered the name_set
check and directly return as an error.

commit a1dd770
Author: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Date:   Wed Jun 15 20:26:21 2016 -0400

    Make channel read timer interval optional in session.xsd

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
commit c2623b8
Author: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Date:   Wed Jun 15 20:25:48 2016 -0400

    Make channel output type optional in session.xsd

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Make channel output type optional in session.xsd

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Make channel name attribute optional in session.xsd

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Rename normal_* template to discard_*

Completion: add --template-path to bash completion for directory listing

Template: add session templates.

For now session type is cast in stone based on the passed template.

Docs: document new --default-output option of create command

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Implement --default-output option of create command

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Docs: document create command --default-name option

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Implement create command --default-name option

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix format and variable name create.c

Fix: generate a full output configuration node

The lttng save --omit-output does not remove the destination node but the full
output node. Make sure to create a full one.

Fix: memory management in create.c

Fix: use temporary data for return of asprintf

Fix session-config: mem leak and uninitialized ret value

Create template: Allow empty destination and name

Allow template with empty name and no destination so create generate them.

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Test: template on create session

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Man: create --template-path

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Fix: use named enums

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Fix: check return of parse_template

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Fix: check return value of config_document_replace_element

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Create commands template feature.

Introduce a new option the create command: --template-path.

The template-path option take a path to a specific template file that follow the
saved sessions schema definition (src/common/config/session.xsd).

Other "create" options are still effective except the mode options for now.

Regular options have priority on the template values and overwrite the
values found in the template.

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Allow optional snapshot name and max_size

Generate a default name and give the default max size if not defined.

Fix: missing goto error on null pointer

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Config: utility functions for direct configurations element manipulations

Introduce two new config struct: config_document and config_element.

config_document represent an instance of a complete configuration (xmlDoc).
config_element represent an instance of a sub configuration element (xmlNode).

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Add error code related to templates handling

Doc: via -> using

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Fix: clear error message

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Fix error message on asprintf

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Fix whitespace

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Fix: make sure ctrl & data url are set for post creation configuration

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Fix: comment for errors propagation

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Fix: use -1 for unknown values of enums

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

refactor: split create_session in 4 functions

validate_command: checks for invalid options configuration
create_session_basic: responsible for session creation and configuration
generate_output: responsible of outputting the info to the user

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>

Docs: document new save command options (--omit-name/--omit-output)

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Honor omit_name and omit_output options in sessiond save command

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Allow nameless sessions in session.xsd

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Add --omit-name and --omit-output options to the save command

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: sessiond ht_match_event() check if filter is NULL

It looks like an agent event's filter expression is NULL when
it's created with -a, for example:

    lttng enable-event -j -a

Since there's no check for this in ht_match_event(), strlen()
makes the session daemon segfault with this scenario:

    lttng create
    lttng enable-event -j -a
    lttng disable-event -j -a

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

configure.ac: fix --enable/disable-kmod option

Now all the following work as expected:

  * No option (defaults to --enable-kmod)
  * --disable-kmod
  * --enable-kmod
  * --enable-kmod=no
  * --enable-kmod=yes

Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Tests: spawn ht_cleanup thread in unit tests

Run the cleanup thread in the unit tests to ensure that the hash
tables are properly cleaned-up even in the context of tests. This
allows us to reliably check for memory leaks in the unit tests.

Lazily initialize max poll set size

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Revert "Assert that a valid clean-up pipe exists on ht_cleanup_push"

This reverts commit fc41443f7ee5fa95a942d0aedd61c75d622697b5.

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

tests: test ust snapshot with discard buffers

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

tests: test kernel snapshot with discard buffers

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Allow channel mode override in snapshot sessions

Allow the lttng cmd line and liblttng-ctl users to override the channel
mode in snapshot sessions.

Note that liblttng-ctl users expecting that an "overwrite" mode
explicitly set at the value "0" (discard) for a snapshot session will
now create a channel in discard mode.

The DEFAULT_CHANNEL_OVERWRITE used by liblttng-ctl
lttng_channel_set_default_attr() is changed to "-1".

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: validate number of subbuffers after tweaking properties

There are properties that are tweaked by each of ust and kernel channel
create functions after a validation on the number of subbuffers for
overwrite channels. Move validation after those properties
modifications.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Use bool type for save command options

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Clean-up: harmonize popt option declarations in save.c

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Clean-up: move mi_writer declaration with other declarations

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Add name omission and output omission on save to lttng-ctl

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Express overwrite attribute as a byte in communication protocol

There is no need for 4 bytes to express a boolean state.

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Assert that a valid clean-up pipe exists on ht_cleanup_push

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: do not refer to objects as .o

The extension of object files is platform dependant,
use $(OBJEXT) instead of .o when referring to objects.

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: do not link against libtool .o objects

We should not link against libtool objects directly
since they have different names in static or shared
only build. Instead link on the full .la

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: Define MANPATH in config.h

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Tests: Make warn_processes.sh more portable

Options to pgrep aren't standardised across coreutils
implementations, use a more common option.

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: add missing script to dist

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: leak of UST app hash tables

The ht_cleanup thread is shut down before the queue of rcu
callbacks is emptied by the rcu_barrier(). Since callbacks added
by call_rcu can push hash tables through the ht_cleanup pipe, we run
into cases where the clean-up thread has been shutdown and
hash tables pushed through the clean-up pipe are leaked.

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: leak of reply buffer on data pending check

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: call xmlCleanupParser to free global libxml2 allocations

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: missing "void" parameter of lib constructor and destructor

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Docs: fix comment typos in lttng-sessiond's main.c

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: lttng-relayd allow binding of privileged ports for non-root users

Non-root users could use authbind to bind to low-numbered ports.
Moreover, the check was inverted; !getuid() will only be true
for the root user.

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

relayd: optimize receive throughput

For channels configured with large sub-buffer size, the relayd copies
the entire trace sub-buffer (trace packet) into a large buffer, and then
copies the large buffer to disk. It is inefficient from a point of view
of cache locality.

Use a 64k buffer on the stack instead, and move the data piece-wise.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Clean-up: harmonize kernctl API error checking

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Introduce LTTNG_IOCTL_CHECK and LTTNG_IOCTL_NO_CHECK macros

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: enforce assumption that lttng-modules ioctl() return <= 0

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: reduce scope of kconsumer consumed_pos and produced_pos

The consumed_pos and produced_pos accesses are protected by the
stream mutex, which is fine as-is. However, consumed_pos is
passed to consumer_get_consume_start_pos() and is flagged by
Coverity as a possible use of a "stale" consumed_pos.

From an analyzer's standpoint, this makes sense since
both lttng_kconsumer_get_produced_snapshot() and
lttng_kconsumer_get_consumed_snapshot() could leave their output
parameter uninitialized and return 0 since they both  assume that
ioctl() will set errno if ret != 0.

IOCTL(3P) specifies that errno is only set if ret < 0.

A bug in lttng-modules could cause ioctl() to return a positive
value, leaving the errno variable unset. In such a case,
both functions would return 0, leaving the positions uninitialized.

A follow-up fix enforces this assumption (ret never > 0) as part
of the kernctl API.

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Docs: document safety of consumer_thread_is_ready access

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Clean-up: unused ret values in thread_manage_health

Reported by Coverity as:
CID 1243014 (#1 of 15): Unused value (UNUSED_VALUE)

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Clean-up: remove dead code in sessiond's set_option()

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: set the logger level to prevent unexpected level inheritance

BSF and other jars can ship with an embedded log4j.properties
file. This causes problem when launching an application with a general
class path (e.g /usr/share/java/*) since log4j will look for a
configuration file in all loaded jars. If any contains a directive for
the root logger, it will affect any logger with no level that are
directly under the root logger. This can result in an unexpected
behaviour (e.g no events triggered etc.).

Link: https://issues.apache.org/jira/browse/BSF-24
Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Typo: Stoping -> Stopping

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Typo: occured -> occurred

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: only perform lttng_consumer_sync_trace_file() in local mode

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Make lttng_consumer_sync_trace_file static

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: don't negate posix_fadvise return value to check error

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix Solaris 10 build: use lttng_strnlen() wrapper

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Rename lttng_error_type to lttng_error_level

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Cleanup error.h __lttng_print() used for message printing

The loglevels have never really been a mask, and it is useless to try to
use them as masks, because the compiler statically knows the value of
the loglevel requested, and can therefore optimise away all the logic.

This takes care of Coverity warning about mixed bitwise and boolean
logic, which was technically correct, but more complex than needed.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>