[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id:
<173171223475.2762542.13729903505753277301.git-patchwork-notify@kernel.org>
Date: Fri, 15 Nov 2024 23:10:34 +0000
From: patchwork-bot+netdevbpf@...nel.org
To: Felix Maurer <fmaurer@...hat.com>
Cc: bpf@...r.kernel.org, bjorn@...nel.org, magnus.karlsson@...el.com,
maciej.fijalkowski@...el.com, jonathan.lemon@...il.com, davem@...emloft.net,
edumazet@...gle.com, kuba@...nel.org, pabeni@...hat.com, horms@...nel.org,
ast@...nel.org, daniel@...earbox.net, hawk@...nel.org,
john.fastabend@...il.com, yoong.siang.song@...el.com, sdf@...ichev.me,
netdev@...r.kernel.org, mschmidt@...hat.com
Subject: Re: [PATCH bpf] xsk: Free skb when TX metadata options are invalid
Hello:
This patch was applied to netdev/net.git (main)
by Jakub Kicinski <kuba@...nel.org>:
On Thu, 14 Nov 2024 12:30:05 +0100 you wrote:
> When a new skb is allocated for transmitting an xsk descriptor, i.e., for
> every non-multibuf descriptor or the first frag of a multibuf descriptor,
> but the descriptor is later found to have invalid options set for the TX
> metadata, the new skb is never freed. This can leak skbs until the send
> buffer is full which makes sending more packets impossible.
>
> Fix this by freeing the skb in the error path if we are currently dealing
> with the first frag, i.e., an skb allocated in this iteration of
> xsk_build_skb.
>
> [...]
Here is the summary with links:
- [bpf] xsk: Free skb when TX metadata options are invalid
https://git.kernel.org/netdev/net/c/0c0d0f42ffa6
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
Powered by blists - more mailing lists