Bug 24830

Summary: Some characters are not drawn well, missing vertical lines with NVIDIA NV18
Product: xorg Reporter: David <david.hernando.m>
Component: Driver/nouveauAssignee: Nouveau Project <nouveau>
Status: RESOLVED FIXED QA Contact: Xorg Project Team <xorg-team>
Severity: normal    
Priority: medium    
Version: unspecified   
Hardware: x86 (IA32)   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
image
none
Xorg.0.log
none
patch
none
patch0.diff
none
patch1.diff
none
image_patch0
none
image_patch1
none
patch2.diff none

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.