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 <[email protected]>
_______________________________________________
Spice-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to