
<html><head></head><body>Hi,<br><br>What would be the point in notifying the accesses and stream filter of pause if it has no implications? For the third time, the *whole* point is to notify that the data flow is suspended.<br><br><div class="gmail_quote">Le 19 novembre 2019 14:22:40 GMT+02:00, Alexandre Janniaux <ajanni@videolabs.io> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<pre class="k9mail">Hi,<br><br>On Tue, Nov 19, 2019 at 10:43:27AM +0200, Rémi Denis-Courmont wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;"> Hi,<br><br> The point is that pause/resume was added because some accesses needed it, and they don't expect data to flow during pause.<br></blockquote><br>Ok, this is clearer for my previous questions, thank you.<br><br>Can't this be reversed so that accesses can signal they expect data to<br>not flow during pause, and so it works for most use case without edge<br>case and we handle the other case separately?<br><br>It seems that there is already a can_pause for those cases, so it is<br>already more or less close to this as far as I can read.<br><br>Then there are also filters changing this can_pause to true, like es<br>timeshift. So if I take your point, the issue with what I said<br>previously is that in this case, we don't really know what we can<br>really do in pause state and it is mostly used for the UI to know<br>whether the input is pausable or not?<br><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">Furthermore, we can't allow reading during pause as it breaks live streaming. Otherwise, you'd get a huge hysteresis on pause/resume as you'd fill the buffer on pause and drain it on resume.<br></blockquote><br>This part concerns specifically the prefetch filter right? Is it<br>because of this hystereris that you're saying this patch is wrong?<br><br>In this case, should frame-by-frame be a different state than pause?<br><br>Regards,<br>--<br>Alexandre Janniaux<br>Videolabs<br><br>><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">Le 18 novembre 2019 23:27:26 GMT+02:00, Alexandre Janniaux <ajanni@videolabs.io> a écrit :<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;">Hi,<br><br>On Mon, Nov 18, 2019 at 10:07:22PM +0200, Rémi Denis-Courmont wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #8ae234; padding-left: 1ex;">Le maanantaina 18. marraskuuta 2019, 19.13.03 EET Alexandre Janniaux<br></blockquote>a écrit :<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #8ae234; padding-left: 1ex;"><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #fcaf3e; padding-left: 1ex;"> Hi,<br><br> On Mon, Nov 18, 2019 at 05:56:43PM +0200, Rémi Denis-Courmont<br></blockquote></blockquote>wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #8ae234; padding-left: 1ex;"><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #fcaf3e; padding-left: 1ex;"><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #e9b96e; padding-left: 1ex;">I don't see how the error message is wrong. It is not permissible<br></blockquote></blockquote></blockquote>to read<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #8ae234; padding-left: 1ex;"><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #fcaf3e; padding-left: 1ex;"><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #e9b96e; padding-left: 1ex;">from a paused stream, by definition. This patch looks wrong to<br></blockquote></blockquote></blockquote>me.<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #8ae234; padding-left: 1ex;"><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #fcaf3e; padding-left: 1ex;">The concept of a playback pipeline being paused is fairly simple<br>considering the user point of view, but from the internal point of<br>view I would have expected that pause is just relative to<br></blockquote></blockquote>audio/video<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #8ae234; padding-left: 1ex;"><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #fcaf3e; padding-left: 1ex;">playback itself, not everything before which would be paced by the<br>pool of available resources.<br></blockquote>The concept of pause is easy. Data does not flow during pause. There<br></blockquote>are no<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #8ae234; padding-left: 1ex;">other reasons why pause was exposed to stream filters and accesses in<br></blockquote>the first<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #8ae234; padding-left: 1ex;">place.<br></blockquote>Sure, I hear that, it is not really a hint about why though.<br><br>I should have phrase my sentence like the following:<br><br>If we want data to not flow during pause, it can be achieved<br>by two effects:<br>+ we don't process data anymore, the whole pipeline is<br>freezed.<br>+ we don't switch the last frame / don't display audio, the<br>pipeline process is stalled because paced.<br><br>What Thomas is doing is the second one in this patch. I<br>tested it and it allows seeking while pausing and brings a<br>better UX.<br><br>The two questions then:<br>+ if we keep doing the first, how pause should be handled to<br>allow frame by frame ?<br>+ if we're going for the second, what are the setbacks?<br><br>Do you have any hints with regards to these ?<br><br>Regards,<br>--<br>Alexandre Janniaux<br>Videolabs<hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></blockquote>--<br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.<br></blockquote><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;"><hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></blockquote><hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a></pre></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>