May 2022 - Wine-Commits - list.winehq.org

Jacek Caban : winemac: Use pthread for synchronization in opengl.c.
by Alexandre Julliard May 18, 2022

May 18, 2022

Module: wine Branch: master Commit: 33e6932c045f1e73709f483ccd4c9d897b2eedd8 URL: https://source.winehq.org/git/wine.git/?a=commit;h=33e6932c045f1e73709f483c… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 17 01:25:22 2022 +0200 winemac: Use pthread for synchronization in opengl.c. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winemac.drv/opengl.c | 84 +++++++++++++++++------------------------------ 1 file changed, 31 insertions(+), 53 deletions(-) diff --git a/dlls/winemac.drv/opengl.c b/dlls/winemac.drv/opengl.c index d61d0a6ea66..f553b93da5a 100644 --- a/dlls/winemac.drv/opengl.c +++ b/dlls/winemac.drv/opengl.c @@ -77,15 +77,7 @@ struct wgl_context }; static struct list context_list = LIST_INIT(context_list); - -static CRITICAL_SECTION context_section; -static CRITICAL_SECTION_DEBUG critsect_debug = -{ - 0, 0, &context_section, - { &critsect_debug.ProcessLocksList, &critsect_debug.ProcessLocksList }, - 0, 0, { (DWORD_PTR)(__FILE__ ": context_section") } -}; -static CRITICAL_SECTION context_section = { &critsect_debug, -1, 0, 0, 0, 0 }; +static pthread_mutex_t context_mutex = PTHREAD_MUTEX_INITIALIZER; struct wgl_pbuffer @@ -99,15 +91,7 @@ struct wgl_pbuffer }; static CFMutableDictionaryRef dc_pbuffers; - -static CRITICAL_SECTION dc_pbuffers_section; -static CRITICAL_SECTION_DEBUG dc_pbuffers_section_debug = -{ - 0, 0, &dc_pbuffers_section, - { &dc_pbuffers_section_debug.ProcessLocksList, &dc_pbuffers_section_debug.ProcessLocksList }, - 0, 0, { (DWORD_PTR)(__FILE__ ": dc_pbuffers_section") } -}; -static CRITICAL_SECTION dc_pbuffers_section = { &dc_pbuffers_section_debug, -1, 0, 0, 0, 0 }; +static pthread_mutex_t dc_pbuffers_mutex = PTHREAD_MUTEX_INITIALIZER; static struct opengl_funcs opengl_funcs; @@ -1363,7 +1347,7 @@ static int get_dc_pixel_format(HDC hdc) { struct wgl_pbuffer *pbuffer; - EnterCriticalSection(&dc_pbuffers_section); + pthread_mutex_lock(&dc_pbuffers_mutex); pbuffer = (struct wgl_pbuffer*)CFDictionaryGetValue(dc_pbuffers, hdc); if (pbuffer) format = pbuffer->format; @@ -1372,7 +1356,7 @@ static int get_dc_pixel_format(HDC hdc) WARN("no window or pbuffer for DC %p\n", hdc); format = 0; } - LeaveCriticalSection(&dc_pbuffers_section); + pthread_mutex_unlock(&dc_pbuffers_mutex); } return format; @@ -1612,13 +1596,13 @@ static void mark_contexts_for_moved_view(macdrv_view view) { struct wgl_context *context; - EnterCriticalSection(&context_section); + pthread_mutex_lock(&context_mutex); LIST_FOR_EACH_ENTRY(context, &context_list, struct wgl_context, entry) { if (context->draw_view == view) InterlockedExchange(&context->view_moved, TRUE); } - LeaveCriticalSection(&context_section); + pthread_mutex_unlock(&context_mutex); } @@ -2885,9 +2869,9 @@ static struct wgl_context *macdrv_wglCreateContextAttribsARB(HDC hdc, return NULL; } - EnterCriticalSection(&context_section); + pthread_mutex_lock(&context_mutex); list_add_tail(&context_list, &context->entry); - LeaveCriticalSection(&context_section); + pthread_mutex_unlock(&context_mutex); return context; } @@ -3090,7 +3074,7 @@ static HDC macdrv_wglGetPbufferDCARB(struct wgl_pbuffer *pbuffer) hdc = CreateDCA("DISPLAY", NULL, NULL, NULL); if (!hdc) return 0; - EnterCriticalSection(&dc_pbuffers_section); + pthread_mutex_lock(&dc_pbuffers_mutex); prev = (struct wgl_pbuffer*)CFDictionaryGetValue(dc_pbuffers, hdc); if (prev) { @@ -3098,7 +3082,7 @@ static HDC macdrv_wglGetPbufferDCARB(struct wgl_pbuffer *pbuffer) HeapFree(GetProcessHeap(), 0, prev); } CFDictionarySetValue(dc_pbuffers, hdc, pbuffer); - LeaveCriticalSection(&dc_pbuffers_section); + pthread_mutex_unlock(&dc_pbuffers_mutex); TRACE("pbuffer %p -> hdc %p\n", pbuffer, hdc); return hdc; @@ -3538,14 +3522,14 @@ static BOOL macdrv_wglMakeContextCurrentARB(HDC draw_hdc, HDC read_hdc, struct w { struct wgl_pbuffer *pbuffer; - EnterCriticalSection(&dc_pbuffers_section); + pthread_mutex_lock(&dc_pbuffers_mutex); pbuffer = (struct wgl_pbuffer*)CFDictionaryGetValue(dc_pbuffers, draw_hdc); if (pbuffer) { if (context->format != pbuffer->format) { WARN("mismatched pixel format draw_hdc %p %u context %p %u\n", draw_hdc, pbuffer->format, context, context->format); - LeaveCriticalSection(&dc_pbuffers_section); + pthread_mutex_unlock(&dc_pbuffers_mutex); SetLastError(ERROR_INVALID_PIXEL_FORMAT); return FALSE; } @@ -3556,7 +3540,7 @@ static BOOL macdrv_wglMakeContextCurrentARB(HDC draw_hdc, HDC read_hdc, struct w else { WARN("no window or pbuffer for DC\n"); - LeaveCriticalSection(&dc_pbuffers_section); + pthread_mutex_unlock(&dc_pbuffers_mutex); SetLastError(ERROR_INVALID_HANDLE); return FALSE; } @@ -3564,7 +3548,7 @@ static BOOL macdrv_wglMakeContextCurrentARB(HDC draw_hdc, HDC read_hdc, struct w context->draw_hwnd = NULL; context->draw_view = NULL; context->draw_pbuffer = pbuffer; - LeaveCriticalSection(&dc_pbuffers_section); + pthread_mutex_unlock(&dc_pbuffers_mutex); } context->read_view = NULL; @@ -3586,9 +3570,9 @@ static BOOL macdrv_wglMakeContextCurrentARB(HDC draw_hdc, HDC read_hdc, struct w } else { - EnterCriticalSection(&dc_pbuffers_section); + pthread_mutex_lock(&dc_pbuffers_mutex); context->read_pbuffer = (struct wgl_pbuffer*)CFDictionaryGetValue(dc_pbuffers, read_hdc); - LeaveCriticalSection(&dc_pbuffers_section); + pthread_mutex_unlock(&dc_pbuffers_mutex); } } @@ -3933,7 +3917,7 @@ static int macdrv_wglReleasePbufferDCARB(struct wgl_pbuffer *pbuffer, HDC hdc) TRACE("pbuffer %p hdc %p\n", pbuffer, hdc); - EnterCriticalSection(&dc_pbuffers_section); + pthread_mutex_lock(&dc_pbuffers_mutex); prev = (struct wgl_pbuffer*)CFDictionaryGetValue(dc_pbuffers, hdc); if (prev) @@ -3946,7 +3930,7 @@ static int macdrv_wglReleasePbufferDCARB(struct wgl_pbuffer *pbuffer, HDC hdc) } else hdc = 0; - LeaveCriticalSection(&dc_pbuffers_section); + pthread_mutex_unlock(&dc_pbuffers_mutex); return hdc && DeleteDC(hdc); } @@ -4109,13 +4093,13 @@ static BOOL macdrv_wglSwapIntervalEXT(int interval) { struct wgl_context *ctx; - EnterCriticalSection(&context_section); + pthread_mutex_lock(&context_mutex); LIST_FOR_EACH_ENTRY(ctx, &context_list, struct wgl_context, entry) { if (ctx != context && ctx->draw_hwnd == context->draw_hwnd) InterlockedExchange(&context->update_swap_interval, TRUE); } - LeaveCriticalSection(&context_section); + pthread_mutex_unlock(&context_mutex); } return TRUE; @@ -4222,21 +4206,17 @@ static void load_extensions(void) } -static BOOL CALLBACK init_opengl(INIT_ONCE *init_once, void *context, void **param) +static void init_opengl(void) { - static BOOL init_done = FALSE; unsigned int i; - if (init_done) return (opengl_handle != NULL); - init_done = TRUE; - TRACE("()\n"); dc_pbuffers = CFDictionaryCreateMutable(NULL, 0, NULL, NULL); if (!dc_pbuffers) { WARN("CFDictionaryCreateMutable failed\n"); - return FALSE; + return; } opengl_handle = dlopen("/System/Library/Frameworks/OpenGL.framework/OpenGL", RTLD_LAZY|RTLD_LOCAL|RTLD_NOLOAD); @@ -4244,7 +4224,7 @@ static BOOL CALLBACK init_opengl(INIT_ONCE *init_once, void *context, void **par { ERR("Failed to load OpenGL: %s\n", dlerror()); ERR("OpenGL support is disabled.\n"); - return FALSE; + return; } for (i = 0; i < ARRAY_SIZE(opengl_func_names); i++) @@ -4285,12 +4265,11 @@ static BOOL CALLBACK init_opengl(INIT_ONCE *init_once, void *context, void **par if (!init_pixel_formats()) goto failed; - return TRUE; + return; failed: dlclose(opengl_handle); opengl_handle = NULL; - return FALSE; } @@ -4419,9 +4398,9 @@ static BOOL WINAPI macdrv_wglDeleteContext(struct wgl_context *context) { TRACE("deleting context %p/%p/%p\n", context, context->context, context->cglcontext); - EnterCriticalSection(&context_section); + pthread_mutex_lock(&context_mutex); list_remove(&context->entry); - LeaveCriticalSection(&context_section); + pthread_mutex_unlock(&context_mutex); macdrv_dispose_opengl_context(context->context); return HeapFree(GetProcessHeap(), 0, context); @@ -4583,9 +4562,9 @@ static BOOL WINAPI macdrv_wglSwapBuffers(HDC hdc) { struct wgl_pbuffer *pbuffer; - EnterCriticalSection(&dc_pbuffers_section); + pthread_mutex_lock(&dc_pbuffers_mutex); pbuffer = (struct wgl_pbuffer*)CFDictionaryGetValue(dc_pbuffers, hdc); - LeaveCriticalSection(&dc_pbuffers_section); + pthread_mutex_unlock(&dc_pbuffers_mutex); if (!pbuffer) { @@ -4634,7 +4613,7 @@ static struct opengl_funcs opengl_funcs = */ struct opengl_funcs *macdrv_wine_get_wgl_driver(UINT version) { - static INIT_ONCE opengl_init = INIT_ONCE_STATIC_INIT; + static pthread_once_t init_once = PTHREAD_ONCE_INIT; if (version != WINE_WGL_DRIVER_VERSION) { @@ -4642,7 +4621,6 @@ struct opengl_funcs *macdrv_wine_get_wgl_driver(UINT version) return NULL; } - if (!InitOnceExecuteOnce(&opengl_init, init_opengl, NULL, NULL)) return (void *)-1; - - return &opengl_funcs; + pthread_once(&init_once, init_opengl); + return opengl_handle ? &opengl_funcs : (void *)-1; }

