[lttng-dev] [PATCH lttng-ust] Add aarch64 support to configure

Signed-off-by: Michael Jeanson mjean...@efficios.com --- configure.ac | 1 + 1 file changed, 1 insertion(+) diff --git a/configure.ac b/configure.ac index f84f426..971cf54 100644 --- a/configure.ac +++ b/configure.ac @@ -238,6 +238,7 @@ changequote([,])dnl s390) NO_UNALIGNED_ACCESS=1

[lttng-dev] [PATCH lttng-ust] Fix: build log4j example when enabled

Signed-off-by: Michael Jeanson mjean...@efficios.com --- .gitignore | 2 +- doc/examples/Makefile.am| 26 +++--- doc/examples/java-jul/run | 3 ++- doc/examples/java-log4j/.intree | 0 doc/examples/java-log4j/run | 5 +++-- 5 files

[lttng-dev] [PATCH lttng-ust] Fix: out of tree build of java agents

Signed-off-by: Michael Jeanson mjean...@efficios.com --- liblttng-ust-java-agent/jni/jul/Makefile.am | 2 +- liblttng-ust-java-agent/jni/log4j/Makefile.am | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/liblttng-ust-java-agent/jni/jul/Makefile.am b/liblttng-ust-java

[lttng-dev] [PATCH lttng-modules] Fix: version checks for kernel 4.0

Signed-off-by: Michael Jeanson mjean...@efficios.com --- probes/Makefile | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/probes/Makefile b/probes/Makefile index 4de157b..f0325c8 100644 --- a/probes/Makefile +++ b/probes/Makefile @@ -61,10 +61,12 @@ obj-m

[lttng-dev] [PATCH lttng-tools] Doc: Update liburcu URL in README.md

Signed-off-by: Michael Jeanson mjean...@efficios.com --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 716fb66..9a95e33 100644 --- a/README.md +++ b/README.md @@ -20,7 +20,7 @@ components: that, the kernel version may probably

[lttng-dev] [PATCH lttng-tools] Tests: Java agent: Add configure switches to enable tests

This patch adds the switches --enable-test-java-agent-(jul|log4j|all) to the configure script, the tests now need to be explicitly enabled and their dependencies are only checked if so. Signed-off-by: Michael Jeanson mjean...@efficios.com --- configure.ac| 94

[lttng-dev] [PATCH] Fix: 32bit values index_major/minor used as 64bit

This patch fixes compiler warnings on big endian architectures where be64toh() is a nop and returns the original type. Signed-off-by: Michael Jeanson mjean...@efficios.com --- formats/ctf/ctf.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/formats/ctf/ctf.c b

[lttng-dev] [PATCH lttng-modules 2.6] Fix: writeback instrumentation for 4.2 kernels

Bandwidth related fields were moved by this commit: commit a88a341a73be4ef035ca26170c849f002797da27 Author: Tejun Heo t...@kernel.org Date: Fri, 22 May 2015 17:13:28 -0400 writeback: move bandwidth related fields from backing_dev_info into bdi_writeback Signed-off-by: Michael Jeanson

[lttng-dev] [PATCH lttng-modules] Fix: writeback instrumentation for 4.2 kernels

Bandwidth related fields were moved by this commit: commit a88a341a73be4ef035ca26170c849f002797da27 Author: Tejun Heo t...@kernel.org Date: Fri, 22 May 2015 17:13:28 -0400 writeback: move bandwidth related fields from backing_dev_info into bdi_writeback Signed-off-by: Michael Jeanson

[lttng-dev] [PATCH lttng-ust] Default to no unaligned access on unsupported archs

Signed-off-by: Michael Jeanson mjean...@efficios.com --- configure.ac | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 03483a7..5901dfb 100644 --- a/configure.ac +++ b/configure.ac @@ -241,7 +241,10 @@ changequote([,])dnl

[lttng-dev] [PATCH lttng-ust] Fix: java class check when uudecode is not present

consistant whether 'uudecode' is present or not. Signed-off-by: Michael Jeanson mjean...@efficios.com --- config/ax_check_class.m4 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/ax_check_class.m4 b/config/ax_check_class.m4 index 098aa77..42b51d7 100644 --- a/config

