Rename ctf_stream to ctf_stream_definition

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

Rename ctf_stream_event to ctf_event_definition

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

Rename ctf_stream_class to ctf_stream_declaration

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

Cleanup: Remove unneeded local variables from function

Reported-by: Yannick Brosseau <yannick.brosseau@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

API Fix : give access to trace_handle and context

When reading multiple traces, we need to know to which context or
trace_handle an event belongs to. Setting the context and trace_handle
is format agnostic, but reading it is not, so the reading part is only
implemented for CTF.

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

API Fix : handle id to use the public functions

The API functions were designed to be used with struct trace_handle as
parameters, but no public function returns such a struct.
This patch fixes that by allowing the user to pass the context and the
handle_id.

Acked-by: Yannick Brosseau <yannick.brosseau@gmail.com>
Signed-off-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix : add the missing seek begin

bt_iter_set_pos can now seek to the beginning of the trace.

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

Update version to 1.0.0-pre4

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

fix : bt_ctf_iter_create defaults to BEGIN pos

With the context notion it does not make sense to start the iterator at
the current position, so we default to BT_SEEK_BEGIN if begin_pos is
NULL.

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

Fix : only one iterator per context

As of now, the API let the user create multiple iterators on the same
context but the underlying code was not ready. This patch restrict to
only one iterator per context. Supporting multiple iterators is
targetted for the next release.

Fixes #166

[ Edit by Mathieu Desnoyers: update API comment ]

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

Fix error checking in bt_context_add_traces_recursive

If a trace directory is inexistant or if the user doesn't have the
permission to open the parent directory, bt_context_add_traces_recursive
returned 0, now it returns -1. This function is not part of the API, so
the programs who copied this function into their code should apply this
fix as well.

Fixes #103

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

bt_context_add_trace, bt_iter_pos and bt_iter needed some more comments.

Fixes #75

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

Fix API: add const qualifiers, privatize struct bt_ctf_event

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

Fix: split ctf/event.c

Ensure the implementation follows the API event.h/iterator.h split.

closes #161

Reported-by: Yannick Brosseau <yannick.brosseau@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix: seek by timestamp

When seeking a stream by timestamp, we need to match the first packet in
the stream that is at (or after) the timestamp to seek to. Failure to do
so is a bug: if we have many streams, and seek all streams to a
timestamp that is after the start time of some of the streams, those
late comers will show no events.

Reported-by: Yannick Brosseau <yannick.brosseau@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix: remove leftover code from seek begin (unimplemented for now)

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

Fix: seek error handling

- EINVAL should not teardown context.
- SEEK_TIME 0 should be allowed.

Reported-by: Yannick Brosseau <yannick.brosseau@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix: callbacks.c: more explicit error msg

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

Fix: ctf/iterator.h bt_ctf_iter_read_event() comment

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

Add missing iterator.h into the git repo

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

API fix: Move ctf iterator API members to new babeltrace/ctf/iterator.h

They did not belong in babeltrace/ctf/event.h.

