
<html><head></head><body>Hi Toke, <br>

<br>

I certainly am thankful for your work, and believe you deserve $CAKE and $BEVERAGE, I am also leaf to believe 'the cake is a lie' <a href="https://m.youtube.com/watch?v=qdrs3gr_GAs">https://m.youtube.com/watch?v=qdrs3gr_GAs</a> ;)<br>

<br>

<br>

<br>

Mange Tak<br>

<br>

Sebastian<br><br><div class="gmail_quote">On April 29, 2018 12:45:26 PM GMT+02:00, "Toke Høiland-Jørgensen" <toke@toke.dk> wrote:<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<pre class="k9mail"><br><br>On 29 April 2018 03:01:55 CEST, Ryan Mounce <ryan@mounce.com.au> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">On 29 April 2018 at 07:19, Toke Høiland-Jørgensen <toke@toke.dk> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;"> Think I finally managed to fix the ACK filter so it works for both<br></blockquote>6in4<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;"> and regular v6 (the latter seems to have been broken since the commit<br> "e6b72c2 ack_filter: make less aggressive by default" which<br></blockquote>accidentally<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;"> reversed the address compare logic). It should also be safe against<br> weirdly fragmented packets (ha! famous last words!).<br><br> If someone wants to either eyeball the code and/or test the ACK<br></blockquote>filter<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;"> please do. I'm planning to resubmit the patch to netdev sometime<br> tomorrow evening :)<br></blockquote><br>Eyeballing now, only a couple of comments.<br><br>- Initialisation of iph and ipv6h can be moved before the while()<br>loop. The compiler probably does this anyway.<br></blockquote><br>Yeah, figured I'd let the compiler deal with that...<br><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">- Whether it is worth the complexity of restoring the<br>skb->encapsulation check within cake_get_tcphdr to handle other cases<br>of encapsulated v4/v6 traffic. I may find some time to test this later<br>today.<br></blockquote><br>The reason I removed the check for skb->encapsulation was that in my tests it wasn't actually set for 6in4 traffic.<br><br>And looking at the code that touches it, that field seems to mostly be related to offloads? Or did you actually see any effect of using it when you added it initially? :P<br><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">Everything else looks good to me, much cleaner.<br></blockquote><br>Awesome, thanks for taking a look!<br><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">Thank you so much for your efforts so far in getting cake upstream,<br>and in particular to rescue ACK filtering. We may have to send you a<br>real cake!<br></blockquote><br>Haha, you're very welcome. Real cake will be appreciated, but don't feel any obligations ;)<br><br>-Toke<br><hr><br>Cake mailing list<br>Cake@lists.bufferbloat.net<br><a href="https://lists.bufferbloat.net/listinfo/cake">https://lists.bufferbloat.net/listinfo/cake</a><br></pre></blockquote></div><br>

-- <br>

Sent from my Android device with K-9 Mail. Please excuse my brevity.</body></html>