Bug 24585

Summary: radeon texture size checks with bit 11
Product: DRI Reporter: Mathias Fröhlich <Mathias.Froehlich>
Component: DRM/RadeonAssignee: 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:
Description Flags
Bit 11 texture check none

Description Mathias Fröhlich 2009-10-17 00:25:02 UTC 
Hi,

I have a problem with my r520 accepting mipmapped textures huger than 2048 with the current in kernel texture size check code.

The problem boils down to the order when the bit11 of the texture size is or'ed to the original width. In the end each mipmap level has the same width or height because of that 11 bit is ored to the scaled down lod with and thus blows up the size again to the full size or more due to the power of two rounding afterwards.

The attached patch changes this order so that the texture sizes are computed correct. Also the on error the yet missing inputs to the size computation are printed which helped me to find out where it really breaks.

Please apply or fix in a more appropriate way.
Thanks

Mathias
Comment 1 Nicolai Hähnle 2009-10-17 04:47:18 UTC 
Thanks again, but it seems the patch is missing here as well?
Comment 2 Mathias Fröhlich 2009-10-17 08:06:59 UTC 
Created attachment 30500 [details] [review]
Bit 11 texture check

Huch, so, now that I can debug mesa, I might need to learn how to use firefox :)

Hopefully attached now ...

Mathias
Comment 3 Alex Deucher 2009-11-05 11:58:57 UTC 
pushed to drm-next:
ceb776bc87280eb8d13f38e4d7afae757e95af44

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.