1 0

Rémi Bernon : ntdll: Increase heap large block min size for sizeof(void *) == 8.
by Alexandre Julliard May 18, 2022

May 18, 2022

Module: wine Branch: master Commit: 69c75b2b1da963add2e3208c5e67541dcbe23741 URL: https://source.winehq.org/git/wine.git/?a=commit;h=69c75b2b1da963add2e3208c… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon May 2 13:21:28 2022 +0200 ntdll: Increase heap large block min size for sizeof(void *) == 8. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/heap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/ntdll/heap.c b/dlls/ntdll/heap.c index 374b4e1f209..75622da7a6a 100644 --- a/dlls/ntdll/heap.c +++ b/dlls/ntdll/heap.c @@ -134,7 +134,7 @@ C_ASSERT( sizeof(ARENA_LARGE) % LARGE_ALIGNMENT == 0 ); /* minimum size that must remain to shrink an allocated block */ #define HEAP_MIN_SHRINK_SIZE (HEAP_MIN_DATA_SIZE+sizeof(ARENA_FREE)) /* minimum size to start allocating large blocks */ -#define HEAP_MIN_LARGE_BLOCK_SIZE 0x7f000 +#define HEAP_MIN_LARGE_BLOCK_SIZE (0x10000 * ALIGNMENT - 0x1000) /* extra size to add at the end of block for tail checking */ #define HEAP_TAIL_EXTRA_SIZE(flags) \ ((flags & HEAP_TAIL_CHECKING_ENABLED) || RUNNING_ON_VALGRIND ? ALIGNMENT : 0)

