November 2022 - Wine-Commits

Torge Matthies : ntdll: Fix incorrectly sized mov in user_mode_callback_return.
by Alexandre Julliard Nov. 9, 2022

Nov. 9, 2022

Module: wine Branch: master Commit: efc45c5e3b49932989b9d061d0a60f504ec7fa6e URL: https://gitlab.winehq.org/wine/wine/-/commit/efc45c5e3b49932989b9d061d0a60f… Author: Torge Matthies <tmatthies(a)codeweavers.com> Date: Wed Nov 9 08:06:28 2022 +0100 ntdll: Fix incorrectly sized mov in user_mode_callback_return. Signed-off-by: Torge Matthies <tmatthies(a)codeweavers.com> --- dlls/ntdll/unix/signal_x86_64.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/ntdll/unix/signal_x86_64.c b/dlls/ntdll/unix/signal_x86_64.c index 58029602130..263bb64c8cc 100644 --- a/dlls/ntdll/unix/signal_x86_64.c +++ b/dlls/ntdll/unix/signal_x86_64.c @@ -1647,7 +1647,7 @@ __ASM_GLOBAL_FUNC( user_mode_callback_return, "movq 0x10(%rbp),%rsi\n\t" /* ret_ptr */ "movq 0x18(%rbp),%rdi\n\t" /* ret_len */ "movq %rcx,(%rsi)\n\t" - "movq %rdx,(%rdi)\n\t" + "movl %edx,(%rdi)\n\t" "movdqa -0xe0(%rbp),%xmm15\n\t" "movdqa -0xd0(%rbp),%xmm14\n\t" "movdqa -0xc0(%rbp),%xmm13\n\t"

1 0

Zebediah Figura : d3d8: Build without -DWINE_NO_LONG_TYPES.
by Alexandre Julliard Nov. 9, 2022

Nov. 9, 2022

Module: wine Branch: master Commit: 52c35814bece03b8668a699b87d856af19ba94c3 URL: https://gitlab.winehq.org/wine/wine/-/commit/52c35814bece03b8668a699b87d856… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Sat Sep 17 23:32:00 2022 -0500 d3d8: Build without -DWINE_NO_LONG_TYPES. --- dlls/d3d8/Makefile.in | 1 - dlls/d3d8/buffer.c | 24 +++--- dlls/d3d8/device.c | 186 +++++++++++++++++++++--------------------- dlls/d3d8/directx.c | 16 ++-- dlls/d3d8/shader.c | 12 +-- dlls/d3d8/surface.c | 10 +-- dlls/d3d8/swapchain.c | 8 +- dlls/d3d8/texture.c | 34 ++++---- dlls/d3d8/vertexdeclaration.c | 28 +++---- dlls/d3d8/volume.c | 5 +- 10 files changed, 159 insertions(+), 165 deletions(-)

1 0

Zebediah Figura : d3d9: Build without -DWINE_NO_LONG_TYPES.
by Alexandre Julliard Nov. 9, 2022

Nov. 9, 2022

Module: wine Branch: master Commit: 9b3742bebb52252d9a3bfec35d17808dda2b641b URL: https://gitlab.winehq.org/wine/wine/-/commit/9b3742bebb52252d9a3bfec35d1780… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Sun Sep 18 22:57:13 2022 -0500 d3d9: Build without -DWINE_NO_LONG_TYPES. --- dlls/d3d9/Makefile.in | 1 - dlls/d3d9/buffer.c | 24 +++---- dlls/d3d9/d3d9_main.c | 12 ++-- dlls/d3d9/device.c | 142 +++++++++++++++++++++--------------------- dlls/d3d9/directx.c | 24 +++---- dlls/d3d9/query.c | 11 ++-- dlls/d3d9/shader.c | 12 ++-- dlls/d3d9/stateblock.c | 6 +- dlls/d3d9/surface.c | 12 ++-- dlls/d3d9/swapchain.c | 10 +-- dlls/d3d9/texture.c | 36 +++++------ dlls/d3d9/vertexdeclaration.c | 12 ++-- dlls/d3d9/volume.c | 4 +- 13 files changed, 152 insertions(+), 154 deletions(-)

1 0

Zebediah Figura : d3d9/tests: Test drawing from a managed texture after Reset().
by Alexandre Julliard Nov. 9, 2022

Nov. 9, 2022

