Bug 33886

Summary: fix gallium sampler view first_level usage
Product: Mesa Reporter: Brian Paul <brianp>
Component: Mesa coreAssignee: mesa-dev
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: medium    
Version: git   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments: patch for softpipe
patch for r300
patch for state tracker
second patch for state tracker
patch for r300 rebased
patch for llvmpipe

Description Brian Paul 2011-02-03 16:19:11 UTC
This is a bug report to avoid losing Henri Verbeet's patch series which fixes gallium's use of the first_level field of sampler views.

Basically, the texture lod should be computed relative to level==first_level, not level=0.  We have patches for softpipe and r300 but llvmpipe (at least) needs to be updated too.
Comment 1 Brian Paul 2011-02-03 16:19:40 UTC
Created attachment 42910 [details] [review]
patch for softpipe
Comment 2 Brian Paul 2011-02-03 16:20:01 UTC
Created attachment 42911 [details] [review]
patch for r300
Comment 3 Brian Paul 2011-02-03 16:20:21 UTC
Created attachment 42912 [details] [review]
patch for state tracker
Comment 4 Brian Paul 2011-02-03 16:20:43 UTC
Created attachment 42913 [details] [review]
second patch for state tracker
Comment 5 Marek Olšák 2011-03-03 19:23:28 UTC
Created attachment 44101 [details] [review]
patch for r300 rebased

Is there any reason not to commit the patches? They work fine on r300g and softpipe.
Comment 6 Brian Paul 2011-03-03 20:52:45 UTC
I'd like to get this resolved for llvmpipe too.  I just haven't had the time.
Comment 7 Fabian Bieler 2011-03-31 07:08:17 UTC
Created attachment 45093 [details] [review]
patch for llvmpipe
Comment 8 Marek Olšák 2011-04-07 12:53:23 UTC
Brian,

Fabian attached the patch for llvmpipe. Can I now push all of them?
Comment 9 Brian Paul 2011-04-07 17:46:32 UTC
I've taken a look at the llvmpipe changes and they look good.  But I haven't had time to apply all the patches and do any testing yet.

If you've tested with piglit, etc. and everything looks good then go ahead and commit the patches.
Comment 10 Marek Olšák 2011-04-07 19:52:41 UTC
All the variants of the tex-miplevel-selection test pass with softpipe, llvmpipe, and r300g (for the first time!). All patches have been pushed. Closing.

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.