<br><br><div class="gmail_quote">On Wed, Dec 1, 2010 at 9:30 PM, Boudewijn Rempt <span dir="ltr"><<a href="mailto:boud@valdyas.org">boud@valdyas.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On Wednesday 01 December 2010, Dmitry Kazakov wrote:<br>
> > We have random failure of the KisLayerTest, and I am wondering if it is not<br>
> > associated with some of the random crash we have in krita. Following Mark's<br>
> > suggestion on IRC I ran helgrind on KisLayerTest (running it on krita is<br>
> > impossible), and I get the following errors. I hope someone (ie Dmitry :D)<br>
> > can<br>
> > figure out something out of it.<br>
> ><br>
><br>
> Hm.. i will try to reproduce this crash.<br>
<br>
</div>It's more than a crash: helgrind clearly shows we have some issues in the threading code for the projection calculation. Whether or not you can actually reproduce the crash isn't that much of an issue, as long as we can make the layer test run cleanly under helgrind :-)<br>
</blockquote><div><br>The problem is, hellgring output is almost useless here, as tiles engine uses a lockless approach for the data access. But the hellgrind is intended to work with the structures protected with mutexes and etc, so it'll think our code a complete solid issue ;). So it doesn't proof much. There is a couple of interesting lines, though.<br>
<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">
> But it'll happen in the beginning<br>
> of the next week. I have to do some stuff for the university right now.<br>
<br>
</div>I think that means that we should wait with the RC until you've had time to look at the issues. To me, this is clearly a release blocking issue.<br></blockquote></div><br>If it is reproducible (esp. on different systems), this is surely an issue =) Just don't rely on hellgrind output much.<br>
<br clear="all"><br>-- <br>Dmitry Kazakov<br>