Bug 62962 - image corruption as of update to latest version of openchrome
Summary: image corruption as of update to latest version of openchrome
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/openchrome (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: Openchrome development list
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-31 13:11 UTC by ma
Modified: 2017-01-22 19:03 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
Xorg.0.log (47.26 KB, text/plain)
2013-03-31 13:11 UTC, ma
no flags Details
picture of the issue (632.17 KB, image/jpeg)
2013-03-31 14:36 UTC, ma
no flags Details
picture of the issue (841.81 KB, image/jpeg)
2013-03-31 14:38 UTC, ma
no flags Details
0.3.2 Xorg.0.log (46.91 KB, text/plain)
2013-04-01 11:18 UTC, ma
no flags Details
Xorg.0.log with noaccel true (46.60 KB, text/plain)
2013-04-09 12:49 UTC, ma
no flags Details
Xorg.0.log with custom compiled kernelTtmGemKms ) (40.52 KB, text/plain)
2013-04-14 11:03 UTC, ma
no flags Details
picture of the issue (custom compiled kernelTtmGemKms ) (385.00 KB, image/jpeg)
2013-04-14 11:17 UTC, ma
no flags Details
Xorg.0.log with custom compiled kernelTtmGemKms and via.modeset=0 (46.56 KB, text/plain)
2013-04-14 11:22 UTC, ma
no flags Details
picture of the issue with custom compiled kernelTtmGemKms and via.modeset=0 (349.32 KB, image/jpeg)
2013-04-14 11:27 UTC, ma
no flags Details
screenshot of the issue (glxgears) with custom compiled kernelTtmGemKms and via.modeset=0 (176.14 KB, image/png)
2013-04-14 11:39 UTC, ma
no flags Details
kernel log with custom compiled kernelTtmGemKms drm.debug=255 and via.modeset=1 (64.11 KB, text/plain)
2013-04-14 13:16 UTC, ma
no flags Details
Fix colors on second crtc (1005 bytes, patch)
2013-05-22 12:33 UTC, James Simmons
no flags Details | Splinter Review
Xorg.0.log with patch from comment #18 applied (58.45 KB, text/plain)
2013-05-22 14:14 UTC, ma
no flags Details
screenshot with patch from comment #18 applied (blueish, watermark) (668.31 KB, image/jpeg)
2013-05-22 14:22 UTC, ma
no flags Details
dmesg with patch from comment #18 applied and via.modeset=1 (36.27 KB, text/plain)
2013-05-22 17:03 UTC, ma
no flags Details
Xorg.0.log with patch from comment #18 applied and via.modeset=1 (40.81 KB, text/plain)
2013-05-22 17:04 UTC, ma
no flags Details
picture of issue with patch from comment #18 applied and via.modeset=1 (681.41 KB, image/jpeg)
2013-06-08 10:12 UTC, ma
no flags Details
Turn off secondary cursor by default. (548 bytes, patch)
2013-06-09 19:30 UTC, James Simmons
no flags Details | Splinter Review
Test patch to see if LVDS is TTL (479 bytes, patch)
2013-06-09 19:32 UTC, James Simmons
no flags Details | Splinter Review
Xorg.0.log with patches from 2013-06-09 (40.90 KB, text/plain)
2013-06-15 14:37 UTC, ma
no flags Details
dmesg with patches from 2013-06-09 (35.37 KB, text/plain)
2013-06-15 14:38 UTC, ma
no flags Details
dmidecode (9.13 KB, text/plain)
2013-06-23 04:09 UTC, ma
no flags Details

Description ma 2013-03-31 13:11:05 UTC
Created attachment 77247 [details]
Xorg.0.log

Hi, since updating openchrome on my Amilo Pro V2030 (running EL 6.4) to version 0.3.0-3.20120806git, image corruption appears. I have also tried the latest version from git, which did not fix the issue.
Xorg.0.log and picture are attached.
Comment 1 ma 2013-03-31 14:36:54 UTC
Created attachment 77250 [details]
picture of the issue

