[Mesa-stable] [Mesa-dev] [PATCH] i965: Initialize inout_offset parameter to brw_search_cache().

Damien Lespiau damien.lespiau at intel.com
Tue Aug 27 16:51:44 PDT 2013


On Wed, Jul 24, 2013 at 10:02:22AM -0700, Kenneth Graunke wrote:
> On 07/24/2013 09:33 AM, Paul Berry wrote:
> >Two callers of brw_search_cache() weren't initializing that function's
> >inout_offset parameter: brw_blorp_const_color_params::get_wm_prog()
> >and brw_blorp_const_color_params::get_wm_prog().
> >
> >That's a benign problem, since the only effect of not initializing
> >inout_offset prior to calling brw_search_cache() is that the bit
> >corresponding to cache_id in brw->state.dirty.cache may not be set
> >reliably.  This is ok, since the cache_id's used by
> >brw_blorp_const_color_params::get_wm_prog() and
> >brw_blorp_blit_params::get_wm_prog() (BRW_BLORP_CONST_COLOR_PROG and
> >BRW_BLORP_BLIT_PROG, respectively) correspond to dirty bits that are
> >not used.
> >
> >However, failing to initialize this parameter causes valgrind to
> >complain.  So let's go ahead and fix it to reduce valgrind noise.
> >
> >Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=66779

Can we have this commit in the 9.2 branch? It was added in the release
blocker bug but never cherry-picked for 9.2.

Thanks!

-- 
Damien


More information about the mesa-stable mailing list