Re: sluggish behaviour with native Xorg and mga(4) [FIXED]

[Jukka Salmi <j+nbsd%2009.salmi.ch@localhost>]

Jukka Salmi --> tech-x11 (2009-07-01 22:05:53 +0200):
> Hello,
> 
> after having switched a -current system running XFree86 to NetBSD 5 and
> (native) Xorg I'm seeing severe performance problems with mga(4): tasks
> like e.g. moving a window or scrolling down a page in firefox take
> several seconds to complete.  The system has a Matrox G450 card, BTW.
> 
> While trying to debug this problem I noticed that if I manually run
> `./configure [...]; make' in xsrc/external/mit/xf86-video-mga/dist and
> use the resulting mga_drv.so instead of the one built by NetBSD's build
> framework (i.e. src/external/mit/xorg/server/drivers/xf86-video-mga),
> then I _don't_ see those performance problems!  Moving windows etc. is
> as fast as it used to be.
> 
> Unfortunately I failed to find out what exactly causes the problems when
> building using the build framework, even after removing many flags from
> CPPFLAGS.  That's why I attached outputs from both my manual build and
> the one using the build framework.  Help is appreciated!

The relevant difference turned out to be XAA (thanks to Matthew Green
for the hint!).  With the attached patch the driver built by NetBSD's
build framework is perfectly usable again.


Regards, Jukka

-- 
This email fills a much-needed gap in the archives.

Index: external/mit/xorg/server/drivers/xf86-video-mga/Makefile
===================================================================
RCS file: 
/cvsroot/src/external/mit/xorg/server/drivers/xf86-video-mga/Makefile,v
retrieving revision 1.4
diff -u -p -r1.4 Makefile
--- external/mit/xorg/server/drivers/xf86-video-mga/Makefile    1 Sep 2008 
04:21:04 -0000       1.4
+++ external/mit/xorg/server/drivers/xf86-video-mga/Makefile    2 Jul 2009 
07:34:01 -0000
@@ -17,10 +17,12 @@ SRCS+=              mga_dri.c
 
 #if USE_EXA
 SRCS+=         mga_exa.c
+CPPFLAGS+=     -DUSE_EXA
 #endif
 
 #if USE_XAA
 SRCS+=         mga_storm.c
+CPPFLAGS+=     -DUSE_XAA
 #endif
 
 CPPFLAGS+=     -I${DESTDIR}${X11INCDIR}/X11