[Spice-devel] [PATCH v9 00/24] Add GStreamer support for video streaming

[Pavel Grunt]

Hi Francois,

I tried your patches with a small patch for the replay utility.
http://paste.fedoraproject.org/313167/33783291/

I used an old recording of a Win7 guest a tried to replay it:
./spice-server-replay -c "spicy -h localhost -p 6000 2>/tmp/log.txt" -p
6000 /tmp/win7record.spice -S 2 -v "gstreamer:mjpeg"

And it crashes: http://paste.fedoraproject.org/313166/33782911/

With "gstreamer:vp8" it hangs
With "gstreamer:h264" it hangs after a while and I see a few of:
((null):24420): Spice-Warning **: gstreamer-encoder.c:726:map_format:
The 8 format has not been tested yet

With "spice:mjpeg" it works as expected

Unsupported "spice:h264", "spice:vp8" crashes too.

Maybe my usage is wrong and/or the crashes are related to "timing" of
the replay utility. Have you seen the crash before?

Thanks,
Pavel

On Mon, 2016-01-18 at 12:03 +0100, Francois Gouget wrote:
> The source changed so here is the new revision of this patchset.
> 
> This patch series adds support for using GStreamer to encode and
> decode 
> the video streams, adding support for VP8 and h264 codecs. As before
> the 
> patches can also be grabbed from the repositories below:
> 
> spice:          https://github.com/fgouget/spice
> spice-gtk:      https://github.com/fgouget/spice-gtk
> xf86-video-qxl: https://github.com/fgouget/xf86-video-qxl
> spice-protocol: https://github.com/fgouget/spice-protocol
> 
> (there's also 'extras' branches with more experimental/future
> patches 
> for the curious)
> 
> 
> Changes from v8:
> * Rebased on the current source.
> * Adjusted the Spice protocol version requirement.
>  
> Changes from v7:
> * When the client has support for GStreamer video codecs it now 
>   checks the presence of each supported codec before advertising
> support 
>   for it.
> 
> Changes from v6:
> * configure.ac uses the new m4 macros to check for GStreamer support 
>   and the presence of the needed plugins.
> * It separates adding checks for the client codec support from 
>   specifying which codec to use in the server and from adding VP8 
>   support.
> 
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-replay-Add-an-option-to-change-video-codec.patch
Type: text/x-patch
Size: 2043 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20160121/e15be4ce/attachment.bin>