February 2021 - Wine-Commits

Alexandre Julliard : vdmdbg: Use --prefer-native instead of DLL_WINE_PREATTACH.
by Alexandre Julliard Feb. 25, 2021

Feb. 25, 2021

Module: wine Branch: master Commit: 8818b890516048baac4a7235499c77c2f69c18b4 URL: https://source.winehq.org/git/wine.git/?a=commit;h=8818b890516048baac4a7235… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Feb 25 08:59:09 2021 +0100 vdmdbg: Use --prefer-native instead of DLL_WINE_PREATTACH. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/vdmdbg/Makefile.in | 2 +- dlls/vdmdbg/vdmdbg.c | 16 ---------------- 2 files changed, 1 insertion(+), 17 deletions(-) diff --git a/dlls/vdmdbg/Makefile.in b/dlls/vdmdbg/Makefile.in index d9294b61763..495812797ad 100644 --- a/dlls/vdmdbg/Makefile.in +++ b/dlls/vdmdbg/Makefile.in @@ -1,7 +1,7 @@ MODULE = vdmdbg.dll IMPORTLIB = vdmdbg -EXTRADLLFLAGS = -mno-cygwin +EXTRADLLFLAGS = -mno-cygwin -Wb,--prefer-native C_SRCS = \ vdmdbg.c diff --git a/dlls/vdmdbg/vdmdbg.c b/dlls/vdmdbg/vdmdbg.c index 4d2656c3a1e..b56c62535ac 100644 --- a/dlls/vdmdbg/vdmdbg.c +++ b/dlls/vdmdbg/vdmdbg.c @@ -42,19 +42,3 @@ INT WINAPI VDMEnumProcessWOW(PROCESSENUMPROC fp, LPARAM lparam) FIXME("%p: stub!\n", fp); return 0; } - -/*********************************************************************** - * DllMain (VDMDBG.@) - */ -BOOL WINAPI DllMain( HINSTANCE inst, DWORD reason, LPVOID reserved ) -{ - switch(reason) - { - case DLL_WINE_PREATTACH: - return FALSE; /* prefer native version */ - case DLL_PROCESS_ATTACH: - DisableThreadLibraryCalls( inst ); - break; - } - return TRUE; -}

1 0

Zebediah Figura : winegstreamer: Inline init_new_decoded_pad() into pad_added_cb().
by Alexandre Julliard Feb. 24, 2021

Feb. 24, 2021

Module: wine Branch: master Commit: e4922b930b10c4006a81201481ffca6c448c05b7 URL: https://source.winehq.org/git/wine.git/?a=commit;h=e4922b930b10c4006a812014… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Tue Feb 23 16:07:01 2021 -0600 winegstreamer: Inline init_new_decoded_pad() into pad_added_cb(). Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winegstreamer/wg_parser.c | 20 +++++++------------- 1 file changed, 7 insertions(+), 13 deletions(-) diff --git a/dlls/winegstreamer/wg_parser.c b/dlls/winegstreamer/wg_parser.c index 95e8611fb57..341ca182749 100644 --- a/dlls/winegstreamer/wg_parser.c +++ b/dlls/winegstreamer/wg_parser.c @@ -987,13 +987,19 @@ static struct wg_parser_stream *create_stream(struct wg_parser *parser) return stream; } -static void init_new_decoded_pad(GstElement *element, GstPad *pad, struct wg_parser *parser) +static void pad_added_cb(GstElement *element, GstPad *pad, gpointer user) { + struct wg_parser *parser = user; struct wg_parser_stream *stream; const char *name; GstCaps *caps; int ret; + GST_LOG("parser %p, element %p, pad %p.", parser, element, pad); + + if (gst_pad_is_linked(pad)) + return; + caps = gst_caps_make_writable(gst_pad_query_caps(pad, NULL)); name = gst_structure_get_name(gst_caps_get_structure(caps, 0)); @@ -1115,18 +1121,6 @@ out: gst_caps_unref(caps); } -static void pad_added_cb(GstElement *element, GstPad *pad, gpointer user) -{ - struct wg_parser *parser = user; - - GST_LOG("parser %p, element %p, pad %p.", parser, element, pad); - - if (gst_pad_is_linked(pad)) - return; - - init_new_decoded_pad(element, pad, parser); -} - static void pad_removed_cb(GstElement *element, GstPad *pad, gpointer user) { struct wg_parser *parser = user;

1 0

Zebediah Figura : winegstreamer: Rename gst_base_src_perform_seek() to src_perform_seek().
by Alexandre Julliard Feb. 24, 2021

Feb. 24, 2021

