All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/4]lttng-2.0 new recipe
@ 2012-01-06  6:11 Zumeng Chen
  2012-01-06  6:11 ` [PATCH 1/4] lttng-ust: Integrate into yocto linux Zumeng Chen
                   ` (3 more replies)
  0 siblings, 4 replies; 22+ messages in thread
From: Zumeng Chen @ 2012-01-06  6:11 UTC (permalink / raw)
  To: yocto, richard.purdie, bruce.ashfield, sgw

Hi All,

This patchset adds support for lttng-2.0, including the folloing
four bb files:
  babeltrace_0.8.bb
  lttng-modules_v2.0.pre11.bb
  lttng-tools_2.0-pre16.bb
  lttng-ust_1.9.4.bb
And two patches for smoothly compiling:
  lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch
  lttng-tools-fix-compiling-error-for-powerpc-arm-and-mips.patch

There is a clear description of the LTTng 2.0 architecture in the
followiing site: 
  http://lttng.org/sites/lttng.org/files/LTTng2_0Architecture_pa3.pdf

Currently lttng-2.0 is still in pre-release phrase, so I add this
into the patch header for reminder. And another goal of this RR 
is intended to bring forward the following issues:

  lttng-2.0 has been added into recipies-kernel, which coexisted
  with lttng(the previous version), because there are some mainly
  changes from lttng-control to lttng-tools, which both monitors
  tracepoints and dynamic probes from kernel and userspace. But
  I'm not sure if we should remove the previous version lttng.
  My suggestion is to preserve for 2.6.37 kernel :-)

lttng-2.0 supports lttng-modules extra builds, which don't need
any patches on Linux kernel vs the previous version of lttng.
As described in README of lttng-modules-2.0, so far, it has been
tested in current 3.0 kernel on x86 32/64-bit, and powerpc 32-bit,
, build tested on ARM. So the related tests on qemux86 and qemuppc
has been validated and a build test on qemuarm passed too with
this patcheset.

Babeltrace provides as a trace converter, which used to convert
LTTng 2.0 traces into human-readable log(see below example for
qemuppc)

lttng-ust_1.9.4 is Linux Trace Toolkit 2.0 Userspace Tracer.

I'm still keep an eye on the development of lttng for updates.

Regard,
Zumeng

root@qemuppc:/# lttng list  -k
Kernel events:
-------------
      sched_kthread_stop (type: tracepoint)
      sched_kthread_stop_ret (type: tracepoint)
      sched_wakeup (type: tracepoint)
      sched_wakeup_new (type: tracepoint)
      sched_switch (type: tracepoint)
      sched_migrate_task (type: tracepoint)
      sched_process_free (type: tracepoint)
      sched_process_exit (type: tracepoint)
      sched_wait_task (type: tracepoint)
      sched_process_wait (type: tracepoint)
      sched_process_fork (type: tracepoint)
      sched_stat_wait (type: tracepoint)
      sched_stat_sleep (type: tracepoint)
      sched_stat_iowait (type: tracepoint)
      sched_stat_runtime (type: tracepoint)
      sched_pi_setprio (type: tracepoint)
      irq_handler_entry (type: tracepoint)
      irq_handler_exit (type: tracepoint)
      softirq_entry (type: tracepoint)
      softirq_exit (type: tracepoint)
      softirq_raise (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]
       (type: tracepoint) [disabled]

root@qemuppc:/# lttng create mysession
Session mysession created.
Traces will be written in /home/root/lttng-traces/mysession-20111230-152610
root@qemuppc:/#  lttng enable-event -a -k
All kernel events are enabled in channel channel0
root@qemuppc:/# lttng enable-event sched_switch,sched_wakeup -k
kernel event sched_switch created in channel channel0
kernel event sched_wakeup created in channel channel0
root@qemuppc:/# lttng enable-event -a -k --tracepoint
All kernel tracepoints are enabled in channel channel0
root@qemuppc:/# lttng add-context -k -e sched_switch -t pid
kernel context pid added to sched_switch event in all channels
root@qemuppc:/# lttng start
Tracing started for session mysession
root@qemuppc:/#  lttng list -u
UST events:
-------------
None

root@qemuppc:/# lttng stop
Tracing stopped for session mysession
root@qemuppc:/# lttng list -u
UST events:
-------------
None

root@qemuppc:/# uname -a
Linux qemuppc 3.0.12-yocto-standard+ #1 PREEMPT Thu Dec 29 22:23:52 CST 2011 ppc GNU/Linux
root@qemuppc:/# 
root@qemuppc:/# babeltrace /home/root/lttng-traces/mysession-20111230-152610 | less
[2078772597067] softirq_raise: { 0 }, { vec = 1 }
[2078773657367] softirq_raise: { 0 }, { vec = 9 }
[2078773943557] sched_stat_runtime: { 0 }, { comm = "lttng-sessiond", tid = 404, runtime = 5104592, vruntime = 61173667721 }
[2078774051587] softirq_entry: { 0 }, { vec = 1 }
[2078774081797] softirq_exit: { 0 }, { vec = 1 }
[2078774086557] softirq_entry: { 0 }, { vec = 9 }
[2078774094247] softirq_exit: { 0 }, { vec = 9 }
[2078774105437] sched_stat_runtime: { 0 }, { comm = "lttng-sessiond", tid = 404, runtime = 230192, vruntime = 61173897913 }
[2078774185387] sched_stat_wait: { 0 }, { comm = "lttng-consumerd", tid = 418, delay = 5334784 }
[2078774328677] sched_switch: { 0 }, 385, { prev_comm = "lttng-sessiond", prev_tid = 404, prev_prio = 20, prev_state = 0, next_comm = "lttng-consumerd", next_tid = 418, next_prio = 20 }
[2078774802747] exit_syscall: { 0 }, { ret = 1 }
[2078774968937] sys_unknown: { 0 }, { id = 102, args = [ [0] = 17, [1] = 1218120660, [2] = 0, [3] = 268526120, [4] = 32, [5] = 4294967232 ] }
[2078775075597] exit_syscall: { 0 }, { ret = 4136 }
[2078775109747] sys_unknown: { 0 }, { id = 167, args = [ [0] = 1218124968, [1] = 2, [2] = 4294967295, [3] = 268526120, [4] = 32, [5] = 4294967232 ] }
[2078775130867] exit_syscall: { 0 }, { ret = 1 }
[2078775339007] sys_unknown: { 0 }, { id = 102, args = [ [0] = 17, [1] = 1218120612, [2] = 0, [3] = 268526120, [4] = 32, [5] = 4294967232 ] }
[2078776095757] exit_syscall: { 0 }, { ret = 1 }
[2078776238357] softirq_raise: { 0 }, { vec = 1 }
[2078776245827] softirq_raise: { 0 }, { vec = 9 }
[2078776265487] sched_stat_runtime: { 0 }, { comm = "lttng-consumerd", tid = 418, runtime = 2149408, vruntime = 61169846169 }
[2078776291187] softirq_entry: { 0 }, { vec = 1 }
[2078776306217] softirq_exit: { 0 }, { vec = 1 }
[2078776310607] softirq_entry: { 0 }, { vec = 9 }
[2078776317337] softirq_exit: { 0 }, { vec = 9 }
....snip..........................

The following changes since commit 5797feac5f06f5cc363869cd440b82b3eaafd456:

  image-mklibs/package_ipk: Remove bashisms (2012-01-05 22:26:25 +0000)

are available in the git repository at:
  git://git.pokylinux.org/poky-contrib zumeng/lttng-2.0
  http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=zumeng/lttng-2.0

Zumeng Chen (4):
  lttng-ust: Integrate into yocto linux
  lttng-tools: Integrated from git repo.
  lttng-modules: from lttng-modules 2.0 git repo.
  babeltrace: Integrated from version 0.8

 meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb    |   30 ++++++++++++++
 ...modules-replace-KERNELDIR-with-KERNEL_SRC.patch |   41 ++++++++++++++++++++
 .../lttng-2.0/lttng-modules_v2.0.pre11.bb          |   23 +++++++++++
 ...-compiling-error-for-powerpc-arm-and-mips.patch |   23 +++++++++++
 .../lttng-2.0/lttng-tools_2.0-pre16.bb             |   24 +++++++++++
 meta/recipes-kernel/lttng-2.0/lttng-ust_1.9.4.bb   |   23 +++++++++++
 6 files changed, 164 insertions(+), 0 deletions(-)
 create mode 100644 meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-modules_v2.0.pre11.bb
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm-and-mips.patch
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-ust_1.9.4.bb

-- 
1.7.5.4



^ permalink raw reply	[flat|nested] 22+ messages in thread

* [PATCH 1/4] lttng-ust: Integrate into yocto linux
  2012-01-06  6:11 [PATCH 0/4]lttng-2.0 new recipe Zumeng Chen
