Re: [PATCH] display: Advertise preferrred compression cap

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

 



On Wed, Sep 16, 2015 at 07:19:51AM +0200, Pavel Grunt wrote:
> Hi Christophe,
> 
> On Tue, 2015-09-15 at 18:08 +0200, Christophe Fergeau wrote:
> > The patches adding a way for the client to set its preferred compression
> > method added a new capability so that the server can indicate support
> > for this feature. However, spice-server was not setting this capability
> > on its display channel, which means clients are not going to try to send
> > 'preferred-compression' messages even though the user request it.
> > ---
> >  server/red_worker.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/server/red_worker.c b/server/red_worker.c
> > index e2feb23..0b22f0d 100644
> > --- a/server/red_worker.c
> > +++ b/server/red_worker.c
> > @@ -10520,6 +10520,7 @@ static RedChannel *__new_channel(RedWorker *worker,
> > int size, uint32_t channel_t
> >                                          handle_parsed,
> >                                          &channel_cbs,
> >                                          migration_flags);
> > +    red_channel_set_cap(channel, SPICE_DISPLAY_CAP_PREF_COMPRESSION);
> 
> Why is it not set in server/red_dispatcher.c, with other display channel caps ?

Ah I did not notice these ones, I'll send a v2 (these look out of place
in red_dispatcher.c, but that's a different matter :p )

Thanks,

Christophe

Attachment: pgpf_ngqwwiPE.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]