November 2022 - Wine-commits

Eric Pouech : rpcrt4: Fix unexpected exception in ndr_marshal tests.
by Alexandre Julliard 07 Nov '22

07 Nov '22

Module: wine Branch: master Commit: c5e23886b2290304c78101d7927898700974b84a URL: https://gitlab.winehq.org/wine/wine/-/commit/c5e23886b2290304c78101d7927898… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Fri Nov 4 15:42:41 2022 +0100 rpcrt4: Fix unexpected exception in ndr_marshal tests. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=52909 Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> --- dlls/rpcrt4/tests/ndr_marshall.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/rpcrt4/tests/ndr_marshall.c b/dlls/rpcrt4/tests/ndr_marshall.c index 5d72f5efc61..bae9b12f7ab 100644 --- a/dlls/rpcrt4/tests/ndr_marshall.c +++ b/dlls/rpcrt4/tests/ndr_marshall.c @@ -154,6 +154,7 @@ static void test_ndr_simple_type(void) StubMsg.BufferLength = 16; StubMsg.RpcMsg->Buffer = StubMsg.BufferStart = StubMsg.Buffer = HeapAlloc(GetProcessHeap(), 0, StubMsg.BufferLength); + StubMsg.BufferEnd = StubMsg.Buffer + StubMsg.BufferLength; l = 0xcafebabe; NdrSimpleTypeMarshall(&StubMsg, (unsigned char*)&l, FC_LONG); ok(StubMsg.Buffer == StubMsg.BufferStart + 4, "%p %p\n", StubMsg.Buffer, StubMsg.BufferStart);

1 0

Alexandre Julliard : xslt: Use WINAPIV calling convention for variadic functions.
by Alexandre Julliard 07 Nov '22

07 Nov '22

Module: wine Branch: master Commit: 1a652d2c9e48fa239d10e53a37be98c102e226af URL: https://gitlab.winehq.org/wine/wine/-/commit/1a652d2c9e48fa239d10e53a37be98… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Nov 4 21:18:16 2022 +0100 xslt: Use WINAPIV calling convention for variadic functions. --- libs/xslt/libxslt/xsltutils.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libs/xslt/libxslt/xsltutils.h b/libs/xslt/libxslt/xsltutils.h index 12e4c75f5b9..7f35cc7ed20 100644 --- a/libs/xslt/libxslt/xsltutils.h +++ b/libs/xslt/libxslt/xsltutils.h @@ -157,7 +157,7 @@ XSLTPUBFUN void XSLTCALL xsltSetTransformErrorFunc (xsltTransformContextPtr ctxt, void *ctx, xmlGenericErrorFunc handler); -XSLTPUBFUN void XSLTCALL +XSLTPUBFUN void WINAPIV xsltTransformError (xsltTransformContextPtr ctxt, xsltStylesheetPtr style, xmlNodePtr node,

1 0

Alexandre Julliard : ntdll: Reimplement KeUserModeCallback in assembly on i386.
by Alexandre Julliard 07 Nov '22

07 Nov '22