Module: wine Branch: master Commit: e4cd936679d9b9e505d600c68f3ac06201c60f08 URL: https://gitlab.winehq.org/wine/wine/-/commit/e4cd936679d9b9e505d600c68f3ac0… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Thu Oct 20 20:04:29 2022 -0500 d3d9/tests: Test drawing from a managed texture after Reset(). --- dlls/d3d9/tests/visual.c | 153 ++++++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 150 insertions(+), 3 deletions(-) diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c index d8fa298d6c7..fc30090562f 100644 --- a/dlls/d3d9/tests/visual.c +++ b/dlls/d3d9/tests/visual.c @@ -28,6 +28,7 @@ * causes visible results in games can be tested in a way that does not depend on pixel exactness */ +#include <stdbool.h> #include <limits.h> #include <math.h> @@ -51,6 +52,14 @@ struct vec4 float x, y, z, w; }; +struct d3d9_test_context +{ + HWND window; + IDirect3D9 *d3d; + IDirect3DDevice9 *device; + IDirect3DSurface9 *backbuffer; +}; + static HWND create_window(void) { HWND hwnd; @@ -249,8 +258,9 @@ static DWORD getPixelColor(IDirect3DDevice9 *device, UINT x, UINT y) return ret; } -#define check_rt_color(a, b) check_rt_color_(__LINE__, a, b) -static void check_rt_color_(unsigned int line, IDirect3DSurface9 *rt, D3DCOLOR expected_color) +#define check_rt_color(a, b) check_rt_color_(__LINE__, a, b, false) +#define check_rt_color_todo(a, b) check_rt_color_(__LINE__, a, b, true) +static void check_rt_color_(unsigned int line, IDirect3DSurface9 *rt, D3DCOLOR expected_color, bool todo) { unsigned int color = 0xdeadbeef; struct surface_readback rb; @@ -274,7 +284,8 @@ static void check_rt_color_(unsigned int line, IDirect3DSurface9 *rt, D3DCOLOR e break; } release_surface_readback(&rb); - ok_(__FILE__, line)(color == expected_color, "Got unexpected color 0x%08x.\n", color); + todo_wine_if (todo) + ok_(__FILE__, line)(color == expected_color, "Got unexpected color 0x%08x.\n", color); } static IDirect3DDevice9 *create_device(IDirect3D9 *d3d, HWND device_window, HWND focus_window, BOOL windowed) @@ -315,6 +326,101 @@ static void cleanup_device(IDirect3DDevice9 *device) } } +static bool init_test_context(struct d3d9_test_context *context) +{ + HRESULT hr; + + memset(context, 0, sizeof(*context)); + + context->window = create_window(); + context->d3d = Direct3DCreate9(D3D_SDK_VERSION); + ok(!!context->d3d, "Failed to create a D3D object.\n"); + if (!(context->device = create_device(context->d3d, context->window, context->window, TRUE))) + { + skip("Failed to create a D3D device.\n"); + IDirect3D9_Release(context->d3d); + DestroyWindow(context->window); + return false; + } + + hr = IDirect3DDevice9_GetRenderTarget(context->device, 0, &context->backbuffer); + ok(hr == S_OK, "Failed to get backbuffer, hr %#lx.\n", hr); + + return true; +} + +#define release_test_context(a) release_test_context_(__LINE__, a) +static void release_test_context_(unsigned int line, struct d3d9_test_context *context) +{ + ULONG refcount; + + IDirect3DSurface9_Release(context->backbuffer); + refcount = IDirect3DDevice9_Release(context->device); + ok(!refcount, "Device has %lu references left.\n", refcount); + refcount = IDirect3D9_Release(context->d3d); + ok(!refcount, "D3D object has %lu references left.\n", refcount); + DestroyWindow(context->window); +} + +static void draw_textured_quad(struct d3d9_test_context *context, IDirect3DTexture9 *texture) +{ + IDirect3DDevice9 *device = context->device; + HRESULT hr; + + static const struct + { + struct vec3 position; + struct vec2 texcoord; + } + quad[] = + { + {{-1.0f, -1.0f, 0.0f}, {0.0f, 0.0f}}, + {{-1.0f, 1.0f, 0.0f}, {0.0f, 1.0f}}, + {{ 1.0f, -1.0f, 0.0f}, {1.0f, 0.0f}}, + {{ 1.0f, 1.0f, 0.0f}, {1.0f, 1.0f}}, + }; + + hr = IDirect3DDevice9_SetRenderState(device, D3DRS_ZENABLE, FALSE); + ok(hr == S_OK, "Got hr %#lx.\n", hr); + hr = IDirect3DDevice9_SetFVF(device, D3DFVF_XYZ | D3DFVF_TEX1); + ok(hr == S_OK, "Got hr %#lx.\n", hr); + hr = IDirect3DDevice9_SetTextureStageState(device, 0, D3DTSS_COLOROP, D3DTOP_SELECTARG1); + ok(hr == S_OK, "Got hr %#lx.\n", hr); + hr = IDirect3DDevice9_SetTextureStageState(device, 0, D3DTSS_COLORARG1, D3DTA_TEXTURE); + ok(hr == S_OK, "Got hr %#lx.\n", hr); + hr = IDirect3DDevice9_SetTexture(device, 0, (IDirect3DBaseTexture9 *)texture); + ok(hr == S_OK, "Got hr %#lx.\n", hr); + hr = IDirect3DDevice9_BeginScene(device); + ok(hr == S_OK, "Got hr %#lx.\n", hr); + hr = IDirect3DDevice9_DrawPrimitiveUP(device, D3DPT_TRIANGLESTRIP, 2, &quad, sizeof(*quad)); + ok(hr == S_OK, "Got hr %#lx.\n", hr); + hr = IDirect3DDevice9_EndScene(device); + ok(hr == S_OK, "Got hr %#lx.\n", hr); +} + +static HRESULT reset_device(struct d3d9_test_context *context) +{ + D3DPRESENT_PARAMETERS present_parameters = {0}; + HRESULT hr; + + IDirect3DSurface9_Release(context->backbuffer); + + present_parameters.BackBufferWidth = 640; + present_parameters.BackBufferHeight = 480; + present_parameters.BackBufferFormat = D3DFMT_A8R8G8B8; + present_parameters.SwapEffect = D3DSWAPEFFECT_DISCARD; + present_parameters.hDeviceWindow = context->window; + present_parameters.Windowed = TRUE; + present_parameters.EnableAutoDepthStencil = TRUE; + present_parameters.AutoDepthStencilFormat = D3DFMT_D24S8; + hr = IDirect3DDevice9_Reset(context->device, &present_parameters); + + if (SUCCEEDED(hr)) + IDirect3DDevice9_GetRenderTarget(context->device, 0, &context->backbuffer); + + return hr; +} + static void test_sanity(void) { IDirect3DDevice9 *device; @@ -3901,6 +4007,15 @@ static void fill_surface(IDirect3DSurface9 *surface, DWORD color, DWORD flags) ok(hr == S_OK, "Got hr %#lx.\n", hr); } +static void fill_texture(IDirect3DTexture9 *texture, DWORD color, DWORD flags) +{ + IDirect3DSurface9 *surface; + + IDirect3DTexture9_GetSurfaceLevel(texture, 0, &surface); + fill_surface(surface, color, flags); + IDirect3DSurface9_Release(surface); +} + static void stretchrect_test(void) { enum surface_type @@ -27723,6 +27838,37 @@ static void test_filling_convention(void) DestroyWindow(window); } +static void test_managed_reset(void) +{ + struct d3d9_test_context context; + IDirect3DTexture9 *texture; + IDirect3DDevice9 *device; + HRESULT hr; + + if (!init_test_context(&context)) + return; + device = context.device; + + hr = IDirect3DDevice9_CreateTexture(device, 256, 256, 1, 0, D3DFMT_A8R8G8B8, D3DPOOL_MANAGED, &texture, NULL); + ok(hr == S_OK, "Got hr %#lx.\n", hr); + fill_texture(texture, 0x0000ff00, 0); + + draw_textured_quad(&context, texture); + check_rt_color(context.backbuffer, 0x0000ff00); + + hr = reset_device(&context); + ok(hr == S_OK, "Got hr %#lx.\n", hr); + + hr = IDirect3DDevice9_Clear(device, 0, NULL, D3DCLEAR_TARGET, 0xffff0000, 0.0, 0); + ok(hr == S_OK, "Got hr %#lx.\n", hr); + + draw_textured_quad(&context, texture); + check_rt_color_todo(context.backbuffer, 0x0000ff00); + + IDirect3DTexture9_Release(texture); + release_test_context(&context); +} + START_TEST(visual) { D3DADAPTER_IDENTIFIER9 identifier; @@ -27874,4 +28020,5 @@ START_TEST(visual) test_sample_mask(); test_dynamic_map_synchronization(); test_filling_convention(); + test_managed_reset(); }

