[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] [RFC] LZ4 compression option for SPICE
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PATCH] [RFC] LZ4 compression option for SPICE |
Date: |
Thu, 08 Jan 2015 12:28:13 +0100 |
On Do, 2015-01-08 at 11:50 +0100, Javier Celaya wrote:
> Hello
>
> Recently, SPICE included the lz4 compression algorithm. This patch adds
> a command line option to select it. However, SPICE_IMAGE_COMPRESS_LZ4 did not
> exist before the commit that added this compression algorithm, so it should
> be
> guarded with conditional compilation. How do you think this should be done?
> Wait for the next stable version of spice-server and check for
> SPICE_SERVER_VERSION? Or add a specific flag?
Given that SPICE_IMAGE_COMPRESS_LZ4 is a enum not define (so you can't
just #ifdef SPICE_IMAGE_COMPRESS_LZ4) using SPICE_SERVER_VERSION looks
best to me.
cheers,
Gerd
- [Qemu-devel] [PATCH] [RFC] LZ4 compression option for SPICE, Javier Celaya, 2015/01/08
- Re: [Qemu-devel] [PATCH] [RFC] LZ4 compression option for SPICE,
Gerd Hoffmann <=
- Re: [Qemu-devel] [Spice-devel] [PATCH] [RFC] LZ4 compression option for SPICE, Fabio Fantoni, 2015/01/08
- Re: [Qemu-devel] [Spice-devel] [PATCH] [RFC] LZ4 compression option for SPICE, Christophe Fergeau, 2015/01/20
- Re: [Qemu-devel] [Spice-devel] [PATCH] [RFC] LZ4 compression option for SPICE, Javier Celaya, 2015/01/23
- Re: [Qemu-devel] [Spice-devel] [PATCH] [RFC] LZ4 compression option for SPICE, Eric Blake, 2015/01/23
- Re: [Qemu-devel] [Spice-devel] [PATCH] [RFC] LZ4 compression option for SPICE, Javier Celaya, 2015/01/26
- Re: [Qemu-devel] [Spice-devel] [PATCH] [RFC] LZ4 compression option for SPICE, Eric Blake, 2015/01/26
- Re: [Qemu-devel] [Spice-devel] [PATCH] [RFC] LZ4 compression option for SPICE, Markus Armbruster, 2015/01/27
- Re: [Qemu-devel] [Spice-devel] [PATCH] [RFC] LZ4 compression option for SPICE, Javier Celaya, 2015/01/28
- Re: [Qemu-devel] [Spice-devel] [PATCH] [RFC] LZ4 compression option for SPICE, Markus Armbruster, 2015/01/28