diff options
author | Brian Gerst <brgerst@gmail.com> | 2023-07-21 12:10:14 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2023-10-05 10:06:42 +0200 |
commit | eec62f61e134d6711f98d4005c6439f24d03d54f (patch) | |
tree | 7619ac49eedb9b32f4ecfd26e80a4768fd598099 | |
parent | eb43c9b1517b48e2ff0d3a584aca197338987d7b (diff) | |
download | linux-eec62f61e134d6711f98d4005c6439f24d03d54f.tar.gz linux-eec62f61e134d6711f98d4005c6439f24d03d54f.tar.bz2 linux-eec62f61e134d6711f98d4005c6439f24d03d54f.zip |
x86/entry/compat: Combine return value test from syscall handler
Move the sysret32_from_system_call label to remove a duplicate test of
the return value from the syscall handler.
Signed-off-by: Brian Gerst <brgerst@gmail.com>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Link: https://lore.kernel.org/r/20230721161018.50214-4-brgerst@gmail.com
-rw-r--r-- | arch/x86/entry/entry_64_compat.S | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/arch/x86/entry/entry_64_compat.S b/arch/x86/entry/entry_64_compat.S index 70150298f8bd..b16272395f1a 100644 --- a/arch/x86/entry/entry_64_compat.S +++ b/arch/x86/entry/entry_64_compat.S @@ -118,9 +118,6 @@ SYM_INNER_LABEL(entry_SYSENTER_compat_after_hwframe, SYM_L_GLOBAL) movq %rsp, %rdi call do_SYSENTER_32 - /* XEN PV guests always use IRET path */ - ALTERNATIVE "testl %eax, %eax; jz swapgs_restore_regs_and_return_to_usermode", \ - "jmp swapgs_restore_regs_and_return_to_usermode", X86_FEATURE_XENPV jmp sysret32_from_system_call .Lsysenter_fix_flags: @@ -212,13 +209,15 @@ SYM_INNER_LABEL(entry_SYSCALL_compat_after_hwframe, SYM_L_GLOBAL) movq %rsp, %rdi call do_fast_syscall_32 + +sysret32_from_system_call: /* XEN PV guests always use IRET path */ ALTERNATIVE "testl %eax, %eax; jz swapgs_restore_regs_and_return_to_usermode", \ "jmp swapgs_restore_regs_and_return_to_usermode", X86_FEATURE_XENPV - /* Opportunistic SYSRET */ -sysret32_from_system_call: /* + * Opportunistic SYSRET + * * We are not going to return to userspace from the trampoline * stack. So let's erase the thread stack right now. */ |