iovisor/bcc

Incorrect calculation of recv throughput in tcptop.py

Open

#5.352 aberto em 4 de jul. de 2025

Ver no GitHub
 (11 comments) (6 reactions) (0 assignees)C (4.051 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (22.409 stars)
Métricas de merge de PR
 (Mesclagem média 12d 6h) (12 fundiu PRs em 30d)

Description

tcp_cleanup_rbuf is called several times in a loop inside tcp_recvmsg. At the same time, it is passed the accumulated parameter "copied". Thus, the same data is taken into account several times.

int tcp_recvmsg(...) {
     ...
     int copied = 0;
     ...
     do {
         ...
         tcp_cleanup_rbuf(sk, copied);
         ...
         copied += used;
         ...
     } while (len > 0);
     
     tcp_cleanup_rbuf(sk, copied);
     return copied;
     ...
}

Guia do colaborador

Direção de pesquisa
Examine a função do kernel tcp recvmsg para entender como o parâmetro 'copied' é passado várias vezes para tcp cleanup rbuf, causando contagem dupla. Em seguida, modifique a sonda BPF no tcptop.py para usar o valor final de 'copied' ou ajustar a contagem para evitar supercontagem.
Pilha de tecnologia
cpython
Domain
networkingperformance
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
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
CBPFBCC
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70