1 0

Jacek Caban : winemac: Use pthread_once for wine_vk_init.
by Alexandre Julliard May 18, 2022

May 18, 2022

Module: wine Branch: master Commit: f7ffa59079837d0b509443ea084f14ed9cc0e8ef URL: https://source.winehq.org/git/wine.git/?a=commit;h=f7ffa59079837d0b509443ea… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed May 18 00:21:29 2022 +0200 winemac: Use pthread_once for wine_vk_init. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winemac.drv/vulkan.c | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/dlls/winemac.drv/vulkan.c b/dlls/winemac.drv/vulkan.c index 67273b8f20b..de99e4035bc 100644 --- a/dlls/winemac.drv/vulkan.c +++ b/dlls/winemac.drv/vulkan.c @@ -101,12 +101,12 @@ static inline struct wine_vk_surface *surface_from_handle(VkSurfaceKHR handle) static void *vulkan_handle; -static BOOL WINAPI wine_vk_init(INIT_ONCE *once, void *param, void **context) +static void wine_vk_init(void) { if (!(vulkan_handle = dlopen(SONAME_LIBMOLTENVK, RTLD_NOW))) { ERR("Failed to load %s\n", SONAME_LIBMOLTENVK); - return TRUE; + return; } #define LOAD_FUNCPTR(f) if ((p##f = dlsym(vulkan_handle, #f)) == NULL) goto fail; @@ -130,12 +130,11 @@ static BOOL WINAPI wine_vk_init(INIT_ONCE *once, void *param, void **context) LOAD_FUNCPTR(vkQueuePresentKHR) #undef LOAD_FUNCPTR - return TRUE; + return; fail: dlclose(vulkan_handle); vulkan_handle = NULL; - return TRUE; } /* Helper function for converting between win32 and MoltenVK compatible VkInstanceCreateInfo. @@ -625,7 +624,7 @@ static void *macdrv_get_vk_instance_proc_addr(VkInstance instance, const char *n static const struct vulkan_funcs *get_vulkan_driver(UINT version) { - static INIT_ONCE init_once = INIT_ONCE_STATIC_INIT; + static pthread_once_t init_once = PTHREAD_ONCE_INIT; if (version != WINE_VULKAN_DRIVER_VERSION) { @@ -633,7 +632,7 @@ static const struct vulkan_funcs *get_vulkan_driver(UINT version) return NULL; } - InitOnceExecuteOnce(&init_once, wine_vk_init, NULL, NULL); + pthread_once(&init_once, wine_vk_init); if (vulkan_handle) return &vulkan_funcs;

