Hi,
On Mon, Jul 20, 2015 at 06:35:28PM -0400, Sandy Stutsman wrote:
> Hi Again.
> >>>>>> diff --git a/src/spice-widget.c b/src/spice-widget.c
> >>>>>> index 59f9792..3ec2e65 100644
> >>>>>> --- a/src/spice-widget.c
> >>>>>> +++ b/src/spice-widget.c
> >>>>>> @@ -293,7 +293,12 @@ static void update_monitor_area(SpiceDisplay
> >>>>>> *display)
> >>>>>> goto whole;
> >>>>>> }
> >>>>>>
> >>>>>> - update_area(display, c->x, c->y, c->width, c->height);
> >>>>>> + /* If only one head on this monitor, update the whole area */
> >>>>>> + if(monitors->len == 1) {
> >>>>>> + update_area(display, 0, 0, c->width, c->height);
> >>>>>> + } else {
> >>>>>> + update_area(display, c->x, c->y, c->width, c->height);
> >>>>>> + }
> I did a little more testing today. I did see the "Waiting for display ...2" message whenever
> I started the remote-viewer with 1 monitor and tried to add a second. It happened both
> with and without the patch. If I added the second monitor, closed the viewer and
> re-opened it, the second monitor will display just fine.
>
Great, so it isn't really related to this patch.
> The good news is that when the this patch is paired with the monitor config qxl patch,
> https://bugzilla.redhat.com/show_bug.cgi?id=1202419, I don't see the problem at all.
Right, this is the patch I was missing then:
http://cgit.freedesktop.org/spice/win32/qxl/commit/?id=ed37b635188893719c59d71c031feddd01408f36
Do you have an qxl installer with the above patch for me to test it locally?
I'm asking because Pavel verified a problem in the following test:
1-) Connected with remove-viewer and both displays enabled
2-) Disable display 1
3-) Disconnect
4-) Connect again
R: Only Display 2 is enabled and it isn't possible to enable Display 1
If the above is also fixed with newer qxl them I don't see problems with
this patch as the main concern was related to Linux guest and I wasn't
able to find any problem...
Best,
Victor Toso
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel