Bug 12242 - allow greater yres_virtual for fbdev devices
Summary: allow greater yres_virtual for fbdev devices
Status: CLOSED 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:
Depends on:
Blocks: xorg-7.3
  Show dependency treegraph
 
Reported: 2007-08-31 16:18 UTC by Alan Hourihane
Modified: 2007-09-05 01:55 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments

Description Alan Hourihane 2007-08-31 16:18:43 UTC
Can someone cherry-pick this ....

87495fc7064d5e0a7575a0713b6895a4172df0fa

for server-1.4-branch ??
Comment 1 Daniel Stone 2007-08-31 18:28:05 UTC
can you please explain why? making the mode equal function not actually test if the modes are equal, doesn't seem like the kind of obvious change that needs zero explanation.  if you need to fudge (which this function seems to be designed to prevent), why not use a fudge factor, instead of taking 1280x768 for 1024x768?
Comment 2 Eric Anholt 2007-08-31 21:08:11 UTC
10101 is 7.4, which you've already committed your change to master for.
Comment 3 Alan Hourihane 2007-09-01 15:23:23 UTC
ah yes, i forgot that it's 1.3 xserver that was released after 7.2 and there was never a 7.3 release.
Comment 4 Alan Hourihane 2007-09-01 15:26:49 UTC
(In reply to comment #1)
> can you please explain why? making the mode equal function not actually test if
> the modes are equal, doesn't seem like the kind of obvious change that needs
> zero explanation.  if you need to fudge (which this function seems to be
> designed to prevent), why not use a fudge factor, instead of taking 1280x768
> for 1024x768?
> 

If you look at the current code, the >= is already done xres_virtual to handle displayWidth vs virtualX issues. 

With fbdev the driver can adjust the virtual y resolution to accelerate YPAN/YWRAP scrolling. Yes, this may well be an fbdev issue, but we need to handle it too. Ideally the fbdev driver would just allocate enough memory to handle the resolution, but with the new acceleration scheme available in fbdev drivers we need to handle these cases.

So allowing yres_virtual allows fbdev drivers that got broken with the change to work again. As the "mode equal" was only introduced in xxerver 1.3 and broke some fbdev drivers.
Comment 5 Eric Anholt 2007-09-04 17:46:43 UTC
OK, justified this change to myself now.  Pushed. (The confusing part was that this isn't about the actual mode resolution in xres/yres, just about the memory backing the display, which we really don't care about)
Comment 6 Alan Hourihane 2007-09-05 01:55:21 UTC
Exactly. Thanks Eric.


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.