<div dir="ltr">Thanks for pointing out that the workers get signalled multiple times, I missed that indeed. In that case, termination of the module under valgrind takes a little longer then I thought it did, yet the problem remains the same. <div>So the upper boundary of 1000 ms for the iteration has to remain fixed ? In that case, we'll have a patch to maintain (or see if we can round up in less time). </div><div><br></div><div>Thanks!</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 23, 2015 at 11:18 PM, Maxim Dounin <span dir="ltr"><<a href="mailto:mdounin@mdounin.ru" target="_blank">mdounin@mdounin.ru</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello!<br>
<span class=""><br>
On Mon, Mar 23, 2015 at 08:39:46PM +0100, Otto van der Schaaf wrote:<br>
<br>
> Hi,<br>
><br>
> For testing quick termination during high loads, while running with<br>
> valgrind, it might be useful to be able to extend the amount of time nginx<br>
> allows child processes to wrap up before sending SIGKILL.<br>
> For ngx_pagespeed, the current hard-coded default of 1 second seems to be<br>
> just short of what we need to be able to reliably test just this scenario,<br>
> so I've made a patch so we can run with different values for with and<br>
> without valgrind.<br>
> Would the following patch be acceptable?<br>
<br>
</span>No.  Note well that current limit isn't 1 second, you are<br>
misunderstanding the code.  It'll signal workers multiple times,<br>
doubling wait time on each iteration, from 50ms to 1000ms<br>
including.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Maxim Dounin<br>
<a href="http://nginx.org/" target="_blank">http://nginx.org/</a><br>
<br>
_______________________________________________<br>
nginx-devel mailing list<br>
<a href="mailto:nginx-devel@nginx.org">nginx-devel@nginx.org</a><br>
<a href="http://mailman.nginx.org/mailman/listinfo/nginx-devel" target="_blank">http://mailman.nginx.org/mailman/listinfo/nginx-devel</a><br>
</font></span></blockquote></div><br></div>