Re: [PATCH net] phonet/pep: disable BH around forwarded sk_receive_skb()

[patchwork-bot+netdevbpf]

Hello:

This patch was applied to netdev/net.git (main)
by Jakub Kicinski <kuba@xxxxxxxxxx>:

On Tue, 19 May 2026 10:26:33 -0700 you wrote:
> The networking receive path is usually run from softirq context, but
> protocols that take the socket lock may have packets stored in the
> backlog and processed later from process context. In that case
> release_sock() -> __release_sock() drops the slock with spin_unlock_bh()
> and then calls sk->sk_backlog_rcv() with bottom halves enabled.
> 
> Typical sk_backlog_rcv handlers process the socket whose backlog is
> being drained, so the BH state at entry is irrelevant for the slocks
> they touch. pep_do_rcv() is different: when the inbound skb targets an
> existing PEP pipe, it forwards the skb to a different *child* socket
> via sk_receive_skb(). That helper takes the child slock with
> bh_lock_sock_nested(), which is just spin_lock_nested() and assumes BH
> is already off. The same child slock therefore ends up acquired with
> BH on (process path) and with BH off (softirq path):
> 
> [...]

Here is the summary with links:
  - [net] phonet/pep: disable BH around forwarded sk_receive_skb()
    https://git.kernel.org/netdev/net/c/dbc81608e3a6

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html