[Spice-devel] [PATCH spice-gtk 2/2] Send monitor config if guest supports sparse monitors

Pavel Grunt pgrunt at redhat.com
Thu Jun 25 06:43:02 PDT 2015


On Fri, 2015-06-19 at 12:51 -0400, Marc-André Lureau wrote:
> hi
> 
> ----- Original Message -----
> > It is ok to send monitor configuration even for displays which are not
> > disabled or enabled. Because the guest having support for the sparse
> > monitor configuration (VD_AGENT_CAP_SPARSE_MONITORS_CONFIG) will not
> > resize the monitor which is disabled / undefined.
> 
> I trust you on this one :) ack
> 
Thank you, pushed.

> > 
> > Fixes:
> > https://bugs.freedesktop.org/show_bug.cgi?id=90914
> > ---
> >  src/channel-main.c | 17 +++++++++--------
> >  1 file changed, 9 insertions(+), 8 deletions(-)
> > 
> > diff --git a/src/channel-main.c b/src/channel-main.c
> > index 4e8046e..c7ba773 100644
> > --- a/src/channel-main.c
> > +++ b/src/channel-main.c
> > @@ -1428,14 +1428,15 @@ static gboolean timer_set_display(gpointer data)
> >  
> >      session = spice_channel_get_session(SPICE_CHANNEL(channel));
> >  
> > -    /* ensure we have an explicit monitor configuration at least for
> > -       number of display channels */
> > -    for (i = 0; i < spice_session_get_n_display_channels(session); i++)
> > -        if (c->display[i].display_state == DISPLAY_UNDEFINED) {
> > -            SPICE_DEBUG("Not sending monitors config, missing monitors");
> > -            return FALSE;
> > -        }
> > -
> > +    if (!spice_main_agent_test_capability(channel,
> > VD_AGENT_CAP_SPARSE_MONITORS_CONFIG)) {
> > +        /* ensure we have an explicit monitor configuration at least for
> > +           number of display channels */
> > +        for (i = 0; i < spice_session_get_n_display_channels(session); i++)
> > +            if (c->display[i].display_state == DISPLAY_UNDEFINED) {
> > +                SPICE_DEBUG("Not sending monitors config, missing
> > monitors");
> > +                return FALSE;
> > +            }
> > +    }
> >      spice_main_send_monitor_config(channel);
> >  
> >      return FALSE;
> > --
> > 2.4.4
> > 
> > _______________________________________________
> > Spice-devel mailing list
> > Spice-devel at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/spice-devel
> > 


More information about the Spice-devel mailing list