https://bugs.winehq.org/show_bug.cgi?id=49535
Bug ID: 49535 Summary: Final Fantasy XI Online: Incorrect Opening Movie Colours Product: Wine Version: 5.12 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: winegstreamer Assignee: wine-bugs@winehq.org Reporter: escomk3@hotmail.com Distribution: ---
After fd25ba65 [1], the opening movie has too little red, and too much blue.
1. https://source.winehq.org/git/wine.git/commit/fd25ba65e0eb9fedfb2cdfa2b7a4b1...
https://bugs.winehq.org/show_bug.cgi?id=49535
Chiitoo escomk3@hotmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1| |fd25ba65e0eb9fedfb2cdfa2b7a | |4b16e0401dfdf Distribution|--- |Gentoo
https://bugs.winehq.org/show_bug.cgi?id=49535
Zebediah Figura z.figura12@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression CC| |z.figura12@gmail.com
--- Comment #1 from Zebediah Figura z.figura12@gmail.com --- Can you please attach a log with WINEDEBUG=+quartz,+gstreamer,+strmbase,+d3d9,+d3d?
https://bugs.winehq.org/show_bug.cgi?id=49535
--- Comment #2 from Chiitoo escomk3@hotmail.com --- Created attachment 67667 --> https://bugs.winehq.org/attachment.cgi?id=67667 ffxi-intro_colours-quartz_gstreamer_strmbase_d3d9_d3d
There will be a single trace line I added for this build:
trace:gstreamer:init_new_decoded_pad Avoid expensive color matrix conversions.
I hope that's okay.
https://bugs.winehq.org/show_bug.cgi?id=49535
--- Comment #3 from Zebediah Figura z.figura12@gmail.com --- Eh, I guess I misidentified the commit ID; I thought this was a different problem...
This is kind of an expected side effect of the linked commit, i.e. colorimetry is a little different. There's not really anything we can do except to revert the commit.
I'd be kind of curious to see if Windows (or even native quartz) produces different colors for this application.
https://bugs.winehq.org/show_bug.cgi?id=49535
--- Comment #4 from Chiitoo escomk3@hotmail.com --- Native 'quartz' results into the same overly green and/or purple colours.
I'll see if I can set up a Windows test at some point. It may take a while, but I would also be able to tell if there's another issue with the video, in that the aspect ratio isn't preserved (the image is stretched horizontally). It might be due to the resolution settings, however, since some things in the game only know 4:3 still.
Thanks!
https://bugs.winehq.org/show_bug.cgi?id=49535
--- Comment #5 from Chiitoo escomk3@hotmail.com --- Forgot to add that the videos of this over at the tubes seem the same (colour-wise) as with Wine before this commit.
https://bugs.winehq.org/show_bug.cgi?id=49535
Zebediah Figura z.figura12@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Final Fantasy XI Online: |Multiple applications have |Incorrect Opening Movie |overly green and purple |Colours |videos due to bad YUV -> | |RGB conversion (Final | |Fantasy XI Online, Megarace | |3)
https://bugs.winehq.org/show_bug.cgi?id=49535
soredake gi85qht0z@relay.firefox.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |gi85qht0z@relay.firefox.com
https://bugs.winehq.org/show_bug.cgi?id=49535
Dmitry Skvortsov (Iglu47) lvb.crd@protonmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |lvb.crd@protonmail.com
--- Comment #6 from Dmitry Skvortsov (Iglu47) lvb.crd@protonmail.com --- Created attachment 68560 --> https://bugs.winehq.org/attachment.cgi?id=68560 bbct-image-example-before-and-after-commits
add please this also affects the 'BlazBlue Calamity Trigger' reverting fd25ba65e0eb9fedfb2cdfa2b7a4b16e0401dfdf and 930dc23c7c9a9221aad19ebf468c3838f86427b3 helps (did not check which one)
I previously posted the output from current wine master (5.20) here: https://bugs.winehq.org/show_bug.cgi?id=25329#attach_68557
https://bugs.winehq.org/show_bug.cgi?id=49535
Olivier F. R. Dierick o.dierick@piezo-forte.be changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |o.dierick@piezo-forte.be Summary|Multiple applications have |Multiple applications have |overly green and purple |overly green and purple |videos due to bad YUV -> |videos due to bad YUV -> |RGB conversion (Final |RGB conversion (Final |Fantasy XI Online, Megarace |Fantasy XI Online, Megarace |3) |3, BlazBlue Calamity | |Trigger)
https://bugs.winehq.org/show_bug.cgi?id=49535
Sagawa sagawa.aki+winebugs@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |sagawa.aki+winebugs@gmail.c | |om
--- Comment #7 from Sagawa sagawa.aki+winebugs@gmail.com --- I'm working on this issue. Hopefully, I'll send a patch in a few weeks later.
https://bugs.winehq.org/show_bug.cgi?id=49535
--- Comment #8 from Dmitry Skvortsov (Iglu47) lvb.crd@protonmail.com --- I looked over and more 'quartz-games': 'BlazBlue Calamity Trigger' 'Guilty Gear X2 Reload' 'Dead or Alive 5 Last Round'. It turned out that reverting fd25ba65e0eb9fedfb2cdfa2b7a4b16e0401dfdf helps with the colors in the video for all of them.
Also for the latest Wine (currently 5.21) this is no longer enough, so I additionally had to cancel 0542a34c247f029a0394f308f9f44c5c41d904fc e484056171bfd57473660243bc75935409320f37 9afc5ed72eaf109726be8f558294a1a21502c496 e47b45e2f55ea33790aa60fdf207cafa53c9adb0 (did not check which ones are specific in this set of patches).
https://bugs.winehq.org/show_bug.cgi?id=49535
--- Comment #9 from Dmitry Skvortsov (Iglu47) lvb.crd@protonmail.com --- Created attachment 68637 --> https://bugs.winehq.org/attachment.cgi?id=68637 doa5-wine-5.21.r69.gb940c5e7c91+yuvfixup.log
I checked https://source.winehq.org/patches/data/195961 on b940c5e7c91bff963336dd7d2c4defc3a82c75a1.
With the yuvfixup patch, colors now look normal in all 3 games I wrote about above. I attach the terminal output from Dead or Alive 5 along with Steam(version for Windows).
Thank You, Akihiro Sagawa
https://bugs.winehq.org/show_bug.cgi?id=49535
Gijs Vermeulen gijsvrm@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Ever confirmed|0 |1 Keywords| |patch Status|UNCONFIRMED |NEW Summary|Multiple applications have |Multiple applications have |overly green and purple |overly green and purple |videos due to bad YUV -> |videos due to bad YUV -> |RGB conversion (Final |RGB conversion (Final |Fantasy XI Online, Megarace |Fantasy XI Online, Megarace |3, BlazBlue Calamity |3, BlazBlue Calamity |Trigger) |Trigger, Mafia: The City of | |Lost Heaven)
--- Comment #10 from Gijs Vermeulen gijsvrm@gmail.com --- Confirming, also affects Mafia: The City of Lost Heaven. (See Bug 48850)
Also adding patch keyword and after this I'll attach a rebased version of Akihiro Sagawa's patch.
https://bugs.winehq.org/show_bug.cgi?id=49535
--- Comment #11 from Gijs Vermeulen gijsvrm@gmail.com --- Created attachment 68793 --> https://bugs.winehq.org/attachment.cgi?id=68793 rebased patch by Akihiro Sagawa
https://bugs.winehq.org/show_bug.cgi?id=49535
Lunknown SolisX007@yahoo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |SolisX007@yahoo.com
--- Comment #12 from Lunknown SolisX007@yahoo.com --- *** Bug 50266 has been marked as a duplicate of this bug. ***
https://bugs.winehq.org/show_bug.cgi?id=49535
Zebediah Figura z.figura12@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |251262a44a8f8403fd963e7bb51 | |0da778944af1b Resolution|--- |FIXED Status|NEW |RESOLVED
--- Comment #13 from Zebediah Figura z.figura12@gmail.com --- This should be fixed by https://source.winehq.org/git/wine.git/commitdiff/251262a44a8f8403fd963e7bb510da778944af1b.
https://bugs.winehq.org/show_bug.cgi?id=49535
--- Comment #14 from Dmitry Skvortsov (Iglu47) lvb.crd@protonmail.com --- (In reply to Dmitry Skvortsov (Iglu47) from comment #8)
Also for the latest Wine (currently 5.21) this is no longer enough, so I additionally had to cancel
Sorry, I didn't quite write. I meant that just `git revert --no-commit fd25ba65e0eb9fedfb2cdfa2b7a4b16e0401dfdf` was no longer enough, because instead of returning it gives a conflict.
https://bugs.winehq.org/show_bug.cgi?id=49535
--- Comment #15 from Chiitoo escomk3@hotmail.com --- (In reply to Zebediah Figura from comment #13)
This should be fixed by https://source.winehq.org/git/wine.git/commitdiff/ 251262a44a8f8403fd963e7bb510da778944af1b.
Quite curiously, the colours in FFXI are still wonky.
I've made pretty sure that I'm building at fac1e40aaf0 [1], and that the particular line isn't there any longer, but maybe I'm just not seeing something that I should...
Will investigate more, hopefully sooner rather than later.
Anyone else see this with the other affected applications?
1. https://source.winehq.org/git/wine.git/commit/fac1e40aaf0726a3e328a922cb4969...
https://bugs.winehq.org/show_bug.cgi?id=49535
--- Comment #16 from Dmitry Skvortsov (Iglu47) lvb.crd@protonmail.com --- Created attachment 68860 --> https://bugs.winehq.org/attachment.cgi?id=68860 additional removing
Please ignore my previous comment: https://bugs.winehq.org/show_bug.cgi?id=49535#c14 I confused this with another issue. In fact, if you check - no conflict exists.
I found out that among those four commits, additionally undoing e484056171bfd57473660243bc75935409320f37 helps. This is still relevant for the current wine, or removing this 1 line helps: https://source.winehq.org/git/wine.git/blob/251262a44a8f8403fd963e7bb510da77...
https://bugs.winehq.org/show_bug.cgi?id=49535
Zebediah Figura z.figura12@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Ever confirmed|1 |0 Resolution|FIXED |--- Status|RESOLVED |UNCONFIRMED
--- Comment #17 from Zebediah Figura z.figura12@gmail.com --- Sorry, that was indeed premature. I've sent another patch which removes the other matrix-mode setting.
https://bugs.winehq.org/show_bug.cgi?id=49535
--- Comment #18 from Chiitoo escomk3@hotmail.com --- Cool.
For fun I tried re-adding the other line, while removing this one, and that alone seems to fix the thing for FFXI.
Thanks!
https://bugs.winehq.org/show_bug.cgi?id=49535
--- Comment #19 from Dmitry Skvortsov (Iglu47) lvb.crd@protonmail.com --- (In reply to Chiitoo from comment #18)
For fun I tried re-adding the other line, while removing this one, and that alone seems to fix the thing for FFXI.
Yes - I did not notice that the presence or absence of the first problematic commit somehow noticeably affects the 'BlazBlue Calamity Trigger', 'Guilty Gear X2 Reload' and 'Dead or Alive 5 Last Round' in the latest versions of Wine. Currently the main thing for them is only - https://source.winehq.org/patches/data/197380.
If it can reduce resource consumption between internal Gstreamer links - then it makes sense for weak machines. However, it looks like applying such a thing to the last link in the output is not well received by the application. (if I understood everything correctly)
https://bugs.winehq.org/show_bug.cgi?id=49535
--- Comment #20 from Dmitry Skvortsov (Iglu47) lvb.crd@protonmail.com --- I mean, not necessarily the last link, but the link where the last color transformation takes place - after all, after it there is no explicit `matrix-mode` restore or something like that. (again if I understood everything correctly)
https://bugs.winehq.org/show_bug.cgi?id=49535
joaopa jeremielapuree@yahoo.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jeremielapuree@yahoo.fr
--- Comment #21 from joaopa jeremielapuree@yahoo.fr --- *** Bug 50145 has been marked as a duplicate of this bug. ***
https://bugs.winehq.org/show_bug.cgi?id=49535
Zebediah Figura z.figura12@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Fixed by SHA1|251262a44a8f8403fd963e7bb51 |6f158754435f403864052e595ab |0da778944af1b |627dadac2666f Status|UNCONFIRMED |RESOLVED
--- Comment #22 from Zebediah Figura z.figura12@gmail.com --- Actually fixed by https://source.winehq.org/git/wine.git/commitdiff/6f158754435f403864052e595ab627dadac2666f.
https://bugs.winehq.org/show_bug.cgi?id=49535
andy andy86@fastwebnet.it changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |andy86@fastwebnet.it
--- Comment #23 from andy andy86@fastwebnet.it --- *** Bug 50286 has been marked as a duplicate of this bug. ***
https://bugs.winehq.org/show_bug.cgi?id=49535
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #24 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 6.0-rc2.
https://bugs.winehq.org/show_bug.cgi?id=49535
Anastasius Focht focht@gmx.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |focht@gmx.net URL| |https://web.archive.org/web | |/20210214111943/http://down | |load.fileplanet.com/ftp1/ga | |medemos/arcade/megarace3_de | |mo.zip?st=NWsgWlFvxM2auaDQ1 | |j2fmQ&e=1613312332
https://bugs.winehq.org/show_bug.cgi?id=49535
Anastasius Focht focht@gmx.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download