[Desktop-packages] [Bug 135738] Re: DisplaySize setting not honored
** Changed in: xorg-server Status: Confirmed => Unknown -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to xorg-server in Ubuntu. https://bugs.launchpad.net/bugs/135738 Title: DisplaySize setting not honored Status in X.Org X server: Unknown Status in xorg-server package in Ubuntu: Fix Released Bug description: Binary package hint: xorg I have this in my xorg.conf: Section "Monitor" Identifier "VG2230wm" Option "DPMS" DisplaySize 444 277 HorizSync 24-82 VertRefresh 50-85 EndSection However, xdpyinfo reports that it's not using it: screen #0: dimensions:1680x1050 pixels (474x303 millimeters) This was working in 7.04, but broke when I upgraded to gusty somewhere around tribe 5. To manage notifications about this bug go to: https://bugs.launchpad.net/xorg-server/+bug/135738/+subscriptions -- Mailing list: https://launchpad.net/~desktop-packages Post to : desktop-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~desktop-packages More help : https://help.launchpad.net/ListHelp
[Desktop-packages] [Bug 135738]
-- GitLab Migration Automatic Message -- This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/xorg/xserver/issues/246. -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to xorg-server in Ubuntu. https://bugs.launchpad.net/bugs/135738 Title: DisplaySize setting not honored Status in X.Org X server: Unknown Status in xorg-server package in Ubuntu: Fix Released Bug description: Binary package hint: xorg I have this in my xorg.conf: Section "Monitor" Identifier "VG2230wm" Option "DPMS" DisplaySize 444 277 HorizSync 24-82 VertRefresh 50-85 EndSection However, xdpyinfo reports that it's not using it: screen #0: dimensions:1680x1050 pixels (474x303 millimeters) This was working in 7.04, but broke when I upgraded to gusty somewhere around tribe 5. To manage notifications about this bug go to: https://bugs.launchpad.net/xorg-server/+bug/135738/+subscriptions -- Mailing list: https://launchpad.net/~desktop-packages Post to : desktop-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~desktop-packages More help : https://help.launchpad.net/ListHelp
[Desktop-packages] [Bug 135738]
Created attachment 88557 patch against current git master to respect DisplaySize (20131103) This should be sufficient to make it work. Always uses user supplied DisplaySize in preference to detected size if specified. Also reports if monitor features and detailed timings report differing sizes, this is/was just to help debug and can be dropped. -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to xorg-server in Ubuntu. https://bugs.launchpad.net/bugs/135738 Title: DisplaySize setting not honored Status in X.Org X server: Confirmed Status in “xorg-server” package in Ubuntu: Fix Released Bug description: Binary package hint: xorg I have this in my xorg.conf: Section Monitor Identifier VG2230wm Option DPMS DisplaySize 444 277 HorizSync 24-82 VertRefresh 50-85 EndSection However, xdpyinfo reports that it's not using it: screen #0: dimensions:1680x1050 pixels (474x303 millimeters) This was working in 7.04, but broke when I upgraded to gusty somewhere around tribe 5. To manage notifications about this bug go to: https://bugs.launchpad.net/xorg-server/+bug/135738/+subscriptions -- Mailing list: https://launchpad.net/~desktop-packages Post to : desktop-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~desktop-packages More help : https://help.launchpad.net/ListHelp
[Desktop-packages] [Bug 135738]
Just to add for clarity, it shouldn't be surprising that I had to remove: hw/xfree86/modes/xf86Crtc.c: handle_detailed_physical_size() ... p-output-mm_width = det_mon-section.d_timings.h_size; p-output-mm_height = det_mon-section.d_timings.v_size; since it seems to be called on each DDC probe and not just when the connected output device has changed. This should at least be conditional on DisplaySize not being set. I don't think this has really been noticed since the core DPI is no longer tied to the output size/resolution, but any app wanting to make use of the actual display characteristics via randr will potentially get some very wrong values! -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to xorg-server in Ubuntu. https://bugs.launchpad.net/bugs/135738 Title: DisplaySize setting not honored Status in X.Org X server: Confirmed Status in “xorg-server” package in Ubuntu: Fix Released Bug description: Binary package hint: xorg I have this in my xorg.conf: Section Monitor Identifier VG2230wm Option DPMS DisplaySize 444 277 HorizSync 24-82 VertRefresh 50-85 EndSection However, xdpyinfo reports that it's not using it: screen #0: dimensions:1680x1050 pixels (474x303 millimeters) This was working in 7.04, but broke when I upgraded to gusty somewhere around tribe 5. To manage notifications about this bug go to: https://bugs.launchpad.net/xorg-server/+bug/135738/+subscriptions -- Mailing list: https://launchpad.net/~desktop-packages Post to : desktop-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~desktop-packages More help : https://help.launchpad.net/ListHelp
[Desktop-packages] [Bug 135738]
There are still bugs here with the current git version. in hw/xfree86/modes/xf86Crtc.c: handle_detailed_physical_size() p-output-mm_width = det_mon-section.d_timings.h_size; p-output-mm_height = det_mon-section.d_timings.v_size; This unconditionally sets mm width and height from the detailed timings. hw/xfree86/ddc/interpret_edid.c: encode_aspect_ratio() This function contains a quirk which determines whether the mm width and height in the detailed timings presented by the monitor are actually the aspect ratio. If so: m-features.hsize = m-features.vsize = 0; This results in a message like this in the log: Quirked EDID physical size to 0x0 cm Back to hw/xfree86/modes/xf86Crtc.c: xf86OutputSetEDID() In this function, if the mm width and height haven't been determined by the DDC detailed timings probe (with no check whether they are valid), they are then set to values in the max size field (features.hsize and features.vsize) as long as it wasn't quirked to 0x0 due to the detailed timings being invalid... ...too late now! Of course, if no information is available, the xrandr output dimensions needs to be set to something, whether that's 0x0 or 160x90 mm doesn't matter so much, either way it's not reflecting reality. However, all this convolution completely overrides the DisplaySize value the user may have set. Since it's sourced in hw/xfree86/modes/xf86RandR12.c, but then overridden during DDC mode probes it makes no difference. -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to xorg-server in Ubuntu. https://bugs.launchpad.net/bugs/135738 Title: DisplaySize setting not honored Status in X.Org X server: Confirmed Status in “xorg-server” package in Ubuntu: Fix Released Bug description: Binary package hint: xorg I have this in my xorg.conf: Section Monitor Identifier VG2230wm Option DPMS DisplaySize 444 277 HorizSync 24-82 VertRefresh 50-85 EndSection However, xdpyinfo reports that it's not using it: screen #0: dimensions:1680x1050 pixels (474x303 millimeters) This was working in 7.04, but broke when I upgraded to gusty somewhere around tribe 5. To manage notifications about this bug go to: https://bugs.launchpad.net/xorg-server/+bug/135738/+subscriptions -- Mailing list: https://launchpad.net/~desktop-packages Post to : desktop-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~desktop-packages More help : https://help.launchpad.net/ListHelp
[Desktop-packages] [Bug 135738]
Created attachment 87022 patch against current git master to respect DisplaySize By setting the output mm sizes to the user values from DisplaySize after the DDC probe but before checking whether they are set and falling back to the max size field gives priority to the user config. (copied from hw/xfree86/modes/xf86RandR12.c) -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to xorg-server in Ubuntu. https://bugs.launchpad.net/bugs/135738 Title: DisplaySize setting not honored Status in X.Org X server: Confirmed Status in “xorg-server” package in Ubuntu: Fix Released Bug description: Binary package hint: xorg I have this in my xorg.conf: Section Monitor Identifier VG2230wm Option DPMS DisplaySize 444 277 HorizSync 24-82 VertRefresh 50-85 EndSection However, xdpyinfo reports that it's not using it: screen #0: dimensions:1680x1050 pixels (474x303 millimeters) This was working in 7.04, but broke when I upgraded to gusty somewhere around tribe 5. To manage notifications about this bug go to: https://bugs.launchpad.net/xorg-server/+bug/135738/+subscriptions -- Mailing list: https://launchpad.net/~desktop-packages Post to : desktop-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~desktop-packages More help : https://help.launchpad.net/ListHelp
[Desktop-packages] [Bug 135738]
Hoped that would be enough, but it seems there's also need to address the issue of unconditionally overriding the value in the handle_detailed_physical_size(). I've just removed the code here (which works with my test case), but that could well break something else... :/ -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to xorg-server in Ubuntu. https://bugs.launchpad.net/bugs/135738 Title: DisplaySize setting not honored Status in X.Org X server: Confirmed Status in “xorg-server” package in Ubuntu: Fix Released Bug description: Binary package hint: xorg I have this in my xorg.conf: Section Monitor Identifier VG2230wm Option DPMS DisplaySize 444 277 HorizSync 24-82 VertRefresh 50-85 EndSection However, xdpyinfo reports that it's not using it: screen #0: dimensions:1680x1050 pixels (474x303 millimeters) This was working in 7.04, but broke when I upgraded to gusty somewhere around tribe 5. To manage notifications about this bug go to: https://bugs.launchpad.net/xorg-server/+bug/135738/+subscriptions -- Mailing list: https://launchpad.net/~desktop-packages Post to : desktop-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~desktop-packages More help : https://help.launchpad.net/ListHelp
