diff mbox series

[RFC,v3,1/4] module: remove module_deallocate() unused param

Message ID 20241108-modules-ro_after_init-v3-1-6dd041b588a5@samsung.com (mailing list archive)
State Handled Elsewhere
Headers show
Series module: refactor ro_after_init failure path | expand

Checks

Context Check Description
mcgrof/vmtest-modules-next-VM_Test-0 fail Logs for Run CI tests
mcgrof/vmtest-modules-next-VM_Test-1 fail Logs for Run CI tests
mcgrof/vmtest-modules-next-VM_Test-4 success Logs for setup / Setup kdevops environment
mcgrof/vmtest-modules-next-VM_Test-5 success Logs for setup / Setup kdevops environment
mcgrof/vmtest-modules-next-PR fail PR summary
mcgrof/vmtest-modules-next-VM_Test-3 success Logs for cleanup / Archive results and cleanup
mcgrof/vmtest-modules-next-VM_Test-2 success Logs for cleanup / Archive results and cleanup

Commit Message

Daniel Gomez via B4 Relay Nov. 8, 2024, 4:12 p.m. UTC 
From: Daniel Gomez <da.gomez@samsung.com>

The module_deallocate() function no longer requires the struct load_info
parameter as of commit 48fd11880b5ef04270be8a87d9a9a9ee2fdae338 ("module:
Fix performance regression on modules with large symbol tables"). This
commit removes the now-unused parameter.

No functional changes.

Signed-off-by: Daniel Gomez <da.gomez@samsung.com>
---
 kernel/module/main.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/kernel/module/main.c b/kernel/module/main.c
index 5399c182b3cb..e0b2a8d9e324 100644
--- a/kernel/module/main.c
+++ b/kernel/module/main.c
@@ -2767,7 +2767,7 @@  static struct module *layout_and_allocate(struct load_info *info, int flags)
 }
 
 /* mod is no longer valid after this! */
-static void module_deallocate(struct module *mod, struct load_info *info)
+static void module_deallocate(struct module *mod)
 {
 	percpu_modfree(mod);
 	module_arch_freeing_init(mod);
@@ -3417,7 +3417,7 @@  static int load_module(struct load_info *info, const char __user *uargs,
 				       mod->mem[type].size);
 	}
 
-	module_deallocate(mod, info);
+	module_deallocate(mod);
  free_copy:
 	/*
 	 * The info->len is always set. We distinguish between