Re: [Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-06-09 Thread Francois Gouget
On Thu, 4 Jun 2015, Fabio Fantoni wrote: [...] > No bandwidth limit, on client I saw use between 1 and 3 mb, both server > and client have 1 gbps cabled network (lan). > Server used for testing is a dell PE T310 with cpu xeon X3450 is asus > K53S with core i5-2430M. > Server cpu usage of qemu proce

Re: [Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-06-04 Thread Fabio Fantoni
Il 02/06/2015 12:29, Francois Gouget ha scritto: > On Tue, 2 Jun 2015, Fabio Fantoni wrote: > [...] >> Video on youtube, > Does it look like the video moves back and forth? Or maybe just the > progress bar area, with some shearing too. That's a known issue caused > by the progress bar popping up

Re: [Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-06-02 Thread Francois Gouget
On Tue, 2 Jun 2015, Fabio Fantoni wrote: [...] > Video on youtube, Does it look like the video moves back and forth? Or maybe just the progress bar area, with some shearing too. That's a known issue caused by the progress bar popping up and down. > hd trailer (file) with vlc (without vp8 works

Re: [Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-06-02 Thread Fabio Fantoni
Il 02/06/2015 01:57, Francois Gouget ha scritto: > On Mon, 1 Jun 2015, Fabio Fantoni wrote: > [...] >> I suppose that these warning I saw on client are related: >> http://pastebin.com/GeTPyscG >>> 0:00:00.150227452 27501 0x29f98f0 WARNvideodecoder >>> gstvideodecoder.c:2026:gst_vid

Re: [Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-06-01 Thread Francois Gouget
On Mon, 1 Jun 2015, Fabio Fantoni wrote: [...] > I suppose that these warning I saw on client are related: > http://pastebin.com/GeTPyscG > > 0:00:00.150227452 27501 0x29f98f0 WARNvideodecoder > > gstvideodecoder.c:2026:gst_video_decoder_chain: Received > > buffer without a new-seg

Re: [Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-06-01 Thread Fabio Fantoni
Il 29/05/2015 15:20, Francois Gouget ha scritto: > On Fri, 29 May 2015, Fabio Fantoni wrote: > [...] >> I'm trying the gst1 support but build fails with this error: >> gstreamer_encoder.c: In function 'construct_pipeline': >> gstreamer_encoder.c:219:41: error: 'GstEncoder' has no member named 'frc'

Re: [Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-05-29 Thread Francois Gouget
On Fri, 29 May 2015, Fabio Fantoni wrote: [...] > I'm trying the gst1 support but build fails with this error: > gstreamer_encoder.c: In function 'construct_pipeline': > gstreamer_encoder.c:219:41: error: 'GstEncoder' has no member named 'frc' > "deadline", encoder->frc.period

Re: [Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-05-29 Thread Fabio Fantoni
Il 29/05/2015 04:29, Francois Gouget ha scritto: > On Thu, 28 May 2015, Fabio Fantoni wrote: > [...] >> Thanks for reply, I'm preparing spice-gtk with vp8 patches armhf for >> test it on arm thin client (for example raspberry 2), I want try to use >> gst-omx for hardware decoding of mjpeg and vp8,

Re: [Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-05-28 Thread Francois Gouget
On Thu, 28 May 2015, Fabio Fantoni wrote: [...] > Thanks for reply, I'm preparing spice-gtk with vp8 patches armhf for > test it on arm thin client (for example raspberry 2), I want try to use > gst-omx for hardware decoding of mjpeg and vp8, can I simply build and > install gst-omx instead of gst-

Re: [Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-05-28 Thread Fabio Fantoni
Il 28/05/2015 16:17, Francois Gouget ha scritto: > On Tue, 26 May 2015, Fabio Fantoni wrote: > [...] >> After din't show gstreamer warning anymore but still have image freeze >> and also spice-gtk crash after open video fullscreen, here the full gdb >> datas: >> http://pastebin.com/idTkZLh0 > It lo

Re: [Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-05-28 Thread Francois Gouget
On Tue, 26 May 2015, Fabio Fantoni wrote: [...] > After din't show gstreamer warning anymore but still have image freeze > and also spice-gtk crash after open video fullscreen, here the full gdb > datas: > http://pastebin.com/idTkZLh0 It looks like the crash happened when trying to access the st->

Re: [Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-05-27 Thread Francois Gouget
On Thu, 21 May 2015, Fabio Fantoni wrote: [...] > Thanks for improving spice, I tried your patches, I applied the > spice-gtk, spice-protocol and I updated the git submodule to use right > spice-protocol but make of spice-common/common generate a new enum.h and > override the spice-protocol patch f

Re: [Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-05-26 Thread Fabio Fantoni
Il 25/05/2015 17:07, Fabio Fantoni ha scritto: > Il 21/05/2015 10:34, Fabio Fantoni ha scritto: >> Il 13/05/2015 22:20, Francois Gouget ha scritto: >>> This is a followup on Jeremy White's concept patch to use GStreamer: >>> http://www.spinics.net/lists/spice-devel/msg10030.html >>> >>> There is st

Re: [Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-05-25 Thread Fabio Fantoni
Il 21/05/2015 10:34, Fabio Fantoni ha scritto: > Il 13/05/2015 22:20, Francois Gouget ha scritto: >> This is a followup on Jeremy White's concept patch to use GStreamer: >> http://www.spinics.net/lists/spice-devel/msg10030.html >> >> There is still a lot of work ahead but this patches series has al

Re: [Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-05-21 Thread Fabio Fantoni
Il 13/05/2015 22:20, Francois Gouget ha scritto: > This is a followup on Jeremy White's concept patch to use GStreamer: > http://www.spinics.net/lists/spice-devel/msg10030.html > > There is still a lot of work ahead but this patches series has all the > infrastructure so we're sending it out to ve

[Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-05-13 Thread Francois Gouget
This is a followup on Jeremy White's concept patch to use GStreamer: http://www.spinics.net/lists/spice-devel/msg10030.html There is still a lot of work ahead but this patches series has all the infrastructure so we're sending it out to verify that it looks ok. First, with this patch series: 1