Re: [PATCH spice-server v3] stream-channel: Tell client we are just streaming data

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

 



Frediano Ziglio writes:

> This give an hint to client which can optimise rendering.
>
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
>  server/stream-channel.c | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> Changes since v2:
> - remove conditional code.

OK, now I'm confused :-)

Does that mean that:

a) you simply want this to be independent from your identity macros patch?

b) you feel it's OK (or maybe even good) to break the build if the wrong
   protocol package is included?

>
> diff --git a/server/stream-channel.c b/server/stream-channel.c
> index c7ca0206..568ceac8 100644
> --- a/server/stream-channel.c
> +++ b/server/stream-channel.c
> @@ -203,6 +203,13 @@ stream_channel_send_item(RedChannelClient *rcc, RedPipeItem *pipe_item)
>              channel->width, channel->height,
>              SPICE_SURFACE_FMT_32_xRGB, SPICE_SURFACE_FLAGS_PRIMARY
>          };
> +
> +        // give an hint to client that we are sending just streaming
> +        // see spice.proto for capability check here
> +        if (red_channel_client_test_remote_cap(rcc, SPICE_DISPLAY_CAP_MULTI_CODEC)) {
> +            surface_create.flags |= SPICE_SURFACE_FLAGS_STREAMING_MODE;
> +        }
> +
>          spice_marshall_msg_display_surface_create(m, &surface_create);
>          break;
>      }


--
Cheers,
Christophe de Dinechin (IRC c3d)
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://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]