Re: [Spice-devel] [PATCH] Make use of the new playback rate interface, if available.

2014-04-28 Thread Andrew Eikum
On Mon, Apr 28, 2014 at 07:04:04AM -0500, Jeremy White wrote: > >> /* disable CELT */ > >> ret = spice_server_set_playback_compression(qxl->spice_server, 0); > >> if (ret < 0) > >> return errno; > >This codepath disables compression for some reason, should we do the > >same

Re: [Spice-devel] [PATCH] Make use of the new playback rate interface, if available.

2014-04-28 Thread Jeremy White
/* disable CELT */ ret = spice_server_set_playback_compression(qxl->spice_server, 0); if (ret < 0) return errno; This codepath disables compression for some reason, should we do the same in the newer codepath? I don't think it's a valuable code path; in fact, I fear

Re: [Spice-devel] [PATCH] Make use of the new playback rate interface, if available.

2014-04-28 Thread Christophe Fergeau
Hey, This has been pushed already, but I had one question: On Mon, Apr 21, 2014 at 09:39:49AM -0500, Jeremy White wrote: > --- > src/spiceqxl_audio.c | 13 - > 1 file changed, 12 insertions(+), 1 deletion(-) > > diff --git a/src/spiceqxl_audio.c b/src/spiceqxl_audio.c > index 5b87

[Spice-devel] [PATCH] Make use of the new playback rate interface, if available.

2014-04-21 Thread Jeremy White
Signed-off-by: Jeremy White --- src/spiceqxl_audio.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/src/spiceqxl_audio.c b/src/spiceqxl_audio.c index 5b87302..7afc089 100644 --- a/src/spiceqxl_audio.c +++ b/src/spiceqxl_audio.c @@ -193,12 +193,17 @@ audio_thr