- wine-gitlab - winehq.org

Re: [PATCH v2 0/4] MR47: kernel32/tests: Add tests for CancelSynchronousIo.
by Jinoh Kang (＠iamahuman) 11 Jul '22

11 Jul '22

Jinoh Kang (@iamahuman) commented about dlls/kernel32/tests/pipe.c: > + > + SetLastError(0xdeadbeef); > + res = pCancelSynchronousIo(GetCurrentThread()); > + ok(!res, "CancelSynchronousIo succeeded unexpectedly\n"); > + todo_wine > + ok(GetLastError() == ERROR_NOT_FOUND, > + "In CancelSynchronousIo failure, expected ERROR_NOT_FOUND, got %ld\n", GetLastError()); > + > + /* synchronous i/o */ > + pipe = CreateNamedPipeA(PIPENAME, PIPE_ACCESS_DUPLEX, > + PIPE_TYPE_MESSAGE | PIPE_READMODE_MESSAGE | PIPE_WAIT, > + 1, 1024, 1024, NMPWAIT_USE_DEFAULT_WAIT, NULL); > + ok(pipe != INVALID_HANDLE_VALUE, "CreateNamedPipe failed with %lu\n", GetLastError()); > + thread = CreateThread(NULL, 0, synchronousIoThreadMain, pipe, 0, NULL); > + /* wait for thread to start listening */ > + Sleep(100); How about the following race-free approach? ```suggestion:-0+0 /* wait for I/O to start, which transitions the pipe handle from signaled to nonsignaled state. */ while ((wait_status = WaitForSingleObject(pipe, 0)) == WAIT_OBJECT_0) Sleep(1); ok(wait_status == WAIT_TIMEOUT, "WaitForSingleObject returned %lu (error %lu)\n", wait_status, GetLastError()); ``` Ditto for other places. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/47#note_3874

1 0

Re: [PATCH v2 0/4] MR47: kernel32/tests: Add tests for CancelSynchronousIo.
by Jinoh Kang (＠iamahuman) 11 Jul '22

11 Jul '22

