Bug 24830 - Some characters are not drawn well, missing vertical lines with NVIDIA NV18
Summary: Some characters are not drawn well, missing vertical lines with NVIDIA NV18
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/nouveau (show other bugs)
Version: unspecified
Hardware: x86 (IA32) Linux (All)
: medium normal
Assignee: Nouveau Project
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-10-31 09:11 UTC by David
Modified: 2009-11-10 14:50 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
image (478.29 KB, image/png)
2009-10-31 09:13 UTC, David
no flags Details
Xorg.0.log (31.63 KB, text/plain)
2009-10-31 09:14 UTC, David
no flags Details
patch (2.30 KB, patch)
2009-10-31 09:15 UTC, David
no flags Details | Splinter Review
patch0.diff (434 bytes, patch)
2009-10-31 10:28 UTC, Francisco Jerez
no flags Details | Splinter Review
patch1.diff (409 bytes, patch)
2009-10-31 10:28 UTC, Francisco Jerez
no flags Details | Splinter Review
image_patch0 (369.28 KB, image/png)
2009-10-31 12:08 UTC, David
no flags Details
image_patch1 (369.17 KB, image/png)
2009-10-31 12:08 UTC, David
no flags Details
patch2.diff (321 bytes, patch)
2009-11-01 06:59 UTC, Francisco Jerez
no flags Details | Splinter Review

Description David 2009-10-31 09:11:43 UTC
The commit f0a3d9f59817ebf2b9606097785f0ad1ae45da12 of xf86-video-nouveau on some characters are not drawn well missing vertical lines. I have a NVIDIA NV18

Post xorg log, image with the problem, patch works, but you have to check it, I do not know why it works.
Comment 1 David 2009-10-31 09:13:49 UTC
Created attachment 30863 [details]
image
Comment 2 David 2009-10-31 09:14:56 UTC
Created attachment 30864 [details]
Xorg.0.log
Comment 3 David 2009-10-31 09:15:40 UTC
Created attachment 30865 [details] [review]
patch
Comment 4 Francisco Jerez 2009-10-31 10:28:05 UTC
Created attachment 30868 [details] [review]
patch0.diff

Does the attached patch help? If it doesn't, could you confirm that the problem is somewhere along the A8+A8 path by running patch1.diff?
Comment 5 Francisco Jerez 2009-10-31 10:28:37 UTC
Created attachment 30869 [details] [review]
patch1.diff
Comment 6 David 2009-10-31 11:20:58 UTC
The patch1 works well, I have not tried Patch0. I try the Patch0?

Thank you.
Comment 7 Francisco Jerez 2009-10-31 11:54:56 UTC
(In reply to comment #6)
> The patch1 works well, I have not tried Patch0. I try the Patch0?
> 
Please do, patch1.diff just prevents A8+A8 from being accelerated.

> Thank you.
> 

Comment 8 David 2009-10-31 12:06:41 UTC
	
With much improved path0.
I only noticed a flaw in a letter across the desk.
You can see in image_patch0. (font size giant to view)

With patch1 is perfect. You can see in image_patch1
Comment 9 David 2009-10-31 12:08:10 UTC
Created attachment 30870 [details]
image_patch0
Comment 10 David 2009-10-31 12:08:59 UTC
Created attachment 30871 [details]
image_patch1
Comment 11 Francisco Jerez 2009-11-01 06:59:32 UTC
Created attachment 30883 [details] [review]
patch2.diff

What about this one?
Comment 12 David 2009-11-01 07:15:31 UTC
With patch2 is perfect
Comment 13 Francisco Jerez 2009-11-10 14:50:10 UTC
This should be fixed in master now, so I'm closing the bug.


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.