<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 30, 2015 at 3:23 PM, Victor Toso <span dir="ltr"><<a href="mailto:victortoso@redhat.com" target="_blank">victortoso@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":685" class="a3s" style="overflow:hidden">This is done only once in order to get the last volume/mute/channels<br>
value of each stream. The main pourpose of it is when we want to<br>
volume-sync when the client connects.<br>
<br>
The volume events are handy for all the other volume-sync possibilities<br>
like migration. Without it, we may send the wrong value to the guest.<br>
<span class=""><br></span></div></blockquote><div><br></div><div>Even when using volume restore only?<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":685" class="a3s" style="overflow:hidden"><span class="">
> > channel-main:<br>
> > - use g_idle_add in audio-sync functions to work around the delay of getting<br>
> > this values of pulseaudio. This is not significantly high in my enviroment<br>
> > (three function calls). Using gstreamer backend it did not fail as it start<br>
> > the pipeline to get the values.<br>
><br>
> What's the guarantee that volume has been received when that idle is run?<br>
<br>
</span>Well, you are right. If gstreamer/pulse backend fails for whatever<br>
reason this will not stop. Do you think a trying a fix number of times<br>
is fine?<br>
<br></div></blockquote><div> </div><div>No :) But I would rather make get_volume() async to properly sync when volume is received (and wait sync before playback/record start to avoid glitch)<br></div></div><br>-- <br><div class="gmail_signature">Marc-André Lureau</div>
</div></div>