James Morse 363bbb5008 module: Expose module_init_layout_section() ... commit 2abcc4b5a6 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: Adam Johnston <adam.johnston@arm.com> Fixes: 055f23b74b ("module: check for exit sections in layout_sections() instead of module_init_section()") Cc: stable@vger.kernel.org Signed-off-by: James Morse <james.morse@arm.com> Signed-off-by: Luis Chamberlain <mcgrof@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>		2023-09-02 09:17:07 +02:00
..
acpi
asm-generic	word-at-a-time: use the same return type for has_zero regardless of endianness	2023-08-11 15:13:49 +02:00
clocksource
crypto
drm	drm/display/dp: Fix the DP DSC Receiver cap size	2023-08-30 16:18:19 +02:00
dt-bindings
keys
kunit
kvm
linux	module: Expose module_init_layout_section()	2023-09-02 09:17:07 +02:00
math-emu
media	media: v4l2-mem2mem: add lock to protect parameter num_rdy	2023-08-26 14:23:23 +02:00
memory
misc
net	bonding: fix macvlan over alb bond support	2023-08-30 16:18:15 +02:00
pcmcia
ras
rdma
scsi
soc
sound
target
trace	jbd2: remove journal_clean_one_cp_list()	2023-08-30 16:18:12 +02:00
uapi	block: Fix a source code comment in include/uapi/linux/blkzoned.h	2023-08-03 10:22:38 +02:00
vdso
video
xen