diff options
| author | Ingo Molnar <mingo@kernel.org> | 2024-03-29 14:20:40 +0100 |
|---|---|---|
| committer | Ingo Molnar <mingo@kernel.org> | 2024-03-29 14:24:50 +0100 |
| commit | 5ca28d24aecd0809d93da2ea73a4f6e4b2ccfa78 (patch) | |
| tree | 25dba28618eff62161da778373116378b0a6905d /arch | |
| parent | 5d31174f3c8c465d9dbe88f6b9d1fe5716f44981 (diff) | |
| download | linux-stable-5ca28d24aecd0809d93da2ea73a4f6e4b2ccfa78.tar.gz linux-stable-5ca28d24aecd0809d93da2ea73a4f6e4b2ccfa78.tar.bz2 linux-stable-5ca28d24aecd0809d93da2ea73a4f6e4b2ccfa78.zip | |
x86/vm86: Make sure the free_vm86(task) definition uses its parameter even in the !CONFIG_VM86 case
I ran into a case in new code where free_vm86(task) was the only
consumer of the 'task' variable:
arch/x86/kernel/process.c: In function ‘exit_thread’:
arch/x86/kernel/process.c:118:31: error: unused variable ‘t’ [-Werror=unused-variable]
Robustify the definition in the !CONFIG_VM86 case.
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Link: https://lore.kernel.org/r/ZgaFfyHMOdLHEKm+@gmail.com
Diffstat (limited to 'arch')
| -rw-r--r-- | arch/x86/include/asm/vm86.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/x86/include/asm/vm86.h b/arch/x86/include/asm/vm86.h index 9e8ac5073ecb..62ee19909903 100644 --- a/arch/x86/include/asm/vm86.h +++ b/arch/x86/include/asm/vm86.h @@ -84,7 +84,7 @@ static inline int handle_vm86_trap(struct kernel_vm86_regs *a, long b, int c) static inline void save_v86_state(struct kernel_vm86_regs *a, int b) { } -#define free_vm86(t) do { } while(0) +#define free_vm86(task) do { (void)(task); } while(0) #endif /* CONFIG_VM86 */ |