On 04/02/2021 21:31, Eric Dumazet wrote:
> From: Eric Dumazet <eduma...@google.com>
>
> Commit c80794323e82 ("net: Fix packet reordering caused by GRO and
> listified RX cooperation") had the unfortunate effect of adding
> latencies in common workloads.
>
> Before the patch, GRO packets were immediately passed to
> upper stacks.
>
> After the patch, we can accumulate quite a lot of GRO
> packets (depdending on NAPI budget).
>
> My fix is counting in napi->rx_count number of segments
> instead of number of logical packets.
>
> Fixes: c80794323e82 ("net: Fix packet reordering caused by GRO and listified
> RX cooperation")
> Signed-off-by: Eric Dumazet <eduma...@google.com>
> Bisected-by: John Sperbeck <jsperb...@google.com>
> Tested-by: Jian Yang <jiany...@google.com>
> Cc: Maxim Mikityanskiy <maxi...@mellanox.com>
> Cc: Alexander Lobakin <aloba...@dlink.ru>
> Cc: Saeed Mahameed <sae...@mellanox.com>
> Cc: Edward Cree <ec...@solarflare.com>
Reviewed-by: Edward Cree <ecree.xil...@gmail.com>
