Valgrind support requires a fork, which I've published to https://gitlab.winehq.org/rbernon/valgrind. The fork implements loading DWARF debug info from PE files, instead of the old and broken upstream PDB support. I've tried to upstream these changes a long time ago but didn't receive any feedback.
I think we could maybe consider keeping a fork, which I'm happy to maintain, as the changes aren't too large. We may want to investigate adding 32-on-64 support, which may require a bit more changes (to VEX specifically, because its amd64 guest doesn't support segment register manipulation).
The changes here are not all related to Valgrind, and I'll create separate MR for those which may make sense independently from Valgrind / GDB.
Also included is a suppression file to silent some annoying false positives, many of which are coming from the cross-stack accesses during syscalls, which are confusing Valgrind's stack heuristics. One can try this out with something like:
`WINELOADERNOEXEC=1 valgrind --suppressions=tools/valgrind.supp wine64/loader/wine64 wine64/programs/winecfg/winecfg.exe`
--
v6: ntdll: Avoid writing to invalid memory in i386 unix dispatcher.
ntdll: Set %rsp before args in x86_64 call_user_mode_callback.
ntdll: Fix incorrect i386 call_user_mode_callback CFI.
ntdll: Fix valgrind notifications from ntdll.so.
ntdll: Import valgrind headers for PE side ntdll.
ntdll: Allocate a truly separate stack for the kernel stack.
ntdll: Maintain a PE module link map and expose it to GDB.
ntdll: Pass a UNICODE_STRING to load_builtin and virtual_map_image.
loader: Expose a shadow copy of ld.so link map to GDB.
ntdll: Add .cfi_signal_frame to __wine_syscall_dispatcher.
https://gitlab.winehq.org/wine/wine/-/merge_requests/1074
The tests shows that, for sink writer created by MFCreateSinkWriterFromURL,
we are not able to grab the media sink object immediately after the creation,
unless the media sink object can be created without setting media type(MP3, ASF).
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2690
All return paths in d3d12_command_queue_flush_ops_locked() must
leave the op mutex locked.
(cherry-picked from upstream commit e27ceddfb4a89470d5d35ab4391d0a5cf4453ef1)
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2654
--
v5: vkd3d: Collect multiple descriptor writes in a buffer and update in one call.
vkd3d: Use atomic exchange for descriptor writes.
vkd3d: Delay writing Vulkan descriptors until submitted to a queue.
vkd3d: Ensure descriptors are pointer aligned.
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/156
--
v4: vkd3d: Collect multiple descriptor writes in a buffer and update in one call.
vkd3d: Use atomic exchange for descriptor writes.
vkd3d: Delay writing Vulkan descriptors until submitted to a queue.
vkd3d: Ensure descriptors are pointer aligned.
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/156