diff mbox

Failing randconfig builds with CONFIG_TRIM_UNUSED_KSYMS

Message ID alpine.LFD.2.20.1604281248300.5091@knanqh.ubzr
State New
Headers show

Commit Message

Nicolas Pitre April 28, 2016, 8:56 p.m. UTC 
On Thu, 28 Apr 2016, Arnd Bergmann wrote:

> Hi Nico,

> 

> My randconfig build testing has encountered a couple of additional

> failures with CONFIG_TRIM_UNUSED_KSYMS=y, both rather rare at

> happening once in a few thousand randconfig builds. I have attached

> two .config files that presumably refer to different problems I

> see on linux-next:

> 

> 0x1A1E54C2_defconfig fails every time with

> ERROR: "__aeabi_idivmod" [lib/cordic.ko] undefined!

> 

> Apparently, this is because lib/cordic.ko is the only module using

> this symbol (some built-in code uses it) here.


Well, the actual explanation is that this module uses only one symbol. 
And when only one symbol was listed, it got ignored.

The fix goes like this:



Nicolas

Comments

Arnd Bergmann April 28, 2016, 9:20 p.m. UTC | #1 
On Thursday 28 April 2016 16:56:23 Nicolas Pitre wrote:
> On Thu, 28 Apr 2016, Arnd Bergmann wrote:

> Well, the actual explanation is that this module uses only one symbol. 

> And when only one symbol was listed, it got ignored.

 
Ok.

> 

> Now fixing this wasn't all that obvious either.  I had a really nice sed 

> rule to parse multiple instances per line, but sed regexp can only do 

> greedy matching. I found out how people work around that limitation but 

> that doesn't work well for string delimiters.

> 

> In the end the best workaround is simple: substitute any ';' with '\n':


Wow, I could have spent days trying to figure these out, glad I gave
up earlier ;-)

I've applied both patches locally now, and have successfully built
all the configurations that failed before. I'll let you know if something
else should come up.

Thanks,

	Arnd
diff mbox

Patch

diff --git a/scripts/adjust_autoksyms.sh b/scripts/adjust_autoksyms.sh
index 5bf538f1ed..3de40db0a1 100755
--- a/scripts/adjust_autoksyms.sh
+++ b/scripts/adjust_autoksyms.sh
@@ -59,7 +59,7 @@  cat > "$new_ksyms_file" << EOT
  */
 
 EOT
-sed -ns -e '3s/ /\n/gp' "$MODVERDIR"/*.mod | sort -u |
+sed -ns -e '3{s/ /\n/g;/^$/!p;}' "$MODVERDIR"/*.mod | sort -u |
 while read sym; do
 	if [ -n "$CONFIG_HAVE_UNDERSCORE_SYMBOL_PREFIX" ]; then
 		sym="${sym#_}"

> 0xEA8A78CD_defconfig is stranger, as it only sometime breaks when I build
> in a newly created object directory or after "make clean", but
> not if I retry the build:
> 
> $ rm -rf build/0xEA8A78CD
> $ mkdir build/0xEA8A78CD
> $ make O=build/0xEA8A78CD 0xEA8A78CD_defconfig
> $ make O=build/0xEA8A78CD -skj12
> ERROR: "memory_cgrp_subsys_enabled_key" [fs/ncpfs/ncpfs.ko] undefined!
> $ make O=build/0xEA8A78CD -skj12
> $ # SUCCESS

I don't understand why you'd get a successful build the second time.  
I'm able to reproduce, however it fails everytime.

This one was less obvious to solve.  The construct is:

#define SUBSYS(_x)                                                         \
        DEFINE_STATIC_KEY_TRUE(_x ## _cgrp_subsys_enabled_key);            \
        DEFINE_STATIC_KEY_TRUE(_x ## _cgrp_subsys_on_dfl_key);             \
        EXPORT_SYMBOL_GPL(_x ## _cgrp_subsys_enabled_key);                 \
        EXPORT_SYMBOL_GPL(_x ## _cgrp_subsys_on_dfl_key);

Duing the build, EXPORT_SYMBOL*() is redefined to anchor the symbol name 
so a sed script can extract them and feed them to scripts/basic/fixdep.  
However in this case the preprocessor output had more than one such 
symbol on a line and the sed script only captured one of them.  
Therefore the pseudo dependency file for memory_cgrp_subsys_enabled_key 
didn't get added to kernel/.cgroup.o.cmd. Even when autoksyms.h was 
updated with that symbol and the dependency file touched, the build 
system didn't know that kernel/cgroup.c had to be rebuilt.

Now fixing this wasn't all that obvious either.  I had a really nice sed 
rule to parse multiple instances per line, but sed regexp can only do 
greedy matching. I found out how people work around that limitation but 
that doesn't work well for string delimiters.

In the end the best workaround is simple: substitute any ';' with '\n':

diff --git a/scripts/Kbuild.include b/scripts/Kbuild.include
index 36e9475395..1f0d41cc73 100644
--- a/scripts/Kbuild.include
+++ b/scripts/Kbuild.include
@@ -281,7 +281,7 @@  ksym_dep_filter =                                                            \
 	    $(CPP) $(call flags_nodeps,a_flags) -D__KSYM_DEPS__ $< ;;        \
 	  boot*|build*|*cpp_lds_S|dtc|host*|vdso*) : ;;                      \
 	  *) echo "Don't know how to preprocess $(1)" >&2; false ;;          \
-	esac | sed -rn 's/^.*=== __KSYM_(.*) ===.*$$/KSYM_\1/p'
+	esac | tr ";" "\n" | sed -rn 's/^.*=== __KSYM_(.*) ===.*$$/KSYM_\1/p'
 
 cmd_and_fixdep =                                                             \
 	$(echo-cmd) $(cmd_$(1));                                             \