envoyproxy/envoy

quic: measure GRO performance and enable it via config

Open

#19.119 geöffnet am 29. Nov. 2021

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/perfarea/quicarea/udphelp wanted

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

UDP GRO is supposed to improve socket read performance on client side but currently our code performs very poorly in nighthawk client, by reading one packet per read while allocating 24k buffer. It will be worthwhile to invest the suitable traffic patten for GRO. Meanwhile this feature is disabled in https://github.com/envoyproxy/envoy/pull/19088 and it will be good to exposed it via a config knob after we know the suitable use case.

And there was a bug reported that the socket reading code allocating 24k buffer per read might lead to MSG_TRUNC and cause packets dropped occasionally. We should change the value to be 64k which is the max GRO receive buffer allowed by the kernel.

Contributor Guide

Research-Richtung
Beginnen Sie mit dem Lesen des referenzierten PR #19088, um zu verstehen, warum GRO deaktiviert wurde. Untersuchen Sie dann den aktuellen Socket Lesecode in Envoy (wahrscheinlich in source/common/network/), um zu identifizieren, wo 24k Puffer zugewiesen werden. Recherchieren Sie das Kernel UDP GRO Verhalten und die maximale Empfangspuffergröße (64k). Schlagen Sie einen Konfigurationsknopf vor, um GRO zu aktivieren, und aktualisieren Sie die Puffergröße auf 64k, um MSG TRUNC zu vermeiden. Testen Sie mit dem Nighthawk Client für geeignete Verkehrsmuster.
Tech Stack
cpp
Domain
backend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++Linux networkingEnvoy development setup
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65