Module: wine Branch: master Commit: d40017ecda0b4974040fcb6a699e88189652af53 URL: https://gitlab.winehq.org/wine/wine/-/commit/d40017ecda0b4974040fcb6a699e88… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Nov 7 12:28:36 2022 +0100 ntdll: Reimplement KeUserModeCallback in assembly on i386. --- dlls/ntdll/unix/signal_i386.c | 136 +++++++++++++++++++++++++++--------------- 1 file changed, 88 insertions(+), 48 deletions(-) diff --git a/dlls/ntdll/unix/signal_i386.c b/dlls/ntdll/unix/signal_i386.c index 49d65ac1b3d..efb7cc497ef 100644 --- a/dlls/ntdll/unix/signal_i386.c +++ b/dlls/ntdll/unix/signal_i386.c @@ -1571,59 +1571,107 @@ NTSTATUS call_user_exception_dispatcher( EXCEPTION_RECORD *rec, CONTEXT *context } -struct user_callback_frame -{ - struct syscall_frame frame; - void **ret_ptr; - ULONG *ret_len; - __wine_jmp_buf jmpbuf; - NTSTATUS status; - void *teb_frame; -}; +/*********************************************************************** + * call_user_mode_callback + */ +extern NTSTATUS CDECL call_user_mode_callback( void *func, void *stack, void **ret_ptr, + ULONG *ret_len, TEB *teb ); +__ASM_GLOBAL_FUNC( call_user_mode_callback, + "pushl %ebp\n\t" + __ASM_CFI(".cfi_adjust_cfa_offset 4\n\t") + __ASM_CFI(".cfi_rel_offset %ebp,0\n\t") + "movl %esp,%ebp\n\t" + __ASM_CFI(".cfi_def_cfa_register %ebp\n\t") + "pushl %ebx\n\t" + __ASM_CFI(".cfi_rel_offset %ebx,-4\n\t") + "pushl %esi\n\t" + __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 */ + "pushl 0(%edx)\n\t" /* teb->Tib.ExceptionList */ + "subl $0x384,%esp\n\t" /* sizeof(struct syscall_frame) + ebp */ + "andl $~63,%esp\n\t" + "movl %ebp,0x380(%esp)\n\t" + "movl 0x1f8(%edx),%ecx\n\t" /* x86_thread_data()->syscall_frame */ + "movl (%ecx),%eax\n\t" /* frame->syscall_flags */ + "movl %eax,(%esp)\n\t" + "movl 0x38(%ecx),%eax\n\t" /* frame->syscall_table */ + "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 */ + "xorl %ebp,%ebp\n\t" + "jmpl *%ecx" ) + + +/*********************************************************************** + * user_mode_callback_return + */ +extern void CDECL DECLSPEC_NORETURN user_mode_callback_return( void *ret_ptr, ULONG ret_len, + NTSTATUS status, TEB *teb ); +__ASM_GLOBAL_FUNC( user_mode_callback_return, + "movl 16(%esp),%edx\n" /* teb */ + "movl 0x1f8(%edx),%eax\n\t" /* x86_thread_data()->syscall_frame */ + "movl 0x3c(%eax),%ecx\n\t" /* frame->prev_frame */ + "movl %ecx,0x1f8(%edx)\n\t" /* x86_thread_data()->syscall_frame */ + "movl 0x380(%eax),%ebp\n\t" /* call_user_mode_callback ebp */ + __ASM_CFI(".cfi_adjust_cfa_offset 4\n\t") + __ASM_CFI(".cfi_rel_offset %ebp,0\n\t") + __ASM_CFI(".cfi_def_cfa_register %ebp\n\t") + __ASM_CFI(".cfi_rel_offset %ebx,-4\n\t") + __ASM_CFI(".cfi_rel_offset %esi,-8\n\t") + __ASM_CFI(".cfi_rel_offset %edi,-12\n\t") + "movl 4(%esp),%esi\n\t" /* ret_ptr */ + "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 %esi,(%ecx)\n\t" + "movl 0x14(%ebp),%ecx\n\t" /* ret_len */ + "movl %edi,(%ecx)\n\t" + "popl 0(%edx)\n\t" /* teb->Tib.ExceptionList */ + "popl %edi\n\t" + __ASM_CFI(".cfi_same_value %edi\n\t") + "popl %esi\n\t" + __ASM_CFI(".cfi_same_value %esi\n\t") + "popl %ebx\n\t" + __ASM_CFI(".cfi_same_value %ebx\n\t") + "leave\n" + __ASM_CFI(".cfi_def_cfa %esp,4\n\t") + __ASM_CFI(".cfi_same_value %ebp\n\t") + "ret" ) + /*********************************************************************** * KeUserModeCallback */ NTSTATUS WINAPI KeUserModeCallback( ULONG id, const void *args, ULONG len, void **ret_ptr, ULONG *ret_len ) { - struct user_callback_frame callback_frame = { { 0 }, ret_ptr, ret_len }; + struct syscall_frame *frame = x86_thread_data()->syscall_frame; + void *args_data = (void *)((frame->esp - len) & ~15); + ULONG_PTR *stack = args_data; /* if we have no syscall frame, call the callback directly */ - if ((char *)&callback_frame < (char *)ntdll_get_thread_data()->kernel_stack || - (char *)&callback_frame > (char *)x86_thread_data()->syscall_frame) + if ((char *)&frame < (char *)ntdll_get_thread_data()->kernel_stack || + (char *)&frame > (char *)x86_thread_data()->syscall_frame) { NTSTATUS (WINAPI *func)(const void *, ULONG) = ((void **)NtCurrentTeb()->Peb->KernelCallbackTable)[id]; return func( args, len ); } - if ((char *)ntdll_get_thread_data()->kernel_stack + min_kernel_stack > (char *)&callback_frame) + if ((char *)ntdll_get_thread_data()->kernel_stack + min_kernel_stack > (char *)&frame) return STATUS_STACK_OVERFLOW; - if (!__wine_setjmpex( &callback_frame.jmpbuf, NULL )) - { - struct syscall_frame *frame = x86_thread_data()->syscall_frame; - void *args_data = (void *)((frame->esp - len) & ~15); - ULONG_PTR *stack = args_data; - - memcpy( args_data, args, len ); - *(--stack) = 0; - *(--stack) = len; - *(--stack) = (ULONG_PTR)args_data; - *(--stack) = id; - *(--stack) = 0xdeadbabe; - - callback_frame.frame.esp = (ULONG_PTR)stack; - callback_frame.frame.eip = (ULONG_PTR)pKiUserCallbackDispatcher; - callback_frame.frame.eflags = 0x202; - callback_frame.frame.syscall_flags = frame->syscall_flags; - callback_frame.frame.syscall_table = frame->syscall_table; - callback_frame.frame.prev_frame = frame; - callback_frame.teb_frame = NtCurrentTeb()->Tib.ExceptionList; - x86_thread_data()->syscall_frame = &callback_frame.frame; - - __wine_syscall_dispatcher_return( &callback_frame.frame, 0 ); - } - return callback_frame.status; + 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() ); } @@ -1632,16 +1680,8 @@ NTSTATUS WINAPI KeUserModeCallback( ULONG id, const void *args, ULONG len, void */ NTSTATUS WINAPI NtCallbackReturn( void *ret_ptr, ULONG ret_len, NTSTATUS status ) { - struct user_callback_frame *frame = (struct user_callback_frame *)x86_thread_data()->syscall_frame; - - if (!frame->frame.prev_frame) return STATUS_NO_CALLBACK_ACTIVE; - - *frame->ret_ptr = ret_ptr; - *frame->ret_len = ret_len; - frame->status = status; - x86_thread_data()->syscall_frame = frame->frame.prev_frame; - NtCurrentTeb()->Tib.ExceptionList = frame->teb_frame; - __wine_longjmp( &frame->jmpbuf, 1 ); + if (!x86_thread_data()->syscall_frame->prev_frame) return STATUS_NO_CALLBACK_ACTIVE; + user_mode_callback_return( ret_ptr, ret_len, status, NtCurrentTeb() ); }

