diff options
author | Christoph Paasch <cpaasch@apple.com> | 2024-08-15 18:12:01 +0200 |
---|---|---|
committer | Jakub Kicinski <kuba@kernel.org> | 2024-08-17 02:53:49 +0200 |
commit | f4ae8420f6ebdabf97e0c4f5f99412768687985f (patch) | |
tree | 83c016cf129805a11a6e0024fd99f631604fed5c /include/media/v4l2-mem2mem.h | |
parent | net: txgbe: Remove unnecessary NULL check before free (diff) | |
download | linux-f4ae8420f6ebdabf97e0c4f5f99412768687985f.tar.xz linux-f4ae8420f6ebdabf97e0c4f5f99412768687985f.zip |
mpls: Reduce skb re-allocations due to skb_cow()
mpls_xmit() needs to prepend the MPLS-labels to the packet. That implies
one needs to make sure there is enough space for it in the headers.
Calling skb_cow() implies however that one wants to change even the
playload part of the packet (which is not true for MPLS). Thus, call
skb_cow_head() instead, which is what other tunnelling protocols do.
Running a server with this comm it entirely removed the calls to
pskb_expand_head() from the callstack in mpls_xmit() thus having
significant CPU-reduction, especially at peak times.
Cc: Roopa Prabhu <roopa@nvidia.com>
Reported-by: Craig Taylor <cmtaylor@apple.com>
Signed-off-by: Christoph Paasch <cpaasch@apple.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Link: https://patch.msgid.link/20240815161201.22021-1-cpaasch@apple.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'include/media/v4l2-mem2mem.h')
0 files changed, 0 insertions, 0 deletions