babeltrace.git
10 years agoFix: add missing heap_copy
Julien Desfossez [Tue, 24 Jul 2012 15:54:02 +0000 (11:54 -0400)] 
Fix: add missing heap_copy

This function allows the user to copy an existing priority heap.
It is required to fix the seek by position.

Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: babeltrace assert() triggered by directories within trace
Mathieu Desnoyers [Thu, 19 Jul 2012 22:35:15 +0000 (18:35 -0400)] 
Fix: babeltrace assert() triggered by directories within trace

Need to check if the file descriptors are directories and skip them.
Print warning.

Fixes #313

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoSeveral fixes for bt_iter_pos related functions
Francis Deslauriers [Wed, 4 Jul 2012 13:15:25 +0000 (09:15 -0400)] 
Several fixes for bt_iter_pos related functions

Add verification so we free the array only if iterator is of type BT_SEEK_RESTORE
Add return statement at the end of case
Set type of bt_iter_pos returned by bt_iter_get_pos

[ Mathieu Desnoyers edit: check the iter_pos->type before the restore
  pointer in bt_iter_free_pos. The && operator is a sequence point, so
  the first check is done before the second. Since we might not be
  using the union for restore if type != BT_SEEK_RESTORE, check the
  type first before accessing the (possibly wrong) union field. ]

Signed-off-by: Francis Deslauriers <francis.deslauriers@polymtl.ca>
Signed-off-by: Julien Desfossez <jdesfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix iterator: various fixes
Mathieu Desnoyers [Fri, 22 Jun 2012 14:35:54 +0000 (10:35 -0400)] 
Fix iterator: various fixes

- various return values were untested
- seek_file_stream_by_timestamp off-by-one
- multi-trace logic issue
- not testing the return value of g_ptr_array_index() for NULL (possible
  NULL pointer dereference)

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: remove duplicate yydebug var
Mathieu Desnoyers [Mon, 18 Jun 2012 18:14:07 +0000 (14:14 -0400)] 
Fix: remove duplicate yydebug var

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix babeltrace iterator lib: seek at time 0
Julien Desfossez [Thu, 14 Jun 2012 12:09:44 +0000 (08:09 -0400)] 
Fix babeltrace iterator lib: seek at time 0

When seeking by timestamp, we assumed that the timestamp passed as an
argument was a real timestamp from the trace. However LTTV uses
timestamp 0 to refer to the beginning of the trace. This fix ensures
that we read at least the first event after the seek.

Signed-off-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoUpdate version to 1.0.0-rc4 v1.0.0-rc4
Mathieu Desnoyers [Wed, 30 May 2012 18:19:57 +0000 (14:19 -0400)] 
Update version to 1.0.0-rc4

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoAdd CodingStyle to tarball
Mathieu Desnoyers [Wed, 30 May 2012 13:46:56 +0000 (09:46 -0400)] 
Add CodingStyle to tarball

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoAdd coding style document
Mathieu Desnoyers [Wed, 30 May 2012 13:09:22 +0000 (09:09 -0400)] 
Add coding style document

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: babeltrace should use output format argument
Fredrik Oestman [Wed, 30 May 2012 12:36:07 +0000 (08:36 -0400)] 
Fix: babeltrace should use output format argument

"Please make this little change, otherwise ctf-text is always output
regardless of "-o"."

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: accept traces which skip stream ids
Mathieu Desnoyers [Tue, 29 May 2012 22:56:49 +0000 (18:56 -0400)] 
Fix: accept traces which skip stream ids

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoUse mmap_align
Mathieu Desnoyers [Tue, 29 May 2012 22:54:43 +0000 (18:54 -0400)] 
Use mmap_align

Allow reading traces with packet size different from the machine page
size.

Fixes #237

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoalign header: define PAGE_SIZE
Mathieu Desnoyers [Tue, 29 May 2012 22:54:21 +0000 (18:54 -0400)] 
align header: define PAGE_SIZE

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoImplement mmap alignment header
Mathieu Desnoyers [Tue, 29 May 2012 22:53:11 +0000 (18:53 -0400)] 
Implement mmap alignment header