1 0

Alexandre Julliard : ntdll: Reimplement KeUserModeCallback in assembly on x86_64.
by Alexandre Julliard 07 Nov '22

07 Nov '22

Module: wine Branch: master Commit: 8e9551f32435d76df87864147cc2119fd6555ef3 URL: https://gitlab.winehq.org/wine/wine/-/commit/8e9551f32435d76df87864147cc211… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Nov 7 12:28:56 2022 +0100 ntdll: Reimplement KeUserModeCallback in assembly on x86_64. --- dlls/ntdll/unix/signal_x86_64.c | 194 +++++++++++++++++++++++++++++----------- 1 file changed, 144 insertions(+), 50 deletions(-)

1 0

Alexandre Julliard : ntdll: Reimplement KeUserModeCallback in assembly on ARM.
by Alexandre Julliard 07 Nov '22

07 Nov '22

Module: wine Branch: master Commit: b60d076b09315a5b1abb740dcadb136b7440f564 URL: https://gitlab.winehq.org/wine/wine/-/commit/b60d076b09315a5b1abb740dcadb13… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Nov 7 12:29:08 2022 +0100 ntdll: Reimplement KeUserModeCallback in assembly on ARM. --- dlls/ntdll/unix/signal_arm.c | 113 ++++++++++++++++++++++++++----------------- 1 file changed, 69 insertions(+), 44 deletions(-) diff --git a/dlls/ntdll/unix/signal_arm.c b/dlls/ntdll/unix/signal_arm.c index 9300ea2adc3..9eef8ff7c69 100644 --- a/dlls/ntdll/unix/signal_arm.c +++ b/dlls/ntdll/unix/signal_arm.c @@ -1149,55 +1149,88 @@ NTSTATUS call_user_exception_dispatcher( EXCEPTION_RECORD *rec, CONTEXT *context } -struct user_callback_frame -{ - struct syscall_frame frame; - void **ret_ptr; - ULONG *ret_len; - __wine_jmp_buf jmpbuf; - NTSTATUS status; - void *teb_frame; -}; +/*********************************************************************** + * call_user_mode_callback + */ +extern NTSTATUS CDECL call_user_mode_callback( void *func, void *stack, void **ret_ptr, + ULONG *ret_len, TEB *teb ); +__ASM_GLOBAL_FUNC( call_user_mode_callback, + "push {r2-r12,lr}\n\t" + "ldr r4, [sp, #0x30]\n\t" /* teb */ + "ldr r5, [r4]\n\t" /* teb->Tib.ExceptionList */ + "str r5, [sp, #0x28]\n\t" +#ifndef __SOFTFP__ + "sub sp, sp, #0x90\n\t" + "mov r5, sp\n\t" + "vmrs r6, fpscr\n\t" + "vstm r5, {d8-d15}\n\t" + "str r6, [r5, #0x80]\n\t" +#endif + "sub sp, sp, #0x160\n\t" /* sizeof(struct syscall_frame) + registers */ + "ldr r5, [r4, #0x1d8]\n\t" /* arm_thread_data()->syscall_frame */ + "str r5, [sp, #0x4c]\n\t" /* frame->prev_frame */ + "str sp, [r4, #0x1d8]\n\t" /* arm_thread_data()->syscall_frame */ + "ldr r6, [r5, #0x50]\n\t" /* prev_frame->syscall_table */ + "str r6, [sp, #0x50]\n\t" /* frame->syscall_table */ + "mov ip, r0\n\t" + "mov sp, r1\n\t" + "pop {r0-r3}\n\t" + "bx ip" ) + + +/*********************************************************************** + * user_mode_callback_return + */ +extern void CDECL DECLSPEC_NORETURN user_mode_callback_return( void *ret_ptr, ULONG ret_len, + NTSTATUS status, TEB *teb ); +__ASM_GLOBAL_FUNC( user_mode_callback_return, + "ldr r4, [r3, #0x1d8]\n\t" /* arm_thread_data()->syscall_frame */ + "ldr r5, [r4, #0x4c]\n\t" /* frame->prev_frame */ + "str r5, [r3, #0x1d8]\n\t" /* arm_thread_data()->syscall_frame */ + "add r5, r4, #0x160\n\t" +#ifndef __SOFTFP__ + "vldm r5, {d8-d15}\n\t" + "ldr r6, [r5, #0x80]\n\t" + "vmsr fpscr, r6\n\t" + "add r5, r5, #0x90\n\t" +#endif + "mov sp, r5\n\t" + "ldr r5, [sp, #0x28]\n\t" + "str r5, [r3]\n\t" /* teb->Tib.ExceptionList */ + "pop {r5, r6}\n\t" /* ret_ptr, ret_len */ + "str r0, [r5]\n\t" /* ret_ptr */ + "str r1, [r6]\n\t" /* ret_len */ + "mov r0, r2\n\t" /* status */ + "pop {r4-r12,pc}" ) + /*********************************************************************** * KeUserModeCallback */ NTSTATUS WINAPI KeUserModeCallback( ULONG id, const void *args, ULONG len, void **ret_ptr, ULONG *ret_len ) { - struct user_callback_frame callback_frame = { { 0 }, ret_ptr, ret_len }; + struct syscall_frame *frame = arm_thread_data()->syscall_frame; + void *args_data = (void *)((frame->sp - len) & ~15); + ULONG_PTR *stack = args_data; /* if we have no syscall frame, call the callback directly */ - if ((char *)&callback_frame < (char *)ntdll_get_thread_data()->kernel_stack || - (char *)&callback_frame > (char *)arm_thread_data()->syscall_frame) + if ((char *)&frame < (char *)ntdll_get_thread_data()->kernel_stack || + (char *)&frame > (char *)arm_thread_data()->syscall_frame) { NTSTATUS (WINAPI *func)(const void *, ULONG) = ((void **)NtCurrentTeb()->Peb->KernelCallbackTable)[id]; return func( args, len ); } - if ((char *)ntdll_get_thread_data()->kernel_stack + min_kernel_stack > (char *)&callback_frame) + if ((char *)ntdll_get_thread_data()->kernel_stack + min_kernel_stack > (char *)&frame) return STATUS_STACK_OVERFLOW; - if (!__wine_setjmpex( &callback_frame.jmpbuf, NULL )) - { - struct syscall_frame *frame = arm_thread_data()->syscall_frame; - void *args_data = (void *)((frame->sp - len) & ~15); - - memcpy( args_data, args, len ); - - callback_frame.frame.r0 = id; - callback_frame.frame.r1 = (ULONG_PTR)args; - callback_frame.frame.r2 = len; - callback_frame.frame.sp = (ULONG_PTR)args_data; - callback_frame.frame.pc = (ULONG_PTR)pKiUserCallbackDispatcher; - callback_frame.frame.restore_flags = CONTEXT_INTEGER; - callback_frame.frame.syscall_table = frame->syscall_table; - callback_frame.frame.prev_frame = frame; - callback_frame.teb_frame = NtCurrentTeb()->Tib.ExceptionList; - arm_thread_data()->syscall_frame = &callback_frame.frame; - - __wine_syscall_dispatcher_return( &callback_frame.frame, 0 ); - } - return callback_frame.status; + 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() ); } @@ -1206,16 +1239,8 @@ NTSTATUS WINAPI KeUserModeCallback( ULONG id, const void *args, ULONG len, void */ NTSTATUS WINAPI NtCallbackReturn( void *ret_ptr, ULONG ret_len, NTSTATUS status ) { - struct user_callback_frame *frame = (struct user_callback_frame *)arm_thread_data()->syscall_frame; - - if (!frame->frame.prev_frame) return STATUS_NO_CALLBACK_ACTIVE; - - *frame->ret_ptr = ret_ptr; - *frame->ret_len = ret_len; - frame->status = status; - arm_thread_data()->syscall_frame = frame->frame.prev_frame; - NtCurrentTeb()->Tib.ExceptionList = frame->teb_frame; - __wine_longjmp( &frame->jmpbuf, 1 ); + if (!arm_thread_data()->syscall_frame->prev_frame) return STATUS_NO_CALLBACK_ACTIVE; + user_mode_callback_return( ret_ptr, ret_len, status, NtCurrentTeb() ); }