Module: wine Branch: master Commit: 1649389edca12d2998f72a159b2bbbf247176163 URL: https://source.winehq.org/git/wine.git/?a=commit;h=1649389edca12d2998f72a15… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Tue Feb 23 16:07:03 2021 -0600 winegstreamer: Rename gst_base_src_perform_seek() to src_perform_seek(). Avoid polluting the GStreamer namespace. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winegstreamer/wg_parser.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/winegstreamer/wg_parser.c b/dlls/winegstreamer/wg_parser.c index a2eaf1958d6..f76ca903b80 100644 --- a/dlls/winegstreamer/wg_parser.c +++ b/dlls/winegstreamer/wg_parser.c @@ -1387,7 +1387,7 @@ static GstBusSyncReply bus_handler_cb(GstBus *bus, GstMessage *msg, gpointer use return GST_BUS_DROP; } -static gboolean gst_base_src_perform_seek(struct wg_parser *parser, GstEvent *event) +static gboolean src_perform_seek(struct wg_parser *parser, GstEvent *event) { BOOL thread = !!parser->push_thread; GstSeekType cur_type, stop_type; @@ -1444,7 +1444,7 @@ static gboolean src_event_cb(GstPad *pad, GstObject *parent, GstEvent *event) switch (event->type) { case GST_EVENT_SEEK: - ret = gst_base_src_perform_seek(parser, event); + ret = src_perform_seek(parser, event); break; case GST_EVENT_FLUSH_START:

1 0

Alistair Leslie-Hughes : include: Add more SQL_ defines.
by Alexandre Julliard Feb. 24, 2021

Feb. 24, 2021

Module: wine Branch: master Commit: aac82c622e296fff99ee5b54e8e3462054d75fd0 URL: https://source.winehq.org/git/wine.git/?a=commit;h=aac82c622e296fff99ee5b54… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Wed Feb 24 20:30:11 2021 +1100 include: Add more SQL_ defines. Signed-off-by: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- include/sqltypes.h | 3 --- include/sqlucode.h | 5 +++++ 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/include/sqltypes.h b/include/sqltypes.h index 4c45de7e989..70315c271c4 100644 --- a/include/sqltypes.h +++ b/include/sqltypes.h @@ -241,9 +241,6 @@ typedef struct tagSQLGUID typedef SQLULEN BOOKMARK; -#define SQL_WCHAR (-8) -#define SQL_C_WCHAR SQL_WCHAR - #ifdef __cplusplus } #endif diff --git a/include/sqlucode.h b/include/sqlucode.h index c87348cd8b5..7e1fb677cce 100644 --- a/include/sqlucode.h +++ b/include/sqlucode.h @@ -27,6 +27,11 @@ extern "C" { #endif +#define SQL_WCHAR (-8) +#define SQL_C_WCHAR SQL_WCHAR +#define SQL_WVARCHAR (-9) +#define SQL_WLONGVARCHAR (-10) + SQLRETURN WINAPI SQLConnectW(SQLHDBC ConnectionHandle, SQLWCHAR *ServerName, SQLSMALLINT NameLength1, SQLWCHAR *UserName, SQLSMALLINT NameLength2,

1 0

Piotr Caban : msvcrt: Improve asinf compatibility with native ucrtbase.
by Alexandre Julliard Feb. 24, 2021

Feb. 24, 2021

Module: wine Branch: master Commit: 0b8db612713d6a92109396951d6ac53fd23d6039 URL: https://source.winehq.org/git/wine.git/?a=commit;h=0b8db612713d6a9210939695… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Feb 24 16:32:07 2021 +0100 msvcrt: Improve asinf compatibility with native ucrtbase. It reduces number of different asinf results from 5413834 to 639464. Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msvcrt/math.c | 25 +++++++++++++++---------- 1 file changed, 15 insertions(+), 10 deletions(-) diff --git a/dlls/msvcrt/math.c b/dlls/msvcrt/math.c index 83e5d1e65da..6abc1893a52 100644 --- a/dlls/msvcrt/math.c +++ b/dlls/msvcrt/math.c @@ -356,23 +356,25 @@ float CDECL acosf( float x ) static float asinf_R(float z) { /* coefficients for R(x^2) */ - static const float pS0 = 1.6666586697e-01, - pS1 = -4.2743422091e-02, - pS2 = -8.6563630030e-03, - qS1 = -7.0662963390e-01; + static const float p1 = 1.66666672e-01, + p2 = -5.11644611e-02, + p3 = -1.21124933e-02, + p4 = -3.58742251e-03, + q1 = -7.56982703e-01; float p, q; - p = z * (pS0 + z * (pS1 + z * pS2)); - q = 1.0f + z * qS1; + p = z * (p1 + z * (p2 + z * (p3 + z * p4))); + q = 1.0f + z * q1; return p / q; } float CDECL asinf( float x ) { static const double pio2 = 1.570796326794896558e+00; + static const float pio4_hi = 0.785398125648; + static const float pio2_lo = 7.54978941586e-08; - double s; - float z; + float s, z, f, c; unsigned int hx, ix; hx = *(unsigned int*)&x; @@ -391,8 +393,11 @@ float CDECL asinf( float x ) } /* 1 > |x| >= 0.5 */ z = (1 - fabsf(x)) * 0.5f; - s = sqrt(z); - x = pio2 - 2 * (s + s * asinf_R(z)); + s = sqrtf(z); + /* f+c = sqrt(z) */ + *(unsigned int*)&f = *(unsigned int*)&s & 0xffff0000; + c = (z - f * f) / (s + f); + x = pio4_hi - (2 * s * asinf_R(z) - (pio2_lo - 2 * c) - (pio4_hi - 2 * f)); if (hx >> 31) return -x; return x;

1 0

Jacek Caban : ntdll: Use syscall frame in x86_64 call_raise_user_exception_dispatcher implementation.
by Alexandre Julliard Feb. 24, 2021

