[bpf-next,v5,01/11] skmsg: lock ingress_skb when purging

Message ID	20210317022219.24934-2-xiyou.wangcong@gmail.com
State	Superseded
Headers	show Return-Path: <netdev-owner@kernel.org> From: Cong Wang <xiyou.wangcong@gmail.com> To: netdev@vger.kernel.org Cc: bpf@vger.kernel.org, duanxiongchun@bytedance.com, wangdongdong.6@bytedance.com, jiang.wang@bytedance.com, Cong Wang <cong.wang@bytedance.com>, John Fastabend <john.fastabend@gmail.com>, Daniel Borkmann <daniel@iogearbox.net>, Lorenz Bauer <lmb@cloudflare.com>, Jakub Sitnicki <jakub@cloudflare.com> Subject: [Patch bpf-next v5 01/11] skmsg: lock ingress_skb when purging Date: Tue, 16 Mar 2021 19:22:09 -0700 Message-Id: <20210317022219.24934-2-xiyou.wangcong@gmail.com> In-Reply-To: <20210317022219.24934-1-xiyou.wangcong@gmail.com> References: <20210317022219.24934-1-xiyou.wangcong@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk
Series	sockmap: introduce BPF_SK_SKB_VERDICT and support UDP \| expand [bpf-next,v5,00/11] sockmap: introduce BPF_SK_SKB_VERDICT and support UDP [bpf-next,v5,01/11] skmsg: lock ingress_skb when purging [bpf-next,v5,02/11] skmsg: introduce a spinlock to protect ingress_msg [bpf-next,v5,03/11] skmsg: introduce skb_send_sock() for sock_map [bpf-next,v5,04/11] skmsg: avoid lock_sock() in sk_psock_backlog() [bpf-next,v5,05/11] sock_map: introduce BPF_SK_SKB_VERDICT [bpf-next,v5,06/11] sock: introduce sk->sk_prot->psock_update_sk_prot() [bpf-next,v5,07/11] udp: implement ->read_sock() for sockmap [bpf-next,v5,08/11] skmsg: extract __tcp_bpf_recvmsg() and tcp_bpf_wait_data() [bpf-next,v5,09/11] udp: implement udp_bpf_recvmsg() for sockmap [bpf-next,v5,10/11] sock_map: update sock type checks for UDP [bpf-next,v5,11/11] selftests/bpf: add a test case for udp sockmap

Message ID

20210317022219.24934-2-xiyou.wangcong@gmail.com

State

Superseded

Headers

From: Cong Wang <xiyou.wangcong@gmail.com>
To: netdev@vger.kernel.org
Cc: bpf@vger.kernel.org, duanxiongchun@bytedance.com,
	wangdongdong.6@bytedance.com, jiang.wang@bytedance.com,
	Cong Wang <cong.wang@bytedance.com>,
	John Fastabend <john.fastabend@gmail.com>,
	Daniel Borkmann <daniel@iogearbox.net>,
	Lorenz Bauer <lmb@cloudflare.com>, Jakub Sitnicki <jakub@cloudflare.com>
Subject: [Patch bpf-next v5 01/11] skmsg: lock ingress_skb when purging
Date: Tue, 16 Mar 2021 19:22:09 -0700
Message-Id: <20210317022219.24934-2-xiyou.wangcong@gmail.com>
In-Reply-To: <20210317022219.24934-1-xiyou.wangcong@gmail.com>
References: <20210317022219.24934-1-xiyou.wangcong@gmail.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Precedence: bulk

Series

sockmap: introduce BPF_SK_SKB_VERDICT and support UDP | expand

Commit Message

Cong Wang March 17, 2021, 2:22 a.m. UTC

From: Cong Wang <cong.wang@bytedance.com>

Currently we purge the ingress_skb queue only when psock
refcnt goes down to 0, so locking the queue is not necessary,
but in order to be called during ->close, we have to lock it
here.

Cc: John Fastabend <john.fastabend@gmail.com>
Cc: Daniel Borkmann <daniel@iogearbox.net>
Cc: Lorenz Bauer <lmb@cloudflare.com>
Acked-by: Jakub Sitnicki <jakub@cloudflare.com>
Signed-off-by: Cong Wang <cong.wang@bytedance.com>
---
 net/core/skmsg.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/core/skmsg.c b/net/core/skmsg.c
index 07f54015238a..bebf84ed4e30 100644
--- a/net/core/skmsg.c
+++ b/net/core/skmsg.c
@@ -634,7 +634,7 @@  static void sk_psock_zap_ingress(struct sk_psock *psock)
 {
 	struct sk_buff *skb;
 
-	while ((skb = __skb_dequeue(&psock->ingress_skb)) != NULL) {
+	while ((skb = skb_dequeue(&psock->ingress_skb)) != NULL) {
 		skb_bpf_redirect_clear(skb);
 		kfree_skb(skb);
 	}