Bug 11703 - [PATCH] RandR1.2: Set the crtc in the outputs before delivering notifications
Summary: [PATCH] RandR1.2: Set the crtc in the outputs before delivering notifications
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Server/General (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: Xorg Project Team
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2007-07-23 10:22 UTC by Gustavo Pichorim Boiko
Modified: 2007-08-03 07:43 UTC (History)
0 users

See Also:
i915 platform:
i915 features:

Attachments
Set the crtc in the output before notifications are delivered (944 bytes, patch)
2007-07-23 10:26 UTC, Gustavo Pichorim Boiko 		no flags Details | Splinter Review
Set the crtc in the output before notifications are delivered (3.26 KB, patch)
2007-07-23 14:28 UTC, Gustavo Pichorim Boiko 		no flags Details | Splinter Review
Show Obsolete (1) View All

Description Gustavo Pichorim Boiko 2007-07-23 10:22:38 UTC 
The output's randr_output information was not getting the crtc updated on crtc changes causing the clients to receive output change notification events containing outdated information.

The attached patch fixes this problem by setting the output's randr crtc information right when the crtc change occurs.
Comment 1 Gustavo Pichorim Boiko 2007-07-23 10:26:48 UTC 
Created attachment 10848 [details] [review]
Set the crtc in the output before notifications are delivered
Comment 2 Gustavo Pichorim Boiko 2007-07-23 14:28:09 UTC 
Created attachment 10859 [details] [review]
Set the crtc in the output before notifications are delivered

New version of the patch following suggestions from Keith Packard
Comment 3 Gustavo Pichorim Boiko 2007-07-23 16:30:30 UTC 
Patch applied, closing bug.
Comment 4 Gustavo Pichorim Boiko 2007-08-03 07:43:58 UTC 
Just as an update: this fix also requires the already applied commits:
8d230319040f0a7f72231da2bf5ec97dc3612e21
aec0d06469a2fa7440fdd5ee03dc256a68704e77

It might also be worth looking at #11830 which contains some other interesting patches.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.