[PATCH 0/1] lttng-2.0 fixes

[PATCH 0/1] lttng-2.0 fixes

[tom.zanussi]

From: Tom Zanussi <tom.zanussi@intel.com>

When trying out the current lttng-2.0, I found that it didn't work in
its current state.  This patch makes it work again, or at least a
a quick test on qemux86 works:

Before:

root@qemux86:~# lttng create
Spawning a session daemon
FATAL: Module lttng-tracer not found.
Error: Unable to load module lttng-tracer
Error: Unable to remove module lttng-tracer
Warning: No kernel tracer available
Session auto-20121012-190712 created.
Traces will be written in /home/root/lttng-traces/auto-20121012-190712

After:

root@qemux86:~# lttng create
Spawning a session daemon
Session auto-20121012-192342 created.
Traces will be written in /home/root/lttng-traces/auto-20121012-192342
root@qemux86:~# lttng enable-event --kernel --all
All kernel events are enabled in channel channel0
root@qemux86:~# lttng start
Tracing started for session auto-20121012-192342
root@qemux86:~# lttng stop
Tracing stopped for session auto-20121012-192342
root@qemux86:~# lttng view
[19:27:01.661844467] (+?.?????????) hrtimer_cancel: { 0 }, { hrtimer = 3354328992 }
[19:27:01.661854316] (+0.000009849) hrtimer_expire_entry: { 0 }, { hrtimer = 3354328992, now = \
318684045266, function = 3238494992 }
[19:27:01.661870669] (+0.000016353) softirq_raise: { 0 }, { vec = 1 }
[19:27:01.661876340] (+0.000005671) softirq_raise: { 0 }, { vec = 9 }
[19:27:01.661887892] (+0.000011552) sched_stat_runtime: { 0 }, { comm = "lttng-sessiond", tid =


The following changes since commit 0260bb5c6978839c068007fcff2f704937805faf:

  gitignore: Fix for poky repository (2012-10-11 13:52:14 +0100)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib.git tzanussi/lttng-fixes.0
  http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=tzanussi/lttng-fixes.0

Tom Zanussi (2):
  lttng-modules: update SRCREV and remove is_compat_task patch

 ...ls-protect-is_compat_task-from-redefiniti.patch | 41 ----------------------
 .../lttng-2.0/lttng-modules_2.0.5.bb               |  9 +++--
 2 files changed, 9 insertions(+), 50 deletions(-)
 delete mode 100644 meta/recipes-kernel/lttng-2.0/lttng-modules/lttng-sycalls-protect-is_compat_task-from-redefiniti.patch

-- 
1.7.11.4

[PATCH 1/1] lttng-modules: update SRCREV and remove is_compat_task patch

[tom.zanussi]

From: Tom Zanussi <tom.zanussi@intel.com>

The existing SRCREV doesn't point to v2.0.5 as the recipe version
suggests, and is actually building modules that won't work with the
lttng-tools it's supposed to be matching.

Also, remove the is_compat_task patch for lttng-syscalls.c to avoid
do_patch errors, as recent versions of lttng-modules have added this
check.

Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>
---
 ...ls-protect-is_compat_task-from-redefiniti.patch | 41 ----------------------
 .../lttng-2.0/lttng-modules_2.0.5.bb               |  9 +++--
 2 files changed, 4 insertions(+), 46 deletions(-)
 delete mode 100644 meta/recipes-kernel/lttng-2.0/lttng-modules/lttng-sycalls-protect-is_compat_task-from-redefiniti.patch

diff --git a/meta/recipes-kernel/lttng-2.0/lttng-modules/lttng-sycalls-protect-is_compat_task-from-redefiniti.patch b/meta/recipes-kernel/lttng-2.0/lttng-modules/lttng-sycalls-protect-is_compat_task-from-redefiniti.patch
deleted file mode 100644
index 0056633..0000000
--- a/meta/recipes-kernel/lttng-2.0/lttng-modules/lttng-sycalls-protect-is_compat_task-from-redefiniti.patch
+++ /dev/null
@@ -1,41 +0,0 @@
-From 415a1a678cb2fcf603f9777f2c7b71c1e38f8101 Mon Sep 17 00:00:00 2001
-From: Bruce Ashfield <bruce.ashfield@windriver.com>
-Date: Mon, 19 Mar 2012 13:30:31 -0400
-Subject: [PATCH] lttng-sycalls: protect is_compat_task from redefinition
-
-recent -stable kernels have their own fallback definition of is_compat_task
-
-   #define is_compat_task() (0)
-
-To protect against this double definition of is_compat_task we can check
-to see if it is already defined, and skip the lttng variant.
-
-Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
----
- lttng-syscalls.c |    4 ++++
- 1 files changed, 4 insertions(+), 0 deletions(-)
-
-diff --git a/lttng-syscalls.c b/lttng-syscalls.c
-index 9de3c07..80d0f05 100644
---- a/lttng-syscalls.c
-+++ b/lttng-syscalls.c
-@@ -17,12 +17,16 @@
- #include "ltt-events.h"
- 
- #ifndef CONFIG_COMPAT
-+
-+#ifndef is_compat_task
- static inline int is_compat_task(void)
- {
- 	return 0;
- }
- #endif
- 
-+#endif
-+
- static
- void syscall_entry_probe(void *__data, struct pt_regs *regs, long id);
- 
--- 
-1.7.5.4
-
diff --git a/meta/recipes-kernel/lttng-2.0/lttng-modules_2.0.5.bb b/meta/recipes-kernel/lttng-2.0/lttng-modules_2.0.5.bb
index 4c78f36..96ab85f 100644
--- a/meta/recipes-kernel/lttng-2.0/lttng-modules_2.0.5.bb
+++ b/meta/recipes-kernel/lttng-2.0/lttng-modules_2.0.5.bb
@@ -10,13 +10,12 @@ DEPENDS = "virtual/kernel"
 
 inherit module
 
-SRCREV = "4d3e89e379fc66480d729abe8daa5c86eb585400"
-PV = "2.0.pre11+git${SRCREV}"
-PR = "r0"
+SRCREV = "cbb0f0f5203e9901084f38792e484f6fa988e10a"
+PV = "2.0.5+git${SRCREV}"
+PR = "r1"
 
 SRC_URI = "git://git.lttng.org/lttng-modules.git;protocol=git \
-           file://lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch \
-           file://lttng-sycalls-protect-is_compat_task-from-redefiniti.patch"
+           file://lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch"
 
 export INSTALL_MOD_DIR="kernel/lttng-modules"
 export KERNEL_SRC="${STAGING_KERNEL_DIR}"
-- 
1.7.11.4

[PATCH 0/1] replace lttng with lttng-2.0 in core-tools-profile

[tom.zanussi]

From: Tom Zanussi <tom.zanussi@intel.com>

The current packagegroup-core-tools-profile pulls in 'legacy' lttng
packages, which are useless with the linux-yocto kernels since those
don't include lttng support.

lttng-2.0 does seem to work, if the patch for lttng-modules that I just
sent to oe-core is applied as well (lttng-modules: update SRCREV and
remove is_compat_task).

I've only done minimal testing using qemux86, but even so it should at
least be better than the curent situation.

Sending this patch to poky instead of oe-core since it could be that
legacy lttng is what's being used by most people, but it clearly doesn't
work for yocto.

The following changes since commit 0260bb5c6978839c068007fcff2f704937805faf:

  gitignore: Fix for poky repository (2012-10-11 13:52:14 +0100)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib.git tzanussi/lttng-fixes.0
  http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=tzanussi/lttng-fixes.0

Tom Zanussi (2):
  lttng-modules: update SRCREV and remove is_compat_task patch
  packagegroup-core-tools-profile: replace 'legacy' lttng with lttng
    2.0

 .../packagegroup-core-tools-profile.bb             |  9 ++---
 ...ls-protect-is_compat_task-from-redefiniti.patch | 41 ----------------------
 .../lttng-2.0/lttng-modules_2.0.5.bb               |  9 +++--
 3 files changed, 9 insertions(+), 50 deletions(-)
 delete mode 100644 meta/recipes-kernel/lttng-2.0/lttng-modules/lttng-sycalls-protect-is_compat_task-from-redefiniti.patch

-- 
1.7.11.4

[PATCH 1/1] packagegroup-core-tools-profile: replace 'legacy' lttng with lttng 2.0

[tom.zanussi]

From: Tom Zanussi <tom.zanussi@intel.com>

packagegroup-core-tools-profile currently pulls in the 'legacy' lttng
packages, which are useless without legacy lttng support in the kernel.

This makes packagegroup-core-tools-profile pull in the lttng 2.0
packages instead, which don't need any kernel modifications to work.

Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>
---
 .../packagegroups/packagegroup-core-tools-profile.bb             | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
index 1035036..bf6e16e 100644
--- a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
+++ b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
@@ -5,7 +5,7 @@
 SUMMARY = "Profiling tools"
 LICENSE = "MIT"
 
-PR = "r1"
+PR = "r2"
 
 inherit packagegroup
 
@@ -29,8 +29,9 @@ PROFILETOOLS = "\
     oprofileui-server \
     powertop \
     latencytop \
-    lttng-control \
-    lttng-viewer"
+    lttng-tools \
+    lttng-modules \
+    babeltrace"
 
 # systemtap needs elfutils which is not fully buildable on uclibc
 # hence we exclude it from uclibc based builds
@@ -44,7 +45,7 @@ SYSTEMTAP_mips = ""
 # which means we can not use syscall() to call it. So we ignore
 # it for x86_64/uclibc
 
-LTTNGUST = "lttng-ust"
+LTTNGUST = "lttng2-ust"
 LTTNGUST_libc-uclibc = ""
 LTTNGUST_mips = ""
 
-- 
1.7.11.4

Re: [PATCH 1/1] packagegroup-core-tools-profile: replace 'legacy' lttng with lttng 2.0

[Bruce Ashfield]

On 12-10-12 03:53 PM, tom.zanussi@intel.com wrote:
> From: Tom Zanussi<tom.zanussi@intel.com>
>
> packagegroup-core-tools-profile currently pulls in the 'legacy' lttng
> packages, which are useless without legacy lttng support in the kernel.
>
> This makes packagegroup-core-tools-profile pull in the lttng 2.0
> packages instead, which don't need any kernel modifications to work.

Definitely the right thing to do, oe-core needs the email too though!

Acked-by: Bruce Ashfield <bruce.ashfield@windriver.com>

>
> Signed-off-by: Tom Zanussi<tom.zanussi@intel.com>
> ---
>   .../packagegroups/packagegroup-core-tools-profile.bb             | 9 +++++----
>   1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
> index 1035036..bf6e16e 100644
> --- a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
> +++ b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
> @@ -5,7 +5,7 @@
>   SUMMARY = "Profiling tools"
>   LICENSE = "MIT"
>
> -PR = "r1"
> +PR = "r2"
>
>   inherit packagegroup
>
> @@ -29,8 +29,9 @@ PROFILETOOLS = "\
>       oprofileui-server \
>       powertop \
>       latencytop \
> -    lttng-control \
> -    lttng-viewer"
> +    lttng-tools \
> +    lttng-modules \
> +    babeltrace"
>
>   # systemtap needs elfutils which is not fully buildable on uclibc
>   # hence we exclude it from uclibc based builds
> @@ -44,7 +45,7 @@ SYSTEMTAP_mips = ""
>   # which means we can not use syscall() to call it. So we ignore
>   # it for x86_64/uclibc
>
> -LTTNGUST = "lttng-ust"
> +LTTNGUST = "lttng2-ust"
>   LTTNGUST_libc-uclibc = ""
>   LTTNGUST_mips = ""
>

Re: [PATCH 1/1] packagegroup-core-tools-profile: replace 'legacy' lttng with lttng 2.0

[Bruce Ashfield]

On 12-10-12 04:11 PM, Bruce Ashfield wrote:
> On 12-10-12 03:53 PM, tom.zanussi@intel.com wrote:
>> From: Tom Zanussi<tom.zanussi@intel.com>
>>
>> packagegroup-core-tools-profile currently pulls in the 'legacy' lttng
>> packages, which are useless without legacy lttng support in the kernel.
>>
>> This makes packagegroup-core-tools-profile pull in the lttng 2.0
>> packages instead, which don't need any kernel modifications to work.
>
> Definitely the right thing to do, oe-core needs the email too though!

oops. Not for this profile.

Bruce

>
> Acked-by: Bruce Ashfield <bruce.ashfield@windriver.com>
>
>>
>> Signed-off-by: Tom Zanussi<tom.zanussi@intel.com>
>> ---
>> .../packagegroups/packagegroup-core-tools-profile.bb | 9 +++++----
>> 1 file changed, 5 insertions(+), 4 deletions(-)
>>
>> diff --git
>> a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
>> b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
>> index 1035036..bf6e16e 100644
>> --- a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
>> +++ b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
>> @@ -5,7 +5,7 @@
>> SUMMARY = "Profiling tools"
>> LICENSE = "MIT"
>>
>> -PR = "r1"
>> +PR = "r2"
>>
>> inherit packagegroup
>>
>> @@ -29,8 +29,9 @@ PROFILETOOLS = "\
>> oprofileui-server \
>> powertop \
>> latencytop \
>> - lttng-control \
>> - lttng-viewer"
>> + lttng-tools \
>> + lttng-modules \
>> + babeltrace"
>>
>> # systemtap needs elfutils which is not fully buildable on uclibc
>> # hence we exclude it from uclibc based builds
>> @@ -44,7 +45,7 @@ SYSTEMTAP_mips = ""
>> # which means we can not use syscall() to call it. So we ignore
>> # it for x86_64/uclibc
>>
>> -LTTNGUST = "lttng-ust"
>> +LTTNGUST = "lttng2-ust"
>> LTTNGUST_libc-uclibc = ""
>> LTTNGUST_mips = ""
>>
>
> _______________________________________________
> poky mailing list
> poky@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/poky

Re: [PATCH 1/1] lttng-modules: update SRCREV and remove is_compat_task patch

[Bruce Ashfield]

On 12-10-12 03:52 PM, tom.zanussi@intel.com wrote:
> From: Tom Zanussi<tom.zanussi@intel.com>
>
> The existing SRCREV doesn't point to v2.0.5 as the recipe version
> suggests, and is actually building modules that won't work with the
> lttng-tools it's supposed to be matching.
>
> Also, remove the is_compat_task patch for lttng-syscalls.c to avoid
> do_patch errors, as recent versions of lttng-modules have added this
> check.

Acked-by: Bruce Ashfield <bruce.ashfield@windriver.com>

>
> Signed-off-by: Tom Zanussi<tom.zanussi@intel.com>
> ---
>   ...ls-protect-is_compat_task-from-redefiniti.patch | 41 ----------------------
>   .../lttng-2.0/lttng-modules_2.0.5.bb               |  9 +++--
>   2 files changed, 4 insertions(+), 46 deletions(-)
>   delete mode 100644 meta/recipes-kernel/lttng-2.0/lttng-modules/lttng-sycalls-protect-is_compat_task-from-redefiniti.patch
>
> diff --git a/meta/recipes-kernel/lttng-2.0/lttng-modules/lttng-sycalls-protect-is_compat_task-from-redefiniti.patch b/meta/recipes-kernel/lttng-2.0/lttng-modules/lttng-sycalls-protect-is_compat_task-from-redefiniti.patch
> deleted file mode 100644
> index 0056633..0000000
> --- a/meta/recipes-kernel/lttng-2.0/lttng-modules/lttng-sycalls-protect-is_compat_task-from-redefiniti.patch
> +++ /dev/null
> @@ -1,41 +0,0 @@
> -From 415a1a678cb2fcf603f9777f2c7b71c1e38f8101 Mon Sep 17 00:00:00 2001
> -From: Bruce Ashfield<bruce.ashfield@windriver.com>
> -Date: Mon, 19 Mar 2012 13:30:31 -0400
> -Subject: [PATCH] lttng-sycalls: protect is_compat_task from redefinition
> -
> -recent -stable kernels have their own fallback definition of is_compat_task
> -
> -   #define is_compat_task() (0)
> -
> -To protect against this double definition of is_compat_task we can check
> -to see if it is already defined, and skip the lttng variant.
> -
> -Signed-off-by: Bruce Ashfield<bruce.ashfield@windriver.com>
> ----
> - lttng-syscalls.c |    4 ++++
> - 1 files changed, 4 insertions(+), 0 deletions(-)
> -
> -diff --git a/lttng-syscalls.c b/lttng-syscalls.c
> -index 9de3c07..80d0f05 100644
> ---- a/lttng-syscalls.c
> -+++ b/lttng-syscalls.c
> -@@ -17,12 +17,16 @@
> - #include "ltt-events.h"
> -
> - #ifndef CONFIG_COMPAT
> -+
> -+#ifndef is_compat_task
> - static inline int is_compat_task(void)
> - {
> - 	return 0;
> - }
> - #endif
> -
> -+#endif
> -+
> - static
> - void syscall_entry_probe(void *__data, struct pt_regs *regs, long id);
> -
> ---
> -1.7.5.4
> -
> diff --git a/meta/recipes-kernel/lttng-2.0/lttng-modules_2.0.5.bb b/meta/recipes-kernel/lttng-2.0/lttng-modules_2.0.5.bb
> index 4c78f36..96ab85f 100644
> --- a/meta/recipes-kernel/lttng-2.0/lttng-modules_2.0.5.bb
> +++ b/meta/recipes-kernel/lttng-2.0/lttng-modules_2.0.5.bb
> @@ -10,13 +10,12 @@ DEPENDS = "virtual/kernel"
>
>   inherit module
>
> -SRCREV = "4d3e89e379fc66480d729abe8daa5c86eb585400"
> -PV = "2.0.pre11+git${SRCREV}"
> -PR = "r0"
> +SRCREV = "cbb0f0f5203e9901084f38792e484f6fa988e10a"
> +PV = "2.0.5+git${SRCREV}"
> +PR = "r1"
>
>   SRC_URI = "git://git.lttng.org/lttng-modules.git;protocol=git \
> -           file://lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch \
> -           file://lttng-sycalls-protect-is_compat_task-from-redefiniti.patch"
> +           file://lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch"
>
>   export INSTALL_MOD_DIR="kernel/lttng-modules"
>   export KERNEL_SRC="${STAGING_KERNEL_DIR}"

Re: [PATCH 0/1] lttng-2.0 fixes

[Bruce Ashfield]

On 12-10-12 03:52 PM, tom.zanussi@intel.com wrote:
> From: Tom Zanussi<tom.zanussi@intel.com>
>
> When trying out the current lttng-2.0, I found that it didn't work in
> its current state.  This patch makes it work again, or at least a
> a quick test on qemux86 works:
>
> Before:
>
> root@qemux86:~# lttng create
> Spawning a session daemon
> FATAL: Module lttng-tracer not found.
> Error: Unable to load module lttng-tracer
> Error: Unable to remove module lttng-tracer
> Warning: No kernel tracer available
> Session auto-20121012-190712 created.
> Traces will be written in /home/root/lttng-traces/auto-20121012-190712
>
> After:
>
> root@qemux86:~# lttng create
> Spawning a session daemon
> Session auto-20121012-192342 created.
> Traces will be written in /home/root/lttng-traces/auto-20121012-192342
> root@qemux86:~# lttng enable-event --kernel --all
> All kernel events are enabled in channel channel0
> root@qemux86:~# lttng start
> Tracing started for session auto-20121012-192342
> root@qemux86:~# lttng stop
> Tracing stopped for session auto-20121012-192342
> root@qemux86:~# lttng view
> [19:27:01.661844467] (+?.?????????) hrtimer_cancel: { 0 }, { hrtimer = 3354328992 }
> [19:27:01.661854316] (+0.000009849) hrtimer_expire_entry: { 0 }, { hrtimer = 3354328992, now = \
> 318684045266, function = 3238494992 }
> [19:27:01.661870669] (+0.000016353) softirq_raise: { 0 }, { vec = 1 }
> [19:27:01.661876340] (+0.000005671) softirq_raise: { 0 }, { vec = 9 }
> [19:27:01.661887892] (+0.000011552) sched_stat_runtime: { 0 }, { comm = "lttng-sessiond", tid =
>
>

That's what I'd expect.

Looks good from here.

Bruce

> The following changes since commit 0260bb5c6978839c068007fcff2f704937805faf:
>
>    gitignore: Fix for poky repository (2012-10-11 13:52:14 +0100)
>
> are available in the git repository at:
>
>    git://git.yoctoproject.org/poky-contrib.git tzanussi/lttng-fixes.0
>    http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=tzanussi/lttng-fixes.0
>
> Tom Zanussi (2):
>    lttng-modules: update SRCREV and remove is_compat_task patch
>
>   ...ls-protect-is_compat_task-from-redefiniti.patch | 41 ----------------------
>   .../lttng-2.0/lttng-modules_2.0.5.bb               |  9 +++--
>   2 files changed, 9 insertions(+), 50 deletions(-)
>   delete mode 100644 meta/recipes-kernel/lttng-2.0/lttng-modules/lttng-sycalls-protect-is_compat_task-from-redefiniti.patch
>

Re: [PATCH 1/1] packagegroup-core-tools-profile: replace 'legacy' lttng with lttng 2.0

[Saul Wold]

On 10/12/2012 12:53 PM, tom.zanussi@intel.com wrote:
> From: Tom Zanussi <tom.zanussi@intel.com>
>
> packagegroup-core-tools-profile currently pulls in the 'legacy' lttng
> packages, which are useless without legacy lttng support in the kernel.
>
> This makes packagegroup-core-tools-profile pull in the lttng 2.0
> packages instead, which don't need any kernel modifications to work.
>
> Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>
> ---
>   .../packagegroups/packagegroup-core-tools-profile.bb             | 9 +++++----
>   1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
> index 1035036..bf6e16e 100644
> --- a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
> +++ b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
> @@ -5,7 +5,7 @@
>   SUMMARY = "Profiling tools"
>   LICENSE = "MIT"
>
> -PR = "r1"
> +PR = "r2"
>
>   inherit packagegroup
>
> @@ -29,8 +29,9 @@ PROFILETOOLS = "\
>       oprofileui-server \
>       powertop \
>       latencytop \
> -    lttng-control \
> -    lttng-viewer"
> +    lttng-tools \
> +    lttng-modules \
> +    babeltrace"
>
Not sure this will work since babeltrace is not buildable for mips machines.

Sau!

>   # systemtap needs elfutils which is not fully buildable on uclibc
>   # hence we exclude it from uclibc based builds
> @@ -44,7 +45,7 @@ SYSTEMTAP_mips = ""
>   # which means we can not use syscall() to call it. So we ignore
>   # it for x86_64/uclibc
>
> -LTTNGUST = "lttng-ust"
> +LTTNGUST = "lttng2-ust"
>   LTTNGUST_libc-uclibc = ""
>   LTTNGUST_mips = ""
>
>

Re: [PATCH 1/1] packagegroup-core-tools-profile: replace 'legacy' lttng with lttng 2.0

[Darren Hart]

On 10/15/2012 04:51 PM, Saul Wold wrote:
> On 10/12/2012 12:53 PM, tom.zanussi@intel.com wrote:
>> From: Tom Zanussi <tom.zanussi@intel.com>
>>
>> packagegroup-core-tools-profile currently pulls in the 'legacy' lttng
>> packages, which are useless without legacy lttng support in the kernel.
>>
>> This makes packagegroup-core-tools-profile pull in the lttng 2.0
>> packages instead, which don't need any kernel modifications to work.
>>
>> Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>
>> ---
>>   .../packagegroups/packagegroup-core-tools-profile.bb             | 9 +++++----
>>   1 file changed, 5 insertions(+), 4 deletions(-)
>>
>> diff --git a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
>> index 1035036..bf6e16e 100644
>> --- a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
>> +++ b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
>> @@ -5,7 +5,7 @@
>>   SUMMARY = "Profiling tools"
>>   LICENSE = "MIT"
>>
>> -PR = "r1"
>> +PR = "r2"
>>
>>   inherit packagegroup
>>
>> @@ -29,8 +29,9 @@ PROFILETOOLS = "\
>>       oprofileui-server \
>>       powertop \
>>       latencytop \
>> -    lttng-control \
>> -    lttng-viewer"
>> +    lttng-tools \
>> +    lttng-modules \
>> +    babeltrace"
>>
> Not sure this will work since babeltrace is not buildable for mips machines.

I guess that means we just need a BABELTRACE and BABELTRACE_mips
definition? Or perhaps a set for LTTNG similar to those below for LTTNGUST ?

--
Darren

> 
> Sau!
> 
>>   # systemtap needs elfutils which is not fully buildable on uclibc
>>   # hence we exclude it from uclibc based builds
>> @@ -44,7 +45,7 @@ SYSTEMTAP_mips = ""
>>   # which means we can not use syscall() to call it. So we ignore
>>   # it for x86_64/uclibc
>>
>> -LTTNGUST = "lttng-ust"
>> +LTTNGUST = "lttng2-ust"
>>   LTTNGUST_libc-uclibc = ""
>>   LTTNGUST_mips = ""
>>
>>

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Technical Lead - Linux Kernel

Re: [PATCH 1/1] packagegroup-core-tools-profile: replace 'legacy' lttng with lttng 2.0

[Tom Zanussi]

On Mon, 2012-10-15 at 16:51 -0700, Saul Wold wrote:
> On 10/12/2012 12:53 PM, tom.zanussi@intel.com wrote:
> > From: Tom Zanussi <tom.zanussi@intel.com>
> >
> > packagegroup-core-tools-profile currently pulls in the 'legacy' lttng
> > packages, which are useless without legacy lttng support in the kernel.
> >
> > This makes packagegroup-core-tools-profile pull in the lttng 2.0
> > packages instead, which don't need any kernel modifications to work.
> >
> > Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>
> > ---
> >   .../packagegroups/packagegroup-core-tools-profile.bb             | 9 +++++----
> >   1 file changed, 5 insertions(+), 4 deletions(-)
> >
> > diff --git a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
> > index 1035036..bf6e16e 100644
> > --- a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
> > +++ b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
> > @@ -5,7 +5,7 @@
> >   SUMMARY = "Profiling tools"
> >   LICENSE = "MIT"
> >
> > -PR = "r1"
> > +PR = "r2"
> >
> >   inherit packagegroup
> >
> > @@ -29,8 +29,9 @@ PROFILETOOLS = "\
> >       oprofileui-server \
> >       powertop \
> >       latencytop \
> > -    lttng-control \
> > -    lttng-viewer"
> > +    lttng-tools \
> > +    lttng-modules \
> > +    babeltrace"
> >
> Not sure this will work since babeltrace is not buildable for mips machines.
> 

Yeah, I guess things like this are another reason it hasn't been
switched over (though I wouldn't have expected a userspace translation
tool to be the thing that choked on mips).

So might as well hold off on this until I can test lttng on all the
arches.

Tom

> Sau!
> 
> >   # systemtap needs elfutils which is not fully buildable on uclibc
> >   # hence we exclude it from uclibc based builds
> > @@ -44,7 +45,7 @@ SYSTEMTAP_mips = ""
> >   # which means we can not use syscall() to call it. So we ignore
> >   # it for x86_64/uclibc
> >
> > -LTTNGUST = "lttng-ust"
> > +LTTNGUST = "lttng2-ust"
> >   LTTNGUST_libc-uclibc = ""
> >   LTTNGUST_mips = ""
> >
> >

[PATCH 1/1] packagegroup-core-tools-profile: replace 'legacy' lttng with lttng 2.0

[tom.zanussi]

From: Tom Zanussi <tom.zanussi@intel.com>

packagegroup-core-tools-profile currently pulls in the 'legacy' lttng
packages, which are useless without legacy lttng support in the kernel.

This makes packagegroup-core-tools-profile pull in the lttng 2.0
packages instead, which don't need any kernel modifications to work.

Signed-off-by: Tom Zanussi <tom.zanussi@intel.com>
---
 .../packagegroup-core-tools-profile.bb             | 22 ++++++++++++++++++----
 1 file changed, 18 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
index 1035036..0ab2bc9 100644
--- a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
+++ b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
@@ -5,7 +5,7 @@
 SUMMARY = "Profiling tools"
 LICENSE = "MIT"
 
-PR = "r1"
+PR = "r2"
 
 inherit packagegroup
 
@@ -29,8 +29,7 @@ PROFILETOOLS = "\
     oprofileui-server \
     powertop \
     latencytop \
-    lttng-control \
-    lttng-viewer"
+    "
 
 # systemtap needs elfutils which is not fully buildable on uclibc
 # hence we exclude it from uclibc based builds
@@ -44,10 +43,22 @@ SYSTEMTAP_mips = ""
 # which means we can not use syscall() to call it. So we ignore
 # it for x86_64/uclibc
 
-LTTNGUST = "lttng-ust"
+LTTNGUST = "lttng2-ust"
 LTTNGUST_libc-uclibc = ""
 LTTNGUST_mips = ""
 
+# lttng-tools, lttng-modules and babeltrace all depend on liburcu
+# which currentl doesn't build on mips
+
+LTTNGTOOLS = "lttng-tools"
+LTTNGTOOLS_mips = ""
+
+LTTNGMODULES = "lttng-modules"
+LTTNGMODULES_mips = ""
+
+BABELTRACE = "babeltrace"
+BABELTRACE_mips = ""
+
 # valgrind does not work on mips
 
 VALGRIND = "valgrind"
@@ -65,6 +76,9 @@ VALGRIND_arm = ""
 RDEPENDS_${PN} = "\
     ${PROFILETOOLS} \
     ${LTTNGUST} \
+    ${LTTNGTOOLS} \
+    ${LTTNGMODULES} \
+    ${BABELTRACE} \
     ${SYSTEMTAP} \
     ${VALGRIND} \
     "
-- 
1.7.11.4