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 <[email protected]> Reviewed-by: Tiwei Bie <[email protected]> Signed-off-by: Richard Weinberger <[email protected]>
1 parent 2caa498 commit dac847a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

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)