Cleanup: Missing whitespace in comment

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

Fix: Only allow setting a packet context on a non-frozen stream class

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

Add stream_event_context accessors

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

Implement bt_ctf_field deep copy

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

Fix: Incorrect variant tag validation

An erroneous tag validation causes bt_ctf_field_variant_get_field() to
fail when called a second time on a given variant field.

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

Fix: Missing description NULL check in bt_ctf_clock when serializing

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

Fix: Ensure a packet context is a structure

bt_ctf_stream_class_set_packet_context_type was checking the current
stream packet context's type instead of the one being passed.

Added a test to validate that the check is done appropriately.

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

Add comment indicating the type of ctf_traces' elements

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

Fix: sanity check in iterator creation

This function is part of the public API; we need more
sanity checks before using the structures provided by the caller.

Refs: #827

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

Fix: iterate manually over each live trace to add to catch errors

Instead of using g_hash_table_foreach, we now control the iteration over
the hash table, so we can handle properly the errors.
For now, if there is an error for any trace, we consider this a fatal
issue and exit.

We could allow the process to continue and only skip the problematic
traces, but we will wait until this becomes a real use-case.

Fixes: #827
Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Remove duplicate include of unistd in test_ctf_writer

Signed-off-by: Christian Babeux <christian.babeux@efficios.com>

Fix: make sure we can exit the get_new_metadata loop

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>

Fix: respect signed integers' length when printing in hex base

Make sure we don't print a full 64-bit sign-extended value
when printing a negative integer in hexadecimal base.

Fixes #848

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

Ensure types used for index and count in CTF IR API match

This makes it easier for users to write code not producing
warnings.

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

Cleanup: Missing space after cast operator

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

Tests: add tests for the new trace and trace clock accessors

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

Add trace accessor to CTF Writer

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

Add clock accessor to CTF IR Trace

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

Reuse existing g_string instance when setting a clock name

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

Don't automatically generate a clock UUID in _bt_ctf_clock_create

While the public clock constructor still generates a UUID,
it should not be done when creating a clock during TSDL metadata
parsing.

This makes it possible to use bt_ctf_clock_get_uuid to check
if a UUID has already been set during clock definition parsing.

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

Add missing trace-internal.h header file

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

Add an alias name attribute and accessor to CTF IR field type

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

Add internal nameless clock creation API

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

Add Trace CTF IR type

Refactored the Writer interface to expose a Trace object which
will be exposed by the CTF reader.

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

Python bindings clean-up: missing file headers style fixes

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

Docs fix: Change bt_ctf_writer_create_stream return description

writer -> stream

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

Python bindings clean-up: file headers style fix

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

Change CTF IR header descriptions from CTF Writer to CTF IR

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

Detect Python packages directory on configure

The Python interpreter on Debian is configurated to use
...lib/python.../dist-packages instead of
.../lib/python.../site-packages to look for extra modules.

python_modules.m4 performs a runtime python check of the
sys.path variable to check wether the selected interpreter is
configurated to use site-packages vs dist-packages.

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

Python bindings: fail configure if python version < 3.0

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

Add PEP8 style guide requirement to the CodingStyle

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

Python bindings: Make bindings PEP8 compliant

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

Python bindings: Make examples PEP8 compliant

Python bindings: clarify invalid argument type exception

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

Python bindings docs: clarify add_event_class comments

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

Python bindings: Add packet context accessors

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

Documentation fix: add expected types to packet context accessors

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

Python bindings: Add UUID accessors to the Clock class

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

Clean-up: remove extra whitespace in ctf-ir/clock.h

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

Python bindings: cleanup whitespace

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

Python bindings: add "datetime" property to the Event class

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

Tests: check for < 0 rather than -1 specifically

Change test conditions for functions whose documentation
states that "a negative value" is returned on error.

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

Tests: cleanup extra whitespace

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

Tests: Add tests for new clock UUID accessors

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

Add UUID accessors to CTF-IR clock

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

Fix: scan-build warnings in CTF Writer/IR tests

scan-build reported that strcmp could be called on a NULL string
when some tests fail.

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

Fix: offset_s and CTF clocks with frequency != 1GHz

Take into account offset_s from the clock description when calculating
the tc_offset in ctf_get_real_timestamp().

Do not assume that the single_clock->offset field is at a frequency of
1GHz. Take into account the its clock frequency.

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

Fix: incorrect compiler warning

The compiler does not see that *target will necessarily set the values.
Initialize the variables to 0 to silence uninitialized variable warning.

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

Fix: Don't assume that PROT_WRITE grants read permissions

The prot flag passed to mmap() is set to PROT_WRITE when the O_RDWR
access flags is used. This assumes that PROT_WRITE grants read
permissions on the mmap'ed region. While this is true on x86, this
causes a segmentation fault on SPARC and, presumably, other
architectures implementing strict access permissions.

CTF Writer needs read permissions since the unaligned integer writes
may use load instructions as the value is read back to perform the
required shifting/masking is performed in the _bt_bitfield_write_*
macros.

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

