[Bug 46942] New: [REGRESSION] 4.0 introduce lag / performance drop in Lara Croft the guardian of light
https://bugs.winehq.org/show_bug.cgi?id=46942 Bug ID: 46942 Summary: [REGRESSION] 4.0 introduce lag / performance drop in Lara Croft the guardian of light Product: Wine Version: 4.0 Hardware: x86 URL: https://github.com/jzengc/wine-git/commit/afad5636cb3a 49dff1e40054b47a8407629b3c01?diff=unified OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: kernel32 Assignee: wine-bugs(a)winehq.org Reporter: andy86(a)fastwebnet.it CC: julliard(a)winehq.org, sagawa.aki+winebugs(a)gmail.com Regression SHA1: afad5636cb3a49dff1e40054b47a8407629b3c01 Distribution: ArchLinux Wine 4.0 introduce strange performance issue in Lara Croft the guardian of light, in short words both audio and video seems stuttering together (ta - ta - taa - ta) as continued micro-lagging would be present. Due this game result unplayable. Try with 3.21 game is totally playable. I run a regression test and I've obtained this result: afad5636cb3a49dff1e40054b47a8407629b3c01 is the first bad commit commit afad5636cb3a49dff1e40054b47a8407629b3c01 Author: Akihiro Sagawa <sagawa.aki(a)gmail.com> Date: Fri Dec 7 00:33:26 2018 +0900 kernel32: Resolve an MUI time zone name. Signed-off-by: Akihiro Sagawa <sagawa.aki(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> :040000 040000 44ffdc70c82ae81aef577d61d64b7bfe4cd3f7ee c26de1fa184b116d5a89b201560734e15c6bd84e M dlls -- 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=46942 andy <andy86(a)fastwebnet.it> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression CC| |andy86(a)fastwebnet.it -- 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=46942 Gijs Vermeulen <gijsvrm(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- URL|https://github.com/jzengc/w | |ine-git/commit/afad5636cb3a | |49dff1e40054b47a8407629b3c0 | |1?diff=unified | CC|julliard(a)winehq.org | --- Comment #1 from Gijs Vermeulen <gijsvrm(a)gmail.com> --- Have you tested with a more recent wine version, e.g. 4.4 or 4.5? -- 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=46942 --- Comment #2 from andy <andy86(a)fastwebnet.it> --- Yes. I've tested with 4.2, 4.3, 4.4, 4.5, bug still present. -- 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=46942 joaopa <jeremielapuree(a)yahoo.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jeremielapuree(a)yahoo.fr --- Comment #3 from joaopa <jeremielapuree(a)yahoo.fr> --- Does reverting the bad commit you found fix the problem? -- 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=46942 --- Comment #4 from andy <andy86(a)fastwebnet.it> --- Created attachment 64066 --> https://bugs.winehq.org/attachment.cgi?id=64066 Compilation error get reverting bad commit Trying reverting bad commit i get a compilation error, can't test anything. -- 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=46942 --- Comment #5 from andy <andy86(a)fastwebnet.it> --- Okay. I successfully revert the bad commit disabling test (my mistake not run the configure script with the appropriate options) and can test the game. The answers is yes, revert bad commit fix the problem. -- 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=46942 andy <andy86(a)fastwebnet.it> changed: What |Removed |Added ---------------------------------------------------------------------------- Hardware|x86 |x86-64 -- 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=46942 Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |leslie_alistair(a)hotmail.com --- Comment #6 from Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> --- This issue should of been solved before wine 4.0 final release. Other applications had the same slow down. A log of +msvcrt,+time would be required to workout what function is still trying to access the registry. -- 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=46942 --- Comment #7 from andy <andy86(a)fastwebnet.it> --- Are you sure that msvcrt does matter with this bug? Only reference to it is in compilation error relative to 'test' code. Bugged commit is relative to kernel32. Also bug is present using native msvcrt.dll instead of builtin. -- 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=46942 --- Comment #8 from Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> --- (In reply to andy from comment #7)
Are you sure that msvcrt does matter with this bug?
Only reference to it is in compilation error relative to 'test' code.
Bugged commit is relative to kernel32.
Also bug is present using native msvcrt.dll instead of builtin.
Yes, I'm sure. The issue is where msvcrt called into kernel32 which always looked up the MUI values even though they were never used. This has been corrected in multiple places but there is always a chance one was missed. -- 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=46942 --- Comment #9 from andy <andy86(a)fastwebnet.it> --- Created attachment 64156 --> https://bugs.winehq.org/attachment.cgi?id=64156 log with "+msvcrt,+time,+timestamp" Here the log. I though you meant +timestamp not +time, in any case I've put both. -- 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=46942 andy <andy86(a)fastwebnet.it> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #64066|0 |1 is obsolete| | -- 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=46942 andy <andy86(a)fastwebnet.it> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.winehq.org/sho | |w_bug.cgi?id=47008 -- 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=46942 --- Comment #10 from andy <andy86(a)fastwebnet.it> --- Hello. I updated my git-tree and tried testing wine 4.6 for this bug, but I couldn't testing it because in wine 4.6 Lara Croft GoL do not working at all. I found a regressions and send a bug 47008 about it. Reverting that regressions performance issue is still present in wine 4.6 (if I not reverting regressions reported in this bug) -- 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=46942 andy <andy86(a)fastwebnet.it> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also|https://bugs.winehq.org/sho | |w_bug.cgi?id=47008 | --- Comment #11 from andy <andy86(a)fastwebnet.it> --- Still present in wine-4.7-66-g8039941c52 -- 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=46942 --- Comment #12 from Sagawa <sagawa.aki+winebugs(a)gmail.com> --- Could you try with latest git tree? My recent change mitigates delay of MUI loading. Thanks in advance. -- 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=46942 andy <andy86(a)fastwebnet.it> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|UNCONFIRMED |RESOLVED --- Comment #13 from andy <andy86(a)fastwebnet.it> --- I've been try and I can confirm that is resolved fix in wine 4.9 -- 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=46942 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #14 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 4.10. -- 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)
-
wine-bugs@winehq.org