1 0

Alexandre Julliard : ntdll: Reimplement KeUserModeCallback in assembly on ARM64.
by Alexandre Julliard 07 Nov '22

07 Nov '22

Module: wine Branch: master Commit: 4069a8b384c15a90a7af66636ec5650eeb53b391 URL: https://gitlab.winehq.org/wine/wine/-/commit/4069a8b384c15a90a7af66636ec565… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Nov 7 12:29:18 2022 +0100 ntdll: Reimplement KeUserModeCallback in assembly on ARM64. --- dlls/ntdll/unix/signal_arm64.c | 130 +++++++++++++++++++++++++++-------------- 1 file changed, 85 insertions(+), 45 deletions(-) diff --git a/dlls/ntdll/unix/signal_arm64.c b/dlls/ntdll/unix/signal_arm64.c index 19d08e2270a..bc8225d7ec8 100644 --- a/dlls/ntdll/unix/signal_arm64.c +++ b/dlls/ntdll/unix/signal_arm64.c @@ -876,56 +876,104 @@ NTSTATUS call_user_exception_dispatcher( EXCEPTION_RECORD *rec, CONTEXT *context } -struct user_callback_frame -{ - struct syscall_frame frame; - void **ret_ptr; - ULONG *ret_len; - __wine_jmp_buf jmpbuf; - NTSTATUS status; - void *teb_frame; -}; +/*********************************************************************** + * call_user_mode_callback + */ +extern NTSTATUS CDECL call_user_mode_callback( void *func, void *stack, void **ret_ptr, + ULONG *ret_len, TEB *teb ); +__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" + "stp x25, x26, [x29, #0x40]\n\t" + "stp x27, x28, [x29, #0x50]\n\t" + "stp d8, d9, [x29, #0x60]\n\t" + "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" + + "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 */ + "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" ) + + +/*********************************************************************** + * user_mode_callback_return + */ +extern void CDECL DECLSPEC_NORETURN user_mode_callback_return( void *ret_ptr, ULONG ret_len, + NTSTATUS status, TEB *teb ); +__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 */ + "str x5, [x3, #0x2f8]\n\t" /* arm64_thread_data()->syscall_frame */ + "add x29, x4, #0x330\n\t" /* sizeof(struct syscall_frame) */ + "ldp x5, x6, [x29, #0xb0]\n\t" + "str x6, [x3]\n\t" /* teb->Tib.ExceptionList */ + "msr fpcr, x5\n\t" + "lsr x5, x5, #32\n\t" + "msr fpsr, x5\n\t" + "ldp x19, x20, [x29, #0x10]\n\t" + "ldp x21, x22, [x29, #0x20]\n\t" + "ldp x23, x24, [x29, #0x30]\n\t" + "ldp x25, x26, [x29, #0x40]\n\t" + "ldp x27, x28, [x29, #0x50]\n\t" + "ldp d8, d9, [x29, #0x60]\n\t" + "ldp d10, d11, [x29, #0x70]\n\t" + "ldp d12, d13, [x29, #0x80]\n\t" + "ldp d14, d15, [x29, #0x90]\n\t" + "ldp x5, x6, [x29, #0xa0]\n\t" /* ret_ptr, ret_len */ + "str x0, [x5]\n\t" /* ret_ptr */ + "str x1, [x6]\n\t" /* ret_len */ + "mov x0, x2\n\t" /* status */ + "mov sp, x29\n\t" + "ldp x29, x30, [sp], #0xc0\n\t" + "ret" ) + /*********************************************************************** * KeUserModeCallback */ NTSTATUS WINAPI KeUserModeCallback( ULONG id, const void *args, ULONG len, void **ret_ptr, ULONG *ret_len ) { - struct user_callback_frame callback_frame = { {{ 0 }}, ret_ptr, ret_len }; + struct syscall_frame *frame = arm64_thread_data()->syscall_frame; + void *args_data = (void *)((frame->sp - len) & ~15); + ULONG_PTR *stack = args_data; /* if we have no syscall frame, call the callback directly */ - if ((char *)&callback_frame < (char *)ntdll_get_thread_data()->kernel_stack || - (char *)&callback_frame > (char *)arm64_thread_data()->syscall_frame) + if ((char *)&frame < (char *)ntdll_get_thread_data()->kernel_stack || + (char *)&frame > (char *)arm64_thread_data()->syscall_frame) { NTSTATUS (WINAPI *func)(const void *, ULONG) = ((void **)NtCurrentTeb()->Peb->KernelCallbackTable)[id]; return func( args, len ); } - if ((char *)ntdll_get_thread_data()->kernel_stack + min_kernel_stack > (char *)&callback_frame) + if ((char *)ntdll_get_thread_data()->kernel_stack + min_kernel_stack > (char *)&frame) return STATUS_STACK_OVERFLOW; - if (!__wine_setjmpex( &callback_frame.jmpbuf, NULL )) - { - struct syscall_frame *frame = arm64_thread_data()->syscall_frame; - void *args_data = (void *)((frame->sp - len) & ~15); - - memcpy( args_data, args, len ); - - callback_frame.frame.x[0] = id; - callback_frame.frame.x[1] = (ULONG_PTR)args; - callback_frame.frame.x[2] = len; - callback_frame.frame.x[18] = frame->x[18]; - callback_frame.frame.sp = (ULONG_PTR)args_data; - callback_frame.frame.pc = (ULONG_PTR)pKiUserCallbackDispatcher; - callback_frame.frame.restore_flags = CONTEXT_INTEGER; - callback_frame.frame.syscall_table = frame->syscall_table; - callback_frame.frame.prev_frame = frame; - callback_frame.teb_frame = NtCurrentTeb()->Tib.ExceptionList; - arm64_thread_data()->syscall_frame = &callback_frame.frame; - - __wine_syscall_dispatcher_return( &callback_frame.frame, 0 ); - } - return callback_frame.status; + 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() ); } @@ -934,16 +982,8 @@ NTSTATUS WINAPI KeUserModeCallback( ULONG id, const void *args, ULONG len, void */ NTSTATUS WINAPI NtCallbackReturn( void *ret_ptr, ULONG ret_len, NTSTATUS status ) { - struct user_callback_frame *frame = (struct user_callback_frame *)arm64_thread_data()->syscall_frame; - - if (!frame->frame.prev_frame) return STATUS_NO_CALLBACK_ACTIVE; - - *frame->ret_ptr = ret_ptr; - *frame->ret_len = ret_len; - frame->status = status; - arm64_thread_data()->syscall_frame = frame->frame.prev_frame; - NtCurrentTeb()->Tib.ExceptionList = frame->teb_frame; - __wine_longjmp( &frame->jmpbuf, 1 ); + if (!arm64_thread_data()->syscall_frame->prev_frame) return STATUS_NO_CALLBACK_ACTIVE; + user_mode_callback_return( ret_ptr, ret_len, status, NtCurrentTeb() ); }

