Review comments:<div><br></div><div><blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

All things considered, we should probably VSL these and live with<br>it.   SLT_PersistentEvent or something.</blockquote><div><br></div><div>Agreed, logging from persistent needs to be overhauled as a</div><div>whole. Putting on todo list.</div>

<div><br></div><div><blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

We should avoid whole-second sleeps, they have a bad tendency to<br>get things to run in lockstep.  Ideally each silo should sleep<br>for 1+random-small-delta, but at least make it different from 1.0</blockquote><div><br>

</div><div>Changed</div><div><br></div><blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

You should simply let the tread die, and pthread_join() it from<br>smp_close().</blockquote></div><div><br></div><div>Implemented</div><div><br></div><div><br></div><div><br></div>-- <br>Martin Blix Grydeland<br>Varnish Software AS<br>

<br>
</div>