@ 2012-01-06  6:11 ` Zumeng Chen
  2012-01-07  2:01   ` Darren Hart
  2012-01-06  6:11 ` [PATCH 2/4] lttng-tools: Integrated from git repo Zumeng Chen
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 22+ messages in thread
From: Zumeng Chen @ 2012-01-06  6:11 UTC (permalink / raw)
  To: yocto, richard.purdie, bruce.ashfield, sgw

Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
---
 meta/recipes-kernel/lttng-2.0/lttng-ust_1.9.4.bb |   23 ++++++++++++++++++++++
 1 files changed, 23 insertions(+), 0 deletions(-)
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-ust_1.9.4.bb

diff --git a/meta/recipes-kernel/lttng-2.0/lttng-ust_1.9.4.bb b/meta/recipes-kernel/lttng-2.0/lttng-ust_1.9.4.bb
new file mode 100644
index 0000000..22d39df
--- /dev/null
+++ b/meta/recipes-kernel/lttng-2.0/lttng-ust_1.9.4.bb
@@ -0,0 +1,23 @@
+SUMMARY = "Linux Trace Toolkit Userspace Tracer 2.0"
+DESCRIPTION = "The LTTng UST 2.0 package contains the userspace tracer library to trace userspace codes."
+HOMEPAGE = "http://lttng.org/lttng2.0"
+BUGTRACKER = "n/a"
+
+LICENSE = "LGPLv2.1+ & BSD"
+LIC_FILES_CHKSUM = "file://COPYING;md5=e647752e045a8c45b6f583771bd561ef"
+
+inherit autotools
+
+DEPENDS = "liburcu util-linux"
+
+SRC_URI = "http://lttng.org/files/lttng-ust/lttng-ust-${PV}.tar.gz"
+
+SRC_URI[md5sum] = "0f09bb954c99cb99ab6997cd492953c1"
+SRC_URI[sha256sum] = "e9ec3c55c2251b3f3b512bfa1015b0f67e35d55793850ccea5134f7b24ee3ba1"
+
+S = "${WORKDIR}/lttng-ust-${PV}"
+
+# Due to liburcu not building on MIPS currently this recipe needs to
+# be limited also.
+# So here let us first suppport x86/arm/powerpc platforms now.
+COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'
-- 
1.7.5.4



^ permalink raw reply related	[flat|nested] 22+ messages in thread

* [PATCH 2/4] lttng-tools: Integrated from git repo.
  2012-01-06  6:11 [PATCH 0/4]lttng-2.0 new recipe Zumeng Chen
  2012-01-06  6:11 ` [PATCH 1/4] lttng-ust: Integrate into yocto linux Zumeng Chen
@ 2012-01-06  6:11 ` Zumeng Chen
  2012-01-06  6:27   ` Zumeng Chen
  2012-01-07  2:05   ` Darren Hart
  2012-01-06  6:11 ` [PATCH 3/4] lttng-modules: from lttng-modules 2.0 " Zumeng Chen
  2012-01-06  6:11 ` [PATCH 4/4] babeltrace: Integrated from version 0.8 Zumeng Chen
  3 siblings, 2 replies; 22+ messages in thread
From: Zumeng Chen @ 2012-01-06  6:11 UTC (permalink / raw)
  To: yocto, richard.purdie, bruce.ashfield, sgw

lttng-tools 2.0 is currently in pre-release phase (-pre16).

Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
---
 ...-compiling-error-for-powerpc-arm-and-mips.patch |   23 +++++++++++++++++++
 .../lttng-2.0/lttng-tools_2.0-pre16.bb             |   24 ++++++++++++++++++++
 2 files changed, 47 insertions(+), 0 deletions(-)
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm-and-mips.patch
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb

diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm-and-mips.patch b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm-and-mips.patch
new file mode 100644
index 0000000..c971e78
--- /dev/null
+++ b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm-and-mips.patch
@@ -0,0 +1,23 @@
+Upstream-Status: Pending
+
+lttng-tools: fix compiling error for powerpc, arm
+
+Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
+---
+ librunas/runas.c |    1 +
+ 1 files changed, 1 insertions(+), 0 deletions(-)
+
+diff --git a/librunas/runas.c b/librunas/runas.c
+--- a/librunas/runas.c
++++ b/librunas/runas.c
+@@ -29,6 +29,7 @@
+ #include <fcntl.h>
+ #include <sched.h>
+ #include <sys/mman.h>
++#include <asm/mman.h>
+ 
+ #include <lttngerr.h>
+ 
+-- 
+1.7.5.4
+
diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
new file mode 100644
index 0000000..0b7a240
--- /dev/null
+++ b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
@@ -0,0 +1,24 @@
+SECTION = "devel"
+SUMMARY = "Linux Trace Toolkit Control"
+DESCRIPTION = "The Linux trace toolkit is a suite of tools designed to \
+extract program execution details from the Linux operating system and  \
+interpret them."
+LICENSE = "GPLv2.1"
+LIC_FILES_CHKSUM = "file://COPYING;md5=ab682a4729389c3f11913d758affe98e"
+
+DEPENDS = "liburcu popt lttng-ust"
+
+SRC_URI = "http://lttng.org/files/lttng-tools/lttng-tools-${PV}.tar.bz2 \
+	   file://fix-compiling-error-for-powerpc-arm-and-mips.patch"
+
+SRC_URI[md5sum] = "58ed0a15db80a00172d5b0413a55451c" 
+SRC_URI[sha256sum] = "188d262bda80043dd939c9aa7aec1795ca77aeec0cb3c0f23c28bfe754049491"
+
+S = "${WORKDIR}/lttng-tools-${PV}"
+
+
+inherit autotools
+
+export KERNELDIR="${STAGING_KERNEL_DIR}"
+
+COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'
-- 
1.7.5.4



^ permalink raw reply related	[flat|nested] 22+ messages in thread

* [PATCH 3/4] lttng-modules: from lttng-modules 2.0 git repo.
  2012-01-06  6:11 [PATCH 0/4]lttng-2.0 new recipe Zumeng Chen
  2012-01-06  6:11 ` [PATCH 1/4] lttng-ust: Integrate into yocto linux Zumeng Chen
  2012-01-06  6:11 ` [PATCH 2/4] lttng-tools: Integrated from git repo Zumeng Chen
@ 2012-01-06  6:11 ` Zumeng Chen
  2012-01-07  2:10   ` Darren Hart
  2012-01-06  6:11 ` [PATCH 4/4] babeltrace: Integrated from version 0.8 Zumeng Chen
  3 siblings, 1 reply; 22+ messages in thread
From: Zumeng Chen @ 2012-01-06  6:11 UTC (permalink / raw)
  To: yocto, richard.purdie, bruce.ashfield, sgw

LTTng-modules 2.0 is currently in pre-release phase (-pre11)

Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
---
 ...modules-replace-KERNELDIR-with-KERNEL_SRC.patch |   41 ++++++++++++++++++++
 .../lttng-2.0/lttng-modules_v2.0.pre11.bb          |   23 +++++++++++
 2 files changed, 64 insertions(+), 0 deletions(-)
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-modules_v2.0.pre11.bb

diff --git a/meta/recipes-kernel/lttng-2.0/lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch b/meta/recipes-kernel/lttng-2.0/lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch
new file mode 100644
index 0000000..b7b2fbb
--- /dev/null
+++ b/meta/recipes-kernel/lttng-2.0/lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch
@@ -0,0 +1,41 @@
+Upstream-Status: Inappropriate [embedded specific]
+
+lttng-modules: replace KERNELDIR with KERNEL_SRC
+
+Since lttng-modules uses the default way of module.bbclass to
+build and install lttng-modules, so we do this replacement for
+it as-is.
+
+Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
+---
+ Makefile |    7 +++----
+ 1 files changed, 3 insertions(+), 4 deletions(-)
+
+diff --git a/Makefile b/Makefile
+index 5ac13d7..25caad5 100644
+--- a/Makefile
++++ b/Makefile
+@@ -38,17 +38,16 @@ obj-m += lib/
+ endif
+ 
+ else
+-	KERNELDIR ?= /lib/modules/$(shell uname -r)/build
+ 	PWD := $(shell pwd)
+ 	CFLAGS = $(EXTCFLAGS)
+ 
+ default:
+-	$(MAKE) -C $(KERNELDIR) M=$(PWD) modules
++	$(MAKE) -C $(KERNEL_SRC) M=$(PWD) modules
+ 
+ modules_install:
+-	$(MAKE) -C $(KERNELDIR) M=$(PWD) modules_install
++	$(MAKE) -C $(KERNEL_SRC) M=$(PWD) modules_install
+ 
+ clean:
+-	$(MAKE) -C $(KERNELDIR) M=$(PWD) clean
++	$(MAKE) -C $(KERNEL_SRC) M=$(PWD) clean
+ 
+ endif
+-- 
+1.7.5.4
+
diff --git a/meta/recipes-kernel/lttng-2.0/lttng-modules_v2.0.pre11.bb b/meta/recipes-kernel/lttng-2.0/lttng-modules_v2.0.pre11.bb
new file mode 100644
index 0000000..34af847
--- /dev/null
+++ b/meta/recipes-kernel/lttng-2.0/lttng-modules_v2.0.pre11.bb
@@ -0,0 +1,23 @@
+SECTION = "devel"
+SUMMARY = "Linux Trace Toolkit KERNEL MODULE"
+DESCRIPTION = "The lttng-modules 2.0 package contains the kernel tracer modules"
+LICENSE = "LGPLv2.1 & GPLv2"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=1eb086682a7c65a45acd9bcdf6877b3e \
+		   "file://gpl-2.0.txt;md5=751419260aa954499f7abaabaa882bbe \
+		   "file://lgpl-2.1.txt;md5=243b725d71bb5df4a1e5920b344b86ad"
+
+DEPENDS = virtual/kernel 
+
+inherit module
+
+SRCREV = "4d3e89e379fc66480d729abe8daa5c86eb585400"
+PV = "v2.0.pre11+git${SRCPV}"
+PR = "r0"
+
+SRC_URI += "git://git.lttng.org/lttng-modules.git;protocol=git \
+		file://lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch"
+
+export INSTALL_MOD_DIR="kernel/lttng-modules"
+
+S = "${WORKDIR}/git"
+COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'
-- 
1.7.5.4



^ permalink raw reply related	[flat|nested] 22+ messages in thread

* [PATCH 4/4] babeltrace: Integrated from version 0.8
  2012-01-06  6:11 [PATCH 0/4]lttng-2.0 new recipe Zumeng Chen
                   ` (2 preceding siblings ...)
  2012-01-06  6:11 ` [PATCH 3/4] lttng-modules: from lttng-modules 2.0 " Zumeng Chen
@ 2012-01-06  6:11 ` Zumeng Chen
  2012-01-07 15:54   ` Darren Hart
  3 siblings, 1 reply; 22+ messages in thread
From: Zumeng Chen @ 2012-01-06  6:11 UTC (permalink / raw)
  To: yocto, richard.purdie, bruce.ashfield, sgw

It provides trace read and write libraries, as well as a trace converter.

Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
---
 meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb |   30 +++++++++++++++++++++++
 1 files changed, 30 insertions(+), 0 deletions(-)
 create mode 100644 meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb

diff --git a/meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb b/meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb
new file mode 100644
index 0000000..86a9d01
--- /dev/null
+++ b/meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb
@@ -0,0 +1,30 @@
+SUMMARY = "Babeltrace - Trace Format Babel Tower"
+DESCRIPTION = "Babeltrace provides trace read and write libraries in host side,\
+	       as well as a trace converter, which used to convert LTTng 2.0 \
+	       traces into human-readable log."
+HOMEPAGE = "http://www.efficios.com/babeltrace/"
+BUGTRACKER = "n/a"
+
+LICENSE = "MIT & GPLV2"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=8d1a03b3c17bdd158b3cbb34813b1423"
+
+inherit autotools
+
+DEPENDS = "gtk+ util-linux"
+
+SRCREV = "31e670293588d7882a232bc14b93d5e956fa43f4"
+PV = "0.8+git${SRCPV}"
+PR = "r0"
+
+SRC_URI += "git://git.efficios.com/babeltrace.git;protocol=git"
+
+S = "${WORKDIR}/git"
+
+do_configure_prepend () {
+  ${S}/bootstrap
+}
+
+# Due to liburcu not building on MIPS currently this recipe needs to
+# be limited also.
+# So here let us first suppport x86/arm/powerpc platforms now.
+COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'
-- 
1.7.5.4



^ permalink raw reply related	[flat|nested] 22+ messages in thread

* Re: [PATCH 2/4] lttng-tools: Integrated from git repo.
  2012-01-06  6:11 ` [PATCH 2/4] lttng-tools: Integrated from git repo Zumeng Chen
@ 2012-01-06  6:27   ` Zumeng Chen
  2012-01-07  2:05   ` Darren Hart
  1 sibling, 0 replies; 22+ messages in thread
From: Zumeng Chen @ 2012-01-06  6:27 UTC (permalink / raw)
  To: yocto, richard.purdie, bruce.ashfield, sgw

于 2012年01月06日 14:11, Zumeng Chen 写道
> +DEPENDS = "liburcu popt lttng-ust"
>   
And there is a depend on liburcu, which version should
>= 0.6.6,
it just has been updated recently. Lucky and Thanks :)

Regards,
Zumeng
> +
> +SRC_URI = "http://lttng.org/files/lttng-tools/lttng-tools-${PV}.tar.bz2 \
> +	   file://fix-compiling-error-for-powerpc-arm-and-mips.patch"
> +
> +SRC_URI[md5sum] = "58ed0a15db80a00172d5b0413a55451c" 
> +SRC_URI[sha256sum] = "188d262bda80043dd939c9aa7aec1795ca77aeec0cb3c0f23c28bfe754049491"
> +
> +S = "${WORKDIR}/lttng-tools-${PV}"
> +
> +
> +inherit autotools
> +
> +export KERNELDIR="${STAGING_KERNEL_DIR}"
> +
> +COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'
>   



^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 1/4] lttng-ust: Integrate into yocto linux
  2012-01-06  6:11 ` [PATCH 1/4] lttng-ust: Integrate into yocto linux Zumeng Chen
@ 2012-01-07  2:01   ` Darren Hart
  2012-01-09  4:27     ` Zumeng Chen
  0 siblings, 1 reply; 22+ messages in thread
From: Darren Hart @ 2012-01-07  2:01 UTC (permalink / raw)
  To: Zumeng Chen; +Cc: yocto

Hi Zumeng,

Thanks for putting these together.

Please provide a commit message. See:
https://wiki.yoctoproject.org/wiki/Contribution_Guidelines#Patch_Headers_and_Commit_Messages

> +LICENSE = "LGPLv2.1+ & BSD"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=e647752e045a8c45b6f583771bd561ef"

COPYING only covers the LGPL, please add another file to cover the BSD license.

> +
> +inherit autotools
> +
> +DEPENDS = "liburcu util-linux"
> +
> +SRC_URI = "http://lttng.org/files/lttng-ust/lttng-ust-${PV}.tar.gz"
> +
> +SRC_URI[md5sum] = "0f09bb954c99cb99ab6997cd492953c1"
> +SRC_URI[sha256sum] = "e9ec3c55c2251b3f3b512bfa1015b0f67e35d55793850ccea5134f7b24ee3ba1"
> +
> +S = "${WORKDIR}/lttng-ust-${PV}"
> +
> +# Due to liburcu not building on MIPS currently this recipe needs to
> +# be limited also.
> +# So here let us first suppport x86/arm/powerpc platforms now.
> +COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'

I don't think this is what you meant. You are restricting the type of build system,
not the type of target you can build for.

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


^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 2/4] lttng-tools: Integrated from git repo.
  2012-01-06  6:11 ` [PATCH 2/4] lttng-tools: Integrated from git repo Zumeng Chen
  2012-01-06  6:27   ` Zumeng Chen
@ 2012-01-07  2:05   ` Darren Hart
  2012-01-09  4:31     ` Zumeng Chen
  1 sibling, 1 reply; 22+ messages in thread
From: Darren Hart @ 2012-01-07  2:05 UTC (permalink / raw)
  To: Zumeng Chen; +Cc: yocto

This commit message doesn't tell me anything about what this patch is
intended to do.

> +++ b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm-and-mips.patch
> @@ -0,0 +1,23 @@
> +Upstream-Status: Pending
> +
> +lttng-tools: fix compiling error for powerpc, arm
> +

Add some commentary here about what the compilation error is.

> +Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
> +---
> + librunas/runas.c |    1 +
> + 1 files changed, 1 insertions(+), 0 deletions(-)
> +
> +diff --git a/librunas/runas.c b/librunas/runas.c
> +--- a/librunas/runas.c
> ++++ b/librunas/runas.c
> +@@ -29,6 +29,7 @@
> + #include <fcntl.h>
> + #include <sched.h>
> + #include <sys/mman.h>
> ++#include <asm/mman.h>
> + 
> + #include <lttngerr.h>
> + 
> +-- 
> +1.7.5.4
> +
> diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
> new file mode 100644
> index 0000000..0b7a240
> --- /dev/null
> +++ b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
> @@ -0,0 +1,24 @@
> +SECTION = "devel"
> +SUMMARY = "Linux Trace Toolkit Control"
> +DESCRIPTION = "The Linux trace toolkit is a suite of tools designed to \
> +extract program execution details from the Linux operating system and  \
> +interpret them."
> +LICENSE = "GPLv2.1"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=ab682a4729389c3f11913d758affe98e"
> +
> +DEPENDS = "liburcu popt lttng-ust"
> +
> +SRC_URI = "http://lttng.org/files/lttng-tools/lttng-tools-${PV}.tar.bz2 \
> +	   file://fix-compiling-error-for-powerpc-arm-and-mips.patch"

Whitespace error. Indent with spaces after the initial block indent
(which doesn't exist here).

> +
> +SRC_URI[md5sum] = "58ed0a15db80a00172d5b0413a55451c" 
> +SRC_URI[sha256sum] = "188d262bda80043dd939c9aa7aec1795ca77aeec0cb3c0f23c28bfe754049491"
> +
> +S = "${WORKDIR}/lttng-tools-${PV}"
> +
> +
> +inherit autotools
> +
> +export KERNELDIR="${STAGING_KERNEL_DIR}"
> +
> +COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'

This requires a comment as in the previous patch. Again, I don't think
this is what you are looking for. You want to limit which target
architectures you build this for. Right?

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


^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 3/4] lttng-modules: from lttng-modules 2.0 git repo.
  2012-01-06  6:11 ` [PATCH 3/4] lttng-modules: from lttng-modules 2.0 " Zumeng Chen
@ 2012-01-07  2:10   ` Darren Hart
  2012-01-09  4:33     ` Zumeng Chen
  0 siblings, 1 reply; 22+ messages in thread
From: Darren Hart @ 2012-01-07  2:10 UTC (permalink / raw)
  To: Zumeng Chen; +Cc: yocto



On 01/05/2012 10:11 PM, Zumeng Chen wrote:
> LTTng-modules 2.0 is currently in pre-release phase (-pre11)

If the commit message is identical, one could reasonably expect the
patch to be identical ;-) Please provide a complete commit message.

> 
> Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
> ---
>  ...modules-replace-KERNELDIR-with-KERNEL_SRC.patch |   41 ++++++++++++++++++++
>  .../lttng-2.0/lttng-modules_v2.0.pre11.bb          |   23 +++++++++++
>  2 files changed, 64 insertions(+), 0 deletions(-)
>  create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch
>  create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-modules_v2.0.pre11.bb
> 
> diff --git a/meta/recipes-kernel/lttng-2.0/lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch b/meta/recipes-kernel/lttng-2.0/lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch
> new file mode 100644
> index 0000000..b7b2fbb
> --- /dev/null
> +++ b/meta/recipes-kernel/lttng-2.0/lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch
> @@ -0,0 +1,41 @@
> +Upstream-Status: Inappropriate [embedded specific]
> +
> +lttng-modules: replace KERNELDIR with KERNEL_SRC
> +
> +Since lttng-modules uses the default way of module.bbclass to
> +build and install lttng-modules, so we do this replacement for
> +it as-is.

Since and so are redundant: s/so///

> +
> +Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
> +---
> + Makefile |    7 +++----
> + 1 files changed, 3 insertions(+), 4 deletions(-)
> +
> +diff --git a/Makefile b/Makefile
> +index 5ac13d7..25caad5 100644
> +--- a/Makefile
> ++++ b/Makefile
> +@@ -38,17 +38,16 @@ obj-m += lib/
> + endif
> + 
> + else
> +-	KERNELDIR ?= /lib/modules/$(shell uname -r)/build
> + 	PWD := $(shell pwd)
> + 	CFLAGS = $(EXTCFLAGS)
> + 
> + default:
> +-	$(MAKE) -C $(KERNELDIR) M=$(PWD) modules
> ++	$(MAKE) -C $(KERNEL_SRC) M=$(PWD) modules
> + 
> + modules_install:
> +-	$(MAKE) -C $(KERNELDIR) M=$(PWD) modules_install
> ++	$(MAKE) -C $(KERNEL_SRC) M=$(PWD) modules_install
> + 
> + clean:
> +-	$(MAKE) -C $(KERNELDIR) M=$(PWD) clean
> ++	$(MAKE) -C $(KERNEL_SRC) M=$(PWD) clean
> + 
> + endif
> +-- 
> +1.7.5.4
> +
> diff --git a/meta/recipes-kernel/lttng-2.0/lttng-modules_v2.0.pre11.bb b/meta/recipes-kernel/lttng-2.0/lttng-modules_v2.0.pre11.bb
> new file mode 100644
> index 0000000..34af847
> --- /dev/null
> +++ b/meta/recipes-kernel/lttng-2.0/lttng-modules_v2.0.pre11.bb
> @@ -0,0 +1,23 @@
> +SECTION = "devel"
> +SUMMARY = "Linux Trace Toolkit KERNEL MODULE"
> +DESCRIPTION = "The lttng-modules 2.0 package contains the kernel tracer modules"
> +LICENSE = "LGPLv2.1 & GPLv2"
> +LIC_FILES_CHKSUM = "file://LICENSE;md5=1eb086682a7c65a45acd9bcdf6877b3e \
> +		   "file://gpl-2.0.txt;md5=751419260aa954499f7abaabaa882bbe \
> +		   "file://lgpl-2.1.txt;md5=243b725d71bb5df4a1e5920b344b86ad"

Whitespace errors. Please use spaces here.

> +
> +DEPENDS = virtual/kernel 
> +
> +inherit module
> +
> +SRCREV = "4d3e89e379fc66480d729abe8daa5c86eb585400"
> +PV = "v2.0.pre11+git${SRCPV}"
> +PR = "r0"
> +
> +SRC_URI += "git://git.lttng.org/lttng-modules.git;protocol=git \
> +		file://lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch"

Spaces please.

> +
> +export INSTALL_MOD_DIR="kernel/lttng-modules"
> +
> +S = "${WORKDIR}/git"
> +COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'

I think you want a solution for TARGET.

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


^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 4/4] babeltrace: Integrated from version 0.8
  2012-01-06  6:11 ` [PATCH 4/4] babeltrace: Integrated from version 0.8 Zumeng Chen
