Bug#398860: savage: a fix for respawn crash regression

2007-01-03 Thread Alex Deucher

Committed.  thanks!

Alex

On 12/24/06, Alex Deucher <[EMAIL PROTECTED]> wrote:

This is the right fix.  I'll check it in the next few days.

Thanks!

Alex

On 12/22/06, Mikko Rapeli <[EMAIL PROTECTED]> wrote:
> Hello,
>
> The savage driver some time a go started crashing when respawned from
> kdm et al. When MapMMIO and MapFB functions were merged to MapMem[1], one
> MapFB call in SavageScreenInit was not replaced with MapMem:
>
> 
http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-savage.git;a=commitdiff_plain;h=2f8352df6488476b0c1a46798eca5dd38827444b;hp=6f9abbb972834561cd8494a1d4fb47402b285d7d
>
> Here's a patch which fixes the issue. It works on my Thinkpad T20,
> though I don't really understand the details.
>
> -Mikko
>
> --- xserver-xorg-video-savage-2.1.2.orig/src/savage_driver.c
> +++ xserver-xorg-video-savage-2.1.2/src/savage_driver.c
> @@ -3088,6 +3088,9 @@
>
>  SavageEnableMMIO(pScrn);
>
> +if (!SavageMapMem(pScrn))
> +return FALSE;
> +
>  psav->FBStart2nd = 0;
>
>  if (psav->overlayDepth) {
>
> ___
> xorg mailing list
> [EMAIL PROTECTED]
> http://lists.freedesktop.org/mailman/listinfo/xorg
>




--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]



Bug#398860: savage: a fix for respawn crash regression

2006-12-22 Thread Mikko Rapeli
Hello, 

The savage driver some time a go started crashing when respawned from
kdm et al. When MapMMIO and MapFB functions were merged to MapMem[1], one 
MapFB call in SavageScreenInit was not replaced with MapMem:

http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-savage.git;a=commitdiff_plain;h=2f8352df6488476b0c1a46798eca5dd38827444b;hp=6f9abbb972834561cd8494a1d4fb47402b285d7d

Here's a patch which fixes the issue. It works on my Thinkpad T20,
though I don't really understand the details.

-Mikko

--- xserver-xorg-video-savage-2.1.2.orig/src/savage_driver.c
+++ xserver-xorg-video-savage-2.1.2/src/savage_driver.c
@@ -3088,6 +3088,9 @@
  
 SavageEnableMMIO(pScrn);
 
+if (!SavageMapMem(pScrn))
+return FALSE;
+
 psav->FBStart2nd = 0;
 
 if (psav->overlayDepth) {



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]