1 0

Zhiyi Zhang : d2d1/tests: Add hwnd render target pixel format tests.
by Alexandre Julliard 07 Nov '22

07 Nov '22

Module: wine Branch: master Commit: 1e4d31199953a76c1ab4914e2af41b14ef5de2d0 URL: https://gitlab.winehq.org/wine/wine/-/commit/1e4d31199953a76c1ab4914e2af41b… Author: Zhiyi Zhang <zzhang(a)codeweavers.com> Date: Thu Nov 3 17:15:39 2022 +0800 d2d1/tests: Add hwnd render target pixel format tests. --- dlls/d2d1/tests/d2d1.c | 48 ++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 48 insertions(+) diff --git a/dlls/d2d1/tests/d2d1.c b/dlls/d2d1/tests/d2d1.c index 99a1013a333..67cb83bdd4a 100644 --- a/dlls/d2d1/tests/d2d1.c +++ b/dlls/d2d1/tests/d2d1.c @@ -6075,12 +6075,33 @@ static void test_hwnd_target(BOOL d3d11) ID2D1GdiInteropRenderTarget *interop; D2D1_RENDER_TARGET_PROPERTIES desc; ID2D1HwndRenderTarget *rt, *rt2; + D2D1_PIXEL_FORMAT pixel_format; struct d2d1_test_context ctx; ID2D1RenderTarget *rt3; ID2D1Factory *factory; D2D1_SIZE_U size; + unsigned int i; HRESULT hr; + static const struct format_test + { + D2D1_PIXEL_FORMAT format; + D2D1_PIXEL_FORMAT expected_format; + BOOL expected_failure; + BOOL todo; + } + format_tests[] = + { + {{DXGI_FORMAT_UNKNOWN, D2D1_ALPHA_MODE_UNKNOWN}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_IGNORE}}, + {{DXGI_FORMAT_UNKNOWN, D2D1_ALPHA_MODE_PREMULTIPLIED}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_PREMULTIPLIED}, FALSE, TRUE}, + {{DXGI_FORMAT_UNKNOWN, D2D1_ALPHA_MODE_STRAIGHT}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_STRAIGHT}, TRUE}, + {{DXGI_FORMAT_UNKNOWN, D2D1_ALPHA_MODE_IGNORE}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_IGNORE}}, + {{DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_UNKNOWN}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_IGNORE}, FALSE, TRUE}, + {{DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_PREMULTIPLIED}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_PREMULTIPLIED}}, + {{DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_STRAIGHT}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_STRAIGHT}, TRUE}, + {{DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_IGNORE}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_IGNORE}}, + }; + if (!init_test_context(&ctx, d3d11)) return; release_test_context(&ctx); @@ -6132,6 +6153,33 @@ static void test_hwnd_target(BOOL d3d11) ID2D1HwndRenderTarget_Release(rt); + /* Test render target format */ + for (i = 0; i < ARRAY_SIZE(format_tests); ++i) + { + winetest_push_context("test %d", i); + + desc.pixelFormat = format_tests[i].format; + hr = ID2D1Factory_CreateHwndRenderTarget(factory, &desc, &hwnd_rt_desc, &rt); + if (format_tests[i].expected_failure) + { + todo_wine + ok(FAILED(hr), "Got unexpected hr %#lx.\n", hr); + winetest_pop_context(); + continue; + } + ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); + + pixel_format = ID2D1HwndRenderTarget_GetPixelFormat(rt); + ok(pixel_format.format == format_tests[i].expected_format.format, + "Got unexpected format %#x.\n", pixel_format.format); + todo_wine_if(format_tests[i].todo) + ok(pixel_format.alphaMode == format_tests[i].expected_format.alphaMode, + "Got unexpected alpha mode %d.\n", pixel_format.alphaMode); + + ID2D1HwndRenderTarget_Release(rt); + winetest_pop_context(); + } + DestroyWindow(hwnd_rt_desc.hwnd); ID2D1Factory_Release(factory); }