(Closes #161)

Reported-by: Yannick Brosseau <yannick.brosseau@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

API cleanup: Move bt_iter_create/destroy to internal header

bt_iter is an abstract class, so its constructor/destructor should not
be public.

Reported-by: Yannick Brosseau <yannick.brosseau@gmail.com>
Acked-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix: update missing copyrights and ifdef protection mismatch

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

Clarify clock correlation error message

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

Fix build warning: set -lpopt as LDFLAGS rather than CFLAGS

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

Fix: add missing _GNU_SOURCE define to babeltrace-log.c

Since it is using getline.

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

Fix: Add memstream.h dependency to Makefile.am

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

Implement fallback for systems lacking open_memstream and fopenmem

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

Fix: try to include endian.h by default

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

Update version to 1.0.0-pre3

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

Fix uuid in metadata

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

Babeltrace wrapper update

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

uuid wrapper: fix use in visitor

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

Endian wrapper use fix

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

FreeBSD uuid wrapper fixes

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

Fix missing uuid wrapper change

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

Add endian.h wrapper

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

Use standard __LONG_MAX__ instead of __WORDSIZE

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

Create BSD wrapper for uuid

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

fix API : deal with the optional underscore

A field can be prefixed by an underscore internally by the tracer.
This provides the abilility for the user to specify the underscore or
not (depending if he read the metadata or the tracepoint event
definition).
This is a fallback mechanism, so if the request fails, we prefix an
underscore to the field name and request it again.

Also, the bt_ctf_field_name function now removes the optional
underscore.

A quick benchmark shows that calling the bt_ctf_get_field without the
underscore on a field that requires an underscore adds around 151
nanoseconds to the call on a core 2 duo (tested in a loop of 1000000
iterations).

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

Fix API : functions to access fields properties

Add the ability to know :
- the signedness of an int
- the base of an int
- the encoding of an int
- the len of an array
- the encoding of a string

refs #139

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

Update version to 1.0.0-pre2

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

linker: privatize prio_heap and babeltrace_types

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

Linker: privatize libbabeltrace_types

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

Link statically to internal libraries

Previously, we installed all "internal" libs as .so, because static
linking a static lib into a dynamic lib did remove the unused objects.
Use the -Wl,--no-as-needed linker flag to fix this issue, allowing us to
entirely privatize private libs.

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

Combine duplicated API/pretty-print timestamp code

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

API : export the offsetted timestamp

We export now the offsetted view of the timestamp as well as the raw
timestamp to the users.

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

Fix : segfault when printing timestamp on index

If we detect we lost events in a packet, we print an error message
containing the begin and end timestamps of the packet.  If we detect
this packet loss while generating the index (during the open_trace), the
trace_collection is not yet referenced and so the clock information is
not available. This fix prevents the printing at this stage.

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

Fix : coherency in const parameters

This patchs adds the const attribute to the field access functions
(get_int64 and others) in order to keep the coherency with the get_field
and get_field_list function which return a const pointer.

fixes #84

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

Update version to 1.0.0-pre1

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

Generically print tracer version (and env) in verbose mode

closes #82

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

Fix open_mmap_trace missing support

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

fix: context.h add missing documentation for packet_seek parameter

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

Cleanup iterator.h

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

Fix callback.h documentation

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

Cleanup: babeltrace.h does not need to include ctf/events.h

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

API fix: Move callbacks to CTF plugin

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

Removed unneeded ctf_* forward declarations from babeltrace.h

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

Remove unneeded forward declarations (ctf-specific) from iterator.h

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

fix: move read event from iterator to event.h

because it is CTF-specific.

Need to move the format registry to libbabeltrace due to
cross-dependencies between libbabeltrace and formats.

Now, the format libs depend on libbabeltrace, never the opposite.

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

Fix: ctf specific events.c needs to be within CTF plugin

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

API fix: packet_seek should take an index rather than offset

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

Fix API: allow packet_seek override

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

Fix: add lib missing includes

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

Fix: packet_seek is local to plugins, but can be overridden

libbabeltrace should not know about them.

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

Fix API: rename unclear move_pos_slow to packet_seek

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

Fix: Check for unknown format

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

Fix: do not use ctf-specific struct in format.h

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

fix : callback type, doc and links to libbabeltrace

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

Fix : callback chain len

Wrong check caused a segmentation fault when checking for a callback at
the end of the callback chain.

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

Update version to 0.12

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

Typedef bt_intern_str

Remove external dependency to glib

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

Export the list header with a new namespace

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

API : add_trace return the trace_handle id

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

fix manpage: refer to lttng(1)

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

API : cleanup includes

The user just need to include babeltrace/babeltrace.h to use the API.
Also, create a babeltrace/ctf directory to store CTF-related code.

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

API : Access CTF events fields

Update the iterator code to return a bt_ctf_event.
Access to field list within a scope.
Access to field values depending on type.

[ Edit by Mathieu Desnoyers: minor updates: uint64_t for timestamp, and
  style fixes. ]

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

manpage: Add babeltrace-log.1

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

API : seek by timestamp

Add a new function to create a timestamp-based position and the support
in bt_iter_set_pos to seek to certain timestamp.

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

Move the add_traces_recursive out of the library

The exact API for this function is hard to define, so we move back
the function directly to the converter code where it can serve
as an example for applications who want to use this functionnality.

Signed-off-by: Yannick Brosseau <yannick.brosseau@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

manpage: use pp instead of PP

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

Update version to 0.11.1

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

Add trace-collection.h to the installed headers

Or else the tarball generated by 'make dist' fails to compile with

context.c:27:41: fatal error: babeltrace/trace-collection.h: No
                 such file or directory

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

Update version to 0.11

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

Add --clock-force-correlate option

For support of older 2.0-pre traces.

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

lib: open trace error handling fix

When a trace from a trace collection path fails, consider this as a hard
error. Print errors to stderr rather than stdout.

The end goal is to never show an output if any of the trace specified
(or trace within a trace collection) fails to open.

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

Use default if there is no clock declaration

In absence of clock description in metadata, use a default clock
source which increments once per nanosecond.

Without this patch, the converter crashes when given old traces
that are lacking clock description block.

[ Edit by Mathieu Desnoyers: style cleanup, added clock description,
  added comments. ]

Signed-off-by: Juha Niskanen <juniskane@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

iterator: fix type warning on 32-bit build

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

Use 64-bit type for packet and content size

Expression filestats.st_size * CHAR_BIT wraps around very easily
when stored to 32-bit size_t. Currently BabelTrace cannot handle
input larger than 256Mb because of this and diagnostic messages
from this function can be incorrect because of overflow.

This patch fixes my immediate problem, but further work is needed
for proper large file support i.e. handling files bigger than 2Gb.

Signed-off-by: Juha Niskanen <juniskane@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix context API

- bt_context_add_traces_recursive ->
  don't have 2 symbols that just differ from a "s".
- bt_context_add_trace return values:
  - Now, nonzero is error, zero is OK.
  - Streamline with bt_context_add_traces_recursive, so none return
    trace handle IDs.

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

Add missing context-internal.h

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

struct bt_context is opaque

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

Update comment to match context API code

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

Print loglevel names

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

babeltrace lib cleanup, folded with open/remove trace functions

Folded patch from Yannick Brosseau <yannick.brosseau@gmail.com>, along
with various updates and cleanups, related to babeltrace lib.

Original changelog from Yannick Brosseau:
Move the trace_collection into its own file.
Port the converter to uses the new functions

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

API : iterator get and set position

This patch implements the save and restore position on a iterator.

Passed successfully these tests :
- test 1:
save the position
read n events (tested with multiple values of n)
restore
compare the outputs

- test 2:
in loop until the end of the trace :
        save the position
        read 5 events
        restore the position
        read 5 events
compare each block of 5 events

- test 3:
save the position before printing the first event
parse the trace entirely
restore to the beginning
parse the trace entirely
compare the 2 outputs

- test 4:
seek in the future (manually tweaked the values to restore to)
compare with a normal trace starting from the restore point

[ edit: use g_new/g_new0/g_free instead of malloc/calloc/free.
  Use g_new0 anytime a structure is not explicitly initialized by
  a helper, so if it grows in the future with new fields, they will be
  zeroed. ]

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

Update version to 0.10

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

Fix clock freq

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

clock: use freq field

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

Show env. in verbose mode

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