@ 2012-01-07 15:54   ` Darren Hart
  2012-01-09  4:35     ` Zumeng Chen
  0 siblings, 1 reply; 22+ messages in thread
From: Darren Hart @ 2012-01-07 15:54 UTC (permalink / raw)
  To: Zumeng Chen; +Cc: yocto



On 01/05/2012 10:11 PM, Zumeng Chen wrote:
> It provides trace read and write libraries, as well as a trace converter.
> 
> Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
> ---
>  meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb |   30 +++++++++++++++++++++++
>  1 files changed, 30 insertions(+), 0 deletions(-)
>  create mode 100644 meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb
> 
> diff --git a/meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb b/meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb
> new file mode 100644
> index 0000000..86a9d01
> --- /dev/null
> +++ b/meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb
> @@ -0,0 +1,30 @@
> +SUMMARY = "Babeltrace - Trace Format Babel Tower"
> +DESCRIPTION = "Babeltrace provides trace read and write libraries in host side,\
> +	       as well as a trace converter, which used to convert LTTng 2.0 \
> +	       traces into human-readable log."

In the previous patches you aligned the additional lines with column 0,
here you indent with tabs and spaces. A quick survey of the existing
sources suggest the DESCRIPTION field typically does not indent the
subsequent lines.

> +HOMEPAGE = "http://www.efficios.com/babeltrace/"
> +BUGTRACKER = "n/a"
> +
> +LICENSE = "MIT & GPLV2"
> +LIC_FILES_CHKSUM = "file://LICENSE;md5=8d1a03b3c17bdd158b3cbb34813b1423"
> +
> +inherit autotools
> +
> +DEPENDS = "gtk+ util-linux"
> +
> +SRCREV = "31e670293588d7882a232bc14b93d5e956fa43f4"
> +PV = "0.8+git${SRCPV}"
> +PR = "r0"
> +
> +SRC_URI += "git://git.efficios.com/babeltrace.git;protocol=git"
> +

This should be a "=" assignment, not "+=".

> +S = "${WORKDIR}/git"
> +
> +do_configure_prepend () {
> +  ${S}/bootstrap

Please use tabs for indentation here.

> +}
> +
> +# Due to liburcu not building on MIPS currently this recipe needs to
> +# be limited also.
> +# So here let us first suppport x86/arm/powerpc platforms now.
> +COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'

Again, I think you want to filter based on target arch.

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


^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 1/4] lttng-ust: Integrate into yocto linux
  2012-01-07  2:01   ` Darren Hart
@ 2012-01-09  4:27     ` Zumeng Chen
  0 siblings, 0 replies; 22+ messages in thread
From: Zumeng Chen @ 2012-01-09  4:27 UTC (permalink / raw)
  To: Darren Hart; +Cc: yocto

于 2012年01月07日 10:01, Darren Hart 写道:
> Hi Zumeng,
>
> Thanks for putting these together.
>
> Please provide a commit message. See:
> https://wiki.yoctoproject.org/wiki/Contribution_Guidelines#Patch_Headers_and_Commit_Messages
>    
Thanks for your information, Darren.
>    
>> +LICENSE = "LGPLv2.1+&  BSD"
>> +LIC_FILES_CHKSUM = "file://COPYING;md5=e647752e045a8c45b6f583771bd561ef"
>>      
> COPYING only covers the LGPL, please add another file to cover the BSD license.
>    
Yeah. Added.
>    
>> +
>> +inherit autotools
>> +
>> +DEPENDS = "liburcu util-linux"
>> +
>> +SRC_URI = "http://lttng.org/files/lttng-ust/lttng-ust-${PV}.tar.gz"
>> +
>> +SRC_URI[md5sum] = "0f09bb954c99cb99ab6997cd492953c1"
>> +SRC_URI[sha256sum] = "e9ec3c55c2251b3f3b512bfa1015b0f67e35d55793850ccea5134f7b24ee3ba1"
>> +
>> +S = "${WORKDIR}/lttng-ust-${PV}"
>> +
>> +# Due to liburcu not building on MIPS currently this recipe needs to
>> +# be limited also.
>> +# So here let us first suppport x86/arm/powerpc platforms now.
>> +COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'
>>      
> I don't think this is what you meant. You are restricting the type of build system,
> not the type of target you can build for.
>    
Sorry, I copied from liburcu, and I heard its Poky's convention.

Regards,
Zumeng



^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 2/4] lttng-tools: Integrated from git repo.
  2012-01-07  2:05   ` Darren Hart
@ 2012-01-09  4:31     ` Zumeng Chen
  0 siblings, 0 replies; 22+ messages in thread
From: Zumeng Chen @ 2012-01-09  4:31 UTC (permalink / raw)
  To: Darren Hart; +Cc: yocto

于 2012年01月07日 10:05, Darren Hart 写道:
> This commit message doesn't tell me anything about what this patch is
> intended to do.
>
>    
>> +++ b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm-and-mips.patch
>> @@ -0,0 +1,23 @@
>> +Upstream-Status: Pending
>> +
>> +lttng-tools: fix compiling error for powerpc, arm
>> +
>>      
> Add some commentary here about what the compilation error is.
>    
OK. in V2.
>    
>> +Signed-off-by: Zumeng Chen<zumeng.chen@windriver.com>
>> +---
>> + librunas/runas.c |    1 +
>> + 1 files changed, 1 insertions(+), 0 deletions(-)
>> +
>> +diff --git a/librunas/runas.c b/librunas/runas.c
>> +--- a/librunas/runas.c
>> ++++ b/librunas/runas.c
>> +@@ -29,6 +29,7 @@
>> + #include<fcntl.h>
>> + #include<sched.h>
>> + #include<sys/mman.h>
>> ++#include<asm/mman.h>
>> +
>> + #include<lttngerr.h>
>> +
>> +--
>> +1.7.5.4
>> +
>> diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
>> new file mode 100644
>> index 0000000..0b7a240
>> --- /dev/null
>> +++ b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
>> @@ -0,0 +1,24 @@
>> +SECTION = "devel"
>> +SUMMARY = "Linux Trace Toolkit Control"
>> +DESCRIPTION = "The Linux trace toolkit is a suite of tools designed to \
>> +extract program execution details from the Linux operating system and  \
>> +interpret them."
>> +LICENSE = "GPLv2.1"
>> +LIC_FILES_CHKSUM = "file://COPYING;md5=ab682a4729389c3f11913d758affe98e"
>> +
>> +DEPENDS = "liburcu popt lttng-ust"
>> +
>> +SRC_URI = "http://lttng.org/files/lttng-tools/lttng-tools-${PV}.tar.bz2 \
>> +	   file://fix-compiling-error-for-powerpc-arm-and-mips.patch"
>>      
> Whitespace error. Indent with spaces after the initial block indent
> (which doesn't exist here).
>    
fix.ed
>    
>> +
>> +SRC_URI[md5sum] = "58ed0a15db80a00172d5b0413a55451c"
>> +SRC_URI[sha256sum] = "188d262bda80043dd939c9aa7aec1795ca77aeec0cb3c0f23c28bfe754049491"
>> +
>> +S = "${WORKDIR}/lttng-tools-${PV}"
>> +
>> +
>> +inherit autotools
>> +
>> +export KERNELDIR="${STAGING_KERNEL_DIR}"
>> +
>> +COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'
>>      
> This requires a comment as in the previous patch. Again, I don't think
> this is what you are looking for. You want to limit which target
> architectures you build this for. Right?
>    
Right, but the same reason for another email.

Regards,
Zumeng


^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 3/4] lttng-modules: from lttng-modules 2.0 git repo.
  2012-01-07  2:10   ` Darren Hart
@ 2012-01-09  4:33     ` Zumeng Chen
  0 siblings, 0 replies; 22+ messages in thread
From: Zumeng Chen @ 2012-01-09  4:33 UTC (permalink / raw)
  To: Darren Hart; +Cc: yocto

