Hi,
On Thu, Aug 03, 2017 at 09:29:43AM -0400, Marc-André Lureau wrote:
> Hi
>
> ----- Original Message -----
> > From: Victor Toso <me@xxxxxxxxxxxxxx>
> >
> > There should be some effort to make spice-server adjust mmtime correctly
> > instead
> > of relying on audio backend output directly.
> >
> > The latency from audio backend should be forwarded to spice-server but that's
> > yet to be implemented.
> >
> > This two patches allow us to simply turn-off adjusting mmtime in the client
> > directly from audio's backend.
> >
> > A property sounds reasoanble as this might be something nice to test even if
> > mmtime from spice-server looks great, for instance, considering a mocked data
> > from server we can compare if mmtime from server is better then spice-gtk
> > adjustments based on pulseaudio input.
>
> This really looks like test code to me.
>
> Is this really something we want to have checked in? and in the API?
Relying on audio backend works well in general but the correct mmtime
should come from server. We *use* the tweak since 2010 :)
> If it's generally useful tweak, I would rather use an environment
> variable, but I doubt this is the case.
What should be done is to have mmtime sent correctly, which is not the
case.
> I start to be worried about all of our streaming tweaks and issues. Is
> there any effort to use RTP/SRTP instead? I think this would be a big
> opportunity to improve the situation going forward.
There is not effort on that AFAIK.
But I'm interested on improving the mmtime in server side but it'll be
hard if I can't disable this long-term workaround.
If it isn't a workaround, I would like to understand why.
commit fbe3b5ec32e3d93f0a0f41239b85be723d8d91c5
Author: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
Date: Wed Dec 22 15:32:45 2010 +0100
gtk: update mm time based on playback time+delay
Cheers,
toso
>
>
> > Cheers,
> > toso
> >
> > Victor Toso (2):
> > session: introduce "video-sync-on-audio-latency" property
> > spicy: toggle SpiceSession::video-sync-on-audio-latency
> >
> > src/channel-playback.c | 7 ++++++-
> > src/spice-session-priv.h | 1 +
> > src/spice-session.c | 33 +++++++++++++++++++++++++++++++++
> > tools/spicy.c | 43 ++++++++++++++++++++++++++++++++++++++++++-
> > 4 files changed, 82 insertions(+), 2 deletions(-)
> >
> > --
> > 2.13.0
> >
> > _______________________________________________
> > Spice-devel mailing list
> > Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> > https://lists.freedesktop.org/mailman/listinfo/spice-devel
> >
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel