wine-commits June 2023

wine-commits@winehq.org

1 participants
633 discussions

Alexandre Julliard : ntdll: Don't pass arguments on the stack for user callbacks on i386.
by Alexandre Julliard 01 Jun '23

01 Jun '23

Module: wine Branch: master Commit: d8c39b3d5ace3a7349cb2762aad84ced5bb1563a URL: https://gitlab.winehq.org/wine/wine/-/commit/d8c39b3d5ace3a7349cb2762aad84c… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Jun 1 12:44:21 2023 +0200 ntdll: Don't pass arguments on the stack for user callbacks on i386. --- dlls/ntdll/unix/signal_i386.c | 33 ++++++++++++++++----------------- 1 file changed, 16 insertions(+), 17 deletions(-) diff --git a/dlls/ntdll/unix/signal_i386.c b/dlls/ntdll/unix/signal_i386.c index 748828ad99e..28cd29f4960 100644 --- a/dlls/ntdll/unix/signal_i386.c +++ b/dlls/ntdll/unix/signal_i386.c @@ -1574,8 +1574,8 @@ NTSTATUS call_user_exception_dispatcher( EXCEPTION_RECORD *rec, CONTEXT *context /*********************************************************************** * call_user_mode_callback */ -extern NTSTATUS CDECL call_user_mode_callback( void *func, void *stack, void **ret_ptr, - ULONG *ret_len, TEB *teb ) DECLSPEC_HIDDEN; +extern NTSTATUS call_user_mode_callback( ULONG id, void *args, ULONG len, void **ret_ptr, + ULONG *ret_len, void *func, TEB *teb ) DECLSPEC_HIDDEN; __ASM_GLOBAL_FUNC( call_user_mode_callback, "pushl %ebp\n\t" __ASM_CFI(".cfi_adjust_cfa_offset 4\n\t") @@ -1588,7 +1588,7 @@ __ASM_GLOBAL_FUNC( call_user_mode_callback, __ASM_CFI(".cfi_rel_offset %esi,-8\n\t") "pushl %edi\n\t" __ASM_CFI(".cfi_rel_offset %edi,-12\n\t") - "movl 0x18(%ebp),%edx\n\t" /* teb */ + "movl 0x20(%ebp),%edx\n\t" /* teb */ "pushl 0(%edx)\n\t" /* teb->Tib.ExceptionList */ "subl $0x384,%esp\n\t" /* sizeof(struct syscall_frame) + ebp */ "andl $~63,%esp\n\t" @@ -1600,8 +1600,13 @@ __ASM_GLOBAL_FUNC( call_user_mode_callback, "movl %eax,0x38(%esp)\n\t" "movl %ecx,0x3c(%esp)\n\t" /* frame->prev_frame */ "movl %esp,0x1f8(%edx)\n\t" /* x86_thread_data()->syscall_frame */ - "movl 8(%ebp),%ecx\n\t" /* func */ - "movl 12(%ebp),%esp\n\t" /* stack */ + "movl 0x1c(%ebp),%ecx\n\t" /* func */ + "movl 0x0c(%ebp),%edx\n\t" /* args */ + "leal -4(%edx),%esp\n\t" + "pushl 0x10(%ebp)\n\t" /* len */ + "pushl %edx\n\t" /* args */ + "pushl 0x08(%ebp)\n\t" /* id */ + "pushl $0\n\t" "xorl %ebp,%ebp\n\t" "jmpl *%ecx" ) @@ -1609,8 +1614,8 @@ __ASM_GLOBAL_FUNC( call_user_mode_callback, /*********************************************************************** * user_mode_callback_return */ -extern void CDECL DECLSPEC_NORETURN user_mode_callback_return( void *ret_ptr, ULONG ret_len, - NTSTATUS status, TEB *teb ) DECLSPEC_HIDDEN; +extern void DECLSPEC_NORETURN user_mode_callback_return( void *ret_ptr, ULONG ret_len, + NTSTATUS status, TEB *teb ) DECLSPEC_HIDDEN; __ASM_GLOBAL_FUNC( user_mode_callback_return, "movl 16(%esp),%edx\n" /* teb */ "movl 0x1f8(%edx),%eax\n\t" /* x86_thread_data()->syscall_frame */ @@ -1627,9 +1632,9 @@ __ASM_GLOBAL_FUNC( user_mode_callback_return, "movl 8(%esp),%edi\n\t" /* ret_len */ "movl 12(%esp),%eax\n\t" /* status */ "leal -16(%ebp),%esp\n\t" - "movl 0x10(%ebp),%ecx\n\t" /* ret_ptr */ + "movl 0x14(%ebp),%ecx\n\t" /* ret_ptr */ "movl %esi,(%ecx)\n\t" - "movl 0x14(%ebp),%ecx\n\t" /* ret_len */ + "movl 0x18(%ebp),%ecx\n\t" /* ret_len */ "movl %edi,(%ecx)\n\t" "popl 0(%edx)\n\t" /* teb->Tib.ExceptionList */ "popl %edi\n\t" @@ -1651,19 +1656,13 @@ NTSTATUS WINAPI KeUserModeCallback( ULONG id, const void *args, ULONG len, void { struct syscall_frame *frame = x86_thread_data()->syscall_frame; void *args_data = (void *)((frame->esp - len) & ~15); - ULONG_PTR *stack = args_data; if ((char *)ntdll_get_thread_data()->kernel_stack + min_kernel_stack > (char *)&frame) return STATUS_STACK_OVERFLOW; memcpy( args_data, args, len ); - *(--stack) = 0; - *(--stack) = len; - *(--stack) = (ULONG_PTR)args_data; - *(--stack) = id; - *(--stack) = 0xdeadbabe; - - return call_user_mode_callback( pKiUserCallbackDispatcher, stack, ret_ptr, ret_len, NtCurrentTeb() ); + return call_user_mode_callback( id, args_data, len, ret_ptr, ret_len, + pKiUserCallbackDispatcher, NtCurrentTeb() ); }

