As a follow up to the changes to ensure that KBUILD_DEFCONFIG will
always get its defconfig into the processing, we also should ensure that
we don't get both a KBUILD_DEFCONFIG and a SRC_URI defconfig in the
configuration queue. If both are in the configuration queue, we end up
with competing values and a potentially long running check of 5000+
options x 2 against the final .config.

By removing the defconfig found from the SRC_URI when one is found
via KBUILD_DEFCONFIG, we ensure that only one will be processed.

Signed-off-by: Bruce Ashfield <bruce.ashfi...@windriver.com>
---
 meta/classes/kernel-yocto.bbclass | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/meta/classes/kernel-yocto.bbclass 
b/meta/classes/kernel-yocto.bbclass
index 2edf0fd15788..60979a218f12 100644
--- a/meta/classes/kernel-yocto.bbclass
+++ b/meta/classes/kernel-yocto.bbclass
@@ -107,6 +107,8 @@ do_kernel_metadata() {
                                cmp "${WORKDIR}/defconfig" 
"${S}/arch/${ARCH}/configs/${KBUILD_DEFCONFIG}"
                                if [ $? -ne 0 ]; then
                                        bbwarn "defconfig detected in WORKDIR. 
${KBUILD_DEFCONFIG} skipped"
+                               else
+                                       cp -f 
${S}/arch/${ARCH}/configs/${KBUILD_DEFCONFIG} ${WORKDIR}/defconfig
                                fi
                        else
                                cp -f 
${S}/arch/${ARCH}/configs/${KBUILD_DEFCONFIG} ${WORKDIR}/defconfig
@@ -117,10 +119,19 @@ do_kernel_metadata() {
                fi
        fi
 
-       sccs="$sccs ${@" ".join(find_sccs(d))}"
+       sccs_from_src_uri="${@" ".join(find_sccs(d))}"
        patches="${@" ".join(find_patches(d))}"
        feat_dirs="${@" ".join(find_kernel_feature_dirs(d))}"
 
+       # a quick check to make sure we don't have duplicate defconfigs
+       # If there's a defconfig in the SRC_URI, did we also have one from
+       # the KBUILD_DEFCONFIG processing above ?
+       if [ -n "$sccs" ]; then
+           # we did have a defconfig from above. remove any that might be in 
the src_uri
+           sccs_from_src_uri=$(echo $sccs_from_src_uri | sed 's/defconfig//g')
+       fi
+       sccs="$sccs $sccs_from_src_uri"
+
        # check for feature directories/repos/branches that were part of the
        # SRC_URI. If they were supplied, we convert them into include 
directives
        # for the update part of the process
-- 
2.5.0

-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to