Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 912703

Summary: Resolution changing loop
Product: Red Hat Enterprise Virtualization Manager Reporter: Tomas Jamrisko <tjamrisk>
Component: mingw-virt-viewerAssignee: Marc-Andre Lureau <marcandre.lureau>
Status: CLOSED ERRATA QA Contact: Desktop QE <desktop-qa-list>
Severity: high Docs Contact:
Priority: high    
Version: 3.2.0CC: acathrow, bsanford, cfergeau, dblechte, jbiddle, mbarta, mkrcmari, pvine, sgrinber
Target Milestone: ---Keywords: Regression
Target Release: 3.2.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: mingw-virt-viewer-0.5.3-21.el6ev Doc Type: Bug Fix
Doc Text:
Previously, resizing multiple monitor client windows resulted in a "loop" where the the resolution would endlessly resize until the window was closed. The monitors are now sorted using a stable algorithm, preventing the loop error.
 Story Points: ---
Clone Of: Environment:
Last Closed: 2013-06-10 20:02:46 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
vdagent log
none
spice dll without shuffling monitors
none
spice dll x64 without shuffling monitors none

Description Tomas Jamrisko 2013-02-19 13:16:32 UTC 
Description of problem:
Opening two displays on a RHEL6.4 guest can result in a resolution changing loop as resolution of open windows get swapped somewhere along the way, which invokes another resize event... 

Version-Release number of selected component (if applicable):
mingw-virt-viewer-0.5.3-20

How reproducible:
Always

Steps to Reproduce:
1. Connect to a RHEL6.4 VM 
2. open second window
3. Try changing resolution on one of the windows by dragging 
  
Actual results:
Either both resolutions will match, or it will keep changing and swapping resolutions until interrupted (by dragging a window and preventing
Comment 1 Tomas Jamrisko 2013-02-19 13:36:29 UTC 
Created attachment 699457 [details]
vdagent log
Comment 2 Marc-Andre Lureau 2013-02-19 14:07:18 UTC 
Created attachment 699462 [details]
spice dll without shuffling monitors
Comment 3 Marc-Andre Lureau 2013-02-19 14:31:58 UTC 
Created attachment 699470 [details]
spice dll x64 without shuffling monitors
Comment 4 Marc-Andre Lureau 2013-02-19 15:06:30 UTC 
Patch from Hans avoid uses monitor coordinates, which removes the need for stable sort.

http://cgit.freedesktop.org/spice/spice-gtk/commit/?id=c62c5af6e6bb3144fa3952221a03db90feb9aeab

We still need to replace qsort with a stable sort though, otherwise swapping may happend if the windows share same coordinates
Comment 5 Marc-Andre Lureau 2013-02-20 13:14:02 UTC 
fixed already upstream, cherry-picked in dist-git
Comment 6 Bill Sanford 2013-02-21 17:11:41 UTC 
*** Bug 913604 has been marked as a duplicate of this bug. ***
Comment 7 Tomas Jamrisko 2013-02-22 08:39:16 UTC 
*** Bug 913603 has been marked as a duplicate of this bug. ***
Comment 12 errata-xmlrpc 2013-06-10 20:02:46 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHEA-2013-0889.html