]> code.ossystems Code Review - openembedded-core.git/commitdiff
kernel/rm_work: Improve interaction
authorRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 14 May 2015 09:31:51 +0000 (10:31 +0100)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Fri, 15 May 2015 11:05:52 +0000 (12:05 +0100)
The do_shared_workdir task does leave behind the necessary information in
shared-work after it completes. We don't make this a "full" sstate task
however since that means tarring up and copying what is usually a large
amount of data which would be better extracted straight from the original
SCM.

The issue with rm_work occurs since it removes the do_shared_workdir stamp
meaning subsequent builds will add it back if they need to touch any kernel
modules for example. This ends up triggering a near enough complete kernerl
rebuild since if configure reruns, populate_sysroot has to rerun.

This change promotes the task to have a "setscene" variant but it doesn't use
any of the sstate class lifting to generate the sstate file. The sstate function
will therefore never get called since the sstate object will never exist.

We can add the task to the list of tasks rm_work promotes to a setscene variant
and unwanted rebuilds of the kernel should be avoided.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/classes/kernel.bbclass
meta/classes/rm_work.bbclass

index 74092e96aac56a88f09c506585a53f2db878a4ea..54725baaf1d2ccf423d3efb0538761a5e880b62e 100644 (file)
@@ -235,6 +235,11 @@ kernel_do_install() {
 do_install[prefuncs] += "package_get_auto_pr"
 
 addtask shared_workdir after do_compile before do_compile_kernelmodules
+addtask shared_workdir_setscene
+
+do_shared_workdir_setscene () {
+       exit 1
+}
 
 emit_depmod_pkgdata() {
        # Stash data for depmod
index e68d02a7839283eca83ab96d740bece2601d44c8..5e9efc1d06f388a8592e04e10abdf263c71cb08d 100644 (file)
@@ -49,7 +49,7 @@ do_rm_work () {
     cd `dirname ${STAMP}`
     for i in `basename ${STAMP}`*
     do
-        for j in ${SSTATETASKS}
+        for j in ${SSTATETASKS} do_shared_workdir
         do
             case $i in
             *do_setscene*)