also happens with latest upstream version (git) and ubuntu 12.10.
Comment 2 ma 2013-03-31 14:38:26 UTC
Created attachment 77251 [details]
picture of the issue
Comment 3 Xavier Bachelot 2013-03-31 17:58:52 UTC
RHEL 6.4 (and clones) are using an openchrome snapshot between 0.3.0 and 0.3.1. RHEL 6.3 and earlier were using 0.2.904, this is a huge gap.
It would be nice if you can bisect the regression or at least try 0.2.905 and 0.2.906.
Also, can you please provide the X log from 0.3.2 (which is at this point similar to git master) ?
Comment 4 ma 2013-04-01 11:16:43 UTC
hi,  builds of 0.2.905 and 0.2.906 fail because of:
via_accel.c:37:22: error: xaalocal.h: No such file or directory

it seems as if the current version of xorg-x11-server-devel-1.13.0-11.sl6.i686 does not include xaalocal.h anymore. any help on building it anyway is much appreciated.
Comment 5 ma 2013-04-01 11:18:18 UTC
Created attachment 77272 [details]
0.3.2 Xorg.0.log
Comment 6 Xavier Bachelot 2013-04-05 14:24:40 UTC
XAA has been removed from X server 1.13, that's why 0.2.905 and 0.2.906 are not building. It might be possible to backport some fixes to allow building without XAA, I'll try to take a look.

Meanwhile, can you try to run 0.3.2 without 2D acceleration ? You need to use Option "NoAccel" in your X configuration.
Comment 7 ma 2013-04-09 12:49:29 UTC
NoAccel does not solve the problem.
Comment 8 ma 2013-04-09 12:49:54 UTC
Created attachment 77663 [details]
Xorg.0.log with noaccel true
Comment 9 ma 2013-04-14 11:01:42 UTC
So, I've now compiled a KMS kernel based on those instructions: http://www.freedesktop.org/wiki/Openchrome/TtmGemKms

Does not work, image corruption (different now) still appears.

I'll attach logs regarding this issue.
Comment 10 ma 2013-04-14 11:03:04 UTC
Created attachment 77940 [details]
Xorg.0.log with custom compiled kernelTtmGemKms  )
Comment 11 ma 2013-04-14 11:17:07 UTC
Created attachment 77941 [details]
picture of the issue (custom compiled kernelTtmGemKms )

please note that the above data from today all appeard in combination with via.modeset=1. ill also provide data for if via.modeset=0
Comment 12 ma 2013-04-14 11:22:17 UTC
Created attachment 77942 [details]
Xorg.0.log with custom compiled kernelTtmGemKms and  via.modeset=0
Comment 13 ma 2013-04-14 11:27:25 UTC
Created attachment 77943 [details]
picture of the issue with custom compiled kernelTtmGemKms and via.modeset=0

heres the picture belonging to the issue.
the cursor corruption can be probably solved by using swcursor

besides of that, colors are unreal (too much blue, too much red), there's a little bit of image corruption in the middle of the background and sometimes there also was a translucent scientific logo in the middle everywhere.
Comment 14 ma 2013-04-14 11:39:41 UTC
Created attachment 77944 [details]
screenshot of the issue (glxgears) with custom compiled kernelTtmGemKms and via.modeset=0

I just realized that you can even take normal screenshots of the issue. I found it interesting that you can't spot the red wheel in glxgears, here's another screenshot from it.
Comment 15 ma 2013-04-14 13:16:00 UTC
Created attachment 77949 [details]
kernel log with custom compiled kernelTtmGemKms drm.debug=255 and via.modeset=1
Comment 16 James Simmons 2013-05-20 22:08:21 UTC
Hello.

  The image corruption deals with the hardware cursor. I have seen this issue as well on my VX900 laptop but it doesn't happen all the time. Because of this I haven't been able to track down the problem. If you run into this problem all the time then we might be able to solve it. Also I never seen it in UMS mode like you have which tells me the same cursor bug exist for both the KMS kernel and the xorg UMS code for the cursor. Does this show up only on the LVDS or when you have external monitor plugged in? As for the wrong colors that is a kernel drm bug for the second display. I have seen it recently with my multiscreen testing for KMS.