Allows reading traces written on systems with smaller page size on
systems with larger page size.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoCleanup: type warnings
Mathieu Desnoyers [Tue, 29 May 2012 21:04:20 +0000 (17:04 -0400)] 
Cleanup: type warnings

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoCleanup: do not overwrite const argv parameters
Mathieu Desnoyers [Tue, 29 May 2012 20:59:05 +0000 (16:59 -0400)] 
Cleanup: do not overwrite const argv parameters

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoUpdate version to 1.0.0-rc3 v1.0.0-rc3
Mathieu Desnoyers [Tue, 29 May 2012 20:10:06 +0000 (16:10 -0400)] 
Update version to 1.0.0-rc3

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: converter error logic
Mathieu Desnoyers [Wed, 9 May 2012 01:32:26 +0000 (21:32 -0400)] 
Fix: converter error logic

- If a subdirectory is empty, just warn and continue anyway.
- On partial error, babeltrace now returns a nonzero error value to the
  parent process, even though it proceeds anyway.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: report appropriate field in error message
Mathieu Desnoyers [Fri, 4 May 2012 03:00:42 +0000 (23:00 -0400)] 
Fix: report appropriate field in error message

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: support 64-bit events discarded counter types
Mathieu Desnoyers [Wed, 2 May 2012 15:48:52 +0000 (11:48 -0400)] 
Fix: support 64-bit events discarded counter types

Previously, counters coming for 64-bit architectures would be treated in
the same way as 32-bit counters, and would therefore miss 32-bit
overflows (only happens if 2^32 events or more are lost within in a
packet).

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: add missing bt_ctf_get_int_len API
Mathieu Desnoyers [Wed, 2 May 2012 15:47:50 +0000 (11:47 -0400)] 
Fix: add missing bt_ctf_get_int_len API

Useful to track the overflow of integer values.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix bt_context_add_traces_recursive error code
Julien Desfossez [Fri, 27 Apr 2012 20:02:58 +0000 (16:02 -0400)] 
Fix bt_context_add_traces_recursive error code

When an empty directory readable by the user was passed to this
function, the error was not set appropriately, this patch fixes this
problem.

Signed-off-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoUpdate version to 1.0.0-rc2 v1.0.0-rc2
Mathieu Desnoyers [Tue, 24 Apr 2012 20:15:01 +0000 (16:15 -0400)] 
Update version to 1.0.0-rc2

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: return an error if no trace can be opened
Mathieu Desnoyers [Tue, 24 Apr 2012 15:50:43 +0000 (11:50 -0400)] 
Fix: return an error if no trace can be opened

Fixes #222

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix : double destroy in context remove
Julien Desfossez [Mon, 23 Apr 2012 14:41:18 +0000 (10:41 -0400)] 
Fix : double destroy in context remove

The hashtable storing the trace handle is created with a destroy
function that frees the elements when they are removed. Manually calling
the destroy function after removing a handle for the table causes
double-free problems.

Fixes #197

Reported-by: Mathieu Denis <mathieu.denis@polymtl.ca>
Signed-off-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoUpdate version to 1.0.0-rc1 v1.0.0-rc1
Mathieu Desnoyers [Wed, 18 Apr 2012 21:14:14 +0000 (17:14 -0400)] 
Update version to 1.0.0-rc1

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: -n "field names" option should allow "none"
Mathieu Desnoyers [Wed, 18 Apr 2012 21:11:41 +0000 (17:11 -0400)] 
Fix: -n "field names" option should allow "none"

- Update manpage, help, and cmd line arg parsing.
- Clarify that the default behavior is "payload,context", implying
  that specifying -n ... overrides the default behavior.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: Show context field name by default
Mathieu Desnoyers [Wed, 18 Apr 2012 15:37:05 +0000 (11:37 -0400)] 
Fix: Show context field name by default

