Wine-gitlab

Download

wine-gitlab@list.winehq.org

3 participants
33772 discussions

[wine-gitlab] Re: [PATCH 0/3] MR393: wmvcore: Asynchronous reader state transitions.
by Rémi Bernon 06 Jul '22

06 Jul '22

I'll have a look at the time failures, I was seeing them locally but not on the testbot run so I assumed they were flukes caused by some decoder differences, looks like it may be something else after all. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/393#note_3477

1 0

[wine-gitlab] Re: [PATCH v6 0/7] MR140: qasf: More ASF reader filter implementation.
by Rémi Bernon 06 Jul '22

06 Jul '22

On Wed Jul 6 17:52:44 2022 +0000, **** wrote: > Zebediah Figura replied on the mailing list: > ``` > On 7/6/22 05:08, R��mi Bernon (@rbernon) wrote: > >> So yes, we could not block in IWMReader::Stop(), but that's not > >> ultimately going to help anything. We do still need to wait for the > >> streaming thread to stop at some point. > > > > Well we could then call `Stop` and wait for the WMT_STOPPED callback > notification with a condition variable, releasing the filter_cs so the > callbacks are free to enter it to make the thread-safety obviously correct. > Unfortunately we can't release the filter CS while waiting for the > thread to stop. We need to protect the whole operation. > ``` I opened https://gitlab.winehq.org/wine/wine/-/merge_requests/393#note_3473 to make WM async reader state transitions asynchronous, so that we can then do the right thing here and wait for the callback notifications while entering the CS and avoiding the deadlock. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/140#note_3475

1 0

Jump to page: