From: Gabor Juhos Date: Mon, 13 Jul 2009 09:34:49 +0000 (+0000) Subject: MIPS: fix loading of modules with unresolved weak symbols X-Git-Tag: reboot~22971 X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=091e02823996bd67001eb3680d999804cb82099f;p=openwrt%2Fopenwrt.git MIPS: fix loading of modules with unresolved weak symbols SVN-Revision: 16823 --- diff --git a/target/linux/generic-2.6/patches-2.6.28/025-mips_fix_loading_of_modules_with_unresolved_weak_sy.patch b/target/linux/generic-2.6/patches-2.6.28/025-mips_fix_loading_of_modules_with_unresolved_weak_sy.patch new file mode 100644 index 0000000000..20c6132e4f --- /dev/null +++ b/target/linux/generic-2.6/patches-2.6.28/025-mips_fix_loading_of_modules_with_unresolved_weak_sy.patch @@ -0,0 +1,42 @@ +From 819b4bda18d62b52d04789c4a8d4fc3fbf9ce242 Mon Sep 17 00:00:00 2001 +From: Gabor Juhos +Date: Mon, 13 Jul 2009 10:46:49 +0200 +Subject: [PATCH] MIPS: fix loading of modules with unresolved weak symbols + +Loading of modules with unresolved weak symbols fails on MIPS +since '88173507e4fc1e7ecd111b0565e8cba0cb7dae6d'. + +Modules: handle symbols that have a zero value + +The module subsystem cannot handle symbols that are zero. If symbols +are present that have a zero value then the module resolver prints out a +message that these symbols are unresolved. + +We have to use IS_ERR_VALUE() to check that a symbol has been resolved +or not. + +Signed-off-by: Gabor Juhos +--- + arch/mips/kernel/module.c | 4 ++-- + 1 files changed, 2 insertions(+), 2 deletions(-) + +--- a/arch/mips/kernel/module.c ++++ b/arch/mips/kernel/module.c +@@ -303,7 +303,7 @@ int apply_relocate(Elf_Shdr *sechdrs, co + /* This is the symbol it is referring to */ + sym = (Elf_Sym *)sechdrs[symindex].sh_addr + + ELF_MIPS_R_SYM(rel[i]); +- if (!sym->st_value) { ++ if (IS_ERR_VALUE(sym->st_value)) { + /* Ignore unresolved weak symbol */ + if (ELF_ST_BIND(sym->st_info) == STB_WEAK) + continue; +@@ -343,7 +343,7 @@ int apply_relocate_add(Elf_Shdr *sechdrs + /* This is the symbol it is referring to */ + sym = (Elf_Sym *)sechdrs[symindex].sh_addr + + ELF_MIPS_R_SYM(rel[i]); +- if (!sym->st_value) { ++ if (IS_ERR_VALUE(sym->st_value)) { + /* Ignore unresolved weak symbol */ + if (ELF_ST_BIND(sym->st_info) == STB_WEAK) + continue; diff --git a/target/linux/generic-2.6/patches-2.6.29/025-mips_fix_loading_of_modules_with_unresolved_weak_sy.patch b/target/linux/generic-2.6/patches-2.6.29/025-mips_fix_loading_of_modules_with_unresolved_weak_sy.patch new file mode 100644 index 0000000000..20c6132e4f --- /dev/null +++ b/target/linux/generic-2.6/patches-2.6.29/025-mips_fix_loading_of_modules_with_unresolved_weak_sy.patch @@ -0,0 +1,42 @@ +From 819b4bda18d62b52d04789c4a8d4fc3fbf9ce242 Mon Sep 17 00:00:00 2001 +From: Gabor Juhos +Date: Mon, 13 Jul 2009 10:46:49 +0200 +Subject: [PATCH] MIPS: fix loading of modules with unresolved weak symbols + +Loading of modules with unresolved weak symbols fails on MIPS +since '88173507e4fc1e7ecd111b0565e8cba0cb7dae6d'. + +Modules: handle symbols that have a zero value + +The module subsystem cannot handle symbols that are zero. If symbols +are present that have a zero value then the module resolver prints out a +message that these symbols are unresolved. + +We have to use IS_ERR_VALUE() to check that a symbol has been resolved +or not. + +Signed-off-by: Gabor Juhos +--- + arch/mips/kernel/module.c | 4 ++-- + 1 files changed, 2 insertions(+), 2 deletions(-) + +--- a/arch/mips/kernel/module.c ++++ b/arch/mips/kernel/module.c +@@ -303,7 +303,7 @@ int apply_relocate(Elf_Shdr *sechdrs, co + /* This is the symbol it is referring to */ + sym = (Elf_Sym *)sechdrs[symindex].sh_addr + + ELF_MIPS_R_SYM(rel[i]); +- if (!sym->st_value) { ++ if (IS_ERR_VALUE(sym->st_value)) { + /* Ignore unresolved weak symbol */ + if (ELF_ST_BIND(sym->st_info) == STB_WEAK) + continue; +@@ -343,7 +343,7 @@ int apply_relocate_add(Elf_Shdr *sechdrs + /* This is the symbol it is referring to */ + sym = (Elf_Sym *)sechdrs[symindex].sh_addr + + ELF_MIPS_R_SYM(rel[i]); +- if (!sym->st_value) { ++ if (IS_ERR_VALUE(sym->st_value)) { + /* Ignore unresolved weak symbol */ + if (ELF_ST_BIND(sym->st_info) == STB_WEAK) + continue; diff --git a/target/linux/generic-2.6/patches-2.6.30/026-mips_fix_loading_of_modules_with_unresolved_weak_sy.patch b/target/linux/generic-2.6/patches-2.6.30/026-mips_fix_loading_of_modules_with_unresolved_weak_sy.patch new file mode 100644 index 0000000000..20c6132e4f --- /dev/null +++ b/target/linux/generic-2.6/patches-2.6.30/026-mips_fix_loading_of_modules_with_unresolved_weak_sy.patch @@ -0,0 +1,42 @@ +From 819b4bda18d62b52d04789c4a8d4fc3fbf9ce242 Mon Sep 17 00:00:00 2001 +From: Gabor Juhos +Date: Mon, 13 Jul 2009 10:46:49 +0200 +Subject: [PATCH] MIPS: fix loading of modules with unresolved weak symbols + +Loading of modules with unresolved weak symbols fails on MIPS +since '88173507e4fc1e7ecd111b0565e8cba0cb7dae6d'. + +Modules: handle symbols that have a zero value + +The module subsystem cannot handle symbols that are zero. If symbols +are present that have a zero value then the module resolver prints out a +message that these symbols are unresolved. + +We have to use IS_ERR_VALUE() to check that a symbol has been resolved +or not. + +Signed-off-by: Gabor Juhos +--- + arch/mips/kernel/module.c | 4 ++-- + 1 files changed, 2 insertions(+), 2 deletions(-) + +--- a/arch/mips/kernel/module.c ++++ b/arch/mips/kernel/module.c +@@ -303,7 +303,7 @@ int apply_relocate(Elf_Shdr *sechdrs, co + /* This is the symbol it is referring to */ + sym = (Elf_Sym *)sechdrs[symindex].sh_addr + + ELF_MIPS_R_SYM(rel[i]); +- if (!sym->st_value) { ++ if (IS_ERR_VALUE(sym->st_value)) { + /* Ignore unresolved weak symbol */ + if (ELF_ST_BIND(sym->st_info) == STB_WEAK) + continue; +@@ -343,7 +343,7 @@ int apply_relocate_add(Elf_Shdr *sechdrs + /* This is the symbol it is referring to */ + sym = (Elf_Sym *)sechdrs[symindex].sh_addr + + ELF_MIPS_R_SYM(rel[i]); +- if (!sym->st_value) { ++ if (IS_ERR_VALUE(sym->st_value)) { + /* Ignore unresolved weak symbol */ + if (ELF_ST_BIND(sym->st_info) == STB_WEAK) + continue;