[Bug 57199] New: window_surface not always destroyed
https://bugs.winehq.org/show_bug.cgi?id=57199 Bug ID: 57199 Summary: window_surface not always destroyed Product: Wine Version: unspecified Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: win32u Assignee: wine-bugs(a)winehq.org Reporter: alexandros.frantzis(a)collabora.com Distribution: --- Created attachment 77093 --> https://bugs.winehq.org/attachment.cgi?id=77093 DPI unaware app with winex11 not destroying window surfaces With the latest Wine master (tested 03d985369dc9afe943ee3072478a2567b3fe4077, but haven't run a bisection, so can't say when this was introduced), there are scenarios where the previous window_surface for a HWND is not destroyed when a new one is created for that same HWND (e.g., resizing). This creates memory leaks which have different effects depending on the driver (e.g., winewayland eventually runs out of SHM memory). I am seeing this issue with applications that are not DPI aware when the system dpi != 96. I am attaching two logs, one for winex11 (under xwayland) and one for winewayland, from the same application (Crayon Physics Deluxe) while interactively resizing the window, in a prefix with dpi=192 configured (matching the 2x monitor scale). Note the absence of any window surface destruction calls in the logs, whereas multiple new window surfaces (for the same window) are being created during the resize. I have slightly changed the {WAYLAND,X11DRV}_CreateWindowSurface TRACE() so that it prints `*surface` (instead of `surface`) in the logs. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=57199 alexandros.frantzis(a)collabora.com changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|window_surface not always |Window surface leaks with |destroyed |DPI unaware apps -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=57199 --- Comment #1 from alexandros.frantzis(a)collabora.com --- Created attachment 77094 --> https://bugs.winehq.org/attachment.cgi?id=77094 DPI unaware app with winewayland not destroying window surfaces -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=57199 Rémi Bernon <rbernon(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED CC| |rbernon(a)codeweavers.com Fixed by SHA1| |9d1bdcd56f3f8234af88273ce2d | |d5d32e8e9d6fe Resolution|--- |FIXED --- Comment #2 from Rémi Bernon <rbernon(a)codeweavers.com> --- Should be fixed after 9d1bdcd56f3f8234af88273ce2dd5d32e8e9d6fe. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=57199 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #3 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 9.18. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
WineHQ Bugzilla