Bug 1666735 - Screen size change is not reflected in 'logical screen size'
Summary: Screen size change is not reflected in 'logical screen size'
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: xfwm4
Version: 29
Hardware: x86_64
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Mukundan Ragavan
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-01-16 13:43 UTC by Roel van de Kraats
Modified: 2019-05-21 10:43 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-05-18 16:42:25 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Output of "lspci | grep VGA" (58 bytes, text/plain)
2019-01-25 21:49 UTC, Roel van de Kraats
no flags Details
Output of "xrandr" (1.06 KB, text/plain)
2019-01-25 21:49 UTC, Roel van de Kraats
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Xfce 15085 0 None None None 2019-05-27 08:24:25 UTC

Description Roel van de Kraats 2019-01-16 13:43:05 UTC
Description of problem:
Running Fedora 29 xfce + xfwm4 in VMware Workstation. A change of the VM screen size is handled properly by most components, except that 'maximizing' a window changes the window size to the previous screen size. This didn't happen before upgrading from F27 to F29.

xfwm4 complains twice with this warning when the VM screen size is changed:

(xfwm4:55331): xfwm4-WARNING **: 14:31:44.773: output size (1902x1045) and logical screen size (1920x1200) do not match

Changing the screen size again shows that the logical screen size was in the end updated after all:

(xfwm4:55331): xfwm4-WARNING **: 14:31:50.728: output size (1837x987) and logical screen size (1902x1045) do not match


Adding some debug output reveals that the monitors_changed_cb() and the size_changed_cb() event handlers both call myScreenComputeSize(), which leads to the warning being show twice.

It looks like WidthOfScreen() and HeightOfScreen() as called from myScreenComputeSize() still give the old values at the moment of handling the events. Perhaps the order in which Xlib (which I believe provides these functions) and gdk/gtk are updated is notified of a screen size change may not be depended on?

Version-Release number of selected component (if applicable):
Fedora 29
xfwm4 4.13.1
VMware Workstation 12.5.9
MS Windows 10

How reproducible:
Easily on my set-up...

Steps to Reproduce:
1. Run xfce in a VMware Workstation VM
2. Change the VM screen size (by resizing the window in which it is running)
3. Maximize a window handled by xfwm

Actual results:
Window is not full screen

Expected results:
Window is full screen

Additional info:

Comment 1 Mukundan Ragavan 2019-01-16 23:30:11 UTC
That's nice work troubleshooting the issue. Would you be able to file this upstream or would you prefer that I do?

Comment 2 Roel van de Kraats 2019-01-17 08:04:50 UTC
Assuming you know your way upstream much better than I do, I'd really appreciate if you could do so. Thanks in advance!

Comment 3 Roel van de Kraats 2019-01-24 08:54:01 UTC
Since you apparently didn't have time yet to file an upstream bug report, I figured out where 'upstream' actually is, and filed https://bugzilla.xfce.org/show_bug.cgi?id=15085
I hope I did so similar to how you would have done it.

Comment 4 Mukundan Ragavan 2019-01-24 23:03:04 UTC
Thanks Roel. My apologies for the delay.

Comment 5 Raphael Groner 2019-01-25 19:30:07 UTC
Did you verify it's not an issue with graphics? Please be aware it's a virtualized environment as noted in original report.

Please give the output of the command "lspci |grep VGA" executed inside a terminal (e.g. open xfce4-terminal and enter this command line without the quotes) in your VM.

Comment 6 Raphael Groner 2019-01-25 19:31:12 UTC
Besides, you should attach the output of xrandr command, executed inside a terminal in your VM.

Comment 7 Roel van de Kraats 2019-01-25 21:49:08 UTC
Created attachment 1523598 [details]
Output of "lspci | grep VGA"

Comment 8 Roel van de Kraats 2019-01-25 21:49:40 UTC
Created attachment 1523599 [details]
Output of "xrandr"

Comment 9 Roel van de Kraats 2019-01-25 21:53:26 UTC
Raphael, please see the attached command outputs. I'd be surprised if some graphics issue could cause this single 'deferred logical screen size' issue, while keeping everything else working fine. Note that e.g. the desktop background is immediately properly updated on a screen size change.

Comment 10 Raphael Groner 2019-01-25 22:14:49 UTC
Thanks for the additional information.

