From 20bcf08698bbab76ccc7063f94615a07451aafa7 Mon Sep 17 00:00:00 2001 From: Felix Fietkau Date: Sun, 20 Dec 2009 04:41:10 +0000 Subject: kernel: fix plt fixup related crashes when loading kernel modules on mips, (happened when there was not enough physically contiguous memory available) git-svn-id: svn://svn.openwrt.org/openwrt/trunk@18858 3c298f89-4303-0410-b956-a3cf2f4a3e73 --- .../patches-2.6.31/027-mips_module_reloc.patch | 113 ++++++++++++++------- 1 file changed, 75 insertions(+), 38 deletions(-) (limited to 'target/linux/generic-2.6/patches-2.6.31') diff --git a/target/linux/generic-2.6/patches-2.6.31/027-mips_module_reloc.patch b/target/linux/generic-2.6/patches-2.6.31/027-mips_module_reloc.patch index f65a09c94e..c0dc64e269 100644 --- a/target/linux/generic-2.6/patches-2.6.31/027-mips_module_reloc.patch +++ b/target/linux/generic-2.6/patches-2.6.31/027-mips_module_reloc.patch @@ -16,10 +16,10 @@ const struct exception_table_entry *dbe_start; const struct exception_table_entry *dbe_end; + -+ void *plt_tbl; -+ unsigned int core_plt_offset; -+ unsigned int core_plt_size; -+ unsigned int init_plt_offset; ++ void *phys_plt_tbl; ++ void *virt_plt_tbl; ++ unsigned int phys_plt_offset; ++ unsigned int virt_plt_offset; }; typedef uint8_t Elf64_Byte; /* Type for a 8-bit quantity. */ @@ -143,7 +143,7 @@ void *module_alloc(unsigned long size) { #ifdef MODULE_START -@@ -58,21 +169,68 @@ void *module_alloc(unsigned long size) +@@ -58,21 +169,99 @@ void *module_alloc(unsigned long size) return __vmalloc_area(area, GFP_KERNEL, PAGE_KERNEL); #else @@ -188,6 +188,25 @@ + } else { + vfree(module_region); + } ++} ++ ++static void *__module_alloc(int size, bool phys) ++{ ++ void *ptr; ++ ++ if (phys) ++ ptr = kmalloc(size, GFP_KERNEL); ++ else ++ ptr = vmalloc(size); ++ return ptr; ++} ++ ++static void __module_free(void *ptr) ++{ ++ if (is_phys_addr(ptr)) ++ kfree(ptr); ++ else ++ vfree(ptr); } int module_frob_arch_sections(Elf_Ehdr *hdr, Elf_Shdr *sechdrs, @@ -204,17 +223,29 @@ + core_size = get_plt_size(hdr, sechdrs, secstrings, symindex, false); + init_size = get_plt_size(hdr, sechdrs, secstrings, symindex, true); + -+ mod->arch.core_plt_offset = 0; -+ mod->arch.core_plt_size = core_size; -+ mod->arch.init_plt_offset = core_size; -+ mod->arch.plt_tbl = kmalloc(core_size + init_size, GFP_KERNEL); -+ if (!mod->arch.plt_tbl) ++ mod->arch.phys_plt_offset = 0; ++ mod->arch.virt_plt_offset = 0; ++ mod->arch.phys_plt_tbl = NULL; ++ mod->arch.virt_plt_tbl = NULL; ++ ++ if ((core_size + init_size) == 0) ++ return 0; ++ ++ mod->arch.phys_plt_tbl = __module_alloc(core_size + init_size, 1); ++ if (!mod->arch.phys_plt_tbl) + return -ENOMEM; ++ ++ mod->arch.virt_plt_tbl = __module_alloc(core_size + init_size, 0); ++ if (!mod->arch.virt_plt_tbl) { ++ __module_free(mod->arch.phys_plt_tbl); ++ mod->arch.phys_plt_tbl = NULL; ++ return -ENOMEM; ++ } + return 0; } -@@ -95,28 +253,40 @@ static int apply_r_mips_32_rela(struct m +@@ -95,28 +284,36 @@ static int apply_r_mips_32_rela(struct m return 0; } @@ -239,38 +270,35 @@ + /* adjust carry for addiu */ + if (v & 0x00008000) + v += 0x10000; -+ + +- *location = (*location & ~0x03ffffff) | +- ((*location + (v >> 2)) & 0x03ffffff); + tramp[0] = 0x3c190000 | (v >> 16); /* lui t9, hi16 */ + tramp[1] = 0x27390000 | (v & 0xffff); /* addiu t9, t9, lo16 */ + tramp[2] = 0x03200008; /* jr t9 */ + tramp[3] = 0x00000000; /* nop */ -- *location = (*location & ~0x03ffffff) | -- ((*location + (v >> 2)) & 0x03ffffff); +- return 0; + return (Elf_Addr) tramp; -+} -+ -+static Elf_Addr add_plt_entry(struct module *me, void *location, Elf_Addr v) -+{ -+ if (location >= me->module_core && -+ location < me->module_core + me->core_size) -+ return add_plt_entry_to(&me->arch.core_plt_offset, -+ me->arch.plt_tbl, v); -+ -+ if (location >= me->module_init && -+ location < me->module_init + me->init_size) -+ return add_plt_entry_to(&me->arch.init_plt_offset, -+ me->arch.plt_tbl, v); - - return 0; } -static int apply_r_mips_26_rela(struct module *me, u32 *location, Elf_Addr v) ++static Elf_Addr add_plt_entry(struct module *me, void *location, Elf_Addr v) ++{ ++ if (is_phys_addr(location)) ++ return add_plt_entry_to(&me->arch.phys_plt_offset, ++ me->arch.phys_plt_tbl, v); ++ else ++ return add_plt_entry_to(&me->arch.virt_plt_offset, ++ me->arch.virt_plt_tbl, v); ++ ++} ++ +static int set_r_mips_26(struct module *me, u32 *location, u32 ofs, Elf_Addr v) { if (v % 4) { pr_err("module %s: dangerous R_MIPS_26 RELArelocation\n", -@@ -125,17 +295,31 @@ static int apply_r_mips_26_rela(struct m +@@ -125,17 +322,31 @@ static int apply_r_mips_26_rela(struct m } if ((v & 0xf0000000) != (((unsigned long)location + 4) & 0xf0000000)) { @@ -305,18 +333,20 @@ static int apply_r_mips_hi16_rel(struct module *me, u32 *location, Elf_Addr v) { struct mips_hi16 *n; -@@ -400,11 +584,23 @@ int module_finalize(const Elf_Ehdr *hdr, +@@ -400,11 +611,32 @@ int module_finalize(const Elf_Ehdr *hdr, list_add(&me->arch.dbe_list, &dbe_list); spin_unlock_irq(&dbe_lock); } + + /* Get rid of the fixup trampoline if we're running the module + * from physically mapped address space */ -+ if (me->arch.core_plt_offset == 0 && -+ me->arch.init_plt_offset == me->arch.core_plt_size && -+ is_phys_addr(me->module_core)) { -+ kfree(me->arch.plt_tbl); -+ me->arch.plt_tbl = NULL; ++ if (me->arch.phys_plt_offset == 0) { ++ __module_free(me->arch.phys_plt_tbl); ++ me->arch.phys_plt_tbl = NULL; ++ } ++ if (me->arch.virt_plt_offset == 0) { ++ __module_free(me->arch.virt_plt_tbl); ++ me->arch.virt_plt_tbl = NULL; + } + return 0; @@ -324,8 +354,15 @@ void module_arch_cleanup(struct module *mod) { -+ if (mod->arch.plt_tbl) -+ kfree(mod->arch.plt_tbl); ++ if (mod->arch.phys_plt_tbl) { ++ __module_free(mod->arch.phys_plt_tbl); ++ mod->arch.phys_plt_tbl = NULL; ++ } ++ if (mod->arch.virt_plt_tbl) { ++ __module_free(mod->arch.virt_plt_tbl); ++ mod->arch.virt_plt_tbl = NULL; ++ } ++ spin_lock_irq(&dbe_lock); list_del(&mod->arch.dbe_list); spin_unlock_irq(&dbe_lock); -- cgit v1.2.3