1 0

Jacek Caban : winemac: Use pthread for keyboard list locking.
by Alexandre Julliard May 18, 2022

May 18, 2022

Module: wine Branch: master Commit: eae3578d893570a1a43e1d3c7d564323558cd9ae URL: https://source.winehq.org/git/wine.git/?a=commit;h=eae3578d893570a1a43e1d3c… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 17 01:14:57 2022 +0200 winemac: Use pthread for keyboard list locking. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winemac.drv/keyboard.c | 25 +++++++++---------------- 1 file changed, 9 insertions(+), 16 deletions(-) diff --git a/dlls/winemac.drv/keyboard.c b/dlls/winemac.drv/keyboard.c index acd6fe6bf0a..79f208edfc8 100644 --- a/dlls/winemac.drv/keyboard.c +++ b/dlls/winemac.drv/keyboard.c @@ -437,14 +437,7 @@ struct layout BOOL enabled; /* is the input source enabled - ie displayed in the input source selector UI */ }; -static CRITICAL_SECTION layout_list_section; -static CRITICAL_SECTION_DEBUG critsect_debug = -{ - 0, 0, &layout_list_section, - { &critsect_debug.ProcessLocksList, &critsect_debug.ProcessLocksList }, - 0, 0, { (DWORD_PTR)(__FILE__ ": layout_list_section") } -}; -static CRITICAL_SECTION layout_list_section = { &critsect_debug, -1, 0, 0, 0, 0 }; +static pthread_mutex_t layout_list_mutex = PTHREAD_MUTEX_INITIALIZER; int macdrv_layout_list_needs_update = TRUE; @@ -486,7 +479,7 @@ static HKL get_hkl(CFStringRef lang, CFStringRef type) /****************************************************************** * get_layout_from_source * - * Must be called while holding the layout_list_section. + * Must be called while holding the layout_list_mutex. * Note, returned layout may not currently be enabled. */ static struct layout *get_layout_from_source(TISInputSourceRef input) @@ -507,7 +500,7 @@ static struct layout *get_layout_from_source(TISInputSourceRef input) /*********************************************************************** * update_layout_list * - * Must be called while holding the layout_list_section + * Must be called while holding the layout_list_mutex * * If an input source has been disabled (ie. removed from the UI) its * entry remains in the layout list but is marked as such and is not @@ -563,13 +556,13 @@ HKL macdrv_get_hkl_from_source(TISInputSourceRef input) struct layout *layout; HKL ret = 0; - EnterCriticalSection(&layout_list_section); + pthread_mutex_lock(&layout_list_mutex); update_layout_list(); layout = get_layout_from_source(input); if (layout) ret = layout->hkl; - LeaveCriticalSection(&layout_list_section); + pthread_mutex_unlock(&layout_list_mutex); return ret; } @@ -1172,7 +1165,7 @@ BOOL macdrv_ActivateKeyboardLayout(HKL hkl, UINT flags) if (hkl == thread_data->active_keyboard_layout) return TRUE; - EnterCriticalSection(&layout_list_section); + pthread_mutex_lock(&layout_list_mutex); update_layout_list(); LIST_FOR_EACH_ENTRY(layout, &layout_list, struct layout, entry) @@ -1195,7 +1188,7 @@ BOOL macdrv_ActivateKeyboardLayout(HKL hkl, UINT flags) break; } } - LeaveCriticalSection(&layout_list_section); + pthread_mutex_unlock(&layout_list_mutex); return ret; } @@ -1306,7 +1299,7 @@ UINT macdrv_GetKeyboardLayoutList(INT size, HKL *list) TRACE("%d, %p\n", size, list); - EnterCriticalSection(&layout_list_section); + pthread_mutex_lock(&layout_list_mutex); update_layout_list(); @@ -1321,7 +1314,7 @@ UINT macdrv_GetKeyboardLayoutList(INT size, HKL *list) } count++; } - LeaveCriticalSection(&layout_list_section); + pthread_mutex_unlock(&layout_list_mutex); TRACE("returning %d\n", count); return count;