Fix: align objstack on 8 bytes

Architectures such as sparc32 have 32-bit pointers, but require
alignment on 8 bytes multiples for 64-bit integers.

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

Fix Python: Add a comment warning about an API limitation

Add a comment warning about a possible misuse of the events()
API. The objects returned by this function shall not be copied
as they become invalid as soon as the next event is accessed.

It is also invalid to access an event after its generator has
gone out of scope.

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

Replace usages of limits.h with the compatibility header

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

Tests: Return from main instead of calling exit()

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

Use BABELTRACE_HOST_NAME_MAX in test_ctf_writer.c

This fixes the build on FreeBSD.

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

Add limits compatibility header

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

Fix: don't perform unaligned integer read/writes

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

Add stream packet header accessors

Stream packet contexts may now be modified to contain custom
fields. The events_discarded field is now handled like a generic
packet context field.

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

Reuse previously allocated string when setting value

bt_ctf_field_string_set_value() frees and reallocates a new
g_string object if the value is changed. This change ensures
that the previous string is used if possible.

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

Fix: bt_ctf_field_type_structure_add_field argument validation

bt_ctf_field_type_structure_add_field is not setting the return
value to a negative value if the parameter validation fails. The
function would abort while appearing to have succeeded.

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

Fix: mmap trace read the stream_id from the first packet

We were hardcoding the stream_id as 0 for mmap traces, causing a
problem for live traces with multiple channels.
A fix enters in lttng-tools v2.4.2 to send the CTF stream id in the beacon
packets, but until then, we cannot support traces with multiple CTF
streams (LTTng channels).

Fixes: #811
Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix: check the lttng-relayd protocol version

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

Fix: don't update the trace collection if no new streams were received

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

Remove logically dead code

Fixes Coverity warning
** CID 1224341:  Logically dead code  (DEADCODE)

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

Add Python bindings for CTF-IR stream getters

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

Tests: Add stream getter tests

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

Implement CTF-IR stream getters

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

Move CTF-Writer stream to CTF-IR

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

Add Python bindings for CTF-IR stream class getters

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

Tests: Add stream class getter tests

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

Implement CTF-IR stream class getters

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

Add Python bindings for CTF-IR event getters

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

Tests: Add event getter tests

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

Implement CTF-IR event getters

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

Fix: Set the stream class' id in the packet header

The stream id was used in the packet header whereas the
stream class' id should be used.

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

Remove unecessary linking on libbabeltrace-ctf-text

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

Add Python bindings for CTF-IR event-types getters

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

Tests: Add an array type and field test

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

Tests: Add event types getter tests

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

Implement CTF-IR field types getters

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

Clean-up: Remove unecessary line break

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

Add Python bindings for CTF-IR event-field getters

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

Update CTF-Writer and CTF-IR copyrights

Changed the copyrights to stay remain in line with our usual
politic of authors retaining their copyright on the code they
contribute.

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

Tests: Test CTF-IR event-field's getters

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

Docs: CTF-IR sequence's length field must be unsigned

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

Implement CTF-IR event fields getters

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

Fix: Check that enumeration container types are integers

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

Add bt_ctf_field_type_enumeration private search functions

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

Add Python bindings for CTF-IR clock's getters

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

Tests: Test CTF-IR Clock's getters

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

Implement CTF-IR Clock getters

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

Split the CTF-Writer implementation in IR and Writer parts

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

Split the CTF-Writer API into IR and Writer-specific parts

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

Split ctf-writer's stream.h into stream.h and stream-class.h

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

Fix (Python bindings): Return a string if a sequence's element are chars

An AttributeError exception was raised when accessing the value property
of a sequence that contains a string.

Reported-by: Yiyang Dai <daiyiyang@hotmail.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: don't use a return code in _bt_ctf_stream_append_discarded_events

That function is defined as void:
| File "/usr/lib/python3/dist-packages/babeltrace.py", line 2284, in
| append_discarded_events
| if ret < 0:
| TypeError: unorderable types: NoneType() < int()

Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Fix: Report the sequence's length field's value as length

bt_ctf_get_field_list currently returns the underlying array's
size which is always the highest element count encountered to
date.

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

Fix: Align buffers from objstack_alloc on sizeof(void *)

The buffers from objstack_alloc will store pointers, so they must
be aligned on a pointer's size, or else it will cause issues on the
CPUs which do not support unaligned addresses access.

Signed-off-by: Fredrik Markstrom <fredrik.markstrom@gmail.com>
Signed-off-by: Roy Li <rongqing.li@windriver.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

Add fallback for betoh/htobe if they do not exist

For older systems endian.h may not contain the betoh/htobe macros.
If they are missing, add them and make sure that any user of
htobe/betoh uses the compat/endian.h

Signed-off-by: Jesper Derehag <jderehag@hotmail.com>
Tested-by: Jesper Derehag <jderehag@hotmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix: Variable name typo in SequenceField and Event classes

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

Fix python: remove weird char at the end of line

There was the character \xc2 at the end of one line that caused an
import error on some ubuntu distros.

Reported-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>