Bug 22615

Summary: radeon-rewrite: bogus stride for small compressed mipmaps
Product: Mesa Reporter: Roland Scheidegger <sroland>
Component: Drivers/DRI/r200Assignee: Default DRI bug account <dri-devel>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: medium    
Version: unspecified   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments: suggested fix

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.

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.