1 0

Jacek Caban : winemac: Use pthread for display locking.
by Alexandre Julliard May 18, 2022

May 18, 2022

Module: wine Branch: master Commit: f4b032b1d9c87e58533789b99478a32ff570fff0 URL: https://source.winehq.org/git/wine.git/?a=commit;h=f4b032b1d9c87e58533789b9… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 17 01:04:29 2022 +0200 winemac: Use pthread for display locking. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winemac.drv/display.c | 19 +++++-------------- 1 file changed, 5 insertions(+), 14 deletions(-) diff --git a/dlls/winemac.drv/display.c b/dlls/winemac.drv/display.c index c6b54c5fd82..4b6740b9274 100644 --- a/dlls/winemac.drv/display.c +++ b/dlls/winemac.drv/display.c @@ -53,14 +53,7 @@ static const WCHAR pixelencodingW[] = {'P','i','x','e','l','E','n','c','o','d',' static CFArrayRef modes; static BOOL modes_has_8bpp, modes_has_16bpp; static int default_mode_bpp; -static CRITICAL_SECTION modes_section; -static CRITICAL_SECTION_DEBUG critsect_debug = -{ - 0, 0, &modes_section, - { &critsect_debug.ProcessLocksList, &critsect_debug.ProcessLocksList }, - 0, 0, { (DWORD_PTR)(__FILE__ ": modes_section") } -}; -static CRITICAL_SECTION modes_section = { &critsect_debug, -1, 0, 0, 0, 0 }; +static pthread_mutex_t modes_mutex = PTHREAD_MUTEX_INITIALIZER; static BOOL inited_original_display_mode; @@ -515,8 +508,6 @@ static int get_default_bpp(void) { int ret; - EnterCriticalSection(&modes_section); - if (!default_mode_bpp) { CGDisplayModeRef mode = CGDisplayCopyDisplayMode(kCGDirectMainDisplay); @@ -532,8 +523,6 @@ static int get_default_bpp(void) ret = default_mode_bpp; - LeaveCriticalSection(&modes_section); - TRACE(" -> %d\n", ret); return ret; } @@ -895,7 +884,9 @@ LONG macdrv_ChangeDisplaySettingsEx(LPCWSTR devname, LPDEVMODEW devmode, return DISP_CHANGE_FAILED; } + pthread_mutex_lock(&modes_mutex); bpp = get_default_bpp(); + pthread_mutex_unlock(&modes_mutex); if ((devmode->dmFields & DM_BITSPERPEL) && devmode->dmBitsPerPel != bpp) TRACE("using default %d bpp instead of caller's request %d bpp\n", bpp, devmode->dmBitsPerPel); @@ -1088,7 +1079,7 @@ BOOL macdrv_EnumDisplaySettingsEx(LPCWSTR devname, DWORD mode, DEVMODEW *devmode { DWORD count, i; - EnterCriticalSection(&modes_section); + pthread_mutex_lock(&modes_mutex); if (mode == 0 || !modes) { @@ -1168,7 +1159,7 @@ BOOL macdrv_EnumDisplaySettingsEx(LPCWSTR devname, DWORD mode, DEVMODEW *devmode } } - LeaveCriticalSection(&modes_section); + pthread_mutex_unlock(&modes_mutex); } if (!display_mode)