Everyone seems to be baffled by "what is this { 0 }", so change default
behavior to show the context field name: will show e.g. { cpu_id = 0 }.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoManpage fix: babeltrace-log manpage should refer to lttng(1)
Mathieu Desnoyers [Tue, 17 Apr 2012 20:44:14 +0000 (16:44 -0400)] 
Manpage fix: babeltrace-log manpage should refer to lttng(1)

Fixes #217

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoClarify warning message for events discarded
Mathieu Desnoyers [Mon, 16 Apr 2012 14:33:55 +0000 (10:33 -0400)] 
Clarify warning message for events discarded

Reported-by: Fredrik Oestman <Fredrik_Oestman@mentor.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: babeltrace should skip erroneous traces
Mathieu Desnoyers [Wed, 11 Apr 2012 21:24:25 +0000 (17:24 -0400)] 
Fix: babeltrace should skip erroneous traces

When opening a trace collection, if one of the traces has an error,
don't forbid opening the entire collection because of it: just print a
warning and continue.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: put explicit error messages and warnings when opening a trace
Mathieu Desnoyers [Wed, 11 Apr 2012 21:20:19 +0000 (17:20 -0400)] 
Fix: put explicit error messages and warnings when opening a trace

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: segfault on error handling
Mathieu Desnoyers [Wed, 11 Apr 2012 21:05:58 +0000 (17:05 -0400)] 
Fix: segfault on error handling

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: enum must use last value + 1 as next item value
Mathieu Desnoyers [Wed, 11 Apr 2012 20:58:32 +0000 (16:58 -0400)] 
Fix: enum must use last value + 1 as next item value

.. for omitted values.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: warn, and don't assert, when reading a value outside enum range
Mathieu Desnoyers [Wed, 11 Apr 2012 11:41:30 +0000 (07:41 -0400)] 
Fix: warn, and don't assert, when reading a value outside enum range

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoAPI Fix : missing list fields of event decl
Julien Desfossez [Tue, 10 Apr 2012 21:43:31 +0000 (17:43 -0400)] 
API Fix : missing list fields of event decl

This patchs allows the user to list the fields (in all scopes) for each
event declarations (so as soon as the trace is opened).
It uses a lazy allocation, so the memory for storing the fields is only
allocated if the users asks for the list and it is allocated per event.

Signed-off-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoAPI fix : fill the values for timestamp begin and end
Julien Desfossez [Thu, 5 Apr 2012 15:49:40 +0000 (11:49 -0400)] 
API fix : fill the values for timestamp begin and end

The API functions were defined and exported but not implemented.

Signed-off-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoAPI : list of events in the trace
Julien Desfossez [Wed, 4 Apr 2012 00:14:56 +0000 (20:14 -0400)] 
API : list of events in the trace

This function exports the list of events enabled in a trace.

Signed-off-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoUUID field should be optional, as specified by CTF spec
Mathieu Desnoyers [Tue, 3 Apr 2012 18:54:05 +0000 (14:54 -0400)] 
UUID field should be optional, as specified by CTF spec

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoBabeltrace ctf-text duplicated error message
Fredrik Oestman [Tue, 3 Apr 2012 12:55:42 +0000 (08:55 -0400)] 
Babeltrace ctf-text duplicated error message

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoAPI fix/breakage : reexporting bt_ctf_event
Julien Desfossez [Mon, 2 Apr 2012 22:02:00 +0000 (18:02 -0400)] 
API fix/breakage : reexporting bt_ctf_event

Replaces the ctf_event_definition to keep the bt_ namespace.

Signed-off-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agofix/breakage API : replace bt_ctf_event
Julien Desfossez [Fri, 30 Mar 2012 19:50:12 +0000 (15:50 -0400)] 
fix/breakage API : replace bt_ctf_event

Following the namespace cleanup, the bt_ctf_event structure is now a
ctf_event_definition structure. Users of the API only have to do this
substitution.

Signed-off-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoadd ctf_event_definition pointer to ctf_stream_definition
Julien Desfossez [Fri, 30 Mar 2012 19:49:01 +0000 (15:49 -0400)] 
add ctf_event_definition pointer to ctf_stream_definition