Feb. 24, 2021

Module: wine Branch: master Commit: 43900dd621cd0564fec3fdd3896fec84f4377e17 URL: https://source.winehq.org/git/wine.git/?a=commit;h=43900dd621cd0564fec3fdd3… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Feb 24 18:25:49 2021 +0100 ntdll: Use syscall frame in x86_64 call_raise_user_exception_dispatcher implementation. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/signal_x86_64.c | 18 ++++-------------- dlls/ntdll/unix/unix_private.h | 2 +- 2 files changed, 5 insertions(+), 15 deletions(-) diff --git a/dlls/ntdll/unix/signal_x86_64.c b/dlls/ntdll/unix/signal_x86_64.c index df83094750e..cd0d0a2c1b0 100644 --- a/dlls/ntdll/unix/signal_x86_64.c +++ b/dlls/ntdll/unix/signal_x86_64.c @@ -2148,20 +2148,10 @@ __ASM_GLOBAL_FUNC( call_user_apc_dispatcher, /*********************************************************************** * call_raise_user_exception_dispatcher */ -__ASM_GLOBAL_FUNC( call_raise_user_exception_dispatcher, - "movq %gs:0x30,%rdx\n\t" - "movq 0x328(%rdx),%rax\n\t" /* amd64_thread_data()->syscall_frame */ - "leaq -0x340(%rax),%r8\n\t" - "andq $~63,%r8\n\t" - "fxrstor64 (%r8)\n\t" - "movq 0x8(%rax),%rbx\n\t" /* frame->rbx */ - "movq 0x20(%rax),%rsi\n\t" /* frame->rsi */ - "movq 0x28(%rax),%rdi\n\t" /* frame->rdi */ - "movq 0x50(%rax),%r12\n\t" /* frame->r12 */ - "movq 0x98(%rax),%rbp\n\t" /* frame->rbp */ - "movq $0,0x328(%rdx)\n\t" - "leaq 0xa8(%rax),%rsp\n\t" - "jmpq *%rcx" ) +void WINAPI call_raise_user_exception_dispatcher( NTSTATUS (WINAPI *dispatcher)(void) ) +{ + amd64_thread_data()->syscall_frame->rip = (UINT64)dispatcher; +} /*********************************************************************** diff --git a/dlls/ntdll/unix/unix_private.h b/dlls/ntdll/unix/unix_private.h index c98b11701e4..cf680218849 100644 --- a/dlls/ntdll/unix/unix_private.h +++ b/dlls/ntdll/unix/unix_private.h @@ -246,7 +246,7 @@ extern void WINAPI DECLSPEC_NORETURN call_user_apc_dispatcher( CONTEXT *context_ void (WINAPI *dispatcher)(CONTEXT*,ULONG_PTR,ULONG_PTR,ULONG_PTR,PNTAPCFUNC) ) DECLSPEC_HIDDEN; extern void WINAPI DECLSPEC_NORETURN call_user_exception_dispatcher( EXCEPTION_RECORD *rec, CONTEXT *context, NTSTATUS (WINAPI *dispatcher)(EXCEPTION_RECORD*,CONTEXT*) ) DECLSPEC_HIDDEN; -extern void WINAPI DECLSPEC_NORETURN call_raise_user_exception_dispatcher( NTSTATUS (WINAPI *dispatcher)(void) ) DECLSPEC_HIDDEN; +extern void WINAPI call_raise_user_exception_dispatcher( NTSTATUS (WINAPI *dispatcher)(void) ) DECLSPEC_HIDDEN; extern void *get_syscall_frame(void) DECLSPEC_HIDDEN; extern void set_syscall_frame(void *frame) DECLSPEC_HIDDEN;

1 0

Jacek Caban : ntdll: Don't use FeatureSet for XSAVE compaction support flag.
by Alexandre Julliard Feb. 24, 2021

Feb. 24, 2021

