- Wine-GitLab - list.winehq.org

[PATCH v4 0/1] MR2153: win32u: invalidate all cached keys after input
by Henry Goffin (＠AMZN-hgoffin) Feb. 28, 2023

Feb. 28, 2023

This patch addresses an issue in Second Life and potentially other multi-threaded applications which process WM_KEYDOWN in one thread and then verify that the key is "still down" with GetAsyncKeyState from another thread. Wine uses a per-thread key cache, resulting in inconsistent views of key status. Caches are now invalidated when an input event is injected by the driver or via SendInput. -- v4: win32u: invalidate all cached keys after input https://gitlab.winehq.org/wine/wine/-/merge_requests/2153

2 1

Re: [PATCH v4 0/2] MR1924: ntoskrnl.exe/tests: Add KeInitializeDpc tests.
by Rémi Bernon Feb. 28, 2023

Feb. 28, 2023

Rémi Bernon (@rbernon) commented about include/ddk/wdm.h: > typedef struct _KDPC { > - CSHORT Type; > - UCHAR Number; > - UCHAR Importance; > - LIST_ENTRY DpcListEntry; > + UCHAR Type; > + UCHAR Importance; > + volatile USHORT Number; > + SINGLE_LIST_ENTRY DpcListEntry; > + KAFFINITY ProcessorHistory; > PKDEFERRED_ROUTINE DeferredRoutine; > PVOID DeferredContext; > PVOID SystemArgument1; > PVOID SystemArgument2; > - PULONG_PTR Lock; > + PVOID DpcData; This doesn't seem to match `wdm.h` from SDK 10.0.19041.0 at least. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1924#note_25794

1 0

[PATCH v2 0/1] MR2153: win32u: invalidate all cached keys after input
by Henry Goffin (＠AMZN-hgoffin) Feb. 28, 2023

Feb. 28, 2023

This patch addresses an issue in Second Life and potentially other multi-threaded applications which process WM_KEYDOWN in one thread and then verify that the key is "still down" with GetAsyncKeyState from another thread. Wine uses a per-thread key cache, resulting in inconsistent views of key status. Caches are now invalidated when an input event is injected by the driver or via SendInput. -- v2: win32u: invalidate all cached keys after input https://gitlab.winehq.org/wine/wine/-/merge_requests/2153

3 5

[PATCH 0/1] MR2302: gdiplus: Use FILE_SHARE_READ in GdipPrivateAddFontFile().
by w-flo (＠w-flo) Feb. 28, 2023

Feb. 28, 2023

ZusiDisplay sometimes loads the same font file into two different PrivateFontCollections using two threads, so there is a race condition when the file is opened without the FILE_SHARE_READ sharing mode. The second call to GdipPrivateAddFontFile() might fail if the first one hasn't closed the file handle yet. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2302

3 4

[PATCH v3 0/2] MR2016: winex11: Take advantage of RTL_CONSTANT_STRING.
by Alex Henrie (＠alexhenrie) Feb. 28, 2023

Feb. 28, 2023

-- v3: winex11: Avoid calling RtlInitUnicodeString on a static constant. winex11: Use RTL_CONSTANT_STRING instead of reimplementing it. https://gitlab.winehq.org/wine/wine/-/merge_requests/2016

2 3

Re: [PATCH v2 0/2] MR2016: winex11: Take advantage of RTL_CONSTANT_STRING.
by Jacek Caban (＠jacek) Feb. 28, 2023

Feb. 28, 2023

Jacek Caban (@jacek) commented about dlls/winex11.drv/desktop.c: > static void query_desktop_work_area( RECT *rc_work ) > { > static const WCHAR trayW[] = {'S','h','e','l','l','_','T','r','a','y','W','n','d'}; > - UNICODE_STRING str = { sizeof(trayW), sizeof(trayW), (WCHAR *)trayW }; > + UNICODE_STRING str = RTL_CONSTANT_STRING( trayW ); You'd need to null-terminate trayW for this to work. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2016#note_25765

1 0

Re: [PATCH v2 0/1] MR2299: wined3d: Do not switch pixel format on non-private GL contexts.
by Matteo Bruni (＠Mystral) Feb. 28, 2023

Feb. 28, 2023

On Tue Feb 28 14:37:01 2023 +0000, Henri Verbeet wrote: > It's probably worth pointing out that this has quite some potential for > causing regressions in performance. swapchain_blit_gdi() is not fast, > and ideally we'd work on reducing the number of cases where it's needed. !2217 should avoid most of those cases I think, it probably makes sense to wait for that one to go in first. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2299#note_25759

1 0

[PATCH v4 0/5] MR2296: Always use the user images for the D3D12 swapchain
by Giovanni Mascellani (＠giomasce) Feb. 28, 2023

Feb. 28, 2023

-- v4: dxgi: Reset the back buffer index to zero on ResizeBuffers(). dxgi/tests: Test that ResizeBuffers() resets the back buffer index to zero. dxgi: Always assume that a D3D12 swapchain always uses user images. dxgi: Immediately error out when creating a D3D12 swapchain on a non-immediate queue. dxgi/tests: Test that D3D12 swapchains can only be created on direct command queues. https://gitlab.winehq.org/wine/wine/-/merge_requests/2296

2 6

Re: [PATCH v2 0/1] MR2299: wined3d: Do not switch pixel format on non-private GL contexts.
by Matteo Bruni (＠Mystral) Feb. 28, 2023

Feb. 28, 2023

On Tue Feb 28 10:26:07 2023 +0000, Matteo Bruni wrote: > I guess we could add a WARN("Using pixel format %u already set on DC > %p.\n", current, dc); or something. Actually the WARN() in wined3d_context_gl_set_gl_context() is probably enough already, so disregard this. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2299#note_25755

1 0

[PATCH v13 0/4] MR84: vkd3d-shader/spirv: Merge all shader IR fork and join phases into a single phase.
by Conor McCarthy (＠cmccarthy) Feb. 28, 2023

Feb. 28, 2023

Normalise the incoming vkd3d_shader_instruction IR to the shader model 6 pattern where only one patch constant function is emitted. This allows generation of a single patch constant function in SPIR-V. -- v13: vkd3d-shader/spirv: Declare the phase SPIR-V function in spirv_compiler_enter_shader_phase(). vkd3d-shader/spirv: Remove the hull shader phase array. vkd3d-shader/trace: Trace the normalised instruction array after tracing the input. vkd3d-shader/ir: Merge all shader IR fork and join phases into a single phase. https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/84

4 6