Bug 22615 - radeon-rewrite: bogus stride for small compressed mipmaps
radeon-rewrite: bogus stride for small compressed mipmaps
Product: Mesa
Classification: Unclassified
Component: Drivers/DRI/r200
Other All
: medium normal
Assigned To: Default DRI bug account
Depends on:
  Show dependency treegraph
Reported: 2009-07-04 06:09 UTC by Roland Scheidegger
Modified: 2009-08-03 15:23 UTC (History)
0 users

See Also:

suggested fix (789 bytes, patch)
2009-07-04 06:09 UTC, Roland Scheidegger
Details | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Roland Scheidegger 2009-07-04 06:09:29 UTC
Created attachment 27379 [details] [review]
suggested fix

rowstride on small compressed mipmaps is wrong on r200/r100 (tested on r200). A quick look compared to pre-radeon-rewrite code suggests to me it's actually wrong for r300 too, but I'm not too sure about that, hence I'm not fixing this directly someone should test it on r300 as I don't know what the stride should really be on which chips, though I suspect it should be row_align just as for uncompressed textures (it can easily be seen in quake3 for instance when compressed textures are enabled some distant textures turn into random color bits).
Comment 1 Roland Scheidegger 2009-07-31 14:27:26 UTC
The wrong pitch is fixed (801c3fcbca69a17f0696522b91cbfc378094974b) but unfortunately it doesn't completely fix the problem so leaving this bug open.
Comment 2 Roland Scheidegger 2009-08-03 15:23:09 UTC
Fixed with 50c736589ee0edbedf9ac434e883483b82b3030a.