Module: wine Branch: master Commit: 014f249aea46694a89e6b7b507ddac1ff561d241 URL: https://source.winehq.org/git/wine.git/?a=commit;h=014f249aea46694a89e6b7b5… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Feb 24 18:27:16 2021 +0100 ntdll: Don't use FeatureSet for XSAVE compaction support flag. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/signal_i386.c | 4 ++-- dlls/ntdll/unix/signal_x86_64.c | 9 ++++----- dlls/ntdll/unix/system.c | 4 +++- dlls/ntdll/unix/unix_private.h | 2 ++ include/winternl.h | 3 +-- 5 files changed, 12 insertions(+), 10 deletions(-) diff --git a/dlls/ntdll/unix/signal_i386.c b/dlls/ntdll/unix/signal_i386.c index 4d03153bd1f..6cc80809316 100644 --- a/dlls/ntdll/unix/signal_i386.c +++ b/dlls/ntdll/unix/signal_i386.c @@ -716,7 +716,7 @@ static inline NTSTATUS save_xstate( CONTEXT *context ) || context_ex->XState.Length > sizeof(XSTATE)) return STATUS_INVALID_PARAMETER; - if (user_shared_data->XState.CompactionEnabled) + if (xstate_compaction_enabled) { /* xsavec doesn't use anything from the save area. */ __asm__ volatile( "xsavec %0" : "=m"(xsave_area) @@ -1650,7 +1650,7 @@ C_ASSERT( (offsetof(struct stack_layout, xstate) == sizeof(struct stack_layout)) assert(!((ULONG_PTR)dst_xs & 63)); context_init_xstate( &stack->context, stack->xstate ); memset( dst_xs, 0, offsetof(XSTATE, YmmContext) ); - dst_xs->CompactionMask = user_shared_data->XState.CompactionEnabled ? 0x8000000000000004 : 0; + dst_xs->CompactionMask = xstate_compaction_enabled ? 0x8000000000000004 : 0; if (src_xs->Mask & 4) { dst_xs->Mask = 4; diff --git a/dlls/ntdll/unix/signal_x86_64.c b/dlls/ntdll/unix/signal_x86_64.c index cd0d0a2c1b0..509ff6d460f 100644 --- a/dlls/ntdll/unix/signal_x86_64.c +++ b/dlls/ntdll/unix/signal_x86_64.c @@ -1959,16 +1959,15 @@ NTSTATUS WINAPI NtGetContextThread( HANDLE handle, CONTEXT *context ) { struct syscall_xsave *xsave = get_syscall_xsave( frame ); CONTEXT_EX *context_ex = (CONTEXT_EX *)(context + 1); - const BOOL compaction_enabled = user_shared_data->XState.CompactionEnabled; unsigned int mask; if (context_ex->XState.Length < offsetof(XSTATE, YmmContext) || context_ex->XState.Length > sizeof(XSTATE)) return STATUS_INVALID_PARAMETER; - mask = (compaction_enabled ? xstate->CompactionMask : xstate->Mask) & XSTATE_MASK_GSSE; + mask = (xstate_compaction_enabled ? xstate->CompactionMask : xstate->Mask) & XSTATE_MASK_GSSE; xstate->Mask = xsave->xstate.Mask & mask; - xstate->CompactionMask = compaction_enabled ? (0x8000000000000000 | mask) : 0; + xstate->CompactionMask = xstate_compaction_enabled ? (0x8000000000000000 | mask) : 0; memset( xstate->Reserved, 0, sizeof(xstate->Reserved) ); if (xstate->Mask) { @@ -2044,7 +2043,7 @@ static void setup_raise_exception( ucontext_t *sigcontext, EXCEPTION_RECORD *rec assert( !((ULONG_PTR)dst_xs & 63) ); context_init_xstate( &stack->context, stack->xstate ); memset( dst_xs, 0, offsetof(XSTATE, YmmContext) ); - dst_xs->CompactionMask = user_shared_data->XState.CompactionEnabled ? 0x8000000000000004 : 0; + dst_xs->CompactionMask = xstate_compaction_enabled ? 0x8000000000000004 : 0; if (src_xs->Mask & 4) { dst_xs->Mask = 4; @@ -2807,7 +2806,7 @@ void *signal_init_syscalls(void) extern void __wine_syscall_dispatcher_xsavec(void) DECLSPEC_HIDDEN; NtQuerySystemInformation( SystemCpuInformation, &cpu_info, sizeof(cpu_info), NULL ); - if (cpu_info.FeatureSet & CPU_FEATURE_XSAVEC) + if (xstate_compaction_enabled) syscall_dispatcher = __wine_syscall_dispatcher_xsavec; else if (cpu_info.FeatureSet & CPU_FEATURE_XSAVE) syscall_dispatcher = __wine_syscall_dispatcher_xsave; diff --git a/dlls/ntdll/unix/system.c b/dlls/ntdll/unix/system.c index 27ecfae365f..00fca0fd1f0 100644 --- a/dlls/ntdll/unix/system.c +++ b/dlls/ntdll/unix/system.c @@ -178,6 +178,8 @@ static SYSTEM_CPU_INFORMATION cpu_info; */ #if defined(__i386__) || defined(__x86_64__) +BOOL xstate_compaction_enabled = FALSE; + #define AUTH 0x68747541 /* "Auth" */ #define ENTI 0x69746e65 /* "enti" */ #define CAMD 0x444d4163 /* "cAMD" */ @@ -308,7 +310,7 @@ static void get_cpuinfo( SYSTEM_CPU_INFORMATION *info ) if (info->FeatureSet & CPU_FEATURE_XSAVE) { do_cpuid( 0x0000000d, regs3 ); /* get XSAVE details */ - if (regs3[0] & 2) info->FeatureSet |= CPU_FEATURE_XSAVEC; + if (regs3[0] & 2) xstate_compaction_enabled = TRUE; } if (regs[1] == AUTH && regs[3] == ENTI && regs[2] == CAMD) diff --git a/dlls/ntdll/unix/unix_private.h b/dlls/ntdll/unix/unix_private.h index cf680218849..cd11ba1c46b 100644 --- a/dlls/ntdll/unix/unix_private.h +++ b/dlls/ntdll/unix/unix_private.h @@ -307,6 +307,8 @@ struct xcontext }; #if defined(__i386__) || defined(__x86_64__) +extern BOOL xstate_compaction_enabled DECLSPEC_HIDDEN; + static inline XSTATE *xstate_from_context( const CONTEXT *context ) { CONTEXT_EX *xctx = (CONTEXT_EX *)(context + 1); diff --git a/include/winternl.h b/include/winternl.h index 3beecf10ff6..f0ab223ef2e 100644 --- a/include/winternl.h +++ b/include/winternl.h @@ -1754,14 +1754,13 @@ typedef struct _SYSTEM_CPU_INFORMATION { /* FIXME: following values are made up, actual flags are unknown */ #define CPU_FEATURE_SSSE3 0x00008000 /* SSSE3 instructions */ -#define CPU_FEATURE_XSAVEC 0x00400000 /* XSAVEC instructions */ #define CPU_FEATURE_SSE41 0x01000000 /* SSE41 instructions */ #define CPU_FEATURE_SSE42 0x02000000 /* SSE42 instructions */ #define CPU_FEATURE_AVX 0x40000000 /* AVX instructions */ #define CPU_FEATURE_AVX2 0x80000000 /* AVX2 instructions */ - #define CPU_FEATURE_PAE 0x00200000 #define CPU_FEATURE_DAZ 0x00400000 + #define CPU_FEATURE_ARM_VFP_32 0x00000001 #define CPU_FEATURE_ARM_NEON 0x00000002 #define CPU_FEATURE_ARM_V8_CRC32 0x00000004

