Commit 5be2dcdb authored by James Morse's avatar James Morse Committed by sanglipeng
Browse files

module: Expose module_init_layout_section() 

stable inclusion
from stable-v5.10.194
commit ecd62c85120dc74b07c5ccd1532de121e81a4dba
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/I9399M

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=ecd62c85120dc74b07c5ccd1532de121e81a4dba



--------------------------------

commit 2abcc4b5 upstream.

module_init_layout_section() choses whether the core module loader
considers a section as init or not. This affects the placement of the
exit section when module unloading is disabled. This code will never run,
so it can be free()d once the module has been initialised.

arm and arm64 need to count the number of PLTs they need before applying
relocations based on the section name. The init PLTs are stored separately
so they can be free()d. arm and arm64 both use within_module_init() to
decide which list of PLTs to use when applying the relocation.

Because within_module_init()'s behaviour changes when module unloading
is disabled, both architecture would need to take this into account when
counting the PLTs.

Today neither architecture does this, meaning when module unloading is
disabled there are insufficient PLTs in the init section to load some
modules, resulting in warnings:
| WARNING: CPU: 2 PID: 51 at arch/arm64/kernel/module-plts.c:99 module_emit_plt_entry+0x184/0x1cc
| Modules linked in: crct10dif_common
| CPU: 2 PID: 51 Comm: modprobe Not tainted 6.5.0-rc4-yocto-standard-dirty #15208
| Hardware name: QEMU KVM Virtual Machine, BIOS 0.0.0 02/06/2015
| pstate: 20400005 (nzCv daif +PAN -UAO -TCO -DIT -SSBS BTYPE=--)
| pc : module_emit_plt_entry+0x184/0x1cc
| lr : module_emit_plt_entry+0x94/0x1cc
| sp : ffffffc0803bba60
[...]
| Call trace:
|  module_emit_plt_entry+0x184/0x1cc
|  apply_relocate_add+0x2bc/0x8e4
|  load_module+0xe34/0x1bd4
|  init_module_from_file+0x84/0xc0
|  __arm64_sys_finit_module+0x1b8/0x27c
|  invoke_syscall.constprop.0+0x5c/0x104
|  do_el0_svc+0x58/0x160
|  el0_svc+0x38/0x110
|  el0t_64_sync_handler+0xc0/0xc4
|  el0t_64_sync+0x190/0x194

Instead of duplicating module_init_layout_section()s logic, expose it.

Reported-by: default avatarAdam Johnston <adam.johnston@arm.com>
Fixes: 055f23b7 ("module: check for exit sections in layout_sections() instead of module_init_section()")
Cc: stable@vger.kernel.org
Signed-off-by: default avatarJames Morse <james.morse@arm.com>
Signed-off-by: default avatarLuis Chamberlain <mcgrof@kernel.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarsanglipeng <sanglipeng1@jd.com>
parent 755970ac

include/linux/moduleloader.h

+5 −0
Original line number Diff line number Diff line
@@ -39,6 +39,11 @@ bool module_init_section(const char *name); 
 */

bool module_exit_section(const char *name);



/* Describes whether within_module_init() will consider this an init section

 * or not. This behaviour changes with CONFIG_MODULE_UNLOAD.

 */

bool module_init_layout_section(const char *sname);



/*

 * Apply the given relocation to the (simplified) ELF.  Return -error

 * or 0.

kernel/module.c

+1 −1
Original line number Diff line number Diff line
@@ -2301,7 +2301,7 @@ void *__symbol_get(const char *symbol) 
}

EXPORT_SYMBOL_GPL(__symbol_get);



static bool module_init_layout_section(const char *sname)

bool module_init_layout_section(const char *sname)

{

#ifndef CONFIG_MODULE_UNLOAD

	if (module_exit_section(sname))