[Spice-devel] [PATCH spice-gtk v3 1/6] channel-display: Minimize the stream lag by ignoring the server one

[Francois Gouget]

Actually I found something I'm a bit unhappy with in this patch.

So one goal is to make sure all streams are in sync. This is why the 
mmtime conversion is performed globally: if two frames from different 
streams have the same mmtime, the resulting client-side timestamps 
should be identical too. That's only possible by applying the same 
offset to all streams, which is what spice_session_mmtime2client_time() 
does.

But then each stream applies its own delay to make sure it has enough 
margin to ensure a smooth playback. Thus, since the delays are different 
the frame playback may not be in sync across streams.

So I think the delay management should be handled globally too which I 
think should be quite doable. But I have been busy with other stuff so I 
did not get time to investigate and resubmit.


-- 
Francois Gouget <fgouget at codeweavers.com>