1 0

Jacek Caban : ntdll: Use the right cpuid for XSAVEC detection.
by Alexandre Julliard Feb. 24, 2021

Feb. 24, 2021

Module: wine Branch: master Commit: 7b95cdffa6b12eca9c10a6ab45b50fcf1f9fa3a4 URL: https://source.winehq.org/git/wine.git/?a=commit;h=7b95cdffa6b12eca9c10a6ab… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Feb 24 18:27:32 2021 +0100 ntdll: Use the right cpuid for XSAVEC detection. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=50713 Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/system.c | 50 ++++++++++++------------------------------------ 1 file changed, 12 insertions(+), 38 deletions(-) diff --git a/dlls/ntdll/unix/system.c b/dlls/ntdll/unix/system.c index 00fca0fd1f0..56bdd63c89b 100644 --- a/dlls/ntdll/unix/system.c +++ b/dlls/ntdll/unix/system.c @@ -188,36 +188,10 @@ BOOL xstate_compaction_enabled = FALSE; #define INEI 0x49656e69 /* "ineI" */ #define NTEL 0x6c65746e /* "ntel" */ -extern void do_cpuid(unsigned int ax, unsigned int *p); - -#ifdef __i386__ -__ASM_GLOBAL_FUNC( do_cpuid, - "pushl %esi\n\t" - "pushl %ebx\n\t" - "movl 12(%esp),%eax\n\t" - "movl 16(%esp),%esi\n\t" - "xorl %ecx,%ecx\n\t" - "cpuid\n\t" - "movl %eax,(%esi)\n\t" - "movl %ebx,4(%esi)\n\t" - "movl %ecx,8(%esi)\n\t" - "movl %edx,12(%esi)\n\t" - "popl %ebx\n\t" - "popl %esi\n\t" - "ret" ) -#else -__ASM_GLOBAL_FUNC( do_cpuid, - "pushq %rbx\n\t" - "movl %edi,%eax\n\t" - "xorl %ecx,%ecx\n\t" - "cpuid\n\t" - "movl %eax,(%rsi)\n\t" - "movl %ebx,4(%rsi)\n\t" - "movl %ecx,8(%rsi)\n\t" - "movl %edx,12(%rsi)\n\t" - "popq %rbx\n\t" - "ret" ) -#endif +static inline void do_cpuid(unsigned int ax, unsigned int cx, unsigned int *p) +{ + __asm__ ("cpuid" : "=a"(p[0]), "=b" (p[1]), "=c"(p[2]), "=d"(p[3]) : "a"(ax), "c"(cx)); +} #ifdef __i386__ extern int have_cpuid(void); @@ -275,10 +249,10 @@ static void get_cpuinfo( SYSTEM_CPU_INFORMATION *info ) if (!have_cpuid()) return; - do_cpuid( 0x00000000, regs ); /* get standard cpuid level and vendor name */ + do_cpuid( 0x00000000, 0, regs ); /* get standard cpuid level and vendor name */ if (regs[0]>=0x00000001) /* Check for supported cpuid version */ { - do_cpuid( 0x00000001, regs2 ); /* get cpu features */ + do_cpuid( 0x00000001, 0, regs2 ); /* get cpu features */ if (regs2[3] & (1 << 3 )) info->FeatureSet |= CPU_FEATURE_PSE; if (regs2[3] & (1 << 4 )) info->FeatureSet |= CPU_FEATURE_TSC; if (regs2[3] & (1 << 6 )) info->FeatureSet |= CPU_FEATURE_PAE; @@ -303,13 +277,13 @@ static void get_cpuinfo( SYSTEM_CPU_INFORMATION *info ) if (regs[0] >= 0x00000007) { - do_cpuid( 0x00000007, regs3 ); /* get extended features */ + do_cpuid( 0x00000007, 0, regs3 ); /* get extended features */ if (regs3[1] & (1 << 5)) info->FeatureSet |= CPU_FEATURE_AVX2; } if (info->FeatureSet & CPU_FEATURE_XSAVE) { - do_cpuid( 0x0000000d, regs3 ); /* get XSAVE details */ + do_cpuid( 0x0000000d, 1, regs3 ); /* get XSAVE details */ if (regs3[0] & 2) xstate_compaction_enabled = TRUE; } @@ -324,10 +298,10 @@ static void get_cpuinfo( SYSTEM_CPU_INFORMATION *info ) info->Revision |= ((regs2[0] >> 4 ) & 0xf) << 8; /* model */ info->Revision |= regs2[0] & 0xf; /* stepping */ - do_cpuid( 0x80000000, regs ); /* get vendor cpuid level */ + do_cpuid( 0x80000000, 0, regs ); /* get vendor cpuid level */ if (regs[0] >= 0x80000001) { - do_cpuid( 0x80000001, regs2 ); /* get vendor features */ + do_cpuid( 0x80000001, 0, regs2 ); /* get vendor features */ if (regs2[2] & (1 << 2)) info->FeatureSet |= CPU_FEATURE_VIRT; if (regs2[3] & (1 << 20)) info->FeatureSet |= CPU_FEATURE_NX; if (regs2[3] & (1 << 27)) info->FeatureSet |= CPU_FEATURE_TSC; @@ -347,10 +321,10 @@ static void get_cpuinfo( SYSTEM_CPU_INFORMATION *info ) if(regs2[2] & (1 << 5)) info->FeatureSet |= CPU_FEATURE_VIRT; if(regs2[3] & (1 << 21)) info->FeatureSet |= CPU_FEATURE_DS; - do_cpuid( 0x80000000, regs ); /* get vendor cpuid level */ + do_cpuid( 0x80000000, 0, regs ); /* get vendor cpuid level */ if (regs[0] >= 0x80000001) { - do_cpuid( 0x80000001, regs2 ); /* get vendor features */ + do_cpuid( 0x80000001, 0, regs2 ); /* get vendor features */ if (regs2[3] & (1 << 20)) info->FeatureSet |= CPU_FEATURE_NX; if (regs2[3] & (1 << 27)) info->FeatureSet |= CPU_FEATURE_TSC; }

