<div>Please point us to the relevant patches that you believe are missing from 3.0 and I'll review them.</div><div><br></div><div>In the meanwhile, here's the commit that reverted one of your changes due to deadlocks.</div>
<div><br></div><div>Locking should be done on the backend not on the frontend.</div><div><br></div><div>commit 47dd377f8587b466475ac5a6cf548f49b9394d0d</div><div>Author: Rodrigo Kumpera <<a href="mailto:kumpera@gmail.com">kumpera@gmail.com</a>></div>
<div>Date:   Thu Nov 1 17:43:25 2012 -0400</div><div><br></div><div>    Revert "Merge pull request #464 from gregoryyoung/master"</div><div>    </div><div>    This commit causes deadlock in the tpool backend in the following way:</div>
<div>    </div><div>    thread 1:</div><div>        socket_io_add locks io_lock</div><div>        tp_poll_modify waits on new_sem</div><div>    </div><div>    thread 2:</div><div>        tp_poll_wait tries to lock io_lock</div>
<div>    </div><div>    tp_poll_wait is the responsible to post to new_sem, which it can't do since it's</div><div>    blocked on io_lock, held by a thread waiting on new_sem.</div><div>    </div><div>    This reverts commit 11286da0ac2e2bab7b2d8ab04b9f6a4da4e12131, reversing</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Nov 28, 2012 at 1:23 AM, Greg Young <span dir="ltr"><<a href="mailto:gregoryyoung1@gmail.com" target="_blank">gregoryyoung1@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I would be curious in talking a bit more about this because without it<br>
TCP does not work reliably.<br>
<br>
btw: having run billions of calls through TCP under load we have never<br>
seen a deadlock on it. Could you describe the deadlock scenario?<br>
<span class="HOEnZb"><font color="#888888"><br>
Greg<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Tue, Nov 27, 2012 at 9:37 PM, Rodrigo Kumpera <<a href="mailto:kumpera@gmail.com">kumpera@gmail.com</a>> wrote:<br>
> It depends on what patches. One I did merge had to be reverted due to<br>
> causing deadlocks.<br>
><br>
><br>
><br>
> On Tue, Nov 27, 2012 at 1:45 PM, Greg Young <<a href="mailto:gregoryyoung1@gmail.com">gregoryyoung1@gmail.com</a>> wrote:<br>
>><br>
>> 3.0.1? We are seeing some of the same kinds of issues with TCP as<br>
>> previously (eg call beginsend never get an endsend). There is<br>
>> discussion in the history of the list. I can go figure out which<br>
>> relevant patches might be missing but should we be expecting them to<br>
>> have been brought forward?<br>
><br>
><br>
<br>
<br>
<br>
</div></div><div class="HOEnZb"><div class="h5">--<br>
Le doute n'est pas une condition agréable, mais la certitude est absurde.<br>
</div></div></blockquote></div><br></div>