Update version to 1.9.2

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

Add missing ust_libc.h to Makefile.am

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

Use DBG instead of WARN for futex_wake workaround message

This message will only be shown if LTTNG_UST_DEBUG env. var. is set.

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

Make dlopen more robust by using the .0 target for the library

We've experienced a double library load on a setup where the lttng-ust
.so and .so.0 were two physical files instead of a symlink (as they
should be). So let's make liblttng-ust slightly more robust by
specifying the target .so.0 instead of .so. It will also make handling
of library version upgrades easier.

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

Update changelog

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

Update README

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

Update version to 1.9.1

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

Rename liblttng-ust-malloc to liblttng-ust-libc, update to lttng-ust 2.0

- includes a fix to tracepoints to disable the tracepoint sites upon
  unregistration of the library. This is useful for cases where, like
  the libc wrapper, the library dependency for constructor/destructor
  is not quite right due to cross-dependencies (lttng-ust depends on
  malloc/free). This ensures the session teardown will never be in a
  position where it could teardown data structures still in use
  by probes that cannot be unregistered because their associated
  library would already have called its destructor.

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

Update readme file, add example makefile

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

Add missing compat.h file in Makefile.am

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

Remove deprecated documentation files

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

Remove unused variable warning

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

Fix unset metadata buffer self ref

Caused the metadata flush to never be done.

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

add ustctl_release_handle

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

Print compiler warning/runtime warning and truncate too long tracepoint names

- also apply to loglevel names.
- -Wsystem-headers needs to be used to get gcc to show the
  warnings from system headers (which is the category in which the
  tracepoint event declaration headers falls into).

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

Add sock flush buffer ctl API

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

Add LTTNG_HOME_RUNDIR constant for local tracing

Adds the LTTNG_HOME_RUNDIR constant using ".lttng" as local directory
used for local sockets (when the session daemon is not running under
root privileges).

This is part of a change made to lttng-tools in order to support
unprivileged users to use the UST tracer.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix dynamic linking with GNU gold linker

Programs that were not linked explicitely with -lurcu-bp generated no
trace output when linked with GNU gold, because the urcu-bp library
symbols were not used to populate the weak urcu-bp symbols used by the
program. (even though they worked fine with the standard GNU ld).

Solve this by creating those wrapper symbols into tracepoint.c, and
explicitely get them with dlsym() in the constructor.

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

wildcards & loglevels: fix list field for iteration

Was causing segmentation fault on loglevel & wildcard disable operation.

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

Handle _LGPL_SOURCE config

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

Add wildcard support

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

Fix loglevel value enable: use isdigit rather than isalpha

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

Fix uninitialized match variable

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

Support activation of loglevel by number

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

loglevel enable fix: enable event names, not loglevel

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

Fix add_loglevel entry setup

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

loglevel enable fixes

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

loglevels: deal with fixup upon library load

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

Add loglevel enable/disable support

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

Include lttng/tracepoint-types.h from tracepoint.h

Was wrong include path.

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

Fix BUG_ON handling

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

Cleanup tracepoint.c

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

Increase tracepoint provider:name len limit to 255, add compile-time warning

The compiler will now complain with a warning if a tracepoint
provider:name exceeds the 255 char limit (+ \0).

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

Use iterator on tracepoint probes instead of tracepoints per se

- Simplify iteration locking scheme by using a linked list to store all
  the tracepoints to be iterated on.
- Allow getting the loglevel in addition to the tracepoint name.

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

Cleanup the JNI interface options in configure.ac

Fix typos, use macros and skip a redundant check.

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

Add check for 'texinfo' to configure.ac

Fix for http://lttng.org/issue/252

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

Update .gitignore

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

Remove unused import for java tracing wrapper

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

Add missing .class in ust java makefile

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

Update tracepoint used within lttng ust java

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

lttng_ust_java: add missing .h

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

Update lttng-ust java provider following API change

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

Add TRACEPOINT_DEFINE to ust java lib

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

Describe autotools/libtool/automake version dependency

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

Remove m4_ifdef for AC_PROG_LIBTOOL (deprecated)

This trick does not seem to work anyway.

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

Use unsigned int instead of int for bitfields

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

Change API to support exporting loglevel in event listing

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

Remove loglevel iter (will be performed by tracepoint iteration)

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

Tracepoint loglevel: setup all loglevel information at build time

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

ust-ctl API: ustctl_release_object() should return an error instead of assert

Communication error can cause this function to fail, so an assert is
definitely not the right approach. Return error to caller.

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

Divert installation of test plugins into /tmp/

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

Support older autotools

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

Remove unneeded null pointer check

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

Fix demo program link dependencies

Only demo can afford not to link with liblttng-ust, not the providers.

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

ltt_event_create should return error values

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

Apply autoupdate to configure.ac

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

demo program: only depend on libdl

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

Tracepoint c++ support: cast void * to function pointer

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

Introduce c++ test

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

Add new demo files

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

Allow weak binding from instrumented apps to liblttng-ust

This introduce the TRACEPOINT_DEFINE macro. See demo program for usage
example. This is an API change for instrumentation.

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

Update tracepoint comments

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

Rename TP_LOGLEVEL to tp_loglevel

Within the loglevel enumeration, the entries are not separated by a
comma. To match the style used for TP_FIELDS() ctf_*() entries, use
lower-case, which denotes that these entries must not be separated by
commas.

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

Implement tracepoint loglevel macros

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

Handle sessiond application refusal with delayed retry

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

Fix 32-bit compile type size warning

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

