[Bug 29652] New: Git commit

[Bug 29652] New: Git commit

[bugzilla-daemon]

https://bugs.freedesktop.org/show_bug.cgi?id=29652

           Summary: Git commit
           Product: Mesa
           Version: git
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: medium
         Component: Drivers/DRI/R600
        AssignedTo: dri-devel@lists.freedesktop.org
        ReportedBy: jon@severinsson.net




-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 29652] Git commit e62e5b0 breaks Civilization IV (wine)

[bugzilla-daemon]

https://bugs.freedesktop.org/show_bug.cgi?id=29652

Jon Severinsson <jon@severinsson.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Git commit                  |Git commit e62e5b0 breaks
                   |                            |Civilization IV (wine)
                URL|                            |http://jon.severinsson.net/
                   |                            |mesa/Civ4.png
                 CC|                            |amaasikas@gmail.com

--- Comment #1 from Jon Severinsson <jon@severinsson.net> 2010-08-18 12:06:35 PDT ---
Tracking Mesa git I found that Civ IV stopped working recently, and I bisected
it to commit e62e5b0922b6554431234a8a71c7aec7cff91864 with subject "r600: add
support for draw_elements_base_vertex" by Andre Maasikas.

The problem that manifests is that the background for the game menue turns
black (though the menus still work), and that all in game tiles are black,
though all units and tile improvements (but not cities) on top of them still
render [see URL above].

I'll happily test any patches.

lspci identifies my GPU as:
01:00.0 VGA compatible controller: ATI Technologies Inc RV630 [Radeon HD 2600
Series]

PS. Excuse the originally empty bug, I managed to hit enter too early...

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 29652] Git commit e62e5b0 breaks Civilization IV (wine)

[bugzilla-daemon]

https://bugs.freedesktop.org/show_bug.cgi?id=29652

--- Comment #2 from Jon Severinsson <jon@severinsson.net> 2010-08-24 03:53:05 PDT ---
Created an attachment (id=38122)
 View: https://bugs.freedesktop.org/attachment.cgi?id=38122
 Review: https://bugs.freedesktop.org/review?bug=29652&attachment=38122

r600: Partially revert e62e5b0 "r600: add support for
draw_elements_base_vertex"

Looking deeper at the problem, it turned out that the problem was only the
changes to the index bounds checking in r700DrawPrims. This patch reverts those
changes, but leaves the rest of e62e5b0, which solves this bug, and *should*
keep draw_elements_base_vertex working (though I haven't figured out how to
test that).

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 29652] Git commit e62e5b0 breaks Civilization IV (wine)

[bugzilla-daemon]

https://bugs.freedesktop.org/show_bug.cgi?id=29652

--- Comment #3 from Andre Maasikas <amaasikas@gmail.com> 2010-08-25 00:12:05 PDT ---
Created an attachment (id=38136)
 View: https://bugs.freedesktop.org/attachment.cgi?id=38136
 Review: https://bugs.freedesktop.org/review?bug=29652&attachment=38136

vb size fix

Can you try if this patch helps

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 29652] Git commit e62e5b0 breaks Civilization IV (wine)

[bugzilla-daemon]

https://bugs.freedesktop.org/show_bug.cgi?id=29652

--- Comment #4 from Jon Severinsson <jon@severinsson.net> 2010-08-25 06:44:21 PDT ---
(In reply to comment #3)
> Can you try if this patch helps
It does!
I tested with a8177e7 + your patch (without my patch above) and it works
perfectly.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 29652] Git commit e62e5b0 breaks Civilization IV (wine)

[bugzilla-daemon]

https://bugs.freedesktop.org/show_bug.cgi?id=29652

Jon Severinsson <jon@severinsson.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #5 from Jon Severinsson <jon@severinsson.net> 2010-08-26 18:22:00 PDT ---
Fixed in git 059950, thanks.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.