<div dir="ltr">On 27 August 2013 16:51, Damien Lespiau <span dir="ltr"><<a href="mailto:damien.lespiau@intel.com" target="_blank">damien.lespiau@intel.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Wed, Jul 24, 2013 at 10:02:22AM -0700, Kenneth Graunke wrote:<br>
> On 07/24/2013 09:33 AM, Paul Berry wrote:<br>
> >Two callers of brw_search_cache() weren't initializing that function's<br>
> >inout_offset parameter: brw_blorp_const_color_params::get_wm_prog()<br>
> >and brw_blorp_const_color_params::get_wm_prog().<br>
> ><br>
> >That's a benign problem, since the only effect of not initializing<br>
> >inout_offset prior to calling brw_search_cache() is that the bit<br>
> >corresponding to cache_id in brw->state.dirty.cache may not be set<br>
> >reliably.  This is ok, since the cache_id's used by<br>
> >brw_blorp_const_color_params::get_wm_prog() and<br>
> >brw_blorp_blit_params::get_wm_prog() (BRW_BLORP_CONST_COLOR_PROG and<br>
> >BRW_BLORP_BLIT_PROG, respectively) correspond to dirty bits that are<br>
> >not used.<br>
> ><br>
> >However, failing to initialize this parameter causes valgrind to<br>
> >complain.  So let's go ahead and fix it to reduce valgrind noise.<br>
> ><br>
> >Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=66779" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=66779</a><br>
<br>
</div>Can we have this commit in the 9.2 branch? It was added in the release<br>
blocker bug but never cherry-picked for 9.2.<br>
<br>
Thanks!<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Damien<br>
</font></span></blockquote></div><br></div><div class="gmail_extra">Is there a reason for wanting this cherry-picked to 9.2 other than to follow procedure?  As the commit message notes it's a benign problem--all it does is cause false positives from valgrind.  I'd rather not mess with 9.2 if there's not going to be any benefit to users.</div>
</div>