also remove two unnecessary forward declarations of
ctf_stream_definition.

Signed-off-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoRename ctf_event to ctf_event_declaration
Julien Desfossez [Fri, 30 Mar 2012 19:48:32 +0000 (15:48 -0400)] 
Rename ctf_event to ctf_event_declaration

Signed-off-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoRename ctf_stream to ctf_stream_definition
Julien Desfossez [Fri, 30 Mar 2012 19:48:00 +0000 (15:48 -0400)] 
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>
10 years agoRename ctf_stream_event to ctf_event_definition
Julien Desfossez [Fri, 30 Mar 2012 19:47:37 +0000 (15:47 -0400)] 
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>
10 years agoRename ctf_stream_class to ctf_stream_declaration
Julien Desfossez [Fri, 30 Mar 2012 19:47:13 +0000 (15:47 -0400)] 
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>
10 years agoCleanup: Remove unneeded local variables from function
Mathieu Desnoyers [Wed, 28 Mar 2012 00:20:35 +0000 (20:20 -0400)] 
Cleanup: Remove unneeded local variables from function

Reported-by: Yannick Brosseau <yannick.brosseau@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoAPI Fix : give access to trace_handle and context
Julien Desfossez [Tue, 27 Mar 2012 13:12:46 +0000 (09:12 -0400)] 
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>
10 years agoAPI Fix : handle id to use the public functions
Julien Desfossez [Mon, 26 Mar 2012 20:41:56 +0000 (16:41 -0400)] 
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>
10 years agoFix : add the missing seek begin
Julien Desfossez [Mon, 19 Mar 2012 16:25:11 +0000 (12:25 -0400)] 
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>
10 years agoUpdate version to 1.0.0-pre4 v1.0.0-pre4
Mathieu Desnoyers [Fri, 16 Mar 2012 19:41:21 +0000 (15:41 -0400)] 
Update version to 1.0.0-pre4

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agofix : bt_ctf_iter_create defaults to BEGIN pos
Julien Desfossez [Wed, 14 Mar 2012 18:58:08 +0000 (14:58 -0400)] 
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>
10 years agoFix : only one iterator per context
Julien Desfossez [Tue, 13 Mar 2012 19:50:34 +0000 (15:50 -0400)] 
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>
10 years agoFix error checking in bt_context_add_traces_recursive
Julien Desfossez [Mon, 12 Mar 2012 18:59:44 +0000 (14:59 -0400)] 
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>
10 years agobt_context_add_trace, bt_iter_pos and bt_iter needed some more comments.
Julien Desfossez [Mon, 12 Mar 2012 16:00:00 +0000 (12:00 -0400)] 
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>
10 years agoFix API: add const qualifiers, privatize struct bt_ctf_event
Mathieu Desnoyers [Mon, 12 Mar 2012 15:12:12 +0000 (11:12 -0400)] 
Fix API: add const qualifiers, privatize struct bt_ctf_event

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: split ctf/event.c
Mathieu Desnoyers [Mon, 12 Mar 2012 14:21:26 +0000 (10:21 -0400)] 
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>
10 years agoFix: seek by timestamp
Mathieu Desnoyers [Fri, 9 Mar 2012 23:26:09 +0000 (18:26 -0500)] 
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>
10 years agoFix: remove leftover code from seek begin (unimplemented for now)
Mathieu Desnoyers [Fri, 9 Mar 2012 23:16:45 +0000 (18:16 -0500)] 
Fix: remove leftover code from seek begin (unimplemented for now)

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: seek error handling
Mathieu Desnoyers [Fri, 9 Mar 2012 21:06:34 +0000 (16:06 -0500)] 
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>
10 years agoFix: callbacks.c: more explicit error msg
Mathieu Desnoyers [Fri, 9 Mar 2012 20:41:03 +0000 (15:41 -0500)] 
Fix: callbacks.c: more explicit error msg

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: ctf/iterator.h bt_ctf_iter_read_event() comment
Mathieu Desnoyers [Fri, 9 Mar 2012 19:37:08 +0000 (14:37 -0500)] 
Fix: ctf/iterator.h bt_ctf_iter_read_event() comment

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoAdd missing iterator.h into the git repo
Mathieu Desnoyers [Thu, 8 Mar 2012 22:26:13 +0000 (17:26 -0500)] 
Add missing iterator.h into the git repo

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoAPI fix: Move ctf iterator API members to new babeltrace/ctf/iterator.h
Mathieu Desnoyers [Thu, 8 Mar 2012 21:41:32 +0000 (16:41 -0500)] 
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>
10 years agoAPI cleanup: Move bt_iter_create/destroy to internal header
Mathieu Desnoyers [Thu, 8 Mar 2012 21:10:26 +0000 (16:10 -0500)] 
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>
10 years agoFix: update missing copyrights and ifdef protection mismatch
Mathieu Desnoyers [Thu, 8 Mar 2012 17:35:43 +0000 (12:35 -0500)] 
Fix: update missing copyrights and ifdef protection mismatch

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoClarify clock correlation error message
Mathieu Desnoyers [Tue, 6 Mar 2012 23:04:47 +0000 (18:04 -0500)] 
Clarify clock correlation error message

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix build warning: set -lpopt as LDFLAGS rather than CFLAGS
Mathieu Desnoyers [Tue, 6 Mar 2012 23:00:02 +0000 (18:00 -0500)] 
Fix build warning: set -lpopt as LDFLAGS rather than CFLAGS

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: add missing _GNU_SOURCE define to babeltrace-log.c
Mathieu Desnoyers [Tue, 6 Mar 2012 22:55:28 +0000 (17:55 -0500)] 
Fix: add missing _GNU_SOURCE define to babeltrace-log.c

