Skip to content

Commit dac847a

Browse files
jmberg-intelrichardweinberger
authored andcommitted
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: 22e2430 ("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>
1 parent 2caa498 commit dac847a

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

arch/um/kernel/process.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ int get_current_pid(void)
109109
*/
110110
void new_thread_handler(void)
111111
{
112-
int (*fn)(void *), n;
112+
int (*fn)(void *);
113113
void *arg;
114114

115115
if (current->thread.prev_sched != NULL)
@@ -122,7 +122,7 @@ void new_thread_handler(void)
122122
/*
123123
* callback returns only if the kernel thread execs a process
124124
*/
125-
n = fn(arg);
125+
fn(arg);
126126
userspace(&current->thread.regs.regs, current_thread_info()->aux_fp_regs);
127127
}
128128

0 commit comments

Comments
 (0)