Bug 20951 - [945GM] Fullscreen graphical corruption on LVDS
Summary: [945GM] Fullscreen graphical corruption on LVDS
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/intel (show other bugs)
Version: 7.4 (2008.09)
Hardware: Other All
: medium normal
Assignee: ykzhao
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-03-30 09:56 UTC by Eric Appleman
Modified: 2009-06-19 09:38 UTC (History)
5 users (show)

See Also:
i915 platform:
i915 features:


Attachments
lspci output (12.64 KB, text/plain)
2009-03-30 18:06 UTC, Eric Appleman
no flags Details
Xorg.0.log output (60.66 KB, text/plain)
2009-03-30 18:06 UTC, Eric Appleman
no flags Details
xrandr verbose output (3.56 KB, text/plain)
2009-03-30 18:08 UTC, Eric Appleman
no flags Details
please try the debug patch on your machine, thanks. (12.25 KB, application/octet-stream)
2009-04-01 06:29 UTC, MaLing
no flags Details
Camera phone images of graphical corruption (274.17 KB, application/zip)
2009-04-01 14:45 UTC, Eric Appleman
no flags Details
Unpatched Xorg.0.log (23.25 KB, text/plain)
2009-04-02 19:39 UTC, Eric Appleman
no flags Details
Patched Xorg.0.log (24.56 KB, text/plain)
2009-04-02 19:46 UTC, Eric Appleman
no flags Details
Config of default OpenArena settings (6.31 KB, application/octet-stream)
2009-04-02 20:18 UTC, Eric Appleman
no flags Details
Xorg.0.log output with tiling enabled and without UXA (34.90 KB, application/octet-stream)
2009-04-07 12:00 UTC, Eric Appleman
no flags Details
glxinfo verbose (11.25 KB, text/plain)
2009-04-13 08:01 UTC, Eric Appleman
no flags Details
Picture of bug symptoms, as seen at the World of Goo title screen (112.82 KB, image/jpeg)
2009-04-13 09:21 UTC, Eric Appleman
no flags Details
Regdump of corruption while running Touhou 06 in Wine (9.43 KB, text/plain)
2009-04-13 18:24 UTC, Eric Appleman
no flags Details
Regdump of corruption while running OpenArena 0.8.1 natively (9.43 KB, text/plain)
2009-04-13 18:28 UTC, Eric Appleman
no flags Details
please try the patch on your machine, thanks. (1.74 KB, text/plain)
2009-05-07 07:20 UTC, MaLing
no flags Details
pfit debugging (1.10 KB, patch)
2009-05-07 15:01 UTC, Jesse Barnes
no flags Details | Splinter Review
pfit debugging 2 (2.49 KB, patch)
2009-05-07 15:34 UTC, Jesse Barnes
no flags Details | Splinter Review
Xorg.0.log debug output (79.05 KB, text/plain)
2009-05-08 14:35 UTC, Eric Appleman
no flags Details
pfit debugging 3 (2.73 KB, patch)
2009-05-08 15:03 UTC, Jesse Barnes
no flags Details | Splinter Review
(World of Goo) Xorg.0.log debug (25.22 KB, text/plain)
2009-05-08 15:27 UTC, Eric Appleman
no flags Details
(Touhou Youyoumu) Xorg.0.log debug (45.87 KB, text/plain)
2009-05-08 15:30 UTC, Eric Appleman
no flags Details
(World of Goo) Xorg.0.log debug (24.04 KB, text/plain)
2009-05-08 15:35 UTC, Eric Appleman
no flags Details
(Touhou Youyoumu) Xorg.0.log debug (42.05 KB, text/plain)
2009-05-08 15:43 UTC, Eric Appleman
no flags Details
potential fix (3.17 KB, patch)
2009-05-08 16:00 UTC, Jesse Barnes
no flags Details | Splinter Review
(Touhou Eiyashou) Xorg.0.log debug (40.07 KB, text/plain)
2009-05-09 17:15 UTC, Eric Appleman
no flags Details
potential fix #2 (3.17 KB, patch)
2009-05-11 09:15 UTC, Jesse Barnes
no flags Details | Splinter Review
(Touhou Fuujinroku) Xorg.0.log debug (Ignore HAL errors) (336.67 KB, text/plain)
2009-05-11 15:27 UTC, Eric Appleman
no flags Details
New photos of corruption (983.47 KB, application/zip)
2009-05-11 18:19 UTC, Eric Appleman
no flags Details
hi Eric, please try the debug patch on your machine under the same environment with comments #61. thanks (2.04 KB, text/plain)
2009-05-13 07:34 UTC, MaLing
no flags Details
(Touhou Eiyashou) Xorg.0.log debug (Ignore tiling errors) (232.59 KB, text/plain)
2009-05-13 10:53 UTC, Eric Appleman
no flags Details
VBIOS dump (64.00 KB, application/octet-stream)
2009-06-08 11:57 UTC, Eric Appleman
no flags Details
Don't change the hsync/vsync while doing LVdS scaling (3.14 KB, patch)
2009-06-09 23:24 UTC, ykzhao
no flags Details | Splinter Review
Don't change the hsync/vsync width while doing LVDS scaling (4.72 KB, patch)
2009-06-10 02:35 UTC, ykzhao
no flags Details | Splinter Review
Don't change the hsync/vsync width while doing LVDS scaling (4.72 KB, patch)
2009-06-10 02:52 UTC, ykzhao
no flags Details | Splinter Review
Touhou Youyoumu with patch (41.27 KB, image/jpeg)
2009-06-10 04:34 UTC, Eric Appleman
no flags Details
Don't change the hsync/vsync width while doing LVDS scaling (7.52 KB, patch)
2009-06-10 23:58 UTC, ykzhao
no flags Details | Splinter Review
Xorg.0.log output (225.94 KB, application/octet-stream)
2009-06-11 09:08 UTC, Eric Appleman
no flags Details
don't change the blank/sync width while doing LVDS scaling mode (5.97 KB, patch)
2009-06-11 20:24 UTC, ykzhao
no flags Details | Splinter Review
Xorg.0.log output for Touhou Fuujinroku (505.19 KB, text/plain)
2009-06-12 13:15 UTC, Eric Appleman
no flags Details

Description Eric Appleman 2009-03-30 09:56:49 UTC
Hi. I have an i945 chipset and it seems that whenever I play a native Linux game like World of Goo or a Wine-powered game Touhou in fullscreen, I experience a rather strange problem.

The game will trigger a switch to fullscreeen, but almost the entire left half of the screen gives funky bands of color while the right side appears mostly normal. Furthermore, sometimes the left of the screen will look relatively fine, but there will be a visible dividing line between the two halves of the screen. Also, for either scenario, the entire screen will appear speckled.

Finally, it is important that I also point out that I if use an external LCD monitor or TV in standalone or clone mode, the problem will not appear on that other screen.

Thanks in advance.

- Eric
Comment 1 Eric Appleman 2009-03-30 09:57:54 UTC
00:02.0 VGA compatible controller [0300]: Intel Corporation Mobile 945GM/GMS, 943/940GML Express Integrated Graphics Controller [8086:27a2] (rev 03)
Comment 2 Eric Appleman 2009-03-30 18:06:10 UTC
Created attachment 24383 [details]
lspci output
Comment 3 Eric Appleman 2009-03-30 18:06:33 UTC
Created attachment 24384 [details]
Xorg.0.log output
Comment 4 Eric Appleman 2009-03-30 18:08:44 UTC
Created attachment 24385 [details]
xrandr verbose output
Comment 5 Gordon Jin 2009-03-30 19:47:24 UTC
Could you provide more info about your driver version? If it's shipped from distro, what's the detailed distro version?

And what's previous version which worked for you?
Comment 6 Eric Appleman 2009-03-30 21:00:12 UTC
Driver:
xserver-xorg-video-intel - 2:2.6.99.1+git20090327.69c84f2c-0ubuntu0tormod 


