http://bugs.freedesktop.org/show_bug.cgi?id=14129

           Summary: R300 regression caused by git change
                    aceccda56b08338e217991e54607f1c9f18fc3e6
           Product: DRI
           Version: DRI CVS
          Platform: x86 (IA32)
        OS/Version: Linux (All)
            Status: NEW
          Severity: normal
          Priority: medium
         Component: General
        AssignedTo: dri-devel@lists.sourceforge.net
        ReportedBy: [EMAIL PROTECTED]


Playing half-life 2 under wine, I thought I'd try the micro-tiling patches
which allow DXT compression with multitexturing to work correctly.

Unfortunately it turns out that git HEAD has introduced some fairly visible
rendering errors, combined with a very obvious performance drop (without the
microtiling patches applied: this is just the dri git sources).

Doing a git bisect narrows the problem down to this change:

[EMAIL PROTECTED]:~/code/DRI/mesa-bisect$ git bisect good
aceccda56b08338e217991e54607f1c9f18fc3e6 is first bad commit
commit aceccda56b08338e217991e54607f1c9f18fc3e6
Author: Kristian Høgsberg <[EMAIL PROTECTED]>
Date:   Thu May 10 15:52:22 2007 -0400

    Drop __DRInativeDisplay and pass in __DRIscreen pointers instead.

    Many DRI entry points took a __DRInativeDisplay pointer and a screen
    index as arguments.  The only use for the native display pointer was to
    pass it back to the loader when looking up the __DRIscreen for the given
    screen index.

    Instead, let's just pass in the __DRIscreen pointer directly, which
    let's drop the __DRInativeDisplay type and the getScreen function.

    The assumption is now that the loader will be able to retrieve context
    from the __DRIscreen pointer when necessary.

:040000 040000 4c70ec7d8475ee44b25b4d900bf8088e29f30f72
231d46d59379c83c7d7af8a8ff6734cdbbb25cb5 M      include
:040000 040000 6d0383baa53e0b3cc3f4995c6541d8d41386ca8a
17e2b60c4c7d4435b14de5eaa59cc6266549bb7a M      src

I'll attach screenshots of the output of both this code and that which
immediately preceeds it.

It's possible that the DRM code is correct and wine is doing something wrong of
course. I don't see any performance regression with quake3.

The hardware is an AGP Radeon 9550:
01:00.0 VGA compatible controller [0300]: ATI Technologies Inc RV350 AS [Radeon
9550] [1002:4153]
Platform:
Linux kantaka 2.6.23-1-686 #1 SMP Fri Dec 21 13:57:07 UTC 2007 i686 GNU/Linux
(Debian unstable as of today)
Wine version 0.9.52


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to