bpf, sockmap: fix forward allocation accounting in strparser self-pass path - #8576
Open
kernel-patches-daemon-bpf-rc[bot] wants to merge 2 commits into
Open
bpf, sockmap: fix forward allocation accounting in strparser self-pass path#8576kernel-patches-daemon-bpf-rc[bot] wants to merge 2 commits into
kernel-patches-daemon-bpf-rc[bot] wants to merge 2 commits into
Conversation
Author
|
Upstream branch: 80f3c3e |
kernel-patches-daemon-bpf-rc
Bot
force-pushed
the
bpf_base
branch
from
August 1, 2026 16:34
ec7c37b to
0391cfe
Compare
Author
|
Upstream branch: a766247 |
kernel-patches-daemon-bpf-rc
Bot
force-pushed
the
series/1138595=>bpf
branch
from
August 1, 2026 16:36
4b39dfc to
78cf3aa
Compare
kernel-patches-daemon-bpf-rc
Bot
force-pushed
the
bpf_base
branch
from
August 1, 2026 19:03
0391cfe to
0be2140
Compare
Author
|
Upstream branch: fdeba03 |
kernel-patches-daemon-bpf-rc
Bot
force-pushed
the
series/1138595=>bpf
branch
from
August 1, 2026 19:04
78cf3aa to
437db0d
Compare
kernel-patches-daemon-bpf-rc
Bot
force-pushed
the
bpf_base
branch
from
August 2, 2026 23:52
0be2140 to
c953291
Compare
Author
|
Upstream branch: e5fd3f5 |
kernel-patches-daemon-bpf-rc
Bot
force-pushed
the
series/1138595=>bpf
branch
from
August 2, 2026 23:53
437db0d to
889adb2
Compare
kernel-patches-daemon-bpf-rc
Bot
force-pushed
the
bpf_base
branch
from
August 4, 2026 07:32
c953291 to
47219af
Compare
Author
|
Upstream branch: 7cd8645 |
kernel-patches-daemon-bpf-rc
Bot
force-pushed
the
series/1138595=>bpf
branch
from
August 4, 2026 07:34
889adb2 to
bbe0794
Compare
Author
|
Upstream branch: 31a420a |
kernel-patches-daemon-bpf-rc
Bot
force-pushed
the
series/1138595=>bpf
branch
from
August 5, 2026 20:40
bbe0794 to
8f8bdf7
Compare
kernel-patches-daemon-bpf-rc
Bot
force-pushed
the
bpf_base
branch
from
August 6, 2026 00:16
6d62329 to
ab5db42
Compare
Author
|
Upstream branch: 31a420a |
kernel-patches-daemon-bpf-rc
Bot
force-pushed
the
series/1138595=>bpf
branch
from
August 6, 2026 00:21
8f8bdf7 to
b51cff4
Compare
kernel-patches-daemon-bpf-rc
Bot
force-pushed
the
bpf_base
branch
from
August 6, 2026 23:41
ab5db42 to
bef3f93
Compare
Author
|
Upstream branch: 7a3c028 |
kernel-patches-daemon-bpf-rc
Bot
force-pushed
the
series/1138595=>bpf
branch
from
August 6, 2026 23:43
b51cff4 to
ed96fb4
Compare
kernel-patches-daemon-bpf-rc
Bot
force-pushed
the
bpf_base
branch
from
August 7, 2026 15:35
bef3f93 to
0e8ddea
Compare
The strparser SK_PASS path can queue cloned skbs back to the same socket. A single TCP receive skb may be split into multiple strparser messages, and each cloned message still carries the receive owner from the TCP receive path. sk_psock_skb_ingress_self() reassigns receive ownership with skb_set_owner_r(). That first orphans the skb, which runs the existing receive destructor, and then charges the skb to the socket again. When this is repeated for strparser clones, sk_forward_alloc can already be in deficit before the next owner transition. Releasing the queued skbs can then uncharge more memcg pages than were reserved and trigger a page_counter underflow. Call sk_rmem_schedule() with a size of zero before skb_set_owner_r() for strparser self-pass skbs. Use the zero-sized reservation to top up any existing sk_forward_alloc deficit without reserving the skb's full truesize again, then let skb_set_owner_r() perform the receive-owner transition. Apply the same handling when retrying the skb from the psock backlog. Fixes: 144748e ("bpf, sockmap: Fix incorrect fwd_alloc accounting") Reported-by: Sechang Lim <rhkrqnwk98@gmail.com> Suggested-by: Emil Tsalapatis <emil@etsalapatis.com> Signed-off-by: Junseo Lim <zirajs7@gmail.com>
Add a sockmap_strp regression test for strparser SK_PASS delivery to the same socket. A one-byte stream parser splits a single write into many messages, repeatedly exercising receive ownership transitions while the skbs remain queued. Verify through INET_DIAG_MEMINFO that sk_forward_alloc does not become negative after the self-pass path is exercised. Signed-off-by: Junseo Lim <zirajs7@gmail.com>
Author
|
Upstream branch: a13307e |
kernel-patches-daemon-bpf-rc
Bot
force-pushed
the
series/1138595=>bpf
branch
from
August 7, 2026 15:37
ed96fb4 to
90b65b2
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull request for series with
subject: bpf, sockmap: fix forward allocation accounting in strparser self-pass path
version: 2
url: https://patchwork.kernel.org/project/netdevbpf/list/?series=1138595