[lttng-dev] [PATCH lttng-ust] Fix: running java examples out of tree

--- doc/examples/java-jul/run | 7 ++- doc/examples/java-log4j/run | 7 ++- 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/doc/examples/java-jul/run b/doc/examples/java-jul/run index 37a3cfd..6deb6fd 100755 --- a/doc/examples/java-jul/run +++ b/doc/examples/java-jul/run

[lttng-dev] [PATCH lttng-modules] Fix: mm_page_alloc_extfrag instrumentation for kernel 3.18.10

2015 -0800 mm: when stealing freepages, also take pages created by splitting buddy page commit 99592d598eca62bdbbf62b59941c189176dfc614 upstream. This patch adds the proper version check to allow building on 3.18.y. Signed-off-by: Michael Jeanson mjean...@efficios.com --- instrumentation

[lttng-dev] [PATCH lttng-tools] Fix: Improve the error reporting of the track/untrack command

-by: Michael Jeanson mjean...@efficios.com --- include/lttng/lttng-error.h| 2 ++ src/bin/lttng-sessiond/trace-ust.c | 4 ++-- src/bin/lttng/commands/track-untrack.c | 23 +-- src/common/error.c | 2 ++ 4 files changed, 27 insertions(+), 4 deletions

[lttng-dev] [PATCH lttng-tools] Tests: Java agent: update after Java agent refactoring

are found in the CLASSPATH during configure. Signed-off-by: Michael Jeanson mjean...@efficios.com --- .gitignore | 2 + config/ax_check_class.m4| 2 +- configure.ac| 80

[lttng-dev] [PATCH babeltrace 2/3] Port: Add OSX mman.h compat

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- formats/lttng-live/lttng-live-comm.c | 2 +- include/babeltrace/compat/mman.h | 36 2 files changed, 37 insertions(+), 1 deletion(-) create mode 100644 include/babeltrace/compat/mman.h

[lttng-dev] [PATCH babeltrace 3/3] Port: Add OSX libuuid compat

OSX has the libuuid symbols built in the system libraries. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- configure.ac | 48 1 file changed, 28 insertions(+), 20 deletions(-) diff --git a/configure.ac b/configure.ac index f

[lttng-dev] [PATCH babeltrace] Fix: BABELTRACE_HAVE_LIBC_UUID misspelled as BT_HAVE_LIBC_UUID

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- include/babeltrace/compat/uuid.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/babeltrace/compat/uuid.h b/include/babeltrace/compat/uuid.h index 369e478..4237eb5 100644 --- a/include/babeltrace/compat/

[lttng-dev] [PATCH babeltrace] Fix: check for socket lib before function checks

On platforms that require extra libraries to link socket apps, running this macro before the function checks will lead to socket functions being correctly detected. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- configure.ac | 7 +++ 1 file changed, 3 insertions(+), 4 del

[lttng-dev] [PATCH babeltrace] Cleanup: remove unused label 'end'

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- include/babeltrace/compat/send.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/babeltrace/compat/send.h b/include/babeltrace/compat/send.h index 3c6d01a..3998694 100644 --- a/include/babeltrace/compat/

[lttng-dev] [PATCH babeltrace] Fix: posix_fallocate() returns int > 0 on errors

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- formats/ctf/ctf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/formats/ctf/ctf.c b/formats/ctf/ctf.c index 74fcb5b..19f61b5 100644 --- a/formats/ctf/ctf.c +++ b/formats/ctf/ctf.c @@ -906,7 +906,7 @

[lttng-dev] [PATCH lttng-ust] Fix: Use $enableval with AC_ARG_ENABLE

Signed-off-by: Michael Jeanson mjean...@efficios.com --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 8bbf7b9..cb4bb21 100644 --- a/configure.ac +++ b/configure.ac @@ -255,7 +255,7 @@ fi # Check for JNI header files

[lttng-dev] [PATCH lttng-modules] Fix: build failure on 2.6.36

Fixes build failure on the 2.6.36 kernel branch which misses the vzalloc() function. Signed-off-by: Michael Jeanson mjean...@efficios.com --- lttng-events.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/lttng-events.c b/lttng-events.c index b49851b..422a2dd 100644 --- a/lttng

[lttng-dev] [PATCH lttng-modules v2] Fix: build failure on 2.6.36

