Re: [Spice-devel] [spice] Enable mm_time adjustments on startup

2019-05-09 Thread Francois Gouget
On Fri, 3 May 2019, Frediano Ziglio wrote: [...] > > > What will be the default? what will happen to late video frames? > > > > Late frames will be dropped as has always been the case (at least in > > the mjpeg case otherwise it's a bit more complex). > > > > I think the terminology should reall

Re: [Spice-devel] [spice] Enable mm_time adjustments on startup

2019-05-03 Thread Frediano Ziglio
- Original Message - > From: "Frediano Ziglio" > To: "Francois Gouget" > Cc: "Spice devel" > Sent: Friday, 3 May, 2019 10:43:20 AM > Subject: Re: [Spice-devel] [spice] Enable mm_time adjustments on startup > > > On Wed, 17 Apr 2

Re: [Spice-devel] [spice] Enable mm_time adjustments on startup

2019-05-03 Thread Frediano Ziglio
> On Wed, 17 Apr 2019, Snir Sheriber wrote: > [...] > > > The reason for that is that while a minimum 400 ms latency is fine when > > > playing a YouTube video [1], it's very annoying when the whole desktop > > > is being streamed, either through the streaming agent or through the > > > future Virg

Re: [Spice-devel] [spice] Enable mm_time adjustments on startup

2019-05-02 Thread Francois Gouget
On Fri, 26 Apr 2019, Frediano Ziglio wrote: > > > > We send mm_time adjustments to the client whenever there is no audio > > playback. There is no audio playback on startup. Therefore > > mm_time_enabled must be true on startup. QED. > > > > How did you demonstrated it ? Starting Xspice (so no

Re: [Spice-devel] [spice] Enable mm_time adjustments on startup

2019-05-02 Thread Francois Gouget
On Wed, 17 Apr 2019, Snir Sheriber wrote: [...] > > The reason for that is that while a minimum 400 ms latency is fine when > > playing a YouTube video [1], it's very annoying when the whole desktop > > is being streamed, either through the streaming agent or through the > > future Virgl remote acc

Re: [Spice-devel] [spice] Enable mm_time adjustments on startup

2019-04-26 Thread Frediano Ziglio
> > On Thu, 11 Apr 2019, Victor Toso wrote: > > > Hi, > > > > On Wed, Apr 10, 2019 at 11:25:17AM +0200, Francois Gouget wrote: > > > We send mm_time adjustments to the client whenever there is no audio > > > playback. There is no audio playback on startup. Therefore > > > mm_time_enabled must be

Re: [Spice-devel] [spice] Enable mm_time adjustments on startup

2019-04-26 Thread Frediano Ziglio
> > We send mm_time adjustments to the client whenever there is no audio > playback. There is no audio playback on startup. Therefore > mm_time_enabled must be true on startup. QED. > How did you demonstrated it ? > Signed-off-by: Francois Gouget > --- > server/reds.c | 1 + > 1 file changed,

Re: [Spice-devel] [spice] Enable mm_time adjustments on startup

2019-04-17 Thread Snir Sheriber
Hi, On 4/17/19 7:27 AM, Francois Gouget wrote: On Thu, 11 Apr 2019, Victor Toso wrote: Hi, On Wed, Apr 10, 2019 at 11:25:17AM +0200, Francois Gouget wrote: We send mm_time adjustments to the client whenever there is no audio playback. There is no audio playback on startup. Therefore mm_time

Re: [Spice-devel] [spice] Enable mm_time adjustments on startup

2019-04-16 Thread Francois Gouget
On Thu, 11 Apr 2019, Victor Toso wrote: > Hi, > > On Wed, Apr 10, 2019 at 11:25:17AM +0200, Francois Gouget wrote: > > We send mm_time adjustments to the client whenever there is no audio > > playback. There is no audio playback on startup. Therefore > > mm_time_enabled must be true on startup. Q

Re: [Spice-devel] [spice] Enable mm_time adjustments on startup

2019-04-11 Thread Victor Toso
Hi, On Wed, Apr 10, 2019 at 11:25:17AM +0200, Francois Gouget wrote: > We send mm_time adjustments to the client whenever there is no audio > playback. There is no audio playback on startup. Therefore > mm_time_enabled must be true on startup. QED. > > Signed-off-by: Francois Gouget But what ar

[Spice-devel] [spice] Enable mm_time adjustments on startup

2019-04-10 Thread Francois Gouget
We send mm_time adjustments to the client whenever there is no audio playback. There is no audio playback on startup. Therefore mm_time_enabled must be true on startup. QED. Signed-off-by: Francois Gouget --- server/reds.c | 1 + 1 file changed, 1 insertion(+) diff --git a/server/reds.c b/serve