]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/commit - net/ipv4/udp.c
udp: under rx pressure, try to condense skbs
authorEric Dumazet <edumazet@google.com>
Wed, 7 Dec 2016 17:19:33 +0000 (09:19 -0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 8 Dec 2016 18:25:07 +0000 (13:25 -0500)
commitc8c8b127091b758f5768f906bcdeeb88bc9951ca
tree6721fe5d6de0ca0ddd61b4356539d1621538c2bd
parent2408022eeada9b0d96cb6a40bccf0ec2aa280bab
udp: under rx pressure, try to condense skbs

Under UDP flood, many softirq producers try to add packets to
UDP receive queue, and one user thread is burning one cpu trying
to dequeue packets as fast as possible.

Two parts of the per packet cost are :
- copying payload from kernel space to user space,
- freeing memory pieces associated with skb.

If socket is under pressure, softirq handler(s) can try to pull in
skb->head the payload of the packet if it fits.

Meaning the softirq handler(s) can free/reuse the page fragment
immediately, instead of letting udp_recvmsg() do this hundreds of usec
later, possibly from another node.

Additional gains :
- We reduce skb->truesize and thus can store more packets per SO_RCVBUF
- We avoid cache line misses at copyout() time and consume_skb() time,
and avoid one put_page() with potential alien freeing on NUMA hosts.

This comes at the cost of a copy, bounded to available tail room, which
is usually small. (We might have to fix GRO_MAX_HEAD which looks bigger
than necessary)

This patch gave me about 5 % increase in throughput in my tests.

skb_condense() helper could probably used in other contexts.

Signed-off-by: Eric Dumazet <edumazet@google.com>
Cc: Paolo Abeni <pabeni@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/skbuff.h
net/core/skbuff.c
net/ipv4/udp.c