envoyproxy/envoy

quic: measure GRO performance and enable it via config

Open

#19.119 aberto em 29 de nov. de 2021

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
area/perfarea/quicarea/udphelp wanted

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Comece lendo o PR #19088 referenciado para entender por que o GRO foi desativado. Em seguida, examine o código atual de leitura de socket no Envoy (provavelmente em source/common/network/) para identificar onde os buffers de 24k são alocados. Pesquise o comportamento do GRO UDP do kernel e o tamanho máximo do buffer de recepção (64k). Proponha um botão de configuração para habilitar o GRO e atualize o tamanho do buffer para 64k para evitar MSG TRUNC. Teste com o cliente nighthawk para padrões de tráfego adequados.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Linux networkingEnvoy development setup
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65