Received: by 2002:a05:6a10:5bc5:0:0:0:0 with SMTP id os5csp3983700pxb; Tue, 2 Nov 2021 01:54:10 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyrRBuHDjbo+wEi2XijKGyzQAsk/Tx5Jw7SoNWHk5B2epRO1QMDajiU8CM0mlOtlXJJtesq X-Received: by 2002:a17:907:d8e:: with SMTP id go14mr44166641ejc.442.1635843249995; Tue, 02 Nov 2021 01:54:09 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1635843249; cv=none; d=google.com; s=arc-20160816; b=o9HggbrINrLTst/LWpeS/9gBsI8QJh/5yGerQrgFLjonrvkR6twd30OEMLA8W57Pf6 MeCLE76rLz8aHrTlAGf7VNwhHC8gtHDm+mnFBfwUY4tVVGa9dTNyNzwbotBSaFWJa/1r WP5XxBUlIKF0bmauNXQPw94VizQni0D7SRm4pGz5GMHPlWxKJcs/TW7nF3uWz/DD/HoR +3aAijgJf2o4kuqJ4xVMkb8Jsy1ZBApmkdSUAlMDGbenymi5fGtkBlIO1/jPOlK9k8dK 0EeRSZnV9v+QVB5acJhU3+JFZ6chYRUPBovZbVg6rfpYV/QEjoRp2D7t8+ZCWD25Fsof tsmQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=gJQUAFLd9VPS9XghTzRhticodd7nxr5AZRpuYBuPmD0=; b=RasctsYXMERvzvFsRV0jLBxxIN2z/H2Bf2/hxa8nu9m8SSY0SmijAz4l5szRqgQHgn UyfcWDQJRGOJc18ySoe+vAKCrZrZsJPcRUcly9/yu3uVropvjmy5Bm7CUbApOkTwhc9W FQ8ncKxLQa+A7O+8yL23Ff+wTfNcLfBpwucud1kcfIuOAl8QG48P9My5binFB7SYxXUG n0xziWzKGigJ7EAPLvlibiKhyA5OEBdglwZjNshtBdjysH5RKPkXmSasMPs8d6QVZMdU e3j6yOkl88HPrVCvnLT+JXQyy0aLc598kQbVd63TJM6Y/dA4Lk3Y8v9jpbjGxwwgbTv+ 3tSQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=huawei.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id z8si26800179edp.278.2021.11.02.01.53.43; Tue, 02 Nov 2021 01:54:09 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=huawei.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229881AbhKBIwG (ORCPT + 99 others); Tue, 2 Nov 2021 04:52:06 -0400 Received: from szxga01-in.huawei.com ([45.249.212.187]:14696 "EHLO szxga01-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229577AbhKBIwF (ORCPT ); Tue, 2 Nov 2021 04:52:05 -0400 Received: from dggemv704-chm.china.huawei.com (unknown [172.30.72.53]) by szxga01-in.huawei.com (SkyGuard) with ESMTP id 4Hk3SD3jYczZcJc; Tue, 2 Nov 2021 16:47:24 +0800 (CST) Received: from dggpeml500011.china.huawei.com (7.185.36.84) by dggemv704-chm.china.huawei.com (10.3.19.47) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.15; Tue, 2 Nov 2021 16:49:28 +0800 Received: from localhost.localdomain (10.175.101.6) by dggpeml500011.china.huawei.com (7.185.36.84) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.15; Tue, 2 Nov 2021 16:49:27 +0800 From: Di Zhu To: , , , , , , , , , CC: , , Subject: [PATCH bpf-next v4 2/2] selftests: bpf: test BPF_PROG_QUERY for progs attached to sockmap Date: Tue, 2 Nov 2021 16:48:56 +0800 Message-ID: <20211102084856.483534-2-zhudi2@huawei.com> X-Mailer: git-send-email 2.27.0 In-Reply-To: <20211102084856.483534-1-zhudi2@huawei.com> References: <20211102084856.483534-1-zhudi2@huawei.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7BIT Content-Type: text/plain; charset=US-ASCII X-Originating-IP: [10.175.101.6] X-ClientProxiedBy: dggems702-chm.china.huawei.com (10.3.19.179) To dggpeml500011.china.huawei.com (7.185.36.84) X-CFilter-Loop: Reflected Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Add test for querying progs attached to sockmap. we use an existing libbpf query interface to query prog cnt before and after progs attaching to sockmap and check whether the queried prog id is right. Signed-off-by: Di Zhu --- .../selftests/bpf/prog_tests/sockmap_basic.c | 84 +++++++++++++++++++ .../bpf/progs/test_sockmap_progs_query.c | 24 ++++++ 2 files changed, 108 insertions(+) create mode 100644 tools/testing/selftests/bpf/progs/test_sockmap_progs_query.c diff --git a/tools/testing/selftests/bpf/prog_tests/sockmap_basic.c b/tools/testing/selftests/bpf/prog_tests/sockmap_basic.c index 1352ec104149..7f3d5c5da6e1 100644 --- a/tools/testing/selftests/bpf/prog_tests/sockmap_basic.c +++ b/tools/testing/selftests/bpf/prog_tests/sockmap_basic.c @@ -8,6 +8,7 @@ #include "test_sockmap_update.skel.h" #include "test_sockmap_invalid_update.skel.h" #include "test_sockmap_skb_verdict_attach.skel.h" +#include "test_sockmap_progs_query.skel.h" #include "bpf_iter_sockmap.skel.h" #define TCP_REPAIR 19 /* TCP sock is under repair right now */ @@ -315,6 +316,83 @@ static void test_sockmap_skb_verdict_attach(enum bpf_attach_type first, test_sockmap_skb_verdict_attach__destroy(skel); } +static __u32 query_prog_id(int prog) +{ + struct bpf_prog_info info = {}; + __u32 info_len = sizeof(info); + int err; + + err = bpf_obj_get_info_by_fd(prog, &info, &info_len); + if (CHECK_FAIL(err || info_len != sizeof(info))) { + perror("bpf_obj_get_info_by_fd"); + return 0; + } + + return info.id; +} + +static void test_sockmap_progs_query(enum bpf_attach_type attach_type) +{ + struct test_sockmap_progs_query *skel; + int err, map, verdict, duration = 0; + __u32 attach_flags = 0; + __u32 prog_ids[3] = {}; + __u32 prog_cnt = 3; + + skel = test_sockmap_progs_query__open_and_load(); + if (CHECK_FAIL(!skel)) { + perror("test_sockmap_progs_query__open_and_load"); + return; + } + + map = bpf_map__fd(skel->maps.sock_map); + + if (attach_type == BPF_SK_MSG_VERDICT) + verdict = bpf_program__fd(skel->progs.prog_skmsg_verdict); + else + verdict = bpf_program__fd(skel->progs.prog_skb_verdict); + + err = bpf_prog_query(map, attach_type, 0 /* query flags */, + &attach_flags, prog_ids, &prog_cnt); + if (CHECK(err, "bpf_prog_query", "failed\n")) + goto out; + + if (CHECK(attach_flags != 0, "bpf_prog_query", + "wrong attach_flags on query: %u", attach_flags)) + goto out; + + if (CHECK(prog_cnt != 0, "bpf_prog_query", + "wrong program count on query: %u", prog_cnt)) + goto out; + + err = bpf_prog_attach(verdict, map, attach_type, 0); + if (CHECK(err, "bpf_prog_attach", "failed\n")) + goto out; + + prog_cnt = 1; + err = bpf_prog_query(map, attach_type, 0 /* query flags */, + &attach_flags, prog_ids, &prog_cnt); + if (CHECK(err, "bpf_prog_query", "failed\n")) + goto detach; + + if (CHECK(attach_flags != 0, "bpf_prog_query attach_flags", + "wrong attach_flags on query: %u", attach_flags)) + goto detach; + + if (CHECK(prog_cnt != 1, "bpf_prog_query prog_cnt", + "wrong program count on query: %u", prog_cnt)) + goto detach; + + if (CHECK(prog_ids[0] != query_prog_id(verdict), "bpf_prog_query", + "wrong prog_ids on query: %u", prog_ids[0])) + goto detach; + +detach: + bpf_prog_detach2(verdict, map, attach_type); +out: + test_sockmap_progs_query__destroy(skel); +} + void test_sockmap_basic(void) { if (test__start_subtest("sockmap create_update_free")) @@ -341,4 +419,10 @@ void test_sockmap_basic(void) test_sockmap_skb_verdict_attach(BPF_SK_SKB_STREAM_VERDICT, BPF_SK_SKB_VERDICT); } + if (test__start_subtest("sockmap progs query")) { + test_sockmap_progs_query(BPF_SK_MSG_VERDICT); + test_sockmap_progs_query(BPF_SK_SKB_STREAM_PARSER); + test_sockmap_progs_query(BPF_SK_SKB_STREAM_VERDICT); + test_sockmap_progs_query(BPF_SK_SKB_VERDICT); + } } diff --git a/tools/testing/selftests/bpf/progs/test_sockmap_progs_query.c b/tools/testing/selftests/bpf/progs/test_sockmap_progs_query.c new file mode 100644 index 000000000000..9d58d61c0dee --- /dev/null +++ b/tools/testing/selftests/bpf/progs/test_sockmap_progs_query.c @@ -0,0 +1,24 @@ +// SPDX-License-Identifier: GPL-2.0 +#include "vmlinux.h" +#include + +struct { + __uint(type, BPF_MAP_TYPE_SOCKMAP); + __uint(max_entries, 1); + __type(key, __u32); + __type(value, __u64); +} sock_map SEC(".maps"); + +SEC("sk_skb") +int prog_skb_verdict(struct __sk_buff *skb) +{ + return SK_PASS; +} + +SEC("sk_msg") +int prog_skmsg_verdict(struct sk_msg_md *msg) +{ + return SK_PASS; +} + +char _license[] SEC("license") = "GPL"; -- 2.27.0