Re: [PATCH] Comm::TcpAcceptor::doAccept fd limit handling is broken

2014-07-17 Thread Alex Rousskov
On 07/16/2014 06:46 PM, Amos Jeffries wrote: > On 17/07/2014 11:10 a.m., Alex Rousskov wrote: >> On 07/16/2014 02:38 PM, Rainer Weikusat wrote: >>> Alex Rousskov writes: On 07/16/2014 11:11 AM, Rainer Weikusat wrote: > This is broken because it re-enables readable notifications even if no

Re: [PATCH] Comm::TcpAcceptor::doAccept fd limit handling is broken

2014-07-16 Thread Amos Jeffries
On 17/07/2014 11:10 a.m., Alex Rousskov wrote: > On 07/16/2014 02:38 PM, Rainer Weikusat wrote: >> Alex Rousskov writes: >>> On 07/16/2014 11:11 AM, Rainer Weikusat wrote: This is broken because it re-enables readable notifications even if no connection was accepted. > >>> In other wor

Re: [PATCH] Comm::TcpAcceptor::doAccept fd limit handling is broken

2014-07-16 Thread Alex Rousskov
On 07/16/2014 02:38 PM, Rainer Weikusat wrote: > Alex Rousskov writes: >> On 07/16/2014 11:11 AM, Rainer Weikusat wrote: >>> This is broken because it re-enables readable notifications even if no >>> connection was accepted. >> In other words, it re-enables a check for new connections when >> >>