polipo icon indicating copy to clipboard operation
polipo copied to clipboard
verified publisher icon jech

TCP Fast Open

Open pysiak opened this issue 10 years ago • 6 comments

Would you consider adding TCP Fast Open support? There's a small and simple example patch here: https://github.com/dtaht/ceropackages-3.10/blob/cf6fd6a01fdfbddb468f2a3ea27b6450955b674a/net/polipo/patches/001-server_tfo.patch

pysiak avatar May 31 '15 15:05 pysiak

Can you provide tcpdump traces that show that this patch actually does something?

jech avatar May 31 '15 16:05 jech

I did this patch for cerowrt (bufferbloat.net) originally. At the time, the kernel was not fully baked to include TCP Fast Open support. I assume it is now. If adding TCP Fast Open is welcome in polipo, I can try provide the complete patch set along with any supporting logs, pcaps.

ketkulka avatar Feb 17 '16 22:02 ketkulka

If adding TCP Fast Open is welcome in polipo, I can try provide the complete patch set along with any supporting logs, pcaps.

Please. Just a well-chosen pcap will be fine. Latency benchmarks welcome, of course, we all love benchmarks.

jech avatar Feb 17 '16 23:02 jech

@ketkulka Any progress on this?

simontretter avatar Sep 16 '16 04:09 simontretter

@simontretter see this commit - ab52932c20638491f696e256020ad6a20aad7939. I'm afraid it's time to close this and any other issues as WONTFIX.

lemenkov avatar Sep 18 '16 15:09 lemenkov

Ketan, it seems we didn't make it with the FASTOPEN patch soon enough :-) I was really going to produce tcpdumps for it as it''s trivial... Anyway, thanks @jech for polipo. I've been chaining it with privoxy on my home router.

pysiak avatar Sep 18 '16 15:09 pysiak