Send bitness upon registration

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

USt test: fix 32-bit type warning

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

Fix 32-bit type warning

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

Add missing patient_write.c

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

ustctl: do not free object on release_object

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

Do not install share.h (cleanup)

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

Do not install usterr-signal-safe

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

Remove core.h

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

Extend align.h

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

Implement error.h

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

Remove unused ARRAY_SIZE macro

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

Implement __rb_same_type

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

Implement lttng_ust_get_cpu()

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

Cleanup: remove debug output

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

ust-tracepoint-event: fix probe creation

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

Compile fix following moved headers

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

Implement ust-tracepoint-events under MIT license

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

Implement tracepoint event reset header

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

Fix tracepoint event related build errors

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

Basic tracepoint event implementation

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

Implement jhash from public domain source, apply LGPL license

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

More compile fixes

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

Fix lib ring buffer compile errors

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

Fix configure

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

Add missing extern to tracepoint.h

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

Installed headers relicensing and cleanup

- Move all locally-used code outside of include/lttng.
- All headers included in applications for tracing are relicensed
  to MIT license, except for lttng/core.h and
  lttng/usterr-signal-safe.h, which both purposefully only contain
  trivial declarations are static inline functions/macros of 10 lines or
  less. They are therefore OK to include into non-GPL applications
  without being considered as derivative work under the terms of the
  LGPLv2.1 license.
- Update version number support.

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

clock.h: ensure all functions are trivial (10 lines or less)

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

Remove old marker header files

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

usterr-signal-safe.h: ensure all macros are trivial (10 lines max)

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

Implement tracepoint-internal.h as LGPLv2.1 (not exported)

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

Relicense tests probe headers/tp.c files to MIT-style license

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

Remove all LGPL-licensed headers

Remove all headers licensed under LGPLv2.1 for upcoming reimplementation
under the BSD-style MIT license.

Even though technically most of the content of these headers fall under
the exceptions of the LGPL:

"When a "work that uses the Library" uses material from a header file
that is part of the Library, the object code for the work may be a
derivative work of the Library even though the source code is not.
Whether this is true is especially significant if the work can be
linked without the Library, or if the work is itself a library.  The
threshold for this to be true is not precisely defined by law.

  If such an object file uses only numerical parameters, data
structure layouts and accessors, and small macros and small inline
functions (ten lines or less in length), then the use of the object
file is unrestricted, regardless of whether it is legally a derivative
work.  (Executables containing this object code plus portions of the
Library will still fall under Section 6.)"

Our intent is to make sure applications can include these headers
without having to be considered derived code.

Therefore, the license of align.h, bug.h, ringbuffer-abi.h and
ringbuffer-config.h, ust-abi.h, ust-events.h, vatomic.h are changed to
MIT license (I, Mathieu Desnoyers, am the sole author of these files and
agree to these changes).

I am removing tracepoint-event.h, tracepoint-internal.h,
tracepoint-types.h, tracepoint.h, ust-tracepoint-event-reset.h and
ust-tracepoint-event.h because those might be considered derived work
from Ftrace.

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

Tracepoint: evaluate arguments within test block

> On Thu, 2011-11-17 at 04:55 +0100, Eric Dumazet wrote:
> > The general admitted claim of a tracepoint being on x86 a single
> > instruction :
> >
> > jmp +0
> >
> > Is not always true.
> >
> > For example in mm/slub.c, kmem_cache_alloc()
> >
> > void *ret = slab_alloc(s, gfpflags, NUMA_NO_NODE, _RET_IP_);
> > trace_kmem_cache_alloc(_RET_IP_, ret, s->objsize, s->size, gfpflags);
> > return ret;
> >
> > We can check compiler output and see that 4 extra instructions were
> > added because s->objsize & s->size are evaluated.
> >
> > I noticed this in a perf session, because these 4 extra instructions
> > added some noticeable latency/cost.
> >
> > c10e26a4:       8b 5d d8                mov    -0x28(%ebp),%ebx
> > c10e26a7:       85 db                   test   %ebx,%ebx
> > c10e26a9:       75 6d                   jne    c10e2718   (doing the memset())
> > c10e26ab:       8b 76 0c                mov    0xc(%esi),%esi   // extra 1
> > c10e26ae:       8b 5d 04                mov    0x4(%ebp),%ebx   // extra 2
> > c10e26b1:       89 75 f0                mov    %esi,-0x10(%ebp) // extra 3
> > c10e26b4:       89 5d ec                mov    %ebx,-0x14(%ebp) // extra 4
> > c10e26b7:       e9 00 00 00 00          jmp    c10e26bc
> > c10e26bc:       8b 45 d8                mov    -0x28(%ebp),%eax
> > c10e26bf:       83 c4 28                add    $0x28,%esp
> > c10e26c2:       5b                      pop    %ebx
> > c10e26c3:       5e                      pop    %esi
> > c10e26c4:       5f                      pop    %edi
> > c10e26c5:       c9                      leave
> >
> >
> > A fix would be to not declare an inline function but a macro...
> >
> > #define trace_kmem_cache_alloc(...) \
> >     if (static_branch(&__tracepoint_kmem_cache_alloc.key)) \
> >             __DO_TRACE(&__tracepoint_kmem_cache_alloc,      \
> >                     ...
> >
> > Anyone has some clever idea how to make this possible ?

Do this by moving the test into the tracepoint() macro.

Reported-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Remove console output that should be turned into debug

But no debug log facility for this library which is included in
sessiond (yet).

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