Re: [PATCH 1/5] LZ4: Limit LZ4 to RGB formats

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



ACK (a note in the commit log explaining why we don't want to handle the
other formats would be useful).

On Thu, Jan 22, 2015 at 05:21:04PM +0100, Javier Celaya wrote:
> ---
>  server/red_worker.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/server/red_worker.c b/server/red_worker.c
> index a18987a..8f07ee9 100644
> --- a/server/red_worker.c
> +++ b/server/red_worker.c
> @@ -6678,6 +6678,7 @@ static inline int red_compress_image(DisplayChannelClient *dcc,
>          if (!glz) {
>  #ifdef USE_LZ4
>              if (image_compression == SPICE_IMAGE_COMPRESS_LZ4 &&
> +                bitmap_fmt_is_rgb(src->format) &&
>                  red_channel_client_test_remote_cap(&dcc->common.base,
>                          SPICE_DISPLAY_CAP_LZ4_COMPRESSION)) {
>                  ret = red_lz4_compress_image(dcc, dest, src, o_comp_data,
> @@ -8918,6 +8919,7 @@ static void red_marshall_image(RedChannelClient *rcc, SpiceMarshaller *m, ImageI
>          } else {
>  #ifdef USE_LZ4
>              if (comp_mode == SPICE_IMAGE_COMPRESS_LZ4 &&
> +                bitmap_fmt_is_rgb(bitmap.format) &&
>                  red_channel_client_test_remote_cap(&dcc->common.base,
>                          SPICE_DISPLAY_CAP_LZ4_COMPRESSION)) {
>                  comp_succeeded = red_lz4_compress_image(dcc, &red_image, &bitmap,
> -- 
> 1.9.3
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel

Attachment: pgpe1JyprOISP.pgp
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]