1 0

Zebediah Figura : wined3d: Add a dirty rect to managed textures when resetting.
by Alexandre Julliard Nov. 9, 2022

Nov. 9, 2022

Module: wine Branch: master Commit: 9b62f5e57fa8da338f8c71c0251089721f075f36 URL: https://gitlab.winehq.org/wine/wine/-/commit/9b62f5e57fa8da338f8c71c0251089… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Thu Oct 20 20:20:13 2022 -0500 wined3d: Add a dirty rect to managed textures when resetting. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53798 --- dlls/d3d8/tests/visual.c | 2 +- dlls/d3d9/tests/visual.c | 2 +- dlls/wined3d/device.c | 3 +++ 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/dlls/d3d8/tests/visual.c b/dlls/d3d8/tests/visual.c index 05a1fc38844..434474b292d 100644 --- a/dlls/d3d8/tests/visual.c +++ b/dlls/d3d8/tests/visual.c @@ -12119,7 +12119,7 @@ static void test_managed_reset(void) ok(hr == S_OK, "Got hr %#lx.\n", hr); draw_textured_quad(&context, texture); - check_rt_color_todo(context.backbuffer, 0x0000ff00); + check_rt_color(context.backbuffer, 0x0000ff00); IDirect3DTexture8_Release(texture); release_test_context(&context); diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c index fc30090562f..f8375d238e4 100644 --- a/dlls/d3d9/tests/visual.c +++ b/dlls/d3d9/tests/visual.c @@ -27863,7 +27863,7 @@ static void test_managed_reset(void) ok(hr == S_OK, "Got hr %#lx.\n", hr); draw_textured_quad(&context, texture); - check_rt_color_todo(context.backbuffer, 0x0000ff00); + check_rt_color(context.backbuffer, 0x0000ff00); IDirect3DTexture9_Release(texture); release_test_context(&context); diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index 6cf7d8a35a3..913376ee3ee 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -5829,6 +5829,9 @@ HRESULT CDECL wined3d_device_reset(struct wined3d_device *device, { TRACE("Unloading resource %p.\n", resource); wined3d_cs_emit_unload_resource(device->cs, resource); + + if (resource->usage & WINED3DUSAGE_MANAGED) + mark_managed_resource_dirty(resource); } device->adapter->adapter_ops->adapter_uninit_3d(device);