1 0

Rémi Bernon : wmadmod: Remove unnecessary DLL.
by Alexandre Julliard May 18, 2022

May 18, 2022

Module: wine Branch: master Commit: 5387718c6fbd36b7a6309122f4af68d475a344e8 URL: https://source.winehq.org/git/wine.git/?a=commit;h=5387718c6fbd36b7a6309122… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Thu May 12 21:27:38 2022 +0200 wmadmod: Remove unnecessary DLL. It was initially meant for tests, which are all now in dlls/mf as well, and for the WMA decoder. The class is supposed to live in this DLL but because of the winegstreamer implementation, it cannot in Wine. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- configure | 3 - configure.ac | 2 - dlls/wmadmod/Makefile.in | 4 - dlls/wmadmod/main.c | 35 -------- dlls/wmadmod/tests/Makefile.in | 5 -- dlls/wmadmod/tests/wmadmod.c | 178 ----------------------------------------- dlls/wmadmod/wmadmod.spec | 5 -- 7 files changed, 232 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=5387718c6fbd36b7a630…

1 0

Nikolay Sivov : winedump: Cleanup minidump output helpers.
by Alexandre Julliard May 18, 2022

May 18, 2022

Module: wine Branch: master Commit: a9f9c1a696e587424ab26e5465c38b9ec0c266dd URL: https://source.winehq.org/git/wine.git/?a=commit;h=a9f9c1a696e587424ab26e54… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue May 17 22:20:36 2022 +0300 winedump: Cleanup minidump output helpers. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/winedump/minidump.c | 122 ++++++++++++++++++++++++---------------------- 1 file changed, 65 insertions(+), 57 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=a9f9c1a696e587424ab2…

1 0

Jacek Caban : winemac: Use pthread for GDI device locking.
by Alexandre Julliard May 18, 2022

May 18, 2022

Module: wine Branch: master Commit: e24c2fa33f3491a2fd0c98aec7bf2152f8240e19 URL: https://source.winehq.org/git/wine.git/?a=commit;h=e24c2fa33f3491a2fd0c98ae… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 17 01:09:18 2022 +0200 winemac: Use pthread for GDI device locking. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winemac.drv/gdi.c | 28 ++++++++++------------------ 1 file changed, 10 insertions(+), 18 deletions(-) diff --git a/dlls/winemac.drv/gdi.c b/dlls/winemac.drv/gdi.c index 264781c6ac1..673dca40270 100644 --- a/dlls/winemac.drv/gdi.c +++ b/dlls/winemac.drv/gdi.c @@ -47,15 +47,7 @@ static int device_data_valid; /* do the above variables have up-to-date values int retina_on = FALSE; -static CRITICAL_SECTION device_data_section; -static CRITICAL_SECTION_DEBUG critsect_debug = -{ - 0, 0, &device_data_section, - { &critsect_debug.ProcessLocksList, &critsect_debug.ProcessLocksList }, - 0, 0, { (DWORD_PTR)(__FILE__ ": device_data_section") } -}; -static CRITICAL_SECTION device_data_section = { &critsect_debug, -1, 0, 0, 0, 0 }; - +static pthread_mutex_t device_data_mutex = PTHREAD_MUTEX_INITIALIZER; static const struct user_driver_funcs macdrv_funcs; @@ -90,7 +82,7 @@ CGRect macdrv_get_desktop_rect(void) { CGRect ret; - EnterCriticalSection(&device_data_section); + pthread_mutex_lock(&device_data_mutex); if (!device_data_valid) { @@ -99,7 +91,7 @@ CGRect macdrv_get_desktop_rect(void) } ret = desktop_rect; - LeaveCriticalSection(&device_data_section); + pthread_mutex_unlock(&device_data_mutex); TRACE("%s\n", wine_dbgstr_cgrect(ret)); @@ -151,9 +143,9 @@ static void device_init(void) void macdrv_reset_device_metrics(void) { - EnterCriticalSection(&device_data_section); + pthread_mutex_lock(&device_data_mutex); device_data_valid = FALSE; - LeaveCriticalSection(&device_data_section); + pthread_mutex_unlock(&device_data_mutex); } @@ -161,9 +153,9 @@ static MACDRV_PDEVICE *create_mac_physdev(void) { MACDRV_PDEVICE *physDev; - EnterCriticalSection(&device_data_section); + pthread_mutex_lock(&device_data_mutex); if (!device_data_valid) device_init(); - LeaveCriticalSection(&device_data_section); + pthread_mutex_unlock(&device_data_mutex); if (!(physDev = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*physDev)))) return NULL; @@ -227,7 +219,7 @@ static INT CDECL macdrv_GetDeviceCaps(PHYSDEV dev, INT cap) { INT ret; - EnterCriticalSection(&device_data_section); + pthread_mutex_lock(&device_data_mutex); if (!device_data_valid) device_init(); @@ -245,7 +237,7 @@ static INT CDECL macdrv_GetDeviceCaps(PHYSDEV dev, INT cap) case HORZRES: case VERTRES: default: - LeaveCriticalSection(&device_data_section); + pthread_mutex_unlock(&device_data_mutex); dev = GET_NEXT_PHYSDEV( dev, pGetDeviceCaps ); ret = dev->funcs->pGetDeviceCaps( dev, cap ); if ((cap == HORZRES || cap == VERTRES) && retina_on) @@ -255,7 +247,7 @@ static INT CDECL macdrv_GetDeviceCaps(PHYSDEV dev, INT cap) TRACE("cap %d -> %d\n", cap, ret); - LeaveCriticalSection(&device_data_section); + pthread_mutex_unlock(&device_data_mutex); return ret; }