1 0

Zebediah Figura : winegstreamer: Rename some callbacks to more closely match GStreamer terminology.
by Alexandre Julliard Feb. 24, 2021

Feb. 24, 2021

Module: wine Branch: master Commit: 39c97f4c2957868ba9de382df4fd03ff14a2c62e URL: https://source.winehq.org/git/wine.git/?a=commit;h=39c97f4c2957868ba9de382d… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Tue Feb 23 16:06:59 2021 -0600 winegstreamer: Rename some callbacks to more closely match GStreamer terminology. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winegstreamer/wg_parser.c | 32 +++++++++++++++++--------------- 1 file changed, 17 insertions(+), 15 deletions(-) diff --git a/dlls/winegstreamer/wg_parser.c b/dlls/winegstreamer/wg_parser.c index 94f677538c7..63ba95a78cb 100644 --- a/dlls/winegstreamer/wg_parser.c +++ b/dlls/winegstreamer/wg_parser.c @@ -699,7 +699,8 @@ static void CDECL wg_parser_stream_notify_qos(struct wg_parser_stream *stream, gst_pad_push_event(stream->my_sink, event); } -static GstAutoplugSelectResult autoplug_blacklist(GstElement *bin, GstPad *pad, GstCaps *caps, GstElementFactory *fact, gpointer user) +static GstAutoplugSelectResult autoplug_select_cb(GstElement *bin, GstPad *pad, + GstCaps *caps, GstElementFactory *fact, gpointer user) { const char *name = gst_element_factory_get_longname(fact); @@ -865,7 +866,7 @@ static gboolean event_sink(GstPad *pad, GstObject *parent, GstEvent *event) return TRUE; } -static GstFlowReturn got_data_sink(GstPad *pad, GstObject *parent, GstBuffer *buffer) +static GstFlowReturn sink_chain_cb(GstPad *pad, GstObject *parent, GstBuffer *buffer) { struct wg_parser_stream *stream = gst_pad_get_element_private(pad); struct wg_parser_event stream_event; @@ -978,7 +979,7 @@ static struct wg_parser_stream *create_stream(struct wg_parser *parser) sprintf(pad_name, "qz_sink_%u", parser->stream_count); stream->my_sink = gst_pad_new(pad_name, GST_PAD_SINK); gst_pad_set_element_private(stream->my_sink, stream); - gst_pad_set_chain_function(stream->my_sink, got_data_sink); + gst_pad_set_chain_function(stream->my_sink, sink_chain_cb); gst_pad_set_event_function(stream->my_sink, event_sink); gst_pad_set_query_function(stream->my_sink, query_sink); @@ -1114,7 +1115,7 @@ out: gst_caps_unref(caps); } -static void existing_new_pad(GstElement *element, GstPad *pad, gpointer user) +static void pad_added_cb(GstElement *element, GstPad *pad, gpointer user) { struct wg_parser *parser = user; @@ -1126,7 +1127,7 @@ static void existing_new_pad(GstElement *element, GstPad *pad, gpointer user) init_new_decoded_pad(element, pad, parser); } -static void removed_decoded_pad(GstElement *element, GstPad *pad, gpointer user) +static void pad_removed_cb(GstElement *element, GstPad *pad, gpointer user) { struct wg_parser *parser = user; unsigned int i; @@ -1155,7 +1156,8 @@ static void removed_decoded_pad(GstElement *element, GstPad *pad, gpointer user) g_free(name); } -static GstFlowReturn request_buffer_src(GstPad *pad, GstObject *parent, guint64 offset, guint size, GstBuffer **buffer) +static GstFlowReturn src_getrange_cb(GstPad *pad, GstObject *parent, + guint64 offset, guint size, GstBuffer **buffer) { struct wg_parser *parser = gst_pad_get_element_private(pad); GstBuffer *new_buffer = NULL; @@ -1277,7 +1279,7 @@ static void *push_data(void *arg) break; size = min(16384, max_size - parser->next_offset); - if ((ret = request_buffer_src(parser->my_src, NULL, parser->next_offset, size, &buffer)) < 0) + if ((ret = src_getrange_cb(parser->my_src, NULL, parser->next_offset, size, &buffer)) < 0) { GST_ERROR("Failed to read data, ret %s.", gst_flow_get_name(ret)); break; @@ -1347,7 +1349,7 @@ static gboolean activate_mode(GstPad *pad, GstObject *parent, GstPadMode mode, g return FALSE; } -static GstBusSyncReply watch_bus(GstBus *bus, GstMessage *msg, gpointer user) +static GstBusSyncReply bus_handler_cb(GstBus *bus, GstMessage *msg, gpointer user) { struct wg_parser *parser = user; gchar *dbg_info = NULL; @@ -1498,14 +1500,14 @@ static HRESULT CDECL wg_parser_connect(struct wg_parser *parser, uint64_t file_s if (!parser->bus) { parser->bus = gst_bus_new(); - gst_bus_set_sync_handler(parser->bus, watch_bus, parser, NULL); + gst_bus_set_sync_handler(parser->bus, bus_handler_cb, parser, NULL); } parser->container = gst_bin_new(NULL); gst_element_set_bus(parser->container, parser->bus); parser->my_src = gst_pad_new_from_static_template(&src_template, "quartz-src"); - gst_pad_set_getrange_function(parser->my_src, request_buffer_src); + gst_pad_set_getrange_function(parser->my_src, src_getrange_cb); gst_pad_set_query_function(parser->my_src, query_function); gst_pad_set_activatemode_function(parser->my_src, activate_mode); gst_pad_set_event_function(parser->my_src, event_src); @@ -1614,9 +1616,9 @@ static BOOL decodebin_parser_init_gst(struct wg_parser *parser) gst_bin_add(GST_BIN(parser->container), element); parser->decodebin = element; - g_signal_connect(element, "pad-added", G_CALLBACK(existing_new_pad), parser); - g_signal_connect(element, "pad-removed", G_CALLBACK(removed_decoded_pad), parser); - g_signal_connect(element, "autoplug-select", G_CALLBACK(autoplug_blacklist), parser); + g_signal_connect(element, "pad-added", G_CALLBACK(pad_added_cb), parser); + g_signal_connect(element, "pad-removed", G_CALLBACK(pad_removed_cb), parser); + g_signal_connect(element, "autoplug-select", G_CALLBACK(autoplug_select_cb), parser); g_signal_connect(element, "no-more-pads", G_CALLBACK(no_more_pads), parser); parser->their_sink = gst_element_get_static_pad(element, "sink"); @@ -1666,8 +1668,8 @@ static BOOL avi_parser_init_gst(struct wg_parser *parser) gst_bin_add(GST_BIN(parser->container), element); - g_signal_connect(element, "pad-added", G_CALLBACK(existing_new_pad), parser); - g_signal_connect(element, "pad-removed", G_CALLBACK(removed_decoded_pad), parser); + g_signal_connect(element, "pad-added", G_CALLBACK(pad_added_cb), parser); + g_signal_connect(element, "pad-removed", G_CALLBACK(pad_removed_cb), parser); g_signal_connect(element, "no-more-pads", G_CALLBACK(no_more_pads), parser); parser->their_sink = gst_element_get_static_pad(element, "sink");