1 0

Zhiyi Zhang : d2d1: Use D2D1_ALPHA_MODE_IGNORE if D2D1_ALPHA_MODE_UNKNOWN is used for creating hwnd render targets.
by Alexandre Julliard 07 Nov '22

07 Nov '22

Module: wine Branch: master Commit: 225daac8790146fe5886995b330ce4dd77c60775 URL: https://gitlab.winehq.org/wine/wine/-/commit/225daac8790146fe5886995b330ce4… Author: Zhiyi Zhang <zzhang(a)codeweavers.com> Date: Thu Nov 3 17:21:16 2022 +0800 d2d1: Use D2D1_ALPHA_MODE_IGNORE if D2D1_ALPHA_MODE_UNKNOWN is used for creating hwnd render targets. Also, alpha mode should not be overwritten with D2D1_ALPHA_MODE_IGNORE just because DXGI_FORMAT_UNKNOWN is specified. --- dlls/d2d1/hwnd_render_target.c | 4 ++-- dlls/d2d1/tests/d2d1.c | 6 ++---- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/dlls/d2d1/hwnd_render_target.c b/dlls/d2d1/hwnd_render_target.c index 5338ca3383f..4d3fe43a1b3 100644 --- a/dlls/d2d1/hwnd_render_target.c +++ b/dlls/d2d1/hwnd_render_target.c @@ -835,10 +835,10 @@ HRESULT d2d_hwnd_render_target_init(struct d2d_hwnd_render_target *render_target ID2D1Factory1_GetDesktopDpi(factory, &dxgi_rt_desc.dpiX, &dxgi_rt_desc.dpiY); if (dxgi_rt_desc.pixelFormat.format == DXGI_FORMAT_UNKNOWN) - { dxgi_rt_desc.pixelFormat.format = DXGI_FORMAT_B8G8R8A8_UNORM; + + if (dxgi_rt_desc.pixelFormat.alphaMode == D2D1_ALPHA_MODE_UNKNOWN) dxgi_rt_desc.pixelFormat.alphaMode = D2D1_ALPHA_MODE_IGNORE; - } swapchain_desc.BufferDesc.Width = hwnd_rt_desc->pixelSize.width; swapchain_desc.BufferDesc.Height = hwnd_rt_desc->pixelSize.height; diff --git a/dlls/d2d1/tests/d2d1.c b/dlls/d2d1/tests/d2d1.c index 67cb83bdd4a..a065c8ddfb5 100644 --- a/dlls/d2d1/tests/d2d1.c +++ b/dlls/d2d1/tests/d2d1.c @@ -6088,15 +6088,14 @@ static void test_hwnd_target(BOOL d3d11) D2D1_PIXEL_FORMAT format; D2D1_PIXEL_FORMAT expected_format; BOOL expected_failure; - BOOL todo; } format_tests[] = { {{DXGI_FORMAT_UNKNOWN, D2D1_ALPHA_MODE_UNKNOWN}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_IGNORE}}, - {{DXGI_FORMAT_UNKNOWN, D2D1_ALPHA_MODE_PREMULTIPLIED}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_PREMULTIPLIED}, FALSE, TRUE}, + {{DXGI_FORMAT_UNKNOWN, D2D1_ALPHA_MODE_PREMULTIPLIED}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_PREMULTIPLIED}}, {{DXGI_FORMAT_UNKNOWN, D2D1_ALPHA_MODE_STRAIGHT}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_STRAIGHT}, TRUE}, {{DXGI_FORMAT_UNKNOWN, D2D1_ALPHA_MODE_IGNORE}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_IGNORE}}, - {{DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_UNKNOWN}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_IGNORE}, FALSE, TRUE}, + {{DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_UNKNOWN}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_IGNORE}}, {{DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_PREMULTIPLIED}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_PREMULTIPLIED}}, {{DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_STRAIGHT}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_STRAIGHT}, TRUE}, {{DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_IGNORE}, {DXGI_FORMAT_B8G8R8A8_UNORM, D2D1_ALPHA_MODE_IGNORE}}, @@ -6172,7 +6171,6 @@ static void test_hwnd_target(BOOL d3d11) pixel_format = ID2D1HwndRenderTarget_GetPixelFormat(rt); ok(pixel_format.format == format_tests[i].expected_format.format, "Got unexpected format %#x.\n", pixel_format.format); - todo_wine_if(format_tests[i].todo) ok(pixel_format.alphaMode == format_tests[i].expected_format.alphaMode, "Got unexpected alpha mode %d.\n", pixel_format.alphaMode);

