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">

We should clarify the meaning of the smp_sign->length field</blockquote><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">

in include/persistent.h:  Does it include struct smp_sign or not ?</blockquote><div><br></div><div>Comments about this added. </div></div><div><br></div>-- <br>
Martin Blix Grydeland<br>Varnish Software AS<br><br>
</div>
</div></div>