* [meta-oe][for-denzil][PATCH 2/8] recipes-kernel: make perf a standalone package
2012-09-06 11:23 [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning Koen Kooi
@ 2012-09-06 11:23 ` Koen Kooi
2012-09-06 11:23 ` [meta-oe][for-denzil][PATCH 3/8] kernel: Add kernel headers to kernel-dev package Koen Kooi
` (6 subsequent siblings)
7 siblings, 0 replies; 17+ messages in thread
From: Koen Kooi @ 2012-09-06 11:23 UTC (permalink / raw)
To: openembedded-devel; +Cc: Koen Kooi, Eric Bénard
From: Liang Li <liang.li@windriver.com>
perf has been coupled to the kernel packages via kernel.bbclass.
While maintaining the build of perf out of the kernel source tree
is desired the package coupling has proved to be awkward in
several situations such as:
- when a kernel recipe doesn't want to build/provide perf
- when licensing of dependencies would prohibit perf and hence
the kernel from being built.
To solve some of these problems, this recipe is the extraction of
the linux-tools.inc provided perf compilation into a standalone
perf recipe that builds out of the kernel source, but is otherwise
independent.
No new functionality is provided above what the linux-tools.inc
variant provided, but the separate recipe provides baseline for
adding new functionality.
Signed-off-by: Liang Li <liang.li@windriver.com>
Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
---
meta-oe/classes/kernel.bbclass | 7 +------
1 files changed, 1 insertions(+), 6 deletions(-)
diff --git a/meta-oe/classes/kernel.bbclass b/meta-oe/classes/kernel.bbclass
index c44a2f5..6f1b6a4 100644
--- a/meta-oe/classes/kernel.bbclass
+++ b/meta-oe/classes/kernel.bbclass
@@ -475,7 +475,7 @@ python populate_packages_prepend () {
metapkg = "kernel-modules"
d.setVar('ALLOW_EMPTY_' + metapkg, "1")
d.setVar('FILES_' + metapkg, "")
- blacklist = [ 'kernel-dev', 'kernel-image', 'kernel-base', 'kernel-vmlinux', 'perf', 'perf-dbg', 'kernel-misc' ]
+ blacklist = [ 'kernel-dev', 'kernel-image', 'kernel-base', 'kernel-vmlinux', 'kernel-misc' ]
for l in module_deps.values():
for i in l:
pkg = module_pattern % legitimize_package_name(re.match(module_regex, os.path.basename(i)).group(1))
@@ -553,8 +553,3 @@ addtask deploy before do_build after do_install
EXPORT_FUNCTIONS do_deploy
-# perf must be enabled in individual kernel recipes
-PACKAGES =+ "perf-dbg perf"
-FILES_perf = "${bindir}/* \
- ${libexecdir}"
-FILES_perf-dbg = "${FILES_${PN}-dbg}"
--
1.7.7.6
^ permalink raw reply related [flat|nested] 17+ messages in thread* [meta-oe][for-denzil][PATCH 3/8] kernel: Add kernel headers to kernel-dev package
2012-09-06 11:23 [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning Koen Kooi
2012-09-06 11:23 ` [meta-oe][for-denzil][PATCH 2/8] recipes-kernel: make perf a standalone package Koen Kooi
@ 2012-09-06 11:23 ` Koen Kooi
2012-09-06 16:43 ` McClintock Matthew-B29882
2012-09-06 11:23 ` [meta-oe][for-denzil][PATCH 4/8] kernel.bbclass: Dont package kxgettext.o Koen Kooi
` (5 subsequent siblings)
7 siblings, 1 reply; 17+ messages in thread
From: Koen Kooi @ 2012-09-06 11:23 UTC (permalink / raw)
To: openembedded-devel; +Cc: Koen Kooi, Eric Bénard
From: Darren Hart <dvhart@linux.intel.com>
[YOCTO #1614]
Add the kernel headers to the kernel-dev package. This packages what was
already built and kept in sysroots for building modules with bitbake.
Making this available on the target requires removing some additional
host binaries.
Move the location to /usr/src/kernel
Before use on the target, the user will need to:
# cd /usr/src/kernel
# make scripts
This renders the kernel-misc recipe empty, so remove it.
As we use /usr/src/kernel in several places (and I missed one in the
previous version), add a KERNEL_SRC_DIR variable and use that throughout
the class to avoid update errors in the future.
Now that we package the kernel headers, drop the
kernel_package_preprocess function which removed them from PKGD.
All *-sdk image recipes include dev-pkgs, so the kernel-dev package will
be installed by default on all such images.
Signed-off-by: Darren Hart <dvhart@linux.intel.com>
CC: Bruce Ashfield <bruce.ashfield@windriver.com>
CC: Tom Zanussi <tom.zanussi@intel.com>
CC: Khem Raj <raj.khem@gmail.com>
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
---
meta-oe/classes/kernel.bbclass | 27 ++++++++++++---------------
1 files changed, 12 insertions(+), 15 deletions(-)
diff --git a/meta-oe/classes/kernel.bbclass b/meta-oe/classes/kernel.bbclass
index 6f1b6a4..d7e42f6 100644
--- a/meta-oe/classes/kernel.bbclass
+++ b/meta-oe/classes/kernel.bbclass
@@ -82,6 +82,10 @@ EXTRA_OEMAKE = ""
KERNEL_ALT_IMAGETYPE ??= ""
+# Define where the kernel headers are installed on the target as well as where
+# they are staged.
+KERNEL_SRC_PATH = "/usr/src/kernel"
+
KERNEL_IMAGETYPE_FOR_MAKE = "${@(lambda s: s[:-3] if s[-3:] == ".gz" else s)(d.getVar('KERNEL_IMAGETYPE', True))}"
kernel_do_compile() {
@@ -135,7 +139,7 @@ kernel_do_install() {
# Support for external module building - create a minimal copy of the
# kernel source tree.
#
- kerneldir=${D}/kernel
+ kerneldir=${D}${KERNEL_SRC_PATH}
install -d $kerneldir
#
@@ -188,23 +192,18 @@ kernel_do_install() {
cp arch/powerpc/lib/crtsavres.o $kerneldir/arch/powerpc/lib/crtsavres.o
fi
- # Remove the following binaries which cause strip errors
+ # Remove the following binaries which cause strip or arch QA errors
# during do_package for cross-compiled platforms
bin_files="arch/powerpc/boot/addnote arch/powerpc/boot/hack-coff \
- arch/powerpc/boot/mktree"
+ arch/powerpc/boot/mktree scripts/kconfig/zconf.tab.o \
+ scripts/kconfig/conf.o"
for entry in $bin_files; do
rm -f $kerneldir/$entry
done
}
-PACKAGE_PREPROCESS_FUNCS += "kernel_package_preprocess"
-
-kernel_package_preprocess () {
- rm -rf ${PKGD}/kernel
-}
-
sysroot_stage_all_append() {
- sysroot_stage_dir ${D}/kernel ${SYSROOT_DESTDIR}/kernel
+ sysroot_stage_dir ${D}${KERNEL_SRC_PATH} ${SYSROOT_DESTDIR}${KERNEL_SRC_PATH}
}
kernel_do_configure() {
@@ -252,13 +251,11 @@ EXPORT_FUNCTIONS do_compile do_install do_configure
# kernel-base becomes kernel-${KERNEL_VERSION}
# kernel-image becomes kernel-image-${KERNEL_VERISON}
-PACKAGES = "kernel kernel-base kernel-vmlinux kernel-image kernel-dev kernel-misc"
+PACKAGES = "kernel kernel-base kernel-vmlinux kernel-image kernel-dev"
FILES = ""
FILES_kernel-image = "/boot/${KERNEL_IMAGETYPE}*"
-FILES_kernel-dev = "/boot/System.map* /boot/Module.symvers* /boot/config*"
+FILES_kernel-dev = "/boot/System.map* /boot/Module.symvers* /boot/config* ${KERNEL_SRC_PATH}"
FILES_kernel-vmlinux = "/boot/vmlinux*"
-# misc is a package to contain files we need in staging
-FILES_kernel-misc = "/kernel/include/config /kernel/scripts /kernel/drivers/crypto /kernel/drivers/media"
RDEPENDS_kernel = "kernel-base"
# Allow machines to override this dependency if kernel image files are
# not wanted in images as standard
@@ -475,7 +472,7 @@ python populate_packages_prepend () {
metapkg = "kernel-modules"
d.setVar('ALLOW_EMPTY_' + metapkg, "1")
d.setVar('FILES_' + metapkg, "")
- blacklist = [ 'kernel-dev', 'kernel-image', 'kernel-base', 'kernel-vmlinux', 'kernel-misc' ]
+ blacklist = [ 'kernel-dev', 'kernel-image', 'kernel-base', 'kernel-vmlinux' ]
for l in module_deps.values():
for i in l:
pkg = module_pattern % legitimize_package_name(re.match(module_regex, os.path.basename(i)).group(1))
--
1.7.7.6
^ permalink raw reply related [flat|nested] 17+ messages in thread* Re: [meta-oe][for-denzil][PATCH 3/8] kernel: Add kernel headers to kernel-dev package
2012-09-06 11:23 ` [meta-oe][for-denzil][PATCH 3/8] kernel: Add kernel headers to kernel-dev package Koen Kooi
@ 2012-09-06 16:43 ` McClintock Matthew-B29882
2012-09-06 16:47 ` Koen Kooi
0 siblings, 1 reply; 17+ messages in thread
From: McClintock Matthew-B29882 @ 2012-09-06 16:43 UTC (permalink / raw)
To: openembedded-devel@lists.openembedded.org; +Cc: Koen Kooi, Eric Bénard
On Thu, Sep 6, 2012 at 6:23 AM, Koen Kooi <koen@dominion.thruhere.net> wrote:
> From: Darren Hart <dvhart@linux.intel.com>
>
> [YOCTO #1614]
>
> Add the kernel headers to the kernel-dev package. This packages what was
> already built and kept in sysroots for building modules with bitbake.
> Making this available on the target requires removing some additional
> host binaries.
>
> Move the location to /usr/src/kernel
>
> Before use on the target, the user will need to:
>
> # cd /usr/src/kernel
> # make scripts
>
> This renders the kernel-misc recipe empty, so remove it.
>
> As we use /usr/src/kernel in several places (and I missed one in the
> previous version), add a KERNEL_SRC_DIR variable and use that throughout
> the class to avoid update errors in the future.
>
> Now that we package the kernel headers, drop the
> kernel_package_preprocess function which removed them from PKGD.
>
> All *-sdk image recipes include dev-pkgs, so the kernel-dev package will
> be installed by default on all such images.
>
> Signed-off-by: Darren Hart <dvhart@linux.intel.com>
> CC: Bruce Ashfield <bruce.ashfield@windriver.com>
> CC: Tom Zanussi <tom.zanussi@intel.com>
> CC: Khem Raj <raj.khem@gmail.com>
> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
Why did you resent this? Are there changes or by accident? Just asking
before I make a patch for denzil with my previous fix.
-M
> ---
> meta-oe/classes/kernel.bbclass | 27 ++++++++++++---------------
> 1 files changed, 12 insertions(+), 15 deletions(-)
>
> diff --git a/meta-oe/classes/kernel.bbclass b/meta-oe/classes/kernel.bbclass
> index 6f1b6a4..d7e42f6 100644
> --- a/meta-oe/classes/kernel.bbclass
> +++ b/meta-oe/classes/kernel.bbclass
> @@ -82,6 +82,10 @@ EXTRA_OEMAKE = ""
>
> KERNEL_ALT_IMAGETYPE ??= ""
>
> +# Define where the kernel headers are installed on the target as well as where
> +# they are staged.
> +KERNEL_SRC_PATH = "/usr/src/kernel"
> +
> KERNEL_IMAGETYPE_FOR_MAKE = "${@(lambda s: s[:-3] if s[-3:] == ".gz" else s)(d.getVar('KERNEL_IMAGETYPE', True))}"
>
> kernel_do_compile() {
> @@ -135,7 +139,7 @@ kernel_do_install() {
> # Support for external module building - create a minimal copy of the
> # kernel source tree.
> #
> - kerneldir=${D}/kernel
> + kerneldir=${D}${KERNEL_SRC_PATH}
> install -d $kerneldir
>
> #
> @@ -188,23 +192,18 @@ kernel_do_install() {
> cp arch/powerpc/lib/crtsavres.o $kerneldir/arch/powerpc/lib/crtsavres.o
> fi
>
> - # Remove the following binaries which cause strip errors
> + # Remove the following binaries which cause strip or arch QA errors
> # during do_package for cross-compiled platforms
> bin_files="arch/powerpc/boot/addnote arch/powerpc/boot/hack-coff \
> - arch/powerpc/boot/mktree"
> + arch/powerpc/boot/mktree scripts/kconfig/zconf.tab.o \
> + scripts/kconfig/conf.o"
> for entry in $bin_files; do
> rm -f $kerneldir/$entry
> done
> }
>
> -PACKAGE_PREPROCESS_FUNCS += "kernel_package_preprocess"
> -
> -kernel_package_preprocess () {
> - rm -rf ${PKGD}/kernel
> -}
> -
> sysroot_stage_all_append() {
> - sysroot_stage_dir ${D}/kernel ${SYSROOT_DESTDIR}/kernel
> + sysroot_stage_dir ${D}${KERNEL_SRC_PATH} ${SYSROOT_DESTDIR}${KERNEL_SRC_PATH}
> }
>
> kernel_do_configure() {
> @@ -252,13 +251,11 @@ EXPORT_FUNCTIONS do_compile do_install do_configure
>
> # kernel-base becomes kernel-${KERNEL_VERSION}
> # kernel-image becomes kernel-image-${KERNEL_VERISON}
> -PACKAGES = "kernel kernel-base kernel-vmlinux kernel-image kernel-dev kernel-misc"
> +PACKAGES = "kernel kernel-base kernel-vmlinux kernel-image kernel-dev"
> FILES = ""
> FILES_kernel-image = "/boot/${KERNEL_IMAGETYPE}*"
> -FILES_kernel-dev = "/boot/System.map* /boot/Module.symvers* /boot/config*"
> +FILES_kernel-dev = "/boot/System.map* /boot/Module.symvers* /boot/config* ${KERNEL_SRC_PATH}"
> FILES_kernel-vmlinux = "/boot/vmlinux*"
> -# misc is a package to contain files we need in staging
> -FILES_kernel-misc = "/kernel/include/config /kernel/scripts /kernel/drivers/crypto /kernel/drivers/media"
> RDEPENDS_kernel = "kernel-base"
> # Allow machines to override this dependency if kernel image files are
> # not wanted in images as standard
> @@ -475,7 +472,7 @@ python populate_packages_prepend () {
> metapkg = "kernel-modules"
> d.setVar('ALLOW_EMPTY_' + metapkg, "1")
> d.setVar('FILES_' + metapkg, "")
> - blacklist = [ 'kernel-dev', 'kernel-image', 'kernel-base', 'kernel-vmlinux', 'kernel-misc' ]
> + blacklist = [ 'kernel-dev', 'kernel-image', 'kernel-base', 'kernel-vmlinux' ]
> for l in module_deps.values():
> for i in l:
> pkg = module_pattern % legitimize_package_name(re.match(module_regex, os.path.basename(i)).group(1))
> --
> 1.7.7.6
>
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [meta-oe][for-denzil][PATCH 3/8] kernel: Add kernel headers to kernel-dev package
2012-09-06 16:43 ` McClintock Matthew-B29882
@ 2012-09-06 16:47 ` Koen Kooi
2012-09-06 18:13 ` McClintock Matthew-B29882
0 siblings, 1 reply; 17+ messages in thread
From: Koen Kooi @ 2012-09-06 16:47 UTC (permalink / raw)
To: McClintock Matthew-B29882
Cc: openembedded-devel@lists.openembedded.org, Eric Bénard
Op 6 sep. 2012, om 18:43 heeft McClintock Matthew-B29882 <B29882@freescale.com> het volgende geschreven:
> On Thu, Sep 6, 2012 at 6:23 AM, Koen Kooi <koen@dominion.thruhere.net> wrote:
>> From: Darren Hart <dvhart@linux.intel.com>
>>
>> [YOCTO #1614]
>>
>> Add the kernel headers to the kernel-dev package. This packages what was
>> already built and kept in sysroots for building modules with bitbake.
>> Making this available on the target requires removing some additional
>> host binaries.
>>
>> Move the location to /usr/src/kernel
>>
>> Before use on the target, the user will need to:
>>
>> # cd /usr/src/kernel
>> # make scripts
>>
>> This renders the kernel-misc recipe empty, so remove it.
>>
>> As we use /usr/src/kernel in several places (and I missed one in the
>> previous version), add a KERNEL_SRC_DIR variable and use that throughout
>> the class to avoid update errors in the future.
>>
>> Now that we package the kernel headers, drop the
>> kernel_package_preprocess function which removed them from PKGD.
>>
>> All *-sdk image recipes include dev-pkgs, so the kernel-dev package will
>> be installed by default on all such images.
>>
>> Signed-off-by: Darren Hart <dvhart@linux.intel.com>
>> CC: Bruce Ashfield <bruce.ashfield@windriver.com>
>> CC: Tom Zanussi <tom.zanussi@intel.com>
>> CC: Khem Raj <raj.khem@gmail.com>
>> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
>> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
>
> Why did you resent this? Are there changes or by accident? Just asking
> before I make a patch for denzil with my previous fix.
This is for the meta-oe kernel.bbclass, not the oe-core one :)
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [meta-oe][for-denzil][PATCH 3/8] kernel: Add kernel headers to kernel-dev package
2012-09-06 16:47 ` Koen Kooi
@ 2012-09-06 18:13 ` McClintock Matthew-B29882
0 siblings, 0 replies; 17+ messages in thread
From: McClintock Matthew-B29882 @ 2012-09-06 18:13 UTC (permalink / raw)
To: openembedded-devel@lists.openembedded.org
Cc: McClintock Matthew-B29882, Eric Bénard
On Thu, Sep 6, 2012 at 11:47 AM, Koen Kooi <koen@dominion.thruhere.net> wrote:
>
> Op 6 sep. 2012, om 18:43 heeft McClintock Matthew-B29882 <B29882@freescale.com> het volgende geschreven:
>
>> On Thu, Sep 6, 2012 at 6:23 AM, Koen Kooi <koen@dominion.thruhere.net> wrote:
>>> From: Darren Hart <dvhart@linux.intel.com>
>>>
>>> [YOCTO #1614]
>>>
>>> Add the kernel headers to the kernel-dev package. This packages what was
>>> already built and kept in sysroots for building modules with bitbake.
>>> Making this available on the target requires removing some additional
>>> host binaries.
>>>
>>> Move the location to /usr/src/kernel
>>>
>>> Before use on the target, the user will need to:
>>>
>>> # cd /usr/src/kernel
>>> # make scripts
>>>
>>> This renders the kernel-misc recipe empty, so remove it.
>>>
>>> As we use /usr/src/kernel in several places (and I missed one in the
>>> previous version), add a KERNEL_SRC_DIR variable and use that throughout
>>> the class to avoid update errors in the future.
>>>
>>> Now that we package the kernel headers, drop the
>>> kernel_package_preprocess function which removed them from PKGD.
>>>
>>> All *-sdk image recipes include dev-pkgs, so the kernel-dev package will
>>> be installed by default on all such images.
>>>
>>> Signed-off-by: Darren Hart <dvhart@linux.intel.com>
>>> CC: Bruce Ashfield <bruce.ashfield@windriver.com>
>>> CC: Tom Zanussi <tom.zanussi@intel.com>
>>> CC: Khem Raj <raj.khem@gmail.com>
>>> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
>>> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
>>
>> Why did you resent this? Are there changes or by accident? Just asking
>> before I make a patch for denzil with my previous fix.
>
> This is for the meta-oe kernel.bbclass, not the oe-core one :)
Ah! Missed that Thanks.
-M
^ permalink raw reply [flat|nested] 17+ messages in thread
* [meta-oe][for-denzil][PATCH 4/8] kernel.bbclass: Dont package kxgettext.o
2012-09-06 11:23 [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning Koen Kooi
2012-09-06 11:23 ` [meta-oe][for-denzil][PATCH 2/8] recipes-kernel: make perf a standalone package Koen Kooi
2012-09-06 11:23 ` [meta-oe][for-denzil][PATCH 3/8] kernel: Add kernel headers to kernel-dev package Koen Kooi
@ 2012-09-06 11:23 ` Koen Kooi
2012-09-06 11:23 ` [meta-oe][for-denzil][PATCH 5/8] kernel.bbclass: add deploy link to KERNEL_IMAGETYPE Koen Kooi
` (4 subsequent siblings)
7 siblings, 0 replies; 17+ messages in thread
From: Koen Kooi @ 2012-09-06 11:23 UTC (permalink / raw)
To: openembedded-devel; +Cc: Koen Kooi, Eric Bénard
From: Khem Raj <raj.khem@gmail.com>
kxgettext.o is generated when building ppc kernels
so we end up with packaging errors like
> ERROR: QA Issue: Architecture did not match (20 to 62) on
> /work/virtex5-poky-linux/linux-xilinx-2.6.38-r00/packages-split/kernel-dev/usr/src/kernel/scripts/kconfig/kxgettext.o
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Signed-off-by: Saul Wold <sgw@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
---
meta-oe/classes/kernel.bbclass | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/meta-oe/classes/kernel.bbclass b/meta-oe/classes/kernel.bbclass
index d7e42f6..b2f7d0c 100644
--- a/meta-oe/classes/kernel.bbclass
+++ b/meta-oe/classes/kernel.bbclass
@@ -196,7 +196,7 @@ kernel_do_install() {
# during do_package for cross-compiled platforms
bin_files="arch/powerpc/boot/addnote arch/powerpc/boot/hack-coff \
arch/powerpc/boot/mktree scripts/kconfig/zconf.tab.o \
- scripts/kconfig/conf.o"
+ scripts/kconfig/conf.o scripts/kconfig/kxgettext.o"
for entry in $bin_files; do
rm -f $kerneldir/$entry
done
--
1.7.7.6
^ permalink raw reply related [flat|nested] 17+ messages in thread* [meta-oe][for-denzil][PATCH 5/8] kernel.bbclass: add deploy link to KERNEL_IMAGETYPE
2012-09-06 11:23 [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning Koen Kooi
` (2 preceding siblings ...)
2012-09-06 11:23 ` [meta-oe][for-denzil][PATCH 4/8] kernel.bbclass: Dont package kxgettext.o Koen Kooi
@ 2012-09-06 11:23 ` Koen Kooi
2012-09-06 11:23 ` [meta-oe][for-denzil][PATCH 6/8] kernel.bbclass: replace os.system with subprocess.call Koen Kooi
` (3 subsequent siblings)
7 siblings, 0 replies; 17+ messages in thread
From: Koen Kooi @ 2012-09-06 11:23 UTC (permalink / raw)
To: openembedded-devel; +Cc: Koen Kooi, Eric Bénard
From: Christopher Larson <chris_larson@mentor.com>
It's common to provide a non-machine-suffixed link in DEPLOY_DIR_IMAGE, so
let's be consistent and do so here as well.
Signed-off-by: Christopher Larson <chris_larson@mentor.com>
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
---
meta-oe/classes/kernel.bbclass | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/meta-oe/classes/kernel.bbclass b/meta-oe/classes/kernel.bbclass
index b2f7d0c..f68d8a9 100644
--- a/meta-oe/classes/kernel.bbclass
+++ b/meta-oe/classes/kernel.bbclass
@@ -541,6 +541,7 @@ kernel_do_deploy() {
cd ${DEPLOYDIR}
rm -f ${KERNEL_IMAGE_SYMLINK_NAME}.bin
ln -sf ${KERNEL_IMAGE_BASE_NAME}.bin ${KERNEL_IMAGE_SYMLINK_NAME}.bin
+ ln -sf ${KERNEL_IMAGE_BASE_NAME}.bin ${KERNEL_IMAGETYPE}
cp ${COREBASE}/meta/files/deploydir_readme.txt ${DEPLOYDIR}/README_-_DO_NOT_DELETE_FILES_IN_THIS_DIRECTORY.txt
}
--
1.7.7.6
^ permalink raw reply related [flat|nested] 17+ messages in thread* [meta-oe][for-denzil][PATCH 6/8] kernel.bbclass: replace os.system with subprocess.call
2012-09-06 11:23 [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning Koen Kooi
` (3 preceding siblings ...)
2012-09-06 11:23 ` [meta-oe][for-denzil][PATCH 5/8] kernel.bbclass: add deploy link to KERNEL_IMAGETYPE Koen Kooi
@ 2012-09-06 11:23 ` Koen Kooi
2012-09-06 11:23 ` [meta-oe][for-denzil][PATCH 7/8] kernel.bbclass: fix external module building Koen Kooi
` (2 subsequent siblings)
7 siblings, 0 replies; 17+ messages in thread
From: Koen Kooi @ 2012-09-06 11:23 UTC (permalink / raw)
To: openembedded-devel; +Cc: Koen Kooi, Eric Bénard
From: Robert Yang <liezhi.yang@windriver.com>
Replace os.system with subprocess.call since the older function would
fail (more or less) silently if the executed program cannot be found
More info:
http://docs.python.org/library/subprocess.html#subprocess-replacements
[YOCTO #2454]
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
---
meta-oe/classes/kernel.bbclass | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/meta-oe/classes/kernel.bbclass b/meta-oe/classes/kernel.bbclass
index f68d8a9..d5b3914 100644
--- a/meta-oe/classes/kernel.bbclass
+++ b/meta-oe/classes/kernel.bbclass
@@ -315,12 +315,12 @@ module_conf_rfcomm = "alias bt-proto-3 rfcomm"
python populate_packages_prepend () {
def extract_modinfo(file):
- import tempfile, re
+ import tempfile, re, subprocess
tempfile.tempdir = d.getVar("WORKDIR", True)
tf = tempfile.mkstemp()
tmpfile = tf[1]
cmd = "PATH=\"%s\" %sobjcopy -j .modinfo -O binary %s %s" % (d.getVar("PATH", True), d.getVar("HOST_PREFIX", True) or "", file, tmpfile)
- os.system(cmd)
+ subprocess.call(cmd, shell=True)
f = open(tmpfile)
l = f.read().split("\000")
f.close()
--
1.7.7.6
^ permalink raw reply related [flat|nested] 17+ messages in thread* [meta-oe][for-denzil][PATCH 7/8] kernel.bbclass: fix external module building
2012-09-06 11:23 [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning Koen Kooi
` (4 preceding siblings ...)
2012-09-06 11:23 ` [meta-oe][for-denzil][PATCH 6/8] kernel.bbclass: replace os.system with subprocess.call Koen Kooi
@ 2012-09-06 11:23 ` Koen Kooi
2012-09-06 11:23 ` [meta-oe][for-denzil][PATCH 8/8] kernel.bbclass: add non-santized kernel provides Koen Kooi
2012-09-06 11:41 ` [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning Koen Kooi
7 siblings, 0 replies; 17+ messages in thread
From: Koen Kooi @ 2012-09-06 11:23 UTC (permalink / raw)
To: openembedded-devel; +Cc: Koen Kooi, Eric Bénard
From: Denis Carikli <GNUtoo@no-log.org>
Without that fix we have the following while compiling compat-wireless.
include/linux/mmzone.h:18:30: fatal error: generated/bounds.h: No such file or directory
Note that the compat-wireless recipe will be added in another commit.
Signed-off-by: Denis 'GNUtoo' Carikli <GNUtoo@no-log.org>
Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
---
meta-oe/classes/kernel.bbclass | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/meta-oe/classes/kernel.bbclass b/meta-oe/classes/kernel.bbclass
index d5b3914..19fc4c6 100644
--- a/meta-oe/classes/kernel.bbclass
+++ b/meta-oe/classes/kernel.bbclass
@@ -192,6 +192,9 @@ kernel_do_install() {
cp arch/powerpc/lib/crtsavres.o $kerneldir/arch/powerpc/lib/crtsavres.o
fi
+ # Necessary for building modules like compat-wireless.
+ cp include/generated/bounds.h $kerneldir/include/generated/bounds.h
+
# Remove the following binaries which cause strip or arch QA errors
# during do_package for cross-compiled platforms
bin_files="arch/powerpc/boot/addnote arch/powerpc/boot/hack-coff \
--
1.7.7.6
^ permalink raw reply related [flat|nested] 17+ messages in thread* [meta-oe][for-denzil][PATCH 8/8] kernel.bbclass: add non-santized kernel provides
2012-09-06 11:23 [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning Koen Kooi
` (5 preceding siblings ...)
2012-09-06 11:23 ` [meta-oe][for-denzil][PATCH 7/8] kernel.bbclass: fix external module building Koen Kooi
@ 2012-09-06 11:23 ` Koen Kooi
2012-09-06 11:41 ` [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning Koen Kooi
7 siblings, 0 replies; 17+ messages in thread
From: Koen Kooi @ 2012-09-06 11:23 UTC (permalink / raw)
To: openembedded-devel; +Cc: Koen Kooi, Eric Bénard
From: Bruce Ashfield <bruce.ashfield@windriver.com>
If the kernel version string uses characters or symbols that
need to be santized for the package name, we can end up with a
mismatch between module requirements and what the kernel
provides.
The kernel version is pulled from utsrelease.h, which contains
the exact string that was passed to the kernel build, not
one that is santized, this can result in:
echo "CONFIG_LOCALVERSION="\"MYVER+snapshot_standard\" >> ${B}/.config
<build>
% rpm -qp kernel-module-uvesafb-3.4-r0.qemux86.rpm --requires
update-modules
kernel-3.4.3-MYVER+snapshot_standard
% rpm -qp kernel-3.4.3-myver+snapshot-standard-3.4-r0.qemux86.rpm --provides
kernel-3.4.3-myver+snapshot-standard = 3.4-r0
At rootfs assembly time, we'll have a dependency issue with the kernel
providing the santizied string and the modules requiring the utsrelease.h
string.
To not break existing use cases, we can add a second provides to the
kernel packaging with the unsantized version string, and allowing the
kernel module packaging to be unchanged.
RPROVIDES_kernel-base += "kernel-${KERNEL_VERSION}"
% rpm -qp kernel-3.4.3-myver+snapshot-standard-3.4-r0.qemux86.rpm --provides
kernel-3.4.3-MYVER+snapshot_standard
kernel-3.4.3-myver+snapshot-standard = 3.4-r0
Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
---
meta-oe/classes/kernel.bbclass | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/meta-oe/classes/kernel.bbclass b/meta-oe/classes/kernel.bbclass
index 19fc4c6..7b80e9a 100644
--- a/meta-oe/classes/kernel.bbclass
+++ b/meta-oe/classes/kernel.bbclass
@@ -265,6 +265,7 @@ RDEPENDS_kernel = "kernel-base"
RDEPENDS_kernel-base ?= "kernel-image"
PKG_kernel-image = "kernel-image-${@legitimize_package_name('${KERNEL_VERSION}')}"
PKG_kernel-base = "kernel-${@legitimize_package_name('${KERNEL_VERSION}')}"
+RPROVIDES_kernel-base += "kernel-${KERNEL_VERSION}"
ALLOW_EMPTY_kernel = "1"
ALLOW_EMPTY_kernel-base = "1"
ALLOW_EMPTY_kernel-image = "1"
--
1.7.7.6
^ permalink raw reply related [flat|nested] 17+ messages in thread* Re: [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning
2012-09-06 11:23 [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning Koen Kooi
` (6 preceding siblings ...)
2012-09-06 11:23 ` [meta-oe][for-denzil][PATCH 8/8] kernel.bbclass: add non-santized kernel provides Koen Kooi
@ 2012-09-06 11:41 ` Koen Kooi
2012-10-10 21:32 ` Denys Dmytriyenko
7 siblings, 1 reply; 17+ messages in thread
From: Koen Kooi @ 2012-09-06 11:41 UTC (permalink / raw)
To: openembedded-devel
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
One day I will figure out how to write a cover letter :)
The matching OE-core patchset is in denzil-next for testing and will
hopefully get merged soon (provided it passes the testsuite).
My main objective is to support on-target builds of external kernel-modules
and initial reports are that it is working as expected.
Since there's a bitbake.conf change needed the matching oe-core set needs to
go in first and ideally this goes in right after that (provided this passes
the testsuite as well).
As you can imagine, this kind of inter-layer dependency sucks, but I don't
see a way around it in this case with the bitbake.conf change.
Op 06-09-12 13:23, Koen Kooi schreef:
> From: Bruce Ashfield <bruce.ashfield@windriver.com>
>
> The kernel source tree in the sysroot has all unecessary source code
> removed. The existing use case is to support module building out of the
> sysroot, but as more toolsa are moved into the kernel tree itself there
> are new use cases for the kernel sysroot source.
>
> To avoid putting dependencies on the kernel, and to be able to
> individually build and package these tools out of the source tree, we can
> save $kerndir/tools and $kernddir/lib from being removed. This enables
> tools like perf to be built our of the kernel source in the sysroot,
> without significantly increasing the amount of source in the sysroot.
>
> Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com> Signed-off-by: Koen
> Kooi <koen@dominion.thruhere.net> --- meta-oe/classes/kernel.bbclass |
> 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/meta-oe/classes/kernel.bbclass
> b/meta-oe/classes/kernel.bbclass index ce82351..c44a2f5 100644 ---
> a/meta-oe/classes/kernel.bbclass +++ b/meta-oe/classes/kernel.bbclass @@
> -178,7 +178,7 @@ kernel_do_install() { # oe_runmake -C $kerneldir
> CC="${KERNEL_CC}" LD="${KERNEL_LD}" clean make -C $kerneldir
> _mrproper_scripts - find $kerneldir -path $kerneldir/scripts -prune -o
> -name "*.[csS]" -exec rm '{}' \; + find $kerneldir -path $kerneldir/lib
> -prune -o -path $kerneldir/tools -prune -o -path $kerneldir/scripts
> -prune -o -name "*.[csS]" -exec rm '{}' \; find $kerneldir/Documentation
> -name "*.txt" -exec rm '{}' \;
>
> # As of Linux kernel version 3.0.1, the clean target removes
>
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)
Comment: GPGTools - http://gpgtools.org
iD8DBQFQSIwFMkyGM64RGpERAjCgAJ9PFqqoYUsY+deeN9gVy89hod+TqACeJFO3
hJ8tR0ZLZeookvwClEGnpD8=
=CRRt
-----END PGP SIGNATURE-----
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning
2012-09-06 11:41 ` [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning Koen Kooi
@ 2012-10-10 21:32 ` Denys Dmytriyenko
2012-10-15 19:23 ` Denys Dmytriyenko
0 siblings, 1 reply; 17+ messages in thread
From: Denys Dmytriyenko @ 2012-10-10 21:32 UTC (permalink / raw)
To: openembedded-devel
On Thu, Sep 06, 2012 at 01:41:57PM +0200, Koen Kooi wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> One day I will figure out how to write a cover letter :)
>
> The matching OE-core patchset is in denzil-next for testing and will
> hopefully get merged soon (provided it passes the testsuite).
>
> My main objective is to support on-target builds of external kernel-modules
> and initial reports are that it is working as expected.
>
> Since there's a bitbake.conf change needed the matching oe-core set needs to
> go in first and ideally this goes in right after that (provided this passes
> the testsuite as well).
>
> As you can imagine, this kind of inter-layer dependency sucks, but I don't
> see a way around it in this case with the bitbake.conf change.
So, what happened to this patchset? Can we get it pushed in to meta-oe/denzil
now, since corresponding changes to oe-core/denzil finally got in couple
weeks ago and it breaks external kernel-modules now? Thanks!
--
Denys
> Op 06-09-12 13:23, Koen Kooi schreef:
> > From: Bruce Ashfield <bruce.ashfield@windriver.com>
> >
> > The kernel source tree in the sysroot has all unecessary source code
> > removed. The existing use case is to support module building out of the
> > sysroot, but as more toolsa are moved into the kernel tree itself there
> > are new use cases for the kernel sysroot source.
> >
> > To avoid putting dependencies on the kernel, and to be able to
> > individually build and package these tools out of the source tree, we can
> > save $kerndir/tools and $kernddir/lib from being removed. This enables
> > tools like perf to be built our of the kernel source in the sysroot,
> > without significantly increasing the amount of source in the sysroot.
> >
> > Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> > Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com> Signed-off-by: Koen
> > Kooi <koen@dominion.thruhere.net> --- meta-oe/classes/kernel.bbclass |
> > 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/meta-oe/classes/kernel.bbclass
> > b/meta-oe/classes/kernel.bbclass index ce82351..c44a2f5 100644 ---
> > a/meta-oe/classes/kernel.bbclass +++ b/meta-oe/classes/kernel.bbclass @@
> > -178,7 +178,7 @@ kernel_do_install() { # oe_runmake -C $kerneldir
> > CC="${KERNEL_CC}" LD="${KERNEL_LD}" clean make -C $kerneldir
> > _mrproper_scripts - find $kerneldir -path $kerneldir/scripts -prune -o
> > -name "*.[csS]" -exec rm '{}' \; + find $kerneldir -path $kerneldir/lib
> > -prune -o -path $kerneldir/tools -prune -o -path $kerneldir/scripts
> > -prune -o -name "*.[csS]" -exec rm '{}' \; find $kerneldir/Documentation
> > -name "*.txt" -exec rm '{}' \;
> >
> > # As of Linux kernel version 3.0.1, the clean target removes
> >
>
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.5 (Darwin)
> Comment: GPGTools - http://gpgtools.org
>
> iD8DBQFQSIwFMkyGM64RGpERAjCgAJ9PFqqoYUsY+deeN9gVy89hod+TqACeJFO3
> hJ8tR0ZLZeookvwClEGnpD8=
> =CRRt
> -----END PGP SIGNATURE-----
>
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning
2012-10-10 21:32 ` Denys Dmytriyenko
@ 2012-10-15 19:23 ` Denys Dmytriyenko
2012-10-17 12:39 ` Koen Kooi
0 siblings, 1 reply; 17+ messages in thread
From: Denys Dmytriyenko @ 2012-10-15 19:23 UTC (permalink / raw)
To: openembedded-devel
ping! Can we get it pushed to denzil, please? Thanks.
On Wed, Oct 10, 2012 at 05:32:34PM -0400, Denys Dmytriyenko wrote:
> On Thu, Sep 06, 2012 at 01:41:57PM +0200, Koen Kooi wrote:
> > -----BEGIN PGP SIGNED MESSAGE-----
> > Hash: SHA1
> >
> > One day I will figure out how to write a cover letter :)
> >
> > The matching OE-core patchset is in denzil-next for testing and will
> > hopefully get merged soon (provided it passes the testsuite).
> >
> > My main objective is to support on-target builds of external kernel-modules
> > and initial reports are that it is working as expected.
> >
> > Since there's a bitbake.conf change needed the matching oe-core set needs to
> > go in first and ideally this goes in right after that (provided this passes
> > the testsuite as well).
> >
> > As you can imagine, this kind of inter-layer dependency sucks, but I don't
> > see a way around it in this case with the bitbake.conf change.
>
> So, what happened to this patchset? Can we get it pushed in to meta-oe/denzil
> now, since corresponding changes to oe-core/denzil finally got in couple
> weeks ago and it breaks external kernel-modules now? Thanks!
>
> --
> Denys
>
>
> > Op 06-09-12 13:23, Koen Kooi schreef:
> > > From: Bruce Ashfield <bruce.ashfield@windriver.com>
> > >
> > > The kernel source tree in the sysroot has all unecessary source code
> > > removed. The existing use case is to support module building out of the
> > > sysroot, but as more toolsa are moved into the kernel tree itself there
> > > are new use cases for the kernel sysroot source.
> > >
> > > To avoid putting dependencies on the kernel, and to be able to
> > > individually build and package these tools out of the source tree, we can
> > > save $kerndir/tools and $kernddir/lib from being removed. This enables
> > > tools like perf to be built our of the kernel source in the sysroot,
> > > without significantly increasing the amount of source in the sysroot.
> > >
> > > Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
> > > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> > > Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com> Signed-off-by: Koen
> > > Kooi <koen@dominion.thruhere.net> --- meta-oe/classes/kernel.bbclass |
> > > 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
> > >
> > > diff --git a/meta-oe/classes/kernel.bbclass
> > > b/meta-oe/classes/kernel.bbclass index ce82351..c44a2f5 100644 ---
> > > a/meta-oe/classes/kernel.bbclass +++ b/meta-oe/classes/kernel.bbclass @@
> > > -178,7 +178,7 @@ kernel_do_install() { # oe_runmake -C $kerneldir
> > > CC="${KERNEL_CC}" LD="${KERNEL_LD}" clean make -C $kerneldir
> > > _mrproper_scripts - find $kerneldir -path $kerneldir/scripts -prune -o
> > > -name "*.[csS]" -exec rm '{}' \; + find $kerneldir -path $kerneldir/lib
> > > -prune -o -path $kerneldir/tools -prune -o -path $kerneldir/scripts
> > > -prune -o -name "*.[csS]" -exec rm '{}' \; find $kerneldir/Documentation
> > > -name "*.txt" -exec rm '{}' \;
> > >
> > > # As of Linux kernel version 3.0.1, the clean target removes
> > >
> >
> > -----BEGIN PGP SIGNATURE-----
> > Version: GnuPG v1.4.5 (Darwin)
> > Comment: GPGTools - http://gpgtools.org
> >
> > iD8DBQFQSIwFMkyGM64RGpERAjCgAJ9PFqqoYUsY+deeN9gVy89hod+TqACeJFO3
> > hJ8tR0ZLZeookvwClEGnpD8=
> > =CRRt
> > -----END PGP SIGNATURE-----
> >
> >
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel@lists.openembedded.org
> > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning
2012-10-15 19:23 ` Denys Dmytriyenko
@ 2012-10-17 12:39 ` Koen Kooi
2012-10-17 13:26 ` Denys Dmytriyenko
2012-10-17 13:54 ` Eric Bénard
0 siblings, 2 replies; 17+ messages in thread
From: Koen Kooi @ 2012-10-17 12:39 UTC (permalink / raw)
To: openembedded-devel
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Op 15-10-12 21:23, Denys Dmytriyenko schreef:
> ping! Can we get it pushed to denzil, please? Thanks.
It's in now.
>
>
> On Wed, Oct 10, 2012 at 05:32:34PM -0400, Denys Dmytriyenko wrote:
>> On Thu, Sep 06, 2012 at 01:41:57PM +0200, Koen Kooi wrote:
> One day I will figure out how to write a cover letter :)
>
> The matching OE-core patchset is in denzil-next for testing and will
> hopefully get merged soon (provided it passes the testsuite).
>
> My main objective is to support on-target builds of external
> kernel-modules and initial reports are that it is working as expected.
>
> Since there's a bitbake.conf change needed the matching oe-core set needs
> to go in first and ideally this goes in right after that (provided this
> passes the testsuite as well).
>
> As you can imagine, this kind of inter-layer dependency sucks, but I
> don't see a way around it in this case with the bitbake.conf change.
>>>
>>> So, what happened to this patchset? Can we get it pushed in to
>>> meta-oe/denzil now, since corresponding changes to oe-core/denzil
>>> finally got in couple weeks ago and it breaks external kernel-modules
>>> now? Thanks!
>>>
>>> -- Denys
>>>
>>>
> Op 06-09-12 13:23, Koen Kooi schreef:
>>>>> From: Bruce Ashfield <bruce.ashfield@windriver.com>
>>>>>
>>>>> The kernel source tree in the sysroot has all unecessary source
>>>>> code removed. The existing use case is to support module building
>>>>> out of the sysroot, but as more toolsa are moved into the kernel
>>>>> tree itself there are new use cases for the kernel sysroot
>>>>> source.
>>>>>
>>>>> To avoid putting dependencies on the kernel, and to be able to
>>>>> individually build and package these tools out of the source
>>>>> tree, we can save $kerndir/tools and $kernddir/lib from being
>>>>> removed. This enables tools like perf to be built our of the
>>>>> kernel source in the sysroot, without significantly increasing
>>>>> the amount of source in the sysroot.
>>>>>
>>>>> Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
>>>>> Signed-off-by: Richard Purdie
>>>>> <richard.purdie@linuxfoundation.org> Signed-off-by: Martin Jansa
>>>>> <Martin.Jansa@gmail.com> Signed-off-by: Koen Kooi
>>>>> <koen@dominion.thruhere.net> --- meta-oe/classes/kernel.bbclass
>>>>> | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
>>>>>
>>>>> diff --git a/meta-oe/classes/kernel.bbclass
>>>>> b/meta-oe/classes/kernel.bbclass index ce82351..c44a2f5 100644
>>>>> --- a/meta-oe/classes/kernel.bbclass +++
>>>>> b/meta-oe/classes/kernel.bbclass @@ -178,7 +178,7 @@
>>>>> kernel_do_install() { # oe_runmake -C $kerneldir
>>>>> CC="${KERNEL_CC}" LD="${KERNEL_LD}" clean make -C $kerneldir
>>>>> _mrproper_scripts - find $kerneldir -path $kerneldir/scripts
>>>>> -prune -o -name "*.[csS]" -exec rm '{}' \; + find $kerneldir
>>>>> -path $kerneldir/lib -prune -o -path $kerneldir/tools -prune -o
>>>>> -path $kerneldir/scripts -prune -o -name "*.[csS]" -exec rm '{}'
>>>>> \; find $kerneldir/Documentation -name "*.txt" -exec rm '{}' \;
>>>>>
>>>>> # As of Linux kernel version 3.0.1, the clean target removes
>>>>>
>
>>>
>>>
>>> _______________________________________________ Openembedded-devel
>>> mailing list Openembedded-devel@lists.openembedded.org
>>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>>
>>
>>>
_______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)
Comment: GPGTools - http://gpgtools.org
iD8DBQFQfqcQMkyGM64RGpERAuo9AJ0Vvatb00Ssc6W9qjYogMnNeAY4/ACfcdsC
oGpHmBDk73Eu/iC8q3Ne2tY=
=88JB
-----END PGP SIGNATURE-----
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning
2012-10-17 12:39 ` Koen Kooi
@ 2012-10-17 13:26 ` Denys Dmytriyenko
2012-10-17 13:54 ` Eric Bénard
1 sibling, 0 replies; 17+ messages in thread
From: Denys Dmytriyenko @ 2012-10-17 13:26 UTC (permalink / raw)
To: openembedded-devel
On Wed, Oct 17, 2012 at 02:39:44PM +0200, Koen Kooi wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Op 15-10-12 21:23, Denys Dmytriyenko schreef:
> > ping! Can we get it pushed to denzil, please? Thanks.
>
> It's in now.
Thank you!
> > On Wed, Oct 10, 2012 at 05:32:34PM -0400, Denys Dmytriyenko wrote:
> >> On Thu, Sep 06, 2012 at 01:41:57PM +0200, Koen Kooi wrote:
> > One day I will figure out how to write a cover letter :)
> >
> > The matching OE-core patchset is in denzil-next for testing and will
> > hopefully get merged soon (provided it passes the testsuite).
> >
> > My main objective is to support on-target builds of external
> > kernel-modules and initial reports are that it is working as expected.
> >
> > Since there's a bitbake.conf change needed the matching oe-core set needs
> > to go in first and ideally this goes in right after that (provided this
> > passes the testsuite as well).
> >
> > As you can imagine, this kind of inter-layer dependency sucks, but I
> > don't see a way around it in this case with the bitbake.conf change.
> >>>
> >>> So, what happened to this patchset? Can we get it pushed in to
> >>> meta-oe/denzil now, since corresponding changes to oe-core/denzil
> >>> finally got in couple weeks ago and it breaks external kernel-modules
> >>> now? Thanks!
> >>>
> >>> -- Denys
> >>>
> >>>
> > Op 06-09-12 13:23, Koen Kooi schreef:
> >>>>> From: Bruce Ashfield <bruce.ashfield@windriver.com>
> >>>>>
> >>>>> The kernel source tree in the sysroot has all unecessary source
> >>>>> code removed. The existing use case is to support module building
> >>>>> out of the sysroot, but as more toolsa are moved into the kernel
> >>>>> tree itself there are new use cases for the kernel sysroot
> >>>>> source.
> >>>>>
> >>>>> To avoid putting dependencies on the kernel, and to be able to
> >>>>> individually build and package these tools out of the source
> >>>>> tree, we can save $kerndir/tools and $kernddir/lib from being
> >>>>> removed. This enables tools like perf to be built our of the
> >>>>> kernel source in the sysroot, without significantly increasing
> >>>>> the amount of source in the sysroot.
> >>>>>
> >>>>> Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
> >>>>> Signed-off-by: Richard Purdie
> >>>>> <richard.purdie@linuxfoundation.org> Signed-off-by: Martin Jansa
> >>>>> <Martin.Jansa@gmail.com> Signed-off-by: Koen Kooi
> >>>>> <koen@dominion.thruhere.net> --- meta-oe/classes/kernel.bbclass
> >>>>> | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
> >>>>>
> >>>>> diff --git a/meta-oe/classes/kernel.bbclass
> >>>>> b/meta-oe/classes/kernel.bbclass index ce82351..c44a2f5 100644
> >>>>> --- a/meta-oe/classes/kernel.bbclass +++
> >>>>> b/meta-oe/classes/kernel.bbclass @@ -178,7 +178,7 @@
> >>>>> kernel_do_install() { # oe_runmake -C $kerneldir
> >>>>> CC="${KERNEL_CC}" LD="${KERNEL_LD}" clean make -C $kerneldir
> >>>>> _mrproper_scripts - find $kerneldir -path $kerneldir/scripts
> >>>>> -prune -o -name "*.[csS]" -exec rm '{}' \; + find $kerneldir
> >>>>> -path $kerneldir/lib -prune -o -path $kerneldir/tools -prune -o
> >>>>> -path $kerneldir/scripts -prune -o -name "*.[csS]" -exec rm '{}'
> >>>>> \; find $kerneldir/Documentation -name "*.txt" -exec rm '{}' \;
> >>>>>
> >>>>> # As of Linux kernel version 3.0.1, the clean target removes
> >>>>>
> >
> >>>
> >>>
> >>> _______________________________________________ Openembedded-devel
> >>> mailing list Openembedded-devel@lists.openembedded.org
> >>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
> >>
> >>
> >>>
> _______________________________________________
> >> Openembedded-devel mailing list
> >> Openembedded-devel@lists.openembedded.org
> >> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.5 (Darwin)
> Comment: GPGTools - http://gpgtools.org
>
> iD8DBQFQfqcQMkyGM64RGpERAuo9AJ0Vvatb00Ssc6W9qjYogMnNeAY4/ACfcdsC
> oGpHmBDk73Eu/iC8q3Ne2tY=
> =88JB
> -----END PGP SIGNATURE-----
>
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [meta-oe][for-denzil][PATCH 1/8] kernel: save $kerndir/tools and $kerndir/lib from pruning
2012-10-17 12:39 ` Koen Kooi
2012-10-17 13:26 ` Denys Dmytriyenko
@ 2012-10-17 13:54 ` Eric Bénard
1 sibling, 0 replies; 17+ messages in thread
From: Eric Bénard @ 2012-10-17 13:54 UTC (permalink / raw)
To: openembedded-devel
Hi Koen,
Le Wed, 17 Oct 2012 14:39:44 +0200,
Koen Kooi <koen@dominion.thruhere.net> a écrit :
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Op 15-10-12 21:23, Denys Dmytriyenko schreef:
> > ping! Can we get it pushed to denzil, please? Thanks.
>
> It's in now.
>
oops I missed to merge that sorry I was traveling when I got your
email asking for the merge.
Eric
^ permalink raw reply [flat|nested] 17+ messages in thread