selftests/bpf: Migrate all deprecated perf_buffer uses

Migrate all old-style perf_buffer__new() and perf_buffer__new_raw()
calls to new v1.0+ variants.

Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Link: https://lore.kernel.org/bpf/20211111053624.190580-7-andrii@kernel.org

diff --git a/tools/testing/selftests/bpf/benchs/bench_ringbufs.c b/tools/testing/selftests/bpf/benchs/bench_ringbufs.c
index d167bffac679d0f59b2fa07b1402912d001c547e..52d4a2f91dbda987e5450302145786669abac749 100644 (file)
--- a/tools/testing/selftests/bpf/benchs/bench_ringbufs.c
+++ b/tools/testing/selftests/bpf/benchs/bench_ringbufs.c
@@ -394,11 +394,6 @@ static void perfbuf_libbpf_setup()
 {
        struct perfbuf_libbpf_ctx *ctx = &perfbuf_libbpf_ctx;
        struct perf_event_attr attr;
-       struct perf_buffer_raw_opts pb_opts = {
-               .event_cb = perfbuf_process_sample_raw,
-               .ctx = (void *)(long)0,
-               .attr = &attr,
-       };
        struct bpf_link *link;
 
        ctx->skel = perfbuf_setup_skeleton();
@@ -423,7 +418,8 @@ static void perfbuf_libbpf_setup()
        }
 
        ctx->perfbuf = perf_buffer__new_raw(bpf_map__fd(ctx->skel->maps.perfbuf),
-                                           args.perfbuf_sz, &pb_opts);
+                                           args.perfbuf_sz, &attr,
+                                           perfbuf_process_sample_raw, NULL, NULL);
        if (!ctx->perfbuf) {
                fprintf(stderr, "failed to create perfbuf\n");
                exit(1);

diff --git a/tools/testing/selftests/bpf/prog_tests/get_stack_raw_tp.c b/tools/testing/selftests/bpf/prog_tests/get_stack_raw_tp.c
index 569fcc6ed660995255ec2f06b741bfb2ec48323f..4184c399d4c603156a3582dbbab223ecc29c6b74 100644 (file)
--- a/tools/testing/selftests/bpf/prog_tests/get_stack_raw_tp.c
+++ b/tools/testing/selftests/bpf/prog_tests/get_stack_raw_tp.c
@@ -85,7 +85,6 @@ void test_get_stack_raw_tp(void)
        const char *file_err = "./test_get_stack_rawtp_err.o";
        const char *prog_name = "raw_tracepoint/sys_enter";
        int i, err, prog_fd, exp_cnt = MAX_CNT_RAWTP;
-       struct perf_buffer_opts pb_opts = {};
        struct perf_buffer *pb = NULL;
        struct bpf_link *link = NULL;
        struct timespec tv = {0, 10};
@@ -124,8 +123,8 @@ void test_get_stack_raw_tp(void)
        if (!ASSERT_OK_PTR(link, "attach_raw_tp"))
                goto close_prog;
 
-       pb_opts.sample_cb = get_stack_print_output;
-       pb = perf_buffer__new(bpf_map__fd(map), 8, &pb_opts);
+       pb = perf_buffer__new(bpf_map__fd(map), 8, get_stack_print_output,
+                             NULL, NULL, NULL);
        if (!ASSERT_OK_PTR(pb, "perf_buf__new"))
                goto close_prog;
 

diff --git a/tools/testing/selftests/bpf/prog_tests/kfree_skb.c b/tools/testing/selftests/bpf/prog_tests/kfree_skb.c
index 885413ed5c969465ff03e0ff83dc1adb351c3e73..2a49f8fcde06a088a07406e29a4a6621ce130cf7 100644 (file)
--- a/tools/testing/selftests/bpf/prog_tests/kfree_skb.c
+++ b/tools/testing/selftests/bpf/prog_tests/kfree_skb.c
@@ -66,7 +66,6 @@ void serial_test_kfree_skb(void)
        struct bpf_map *perf_buf_map, *global_data;
        struct bpf_program *prog, *fentry, *fexit;
        struct bpf_object *obj, *obj2 = NULL;
-       struct perf_buffer_opts pb_opts = {};
        struct perf_buffer *pb = NULL;
        int err, kfree_skb_fd;
        bool passed = false;
@@ -112,9 +111,8 @@ void serial_test_kfree_skb(void)
                goto close_prog;
 
        /* set up perf buffer */
-       pb_opts.sample_cb = on_sample;
-       pb_opts.ctx = &passed;
-       pb = perf_buffer__new(bpf_map__fd(perf_buf_map), 1, &pb_opts);
+       pb = perf_buffer__new(bpf_map__fd(perf_buf_map), 1,
+                             on_sample, NULL, &passed, NULL);
        if (!ASSERT_OK_PTR(pb, "perf_buf__new"))
                goto close_prog;
 

diff --git a/tools/testing/selftests/bpf/prog_tests/perf_buffer.c b/tools/testing/selftests/bpf/prog_tests/perf_buffer.c
index 4e32f3586a75c793cfe84b7269bca007c9e75385..5fc2b3a0711ee0c85643f48c1c49000739b6bef6 100644 (file)
--- a/tools/testing/selftests/bpf/prog_tests/perf_buffer.c
+++ b/tools/testing/selftests/bpf/prog_tests/perf_buffer.c
@@ -47,7 +47,6 @@ void serial_test_perf_buffer(void)
 {
        int err, on_len, nr_on_cpus = 0, nr_cpus, i, j;
        int zero = 0, my_pid = getpid();
-       struct perf_buffer_opts pb_opts = {};
        struct test_perf_buffer *skel;
        cpu_set_t cpu_seen;
        struct perf_buffer *pb;
@@ -82,9 +81,8 @@ void serial_test_perf_buffer(void)
                goto out_close;
 
        /* set up perf buffer */
-       pb_opts.sample_cb = on_sample;
-       pb_opts.ctx = &cpu_seen;
-       pb = perf_buffer__new(bpf_map__fd(skel->maps.perf_buf_map), 1, &pb_opts);
+       pb = perf_buffer__new(bpf_map__fd(skel->maps.perf_buf_map), 1,
+                             on_sample, NULL, &cpu_seen, NULL);
        if (!ASSERT_OK_PTR(pb, "perf_buf__new"))
                goto out_close;
 

diff --git a/tools/testing/selftests/bpf/prog_tests/xdp_bpf2bpf.c b/tools/testing/selftests/bpf/prog_tests/xdp_bpf2bpf.c
index 3bd5904b4db5ee3a7e2825965f5ff8eed9b5d91b..f99386d1dc4ceba8a607e338565bd5e6c67e645f 100644 (file)
--- a/tools/testing/selftests/bpf/prog_tests/xdp_bpf2bpf.c
+++ b/tools/testing/selftests/bpf/prog_tests/xdp_bpf2bpf.c
@@ -49,7 +49,6 @@ void test_xdp_bpf2bpf(void)
        struct vip key4 = {.protocol = 6, .family = AF_INET};
        struct bpf_program *prog;
        struct perf_buffer *pb = NULL;
-       struct perf_buffer_opts pb_opts = {};
 
        /* Load XDP program to introspect */
        pkt_skel = test_xdp__open_and_load();
@@ -86,10 +85,8 @@ void test_xdp_bpf2bpf(void)
                goto out;
 
        /* Set up perf buffer */
-       pb_opts.sample_cb = on_sample;
-       pb_opts.ctx = &passed;
-       pb = perf_buffer__new(bpf_map__fd(ftrace_skel->maps.perf_buf_map),
-                             1, &pb_opts);
+       pb = perf_buffer__new(bpf_map__fd(ftrace_skel->maps.perf_buf_map), 1,
+                             on_sample, NULL, &passed, NULL);
        if (!ASSERT_OK_PTR(pb, "perf_buf__new"))
                goto out;
 

diff --git a/tools/testing/selftests/bpf/test_tcpnotify_user.c b/tools/testing/selftests/bpf/test_tcpnotify_user.c
index 63111cb082feccf583255779725bfb63a9b20d02..4c5114765b23110514c83c60e9097efb726f1cc8 100644 (file)
--- a/tools/testing/selftests/bpf/test_tcpnotify_user.c
+++ b/tools/testing/selftests/bpf/test_tcpnotify_user.c
@@ -72,7 +72,6 @@ int main(int argc, char **argv)
 {
        const char *file = "test_tcpnotify_kern.o";
        struct bpf_map *perf_map, *global_map;
-       struct perf_buffer_opts pb_opts = {};
        struct tcpnotify_globals g = {0};
        struct perf_buffer *pb = NULL;
        const char *cg_path = "/foo";
@@ -117,8 +116,7 @@ int main(int argc, char **argv)
                return -1;
        }
 
-       pb_opts.sample_cb = dummyfn;
-       pb = perf_buffer__new(bpf_map__fd(perf_map), 8, &pb_opts);
+       pb = perf_buffer__new(bpf_map__fd(perf_map), 8, dummyfn, NULL, NULL, NULL);
        if (!pb)
                goto err;