1 0

Zebediah Figura : winegstreamer: Rename some callbacks to explicitly show they are callbacks.
by Alexandre Julliard Feb. 24, 2021

Feb. 24, 2021

Module: wine Branch: master Commit: 2741b6ca33262ba61d7a2d39f3325bf048f4abe9 URL: https://source.winehq.org/git/wine.git/?a=commit;h=2741b6ca33262ba61d7a2d39… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Tue Feb 23 16:07:00 2021 -0600 winegstreamer: Rename some callbacks to explicitly show they are callbacks. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winegstreamer/wg_parser.c | 26 +++++++++++++------------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/dlls/winegstreamer/wg_parser.c b/dlls/winegstreamer/wg_parser.c index 63ba95a78cb..95e8611fb57 100644 --- a/dlls/winegstreamer/wg_parser.c +++ b/dlls/winegstreamer/wg_parser.c @@ -719,7 +719,7 @@ static GstAutoplugSelectResult autoplug_select_cb(GstElement *bin, GstPad *pad, return GST_AUTOPLUG_SELECT_TRY; } -static void no_more_pads(GstElement *element, gpointer user) +static void no_more_pads_cb(GstElement *element, gpointer user) { struct wg_parser *parser = user; @@ -769,7 +769,7 @@ static GstFlowReturn queue_stream_event(struct wg_parser_stream *stream, return GST_FLOW_OK; } -static gboolean event_sink(GstPad *pad, GstObject *parent, GstEvent *event) +static gboolean sink_event_cb(GstPad *pad, GstObject *parent, GstEvent *event) { struct wg_parser_stream *stream = gst_pad_get_element_private(pad); struct wg_parser *parser = stream->parser; @@ -897,7 +897,7 @@ static GstFlowReturn sink_chain_cb(GstPad *pad, GstObject *parent, GstBuffer *bu return ret; } -static gboolean query_sink(GstPad *pad, GstObject *parent, GstQuery *query) +static gboolean sink_query_cb(GstPad *pad, GstObject *parent, GstQuery *query) { struct wg_parser_stream *stream = gst_pad_get_element_private(pad); @@ -980,8 +980,8 @@ static struct wg_parser_stream *create_stream(struct wg_parser *parser) stream->my_sink = gst_pad_new(pad_name, GST_PAD_SINK); gst_pad_set_element_private(stream->my_sink, stream); gst_pad_set_chain_function(stream->my_sink, sink_chain_cb); - gst_pad_set_event_function(stream->my_sink, event_sink); - gst_pad_set_query_function(stream->my_sink, query_sink); + gst_pad_set_event_function(stream->my_sink, sink_event_cb); + gst_pad_set_query_function(stream->my_sink, sink_query_cb); parser->streams[parser->stream_count++] = stream; return stream; @@ -1209,7 +1209,7 @@ static GstFlowReturn src_getrange_cb(GstPad *pad, GstObject *parent, return ret ? GST_FLOW_OK : GST_FLOW_ERROR; } -static gboolean query_function(GstPad *pad, GstObject *parent, GstQuery *query) +static gboolean src_query_cb(GstPad *pad, GstObject *parent, GstQuery *query) { struct wg_parser *parser = gst_pad_get_element_private(pad); GstFormat format; @@ -1330,7 +1330,7 @@ static gboolean activate_push(GstPad *pad, gboolean activate) return TRUE; } -static gboolean activate_mode(GstPad *pad, GstObject *parent, GstPadMode mode, gboolean activate) +static gboolean src_activate_mode_cb(GstPad *pad, GstObject *parent, GstPadMode mode, gboolean activate) { struct wg_parser *parser = gst_pad_get_element_private(pad); @@ -1440,7 +1440,7 @@ static gboolean gst_base_src_perform_seek(struct wg_parser *parser, GstEvent *ev return TRUE; } -static gboolean event_src(GstPad *pad, GstObject *parent, GstEvent *event) +static gboolean src_event_cb(GstPad *pad, GstObject *parent, GstEvent *event) { struct wg_parser *parser = gst_pad_get_element_private(pad); gboolean ret = TRUE; @@ -1508,9 +1508,9 @@ static HRESULT CDECL wg_parser_connect(struct wg_parser *parser, uint64_t file_s parser->my_src = gst_pad_new_from_static_template(&src_template, "quartz-src"); gst_pad_set_getrange_function(parser->my_src, src_getrange_cb); - gst_pad_set_query_function(parser->my_src, query_function); - gst_pad_set_activatemode_function(parser->my_src, activate_mode); - gst_pad_set_event_function(parser->my_src, event_src); + gst_pad_set_query_function(parser->my_src, src_query_cb); + gst_pad_set_activatemode_function(parser->my_src, src_activate_mode_cb); + gst_pad_set_event_function(parser->my_src, src_event_cb); gst_pad_set_element_private(parser->my_src, parser); parser->start_offset = parser->next_offset = parser->stop_offset = 0; @@ -1619,7 +1619,7 @@ static BOOL decodebin_parser_init_gst(struct wg_parser *parser) g_signal_connect(element, "pad-added", G_CALLBACK(pad_added_cb), parser); g_signal_connect(element, "pad-removed", G_CALLBACK(pad_removed_cb), parser); g_signal_connect(element, "autoplug-select", G_CALLBACK(autoplug_select_cb), parser); - g_signal_connect(element, "no-more-pads", G_CALLBACK(no_more_pads), parser); + g_signal_connect(element, "no-more-pads", G_CALLBACK(no_more_pads_cb), parser); parser->their_sink = gst_element_get_static_pad(element, "sink"); @@ -1670,7 +1670,7 @@ static BOOL avi_parser_init_gst(struct wg_parser *parser) g_signal_connect(element, "pad-added", G_CALLBACK(pad_added_cb), parser); g_signal_connect(element, "pad-removed", G_CALLBACK(pad_removed_cb), parser); - g_signal_connect(element, "no-more-pads", G_CALLBACK(no_more_pads), parser); + g_signal_connect(element, "no-more-pads", G_CALLBACK(no_more_pads_cb), parser); parser->their_sink = gst_element_get_static_pad(element, "sink");

1 0