于 2012年01月07日 10:10, Darren Hart 写道:
>
> On 01/05/2012 10:11 PM, Zumeng Chen wrote:
>    
>> LTTng-modules 2.0 is currently in pre-release phase (-pre11)
>>      
> If the commit message is identical, one could reasonably expect the
> patch to be identical ;-) Please provide a complete commit message.
>    
OK, in V2.
>    
>> Signed-off-by: Zumeng Chen<zumeng.chen@windriver.com>
>> ---
>>   ...modules-replace-KERNELDIR-with-KERNEL_SRC.patch |   41 ++++++++++++++++++++
>>   .../lttng-2.0/lttng-modules_v2.0.pre11.bb          |   23 +++++++++++
>>   2 files changed, 64 insertions(+), 0 deletions(-)
>>   create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch
>>   create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-modules_v2.0.pre11.bb
>>
>> diff --git a/meta/recipes-kernel/lttng-2.0/lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch b/meta/recipes-kernel/lttng-2.0/lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch
>> new file mode 100644
>> index 0000000..b7b2fbb
>> --- /dev/null
>> +++ b/meta/recipes-kernel/lttng-2.0/lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch
>> @@ -0,0 +1,41 @@
>> +Upstream-Status: Inappropriate [embedded specific]
>> +
>> +lttng-modules: replace KERNELDIR with KERNEL_SRC
>> +
>> +Since lttng-modules uses the default way of module.bbclass to
>> +build and install lttng-modules, so we do this replacement for
>> +it as-is.
>>      
> Since and so are redundant: s/so///
>    
fixed.
>    
>> +
>> +Signed-off-by: Zumeng Chen<zumeng.chen@windriver.com>
>> +---
>> + Makefile |    7 +++----
>> + 1 files changed, 3 insertions(+), 4 deletions(-)
>> +
>> +diff --git a/Makefile b/Makefile
>> +index 5ac13d7..25caad5 100644
>> +--- a/Makefile
>> ++++ b/Makefile
>> +@@ -38,17 +38,16 @@ obj-m += lib/
>> + endif
>> +
>> + else
>> +-	KERNELDIR ?= /lib/modules/$(shell uname -r)/build
>> + 	PWD := $(shell pwd)
>> + 	CFLAGS = $(EXTCFLAGS)
>> +
>> + default:
>> +-	$(MAKE) -C $(KERNELDIR) M=$(PWD) modules
>> ++	$(MAKE) -C $(KERNEL_SRC) M=$(PWD) modules
>> +
>> + modules_install:
>> +-	$(MAKE) -C $(KERNELDIR) M=$(PWD) modules_install
>> ++	$(MAKE) -C $(KERNEL_SRC) M=$(PWD) modules_install
>> +
>> + clean:
>> +-	$(MAKE) -C $(KERNELDIR) M=$(PWD) clean
>> ++	$(MAKE) -C $(KERNEL_SRC) M=$(PWD) clean
>> +
>> + endif
>> +--
>> +1.7.5.4
>> +
>> diff --git a/meta/recipes-kernel/lttng-2.0/lttng-modules_v2.0.pre11.bb b/meta/recipes-kernel/lttng-2.0/lttng-modules_v2.0.pre11.bb
>> new file mode 100644
>> index 0000000..34af847
>> --- /dev/null
>> +++ b/meta/recipes-kernel/lttng-2.0/lttng-modules_v2.0.pre11.bb
>> @@ -0,0 +1,23 @@
>> +SECTION = "devel"
>> +SUMMARY = "Linux Trace Toolkit KERNEL MODULE"
>> +DESCRIPTION = "The lttng-modules 2.0 package contains the kernel tracer modules"
>> +LICENSE = "LGPLv2.1&  GPLv2"
>> +LIC_FILES_CHKSUM = "file://LICENSE;md5=1eb086682a7c65a45acd9bcdf6877b3e \
>> +		   "file://gpl-2.0.txt;md5=751419260aa954499f7abaabaa882bbe \
>> +		   "file://lgpl-2.1.txt;md5=243b725d71bb5df4a1e5920b344b86ad"
>>      
> Whitespace errors. Please use spaces here.
>    
fixed.
>    
>> +
>> +DEPENDS = virtual/kernel
>> +
>> +inherit module
>> +
>> +SRCREV = "4d3e89e379fc66480d729abe8daa5c86eb585400"
>> +PV = "v2.0.pre11+git${SRCPV}"
>> +PR = "r0"
>> +
>> +SRC_URI += "git://git.lttng.org/lttng-modules.git;protocol=git \
>> +		file://lttng-modules-replace-KERNELDIR-with-KERNEL_SRC.patch"
>>      
> Spaces please.
>    
fixed.
>    
>> +
>> +export INSTALL_MOD_DIR="kernel/lttng-modules"
>> +
>> +S = "${WORKDIR}/git"
>> +COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'
>>      
> I think you want a solution for TARGET.
>    
The same reason.
Regards,
Zumeng



^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 4/4] babeltrace: Integrated from version 0.8
  2012-01-07 15:54   ` Darren Hart
@ 2012-01-09  4:35     ` Zumeng Chen
  0 siblings, 0 replies; 22+ messages in thread
From: Zumeng Chen @ 2012-01-09  4:35 UTC (permalink / raw)
  To: Darren Hart; +Cc: yocto

于 2012年01月07日 23:54, Darren Hart 写道:
>
> On 01/05/2012 10:11 PM, Zumeng Chen wrote:
>    
>> It provides trace read and write libraries, as well as a trace converter.
>>
>> Signed-off-by: Zumeng Chen<zumeng.chen@windriver.com>
>> ---
>>   meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb |   30 +++++++++++++++++++++++
>>   1 files changed, 30 insertions(+), 0 deletions(-)
>>   create mode 100644 meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb
>>
>> diff --git a/meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb b/meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb
>> new file mode 100644
>> index 0000000..86a9d01
>> --- /dev/null
>> +++ b/meta/recipes-kernel/lttng-2.0/babeltrace_0.8.bb
>> @@ -0,0 +1,30 @@
>> +SUMMARY = "Babeltrace - Trace Format Babel Tower"
>> +DESCRIPTION = "Babeltrace provides trace read and write libraries in host side,\
>> +	       as well as a trace converter, which used to convert LTTng 2.0 \
>> +	       traces into human-readable log."
>>      
> In the previous patches you aligned the additional lines with column 0,
> here you indent with tabs and spaces. A quick survey of the existing
> sources suggest the DESCRIPTION field typically does not indent the
> subsequent lines.
>    
OK. fixed.
>    
>> +HOMEPAGE = "http://www.efficios.com/babeltrace/"
>> +BUGTRACKER = "n/a"
>> +
>> +LICENSE = "MIT&  GPLV2"
>> +LIC_FILES_CHKSUM = "file://LICENSE;md5=8d1a03b3c17bdd158b3cbb34813b1423"
>> +
>> +inherit autotools
>> +
>> +DEPENDS = "gtk+ util-linux"
>> +
>> +SRCREV = "31e670293588d7882a232bc14b93d5e956fa43f4"
>> +PV = "0.8+git${SRCPV}"
>> +PR = "r0"
>> +
>> +SRC_URI += "git://git.efficios.com/babeltrace.git;protocol=git"
>> +
>>      
> This should be a "=" assignment, not "+=".
>    
fixed.
>    
>> +S = "${WORKDIR}/git"
>> +
>> +do_configure_prepend () {
>> +  ${S}/bootstrap
>>      
> Please use tabs for indentation here.
>    
fixed.
>    
>> +}
>> +
>> +# Due to liburcu not building on MIPS currently this recipe needs to
>> +# be limited also.
>> +# So here let us first suppport x86/arm/powerpc platforms now.
>> +COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'
>>      
> Again, I think you want to filter based on target arch.
>    




^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 2/4] lttng-tools: Integrated from git repo.
  2012-01-12 22:04   ` Saul Wold
@ 2012-01-13  0:55     ` zumeng.chen
  0 siblings, 0 replies; 22+ messages in thread
From: zumeng.chen @ 2012-01-13  0:55 UTC (permalink / raw)
  To: Saul Wold; +Cc: Patches and discussions about the oe-core layer

On 2012年01月13日 06:04, Saul Wold wrote:
> On 01/11/2012 07:11 PM, Zumeng Chen wrote:
>> This patch intended to integrate the Linux trace toolkit, which
>> is a suite of tools designed to extract program execution details
>> from the Linux operating system and interpret them.
>>
>> lttng-tools 2.0 is currently in pre-release(-pre16), so we have to
>> update it when official released.
>>
>> Signed-off-by: Zumeng Chen<zumeng.chen@windriver.com>
>> ---
>> ...tools-fix-compiling-error-for-powerpc-arm.patch | 28 
>> ++++++++++++++++++
>> .../lttng-2.0/lttng-tools_2.0-pre16.bb | 30 ++++++++++++++++++++
>> 2 files changed, 58 insertions(+), 0 deletions(-)
>> create mode 100644 
>> meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
>> create mode 100644 
>> meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
>>
>> diff --git 
>> a/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch 
>> b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch 
>>
>> new file mode 100644
>> index 0000000..f8e144f
>> --- /dev/null
>> +++ 
>> b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
>> @@ -0,0 +1,28 @@
>> +From 5d8d75ab0c7f9e0009d90a2e5ad6475034a5c859 Mon Sep 17 00:00:00 2001
>> +From: Zumeng Chen<zumeng.chen@windriver.com>
>> +Date: Tue, 10 Jan 2012 12:41:45 +0800
>> +Subject: [PATCH 1/1] Fix compiling error on powerpc and arm
>> +
>
> No Upstream-Status:
Upstream-Status: Pending
>
>
>> +For powerpc and arm, some MAP_ are defined asm/mman.h,
>> +so including it in header.
>> +
>> +Signed-off-by: Zumeng Chen<zumeng.chen@windriver.com>
>> +---
>> + common/runas.c | 1 +
>> + 1 files changed, 1 insertions(+), 0 deletions(-)
>> +
>> +diff --git a/common/runas.c b/common/runas.c
>> +index f612ccc..62a2791 100644
>> +--- a/common/runas.c
>> ++++ b/common/runas.c
>> +@@ -29,6 +29,7 @@
>> + #include<fcntl.h>
>> + #include<sched.h>
>> + #include<sys/mman.h>
>> ++#include<asm/mman.h>
>> +
>> + #include<lttngerr.h>
>> +
>> +--
>> +1.7.5.4
>> +
>> diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb 
>> b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
>> new file mode 100644
>> index 0000000..cde57f7
>> --- /dev/null
>> +++ b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
>> @@ -0,0 +1,30 @@
>> +SECTION = "devel"
>> +SUMMARY = "Linux Trace Toolkit Control"
>> +DESCRIPTION = "The Linux trace toolkit is a suite of tools designed \
>> +to extract program execution details from the Linux operating system \
>> +and interpret them."
>> +
>> +LICENSE = "GPLv2.1"
>
> You have gpl-2.0 and lgpl-2.1 list below which is the correct LICENSE?
Yes, fixed, LICENSE = "GPLv2 & LGPLv2.1"

