[PATCH v3 1/1] winegstreamer: reintroduce flushing for wg_parser

Zebediah Figura zfigura at codeweavers.com
Tue Jun 7 16:56:13 CDT 2022


On 6/1/22 20:58, zlice wrote:
> From: zlice <zlice555 at gmail.com>
> 
> Previous commit removed flushing but this causes issues in some games.
> 
> - add flushing back
> - move variable creation after initial condition returns in wm_reader_get_stream_sample. This caused some slow down.
> - Remove EOS sets. Some are not what flushing did before. Had some seg faults on exiting Fallout 3 with them in.

As stated in my original reply to this patch [1]:

I don't think "reintroduce flushing" is the right answer here, not
without understanding why it matters.

The point of 5144b2766 is that flushing should not make a difference. It
allows wg_parser_stream_get_event() to return more quickly, but that
same cost is added to the subsequent seek or stop request, so
GST_Seeking_SetPositions() or parser_cleanup_stream() will end up taking
just as long.

If flushing does make a difference, I think we need to understand why,
and quite likely solve this a different way.

[1] https://www.winehq.org/pipermail/wine-devel/2022-May/217461.html



More information about the wine-devel mailing list