Do you mean these two?<div><span class="Apple-style-span" style="border-collapse: collapse; color: rgb(32, 32, 32); font-family: 'Droid Sans', arial, sans-serif; font-size: 13px; "><br>* L139: binary:match is probably too expensive to simply find byte 255.<br>

* L176: we are constructing a new binary that copies the message to send</span></div><div><font class="Apple-style-span" color="#202020" face="'Droid Sans', arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;"><br>

</span></font></div><div><font class="Apple-style-span" color="#202020" face="'Droid Sans', arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;">I tried to convert the second one to iolist, with no effect. The first one I didn't change. </span></font></div>

<div><font class="Apple-style-span" color="#202020" face="'Droid Sans', arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;"><br></span></font></div><div><font class="Apple-style-span" color="#202020" face="'Droid Sans', arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;">All my change to Cowboy is attached in a diff file in my previous post. Can you cook a new diff that I can apply to your stock version with your proposed change so that I can try?</span></font></div>

<div><font class="Apple-style-span" color="#202020" face="'Droid Sans', arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;"><br></span></font></div><div><font class="Apple-style-span" color="#202020" face="'Droid Sans', arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;">Thanks</span></font></div>

<div><font class="Apple-style-span" color="#202020" face="'Droid Sans', arial, sans-serif"><span class="Apple-style-span" style="border-collapse: collapse;">Andy<br></span></font><br><div class="gmail_quote">On Tue, May 31, 2011 at 5:30 PM, Loïc Hoguin <span dir="ltr"><<a href="mailto:essen@dev-extend.eu">essen@dev-extend.eu</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Hello,<br>
<br>
Did you try my proposed changes?<br>
<div class="im"><br>
On 05/31/2011 08:18 AM, Andy W. Song wrote:<br>
> Cowboy can't pass my test this time. Connections start dropping from<br>
> 20K, don't know why.<br>
<br>
</div>Works fine here.<br></blockquote><div>Do you mean using my C version client? </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
I've noted a few more things however:<br>
<br>
* Your hibernate call is in the wrong place, it should be right before<br>
the receive, as there could be more than one frame queued and you'll<br>
ignore it by hibernating right after the reply to the first frame.<br></blockquote><div>I don't think so. If before receive, what parameter do you give hibernate?</div><div>And in my test, each connection will receive only one packet in every 5 minutes. So there is no queued up frame at all and that is the reason to use hibernate.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
* You should use erlang:hibernate, not proc_lib:hibernate, as this<br>
process isn't started using proc_lib.<br></blockquote><div>Tried, no difference.  </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
* If you don't need the HTTP headers or any HTTP stuff from the request<br>
after socket initialization, you should just remove it from the record.<br>
Include "cowboy/http.hrl" and change the #http_req contents to remove<br>
the extra cruft. You'll use less than half you currently use doing this.<br>
<font color="#888888"><br>
--<br>
Loïc Hoguin<br>
Dev:Extend<br>
</font></blockquote></div><br><br clear="all"><br>-- <br><div>---------------------------------------------------------------</div><div>有志者，事竟成，破釜沉舟，百二秦关终属楚</div><div>苦心人，天不负，卧薪尝胆，三千越甲可吞吴</div><br>
</div>