[Bug 220724] New: radeon: grey line at the bottom of the screen when the screen gets scalled to the flatpanel resolution
https://bugzilla.novell.com/show_bug.cgi?id=220724 Summary: radeon: grey line at the bottom of the screen when the screen gets scalled to the flatpanel resolution Product: openSUSE 10.2 Version: Beta 2 Platform: 32bit OS/Version: Other Status: NEW Severity: Minor Priority: P5 - None Component: X.Org AssignedTo: sndirsch@novell.com ReportedBy: ssommer@novell.com QAContact: sndirsch@novell.com Hi, I wonder if my patch at https://bugs.freedesktop.org/show_bug.cgi?id=8983 could be integrated. The problem is that whenever I choose a resolution other than 1400x1050 for my mobility radeon 9700 the most right part of the screen is corrupted and a grey line occurres on the bottom of the screen. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=220724 sndirsch@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mhopf@novell.com, sndirsch@novell.com Severity|Minor |Normal Status|NEW |ASSIGNED Priority|P5 - None |P2 - High -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=220724 ------- Comment #1 from mhopf@novell.com 2006-11-14 05:24 MST ------- Wow. I would typically imagine that *adding* rounding code would fix floating ponint issues - this time it is the other way round... I would either have this tested on other machines (I haven't heard of this issue yet, and there are more 1400x1050 displays with radeon chips available), or commented positively upstream before unconditional inclusion. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=220724 ------- Comment #2 from ssommer@novell.com 2006-11-14 06:02 MST ------- I wonder how noticeable this is to the generic user. I probably would not have noticed it if radeonfb didn't suffer from the same bug. Most of the time the grey line won't be noticed due to the kde-panel and then there are probably not many people using the flatpanel with another resolution than the native one anyway. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=220724 sndirsch@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P2 - High |P3 - Medium ------- Comment #4 from sndirsch@novell.com 2006-11-18 15:08 MST ------- updating priority -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=220724 sndirsch@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEEDINFO Info Provider| |ssommer@novell.com ------- Comment #5 from sndirsch@novell.com 2006-12-06 09:02 MST ------- Setting to NEEDINFO as long as we don't have an upstream decision. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=220724 sndirsch@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |RESOLVED Info Provider|ssommer@novell.com | Resolution| |LATER ------- Comment #6 from sndirsch@novell.com 2007-01-30 13:37 MST ------- Setting to LATER as long as we don't have an upstream decision. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=220724#c7 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|LATER | --- Comment #7 from Stefan Dirsch <sndirsch@novell.com> 2007-08-09 13:27:13 MST --- Reopen. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724#c8 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |NEEDINFO Info Provider| |ssommer@novell.com --- Comment #8 from Stefan Dirsch <sndirsch@novell.com> 2007-08-09 13:38:10 MST --- Any improvements with openSUSE 10.3 Beta1? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QAContact|sndirsch@novell.com |xorg-maintainer-bugs@forge.provo.novell.com -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724#c9 Sascha Sommer <ssommer@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |REOPENED Info Provider|ssommer@novell.com | --- Comment #9 from Sascha Sommer <ssommer@novell.com> 2007-08-14 02:52:07 MST --- No ;( -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724#c10 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |RESOLVED Resolution| |LATER --- Comment #10 from Stefan Dirsch <sndirsch@novell.com> 2007-08-14 03:05:41 MST --- Right. My bad. It still has not been handled upstream. Probably you need to subscribe to the xorg ML and discuss this issue there (at least point to the bugreport), since most bugreports are currently ignored. :-( -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724#c11 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|LATER | --- Comment #11 from Stefan Dirsch <sndirsch@novell.com> 2007-08-16 14:45:17 MST --- Luc/Matthias wanted to handle this one. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724#c12 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|sndirsch@novell.com |lverhaegen@novell.com Status|REOPENED |NEW --- Comment #12 from Stefan Dirsch <sndirsch@novell.com> 2007-08-16 14:45:49 MST --- Reassigning. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724#c13 --- Comment #13 from Sascha Sommer <ssommer@novell.com> 2007-08-19 07:29:59 MST --- I resent the patch to the xorg mailinglist. Maybe I get an answer this time. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724#c15 Egbert Eich <eich@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Info Provider| |atilinuxnovellbugs@ati.com --- Comment #15 from Egbert Eich <eich@novell.com> 2007-08-20 02:27:26 MST --- Question for ATI: what would be the right alorithm to calculate the integer value written to the RADEON_FP_HORZ_STRETCH (MMIO:0x028c) and RADEON_FP_VERT_STRETCH (MMIO:0x0290)? currently the code in radeon_driver.c does: float ratio = resolution / panel_size; stretch = (ratio * 4096 + 0.5); This means that any float whose fractional part is > 0.5 gets rounded up before converting it to an int. This gives a slight overscaling in these cases. In the sources this algorithm can be followed all the way back to the XFree86 code. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724#c16 --- Comment #16 from Lisa Wu <liswu@ati.com> 2007-08-20 21:40:41 MST --- when RMX_AUTO_RATIO_HORZ_INC set to 1,Horizontal auto ratio result trucated, and then incremented by 1. Horz_Ratio = ( ((Active display width in characters (including overscan) + 1) / (Panel width in characters)) x 4096 + 1 ) else Horz_Ratio = ( ((Active display width in characters (including overscan)) / (Panel width in characters)) x 4096 + 1 ) when RMX_AUTO_RATIO_VERT_INC set to 1, Vertical auto ratio result trucated, and then incremented by 1. Vert_Ration = ( ((Active display width in characters (including overscan) + 1) / (Panel width in characters)) x 4096 + 1) else Vert_Ration = ( ((Active display width in characters (including overscan)) / (Panel width in characters)) x 4096 + 1) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724#c17 --- Comment #17 from Egbert Eich <eich@novell.com> 2007-08-20 23:00:17 MST --- Cool! Thx Lisa! I guess this algorithm is easy to implement. RMX_AUTO_RATIO_HORZ_INC seems to be called RADEON_HORZ_AUTO_RATIO_INC in the radeon driver and is bit 31 in register RADEON_FP_HORZ2_STRETCH (at 0x038c). There is no mention of a RADEON_VERT_AUTO_RATIO_INC in the header file. I'd assume it should be in register RADEON_FP_VERT2_STRETCH (at 0x0390).
From analogy I'd assume it's bit 31 also, however this bit is marked reserved. # define RADEON_VERT_STRETCH_RESERVED 0xf1000000 Could it be that this bit did not exist on some generations of the hardware? I assume if we just read it (not set it) it should be save in any case as if it doesn't exist on some hardware it should return 0.
-- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724#c18 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|lverhaegen@novell.com |eich@novell.com Status|NEEDINFO |NEW Info Provider|atilinuxnovellbugs@ati.com | --- Comment #18 from Stefan Dirsch <sndirsch@novell.com> 2007-08-20 23:41:55 MST --- Reassigning to Egbert. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724#c19 --- Comment #19 from Lisa Wu <liswu@ati.com> 2007-08-21 00:33:06 MST --- i don't know why bit RMX_AUTO_RATIO_VERT_INC initially marked reserved in the driver, as far as I know, of most ati chips, RMX_AUTO_RATIO_VERT_INC is bit31 of RADEON_FP_VERT_STRETCH. however, i agree it's better just read this bit but not set it. and here is some mistake I would like to point out, in fact, RMX register only take effect on primary CRTC, that is to say, there is ,in fact, no RADEON_FP_VERT2_STRETCH (at 0x0390) nor RADEON_FP_HORZ2_STRETCH (at 0x038c) at all. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724 User eich@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=220724#c20 --- Comment #20 from Egbert Eich <eich@novell.com> 2008-04-14 22:37:45 MST --- Created an attachment (id=207880) --> (https://bugzilla.novell.com/attachment.cgi?id=207880) Implementation of the RMX stretch algorithm for latest GIT head. This patch attempts to implement the stretching algorithm as described by Lisa. It furthermore avoids the use of floating point variables by adjusting the order of multiplication and divisions appropriately. Unfortunately I cannot test this patch myself other than make sure it builds cleanly. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=220724#c22 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Info Provider| |ssommer@novell.com --- Comment #22 from Stefan Dirsch <sndirsch@novell.com> 2008-04-15 00:23:44 MST --- Sascha, could you give Egbert's patch a try and verify that it works for you? Thanks. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=220724#c23 --- Comment #23 from Stefan Dirsch <sndirsch@novell.com> 2008-04-23 06:15:42 MST --- Sascha, still available for feedback? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=220724#c24 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|sndirsch@novell.com |eich@novell.com Status|NEEDINFO |NEW Info Provider|ssommer@novell.com | --- Comment #24 from Stefan Dirsch <sndirsch@novell.com> 2008-04-25 14:47:36 MST --- I'm afraid Sascha is no longer available for testing. Reassigning back to Egbert to discuss the patch with Alex. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724 User eich@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=220724#c25 Egbert Eich <eich@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #25 from Egbert Eich <eich@novell.com> 2008-08-26 02:25:57 MDT --- I've fixed and pushed this upstream in May/June already. Since there is no one to test this any more there is no point in keeping this bug open. With the upstream fix I assume the issue is resolved and the problem is FIXED. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=220724 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=220724#c26 --- Comment #26 from Stefan Dirsch <sndirsch@novell.com> 2008-08-26 02:46:21 MDT --- Yes, the patch is already included in xf86-video-ati 6.9.0, which will be shipped with openSUSE 11.1. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com