Fixes build failure on the 2.6.36 kernel branch which misses the vzalloc() function. Signed-off-by: Michael Jeanson mjean...@efficios.com --- lttng-events.c| 5 +++-- wrapper/vzalloc.h | 46 ++ 2 files changed, 49 insertions(+), 2 deletions

[lttng-dev] [PATCH lttng-modules v3] Fix: build failure on 2.6.36

Fixes build failure on the 2.6.36 kernel branch which misses the vzalloc() function. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- lttng-events.c| 5 +++-- wrapper/vzalloc.h | 47 +++ 2 files changed, 50 insertions(+), 2 del

[lttng-dev] [PATCH babeltrace] Fix: make check in out of tree build

Use the tracefiles from the source dir when running make check in an out of tree builddir. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- .gitignore | 2 ++ tests/bin/Makefile.am| 22 ++-- tests/bin/test_trace_read

[lttng-dev] [PATCH lttng-ust] Fix: make check in out of tree build

Use the elf data files from the source dir when running make check in an out of tree builddir. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- tests/ust-elf/Makefile.am | 22 ++ tests/ust-elf/test_ust_elf| 4 tests/ust-elf/test_ust_elf.in | 4 +

[lttng-dev] [PATCH lttng-ust v2] Fix: make check in out of tree build

Use the elf data files from the source dir when running make check in an out of tree builddir. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- .gitignore| 1 + tests/ust-elf/Makefile.am | 22 ++ tests/ust-elf/test_ust_elf

[lttng-dev] [PATCH lttng-ust v3] Fix: make check in out of tree build

Use the elf data files from the source dir when running make check in an out of tree builddir and add the data files to dist. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- .gitignore| 1 + tests/ust-elf/Makefile.am | 22 ++ tests/u

[lttng-dev] [PATCH lttng-ust] Fix: add missing ust-elf.h header to dist

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- include/Makefile.am | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/include/Makefile.am b/include/Makefile.am index 44102ef..3c9cf24 100644 --- a/include/Makefile.am +++ b/include/Makefile.am @@ -24,7

[lttng-dev] [PATCH lttng-modules v2] Fix: lttng-test build failure for kernels < 3.0

Fixes an undefined reference to kstrtouint_from_user. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- probes/lttng-test.c | 3 ++- wrapper/kstrtox.h | 58 + 2 files changed, 60 insertions(+), 1 deletion(-) create mode

[lttng-dev] [PATCH RFC lttng-modules] Fix: lttng-test build failure for kernels < 3.0

Fixes an undefined reference to kstrtouint_from_user. The provided wrapper is just an example and should be carefully reviewed and probably reimplemented from scratch considering my proficiency in kernel c. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- probes/lttng-test.

[lttng-dev] [PATCH babeltrace] Fix: add compat for glib < 2.32

