kwin resizes hidden windows most of the time. I see the existing flaky markers triggered on Windows 11 occasionally. This MR makes the ddraw-d3d9 tests run reliably on kwin (I tested version 5.27.7) in Xephyr and a real display.
This makes the !WS_VISIBLE tests fairly weak since we ignore the random failures we get on Windows and Wine.
According to https://marc.info/?l=wine-patches&m=141642450913299&w=2 , the mail to wine-patches behind commit f4d520d6e2, which added this test, those tests intended to cover some differences between d3d9 and d3d9ex, not a specific game behavior. I vaguely remember that I did run into something that needed this behavior, but I can't find out any more what this might have been. The memory of those 9 year old patches is getting blurry.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3779
This series fixes a few bugs in the d3d tests that cause them to crash on my Windows XP laptop with the r200 GPU (dx8 era).
There are plenty more problems on this GPU on Windows. d3d8:visual hangs the entire system. d3d9:device has some failing tests. d3d9:visual crashes, which is probably good because otherwise it would also cause a kernel panic.
--
v2: d3d9/tests: Fix skipping ATI2N in test_resource_access.
d3d8/tests: Fix skipping ATI2N in test_resource_access.
d3d9/tests: Request a 3 mip level cube texture in test_getdc().
d3d8/tests: Zero stride tests crash on Windows XP, r200 GPU.
d3d9/tests: Zero stride tests crash on Windows XP, r200 GPU.
https://gitlab.winehq.org/wine/wine/-/merge_requests/3757
On Thu Sep 14 07:06:18 2023 +0000, Ziqing Hui wrote:
> This was added because mp4mux is requiring it: https://gitlab.freedesktop.org/gstreamer/gstreamer/-/blob/main/subprojects/…
> Without codec_data set, mp4mux element fails. And according to my test,
> setting codec_data to MF_MT_MPEG_SEQUENCE_HEADER is OK, becuase I
> finally successfully generate a mp4 file which is able to be played by
> video player.
I didn't dig into how mp4mux element handle h264 codec_data. But the truth is it do accpet MF_MT_MPEG_SEQUENCE_HEADER.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3636#note_45304
On Thu Sep 14 07:06:18 2023 +0000, Zebediah Figura wrote:
> Hi Ziqing, I've been doing some research, and I believe patch 2/6
> (committed as 320383c594c4) is incorrect. GStreamer's "codec-data" field
> is an AVCDecoderConfigurationRecord, as defined by ISO/IEC 14496-15
> (which you can see at [1]). MF_MT_MPEG_SEQUENCE_HEADER, on the other
> hand, seems to contain the raw SPS/PPS NALUs, start codes and all.
> Since Microsoft uses Annex B everywhere, it shouldn't need the
> codec-data anyway, so I'm curious now why this was added—was there a
> need for it, or did you just add it on the assumption that the two
> attributes mapped to each other?
> [1] https://doc-kurento.readthedocs.io/en/latest/knowledge/h264.html#gstreamer-…
This was added because mp4mux is requiring it: https://gitlab.freedesktop.org/gstreamer/gstreamer/-/blob/main/subprojects/…
Without codec_data set, mp4mux element fails. And according to my test, setting codec_data to MF_MT_MPEG_SEQUENCE_HEADER is OK, becuase I finally successfully generate a mp4 file which is able to be played by video player.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3636#note_45303