1 0

Jinoh Kang : comctl32/tests: Flush events before testing edit control SetFocus() messages.
by Alexandre Julliard 07 Nov '22

07 Nov '22

Module: wine Branch: master Commit: 7a772a7feb69c0fb0a6e24ac0471fb03b31e8f3c URL: https://gitlab.winehq.org/wine/wine/-/commit/7a772a7feb69c0fb0a6e24ac0471fb… Author: Jinoh Kang <jinoh.kang.kr(a)gmail.com> Date: Fri Nov 4 00:31:26 2022 +0900 comctl32/tests: Flush events before testing edit control SetFocus() messages. --- dlls/comctl32/tests/edit.c | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/dlls/comctl32/tests/edit.c b/dlls/comctl32/tests/edit.c index 5f23fff2c9c..ccdeed95d92 100644 --- a/dlls/comctl32/tests/edit.c +++ b/dlls/comctl32/tests/edit.c @@ -54,6 +54,24 @@ struct edit_notify { static struct edit_notify notifications; +/* try to make sure pending X events have been processed before continuing */ +static void flush_events(void) +{ + MSG msg; + int diff = 200; + int min_timeout = 100; + DWORD time = GetTickCount() + diff; + + while (diff > 0) + { + if (MsgWaitForMultipleObjects(0, NULL, FALSE, min_timeout, QS_ALLINPUT) == WAIT_TIMEOUT) + break; + while (PeekMessageA(&msg, 0, 0, 0, PM_REMOVE)) + DispatchMessageA(&msg); + diff = time - GetTickCount(); + } +} + static INT_PTR CALLBACK multi_edit_dialog_proc(HWND hdlg, UINT msg, WPARAM wparam, LPARAM lparam) { static int num_ok_commands = 0; @@ -3445,6 +3463,7 @@ static void test_change_focus(void) SetWindowLongPtrA(hwnd, GWLP_USERDATA, (LONG_PTR)oldproc); SetCursorPos(400, 400); + flush_events(); SetFocus(parent_wnd); flush_sequences(sequences, NUM_MSG_SEQUENCES);

