Re: [Spice-devel] [PATCH 00/13] Add GStreamer support for video streams (take 4)

2015-08-04 Thread Christophe Fergeau
Hey,

One comment regarding the subject line of this series, please pass -v4
to git format-patch. Appending (take 4) to the end makes it harder to
apply the patch with git am as this has to be manually stripped.

Christophe

On Tue, Jul 21, 2015 at 08:00:34PM +0200, Francois Gouget wrote:
 
 This follows up on the previous patch series with the addition of full 
 bit rate support. This brings the GStreamer video encoder to feature 
 parity with the builtin Spice MJPEG one so I think it's ready to be 
 committed.
 
 To summarize the changes since the last round:
  - The bit rate is automatically adjusted based on the network 
conditions, both down and up.
 
  - Zero-copy for the compressed output buffer.
 
  - Proper fallback in case the server and client have no video codec in 
common (which would only happen if the client does not support MJPEG 
streams).
 
 
 As for the previous round I'm only sending the patches needed for the 
 Spice server to limit the size of this series. The GStreamer MJPEG 
 encoder is fully compatible with existing clients so this should not 
 hinder testing. I will post a new spice-gtk patch series soon but in the 
 meantime one can fetch patches from GitHub to test the VP8 and h264 
 codecs. See the gst branch of 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-common:   https://github.com/fgouget/spice-common
 spice-protocol: https://github.com/fgouget/spice-protocol
 
 (there's also 'extras' branches with more experimental/future patches 
 for the curious)
 
 For spice-html5 and QEMU one would have to refer to the patches posted 
 previously on spice-devel. They should still work with this series.
 
 Let me know if there are changes that are needed for inclusion. 
 
 
 -- 
 Francois Gouget fgou...@codeweavers.com
 ___
 Spice-devel mailing list
 Spice-devel@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/spice-devel


pgpEFf5JnClOD.pgp
Description: PGP signature
___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel


[Spice-devel] [PATCH 00/13] Add GStreamer support for video streams (take 4)

2015-07-21 Thread Francois Gouget

This follows up on the previous patch series with the addition of full 
bit rate support. This brings the GStreamer video encoder to feature 
parity with the builtin Spice MJPEG one so I think it's ready to be 
committed.

To summarize the changes since the last round:
 - The bit rate is automatically adjusted based on the network 
   conditions, both down and up.

 - Zero-copy for the compressed output buffer.

 - Proper fallback in case the server and client have no video codec in 
   common (which would only happen if the client does not support MJPEG 
   streams).


As for the previous round I'm only sending the patches needed for the 
Spice server to limit the size of this series. The GStreamer MJPEG 
encoder is fully compatible with existing clients so this should not 
hinder testing. I will post a new spice-gtk patch series soon but in the 
meantime one can fetch patches from GitHub to test the VP8 and h264 
codecs. See the gst branch of 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-common:   https://github.com/fgouget/spice-common
spice-protocol: https://github.com/fgouget/spice-protocol

(there's also 'extras' branches with more experimental/future patches 
for the curious)

For spice-html5 and QEMU one would have to refer to the patches posted 
previously on spice-devel. They should still work with this series.

Let me know if there are changes that are needed for inclusion. 


-- 
Francois Gouget fgou...@codeweavers.com
___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel