<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 4, 2015 at 2:26 PM, Rafael Zalamena <span dir="ltr"><<a href="mailto:rafaelfz@taghos.com.br" target="_blank">rafaelfz@taghos.com.br</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Em Fri, 4 Sep 2015 13:42:40 +0100<br>
Federico Schwindt <<a href="mailto:fgsch@lodoss.net">fgsch@lodoss.net</a>> escreveu:<br>
<span class=""><br>
> Diff aside looking at the code my impression is that the VTCP_filter_http()<br>
> function is meant to be compiled in always so erroring out if it's not<br>
> supported might be wrong here, or at least not when errno is EOPNOTSUPP/<br>
> multiple times.<br>
<br>
</span>People without it must have a way to find it out. The old way was not<br>
including any code and avoid it at all, but then we had a code that behaves<br>
differently according to the system which seems better than not giving<br>
any clues.<br></blockquote><div><br></div><div>The older code will always include VTCP_filter_http() for Linux, it was not calling it though.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> On 4 Sep 2015 1:35 pm, "Federico Schwindt" <<a href="mailto:fgsch@lodoss.net">fgsch@lodoss.net</a>> wrote:<br>
><br>
> > Did you see my second diff? :)<br>
</span>> --- SNIPPED ---<br>
<br>
Yes, I read your second diff.<br>
<br>
I agree with you that it would be better to kill the accept_filter<br>
param ifdef guards to keep it avaliable in every system, but I don't<br>
think it's a good idea to break the old behavior because people might<br>
be expecting it. That's why I suggested the TCP_DEFER_ACCEPT detection<br>
to make the code more portable instead of just looking for __linux.<br></blockquote><div><br></div>My second diff doesn't change any behaviour so nothing will break unless you enable it.<br></div><br></div></div>