1 0

Jacek Caban : winemac: Use pthread for cursor cache locking.
by Alexandre Julliard May 18, 2022

May 18, 2022

Module: wine Branch: master Commit: 3ab98869bb983bc02aafcaead0b802d4ae585697 URL: https://source.winehq.org/git/wine.git/?a=commit;h=3ab98869bb983bc02aafcaea… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 17 01:16:59 2022 +0200 winemac: Use pthread for cursor cache locking. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winemac.drv/mouse.c | 22 +++++++--------------- 1 file changed, 7 insertions(+), 15 deletions(-) diff --git a/dlls/winemac.drv/mouse.c b/dlls/winemac.drv/mouse.c index 55bed52ae8e..d101d2513c8 100644 --- a/dlls/winemac.drv/mouse.c +++ b/dlls/winemac.drv/mouse.c @@ -32,15 +32,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(cursor); -static CRITICAL_SECTION cursor_cache_section; -static CRITICAL_SECTION_DEBUG critsect_debug = -{ - 0, 0, &cursor_cache_section, - { &critsect_debug.ProcessLocksList, &critsect_debug.ProcessLocksList }, - 0, 0, { (DWORD_PTR)(__FILE__ ": cursor_cache_section") } -}; -static CRITICAL_SECTION cursor_cache_section = { &critsect_debug, -1, 0, 0, 0, 0 }; - +static pthread_mutex_t cursor_cache_mutex = PTHREAD_MUTEX_INITIALIZER; static CFMutableDictionaryRef cursor_cache; @@ -647,10 +639,10 @@ void macdrv_DestroyCursorIcon(HCURSOR cursor) { TRACE("cursor %p\n", cursor); - EnterCriticalSection(&cursor_cache_section); + pthread_mutex_lock(&cursor_cache_mutex); if (cursor_cache) CFDictionaryRemoveValue(cursor_cache, cursor); - LeaveCriticalSection(&cursor_cache_section); + pthread_mutex_unlock(&cursor_cache_mutex); } @@ -731,7 +723,7 @@ void macdrv_SetCursor(HCURSOR cursor) { ICONINFOEXW info; - EnterCriticalSection(&cursor_cache_section); + pthread_mutex_lock(&cursor_cache_mutex); if (cursor_cache) { CFTypeRef cached_cursor = CFDictionaryGetValue(cursor_cache, cursor); @@ -743,7 +735,7 @@ void macdrv_SetCursor(HCURSOR cursor) cursor_frames = CFRetain(cached_cursor); } } - LeaveCriticalSection(&cursor_cache_section); + pthread_mutex_unlock(&cursor_cache_mutex); if (cursor_name || cursor_frames) goto done; @@ -790,13 +782,13 @@ void macdrv_SetCursor(HCURSOR cursor) if (cursor_name || cursor_frames) { - EnterCriticalSection(&cursor_cache_section); + pthread_mutex_lock(&cursor_cache_mutex); if (!cursor_cache) cursor_cache = CFDictionaryCreateMutable(NULL, 0, NULL, &kCFTypeDictionaryValueCallBacks); CFDictionarySetValue(cursor_cache, cursor, cursor_name ? (CFTypeRef)cursor_name : (CFTypeRef)cursor_frames); - LeaveCriticalSection(&cursor_cache_section); + pthread_mutex_unlock(&cursor_cache_mutex); } else cursor_name = CFRetain(CFSTR("arrowCursor"));

