From d3630f81747157198316fef874d59f9f120969d4 Mon Sep 17 00:00:00 2001
From: Richard Purdie <rpurdie@linux.intel.com>
Date: Tue, 10 Aug 2010 14:47:41 +0100
Subject: kernel/package.bbclass: Handle kernel module stripping centrally

Signed-off-by: Richard Purdie <rpurdie@linux.intel.com>
---
 meta/classes/kernel.bbclass       |  4 +---
 meta/classes/module_strip.bbclass | 22 ----------------------
 meta/classes/package.bbclass      |  4 ++++
 3 files changed, 5 insertions(+), 25 deletions(-)

(limited to 'meta/classes')

diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index 3963e38eb8..1390afce3f 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -218,15 +218,13 @@ 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-image kernel-dev kernel-vmlinux kernel-misc kernel-dbg"
+PACKAGES = "kernel kernel-base kernel-image kernel-dev kernel-vmlinux kernel-misc"
 FILES = ""
 FILES_kernel-image = "/boot/${KERNEL_IMAGETYPE}*"
 FILES_kernel-dev = "/boot/System.map* /boot/Module.symvers* /boot/config*"
 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"
-# We don't care about dbg files for the kernel but split them out here to keep the packaging clean
-FILES_kernel-dbg = "/lib/modules/${KERNEL_VERSION}/kernel/*/*/.debug /lib/modules/${KERNEL_VERSION}/kernel/*/.debug /lib/modules/${KERNEL_VERSION}/kernel/*/*/*/.debug"
 RDEPENDS_kernel = "kernel-base"
 # Allow machines to override this dependency if kernel image files are 
 # not wanted in images as standard
diff --git a/meta/classes/module_strip.bbclass b/meta/classes/module_strip.bbclass
index 7636f1f403..e69de29bb2 100644
--- a/meta/classes/module_strip.bbclass
+++ b/meta/classes/module_strip.bbclass
@@ -1,22 +0,0 @@
-#DEPENDS_append = " module-strip"
-
-do_strip_modules () {
-	for p in ${PACKAGES}; do
-		if test -e ${WORKDIR}/install/$p/lib/modules; then
-			modules="`find ${WORKDIR}/install/$p/lib/modules -name \*${KERNEL_OBJECT_SUFFIX}`"
-			if [ -n "$modules" ]; then
-				for module in $modules ; do
-					if ! [ -d "$module"  ] ; then
-						${STRIP} -v -g $module
-					fi
-				done
-#				NM="${STAGING_BINDIR_NATIVE}/${HOST_PREFIX}nm" OBJCOPY="${STAGING_BINDIR_NATIVE}/${HOST_PREFIX}objcopy" strip_module $modules
-			fi
-		fi
-	done
-}
-
-python do_package_append () {
-	if (bb.data.getVar('INHIBIT_PACKAGE_STRIP', d, 1) != '1'):
-		bb.build.exec_func('do_strip_modules', d)
-}
diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
index 8611d94774..d8e480f1ff 100644
--- a/meta/classes/package.bbclass
+++ b/meta/classes/package.bbclass
@@ -155,6 +155,10 @@ def runstrip(file, d):
     strip = bb.data.getVar("STRIP", d, True)
     objcopy = bb.data.getVar("OBJCOPY", d, True)
 
+    # Handle kernel modules specifically - .debug directories here are pointless
+    if file.find("/lib/modules/") != -1 and file.endswith(".ko"):
+        return os.system("%s'%s' -g --remove-section=.comment --remove-section=.note '%s'" % (pathprefix, strip, file))
+
     newmode = None
     if not os.access(file, os.W_OK):
         origmode = os.stat(file)[stat.ST_MODE]
-- 
cgit v1.2.3