Patch for bug #1912 applied incorrectly to 6.8 branch
Alan Coopersmith
Alan.Coopersmith at Sun.COM
Wed Feb 9 21:18:42 PST 2005
Michel Dänzer wrote:
> On Wed, 2005-02-09 at 20:53 -0800, Alan Coopersmith wrote:
>>>>https://bugs.freedesktop.org/attachment.cgi?id=1433
>>>>
>>>>a RADEONSetFBLocation() call was added to RADEONAdjustFrame().
>>>>RADEONSetFBLocation() should be called as rarely as possible because it
>>>>messes with the guts of the GPU memory controller.
>>
>>According to CVSweb, the RADEONSetFBLocation in RADEONAdjustFrame() came
>>from this patch, which was checked into 6.8.2 in Decemeber (rev 1.19.2.7):
>> * xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_driver.c
>> Bugzilla #1220 (https://bugs.freedesktop.org/show_bug.cgi?id=1220)
>> attachment #980 (https://bugs.freedesktop.org/attachment.cgi?id=980):
>> Fix garbage screen on radeon cards which may appear after a system
>> suspend/resume cycle.
>> Patch by Matthias Hopf <mhopf at suse.de>
>>
>>It appears a previous patch adding RADEONSetFBLocation to fix this same
>>bug [...]
>
>
> Same bug? This commit was supposed to be attachment #1433 for bug #1912
Sorry, I meant both of the patches listed in bugzilla for bug #1220 were
applied, that was the "same bug" I was referring to - the first one presumably
should have been removed when the second one went in, since they both added
RADEONSetFBLocation calls in different places.
--
-Alan Coopersmith- alan.coopersmith at sun.com
Sun Microsystems, Inc. - X Window System Engineering
More information about the xorg
mailing list