1 0

Rémi Bernon : ntdll: Simplify the fallback case in heap_reallocate.
by Alexandre Julliard May 18, 2022

May 18, 2022

Module: wine Branch: master Commit: 419e4730ba81cb171eba6bc8920d940183d9597d URL: https://source.winehq.org/git/wine.git/?a=commit;h=419e4730ba81cb171eba6bc8… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Wed May 18 09:34:18 2022 +0200 ntdll: Simplify the fallback case in heap_reallocate. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/heap.c | 45 ++++++++------------------------------------- 1 file changed, 8 insertions(+), 37 deletions(-) diff --git a/dlls/ntdll/heap.c b/dlls/ntdll/heap.c index fa9f617f482..eb45a94da95 100644 --- a/dlls/ntdll/heap.c +++ b/dlls/ntdll/heap.c @@ -1663,9 +1663,11 @@ BOOLEAN WINAPI DECLSPEC_HOTPATCH RtlFreeHeap( HANDLE heap, ULONG flags, void *pt static NTSTATUS heap_reallocate( HEAP *heap, ULONG flags, void *ptr, SIZE_T size, void **ret ) { + struct block *next; ARENA_INUSE *pArena; SUBHEAP *subheap; SIZE_T oldBlockSize, oldActualSize, rounded_size; + NTSTATUS status; rounded_size = ROUND_SIZE(size) + HEAP_TAIL_EXTRA_SIZE(flags); if (rounded_size < size) return STATUS_NO_MEMORY; /* overflow */ @@ -1684,20 +1686,8 @@ static NTSTATUS heap_reallocate( HEAP *heap, ULONG flags, void *ptr, SIZE_T size oldActualSize = (pArena->size & ARENA_SIZE_MASK) - pArena->unused_bytes; if (rounded_size > oldBlockSize) { - struct block *next; - - if (rounded_size >= HEAP_MIN_LARGE_BLOCK_SIZE) - { - if (flags & HEAP_REALLOC_IN_PLACE_ONLY) return STATUS_NO_MEMORY; - if (!(*ret = allocate_large_block( heap, flags, size ))) return STATUS_NO_MEMORY; - memcpy( *ret, pArena + 1, oldActualSize ); - notify_free( pArena + 1 ); - HEAP_MakeInUseBlockFree( subheap, pArena ); - return STATUS_SUCCESS; - } - if ((next = next_block( subheap, pArena )) && (block_get_flags( next ) & ARENA_FLAG_FREE) && - (oldBlockSize + block_get_size( next ) >= rounded_size)) + rounded_size < HEAP_MIN_LARGE_BLOCK_SIZE && rounded_size <= oldBlockSize + block_get_size( next )) { /* The next block is free and large enough */ struct entry *entry = (struct entry *)next; @@ -1707,34 +1697,15 @@ static NTSTATUS heap_reallocate( HEAP *heap, ULONG flags, void *ptr, SIZE_T size notify_realloc( pArena + 1, oldActualSize, size ); HEAP_ShrinkBlock( subheap, pArena, rounded_size ); } - else /* Do it the hard way */ + else { - ARENA_FREE *pNew; - ARENA_INUSE *pInUse; - SUBHEAP *newsubheap; - if (flags & HEAP_REALLOC_IN_PLACE_ONLY) return STATUS_NO_MEMORY; - if (!(pNew = HEAP_FindFreeBlock( heap, rounded_size, &newsubheap ))) return STATUS_NO_MEMORY; - - /* Build the in-use arena */ - - list_remove( &pNew->entry ); - pInUse = (ARENA_INUSE *)pNew; - pInUse->size = (pInUse->size & ~ARENA_FLAG_FREE) - + sizeof(ARENA_FREE) - sizeof(ARENA_INUSE); - pInUse->magic = ARENA_INUSE_MAGIC; - HEAP_ShrinkBlock( newsubheap, pInUse, rounded_size ); - - mark_block_initialized( pInUse + 1, oldActualSize ); - notify_alloc( pInUse + 1, size, FALSE ); - memcpy( pInUse + 1, pArena + 1, oldActualSize ); - - /* Free the previous block */ - + if ((status = heap_allocate( heap, flags & ~HEAP_ZERO_MEMORY, size, ret ))) return status; + memcpy( *ret, pArena + 1, oldActualSize ); + if (flags & HEAP_ZERO_MEMORY) memset( (char *)*ret + oldActualSize, 0, size - oldActualSize ); notify_free( pArena + 1 ); HEAP_MakeInUseBlockFree( subheap, pArena ); - subheap = newsubheap; - pArena = pInUse; + return STATUS_SUCCESS; } } else

1 0