xf86-video-savage: add missing xf86CrtcConfigInit in SavagePreInit

Alex Villací­s Lasso a_villacis at palosanto.com
Tue Jan 13 10:40:49 PST 2009


Alex Deucher escribió:
> On Tue, Jan 13, 2009 at 11:36 AM, Alex Villací­s Lasso
> <a_villacis at palosanto.com> wrote:
>   
>> With the latest git for xserver and the latest git for savage, an attempt to
>> start up crashes the server unless this patch is applied. It seems that an
>> explicit xf86CrtcConfigInit() is now required in the PreInit stage of the
>> driver setup. I found via valgrind that the absence of this call causes
>> several invalid reads, but only the one triggered by DRIFinishScreenInit()
>> actually crashes the server on startup. Grepping through the git code, only
>> xf86-video-intel, xf86-video-ati, xf86-video-nv and xf86-video-siliconmotion
>> have this call in their code. Does this mean that any other driver that
>> tries to init DRI will crash in the same way with latest xserver?
>>     
>
> These just all happen to be drivers with xrandr 1.2 support.
> Something changed in the xserver that breaks non xrandr 1.2 drivers.
> You should probably file a bug (https://bugs.freedesktop.org) and make
> sure this gets fixed before xserver 1.6 ships.
>
> Alex
>
>   
Bug #19539 has been opened with the proposed patch.

-- 
perl -e '$x=2.4;print sprintf("%.0f + %.0f = %.0f\n",$x,$x,$x+$x);'




More information about the xorg mailing list