<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [bisected] Anvil accesses uninitialized memory while compiling shaders"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=104226#c12">Comment # 12</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [bisected] Anvil accesses uninitialized memory while compiling shaders"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=104226">bug 104226</a>
              from <span class="vcard"><a class="email" href="mailto:currojerez@riseup.net" title="Francisco Jerez <currojerez@riseup.net>"> <span class="fn">Francisco Jerez</span></a>
</span></b>
        <pre>(In reply to Józef Kucia from <a href="show_bug.cgi?id=104226#c11">comment #11</a>)
<span class="quote">> (In reply to Francisco Jerez from <a href="show_bug.cgi?id=104226#c10">comment #10</a>)
> > Created <span class=""><a href="attachment.cgi?id=136240" name="attach_136240" title="0001-intel-fs-bank_conflicts-Use-posix_memalign-instead-o.patch">attachment 136240</a> <a href="attachment.cgi?id=136240&action=edit" title="0001-intel-fs-bank_conflicts-Use-posix_memalign-instead-o.patch">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=104226&attachment=136240'>[review]</a> [review] [review]
> > 0001-intel-fs-bank_conflicts-Use-posix_memalign-instead-o.patch
> > 
> > I think I would prefer to address the problem at the allocation point as in
> > the attached patch.  Can you confirm whether the attachment fixes the crash
> > for you since I'm unable to reproduce it?
> The patch fixes the crash.</span >

Great, I'll send it for review once I get some results from our CI.

<span class="quote">> I'm not sure if it's the best solution though, e.g. _mesa_align_malloc() might
> be preferred instead of posix_memalign().</span >

I shortly considered using _mesa_align_malloc(), however I don't think it's
available to the vulkan driver.  Not a big deal, I think posix_memalign should
be fine, the i965 driver is not expected to run on Windows.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the QA Contact for the bug.</li>
      </ul>
    </body>
</html>