Bug 29652 - Git commit e62e5b0 breaks Civilization IV (wine)
Summary: Git commit e62e5b0 breaks Civilization IV (wine)
Status: RESOLVED FIXED
Alias: None
Product: Mesa
Classification: Unclassified
Component: Drivers/DRI/R600 (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: Default DRI bug account
QA Contact:
URL: http://jon.severinsson.net/mesa/Civ4.png
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-18 11:53 UTC by Jon Severinsson
Modified: 2010-08-26 18:22 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
r600: Partially revert e62e5b0 "r600: add support for draw_elements_base_vertex" (1.30 KB, patch)
2010-08-24 03:53 UTC, Jon Severinsson
Details | Splinter Review
vb size fix (696 bytes, patch)
2010-08-25 00:12 UTC, Andre Maasikas
Details | Splinter Review

Description Jon Severinsson 2010-08-18 11:53:56 UTC

    
Comment 1 Jon Severinsson 2010-08-18 12:06:35 UTC
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...
Comment 2 Jon Severinsson 2010-08-24 03:53:05 UTC
Created attachment 38122 [details] [review]
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).
Comment 3 Andre Maasikas 2010-08-25 00:12:05 UTC
Created attachment 38136 [details] [review]
vb size fix

Can you try if this patch helps
Comment 4 Jon Severinsson 2010-08-25 06:44:21 UTC
(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.
Comment 5 Jon Severinsson 2010-08-26 18:22:00 UTC
Fixed in git 059950, thanks.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.