In order to measure compression time, you mean the time between jpeg_start_compress() and jpeg_finish_compress, or the time of encode_frame() function? <div class="gmail_quote">2011/7/5 Christophe Fergeau <<a href="mailto:cfergeau@redhat.com">cfergeau@redhat.com</a>> <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">I finally ran some performance test on these changes on a small theora clip, average compression time goes down from 17,7 ms per frame to 14 ms per frame which corresponds to a 20% improvement. Christophe <div><div class="h5"> On Tue, Jun 28, 2011 at 01:18:01PM +0200, Christophe Fergeau wrote: > Hi, > > Currently, the mjpeg encoder has to preprocess the images it gets from > spice-server to make sure the pixel order is the one libjpeg expects (RGB). > This means iterating over the whole images swapping pixels since the input > images are BGR or BGRX. > However, libjpeg-turbo, in addition to SIMD optimization to JPEG decoding, > also brings additional input colorspaces, including BGR and BGRX. This patch > series make use of these new colorspaces in the mjpeg encoder to avoid > the conversion step. > libjpeg-turbo use is optional and the code will still do the conversion > step if libjpeg-turbo wasn't present at compile time. Most of these patches > are refactoring, the one introducing the libjpeg-turbo code is the one > before the last. > > Christophe > > Christophe Fergeau (9): > red_worker: remove 2 empty functions > red_worker: factor pixel conversion code > mjpeg_encoder: rework output buffer allocation > red_worker: simplify red_rgb_to_24bpp prototype > mjpeg_encoder: add mjpeg_encoder_encode_scanline > mjpeg_encoder: add mjpeg_encoder_get_bytes_per_pixel > red_worker: use new mjpeg_encoder_encode_scanline API > mjpeg_encoder: use libjpeg-turbo extra colorspaces > mjpeg_encoder: remove unused functions > > server/mjpeg_encoder.c | 226 +++++++++++++++++++++++++++++++++++++++++------- > server/mjpeg_encoder.h | 10 ++- > server/red_worker.c | 178 +++----------------------------------- > 3 files changed, 213 insertions(+), 201 deletions(-) > > -- > 1.7.5.4 > > _______________________________________________ > Spice-devel mailing list > <a href="mailto:Spice-devel@lists.freedesktop.org">Spice-devel@lists.freedesktop.org</a> > <a href="http://lists.freedesktop.org/mailman/listinfo/spice-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/spice-devel</a> </div></div> _______________________________________________ Spice-devel mailing list <a href="mailto:Spice-devel@lists.freedesktop.org">Spice-devel@lists.freedesktop.org</a> <a href="http://lists.freedesktop.org/mailman/listinfo/spice-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/spice-devel</a> </blockquote></div>