xf86UDelay was used - removal breaks drivers

Alan Coopersmith Alan.Coopersmith at Sun.COM
Fri Nov 6 17:50:35 PST 2009


Luc Verhaegen wrote:
> On Fri, Nov 06, 2009 at 03:31:27PM -0800, Alan Coopersmith wrote:
>> This commit:
>>
>>    commit 9522ab9a98e98bc394703038e34b1c12aa39a179
>>    Author: Matt Turner <mattst88 at gmail.com>
>>    Date:   Mon Aug 31 16:12:29 2009 -0400
>>
>>     Remove unused Delay.c
>>
>> has broken a number of drivers that used the previously exported
>> xf86UDelay function, including:
>> 	xf86-video-ast
>> 	xf86-video-chips
>> 	xf86-video-glint
>> 	xf86-video-neomagic
>> 	xf86-video-rendition
>>
>> Are updates going to be issued to all these drivers to use a different
>> function, or should the removal be reverted?
> 
> Apart from the fact that i find changes like this horribly shortsighted 
> and stupid, i wonder why tinderbox did not catch this.

We caught it while working on the updates to our packages, since we force
resolution of all symbols when building drivers, but I don't think the
X.Org upstream builds do that.   (For it to work for us on Solaris, we
have to generate a linker mapfile listing the exported symbols from the
Xorg binary that's used to satisfy the linker when checking.)

It does appear some distros have had bugs reported on it though:
	http://bugs.gentoo.org/291825
	https://bugzilla.redhat.com/show_bug.cgi?id=523800

-- 
	-Alan Coopersmith-           alan.coopersmith at sun.com
	 Sun Microsystems, Inc. - X Window System Engineering



More information about the xorg-devel mailing list