1 0

Jinoh Kang : comctl32/tests: Flush events before testing edit control IME messages.
by Alexandre Julliard 07 Nov '22

07 Nov '22

Module: wine Branch: master Commit: b6a41a2f98c815689525ce343b8bd3a93bceaaae URL: https://gitlab.winehq.org/wine/wine/-/commit/b6a41a2f98c815689525ce343b8bd3… Author: Jinoh Kang <jinoh.kang.kr(a)gmail.com> Date: Fri Nov 4 00:32:09 2022 +0900 comctl32/tests: Flush events before testing edit control IME messages. --- dlls/comctl32/tests/edit.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/comctl32/tests/edit.c b/dlls/comctl32/tests/edit.c index ccdeed95d92..876cddf0de2 100644 --- a/dlls/comctl32/tests/edit.c +++ b/dlls/comctl32/tests/edit.c @@ -3641,6 +3641,7 @@ static void test_ime(void) /* Test IME messages when EIMES_GETCOMPSTRATONCE is not set */ old_proc = (WNDPROC)SetWindowLongPtrA(hwnd, GWLP_WNDPROC, (LONG_PTR)edit_ime_subclass_proc); SetWindowLongPtrA(hwnd, GWLP_USERDATA, (LONG_PTR)old_proc); + flush_events(); himc = ImmGetContext(hwnd); ret = ImmSetCompositionStringA(himc, SCS_SETSTR, "Wine", 4, NULL, 0);

1 0