<div dir="ltr"><div>Moving all ticky bumps after the heapcheck did prevent the wiggling I noticed.</div><div> </div><div>Could you also weigh in on the location of the cost centre profiling entries?</div><div> </div><div>Thanks!</div>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Feb 18, 2013 at 2:19 PM, Simon Marlow <span dir="ltr">&lt;<a href="mailto:marlowsd@gmail.com" target="_blank">marlowsd@gmail.com</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 16/02/13 16:32, Nicolas Frisby wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid">
I noticed some suspicious wiggling in my ticky reports&#39; entry counts.<br>
SPJ suggested they were due to the counter ticking before the heap<br>
check: if it fails, then it ticks again when we re-enter after the<br>
stg_gc function does its thing.<br>
After looking into it a bit further, I have a couple questions. Here&#39;s<br>
what I found.<br>
   * for functions (StgCmmBind.closureCodeBody), tickyEnterFun and<br>
enterCostCentreFun are before the heap check<br>
   * for thunks (StgCmmBind.thunkCode), tickyEnterThunk is before the<br>
heap check and enterCostCentreThunk is after.<br>
Questions:<br>
   * Does any object to me moving both ticky counters after the heap check?<br>
   * Is there a reason the two cost centre profiling entries are on<br>
opposite sides of the heap check?<br>
</blockquote>
<br></div></div>
It sounds wrong to me.  If you could clean this up that would be great.  These counters should never be bumped before the heap check, because if the heap check fails then the bumps will happen twice.<br>
<br>
Cheers,<br>
        Simon<br>
<br>
</blockquote></div><br></div>