Hi, I installed the packages from opensuse/repositories/X11:/Xorg:/testing on my openSUSE 11.0 / x86_64 machine and current mesa master. Chipset is 690V. - I'm seeing tiny pixel corruptions all over the screen. Maybe some exa accel bugs. Is there some extension I could disable to debug? - My /var/log/messages is full of: kernel: [drm] wait idle failed status : 0x9401C100 0x00080100 kernel: [drm] wait for fifo failed status : 0x9401C100 0x00080100 messages. (numbers are changing sometimes to 0xD411C100 0xBD080100 ) - glxgears performance is ~350 fps compared to ~450 fps in software rendering mode :-( Attached is my xorg.log Thanks! Marc On Monday 30 June 2008 12:09:45 Egbert Eich wrote:
Reporting from xf86-video-radeonhd: Radeon HD video driver
Please visit: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd Or get your own copy by using: git-clone git://anongit.freedesktop.org/xorg/driver/xf86-video-radeonhd
15 commits against quick_and_dirty_2d at 205efb64...: src/Makefile.am | 4 src/radeon_accel.c | 207 ++----------------------------- src/radeon_accel.h | 136 ++++++++++++++++++++ src/radeon_accelfuncs.c | 6 src/radeon_commonfuncs.c | 16 +- src/radeon_exa.c | 43 +++++- src/radeon_exa.h | 39 +++++ src/radeon_exa_funcs.c | 9 - src/radeon_exa_render.c | 187 +++++++++++++--------------- src/radeon_textured_video.c | 7 - src/radeon_textured_videofuncs.c | 2 src/radeon_video.c | 10 - src/radeon_xaa.c | 256 +++++++++++++++++++++++++++++++++++++++ src/radeon_xaa.h | 40 ++++++ src/rhd.h | 151 ----------------------- src/rhd_cp.h | 82 ++++++++++-- src/rhd_dri.c | 36 +++++ src/rhd_driver.c | 118 ++++++++++------- 18 files changed, 822 insertions(+), 527 deletions(-)
commit 14f49f1632790623738d05a48a065d8622204de0 Author: Egbert Eich
Date: Mon Jun 30 12:00:30 2008 +0200 DRI: Add more debugging output to DRI.
diffstat: src/rhd_dri.c | 35 ++++++++++++++++++++++++++++++++++- 1 file changed, 34 insertions(+), 1 deletion(-)
gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h =14f49f16
commit 570039614d32d25f0b5ee7a796095ccb56e00e46 Author: Egbert Eich
Date: Mon Jun 30 10:30:27 2008 +0200 EXA: Get rid of static variables.
In EXA variables carrying state between different accel functions have been made static. Static variables are generally not a good idea in X drivers as multiple instances of these drivers may exist. Although this may no matter in some use cases (ie. when static variables are only used to pass information between functions that are executed within the same drawing command) it's generally a good idea to avoid them.
diffstat: src/radeon_accel.h | 12 ++++ src/radeon_exa.c | 12 +--- src/radeon_exa_render.c | 132 +++++++++++++++++++++----------------------- 3 files changed, 80 insertions(+), 76 deletions(-)
gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h =14f49f16
commit 7794869b198df85c26232b46a52a9dccab31a6d9 Author: Egbert Eich
Date: Mon Jun 30 09:41:49 2008 +0200 2D: Add Close() functions for XAA and EXA.
This allows to push some of the detailed knowledge about the accel methods' internals down to their subsystems.
diffstat: src/radeon_exa.c | 13 +++++++++++++ src/radeon_exa.h | 1 + src/radeon_xaa.c | 17 +++++++++++++++-- src/radeon_xaa.h | 1 + src/rhd_driver.c | 43 ++++++++----------------------------------- 5 files changed, 38 insertions(+), 37 deletions(-)
gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h =14f49f16
commit 157aa4fded056a72eea2e747ca3f6e0e46017831 Author: Egbert Eich
Date: Mon Jun 30 08:21:02 2008 +0200 2D: Split RADEONAccelInit() into XAA and EXA specific function.
This pushes the accel specific init code into the accel specific subsystem.
diffstat: src/radeon_accel.c | 49 ------------------------------------------------- src/radeon_accel.h | 1 - src/radeon_exa.c | 16 ++++++++++++++++ src/radeon_exa.h | 7 +------ src/radeon_xaa.c | 38 +++++++++++++++++++++++++++++++++++--- src/radeon_xaa.h | 7 +------ src/rhd_driver.c | 17 ++++++----------- 7 files changed, 59 insertions(+), 76 deletions(-)
gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h =14f49f16
commit 08eb8945f4b23ae746e7c04e3cbdf9cec6ef7388 Author: Egbert Eich
Date: Mon Jun 30 10:52:33 2008 +0200 2D: Separate XAA from common accel code, separate func decls to XAA, EXA and common accel headers.
This patch separates the XAA and EXA subsystems from the common accel code.
diffstat: src/Makefile.am | 4 src/radeon_accel.c | 165 ++++--------------------------- src/radeon_accel.h | 125 +++++++++++++++++++++++ src/radeon_commonfuncs.c | 2 src/radeon_exa.c | 2 src/radeon_exa.h | 43 ++++++++ src/radeon_textured_video.c | 1 src/radeon_video.c | 1 src/radeon_xaa.c | 211 ++++++++++++++++++++++++++++++++++++++++ src/radeon_xaa.h | 44 ++++++++ src/rhd.h | 131 ------------------------ src/rhd_cp.h | 4 src/rhd_dri.c | 1 src/rhd_driver.c | 5 14 files changed, 464 insertions(+), 275 deletions(-)
gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h =14f49f16
commit 46a0006177a7af85129f4bbb6899ffdea962fc86 Author: Egbert Eich
Date: Sat Jun 28 10:53:32 2008 +0200 2D: Comment out setctions not needed by this driver.
diffstat: src/radeon_commonfuncs.c | 14 +++++++++----- src/radeon_exa_funcs.c | 9 +++++---- 2 files changed, 14 insertions(+), 9 deletions(-)
gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h =14f49f16
commit f3e29a4dc84ab72ab39bd7f8e607900f7493d024 Author: Egbert Eich
Date: Sat Jun 28 10:45:03 2008 +0200 2D: Changed test for AccelMethod to test for presence of method specific data struct.
Testing for the presence of the data struct is more portable. It requires that it is made shure the data struct is only there when the accel method is really initialized. This patch also changees the (historic) name accel to xaa to clearer reflect that this data structure belong to XAA.
diffstat: src/radeon_accel.c | 2 +- src/radeon_textured_video.c | 6 +++--- src/radeon_textured_videofuncs.c | 2 +- src/radeon_video.c | 8 ++++---- src/rhd.h | 21 +++++++++++---------- src/rhd_cp.h | 10 +++++----- src/rhd_driver.c | 12 ++++++------ 7 files changed, 31 insertions(+), 30 deletions(-)
gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h =14f49f16
commit b4e61f1a0339af9a83e5b218ebc4622f076ef416 Author: Egbert Eich
Date: Fri Jun 27 21:19:03 2008 +0200 2D: Removed unneeded variable XAAForceTransBlit from accel_state.
diffstat: src/radeon_accelfuncs.c | 6 +++++- src/rhd.h | 7 ------- 2 files changed, 5 insertions(+), 8 deletions(-)
gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h =14f49f16
commit 3348c60762628ef33dbeb90ec66f2eea09b35d70 Author: Egbert Eich
Date: Fri Jun 27 21:13:27 2008 +0200 2D: Removed unneeded variables re_top_left re_width_height from accel_state.
diffstat: src/radeon_accel.c | 7 ------- src/rhd.h | 8 +++----- src/rhd_cp.h | 5 +++-- 3 files changed, 6 insertions(+), 14 deletions(-)
gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h =14f49f16
commit 9abcc4dcb82e513056f10b65aa6625a20049377f Author: Egbert Eich
Date: Fri Jun 27 21:01:02 2008 +0200 2D: Removed unneeded scissor variables from accel_state.
diffstat: src/radeon_accel.c | 4 ---- src/rhd.h | 8 +------- 2 files changed, 1 insertion(+), 11 deletions(-)
gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h =14f49f16
commit d7b8c57bc06ecf22679a06a3f3d01470f7809b71 Author: Egbert Eich
Date: Fri Jun 27 20:57:11 2008 +0200 2D: Removed unneeded variable 'datatype' from accel_state.
diffstat: src/radeon_accel.c | 13 +++++++------ src/rhd.h | 1 - 2 files changed, 7 insertions(+), 7 deletions(-)
gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h =14f49f16
commit d456b7239e921ab4e01b8d2c7d9a3f2169962911 Author: Egbert Eich
Date: Fri Jun 27 20:40:07 2008 +0200 2D: Removed unneeded variable 'pitch' from accel_state.
diffstat: src/radeon_accel.c | 5 +++-- src/rhd.h | 1 - 2 files changed, 3 insertions(+), 3 deletions(-)
gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h =14f49f16
commit 2b1fa2a5baec39d76cca8c9b559b7b43331142d5 Author: Egbert Eich
Date: Fri Jun 27 20:24:17 2008 +0200 EXA: Restructure file to group together pieces required by this driver.
This patch also #ifdefs out pieces not needed by this driver.
diffstat: src/radeon_exa_render.c | 55 +++++++++++++++++++------------------------- 1 file changed, 24 insertions(+), 31 deletions(-)
gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h =14f49f16
commit 5344ea9afa8b7d8ae1b455344f0391f4e71f00c8 Author: Egbert Eich
Date: Wed Jun 25 22:55:27 2008 +0200 2D: Consolidated variables on which acceleration is used.
Both accelOn and useEXA were consolidated to AccelMethod. 2D memory setup and init was moved to a separate function.
diffstat: src/radeon_accel.c | 4 - src/radeon_textured_video.c | 6 - src/radeon_textured_videofuncs.c | 2 src/radeon_video.c | 9 +- src/rhd.h | 2 src/rhd_cp.h | 8 +- src/rhd_driver.c | 115 ++++++++++++++++++++++++----------- 7 files changed, 96 insertions(+), 50 deletions(-)
gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h =14f49f16
commit c1ee81fb896928f8baa8ace1e2c738e9dc8c72a4 Author: Egbert Eich
Date: Wed Jun 25 16:44:19 2008 +0200 CP: Add missing license and copyright statements to rhd_cp.h.
diffstat: src/rhd_cp.h | 61 +++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 61 insertions(+)
gitweb url: http://gitweb.freedesktop.org/?p=xorg/driver/xf86-video-radeonhd;a=commit;h =14f49f16
-- Dipl. Phys. Marc Dietrich Institut für Angewandte Physik - AG Kleinkühler - Justus-Liebig-Universität Gießen Heinrich-Buff-Ring 16 D-35392 Gießen Germany Tel. +49 641 99 33462 Fax. +49 641 99 33409 email: marc.dietrich@ap.physik.uni-giessen.de www: http://wwwiap.physik.uni-giessen.de/kleinkuehler/kleinkuehler.htm