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>

Remove all code that depends on trace collection directory structure

We now perform equivalent functionnality with environment within trace
metadata.

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

Use environment information for extra fields

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

Parse environment declarations from LTTng-modules and LTTng-UST

LTTng-UST:
vpid, procname, domain.

LTTng-modules:
domain, sysname, release, version.

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

Fix typo

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

Add babeltrace.1 manpage

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

Update help

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

Install license files and dependency listing into the system doc

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

Add ChangeLog to doc install

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

Update version to 0.9

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

Allow signed values in get_signed. Cast unsigned into signed.

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

Show loglevel information only with value

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

env needs to be a keyword too

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

Add env {} warning test case

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

Add environment (env {}) parser-level support

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

Revert "Remove leading underscores from identifiers directly in lexer"

This reverts commit f071e095a2a8f76dd78becfa103b952af68e148d.

The reason why the underscores are needed in the first place is because
we want to make sure sequences and variants to not have clash between
fields called e.g. "stream" and the dynamic scope
"stream.event.header.". Therefore, we need to keep the leading
underscores as part of the internal identifier representation.

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

Remove leading underscores from identifiers directly in lexer

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

Fix: do not remove field names when adding fields

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

API : iterator namespace cleanup

For coherence, rename babeltrace_iter to bt_iter, and
trace_collection_pos to bt_iter_pos.

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

Time offset: use trace offset average

Use the average offset for a given clock across all trace descriptions
as offset for the trace collection. This ensures that the time-stamps
stay invariant for a given trace collection: it won't change if the
traversal order of the trace directories change.

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

API : trace_handle

Add the trace_handle structure to allow the user to manipulate a trace.
Will be useful for getting information on specific trace files and
remove traces from a trace_collection without rebuilding the
trace_collection entirely.

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

Print event discarded at end of stream

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

Discarded event time range is between last packet event and timestamp_end

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

Print time range during which events have been dropped

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

Print events discarded

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

Add clock offset support

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

Warn user of clock offsets do not match

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

Add clocks to trace collection

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

clock: show as time of day

- todo: implement offset at trace collection level.

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

Implement clock mapping within integer type

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

Implement clock IR generation

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

Add warning tests

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

Relax unknown field handling: warn instead of error

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

Fix erroneous use of "-n" for additional fields, add "-f"

The -n option is for enabling "field name" printing.
Add a new "-f" option for enabling optional fields.

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

Output warnings and errors to stderr

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

Fix double typedef of bt_event_name

Reported-by: Tan Dung Le Tran <tan.dung.le.tran@ericsson.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Parse clock descriptions

Parse basic clock descriptions and integer "map = identifiers...;".

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

API : split iterator headers from babeltrace.h

Removed the iterator specific prototypes from babeltrace.h
No change in functionnality.

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

Add missing context.h/context.c

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

Introduce contexts

The context is an object in which the trace_collection is opened. The
iterator (and later callbacks) depend on the context. This patch, does
not modify the behaviour, it is just the introduction of the contexts.
Also moved convert_trace() in the converter code since it is not a
public API function.

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

Fix parallel build for lib/

When we have a dependency from a library on another library (with
LIBADD), we need to put the lib we depend on in another directory, and
build that directory prior to the current directory. This is what is
done in the http://sourceware.org/autobook/autobook/autobook_92.html
"11.5 Convenience Libraries" example.

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

Organise libbabeltrace

Move the library from converter/ to lib/.

Split functionnalities into separate files :
- babeltrace.c
- iterator.c
- callbacks.c

New internal include files for shared structures between iterator and
callbacks :
- include/babeltrace/callbacks-internal.h
- include/babeltrace/iterator-internal.h

No new functionnalities, just a cleanup of the structure.

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

Export the format.h header

This header is necessary to open traces since it deals with the various
formats available.
Install it in the include/babeltrace/ folder and make sure it is
included when we include babeltrace.h

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

Create the babeltrace include dir on install

And for now just install babeltrace.h in it.

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

callbacks: use correct type for element size

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

callbacks: Fix set size array size

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

Fix callback chain array usage

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

Add time delta to ctf-text

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

ctf-text: show timestamp in sec.nsec

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

Fix wrong variable error check

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

Use the generic call to move_pos_slow

Replace two hardcoded calls to ctf_move_pos_slow by calls to the
function pointer registered when opening a trace.

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

Version 0.8

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

Add loglevel support

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

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 libtool version dependency

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

AC_PROG_LIBTOOL is deprecated in favor of LT_INIT

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

Use realpath for path

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

Helper function to extract string from fields

Following the same principle as get_char_array and get_(un)signed_int,
this function returns a char* to a string from an event field.

[ Edit by Mathieu Desnoyers: add header prototype. ]

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

Use realpath on trace collection path

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

allow printing domains when no procname are present, handle various / patterns

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

Add trace:domain,trace:procname,trace:pid support

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

Add "-n trace" option to print trace name

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

Do not install ctf-parser-test

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

Make ctf open errors more verbose

With many files, it is useful to know the cause of open errors, e.g.

File stream openat(): Too many open files

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