Thanks,
Zumeng
>
> Sau!
>
>> +LIC_FILES_CHKSUM = 
>> "file://COPYING;md5=ab682a4729389c3f11913d758affe98e \
>> + file://gpl-2.0.txt;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
>> + file://lgpl-2.1.txt;md5=0f0d71500e6a57fd24d825f33242b9ca"
>> +
>> +DEPENDS = "liburcu popt lttng-ust"
>> +
>> +SRCREV = "f2ca2e251d8f49b0dbbcca529dd61b3562c1147f"
>> +PV = "v2.0.pre16+git${SRCREV}"
>> +PR = "r0"
>> +
>> +SRC_URI = "git://git.lttng.org/lttng-tools.git;protocol=git\
>> + file://lttng-tools-fix-compiling-error-for-powerpc-arm.patch"
>> +
>> +S = "${WORKDIR}/git"
>> +
>> +inherit autotools
>> +
>> +export KERNELDIR="${STAGING_KERNEL_DIR}"
>> +
>> +# Due to liburcu not building for MIPS currently this recipe needs to
>> +# be limited also.
>> +# So here let us first suppport x86/arm/powerpc platforms now.
>> +COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'




^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 2/4] lttng-tools: Integrated from git repo.
  2012-01-12  3:11 ` [PATCH 2/4] lttng-tools: Integrated from git repo Zumeng Chen
@ 2012-01-12 22:04   ` Saul Wold
  2012-01-13  0:55     ` zumeng.chen
  0 siblings, 1 reply; 22+ messages in thread
From: Saul Wold @ 2012-01-12 22:04 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On 01/11/2012 07:11 PM, Zumeng Chen wrote:
> This patch intended to integrate the Linux trace toolkit, which
> is a suite of tools designed to extract program execution details
> from the Linux operating system and interpret them.
>
> lttng-tools 2.0 is currently in pre-release(-pre16), so we have to
> update it when official released.
>
> Signed-off-by: Zumeng Chen<zumeng.chen@windriver.com>
> ---
>   ...tools-fix-compiling-error-for-powerpc-arm.patch |   28 ++++++++++++++++++
>   .../lttng-2.0/lttng-tools_2.0-pre16.bb             |   30 ++++++++++++++++++++
>   2 files changed, 58 insertions(+), 0 deletions(-)
>   create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
>   create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
>
> diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
> new file mode 100644
> index 0000000..f8e144f
> --- /dev/null
> +++ b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
> @@ -0,0 +1,28 @@
> +From 5d8d75ab0c7f9e0009d90a2e5ad6475034a5c859 Mon Sep 17 00:00:00 2001
> +From: Zumeng Chen<zumeng.chen@windriver.com>
> +Date: Tue, 10 Jan 2012 12:41:45 +0800
> +Subject: [PATCH 1/1] Fix compiling error on powerpc and arm
> +

No Upstream-Status:


> +For powerpc and arm, some MAP_ are defined asm/mman.h,
> +so including it in header.
> +
> +Signed-off-by: Zumeng Chen<zumeng.chen@windriver.com>
> +---
> + common/runas.c |    1 +
> + 1 files changed, 1 insertions(+), 0 deletions(-)
> +
> +diff --git a/common/runas.c b/common/runas.c
> +index f612ccc..62a2791 100644
> +--- a/common/runas.c
> ++++ b/common/runas.c
> +@@ -29,6 +29,7 @@
> + #include<fcntl.h>
> + #include<sched.h>
> + #include<sys/mman.h>
> ++#include<asm/mman.h>
> +
> + #include<lttngerr.h>
> +
> +--
> +1.7.5.4
> +
> diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
> new file mode 100644
> index 0000000..cde57f7
> --- /dev/null
> +++ b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
> @@ -0,0 +1,30 @@
> +SECTION = "devel"
> +SUMMARY = "Linux Trace Toolkit Control"
> +DESCRIPTION = "The Linux trace toolkit is a suite of tools designed \
> +to extract program execution details from the Linux operating system \
> +and interpret them."
> +
> +LICENSE = "GPLv2.1"

You have gpl-2.0  and lgpl-2.1 list below which is the correct LICENSE?

Sau!

> +LIC_FILES_CHKSUM = "file://COPYING;md5=ab682a4729389c3f11913d758affe98e \
> +                    file://gpl-2.0.txt;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
> +                    file://lgpl-2.1.txt;md5=0f0d71500e6a57fd24d825f33242b9ca"
> +
> +DEPENDS = "liburcu popt lttng-ust"
> +
> +SRCREV = "f2ca2e251d8f49b0dbbcca529dd61b3562c1147f"
> +PV = "v2.0.pre16+git${SRCREV}"
> +PR = "r0"
> +
> +SRC_URI = "git://git.lttng.org/lttng-tools.git;protocol=git\
> +           file://lttng-tools-fix-compiling-error-for-powerpc-arm.patch"
> +
> +S = "${WORKDIR}/git"
> +
> +inherit autotools
> +
> +export KERNELDIR="${STAGING_KERNEL_DIR}"
> +
> +# Due to liburcu not building for MIPS currently this recipe needs to
> +# be limited also.
> +# So here let us first suppport x86/arm/powerpc platforms now.
> +COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'



^ permalink raw reply	[flat|nested] 22+ messages in thread

* [PATCH 2/4] lttng-tools: Integrated from git repo.
  2012-01-12  3:10 [V4] [PATCH 0/4] lttng-2.0 new recipe for merge Zumeng Chen
@ 2012-01-12  3:11 ` Zumeng Chen
  2012-01-12 22:04   ` Saul Wold
  0 siblings, 1 reply; 22+ messages in thread
From: Zumeng Chen @ 2012-01-12  3:11 UTC (permalink / raw)
  To: openembedded-core

This patch intended to integrate the Linux trace toolkit, which
is a suite of tools designed to extract program execution details
from the Linux operating system and interpret them.

lttng-tools 2.0 is currently in pre-release(-pre16), so we have to
update it when official released.

Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
---
 ...tools-fix-compiling-error-for-powerpc-arm.patch |   28 ++++++++++++++++++
 .../lttng-2.0/lttng-tools_2.0-pre16.bb             |   30 ++++++++++++++++++++
 2 files changed, 58 insertions(+), 0 deletions(-)
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb

diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
new file mode 100644
index 0000000..f8e144f
--- /dev/null
+++ b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
@@ -0,0 +1,28 @@
+From 5d8d75ab0c7f9e0009d90a2e5ad6475034a5c859 Mon Sep 17 00:00:00 2001
+From: Zumeng Chen <zumeng.chen@windriver.com>
+Date: Tue, 10 Jan 2012 12:41:45 +0800
+Subject: [PATCH 1/1] Fix compiling error on powerpc and arm
+
+For powerpc and arm, some MAP_ are defined asm/mman.h,
+so including it in header.
+
+Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
+---
+ common/runas.c |    1 +
+ 1 files changed, 1 insertions(+), 0 deletions(-)
+
+diff --git a/common/runas.c b/common/runas.c
+index f612ccc..62a2791 100644
+--- a/common/runas.c
++++ b/common/runas.c
+@@ -29,6 +29,7 @@
+ #include <fcntl.h>
+ #include <sched.h>
+ #include <sys/mman.h>
++#include <asm/mman.h>
+
+ #include <lttngerr.h>
+
+-- 
+1.7.5.4
+
diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
new file mode 100644
index 0000000..cde57f7
--- /dev/null
+++ b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
@@ -0,0 +1,30 @@
+SECTION = "devel"
+SUMMARY = "Linux Trace Toolkit Control"
+DESCRIPTION = "The Linux trace toolkit is a suite of tools designed \
+to extract program execution details from the Linux operating system \
+and interpret them."
+
+LICENSE = "GPLv2.1"
+LIC_FILES_CHKSUM = "file://COPYING;md5=ab682a4729389c3f11913d758affe98e \
+                    file://gpl-2.0.txt;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
+                    file://lgpl-2.1.txt;md5=0f0d71500e6a57fd24d825f33242b9ca"
+
+DEPENDS = "liburcu popt lttng-ust"
+
+SRCREV = "f2ca2e251d8f49b0dbbcca529dd61b3562c1147f"
+PV = "v2.0.pre16+git${SRCREV}"
+PR = "r0"
+
+SRC_URI = "git://git.lttng.org/lttng-tools.git;protocol=git\
+           file://lttng-tools-fix-compiling-error-for-powerpc-arm.patch"
+
+S = "${WORKDIR}/git"
+
+inherit autotools
+
+export KERNELDIR="${STAGING_KERNEL_DIR}"
+
+# Due to liburcu not building for MIPS currently this recipe needs to
+# be limited also.
+# So here let us first suppport x86/arm/powerpc platforms now.
+COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'
-- 
1.7.5.4




^ permalink raw reply related	[flat|nested] 22+ messages in thread

* [PATCH 2/4] lttng-tools: Integrated from git repo.
       [not found] <cover.1326244852.git.zumeng.chen@windriver.com>
@ 2012-01-11  1:30 ` Zumeng Chen
  0 siblings, 0 replies; 22+ messages in thread
