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

Re: [PATCH v15 0/6] MR7650: Add support for AF_UNIX sockets (followup)
by Ralf Habacker (＠rhabacker) April 15, 2025

April 15, 2025

On Tue Apr 15 07:51:04 2025 +0000, Ralf Habacker wrote: > and added to the opened inode list. This is normal behavior and not fd_unix related. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7650#note_100890

1 0

Re: [PATCH v15 0/6] MR7650: Add support for AF_UNIX sockets (followup)
by Ralf Habacker (＠rhabacker) April 15, 2025

April 15, 2025

On Tue Apr 15 07:50:43 2025 +0000, Ralf Habacker wrote: > and is set as the `fd->closed` Here is no fd_unix related issue. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7650#note_100888

1 0

Re: [PATCH v15 0/6] MR7650: Add support for AF_UNIX sockets (followup)
by Ralf Habacker (＠rhabacker) April 15, 2025

April 15, 2025

On Tue Apr 15 07:50:06 2025 +0000, Ralf Habacker wrote: > Here the `unix_fd` is saved for closing the file, which is -1 in the > case of sockets. `unix_fd` is used in `inode_close_pending()`, `inode_destroy()`, `inode_add_closed_fd()`, `fd_destroy()` and `unmount_fd()` to close the file. In any of the mentioned location the call to close() is guarded with `if (fd->unix_fd != -1)`, so these function are already save the socket case. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7650#note_100886

1 0

Re: [PATCH v15 0/6] MR7650: Add support for AF_UNIX sockets (followup)
by Ralf Habacker (＠rhabacker) April 15, 2025

April 15, 2025

Ralf Habacker (@rhabacker) commented about server/fd.c: > fd->inode = inode; > fd->closed = closed_fd; > fd->cacheable = !inode->device->removable; > list_add_head( &inode->open, &fd->inode_entry ); and added to the opened inode list. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7650#note_100884

1 0

Re: [PATCH v15 0/6] MR7650: Add support for AF_UNIX sockets (followup)
by Ralf Habacker (＠rhabacker) April 15, 2025

April 15, 2025

Ralf Habacker (@rhabacker) commented about server/fd.c: > closed_fd->unix_name = fd->unix_name; > closed_fd->disp_flags = 0; > fd->inode = inode; > fd->closed = closed_fd; and is set as the `fd->closed` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7650#note_100883

1 0

Re: [PATCH v15 0/6] MR7650: Add support for AF_UNIX sockets (followup)
by Ralf Habacker (＠rhabacker) April 15, 2025

April 15, 2025

Ralf Habacker (@rhabacker) commented about server/fd.c: > free( path ); > } > > closed_fd->unix_fd = fd->unix_fd; Here the `unix_fd` is saved for closing the file, which is -1 in the case of sockets. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7650#note_100882

1 0

[PATCH 0/1] MR7821: winhttp/tests: Add broken to test_redirect.
by Bernhard Übelacker April 15, 2025

April 15, 2025

Followup of d2e03e5b43. [This is a testbot run.](https://testbot.winehq.org/JobDetails.pl?Key=157865) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7821

2 2

[PATCH v6 0/1] MR7642: server: Use dlsym() to check for epoll_pwait2 at run-time.
by William Horvath (＠whrvt) April 15, 2025

April 15, 2025

This fixes a real problem and improves behavior at the same time. - ~~Fixes: hanging when trying to run Wine built with epoll_pwait2 headers available but epoll_pwait2 is missing in the run-time kernel (i.e. build 5.11->run 5.10)~~ (already fixed.) - Improvement: bring the improved timeout resolution to Wine builds which didn't have epoll_pwait2 at compile time, if the run-time kernel supports it (i.e. build 5.10->run 6.14) This last point is especially important in my opinion, as it applies to official Proton builds. Proton is (currently) built in Debian 11 with kernel 5.10, but SteamOS is a moving target with a much newer kernel version being used to run these Wine builds. These builds use the lower resolution timeouts if epoll_pwait2 is only checked for at compile-time, when it can cheaply and easily be done at run-time instead. ~~Fixes: 87ca5db40e2c1b37423bdc25101a5c5e39e67e6f~~ ~~An alternative to !7640 that kills two birds with one stone.~~ -- v6: server: Use dlsym() to check for epoll_pwait2 at run-time. https://gitlab.winehq.org/wine/wine/-/merge_requests/7642

2 5

[PATCH v2 0/6] MR7775: shell32: Test and rework rename operation.
by Ziqing Hui (＠zhui) April 15, 2025

April 15, 2025

-- v2: shell32: Rework rename_files. shell32: Don't parse wildcard for rename operation. shell32: Rework add_file_entry, add more parameters. shell32/tests: Avoid showing UI when testing. shell32/tests: Test NULL and empty file name for SHFileOperation. shell32/tests: Add more tests to test_rename. https://gitlab.winehq.org/wine/wine/-/merge_requests/7775

2 7

[PATCH v2 0/4] MR7570: mfreadwrite: Implement encoder creation. (Part 1)
by Ziqing Hui (＠zhui) April 15, 2025

April 15, 2025

I split the implementations into 2 parts to make it looks easier to review. So plz first review this MR and make sure it's good so that we don't have to rebase. The follwing patches is in !7571. -- v2: mfreadwrite: Implement sink_writer_SetInputMediaType. mfreadwrite: Implement IMFSinkWriterEx. mfreadwrite: Add converter transform to stream. mfreadwrite: Add attributes member to writer struct. https://gitlab.winehq.org/wine/wine/-/merge_requests/7570

3 9