1 0

Zebediah Figura : wined3d: Always invalidate ~map_binding in texture write maps.
by Alexandre Julliard Nov. 9, 2022

Nov. 9, 2022

Module: wine Branch: master Commit: 11f00e8457fcc9b8a9b97de1ea7b29b37c93bccf URL: https://gitlab.winehq.org/wine/wine/-/commit/11f00e8457fcc9b8a9b97de1ea7b29… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Thu Oct 20 20:22:45 2022 -0500 wined3d: Always invalidate ~map_binding in texture write maps. Regardless of whether WINED3D_MAP_NO_DIRTY_UPDATE is specified. Dirty rects are independent of resource location management. --- dlls/wined3d/texture.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/dlls/wined3d/texture.c b/dlls/wined3d/texture.c index f385394497b..c9d5193e291 100644 --- a/dlls/wined3d/texture.c +++ b/dlls/wined3d/texture.c @@ -3748,8 +3748,7 @@ static HRESULT texture_resource_sub_resource_map(struct wined3d_resource *resour && !(flags & WINED3D_MAP_NO_DIRTY_UPDATE) && !texture_level) wined3d_texture_dirty_region_add(texture, sub_resource_idx / texture->level_count, box); - if (flags & WINED3D_MAP_WRITE - && (!(flags & WINED3D_MAP_NO_DIRTY_UPDATE) || (resource->usage & WINED3DUSAGE_DYNAMIC))) + if (flags & WINED3D_MAP_WRITE) wined3d_texture_invalidate_location(texture, sub_resource_idx, ~resource->map_binding); wined3d_texture_get_bo_address(texture, sub_resource_idx, &data, resource->map_binding);

1 0

Zebediah Figura : wined3d: Factor out a mark_managed_resource_dirty() helper.
by Alexandre Julliard Nov. 9, 2022

Nov. 9, 2022

Module: wine Branch: master Commit: df70d0991208d058f693423c2ce6696aa2b53692 URL: https://gitlab.winehq.org/wine/wine/-/commit/df70d0991208d058f693423c2ce669… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Thu Oct 20 20:05:11 2022 -0500 wined3d: Factor out a mark_managed_resource_dirty() helper. --- dlls/wined3d/device.c | 27 ++++++++++++++++----------- 1 file changed, 16 insertions(+), 11 deletions(-) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index c314292031d..6cf7d8a35a3 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -5509,6 +5509,21 @@ BOOL CDECL wined3d_device_show_cursor(struct wined3d_device *device, BOOL show) return oldVisible; } +static void mark_managed_resource_dirty(struct wined3d_resource *resource) +{ + if (resource->type != WINED3D_RTYPE_BUFFER) + { + struct wined3d_texture *texture = texture_from_resource(resource); + unsigned int i; + + if (texture->dirty_regions) + { + for (i = 0; i < texture->layer_count; ++i) + wined3d_texture_add_dirty_region(texture, i, NULL); + } + } +} + void CDECL wined3d_device_evict_managed_resources(struct wined3d_device *device) { struct wined3d_resource *resource, *cursor; @@ -5527,17 +5542,7 @@ void CDECL wined3d_device_evict_managed_resources(struct wined3d_device *device) wined3d_cs_emit_unload_resource(device->cs, resource); } - if (resource->type != WINED3D_RTYPE_BUFFER) - { - struct wined3d_texture *texture = texture_from_resource(resource); - unsigned int i; - - if (texture->dirty_regions) - { - for (i = 0; i < texture->layer_count; ++i) - wined3d_texture_add_dirty_region(texture, i, NULL); - } - } + mark_managed_resource_dirty(resource); } } }

1 0

Eric Pouech : dbghelp: Introduce a helper for module lookup.
by Alexandre Julliard Nov. 9, 2022

Nov. 9, 2022

Module: wine Branch: master Commit: 31e76b533326b55ae616f6bb1bf4e38e9ffb00f5 URL: https://gitlab.winehq.org/wine/wine/-/commit/31e76b533326b55ae616f6bb1bf4e3… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Wed Nov 9 11:17:42 2022 +0100 dbghelp: Introduce a helper for module lookup. Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> --- dlls/dbghelp/path.c | 55 ++++++++++++++++++----------------------------------- 1 file changed, 18 insertions(+), 37 deletions(-) diff --git a/dlls/dbghelp/path.c b/dlls/dbghelp/path.c index 5b4172e4001..903087695aa 100644 --- a/dlls/dbghelp/path.c +++ b/dlls/dbghelp/path.c @@ -705,14 +705,24 @@ WCHAR *get_dos_file_name(const WCHAR *filename) return dos_path; } +static BOOL try_match_file(const WCHAR *name, BOOL (*match)(void*, HANDLE, const WCHAR*), void *param) +{ + HANDLE file = CreateFileW(name, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL); + if (file != INVALID_HANDLE_VALUE) + { + BOOL ret = match(param, file, name); + CloseHandle(file); + return ret; + } + return FALSE; +} + BOOL search_dll_path(const struct process *process, const WCHAR *name, BOOL (*match)(void*, HANDLE, const WCHAR*), void *param) { const WCHAR *env; WCHAR *p, *end; size_t len, i; - HANDLE file; WCHAR *buf; - BOOL ret; name = file_name(name); @@ -733,13 +743,7 @@ BOOL search_dll_path(const struct process *process, const WCHAR *name, BOOL (*ma p = end + lstrlenW(end); *p++ = '\\'; lstrcpyW(p, name); - file = CreateFileW(buf, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL); - if (file != INVALID_HANDLE_VALUE) - { - ret = match(param, file, buf); - CloseHandle(file); - if (ret) goto found; - } + if (try_match_file(buf, match, param)) goto found; memcpy(end, programsW, sizeof(programsW)); end += ARRAY_SIZE(programsW); @@ -749,13 +753,7 @@ BOOL search_dll_path(const struct process *process, const WCHAR *name, BOOL (*ma p = end + lstrlenW(end); *p++ = '\\'; lstrcpyW(p, name); - file = CreateFileW(buf, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL); - if (file != INVALID_HANDLE_VALUE) - { - ret = match(param, file, buf); - CloseHandle(file); - if (ret) goto found; - } + if (try_match_file(buf, match, param)) goto found; heap_free(buf); } @@ -771,21 +769,9 @@ BOOL search_dll_path(const struct process *process, const WCHAR *name, BOOL (*ma swprintf(buf, len, L"%s%s\\%s", env, so_dir, name); else swprintf(buf, len, L"%s%s\\%s", env, pe_dir, name); - file = CreateFileW(buf, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL); - if (file != INVALID_HANDLE_VALUE) - { - ret = match(param, file, buf); - CloseHandle(file); - if (ret) goto found; - } + if (try_match_file(buf, match, param)) goto found; swprintf(buf, len, L"%s\\%s", env, name); - file = CreateFileW(buf, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL); - if (file != INVALID_HANDLE_VALUE) - { - ret = match(param, file, buf); - CloseHandle(file); - if (ret) goto found; - } + if (try_match_file(buf, match, param)) goto found; heap_free(buf); } @@ -822,13 +808,8 @@ BOOL search_unix_path(const WCHAR *name, const WCHAR *path, BOOL (*match)(void*, WideCharToMultiByte(CP_UNIXCP, 0, name, -1, buf + len, size - len, NULL, NULL); if ((dos_path = wine_get_dos_file_name(buf))) { - HANDLE file = CreateFileW(dos_path, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL); - if (file != INVALID_HANDLE_VALUE) - { - ret = match(param, file, dos_path); - CloseHandle(file); - if (ret) TRACE("found %s\n", debugstr_w(dos_path)); - } + ret = try_match_file(dos_path, match, param); + if (ret) TRACE("found %s\n", debugstr_w(dos_path)); heap_free(dos_path); if (ret) break; }

