diff options
author | Johannes Berg <johannes.berg@intel.com> | 2024-03-28 10:06:38 +0100 |
---|---|---|
committer | Richard Weinberger <richard@nod.at> | 2024-04-22 22:30:11 +0200 |
commit | dac847ae2b718d41b72bd68eb911ca2862ecfb38 (patch) | |
tree | 36682bef75924a6b673288d345fe7861e0fd27ad /arch/um | |
parent | um: vector: remove unused len variable/calculation (diff) | |
download | linux-dac847ae2b718d41b72bd68eb911ca2862ecfb38.tar.xz linux-dac847ae2b718d41b72bd68eb911ca2862ecfb38.zip |
um: process: remove unused 'n' variable
The return value of fn() wasn't used for a long time,
so no need to assign it to a variable, addressing a
W=1 warning.
This seems to be - with patches from others posted to
the list before - the last W=1 warning in arch/um/.
Fixes: 22e2430d60db ("x86, um: convert to saner kernel_execve() semantics")
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Reviewed-by: Tiwei Bie <tiwei.btw@antgroup.com>
Signed-off-by: Richard Weinberger <richard@nod.at>
Diffstat (limited to 'arch/um')
-rw-r--r-- | arch/um/kernel/process.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/um/kernel/process.c b/arch/um/kernel/process.c index 1395ff3017f0..e5d7d24045a2 100644 --- a/arch/um/kernel/process.c +++ b/arch/um/kernel/process.c @@ -109,7 +109,7 @@ int get_current_pid(void) */ void new_thread_handler(void) { - int (*fn)(void *), n; + int (*fn)(void *); void *arg; if (current->thread.prev_sched != NULL) @@ -122,7 +122,7 @@ void new_thread_handler(void) /* * callback returns only if the kernel thread execs a process */ - n = fn(arg); + fn(arg); userspace(¤t->thread.regs.regs, current_thread_info()->aux_fp_regs); } |