From: Zumeng Chen @ 2012-01-11  1:30 UTC (permalink / raw)
  To: openembedded-core, dvhart

This patch intended to integrate the Linux trace toolkit, which
is a suite of tools designed to extract program execution details
from the Linux operating system and interpret them.

lttng-tools 2.0 is currently in pre-release(-pre16), so we have to
update it when official released.

Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
---
 ...tools-fix-compiling-error-for-powerpc-arm.patch |   28 ++++++++++++++++++
 .../lttng-2.0/lttng-tools_2.0-pre16.bb             |   30 ++++++++++++++++++++
 2 files changed, 58 insertions(+), 0 deletions(-)
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb

diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
new file mode 100644
index 0000000..f8e144f
--- /dev/null
+++ b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
@@ -0,0 +1,28 @@
+From 5d8d75ab0c7f9e0009d90a2e5ad6475034a5c859 Mon Sep 17 00:00:00 2001
+From: Zumeng Chen <zumeng.chen@windriver.com>
+Date: Tue, 10 Jan 2012 12:41:45 +0800
+Subject: [PATCH 1/1] Fix compiling error on powerpc and arm
+
+For powerpc and arm, some MAP_ are defined asm/mman.h,
+so including it in header.
+
+Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
+---
+ common/runas.c |    1 +
+ 1 files changed, 1 insertions(+), 0 deletions(-)
+
+diff --git a/common/runas.c b/common/runas.c
+index f612ccc..62a2791 100644
+--- a/common/runas.c
++++ b/common/runas.c
+@@ -29,6 +29,7 @@
+ #include <fcntl.h>
+ #include <sched.h>
+ #include <sys/mman.h>
++#include <asm/mman.h>
+
+ #include <lttngerr.h>
+
+-- 
+1.7.5.4
+
diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
new file mode 100644
index 0000000..cde57f7
--- /dev/null
+++ b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
@@ -0,0 +1,30 @@
+SECTION = "devel"
+SUMMARY = "Linux Trace Toolkit Control"
+DESCRIPTION = "The Linux trace toolkit is a suite of tools designed \
+to extract program execution details from the Linux operating system \
+and interpret them."
+
+LICENSE = "GPLv2.1"
+LIC_FILES_CHKSUM = "file://COPYING;md5=ab682a4729389c3f11913d758affe98e \
+                    file://gpl-2.0.txt;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
+                    file://lgpl-2.1.txt;md5=0f0d71500e6a57fd24d825f33242b9ca"
+
+DEPENDS = "liburcu popt lttng-ust"
+
+SRCREV = "f2ca2e251d8f49b0dbbcca529dd61b3562c1147f"
+PV = "v2.0.pre16+git${SRCREV}"
+PR = "r0"
+
+SRC_URI = "git://git.lttng.org/lttng-tools.git;protocol=git\
+           file://lttng-tools-fix-compiling-error-for-powerpc-arm.patch"
+
+S = "${WORKDIR}/git"
+
+inherit autotools
+
+export KERNELDIR="${STAGING_KERNEL_DIR}"
+
+# Due to liburcu not building for MIPS currently this recipe needs to
+# be limited also.
+# So here let us first suppport x86/arm/powerpc platforms now.
+COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'
-- 
1.7.5.4




^ permalink raw reply related	[flat|nested] 22+ messages in thread

* [PATCH 2/4] lttng-tools: Integrated from git repo.
       [not found] <cover.1326210673.git.zumeng.chen@windriver.com>
@ 2012-01-10 16:11 ` Zumeng Chen
  0 siblings, 0 replies; 22+ messages in thread
From: Zumeng Chen @ 2012-01-10 16:11 UTC (permalink / raw)
  To: yocto, dvhart, bruce.ashfield, sgw, richard.purdie; +Cc: openembedded-core

This patch intended to integrate the Linux trace toolkit, which
is a suite of tools designed to extract program execution details
from the Linux operating system and interpret them.

lttng-tools 2.0 is currently in pre-release(-pre16), so we have to
update it when official released.

Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
---
 ...tools-fix-compiling-error-for-powerpc-arm.patch |   28 +++++++++++++++++++
 .../lttng-2.0/lttng-tools_2.0-pre16.bb             |   29 ++++++++++++++++++++
 2 files changed, 57 insertions(+), 0 deletions(-)
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb

diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
new file mode 100644
index 0000000..f8e144f
--- /dev/null
+++ b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
@@ -0,0 +1,28 @@
+From 5d8d75ab0c7f9e0009d90a2e5ad6475034a5c859 Mon Sep 17 00:00:00 2001
+From: Zumeng Chen <zumeng.chen@windriver.com>
+Date: Tue, 10 Jan 2012 12:41:45 +0800
+Subject: [PATCH 1/1] Fix compiling error on powerpc and arm
+
+For powerpc and arm, some MAP_ are defined asm/mman.h,
+so including it in header.
+
+Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
+---
+ common/runas.c |    1 +
+ 1 files changed, 1 insertions(+), 0 deletions(-)
+
+diff --git a/common/runas.c b/common/runas.c
+index f612ccc..62a2791 100644
+--- a/common/runas.c
++++ b/common/runas.c
+@@ -29,6 +29,7 @@
+ #include <fcntl.h>
+ #include <sched.h>
+ #include <sys/mman.h>
++#include <asm/mman.h>
+
+ #include <lttngerr.h>
+
+-- 
+1.7.5.4
+
diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
new file mode 100644
index 0000000..d771ed9
--- /dev/null
+++ b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
@@ -0,0 +1,29 @@
+SECTION = "devel"
+SUMMARY = "Linux Trace Toolkit Control"
+DESCRIPTION = "The Linux trace toolkit is a suite of tools designed \
+to extract program execution details from the Linux operating system \
+and interpret them."
+
+LICENSE = "GPLv2.1"
+LIC_FILES_CHKSUM = "file://COPYING;md5=ab682a4729389c3f11913d758affe98e \
+                    file://gpl-2.0.txt;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
+                    file://lgpl-2.1.txt;md5=0f0d71500e6a57fd24d825f33242b9ca"
+
+DEPENDS = "liburcu popt lttng-ust"
+
+SRCREV = "f2ca2e251d8f49b0dbbcca529dd61b3562c1147f"
+PV = "v2.0.pre16+git${SRCREV}"
+PR = "r0"
+
+SRC_URI = "git://git.lttng.org/lttng-tools.git;protocol=git\
+           file://lttng-tools-fix-compiling-error-for-powerpc-arm.patch"
+
+S = "${WORKDIR}/git"
+
+inherit autotools
+
+export KERNELDIR="${STAGING_KERNEL_DIR}"
+
+# Currenly lttng-tools doesn't supprt MIPS officially.
+# So here let us keep as-is.
+COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'
-- 
1.7.5.4



^ permalink raw reply related	[flat|nested] 22+ messages in thread

* Re: [PATCH 2/4] lttng-tools: Integrated from git repo.
  2012-01-09 18:00   ` Darren Hart
@ 2012-01-10  3:22     ` Zumeng Chen
  0 siblings, 0 replies; 22+ messages in thread
From: Zumeng Chen @ 2012-01-10  3:22 UTC (permalink / raw)
  To: Darren Hart; +Cc: yocto

于 2012年01月10日 02:00, Darren Hart 写道:
>
> On 01/09/2012 01:24 AM, Zumeng Chen wrote:
>    
>> This patch intended to integrate the Linux trace toolkit, which
>> is a suite of tools designed to extract program execution details
>> from the Linux operating system and interpret them.
>>
>> lttng-tools 2.0 is currently in pre-release(-pre16), so we have to
>> update it when official released.
>>
>> Signed-off-by: Zumeng Chen<zumeng.chen@windriver.com>
>> ---
>>   ...tools-fix-compiling-error-for-powerpc-arm.patch |   26 ++++++++++++++++++++
>>   .../lttng-2.0/lttng-tools_2.0-pre16.bb             |   21 ++++++++++++++++
>>   2 files changed, 47 insertions(+), 0 deletions(-)
>>   create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
>>   create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
>>
>> diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
>> new file mode 100644
>> index 0000000..47b7b98
>> --- /dev/null
>> +++ b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
>> @@ -0,0 +1,26 @@
>> +Upstream-Status: Pending
>> +
>> +lttng-tools: fix compiling error for powerpc, arm
>> +
>> +For powerpc and arm system, some MAP_ defined in
>> +asm/mman.h, so we include it in header.
>> +
>> +Signed-off-by: Zumeng Chen<zumeng.chen@windriver.com>
>> +---
>> + librunas/runas.c |    1 +
>> + 1 files changed, 1 insertions(+), 0 deletions(-)
>> +
>> +diff --git a/librunas/runas.c b/librunas/runas.c
>> +--- a/librunas/runas.c
>> ++++ b/librunas/runas.c
>> +@@ -29,6 +29,7 @@
>> + #include<fcntl.h>
>> + #include<sched.h>
>> + #include<sys/mman.h>
>> ++#include<asm/mman.h>
>> +
>> + #include<lttngerr.h>
>> +
>> +--
>> +1.7.5.4
>> +
>> diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
>> new file mode 100644
>> index 0000000..8a688c9
>> --- /dev/null
>> +++ b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
>> @@ -0,0 +1,21 @@
>> +SECTION = "devel"
>> +SUMMARY = "Linux Trace Toolkit Control"
>> +DESCRIPTION = "The Linux trace toolkit is a suite of tools designed to extract program execution details from the Linux operating system and interpret them."
>>      
> Please do wrap long lines, just don't indent them with tabs past the
> previous line:
>
> DESCRIPTION = "The Linux trace toolkit is a suite of tools designed \
> to extract program execution details from the Linux operating system \
> and interpret them."
>    
OK. fixed.
>    
>> +LICENSE = "GPLv2.1"
>> +LIC_FILES_CHKSUM = "file://COPYING;md5=ab682a4729389c3f11913d758affe98e"
>> +
>> +DEPENDS = "liburcu popt lttng-ust"
>> +
>> +SRC_URI = "http://lttng.org/files/lttng-tools/lttng-tools-${PV}.tar.bz2 \
>> +           file://lttng-tools-fix-compiling-error-for-powerpc-arm.patch"
>> +
>> +SRC_URI[md5sum] = "58ed0a15db80a00172d5b0413a55451c"
>> +SRC_URI[sha256sum] = "188d262bda80043dd939c9aa7aec1795ca77aeec0cb3c0f23c28bfe754049491"
>> +
>> +S = "${WORKDIR}/lttng-tools-${PV}"
>> +
>> +inherit autotools
>> +
>> +export KERNELDIR="${STAGING_KERNEL_DIR}"
>> +
>>      
> Please include the reason for the COMPATIBLE_HOST test in all recipes.
>    
OK.
>    
>> +COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'
>>      
>    



^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH 2/4] lttng-tools: Integrated from git repo.
  2012-01-09  9:24 ` [PATCH 2/4] lttng-tools: Integrated from git repo Zumeng Chen