1 0

Zebediah Figura : wined3d: Set the window state thread description only if it's actually a new thread.
by Alexandre Julliard Nov. 9, 2022

Nov. 9, 2022

Module: wine Branch: master Commit: efcb73441428705cdd49bf41bd396c4e829ac5c8 URL: https://gitlab.winehq.org/wine/wine/-/commit/efcb73441428705cdd49bf41bd396c… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Wed Oct 19 12:35:35 2022 -0500 wined3d: Set the window state thread description only if it's actually a new thread. Fixes: a2c03e23565779d2743f5750c37de0be3f7222bf --- dlls/wined3d/swapchain.c | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/dlls/wined3d/swapchain.c b/dlls/wined3d/swapchain.c index 4b420a749f3..346fd1e6497 100644 --- a/dlls/wined3d/swapchain.c +++ b/dlls/wined3d/swapchain.c @@ -2200,8 +2200,6 @@ static DWORD WINAPI set_window_state_thread(void *ctx) struct wined3d_window_state *s = ctx; bool filter; - SetThreadDescription(GetCurrentThread(), L"wined3d_set_window_state"); - filter = wined3d_filter_messages(s->window, TRUE); if (s->set_style) @@ -2229,11 +2227,18 @@ static void set_window_state(struct wined3d_window_state *s) * position can potentially deadlock if that thread isn't processing * messages. */ if (window_tid == tid) + { set_window_state_thread(s); - else if (!(thread = CreateThread(NULL, 0, set_window_state_thread, s, 0, NULL))) - ERR("Failed to create thread.\n"); - else + } + else if ((thread = CreateThread(NULL, 0, set_window_state_thread, s, 0, NULL))) + { + SetThreadDescription(thread, L"wined3d_set_window_state"); CloseHandle(thread); + } + else + { + ERR("Failed to create thread.\n"); + } } HRESULT wined3d_swapchain_state_setup_fullscreen(struct wined3d_swapchain_state *state,

1 0

Eric Pouech : dbghelp: Support loading modules from Wine's multi-arch build tree.
by Alexandre Julliard Nov. 9, 2022

Nov. 9, 2022

Module: wine Branch: master Commit: 1ed24f7ec624e7c3bacfbd166770b3c94635d728 URL: https://gitlab.winehq.org/wine/wine/-/commit/1ed24f7ec624e7c3bacfbd166770b3… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Wed Nov 9 11:17:43 2022 +0100 dbghelp: Support loading modules from Wine's multi-arch build tree. Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> --- dlls/dbghelp/dbghelp.c | 10 +++++++ dlls/dbghelp/dbghelp_private.h | 1 + dlls/dbghelp/path.c | 67 +++++++++++++++++++++++++++--------------- 3 files changed, 55 insertions(+), 23 deletions(-) diff --git a/dlls/dbghelp/dbghelp.c b/dlls/dbghelp/dbghelp.c index 32ce7e13865..2facc58585e 100644 --- a/dlls/dbghelp/dbghelp.c +++ b/dlls/dbghelp/dbghelp.c @@ -332,6 +332,16 @@ const WCHAR *process_getenv(const struct process *process, const WCHAR *name) return NULL; } +const struct cpu* process_get_cpu(const struct process* pcs) +{ + const struct module* m = pcs->lmodules; + + /* main module is the last one in list */ + if (!m) return dbghelp_current_cpu; + while (m->next) m = m->next; + return m->cpu; +} + /****************************************************************** * check_live_target * diff --git a/dlls/dbghelp/dbghelp_private.h b/dlls/dbghelp/dbghelp_private.h index 01163c0e98a..22e9eec8805 100644 --- a/dlls/dbghelp/dbghelp_private.h +++ b/dlls/dbghelp/dbghelp_private.h @@ -696,6 +696,7 @@ extern void* fetch_buffer(struct process* pcs, unsigned size) DECLSPEC_HI extern const char* wine_dbgstr_addr(const ADDRESS64* addr) DECLSPEC_HIDDEN; extern struct cpu* cpu_find(DWORD) DECLSPEC_HIDDEN; extern const WCHAR *process_getenv(const struct process *process, const WCHAR *name) DECLSPEC_HIDDEN; +extern const struct cpu* process_get_cpu(const struct process* pcs) DECLSPEC_HIDDEN; extern DWORD calc_crc32(HANDLE handle) DECLSPEC_HIDDEN; /* elf_module.c */ diff --git a/dlls/dbghelp/path.c b/dlls/dbghelp/path.c index 903087695aa..2320c9da5c3 100644 --- a/dlls/dbghelp/path.c +++ b/dlls/dbghelp/path.c @@ -31,22 +31,19 @@ WINE_DEFAULT_DEBUG_CHANNEL(dbghelp); -#ifdef __i386__ -static const WCHAR pe_dir[] = L"\\i386-windows"; -static const WCHAR so_dir[] = L"\\i386-unix"; -#elif defined __x86_64__ -static const WCHAR pe_dir[] = L"\\x86_64-windows"; -static const WCHAR so_dir[] = L"\\x86_64-unix"; -#elif defined __arm__ -static const WCHAR pe_dir[] = L"\\arm-windows"; -static const WCHAR so_dir[] = L"\\arm-unix"; -#elif defined __aarch64__ -static const WCHAR pe_dir[] = L"\\aarch64-windows"; -static const WCHAR so_dir[] = L"\\aarch64-unix"; -#else -static const WCHAR pe_dir[] = L""; -static const WCHAR so_dir[] = L""; -#endif +static const struct machine_dir +{ + DWORD machine; + const WCHAR *pe_dir; + const WCHAR *so_dir; +} + all_machine_dir[] = +{ + {IMAGE_FILE_MACHINE_I386, L"\\i386-windows\\", L"\\i386-unix\\"}, + {IMAGE_FILE_MACHINE_AMD64, L"\\x86_64-windows\\", L"\\x86_64-unix\\"}, + {IMAGE_FILE_MACHINE_ARMNT, L"\\arm-windows\\", L"\\arm-unix\\"}, + {IMAGE_FILE_MACHINE_ARM64, L"\\aarch64-windows\\", L"\\aarch64-unix\\"}, +}; static inline BOOL is_sepA(char ch) {return ch == '/' || ch == '\\';} static inline BOOL is_sep(WCHAR ch) {return ch == '/' || ch == '\\';} @@ -705,6 +702,14 @@ WCHAR *get_dos_file_name(const WCHAR *filename) return dos_path; } +static inline const WCHAR* get_machine_dir(const struct machine_dir *machine_dir, const WCHAR *name) +{ + WCHAR *ptr; + if ((ptr = wcsrchr(name, L'.')) && !lstrcmpW(ptr, L".so")) + return machine_dir->so_dir; + return machine_dir->pe_dir; +} + static BOOL try_match_file(const WCHAR *name, BOOL (*match)(void*, HANDLE, const WCHAR*), void *param) { HANDLE file = CreateFileW(name, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL); @@ -721,18 +726,27 @@ BOOL search_dll_path(const struct process *process, const WCHAR *name, BOOL (*ma { const WCHAR *env; WCHAR *p, *end; - size_t len, i; + size_t len, i, machine_dir_len; WCHAR *buf; + const struct cpu* cpu; + const struct machine_dir* machine_dir; name = file_name(name); + cpu = process_get_cpu(process); + for (machine_dir = all_machine_dir; machine_dir < all_machine_dir + ARRAY_SIZE(all_machine_dir); machine_dir++) + if (machine_dir->machine == cpu->machine) break; + if (machine_dir >= all_machine_dir + ARRAY_SIZE(all_machine_dir)) return FALSE; + machine_dir_len = max(wcslen(machine_dir->pe_dir), wcslen(machine_dir->so_dir)); + if ((env = process_getenv(process, L"WINEBUILDDIR"))) { const WCHAR dllsW[] = { '\\','d','l','l','s','\\' }; const WCHAR programsW[] = { '\\','p','r','o','g','r','a','m','s','\\' }; len = lstrlenW(env); - if (!(buf = heap_alloc((len + 8 + 3 * lstrlenW(name)) * sizeof(WCHAR)))) return FALSE; + if (!(buf = heap_alloc((len + ARRAY_SIZE(programsW) + machine_dir_len + + 2 * lstrlenW(name) + 1) * sizeof(WCHAR)))) return FALSE; wcscpy(buf, env); end = buf + len; @@ -741,6 +755,11 @@ BOOL search_dll_path(const struct process *process, const WCHAR *name, BOOL (*ma if ((p = wcsrchr(end, '.')) && !lstrcmpW(p, L".so")) *p = 0; if ((p = wcsrchr(end, '.')) && !lstrcmpW(p, L".dll")) *p = 0; p = end + lstrlenW(end); + /* try multi-arch first */ + wcscpy(p, get_machine_dir(machine_dir, name)); + wcscpy(p + wcslen(p), name); + if (try_match_file(buf, match, param)) goto found; + /* then old mono-arch */ *p++ = '\\'; lstrcpyW(p, name); if (try_match_file(buf, match, param)) goto found; @@ -751,6 +770,11 @@ BOOL search_dll_path(const struct process *process, const WCHAR *name, BOOL (*ma if ((p = wcsrchr(end, '.')) && !lstrcmpW(p, L".so")) *p = 0; if ((p = wcsrchr(end, '.')) && !lstrcmpW(p, L".exe")) *p = 0; p = end + lstrlenW(end); + /* try multi-arch first */ + wcscpy(p, get_machine_dir(machine_dir, name)); + wcscpy(p + wcslen(p), name); + if (try_match_file(buf, match, param)) goto found; + /* then old mono-arch */ *p++ = '\\'; lstrcpyW(p, name); if (try_match_file(buf, match, param)) goto found; @@ -763,12 +787,9 @@ BOOL search_dll_path(const struct process *process, const WCHAR *name, BOOL (*ma WCHAR env_name[64]; swprintf(env_name, ARRAY_SIZE(env_name), L"WINEDLLDIR%u", i); if (!(env = process_getenv(process, env_name))) return FALSE; - len = wcslen(env) + wcslen(pe_dir) + wcslen(name) + 2; + len = wcslen(env) + machine_dir_len + wcslen(name) + 1; if (!(buf = heap_alloc(len * sizeof(WCHAR)))) return FALSE; - if ((p = wcsrchr(name, '.')) && !lstrcmpW(p, L".so")) - swprintf(buf, len, L"%s%s\\%s", env, so_dir, name); - else - swprintf(buf, len, L"%s%s\\%s", env, pe_dir, name); + swprintf(buf, len, L"%s%s%s", env, get_machine_dir(machine_dir, name), name); if (try_match_file(buf, match, param)) goto found; swprintf(buf, len, L"%s\\%s", env, name); if (try_match_file(buf, match, param)) goto found;

1 0