March 2025 - Wine-GitLab - list.winehq.org

[PATCH 0/6] MR7605: PDB rewrite #3
by eric pouech (＠epo) March 18, 2025

March 18, 2025

This serie (PDB rewrite #3) mostly handles the debug information needed to support C++ enum underlying integral type. - it generalizes the use of VARIANT to gather enum values, - to make things simple, in PDB, the type of variant used has nothing to do with the size of the C++ integral type, but is the smallest size to hold the considered enum value, - adapt the dwarf backend to behave the same way as the PDB one, - adapt debugger to support up to 8-bytes variant types. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7605

2 6

[PATCH v7 0/1] MR6938: winewayland.drv: Implement support for xdg-toplevel-icon.
by Etaash Mathamsetty (＠etaash.mathamsetty) March 18, 2025

March 18, 2025

-- v7: winewayland.drv: Implement support for xdg-toplevel-icon. https://gitlab.winehq.org/wine/wine/-/merge_requests/6938

3 2

[PATCH v5 0/2] MR7290: kernelbase: Handle correctly paths with forward slashes in ReplaceFileW.
by Roman Pišl March 18, 2025

March 18, 2025

PathRemoveFileSpecW keeps only the drive if the path contains all forward slashes as shown in tests. But then the temporary file is created in the root folder which fails for drive Z:. This fixes a corner case when using CMake in Wine. Another usage of PathRemoveFileSpecW is in PathRelativePathToW which should be also checked. I'm leaving this as a draft until getting some feedback and adding tests. -- v5: kernelbase: Handle correctly paths with forward slashes in ReplaceFileW. kernel32: Test ReplaceFileW with forward slashes. https://gitlab.winehq.org/wine/wine/-/merge_requests/7290

1 2

[PATCH 0/1] MR7603: kerberos: Fix imported target name leak.
by Dmitry Timoshkov (＠dmitry) March 18, 2025

March 18, 2025

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

3 2

[PATCH v2 0/4] MR7540: kernel32/tests: Add tests for maximum path length limitation.
by Yongjie Yao (＠yaoyongjie) March 18, 2025

March 18, 2025

In CopyFileEx, and DeleteFile functions, by default, the file name and path are limited to MAX_PATH characters. To extend this limit to 32,767 wide characters, need prepend "\\\\?\\" to the path. -- v2: kernelbase: Limit the maximum path length for DeleteFile. kernelbase: Fix DeleteFileA doesn't support long path. kernelbase: Limit the maximum path length for filesystem. kernel32/tests: Add tests for maximum path length limitation. https://gitlab.winehq.org/wine/wine/-/merge_requests/7540

3 10

Re: [PATCH v3 0/1] MR5052: krnl386: Restore simulated real mode interrupts. - closed
by Alex Henrie (＠alexhenrie) March 18, 2025

March 18, 2025

This merge request was closed by Alex Henrie. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5052

1 0

Re: [PATCH v3 0/1] MR5052: krnl386: Restore simulated real mode interrupts.
by Alex Henrie (＠alexhenrie) March 18, 2025

March 18, 2025

On Mon Mar 17 22:03:12 2025 +0000, Alexandre Julliard wrote: > You should add a separate handler just for what the app needs, instead > of forcing all handlers to support real-mode pointers. > I added an example in 7e3bfd4e5e4f9ca689e1734da05a90b1b14351c9, it seems > to work here but it could be extended if necessary. I see what you mean now about having a separate handler. I still don't understand why it's better to have two implementations of the DOS "get volume serial number" ioctl, a more realistic implementation in [int21.c](https://gitlab.winehq.org/wine/wine/-/blob/6298b0cab2086ae61f46b28… and a less realistic implementation in [int31.c](https://gitlab.winehq.org/wine/wine/-/blob/6298b0cab2086ae61f46b28…, or why it's hard to support real-mode pointers (the original [CTX_SEG_OFF_TO_LIN](https://gitlab.winehq.org/wine/wine/-/commit/526b245237… macro seemed to work just fine). But the two-implementation solution works, so my unresolved questions are just curiosities at this point. Thank you very much for taking the time to fix Myst. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5052#note_98205

1 0

[PATCH v3 0/6] MR6866: Draft: ntdll: On macOS x86_64, swap GSBASE between the TEB and macOS TSD when entering/leaving PE code.
by Brendan Shanks (＠bshanks) March 17, 2025

March 17, 2025

x86_64 Windows and macOS both use `%gs` to access thread-specific data (Windows TEB, macOS TSD). To date, Wine has worked around this conflict by filling the most important TEB fields (`0x30`/`Self`, `0x58`/`ThreadLocalStorage`) in the macOS TSD structure (Apple reserved the fields for our use). This was sufficient for most Windows apps. CrossOver's Wine had an additional hack to handle `0x60`/`ProcessEnvironmentBlock`, and binary patches for certain CEF binaries which directly accessed `0x8`/`StackBase`. Additionally, Apple's libd3dshared could activate a special mode in Rosetta 2 where code executing in certain regions would use the Windows TEB when accessing `%gs`. Now that the PE separation is complete, GSBASE can be swapped when entering/exiting PE code. This is done in the syscall dispatcher, unix-call dispatcher, and for user-mode callbacks. GSBASE also needs to be set to the macOS TSD when entering signal handlers (in `init_handler()`), and then restored to the Windows TEB when exiting (in `leave_handler()`). There is a special-case needed in `usr1_handler`: when inside a syscall (on the kernel stack), GSBASE may need to be reset to either the TEB or the TSD. The only way to tell is to determine what GSBASE was set to on entry to the signal handler. --- macOS does not have a public API for setting GSBASE, but the private `_thread_set_tsd_base()` works and was added in macOS 10.12. `_thread_set_tsd_base()` is a small thunk that sets `%esi`, `%eax`, and does the `syscall`: https://github.com/apple-oss-distributions/xnu/blob/8d741a5de7ff4191bf97d57…. The syscall instruction itself clobbers `%rcx` and `%r11`. I've tried to save as few registers as possible when calling `_thread_set_tsd_base()`, but there may be room for improvement there. --- I've tested this successfully on macOS 15 (Apple Silicon and Intel) with several apps and games, including the `cefclient.exe` CEF sample. I still need to test this patch on macOS 10.13, and I'd also like to do some performance testing. --- I also tested an alternate implementation strategy for this which took advantage of the expanded "full" thread state which is passed to signal handlers when a process has set a user LDT. The full thread state includes GSBASE, so GSBASE is set back to whatever is in the sigcontext on return (like every other field in the context). This would avoid needing to explicitly reset GSBASE in `leave_handler()`, and avoid the special-case in `usr1_handler()`. This strategy was simpler, but I'm not using it for 2 reasons: - the "full" thread state is only available starting with macOS 10.15, and we still support 10.13. - more crucially, Rosetta 2 doesn't seem to correctly implement the GS.base field of the full thread state. It's set to 0 on entry, and isn't read on exit. -- v3: ntdll: Remove x86_64 Mac-specific TEB access workarounds that are no longer needed. ntdll: On macOS x86_64, swap GSBASE between the TEB and macOS TSD when entering/leaving PE code. ntdll: Ensure init_handler runs in signal handlers before any compiler-generated memset calls. ntdll: Remove ugly fallback method for getting a thread's GSBASE on macOS. ntdll: Leave kernel stack before accessing %gs in x86_64 syscall dispatcher. ntdll: Do %gs accesses before switching to kernel stack in x86_64 syscall dispatcher. https://gitlab.winehq.org/wine/wine/-/merge_requests/6866

5 11

Re: [PATCH v2 0/5] MR7600: PDB rewrite #2 - approved
by Alexandre Julliard (＠julliard) March 17, 2025

March 17, 2025

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7600

1 0

Re: [PATCH v3 0/1] MR7510: msxml6/tests: Make test pass with Windows 7. - approved
by Alexandre Julliard (＠julliard) March 17, 2025

March 17, 2025

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7510

1 0