Comment 17 ma 2013-05-22 09:12:36 UTC
I tried using an external monitor but it does not work in any scenario (black image) - either because of the VGA-port being defective or a bug in the driver.
Comment 18 James Simmons 2013-05-22 12:33:19 UTC
Created attachment 79651 [details] [review]
Fix colors on second crtc

Can you try this patch against the drm-opernchrome tree. This should fix the wrong color issues you see. The cursor is still not fixed. I thing it is a initialization issue.
Comment 19 ma 2013-05-22 14:13:13 UTC
drm-openchrome tree with that patch (via.modeset=0) shows the same behavior.
Comment 20 ma 2013-05-22 14:14:16 UTC
Created attachment 79659 [details]
Xorg.0.log with patch from comment #18 applied
Comment 21 ma 2013-05-22 14:22:03 UTC
Created attachment 79660 [details]
screenshot with patch from comment #18 applied (blueish, watermark)
Comment 22 James Simmons 2013-05-22 14:23:58 UTC
Can you try with modeset=1. This patch only impacts KMS mode setting
Comment 23 ma 2013-05-22 14:34:58 UTC
with via.modeset=1: colors ok, image still blurry.
Comment 24 James Simmons 2013-05-22 14:39:39 UTC
Can you post Xorg.log and dmesg.
Comment 25 ma 2013-05-22 17:03:52 UTC
Created attachment 79673 [details]
dmesg with patch from comment #18 applied and via.modeset=1
Comment 26 ma 2013-05-22 17:04:42 UTC
Created attachment 79674 [details]
Xorg.0.log with patch from comment #18 applied and via.modeset=1
Comment 27 ma 2013-06-08 10:12:37 UTC
Created attachment 80515 [details]
picture of issue with patch from comment #18 applied and via.modeset=1

heres another picture of the issue. any chance that it gets fixed?
Comment 28 James Simmons 2013-06-09 19:30:11 UTC
Created attachment 80581 [details] [review]
Turn off secondary cursor by default.

Here is a patch for broke cursor issue. Let me know if you can now use the second cursor.
Comment 29 James Simmons 2013-06-09 19:32:00 UTC
Created attachment 80582 [details] [review]
Test patch to see if LVDS is TTL

I have been thinking about why your LVDS is blurry. It could be a TTL based LVDS. Please try the test patch attached.
Comment 30 ma 2013-06-15 14:36:25 UTC
applied the "Test patch to see if LVDS is TTL", and it works - great.

Not sure what the cursor patch is supposed to do (applied it, though), as far as I know the cursor issue only appeard with via.modeset=0 (I have taken out the swcursor option in xorg.conf)
Comment 31 ma 2013-06-15 14:37:40 UTC
Created attachment 80870 [details]
Xorg.0.log with patches from 2013-06-09
Comment 32 ma 2013-06-15 14:38:17 UTC
Created attachment 80871 [details]
dmesg with patches from 2013-06-09
Comment 33 James Simmons 2013-06-17 14:27:28 UTC
Much better. Could you post the output of dmidecode. I can use that output to handle your LVDS special case.
Comment 34 ma 2013-06-23 04:09:22 UTC
Created attachment 81236 [details]
dmidecode

sorry for answering late, heres the dmidecode output
Comment 35 James Simmons 2013-07-05 22:31:18 UTC
Thanks. I updated the drm-openchrome tree to handle your case automatically.
Comment 36 Kevin Brace 2017-01-22 19:01:48 UTC
It appears that this issue was resolved.
I will be closing the bug.
Comment 37 Kevin Brace 2017-01-22 19:02:30 UTC
If the problem persists with the latest OpenChrome DDX UMS code, let us know.


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.