@ 2012-01-09 18:00   ` Darren Hart
  2012-01-10  3:22     ` Zumeng Chen
  0 siblings, 1 reply; 22+ messages in thread
From: Darren Hart @ 2012-01-09 18:00 UTC (permalink / raw)
  To: Zumeng Chen; +Cc: yocto



On 01/09/2012 01:24 AM, Zumeng Chen wrote:
> This patch intended to integrate the Linux trace toolkit, which
> is a suite of tools designed to extract program execution details
> from the Linux operating system and interpret them.
> 
> lttng-tools 2.0 is currently in pre-release(-pre16), so we have to
> update it when official released.
> 
> Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
> ---
>  ...tools-fix-compiling-error-for-powerpc-arm.patch |   26 ++++++++++++++++++++
>  .../lttng-2.0/lttng-tools_2.0-pre16.bb             |   21 ++++++++++++++++
>  2 files changed, 47 insertions(+), 0 deletions(-)
>  create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
>  create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
> 
> diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
> new file mode 100644
> index 0000000..47b7b98
> --- /dev/null
> +++ b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
> @@ -0,0 +1,26 @@
> +Upstream-Status: Pending
> +
> +lttng-tools: fix compiling error for powerpc, arm
> +
> +For powerpc and arm system, some MAP_ defined in
> +asm/mman.h, so we include it in header.
> +
> +Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
> +---
> + librunas/runas.c |    1 +
> + 1 files changed, 1 insertions(+), 0 deletions(-)
> +
> +diff --git a/librunas/runas.c b/librunas/runas.c
> +--- a/librunas/runas.c
> ++++ b/librunas/runas.c
> +@@ -29,6 +29,7 @@
> + #include <fcntl.h>
> + #include <sched.h>
> + #include <sys/mman.h>
> ++#include <asm/mman.h>
> +
> + #include <lttngerr.h>
> +
> +--
> +1.7.5.4
> +
> diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
> new file mode 100644
> index 0000000..8a688c9
> --- /dev/null
> +++ b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
> @@ -0,0 +1,21 @@
> +SECTION = "devel"
> +SUMMARY = "Linux Trace Toolkit Control"
> +DESCRIPTION = "The Linux trace toolkit is a suite of tools designed to extract program execution details from the Linux operating system and interpret them."

Please do wrap long lines, just don't indent them with tabs past the
previous line:

DESCRIPTION = "The Linux trace toolkit is a suite of tools designed \
to extract program execution details from the Linux operating system \
and interpret them."

> +LICENSE = "GPLv2.1"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=ab682a4729389c3f11913d758affe98e"
> +
> +DEPENDS = "liburcu popt lttng-ust"
> +
> +SRC_URI = "http://lttng.org/files/lttng-tools/lttng-tools-${PV}.tar.bz2 \
> +           file://lttng-tools-fix-compiling-error-for-powerpc-arm.patch"
> +
> +SRC_URI[md5sum] = "58ed0a15db80a00172d5b0413a55451c"
> +SRC_URI[sha256sum] = "188d262bda80043dd939c9aa7aec1795ca77aeec0cb3c0f23c28bfe754049491"
> +
> +S = "${WORKDIR}/lttng-tools-${PV}"
> +
> +inherit autotools
> +
> +export KERNELDIR="${STAGING_KERNEL_DIR}"
> +

Please include the reason for the COMPATIBLE_HOST test in all recipes.

> +COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'

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


^ permalink raw reply	[flat|nested] 22+ messages in thread

* [PATCH 2/4] lttng-tools: Integrated from git repo.
       [not found] <cover.1326099771.git.zumeng.chen@windriver.com>
@ 2012-01-09  9:24 ` Zumeng Chen
  2012-01-09 18:00   ` Darren Hart
  0 siblings, 1 reply; 22+ messages in thread
From: Zumeng Chen @ 2012-01-09  9:24 UTC (permalink / raw)
  To: yocto, dvhart

This patch intended to integrate the Linux trace toolkit, which
is a suite of tools designed to extract program execution details
from the Linux operating system and interpret them.

lttng-tools 2.0 is currently in pre-release(-pre16), so we have to
update it when official released.

Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
---
 ...tools-fix-compiling-error-for-powerpc-arm.patch |   26 ++++++++++++++++++++
 .../lttng-2.0/lttng-tools_2.0-pre16.bb             |   21 ++++++++++++++++
 2 files changed, 47 insertions(+), 0 deletions(-)
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
 create mode 100644 meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb

diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
new file mode 100644
index 0000000..47b7b98
--- /dev/null
+++ b/meta/recipes-kernel/lttng-2.0/lttng-tools-fix-compiling-error-for-powerpc-arm.patch
@@ -0,0 +1,26 @@
+Upstream-Status: Pending
+
+lttng-tools: fix compiling error for powerpc, arm
+
+For powerpc and arm system, some MAP_ defined in
+asm/mman.h, so we include it in header.
+
+Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
+---
+ librunas/runas.c |    1 +
+ 1 files changed, 1 insertions(+), 0 deletions(-)
+
+diff --git a/librunas/runas.c b/librunas/runas.c
+--- a/librunas/runas.c
++++ b/librunas/runas.c
+@@ -29,6 +29,7 @@
+ #include <fcntl.h>
+ #include <sched.h>
+ #include <sys/mman.h>
++#include <asm/mman.h>
+
+ #include <lttngerr.h>
+
+--
+1.7.5.4
+
diff --git a/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
new file mode 100644
index 0000000..8a688c9
--- /dev/null
+++ b/meta/recipes-kernel/lttng-2.0/lttng-tools_2.0-pre16.bb
@@ -0,0 +1,21 @@
+SECTION = "devel"
+SUMMARY = "Linux Trace Toolkit Control"
+DESCRIPTION = "The Linux trace toolkit is a suite of tools designed to extract program execution details from the Linux operating system and interpret them."
+LICENSE = "GPLv2.1"
+LIC_FILES_CHKSUM = "file://COPYING;md5=ab682a4729389c3f11913d758affe98e"
+
+DEPENDS = "liburcu popt lttng-ust"
+
+SRC_URI = "http://lttng.org/files/lttng-tools/lttng-tools-${PV}.tar.bz2 \
+           file://lttng-tools-fix-compiling-error-for-powerpc-arm.patch"
+
+SRC_URI[md5sum] = "58ed0a15db80a00172d5b0413a55451c"
+SRC_URI[sha256sum] = "188d262bda80043dd939c9aa7aec1795ca77aeec0cb3c0f23c28bfe754049491"
+
+S = "${WORKDIR}/lttng-tools-${PV}"
+
+inherit autotools
+
+export KERNELDIR="${STAGING_KERNEL_DIR}"
+
+COMPATIBLE_HOST = '(x86_64.*|i.86.*|arm.*|powerpc.*)-linux'
-- 
1.7.5.4



^ permalink raw reply related	[flat|nested] 22+ messages in thread

end of thread, other threads:[~2012-01-13  1:03 UTC | newest]

Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-01-06  6:11 [PATCH 0/4]lttng-2.0 new recipe Zumeng Chen
2012-01-06  6:11 ` [PATCH 1/4] lttng-ust: Integrate into yocto linux Zumeng Chen
2012-01-07  2:01   ` Darren Hart
2012-01-09  4:27     ` Zumeng Chen
2012-01-06  6:11 ` [PATCH 2/4] lttng-tools: Integrated from git repo Zumeng Chen
2012-01-06  6:27   ` Zumeng Chen
2012-01-07  2:05   ` Darren Hart
2012-01-09  4:31     ` Zumeng Chen
2012-01-06  6:11 ` [PATCH 3/4] lttng-modules: from lttng-modules 2.0 " Zumeng Chen
2012-01-07  2:10   ` Darren Hart
2012-01-09  4:33     ` Zumeng Chen
2012-01-06  6:11 ` [PATCH 4/4] babeltrace: Integrated from version 0.8 Zumeng Chen
2012-01-07 15:54   ` Darren Hart
2012-01-09  4:35     ` Zumeng Chen
     [not found] <cover.1326099771.git.zumeng.chen@windriver.com>
2012-01-09  9:24 ` [PATCH 2/4] lttng-tools: Integrated from git repo Zumeng Chen
2012-01-09 18:00   ` Darren Hart
2012-01-10  3:22     ` Zumeng Chen
     [not found] <cover.1326210673.git.zumeng.chen@windriver.com>
2012-01-10 16:11 ` Zumeng Chen
     [not found] <cover.1326244852.git.zumeng.chen@windriver.com>
2012-01-11  1:30 ` Zumeng Chen
2012-01-12  3:10 [V4] [PATCH 0/4] lttng-2.0 new recipe for merge Zumeng Chen
2012-01-12  3:11 ` [PATCH 2/4] lttng-tools: Integrated from git repo Zumeng Chen
2012-01-12 22:04   ` Saul Wold
2012-01-13  0:55     ` zumeng.chen

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.