Also, I am using Ubuntu 9.04.

I can't clearly remember the last version that worked for me, but if I had to hazard a safe guess, I'd say definitely Ubuntu 8.04 and possibly even Ubuntu 8.10.

I'll have to do a LiveUSB test to double-check.
Comment 7 Eric Appleman 2009-03-30 21:08:34 UTC
BTW, just in case I was vague.

Driver versions

8.04: 2.2.2
8.10: 2.4.1

I'm quite confident that the 2.5.x driver series introduced the error, but as I mentioned, I'll have to check.
Comment 8 Gordon Jin 2009-03-30 21:50:24 UTC
Let's sync the reproduce method first. Eric, could you reproduce with some other native Linux games like openarena, ut2004-demo, quake3-demo or torcs. It's easier for us to test them, and in my impression they don't have this problem with upstream driver.
Comment 9 Eric Appleman 2009-03-30 23:03:20 UTC
Confirming that the problem is present in fresh LiveUSB sessions of Ubuntu 9.04, but not Ubuntu 8.10.

In OpenArena, the corruption was present in all resolutions except for 1360x768 where the screen rendered everything rather normally aside from a flashing black box that occasionally appeared. I imagine that if the game had a native 1440x900 resolution mode, it would render similarly, if not perfectly.

Comment 10 Gordon Jin 2009-03-30 23:09:51 UTC
Haien, do you see this problem with openarena with 2.7 branch on 945gm?
Comment 11 zhao jian 2009-03-31 00:04:20 UTC
(In reply to comment #10)
> Haien, do you see this problem with openarena with 2.7 branch on 945gm?
I just played openarena with 2.7 branch on 945gm, it works well. 
Comment 12 Eric Appleman 2009-03-31 11:53:15 UTC
I'm pretty convinced that this is a modeline issue.

Also, zhao, define "it works well".

As far as I am concerned the i945gm chipset running on 2.7.x drivers is still plagued by a17 tiling issues that lead to a ~6x performance loss compared to the 2.4.x series drivers.
Comment 13 MaLing 2009-04-01 06:29:37 UTC
Created attachment 24430 [details]
please try the debug patch on your machine, thanks.
Comment 14 Eric Appleman 2009-04-01 09:39:12 UTC
The patch is useless since the DRM and intel git break my system. If you can debian package libdrm 2.4.6 and a patched intel driver for me, I'll be able to test.
Comment 15 Eric Appleman 2009-04-01 14:45:04 UTC
Created attachment 24446 [details]
Camera phone images of graphical corruption
Comment 16 Gordon Jin 2009-04-01 18:59:40 UTC
(In reply to comment #14)
> The patch is useless since the DRM and intel git break my system. 

Is there a bug filed for this? I'll consider this bug is blocked by that bug.
Comment 17 Eric Appleman 2009-04-01 19:04:49 UTC
I wouldn't bother. I probably just need to clean up my 2.6.29 and 2.6.28 headers. Give me a day or two.

:3
Comment 18 Eric Appleman 2009-04-02 12:20:55 UTC
Okay. I got my machine working with the git drivers and applied the patch.

There is no change in the fullscreen behavior.
Comment 19 liuhaien 2009-04-02 18:57:42 UTC
(In reply to comment #18)
> Okay. I got my machine working with the git drivers and applied the patch.
> 
> There is no change in the fullscreen behavior.
> 

we can not reproduce this issue on our 945gm.what is the version of your openarena? could you give me the config file of the game?
Comment 20 Eric Appleman 2009-04-02 19:24:45 UTC
Fist of all, have the pictures helped you guys at all?

Also, I'd like to reiterate that this issue occurs on practically all non-native modelines and I have repeatedly reproduced this on clean Jaunty live sessions with my laptop.

Let me upload a few more Xorg.0.log files that better reflect my system before and after the patch.
Comment 21 Eric Appleman 2009-04-02 19:39:00 UTC
FYI. I'm using the latest version of OpenArena, 0.8.1.

I have no idea what you are talking about with regard to a config file.
Comment 22 Eric Appleman 2009-04-02 19:39:37 UTC
Created attachment 24482 [details]
Unpatched Xorg.0.log
Comment 23 Eric Appleman 2009-04-02 19:46:35 UTC
Created attachment 24483 [details]
Patched Xorg.0.log
Comment 24 MaLing 2009-04-02 20:06:38 UTC
Because the game can not triger fullscreen in our machine, and we have to do it manually, but failed to reproduce it. So we hope to use your config file of this game, could you please upload it, and what's your game version?

Thanks
Ma Ling


Comment 25 liuhaien 2009-04-02 20:10:16 UTC
(In reply to comment #21)
> FYI. I'm using the latest version of OpenArena, 0.8.1.
> 
> I have no idea what you are talking about with regard to a config file.
> 

maybe you can find it in the directory /root/.openarena/baseoa
the name is q3config.cfg, upload it if you find. thanks.
Comment 26 Eric Appleman 2009-04-02 20:18:21 UTC
Created attachment 24487 [details]
Config of default OpenArena settings
Comment 27 Eric Appleman 2009-04-02 23:09:29 UTC
>Because the game can not triger fullscreen in our machine, and we have to do it
>manually, but failed to reproduce it. So we hope to use your config file of
>this game, could you please upload it, and what's your game version?

This bug has nothing to do with OpenArena. But in case it means anything, the game fullscreens with the default config.
Comment 28 Eric Appleman 2009-04-05 15:10:08 UTC
So... Any ideas?
Comment 29 liuhaien 2009-04-06 19:01:04 UTC
(In reply to comment #28)
> So... Any ideas?
> 

sorry,we still cann't reproduce this issue,could you retry it using the latest driver?
Libdrm:		(master)51d6346f9f3c425f49e57d185530c6bcaeb94f5e
Mesa:		(mesa_7_4_branch)de197cf991416f0cd65ad2e2d2ca9aa599b52075
Xserver:	(server-1.6-branch)60c161545af80eb78eb790a05bde79409dfdf16e
Xf86_video_intel:		(2.7)10b5014c42dc055d9559ee112cc7a017e887d813
Kernel:       (drm-intel-2.6.29)0e56a4d653b66d4729f944b23935a00c4472f987

thanks.
Comment 30 Jesse Barnes 2009-04-07 10:45:45 UTC
This sounds a lot like the bug we had awhile ago when non-native modes were added to the LVDS output, but apparently even centered panel fitting doesn't help in this case.  I don't see the modeline being used for the game though, has that been checked for validity?  Maybe one of the modes we add to the LVDS output isn't correct for this machine?

But now that I look, the pictures don't really seem like modeline issues, they seem like rendering problems.  This is an UXA/DRI2 configuration, so we don't need to do any sarea updates for resolution changes, but maybe something else is missing?

Eric what version of Mesa are you running?

I also see that tiling is disabled in this config:
(EE) intel(0): Failed to set tiling on front buffer: rejected by kernel

Which may also be related (though the pictures don't really look like tiled rendering problems).
Comment 31 Eric Appleman 2009-04-07 11:25:58 UTC
Mesa: 7.4 branch (Jaunty) *can also be replicated on 7.3 branch as far as I can remember*
Intel: Master or xorg-edgers PPA (usually less than 10 commits apart)
DRM: Master or xorg-edgers PPA (usually less than 10 commits apart)
Comment 32 Eric Appleman 2009-04-07 11:26:47 UTC
And yes, at the discretion of many, I have GEM/tiling disabled.
Comment 33 Eric Appleman 2009-04-07 11:28:30 UTC
Finally, this is not limited to UXA and DRI2, EXA and DRI1 are also affected.
Comment 34 Eric Appleman 2009-04-07 12:00:53 UTC
Created attachment 24650 [details]
Xorg.0.log output with tiling enabled and without UXA
Comment 35 Eric Appleman 2009-04-08 21:53:30 UTC
Is there any other hard data that you guys need?
Comment 36 Michael Fu 2009-04-11 23:27:12 UTC
if you have tried Option "Tiling" "False", how about

Option "FrameBufferCompression" "False"

Comment 37 Eric Appleman 2009-04-12 12:07:27 UTC
Unfortunately, that didn't help.

Also, must I reiterate that the regression window for this bug is between 2.4.1 and current Intel.
Comment 38 Eric Appleman 2009-04-13 06:59:22 UTC
Just taking a shot in the dark, but could this be a dithering problem?
Comment 39 Eric Appleman 2009-04-13 07:23:46 UTC
Does a dusty or speckled rendering ring any bells either?
Comment 40 Eric Appleman 2009-04-13 08:01:37 UTC
Created attachment 24753 [details]
glxinfo verbose
Comment 41 Eric Appleman 2009-04-13 09:21:32 UTC
Created attachment 24759 [details]
Picture of bug symptoms, as seen at the World of Goo title screen
Comment 42 Jesse Barnes 2009-04-13 17:21:36 UTC
Eric narrowed this down (thanks a ton Eric) to the panel fitting change I pushed awhile back: http://cgit.freedesktop.org/xorg/driver/xf86-video-intel/commit/?id=667923559219429b0c5fec12a0164f7eba1f8f2d

Which means either the panel fitting regs are broken on his machine or we're getting the timings slightly wrong.  Can you take a look at the register dump that Eric will attach, Ma Ling?  If this hw truly is broken (i.e. the regs look correct but there's still corruption) we may have to quirk it.
Comment 43 Eric Appleman 2009-04-13 18:24:47 UTC
Created attachment 24774 [details]
Regdump of corruption while running Touhou 06 in Wine
Comment 44 Eric Appleman 2009-04-13 18:28:06 UTC
Created attachment 24775 [details]
Regdump of corruption while running OpenArena 0.8.1 natively
Comment 45 Eric Appleman 2009-04-13 19:53:59 UTC
Workaround.

xrandr --output LVDS --set PANEL_FITTING full
Comment 46 Eric Appleman 2009-04-20 03:09:38 UTC
*sigh*

It looks like Jaunty will be shipping with this bug unresolved. Hopefully the pending fix will be backported or offered as an update.
Comment 47 MaLing 2009-05-07 07:20:39 UTC
Created attachment 25589 [details]
please try the patch on your machine, thanks.

I checked dump register file in comments #43 and #44, they use resolution 1152x864 and lvds fixed mode is 1440x900,no any incorrect. The patch intends to make quirk for the platform, fore it to use FULL pannel fitting, instead of original FULL_ASPECT. please try it on your machine.

Thanks 
Ma Ling
Comment 48 Eric Appleman 2009-05-07 09:23:46 UTC
Ma Ling, I appreciate the patch and it does what you say it does, but I do not consider it to be an acceptable solution to my problem.

My native 640x480 games are still stretched and I can't play them like this since they require pixel-level precision.

In short, I need FULL_ASPECT.
Comment 49 Jesse Barnes 2009-05-07 15:01:02 UTC
Created attachment 25612 [details] [review]
pfit debugging

We can use this to check whether the pfit ratios are sane.  Can you apply it and reproduce the corruption, then attach your log?
Comment 50 Jesse Barnes 2009-05-07 15:34:25 UTC
Created attachment 25616 [details] [review]
pfit debugging 2

Actually, try this one.  It'll give us more info that should allow us to track down the problem if it's related to bad timing (which I suspect).
Comment 51 MaLing 2009-05-07 18:17:31 UTC
(In reply to comment #50)
> Created an attachment (id=25616) [details]
> pfit debugging 2
> Actually, try this one.  It'll give us more info that should allow us to track
> down the problem if it's related to bad timing (which I suspect).

Eric, when you using this patch, could you please upload log file with modedebug optoin on ?

thanks
Ma Ling
Comment 52 Eric Appleman 2009-05-08 14:35:22 UTC
Created attachment 25647 [details]
Xorg.0.log debug output
Comment 53 Jesse Barnes 2009-05-08 15:03:43 UTC
Created attachment 25648 [details] [review]
pfit debugging 3

The ErrorFs I added aren't in your log.  This patch makes them into more normal driver error messages.  If we still don't see them with this patch applied while you're seeing the corruption, then either you're not running the patched driver or the game you're running isn't going through this path to set the mode...
Comment 54 Eric Appleman 2009-05-08 15:27:14 UTC
Created attachment 25649 [details]
(World of Goo) Xorg.0.log debug
Comment 55 Eric Appleman 2009-05-08 15:30:18 UTC
Created attachment 25650 [details]
(Touhou Youyoumu) Xorg.0.log debug
Comment 56 Eric Appleman 2009-05-08 15:35:19 UTC
Created attachment 25651 [details]
(World of Goo) Xorg.0.log debug
Comment 57 Eric Appleman 2009-05-08 15:43:24 UTC
Created attachment 25652 [details]
(Touhou Youyoumu) Xorg.0.log debug
Comment 58 Jesse Barnes 2009-05-08 16:00:56 UTC
Created attachment 25654 [details] [review]
potential fix

If your panel is in dual-channel mode, this patch might fix things by making the panel fit timings even instead of odd.
Comment 59 Eric Appleman 2009-05-08 16:45:09 UTC
That patch helps a lot. The left side corruption is gone.

However, the overall "fuzziness" and thin vertical line of corruption down the center of screen are still present.
Comment 60 Jesse Barnes 2009-05-08 17:01:51 UTC
Can you post a screenshot of the new corruption?
Comment 61 Eric Appleman 2009-05-09 17:15:05 UTC
Created attachment 25677 [details]
(Touhou Eiyashou) Xorg.0.log debug
Comment 62 Eric Appleman 2009-05-09 19:58:51 UTC
I'm also starting to notice faint shadows of previously rendered objects (ie. Touhou seen in World of Goo)
Comment 63 Jesse Barnes 2009-05-11 09:15:00 UTC
Created attachment 25732 [details] [review]
potential fix #2

In some cases the sync range wouldn't fall inside the blank range with the last patch.  This one should fix that case.  Can you try it out and attach the log again if it fails?
Comment 64 Eric Appleman 2009-05-11 15:27:07 UTC
Created attachment 25768 [details]
(Touhou Fuujinroku) Xorg.0.log debug (Ignore HAL errors)
Comment 65 Eric Appleman 2009-05-11 15:54:40 UTC
No visible improvement with newest patch.
Comment 66 Jesse Barnes 2009-05-11 16:02:19 UTC
Ling should be coming online soon.  Any ideas on the "middle line corruption" part of this, Ling?  Sounds like my patch is at least a partial fix (though it needs a little cleanup, I'd appreciate your review; feel free to push it today too if you like it).

The 945 chips have a sub-pixel rendering option for panel fitting, maybe that's off in this configuration somehow?  Or our scaling calculation results in an odd pixel count for the scaled width?
Comment 67 Eric Appleman 2009-05-11 18:19:37 UTC
Created attachment 25781 [details]
New photos of corruption

New photos of vertical corruption line at dead center of screen.

Aforementioned fuzziness is not visible in these photos.

Games used: World of Goo, Touhou Koumakyou, and Touhou Youyoumu.
Comment 68 MaLing 2009-05-13 07:34:40 UTC
Created attachment 25820 [details]
hi Eric, please try the debug patch on your machine under the same environment with comments #61. thanks
Comment 69 Eric Appleman 2009-05-13 10:53:55 UTC
Created attachment 25831 [details]
 (Touhou Eiyashou) Xorg.0.log debug (Ignore tiling errors)

Ma Ling, unfortunately your patch seemed to have no noticeable effect. Everything looks the same as an unpatched driver.

- Blurry and fuzzy rendering remains
- Left-side corruption has returned
- Line of corruption down center of screen blends with left-side corruption
Comment 70 Eric Appleman 2009-05-25 13:19:18 UTC
Are the current patches going to be added to the git?
Comment 71 Michael Fu 2009-06-05 00:22:51 UTC
Eric, does CENTER mode works when you play those games? 
Comment 72 Eric Appleman 2009-06-05 19:43:40 UTC
It works, but it's also affected by this bug.
Comment 73 ykzhao 2009-06-07 23:19:54 UTC
Will you please attach the output of vbios dump?
   >echo 1 > /sys/devices/pci0000:00/0000:00:02.0/rom
   > cat /sys/devices/pci0000:00/0000:00:02.0/rom > vbios.dump
   
Thanks.
Comment 74 Eric Appleman 2009-06-08 11:57:28 UTC
Created attachment 26552 [details]
VBIOS dump

00:02.0 VGA compatible controller [0300]: Intel Corporation Mobile 945GM/GMS, 943/940GML Express Integrated Graphics Controller [8086:27a2] (rev 03)
Comment 75 ykzhao 2009-06-09 23:24:21 UTC
Created attachment 26617 [details] [review]
Don't change the hsync/vsync while doing LVdS scaling
Comment 76 ykzhao 2009-06-09 23:25:19 UTC
Will you please try the debug patch and see whether the issue still exists?
   Thanks.
Comment 77 ykzhao 2009-06-10 02:35:34 UTC
Created attachment 26632 [details] [review]
Don't change the hsync/vsync width while doing LVDS scaling

Will you please try the updated patch ?
Thanks.
Comment 78 ykzhao 2009-06-10 02:52:42 UTC
Created attachment 26633 [details] [review]
Don't change the hsync/vsync width while doing LVDS scaling

Sorry that the incorrect patch is attached.
    
Will you please try the updated debug patch and see whether the issue still exists?
   Thanks.
Comment 79 Eric Appleman 2009-06-10 04:34:11 UTC
Created attachment 26635 [details]
Touhou Youyoumu with patch

The fuzziness and corruption are gone, but the dividing line between the two halves of the screen is still present.

At least we're making progress. ^_^
Comment 80 ykzhao 2009-06-10 23:58:47 UTC
Created attachment 26657 [details] [review]
Don't change the hsync/vsync width while doing LVDS scaling 

Will you please try the updated patch and see whether the issue still exists?
   In this debug patch the hsync/vsync width is not changed. And the blank width is printed.
   After the test, please attach the Xorg log.
   Thanks.
Comment 81 Eric Appleman 2009-06-11 09:08:17 UTC
Created attachment 26689 [details]
Xorg.0.log output

Patch works. All symptoms of the issue are gone.

Will this bug be committed in time for 2.8?
Comment 82 ykzhao 2009-06-11 18:24:00 UTC
Thanks for the test and so quick response.

Will you please try the center scaling mode and see whether it still works for you? 
 After the test, please also attach the Xorg log.
Thanks.
Comment 83 ykzhao 2009-06-11 20:24:39 UTC
Created attachment 26698 [details] [review]
don't change the blank/sync width while doing LVDS scaling mode

Will you please try the updated patch?
Thanks.
Comment 84 Eric Appleman 2009-06-12 13:15:30 UTC
Created attachment 26739 [details]
Xorg.0.log output for Touhou Fuujinroku

Center and full-aspect scaling work properly.
Comment 85 ykzhao 2009-06-16 01:32:09 UTC
Thanks for the test.
    I will send the patch to intel-gfx mailing list ASAP.
     
    As the bug can be fixed by the attached patch, the bug will be marked as "resolved".
    Thanks.
Comment 86 Eric Appleman 2009-06-16 20:25:48 UTC
Damn. Why didn't I see your message before I submitted the patch to the mailing list myself.

Sorry.
Comment 87 Jesse Barnes 2009-06-19 09:38:47 UTC
commit 534e73ad4f234a04755917f2bf17ba821c27eb52
Author: Zhao Yakui <yakui.zhao@intel.com>
Date:   Thu Jun 18 09:46:32 2009 +0800

    Don't change the blank/sync width when calculating scaled modes


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.