The threads are full about "VGA compatible controller: VMware SVGA II Adapter". Please use your favored search engine. In the hope there's a working solution somewhere.

Feel free to report again if there's still any issue. Be aware we can't support VMware related stuff because it's proprietary software and we've to verify there's really no valid issue from the other and evil side.

Comment 11 Raphael Groner 2019-01-25 22:18:16 UTC
Sorry but I can't give any better help. E.g. did you read through and try the suggestions of following links?

https://communities.vmware.com/thread/590991
https://forums.linuxmint.com/viewtopic.php?t=188563

Comment 12 Roel van de Kraats 2019-01-25 23:34:47 UTC
So far I wasn't able to find anything useful, but I'll continue searching. Note that the first link is about pass-through of a 2nd video adapter, which is not applicable for my situation, and the 2nd link only suggests running Mint natively. Perhaps I should try that ;-)

Please also note that this issue didn't occur with F27 (running on the same VM, with the same VMware Workstation version), so it could be considered a regression. And AFAIK, the vmware video driver is open source; I'm also using the open-vm-tools package(s) instead of the 'official' VMware tools.

I'm currently using an ugly workaround where myScreenComputeSize() is called from handleConfigureNotify(), which is called quite regularly. It is called quickly after size_changed_cb(), but with up-to-date values for WidthOfScreen() and HeightOfScreen(). This way, at least the internal logical screen size state is updated, which allows for (manual) re-maximizing a window.

Comment 13 Raphael Groner 2019-01-26 06:19:29 UTC
Great work! We should try to file your patch to upstream.

Comment 14 Roel van de Kraats 2019-01-26 19:23:37 UTC
See upstream ticket https://bugzilla.xfce.org/show_bug.cgi?id=15085 attachment.

Comment 15 Raphael Groner 2019-01-27 06:01:40 UTC
(In reply to Roel van de Kraats from comment #14)
> See upstream ticket https://bugzilla.xfce.org/show_bug.cgi?id=15085
> attachment.

OK, thanks. Hopefully there's a reply from upstream soonish.

Comment 16 Roel van de Kraats 2019-04-16 19:50:54 UTC
See the fix in the upstream ticket. I successfully tested the fix against the Fedora xfwm4-4.13.1-4 package on my F29 system.

Comment 17 Mukundan Ragavan 2019-04-22 20:52:53 UTC
I have applied the upstream patch. Can you test this build please?

https://koji.fedoraproject.org/koji/taskinfo?taskID=34346797

Comment 18 Roel van de Kraats 2019-04-23 20:10:38 UTC
Thanks Mukundan!

xfwm4-4.13.1-7.fc29.x86_64.rpm is running fine on my system (the one mentioned in the bug report).

Comment 19 Mukundan Ragavan 2019-04-23 22:01:42 UTC
Thanks for testing. I will push this as an update.

Comment 20 Fedora Update System 2019-05-01 22:15:19 UTC
xfwm4-4.13.1-8.fc30 has been submitted as an update to Fedora 30. https://bodhi.fedoraproject.org/updates/FEDORA-2019-fa398c683c

Comment 21 Fedora Update System 2019-05-01 22:15:23 UTC
xfwm4-4.13.1-8.fc29 has been submitted as an update to Fedora 29. https://bodhi.fedoraproject.org/updates/FEDORA-2019-f1b877216d

Comment 22 Fedora Update System 2019-05-02 02:38:32 UTC
xfwm4-4.13.1-8.fc29 has been pushed to the Fedora 29 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2019-f1b877216d

Comment 23 Fedora Update System 2019-05-02 02:59:46 UTC
xfwm4-4.13.1-8.fc30 has been pushed to the Fedora 30 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2019-fa398c683c

Comment 24 Fedora Update System 2019-05-17 01:04:00 UTC
xfwm4-4.13.1-8.fc30 has been pushed to the Fedora 30 stable repository. If problems still persist, please make note of it in this bug report.

Comment 25 Fedora Update System 2019-05-17 03:17:04 UTC
xfwm4-4.13.1-8.fc29 has been pushed to the Fedora 29 stable repository. If problems still persist, please make note of it in this bug report.

Comment 26 Jan Pazdziora 2019-05-21 10:43:42 UTC
For the record, the new build also seems to have fixed bug 1656444, thanks.


Note You need to log in before you can comment on or make changes to this bug.