Jinoh Kang (@iamahuman) commented about server/async.c: > return woken; > } > > +static int cancel_blocking( struct process *process, struct thread *thread, client_ptr_t iosb ) > +{ > + struct async *async; > + int woken = 0; > + > +restart: > + LIST_FOR_EACH_ENTRY( async, &process->asyncs, struct async, process_entry ) > + { > + if (async->terminated || async->canceled) continue; > + if (async->blocking && (async->thread == thread) && Maybe you forgot to remove the extraneous parentheses? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/47#note_3875

1 0

Re: [PATCH v2 0/4] MR47: kernel32/tests: Add tests for CancelSynchronousIo.
by Jinoh Kang (＠iamahuman) 11 Jul '22

11 Jul '22

> the test would hang on wine before the implementation is in place. Thanks! I missed that the current test cannot verify the Windows behaviour before the commit that introduces the Wine implementation. > the CloseHandle before the WaitForSingleObject is a workaround to get the thread to exit I'm afraid that closing a handle in use is racy by nature; it's effectively pulling the rug out from underneath. A better approach would be to use `CancelIoEx` on the `OVERLAPPED` structure, or `NtCancelIoFileEx` on the `IO_STATUS_BLOCK` structure to cancel the pending operation. > > moving the CloseHandle was meant to show that workaround is no longer needed and that it now performs like windows > > i can remove that if preferred -- https://gitlab.winehq.org/wine/wine/-/merge_requests/47#note_3873

1 0

[PATCH v3 0/2] MR360: Draft: winepulse.drv: add support for containerId property from sysfs path
by Claire (＠ClearlyClaire) 11 Jul '22

11 Jul '22

Some games with support for the haptic feedback and speaker features of the Sony DualSense controller select the controller's audio output by filtering on the ContainerId IMMDevice property to find one that matches the controller's HID's. This MR, together with !359, adds support for exposing such a ContainerId to applications. I marked this MR as a draft because I understand that the way the ��guid�� is generated is far from ideal. Furthermore, I will need to map from sysfs to container Id in other components as well (`winebus` for the HID device, and possibly `winealsa` and other audio drivers), so moving that part elsewhere would make sense. However, I think I will need help with those tasks. -- v3: winepulse: Add support for containerId property from sysfs path winepulse: Store PulseAudio device's sysfs path when available https://gitlab.winehq.org/wine/wine/-/merge_requests/360

2 2

[PATCH v4 0/4] MR359: mmdevapi: copy ContainerID property from audio driver if available
by Claire (＠ClearlyClaire) 11 Jul '22

11 Jul '22

Some games with support for the haptic feedback and speaker features of the Sony DualSense controller select the controller's audio output by filtering on the ContainerId IMMDevice property to find one that matches the controller's HID's. This MR allows this information to be accessible from the IMMDevice's property store when the audio driver provides it (none for now, but I intend to add that feature to `winepulse.drv` and maybe `winealsa.drv`) This is made specific to containerId rather than supporting VT_CLSID on every property because Wine currently stores VT_BLOBs directly in the registry value, which does not allow us to safely disambiguate between VT_CLSID and VT_BLOB values when reading from registry. -- v4: mmdevapi: Invalidate ContainerID of unavailable audio devices mmdevapi: copy ContainerID from audio driver if available mmdevapi: decode ContainerId property to CLSID in MMDevice_GetPropValue mmdevapi: support VT_CLSID for containerId property in MMDevice_SetPropValue https://gitlab.winehq.org/wine/wine/-/merge_requests/359

3 4

[PATCH 0/6] MR420: imm32: win32u conversion
by Jacek Caban (＠jacek) 11 Jul '22

11 Jul '22

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/420

3 7

[PATCH 0/1] MR419: user32/tests: Check that a message window brought topmost does not defocus other windows.
by Giovanni Mascellani (＠giomasce) 11 Jul '22

11 Jul '22

libOVR brings a message window topmost claiming that this way it receives WM_DEVICECHANGE faster. Currently, in Wine, this causes a WM_KILLFOCUS to be sent to the actual topmost window, which in turn causes "Shantae: Risky's Revenge" to minimize. The effect is that the game automatically minimizes as soon as it is launched, which is incorrect. Signed-off-by: Giovanni Mascellani <gmascellani(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/419

2 1

[PATCH v2 0/1] MR409: crypt32: Recognize 'Microsoft Root Certificate Authority 2011' when verifying...
by Paul Gofman (＠gofman) 11 Jul '22

11 Jul '22

crypt32: Recognize 'Microsoft Root Certificate Authority 2011' when verifying the Microsoft root policy. Signed-off-by: Paul Gofman <pgofman(a)codeweavers.com> -- v2: crypt32: Support MICROSOFT_ROOT_CERT_CHAIN_POLICY_CHECK_APPLICATION_ROOT_FLAG. https://gitlab.winehq.org/wine/wine/-/merge_requests/409

3 3

[PATCH v4 0/27] MR398: Draft: xactengine3_7/tests: Test notifications when loading a wave bank.
by Giovanni Mascellani (＠giomasce) 11 Jul '22

11 Jul '22

"The King of Fighters '98 Ultimate Match Final Edition" depends on this behavior. At least, the build I have is; it seems other builds are not. Signed-off-by: Giovanni Mascellani <gmascellani(a)codeweavers.com> -- v4: xactengine3_7/tests: Test notifications when loading a wave bank. fixup! include/wine: Add ANSI color codes. ntdll: Dump executable. include/wine: Add ANSI color codes. ntdll: Introduce x86emu. ntdll: Allowing injecting an int3. ntdll: Allow setting a watchpoint. include/wine: Introduce STAR to dereference a generic number. widl: Generate empty interface implementations. ntdll: Log architecture. ntdll: Implement debug writing to file. ntdll: Mark Wine code in mmap output. ntdll: Support dynamically enabling trace messages. ntdll: Log signals early. ntdll: Dump a single symbol. ntdll: Capture stack backtrace from UNIX side. ntdll: Add facility to disassemble code. ntdll: Print the command line when a process is started. ntdll: Dump memory mappings and unmappings. ntdll: Make dynamic unwind entries accessible from UNIX side. ntdll: Add +microsecs channel for precise timestamps. ntdll: Log return address. ntdll: Log file and line number. ntdll: Do not suppress useful log fields on \1. ntdll: Implement log locking and disabling. ntdll: Add a mechanism to ease development calls. ntdll: Support gio channel by default. This merge request has too many patches to be relayed via email. Please visit the URL below to see the contents of the merge request. https://gitlab.winehq.org/wine/wine/-/merge_requests/398

2 2

[PATCH v3 0/27] MR398: Draft: xactengine3_7/tests: Test notifications when loading a wave bank.
by Giovanni Mascellani (＠giomasce) 11 Jul '22

11 Jul '22

"The King of Fighters '98 Ultimate Match Final Edition" depends on this behavior. At least, the build I have is; it seems other builds are not. Signed-off-by: Giovanni Mascellani <gmascellani(a)codeweavers.com> -- v3: xactengine3_7/tests: Test notifications when loading a wave bank. fixup! include/wine: Add ANSI color codes. ntdll: Dump executable. include/wine: Add ANSI color codes. ntdll: Introduce x86emu. ntdll: Allowing injecting an int3. ntdll: Allow setting a watchpoint. include/wine: Introduce STAR to dereference a generic number. widl: Generate empty interface implementations. ntdll: Log architecture. ntdll: Implement debug writing to file. ntdll: Mark Wine code in mmap output. ntdll: Support dynamically enabling trace messages. ntdll: Log signals early. ntdll: Dump a single symbol. ntdll: Capture stack backtrace from UNIX side. ntdll: Add facility to disassemble code. ntdll: Print the command line when a process is started. ntdll: Dump memory mappings and unmappings. ntdll: Make dynamic unwind entries accessible from UNIX side. ntdll: Add +microsecs channel for precise timestamps. ntdll: Log return address. ntdll: Log file and line number. ntdll: Do not suppress useful log fields on \1. ntdll: Implement log locking and disabling. ntdll: Add a mechanism to ease development calls. ntdll: Support gio channel by default. This merge request has too many patches to be relayed via email. Please visit the URL below to see the contents of the merge request. https://gitlab.winehq.org/wine/wine/-/merge_requests/398

1 1

2025

2024

2023

2022

wine-gitlab