2 and add a compatibility header. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- include/Makefile.am | 1 + include/babeltrace/compat/glib.h | 80 lib/values.c | 6 +-- 3 files changed, 84 insertions(

[lttng-dev] [PATCH lttng-tools v2] Fix: tests: support systems where PAGE_SIZE is not 4096

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- tests/regression/tools/snapshots/ust_test | 5 +- .../tools/tracefile-limits/test_tracefile_size | 56 +- 2 files changed, 58 insertions(+), 3 deletions(-) diff --git a/tests/regression/tools/sna

[lttng-dev] [PATCH babeltrace] Fix: glib >= 2.32 required for g_hash_table_contains() in lib/values.c

--- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index dc23daa..a815274 100644 --- a/configure.ac +++ b/configure.ac @@ -54,7 +54,7 @@ if test ! -f "$srcdir/formats/ctf/metadata/ctf-lexer.c"; then fi -AM_PATH_GLIB_2_0(2.22.0,

[lttng-dev] [PATCH lttng-tools] Fix: Possible dereference of null pointers

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- src/common/sessiond-comm/unix.c | 6 ++ src/lib/lttng-ctl/filter/filter-visitor-set-parent.c | 4 2 files changed, 10 insertions(+) diff --git a/src/common/sessiond-comm/unix.c b/src/common/session

[lttng-dev] [PATCH lttng-tools] Fix: int printed as uint

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- src/common/ust-consumer/ust-consumer.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/common/ust-consumer/ust-consumer.c b/src/common/ust-consumer/ust-consumer.c index 08e6ee0..481bbeb 100644 ---

[lttng-dev] [RFC PATCH v2 babeltrace] Fix: Check return value of fpathconf

for dirent and then overflow on the readdir_r call. This patch is an RFC, I'm not sure what is the best way to handle the failure, should we instead fallback on using the global "NAME_MAX"? Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- formats/ctf/ctf.c | 12 ++-- 1

[lttng-dev] [RFC PATCH babeltrace] Fix: Check return value of fpathconf

for dirent and then overflow on the readdir_r call. This patch is an RFC, I'm not sure what is the best way to handle the failure, should we instead fallback on using the global "NAME_MAX"? Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- formats/ctf/ctf.c | 12 ++-- 1

[lttng-dev] [PATCH lttng-modules] Fix: Use generic irqs_disabled_flags

instead of arch_irqs_disabled_flags, this fixes the build for kernel < 2.6.37 See linux commit df9ee29270c11dba7d0fe0b83ce47a4d8e8d2101 for more details. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- wrapper/irqflags.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[lttng-dev] [RFC PATCH lttng-modules] Add support for builtin kernel build

d as usual and voila! This is an early implementation with a single configuration that enable or disable everything, if it's useful we could add fine grained config options for each modules or group of modules. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- Kconfig

[lttng-dev] [PATCH lttng-tools] Fix: missing $ in SED variable

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 5537c15..f851be8 100644 --- a/configure.ac +++ b/configure.ac @@ -90,7 +90,7 @@ AS_IF([test "x$libtool_fix

[lttng-dev] [PATCH lttng-tools 2/3] Fix: do not link against libtool .o objects

We should not link against libtool objects directly since they have different names in static or shared only build. Instead link on the full .la Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- tests/unit/Makefile.am | 13 - 1 file changed, 4 insertions(+), 9 del

[lttng-dev] [PATCH lttng-tools 1/3] Fix: Define MANPATH in config.h

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- configure.ac | 2 ++ src/common/Makefile.am | 3 +-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index c94da7a..d6149fe 100644 --- a/configure.ac +++ b/configure.ac @@ -621,6

[lttng-dev] [PATCH lttng-tools 3/3] Fix: do not refer to objects as .o

The extension of object files is platform dependant, use $(OBJEXT) instead of .o when referring to objects. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- tests/regression/tools/live/Makefile.am | 8 +++ tests/unit/Makefile.am

[lttng-dev] [PATCH lttng-tools 2/2] Tests: Make warn_processes.sh more portable

Options to pgrep aren't standardised across coreutils implementations, use a more common option. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- tests/Makefile.am | 2 +- tests/utils/Makefile.am | 2 +- tests/utils/warn_lttng_processes.s

[lttng-dev] [PATCH lttng-tools 1/2] Fix: add missing script to dist

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- tests/utils/Makefile.am | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/utils/Makefile.am b/tests/utils/Makefile.am index 4208648..bd79d20 100644 --- a/tests/utils/Makefile.am +++ b/tests/utils/Makef

[lttng-dev] [PATCH babeltrace 1/2] sys/param.h is required for MAXNAMLEN on musl libc

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- formats/lttng-live/lttng-live.h | 1 + 1 file changed, 1 insertion(+) diff --git a/formats/lttng-live/lttng-live.h b/formats/lttng-live/lttng-live.h index e00a51e..b315458 100644 --- a/formats/lttng-live/lttng-live.h +++ b/formats

[lttng-dev] [PATCH babeltrace 2/2] Fix: strerror_r behavior is glibc specific

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- include/babeltrace/compat/string.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/babeltrace/compat/string.h b/include/babeltrace/compat/string.h index 7a574e6..6092616 100644 --- a/include/babeltrace/

[lttng-dev] [PATCH lttng-ust 4/4] Use config.h to detect sched_getcpu support

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- libringbuffer/getcpu.h | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/libringbuffer/getcpu.h b/libringbuffer/getcpu.h index fbddb79..7b009dc 100644 --- a/libringbuffer/getcpu.h +++ b/libring

[lttng-dev] [PATCH lttng-ust 1/4] Add support for musl libc to ust-dlfcn.h

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- include/lttng/ust-dlfcn.h | 4 1 file changed, 4 insertions(+) diff --git a/include/lttng/ust-dlfcn.h b/include/lttng/ust-dlfcn.h index c92f1bb..7471d94 100644 --- a/include/lttng/ust-dlfcn.h +++ b/include/lttng/ust-dlfcn.h @@

[lttng-dev] [PATCH lttng-ust 3/4] Fix: use limits.h in ust-elf test

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- tests/ust-elf/prog.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/ust-elf/prog.c b/tests/ust-elf/prog.c index da4f396..4036563 100644 --- a/tests/ust-elf/prog.c +++ b/tests/ust-elf/prog.c @@ -16,7

[lttng-dev] [PATCH lttng-ust 2/4] Fix: strerror_r behavior is glibc specific

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- include/usterr-signal-safe.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/usterr-signal-safe.h b/include/usterr-signal-safe.h index bc68fbf..1df5ada 100644 --- a/include/usterr-signal-safe.h +++ b/i

[lttng-dev] [PATCH lttng-ust] Only build python lib when agent is enabled

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- Makefile.am | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Makefile.am b/Makefile.am index 1db7480..e498719 100644 --- a/Makefile.am +++ b/Makefile.am @@ -6,7 +6,6 @@ SUBDIRS = . include snprintf libring

[lttng-dev] [PATCH lttng-tools 2/6] Fix: error.h -> common/error.h

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- src/bin/lttng-sessiond/agent.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/bin/lttng-sessiond/agent.c b/src/bin/lttng-sessiond/agent.c index 8e1ef08..5b0b6f5 100644 --- a/src/bin/lttng-sessiond/agent.c

[lttng-dev] [PATCH lttng-tools 1/6] Fix: snapshot del-output with name on musl

Some implementations of strtol() like the one in musl will return EINVAL in perror when no valid number was found in the string. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- src/bin/lttng/commands/snapshot.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[lttng-dev] [PATCH lttng-tools 5/6] Fix: location of various standard headers

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- src/common/daemonize.c | 2 +- src/common/runas.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/common/daemonize.c b/src/common/daemonize.c index 300851b..b12194e 100644 --- a/src/common/daemonize.c

[lttng-dev] [PATCH lttng-tools 6/6] Fix: Set thread stack size to ulimit soft value

Some libc don't honor the limit set for the stack size and use their own empirically chosen static value. Detect this behavior by checking if the current stack size is smally than the soft limit and in that case set the pthread stack size to soft limit value. Signed-off-by: Michael Jeanson <mj

[lttng-dev] [PATCH lttng-tools 4/6] Fix: missing include ctype.h for isdigit()

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- src/bin/lttng-sessiond/main.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/bin/lttng-sessiond/main.c b/src/bin/lttng-sessiond/main.c index ee19ef8..bab92a4 100644 --- a/src/bin/lttng-sessiond/main.c +++ b/src/bin/lttng-se

Re: [lttng-dev] In Redhat 7.1 kmod-lttng-modules is not installed

Hi, The 'kmod-lttng-modules' binary package we provide only supports the latest RHEL 7.x kernel, you can either upgrade your system to the latest kernel or get the source package [1] and build it against your current kernel. Regards, Michael [1]

Re: [lttng-dev] Suggestions concerning lttng analyses

- On Jan 6, 2016, at 3:09 PM, Tetreault, Francois ftetr...@ciena.com wrote: > Some comments concerning lttng analyses. > > 1- How to figure which version of the lttng analyses package is installed? > 1.1- Version is "0-unknown" when using "pip3 show lttnganalyses". > Would be nice to

[lttng-dev] [PATCH lttng-modules 2/2] Fix: RHEL 7.2 scsi instrumentation

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- instrumentation/events/lttng-module/scsi.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/instrumentation/events/lttng-module/scsi.h b/instrumentation/events/lttng-module/scsi.h index c23c4f2..c0088c6

Re: [lttng-dev] [PATCH] Fix: RHEL 7.2 kvm instrumentation

Hi, This patch does not seem to target master or even 2.8 since the LTTNG_RHEL_KERNEL_RANGE macro was modified to take 6 arguments per RHEL kernel version. We usually only accept patches targeted at master that are then backported to supported versions. Regards, Michael - On Jun 27,

[lttng-dev] [PATCH lttng-modules 2/2] Fix: sched instrumentation for 4.1-rt11

--- instrumentation/events/lttng-module/sched.h | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/instrumentation/events/lttng-module/sched.h b/instrumentation/events/lttng-module/sched.h index 4f3676b..116d65e 100644 --- a/instrumentation/events/lttng-module/sched.h +++

[lttng-dev] [PATCH lttng-modules] Fix: Define lttng-tracer metadata only once

Fixes sysfs trying to create the lttng-tracer module version node multiple times. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- Makefile| 8 lttng-context-cpu-id.c | 8 lttng-context-hostname.c| 8 lttng-c

[lttng-dev] [PATCH lttng-modules] Fix: Add CONFIG_LTTNG to modules_install target

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index 6a8c5c7..5d6be94 100644 --- a/Makefile +++ b/Makefile @@ -103,12 +103,12 @@ default: $(MAKE) -C $(KERNELDIR) M

[lttng-dev] [PATCH lttng-modules] Cleanup: Remove unused lttng-types module

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- lttng-events.h | 2 -- probes/Kbuild| 1 - probes/lttng-type-list.h | 33 probes/lttng-types.c | 66 4 files changed, 102 del

[lttng-dev] [RFC PATCH lttng-tools 4/5] Add comments to kmod functions

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- src/bin/lttng-sessiond/modprobe.c | 52 +++ 1 file changed, 52 insertions(+) diff --git a/src/bin/lttng-sessiond/modprobe.c b/src/bin/lttng-sessiond/modprobe.c index 90fec71..3e41f5c 100644 ---

[lttng-dev] [RFC PATCH lttng-tools 3/5] Add libkmod rmmod support

-off-by: Michael Jeanson <mjean...@efficios.com> --- src/bin/lttng-sessiond/modprobe.c | 104 ++ 1 file changed, 95 insertions(+), 9 deletions(-) diff --git a/src/bin/lttng-sessiond/modprobe.c b/src/bin/lttng-sessiond/modprobe.c index 6f9a997..90fec71

[lttng-dev] [PATCH lttng-modules 1/2] Fix: Ubuntu kernel range for mm_page_alloc_extfrag

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- instrumentation/events/lttng-module/kmem.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/instrumentation/events/lttng-module/kmem.h b/instrumentation/events/lttng-module/kmem.h index 25e58cb..cce8da4

[lttng-dev] [PATCH lttng-modules 2/2] Fix: Ubuntu kernel range for block_rq_complete

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- instrumentation/events/lttng-module/block.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/instrumentation/events/lttng-module/block.h b/instrumentation/events/lttng-module/block.h index 1a2ba6b..6b0c788

Re: [lttng-dev] [RFC PATCH lttng-tools 1/5] Drop optionnal control modules load and unload

Fixed in git, I'll wait for more feedback on the patchset before sending a V2. Thanks, Michael - On Feb 12, 2016, at 4:40 PM, Mathieu Desnoyers mathieu.desnoy...@efficios.com wrote: > > optionnal -> optional > trough -> through > > Thanks, > > Mathieu

[lttng-dev] [RFC PATCH lttng-tools 5/5] Add builtin modules support to kmod modprobe

Report success when loading a module that is builtin the kernel. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- src/bin/lttng-sessiond/modprobe.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/bin/lttng-sessiond/modprobe.c b/src/bin/lttng-se

[lttng-dev] [RFC PATCH lttng-tools 2/5] Move libkmod ifdef to beginning of file

This will be useful to add the libkmod implementation of the modprobe_remove_lttng() function in a single ifdef. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- src/bin/lttng-sessiond/modprobe.c | 150 +++--- 1 file changed, 75 insertions(

[lttng-dev] [RFC PATCH lttng-tools 1/5] Drop optionnal control modules load and unload

The optionnal control modules are loaded and unloaded trought modules dependencies by modprobe or libkmod. Trying to load them manually leads to spurious log messages and slows down the sessiond startup. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- src/bin/lttng-sessiond/modp

[lttng-dev] [PATCH lttng-ust] Fix: Use Java 6 syntax in JUL examples

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- doc/examples/java-jul/ApplicationContextExample.java | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/doc/examples/java-jul/ApplicationContextExample.java b/doc/examples/java-jul/ApplicationContextExampl

[lttng-dev] [PATCH lttng-tools] Fix: Use ipv6 loopback adress in test_uri

of localhost. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- tests/unit/test_uri.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/unit/test_uri.c b/tests/unit/test_uri.c index 432e2b3..cd95753 100644 --- a/tests/unit/test_uri.c +++ b/tests/unit/test

[lttng-dev] [PATCH lttng-tools stable-2.6 1/3] Tests: Introduce conf_proc_count()

From: Jérémie Galarneau Tests are using the nproc utility which return the number of _available_ CPUs. The distinction between online, available and configured CPUs is subtle, but important. The number of "online" CPUs can change at runtime as CPUs are

[lttng-dev] [PATCH lttng-tools stable-2.6 2/3] Tests: use configured processor count in snapshot tests

From: Jérémie Galarneau Signed-off-by: Jérémie Galarneau --- tests/regression/tools/snapshots/ust_test | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/regression/tools/snapshots/ust_test

[lttng-dev] [PATCH lttng-modules] Fix: Add gfpflags.h wrapper for kernel >= 4.6.0

points and printk Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- instrumentation/events/lttng-module/btrfs.h | 2 +- instrumentation/events/lttng-module/compaction.h | 2 +- wrapper/gfpflags.h | 32 3 files changed, 34 inse

Re: [lttng-dev] [PATCH lttng-modules] Fix: Add gfpflags.h wrapper for kernel >= 4.6.0

/events/lttng-module/btrfs.h:9:35: fatal error: trace/events/gfpflags.h: No such file or directory compilation terminated. - On Apr 5, 2016, at 3:13 PM, Mathieu Desnoyers mathieu.desnoy...@efficios.com wrote: > - On Apr 5, 2016, at 3:07 PM, Michael Jeanson mjean...@efficios.com wr

[lttng-dev] [PATCH lttng-modules] Fix: Add granularity to RHEL kernel version checks

Our RHEL kernel version macros were based on the major and minor version of the distribution version like "7.2" instead of the full kernel version like "327.4.4". This prevented us from adding compatibility changes with sufficient granularity. Signed-off-by: Michael Jeanson &l

[lttng-dev] [PATCH lttng-ust] Add missing line continuation and fix indentation

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- liblttng-ust-java-agent/jni/jul/Makefile.am | 10 +- liblttng-ust-java-agent/jni/log4j/Makefile.am | 10 +- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/liblttng-ust-java-agent/jni/jul/Makefile

[lttng-dev] [PATCH lttng-ust] Doc: Update README

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- README.md | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/README.md b/README.md index a2d9e63..728c103 100644 --- a/README.md +++ b/README.md @@ -33,7 +33,7 @@ compile the Git repositor

[lttng-dev] [PATCH lttng-modules] Fix: writeback probe on RHEL >= 3.10.0-327.10.1

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- instrumentation/events/lttng-module/writeback.h | 37 ++--- 1 file changed, 27 insertions(+), 10 deletions(-) diff --git a/instrumentation/events/lttng-module/writeback.h b/instrumentation/events/lttng-

Re: [lttng-dev] [PATCH 1/3] tests/lib/Makefile.am: Remove test_seek_empty_packet and test_seek_big_trace from SCRIPT_LIST

- On Apr 27, 2016, at 11:04 PM, Simon Marchi simon.mar...@polymtl.ca wrote: > On 2016-04-27 22:52, Simon Marchi wrote: >> Since these files are generated by autoconf, they shouldn't be included >> in SCRIPT_LIST, which is the list of scripts to copy from the source >> directory to the build

Re: [lttng-dev] [PATCH 1/3] tests/lib/Makefile.am: Remove test_seek_empty_packet and test_seek_big_trace from SCRIPT_LIST

- On Apr 28, 2016, at 12:44 PM, Simon Marchi simon.mar...@polymtl.ca wrote: > > Ok thanks. > > Just an idea: I find it quite cumbersome to add a new entry in > configure.ac for each test script that we want autoconf to process (such > as in patch 3/3). I think it could be easier to have a

[lttng-dev] [PATCH lttng-ust] Fix: merge tap tests stdout and stderr

This makes the output and error statement ordered in the log file and ensure that the first line is the tap test plan. Some tap parser are confused if the test plan is not on the first line. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- tests/Makefile.am | 1 + 1 file chan

[lttng-dev] [PATCH babeltrace] Fix: merge tap tests stdout and stderr

This makes the output and error statement ordered in the log file and ensure that the first line is the tap test plan. Some tap parser are confused if the test plan is not on the first line. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- tests/Makefile.am | 1 + 1 file chan

[lttng-dev] [PATCH lttng-tools] Fix: merge tap tests stdout and stderr

This makes the output and error statement ordered in the log file and ensure that the first line is the tap test plan. Some tap parser are confused if the test plan is not on the first line. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- tests/regression/Makefile.am | 1 + test

[lttng-dev] [PATCH lttng-tools 1/2] Typo: occured -> occurred

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- doc/proposals/0006-lttng-snapshot.txt| 2 +- include/lttng/lttng-error.h | 2 +- src/bin/lttng-crash/lttng-crash.c| 2 +- src/bin/lttng/commands/disable_events.c | 2 +- src/bin/lttng/co

[lttng-dev] [PATCH lttng-tools 2/2] Typo: Stoping -> Stopping

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- doc/man/lttng-stop.1.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/man/lttng-stop.1.txt b/doc/man/lttng-stop.1.txt index 6405d7b..e2997e6 100644 --- a/doc/man/lttng-stop.1.txt +++ b/doc/man/lttng-stop

[lttng-dev] [PATCH lttng-modules] Add modules target to base Makefile

This way running 'make modules' will indeed build the modules like in the upstream build system. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- Makefile | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 588ca96..738a35c

[lttng-dev] [PATCH RFC lttng-tools] Drop installcheck target

The installcheck target is supposed to be used to run tests on the installed files, however our current implementation runs the exact same tests in the build directory as the check target, which makes it useless. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- tests/Makefile.a

[lttng-dev] [PATCH lttng-ust 1/2] Tests: Replace prove by autotools tap runner

-exit' \ TESTS='foo.test baz.test' make -e check This tap driver also creates a log file for each test that can then be used by another system to build a test report. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- .gitignore| 4 Makefile.am

Re: [lttng-dev] [PATCH lttng-ust 1/2] Tests: Replace prove by autotools tap runner

- On May 12, 2016, at 11:30 AM, Mathieu Desnoyers mathieu.desnoy...@efficios.com wrote: > - On May 12, 2016, at 11:01 AM, Michael Jeanson mjean...@efficios.com > wrote: > >> This patch removes the dependency on the prove perl script >> to run the TAP te

[lttng-dev] [PATCH lttng-tools] Tests: Replace prove by autotools tap runner

-exit' \ TESTS='foo.test baz.test' make -e check This tap driver also creates a log file for each test that can then be used by another system to build a test report. Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- .gitignore| 3 + conf

[lttng-dev] [PATCH lttng-tools] Fix: Double free in utils_partial_realpath error path

Fixes coverity #1355521 Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- src/common/utils.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/common/utils.c b/src/common/utils.c index 30a5f10..901ddb0 100644 --- a/src/common/utils.c +++ b/src/common/u

[lttng-dev] [PATCH lttng-modules] Fix: mm_page_alloc_extfrag instrumentation for kernel 3.16.35

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- instrumentation/events/lttng-module/kmem.h | 1 + 1 file changed, 1 insertion(+) diff --git a/instrumentation/events/lttng-module/kmem.h b/instrumentation/events/lttng-module/kmem.h index 9ee3aa2..ad7bf77 100644 --- a/instrumen

[lttng-dev] [PATCH babeltrace] Add missing generated tests to gitignore

Signed-off-by: Michael Jeanson <mjean...@efficios.com> --- .gitignore | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.gitignore b/.gitignore index 7aa1896..527d195 100644 --- a/.gitignore +++ b/.gitignore @@ -13,6 +13,9 @@ /tests/lib/test_seek_empty_packet /tests/lib/test_dwarf

  1   2   3   4   5   6   >