1 0

Alexandre Julliard : ntdll: Don't pass arguments on the stack for user callbacks on x86-64.
by Alexandre Julliard 01 Jun '23

01 Jun '23

Module: wine Branch: master Commit: 50ac25554c8ecc799d5e94addb748cad503170ea URL: https://gitlab.winehq.org/wine/wine/-/commit/50ac25554c8ecc799d5e94addb748c… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Jun 1 12:06:56 2023 +0200 ntdll: Don't pass arguments on the stack for user callbacks on x86-64. And convert to the sysv ABI. --- dlls/ntdll/unix/signal_x86_64.c | 136 ++++++++++++++-------------------------- 1 file changed, 48 insertions(+), 88 deletions(-)

1 0

Alexandre Julliard : ntdll: Don't pass arguments on the stack for user callbacks on ARM64.
by Alexandre Julliard 01 Jun '23

01 Jun '23

Module: wine Branch: master Commit: efa99eab0bc3edbf3f23bee9da316d901f7a14cb URL: https://gitlab.winehq.org/wine/wine/-/commit/efa99eab0bc3edbf3f23bee9da316d… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Jun 1 11:26:22 2023 +0200 ntdll: Don't pass arguments on the stack for user callbacks on ARM64. --- dlls/ntdll/unix/signal_arm64.c | 45 +++++++++++++++++------------------------- 1 file changed, 18 insertions(+), 27 deletions(-) diff --git a/dlls/ntdll/unix/signal_arm64.c b/dlls/ntdll/unix/signal_arm64.c index 75ba53acda3..eb9fd4c8442 100644 --- a/dlls/ntdll/unix/signal_arm64.c +++ b/dlls/ntdll/unix/signal_arm64.c @@ -1092,14 +1092,11 @@ NTSTATUS call_user_exception_dispatcher( EXCEPTION_RECORD *rec, CONTEXT *context /*********************************************************************** * call_user_mode_callback */ -extern NTSTATUS CDECL call_user_mode_callback( void *func, void *stack, void **ret_ptr, - ULONG *ret_len, TEB *teb ) DECLSPEC_HIDDEN; +extern NTSTATUS call_user_mode_callback( ULONG id, void *args, ULONG len, void **ret_ptr, + ULONG *ret_len, void *func, TEB *teb ) DECLSPEC_HIDDEN; __ASM_GLOBAL_FUNC( call_user_mode_callback, "stp x29, x30, [sp,#-0xc0]!\n\t" "mov x29, sp\n\t" - "mov x16, x0\n\t" /* func */ - "mov x17, x1\n\t" /* stack */ - "mov x18, x4\n\t" /* teb */ "stp x19, x20, [x29, #0x10]\n\t" "stp x21, x22, [x29, #0x20]\n\t" "stp x23, x24, [x29, #0x30]\n\t" @@ -1109,29 +1106,28 @@ __ASM_GLOBAL_FUNC( call_user_mode_callback, "stp d10, d11, [x29, #0x70]\n\t" "stp d12, d13, [x29, #0x80]\n\t" "stp d14, d15, [x29, #0x90]\n\t" - "stp x2, x3, [x29, #0xa0]\n\t" /* ret_ptr, ret_len */ - "mrs x5, fpcr\n\t" - "mrs x6, fpsr\n\t" - "bfi x5, x6, #0, #32\n\t" - "ldr x6, [x18]\n\t" /* teb->Tib.ExceptionList */ - "stp x5, x6, [x29, #0xb0]\n\t" + "stp x3, x4, [x29, #0xa0]\n\t" /* ret_ptr, ret_len */ + "mov x18, x6\n\t" /* teb */ + "mrs x3, fpcr\n\t" + "mrs x4, fpsr\n\t" + "bfi x3, x4, #0, #32\n\t" + "ldr x4, [x18]\n\t" /* teb->Tib.ExceptionList */ + "stp x3, x4, [x29, #0xb0]\n\t" "ldr x7, [x18, #0x2f8]\n\t" /* arm64_thread_data()->syscall_frame */ - "sub x5, sp, #0x330\n\t" /* sizeof(struct syscall_frame) */ - "str x5, [x18, #0x2f8]\n\t" /* arm64_thread_data()->syscall_frame */ + "sub x3, sp, #0x330\n\t" /* sizeof(struct syscall_frame) */ + "str x3, [x18, #0x2f8]\n\t" /* arm64_thread_data()->syscall_frame */ "ldr x8, [x7, #0x118]\n\t" /* prev_frame->syscall_table */ - "ldp x0, x1, [x17]\n\t" /* id, args */ - "ldr x2, [x17, #0x10]\n\t" /* len */ - "mov sp, x17\n\t" - "stp x7, x8, [x5, #0x110]\n\t" /* frame->prev_frame, frame->syscall_table */ - "br x16" ) + "mov sp, x1\n\t" /* stack */ + "stp x7, x8, [x3, #0x110]\n\t" /* frame->prev_frame, frame->syscall_table */ + "br x5" ) /*********************************************************************** * user_mode_callback_return */ -extern void CDECL DECLSPEC_NORETURN user_mode_callback_return( void *ret_ptr, ULONG ret_len, - NTSTATUS status, TEB *teb ) DECLSPEC_HIDDEN; +extern void DECLSPEC_NORETURN user_mode_callback_return( void *ret_ptr, ULONG ret_len, + NTSTATUS status, TEB *teb ) DECLSPEC_HIDDEN; __ASM_GLOBAL_FUNC( user_mode_callback_return, "ldr x4, [x3, #0x2f8]\n\t" /* arm64_thread_data()->syscall_frame */ "ldr x5, [x4, #0x110]\n\t" /* prev_frame */ @@ -1167,18 +1163,13 @@ NTSTATUS WINAPI KeUserModeCallback( ULONG id, const void *args, ULONG len, void { struct syscall_frame *frame = arm64_thread_data()->syscall_frame; void *args_data = (void *)((frame->sp - len) & ~15); - ULONG_PTR *stack = args_data; if ((char *)ntdll_get_thread_data()->kernel_stack + min_kernel_stack > (char *)&frame) return STATUS_STACK_OVERFLOW; memcpy( args_data, args, len ); - *(--stack) = 0; - *(--stack) = len; - *(--stack) = (ULONG_PTR)args_data; - *(--stack) = id; - - return call_user_mode_callback( pKiUserCallbackDispatcher, stack, ret_ptr, ret_len, NtCurrentTeb() ); + return call_user_mode_callback( id, args_data, len, ret_ptr, ret_len, + pKiUserCallbackDispatcher, NtCurrentTeb() ); }

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits June 2023