Since it is using getline.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: Add memstream.h dependency to Makefile.am
Mathieu Desnoyers [Tue, 6 Mar 2012 22:51:06 +0000 (17:51 -0500)] 
Fix: Add memstream.h dependency to Makefile.am

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoImplement fallback for systems lacking open_memstream and fopenmem
Mathieu Desnoyers [Tue, 6 Mar 2012 22:47:56 +0000 (17:47 -0500)] 
Implement fallback for systems lacking open_memstream and fopenmem

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix: try to include endian.h by default
Mathieu Desnoyers [Mon, 5 Mar 2012 17:38:39 +0000 (12:38 -0500)] 
Fix: try to include endian.h by default

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoUpdate version to 1.0.0-pre3 v1.0.0-pre3
Mathieu Desnoyers [Fri, 2 Mar 2012 22:46:57 +0000 (17:46 -0500)] 
Update version to 1.0.0-pre3

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix uuid in metadata
Mathieu Desnoyers [Thu, 1 Mar 2012 23:50:32 +0000 (18:50 -0500)] 
Fix uuid in metadata

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoBabeltrace wrapper update
Mathieu Desnoyers [Thu, 1 Mar 2012 23:36:07 +0000 (18:36 -0500)] 
Babeltrace wrapper update

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agouuid wrapper: fix use in visitor
Mathieu Desnoyers [Thu, 1 Mar 2012 23:27:24 +0000 (18:27 -0500)] 
uuid wrapper: fix use in visitor

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoEndian wrapper use fix
Mathieu Desnoyers [Thu, 1 Mar 2012 23:18:00 +0000 (18:18 -0500)] 
Endian wrapper use fix

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFreeBSD uuid wrapper fixes
Mathieu Desnoyers [Thu, 1 Mar 2012 23:15:38 +0000 (18:15 -0500)] 
FreeBSD uuid wrapper fixes

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix missing uuid wrapper change
Mathieu Desnoyers [Thu, 1 Mar 2012 23:06:14 +0000 (18:06 -0500)] 
Fix missing uuid wrapper change

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoAdd endian.h wrapper
Mathieu Desnoyers [Thu, 1 Mar 2012 23:04:47 +0000 (18:04 -0500)] 
Add endian.h wrapper

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoUse standard __LONG_MAX__ instead of __WORDSIZE
Mathieu Desnoyers [Thu, 1 Mar 2012 23:01:01 +0000 (18:01 -0500)] 
Use standard __LONG_MAX__ instead of __WORDSIZE

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoCreate BSD wrapper for uuid
Mathieu Desnoyers [Thu, 1 Mar 2012 22:49:30 +0000 (17:49 -0500)] 
Create BSD wrapper for uuid

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agofix API : deal with the optional underscore
Julien Desfossez [Thu, 1 Mar 2012 16:14:44 +0000 (11:14 -0500)] 
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>
10 years agoFix API : functions to access fields properties
Julien Desfossez [Thu, 1 Mar 2012 14:10:50 +0000 (09:10 -0500)] 
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>
10 years agoUpdate version to 1.0.0-pre2 v1.0.0-pre2
Mathieu Desnoyers [Thu, 23 Feb 2012 18:59:51 +0000 (13:59 -0500)] 
Update version to 1.0.0-pre2

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agolinker: privatize prio_heap and babeltrace_types
Mathieu Desnoyers [Thu, 23 Feb 2012 18:53:28 +0000 (13:53 -0500)] 
linker: privatize prio_heap and babeltrace_types

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoLinker: privatize libbabeltrace_types
Mathieu Desnoyers [Thu, 23 Feb 2012 18:39:16 +0000 (13:39 -0500)] 
Linker: privatize libbabeltrace_types

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoLink statically to internal libraries
Mathieu Desnoyers [Thu, 23 Feb 2012 18:25:05 +0000 (13:25 -0500)] 
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>
10 years agoCombine duplicated API/pretty-print timestamp code
Mathieu Desnoyers [Thu, 23 Feb 2012 12:54:23 +0000 (07:54 -0500)] 
Combine duplicated API/pretty-print timestamp code

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoAPI : export the offsetted timestamp
Signed-off-by: Julien Desfossez [Thu, 23 Feb 2012 12:37:06 +0000 (07:37 -0500)] 
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>
10 years agoFix : segfault when printing timestamp on index
Julien Desfossez [Tue, 21 Feb 2012 21:47:15 +0000 (16:47 -0500)] 
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>
10 years agoFix : coherency in const parameters
Julien Desfossez [Tue, 21 Feb 2012 15:45:14 +0000 (10:45 -0500)] 
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>
10 years agoUpdate version to 1.0.0-pre1 v1.0.0-pre1
Mathieu Desnoyers [Mon, 20 Feb 2012 20:47:52 +0000 (15:47 -0500)] 
Update version to 1.0.0-pre1

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoGenerically print tracer version (and env) in verbose mode
Mathieu Desnoyers [Mon, 20 Feb 2012 20:27:51 +0000 (15:27 -0500)] 
Generically print tracer version (and env) in verbose mode

closes #82

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix open_mmap_trace missing support
Mathieu Desnoyers [Fri, 17 Feb 2012 20:03:59 +0000 (15:03 -0500)] 
Fix open_mmap_trace missing support

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agofix: context.h add missing documentation for packet_seek parameter
Mathieu Desnoyers [Fri, 17 Feb 2012 19:53:16 +0000 (14:53 -0500)] 
fix: context.h add missing documentation for packet_seek parameter

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoCleanup iterator.h
Mathieu Desnoyers [Fri, 17 Feb 2012 19:53:07 +0000 (14:53 -0500)] 
Cleanup iterator.h

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoFix callback.h documentation
Mathieu Desnoyers [Fri, 17 Feb 2012 19:43:31 +0000 (14:43 -0500)] 
Fix callback.h documentation

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
10 years agoCleanup: babeltrace.h does not need to include ctf/events.h
Mathieu Desnoyers [Fri, 17 Feb 2012 19:42:28 +0000 (14:42 -0500)] 
Cleanup: babeltrace.h does not need to include ctf/events.h

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
This page took 0.064558 seconds and 4 git commands to generate.