Oskar Mansfeld

Results 37 comments of Oskar Mansfeld

> Minor comments. > > Once you have been able to integrate this in Firefox and we see it working, I suggest we merge here. Sounds good? Sounds good, I...

Added a TODO for potentially tracking spurious congestion events per congestion epoch. I have this integrated in firefox now, but not sure how to best upload the patch because it...

#3111 got me thinking: Should we include ECN congestion events here? They cannot be spurious, so I think it is correct to only include congestion events due to loss in...

I implemented the above in the latest commit. We now have the stats `congestion_events_due_to_ecn` and `congestion_events_due_to_loss` and can then decide in `neqo_glue` on the Firefox side how exactly we want...

> Exciting to see this happening in a real network, i.e. your home network. > > Did you already start the integration into Firefox? I am curious whether you see...

Thanks for the input and shared slide deck, this is very interesting. I will also have to give this more thought and take some time to go over the material.

I have opened #3162 for further discussion on the point of loss based slow start exit having multiple losses with some initial experimentation data on tuning the reduction factor in...

quiche implements it: https://github.com/cloudflare/quiche/blob/master/quiche/src/recovery/congestion/cubic.rs#L207C1-L228C6 so does picoquic: https://huitema.wordpress.com/2019/11/11/implementing-cubic-congestion-control-in-quic/#spurious and see original discussion on including it in the draft: https://ntap.github.io/rfc8312bis/issues.html (cannot properly link, search for "spurious" on that page) ^ this...

Alright, then I'll tackle this after (or during if it makes more sense, but as a separate PR ideally) #2535. And yes, kind of a shame that the repos don't...

> May I suggest, before doing the actual implementation in `neqo-transport/src/cc/` to land a metric counting the number of spurious re-transmits we see in the wild? We can then record...