Bug 1261435

Summary: The resolution is set to incorrect value after clicking Revert Setting button.
Product: Red Hat Enterprise Linux 7 Reporter: Xiaodai Wang <xiaodwan>
Component: mutterAssignee: Florian Müllner <fmuellner>
Status: CLOSED WONTFIX QA Contact: Desktop QE <desktop-qa-list>
Severity: medium Docs Contact:
Priority: medium    
Version: 7.2CC: juzhou, mclasen, mxie, mzhan, rstrode, tzheng, xiaodwan
Target Milestone: rc   
Target Release: ---   
Hardware: x86_64   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-09-11 17:58:27 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
journalctl logs
none
virt-viewer debug logs none

Description Xiaodai Wang 2015-09-09 10:25:42 UTC 
Description of problem:
The resolution is set to incorrect value after clicking Revert Setting button.

Version-Release number of selected component (if applicable):
virt-viewer-2.0-6.el7.x86_64
spice-gtk3-0.26-5.el7.x86_64
spice-xpi-2.8-8.el7.x86_64
spice-glib-0.26-5.el7.x86_64
spice-gtk-0.26-5.el7.x86_64
spice-protocol-0.12.6-2.el7.noarch
spice-gtk-python-0.26-5.el7.x86_64
spice-vdagent-0.14.0-10.el7.x86_64
spice-server-0.12.4-13.el7.x86_64

Guest version:
http://download.englab.nay.redhat.com/pub/rhel/rel-eng/RHEL-7.2-20150904.0/compose/Server/x86_64/

How reproducible:
100%

Steps to Reproduce:
1. Open the guest by virt-viewer in fullscreen mode, check the resolution is 1680x1050.
#virt-viewer rhel7.2 -f
2. set the resolution to 1440x900, click Apply->Revert Settings, check the resolution again. it is set to 1680x1021. This is not same as previous.
3. set the resolution to 1440x900 again, click Apply->Revert Settings, check the resolution. it is set to 1440x900. 

Actual results:
See the steps.

Expected results:
It should always set to correct resolution after click "Revert Settings".

Additional info:
1. I tested it with my rhel7.2 host(same path as the guest above) by changing the resolution and revert it back, it has same problem (but not 100% reproduce).
2. I'm not sure which component it should be, if I'm wrong, please correct me.
Comment 2 Ray Strode [halfline] 2015-09-09 13:17:18 UTC 
So I believe virt-viewer has some logic to automatically resize the window, to the guest resolution and I believe the guest vdagent some logic to change the guest resolution based on the window size. If you run:

$ pkill -f spice-vdagent

before you try your guest experiments, do the results change?

If you're seeing this in the host there may be a mutter bug too.

can you post the output of

# sudo journalctl -b -a

after the problem happens?

I'd like the entire log, but in particular I wonder if you're seeing this in the log

Configuring CRTC %d with mode %d (%d x %d @ %f) at position %d, %d and transfrom %u failed
Comment 3 Ray Strode [halfline] 2015-09-09 14:09:35 UTC 
also when you run virt-viewer can you run it from the command line and report any output that is sent to stderr (if any)
Comment 4 Xiaodai Wang 2015-09-10 06:24:38 UTC 
(In reply to Ray Strode [halfline] from comment #2)
> So I believe virt-viewer has some logic to automatically resize the window,
> to the guest resolution and I believe the guest vdagent some logic to change
> the guest resolution based on the window size. If you run:
> 
> $ pkill -f spice-vdagent
> 
> before you try your guest experiments, do the results change?
> 
> If you're seeing this in the host there may be a mutter bug too.
> 
> can you post the output of
> 
> # sudo journalctl -b -a
> 
> after the problem happens?
> 
> I'd like the entire log, but in particular I wonder if you're seeing this in
> the log
> 
> Configuring CRTC %d with mode %d (%d x %d @ %f) at position %d, %d and
> transfrom %u failed

I can't reproduce it with my host now, but still can reproduce it no matter spice-vdagnet is on or off.
I didn't see logs "Configuring CRTC %d with mode %d (%d x %d @ %f) at position %d, %d" by journalctl. I attached the journalctl and virt-viewer logs.
Comment 5 Xiaodai Wang 2015-09-10 06:25:18 UTC 
Created attachment 1072028 [details]
journalctl logs
Comment 6 Xiaodai Wang 2015-09-10 06:25:48 UTC 
Created attachment 1072029 [details]
virt-viewer debug logs