selftests/bpf: fix lots of silly mistakes pointed out by compiler
authorAndrii Nakryiko <andrii@kernel.org>
Thu, 9 Mar 2023 05:40:14 +0000 (21:40 -0800)
committerAlexei Starovoitov <ast@kernel.org>
Fri, 10 Mar 2023 16:14:08 +0000 (08:14 -0800)
Once we enable -Wall for BPF sources, compiler will complain about lots
of unused variables, variables that are set but never read, etc.

Fix all these issues first before enabling -Wall in Makefile.

Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Link: https://lore.kernel.org/r/20230309054015.4068562-4-andrii@kernel.org
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
81 files changed:
tools/testing/selftests/bpf/progs/bpf_iter_ksym.c
tools/testing/selftests/bpf/progs/bpf_iter_setsockopt.c
tools/testing/selftests/bpf/progs/bpf_loop.c
tools/testing/selftests/bpf/progs/cb_refs.c
tools/testing/selftests/bpf/progs/cgroup_skb_sk_lookup_kern.c
tools/testing/selftests/bpf/progs/cgrp_kfunc_failure.c
tools/testing/selftests/bpf/progs/cgrp_ls_attach_cgroup.c
tools/testing/selftests/bpf/progs/cgrp_ls_sleepable.c
tools/testing/selftests/bpf/progs/core_kern.c
tools/testing/selftests/bpf/progs/cpumask_failure.c
tools/testing/selftests/bpf/progs/cpumask_success.c
tools/testing/selftests/bpf/progs/dynptr_fail.c
tools/testing/selftests/bpf/progs/dynptr_success.c
tools/testing/selftests/bpf/progs/fexit_bpf2bpf.c
tools/testing/selftests/bpf/progs/freplace_attach_probe.c
tools/testing/selftests/bpf/progs/iters.c
tools/testing/selftests/bpf/progs/linked_funcs1.c
tools/testing/selftests/bpf/progs/linked_funcs2.c
tools/testing/selftests/bpf/progs/linked_list.c
tools/testing/selftests/bpf/progs/linked_list_fail.c
tools/testing/selftests/bpf/progs/local_storage.c
tools/testing/selftests/bpf/progs/map_kptr.c
tools/testing/selftests/bpf/progs/netcnt_prog.c
tools/testing/selftests/bpf/progs/netif_receive_skb.c
tools/testing/selftests/bpf/progs/perfbuf_bench.c
tools/testing/selftests/bpf/progs/pyperf.h
tools/testing/selftests/bpf/progs/rbtree_btf_fail__wrong_node_type.c
tools/testing/selftests/bpf/progs/rbtree_fail.c
tools/testing/selftests/bpf/progs/rcu_read_lock.c
tools/testing/selftests/bpf/progs/read_bpf_task_storage_busy.c
tools/testing/selftests/bpf/progs/recvmsg4_prog.c
tools/testing/selftests/bpf/progs/recvmsg6_prog.c
tools/testing/selftests/bpf/progs/sendmsg4_prog.c
tools/testing/selftests/bpf/progs/sockmap_verdict_prog.c
tools/testing/selftests/bpf/progs/strobemeta.h
tools/testing/selftests/bpf/progs/tailcall_bpf2bpf3.c
tools/testing/selftests/bpf/progs/tailcall_bpf2bpf6.c
tools/testing/selftests/bpf/progs/task_kfunc_failure.c
tools/testing/selftests/bpf/progs/task_kfunc_success.c
tools/testing/selftests/bpf/progs/test_bpf_nf.c
tools/testing/selftests/bpf/progs/test_cls_redirect_dynptr.c
tools/testing/selftests/bpf/progs/test_core_reloc_bitfields_probed.c
tools/testing/selftests/bpf/progs/test_global_func1.c
tools/testing/selftests/bpf/progs/test_global_func2.c
tools/testing/selftests/bpf/progs/test_hash_large_key.c
tools/testing/selftests/bpf/progs/test_ksyms_btf_write_check.c
tools/testing/selftests/bpf/progs/test_legacy_printk.c
tools/testing/selftests/bpf/progs/test_map_lock.c
tools/testing/selftests/bpf/progs/test_obj_id.c
tools/testing/selftests/bpf/progs/test_parse_tcp_hdr_opt.c
tools/testing/selftests/bpf/progs/test_parse_tcp_hdr_opt_dynptr.c
tools/testing/selftests/bpf/progs/test_pkt_access.c
tools/testing/selftests/bpf/progs/test_ringbuf.c
tools/testing/selftests/bpf/progs/test_ringbuf_map_key.c
tools/testing/selftests/bpf/progs/test_ringbuf_multi.c
tools/testing/selftests/bpf/progs/test_select_reuseport_kern.c
tools/testing/selftests/bpf/progs/test_sk_assign.c
tools/testing/selftests/bpf/progs/test_sk_lookup.c
tools/testing/selftests/bpf/progs/test_sk_lookup_kern.c
tools/testing/selftests/bpf/progs/test_sock_fields.c
tools/testing/selftests/bpf/progs/test_sockmap_kern.h
tools/testing/selftests/bpf/progs/test_spin_lock.c
tools/testing/selftests/bpf/progs/test_tc_dtime.c
tools/testing/selftests/bpf/progs/test_tc_neigh.c
tools/testing/selftests/bpf/progs/test_tcpbpf_kern.c
tools/testing/selftests/bpf/progs/test_tunnel_kern.c
tools/testing/selftests/bpf/progs/test_usdt_multispec.c
tools/testing/selftests/bpf/progs/test_verif_scale1.c
tools/testing/selftests/bpf/progs/test_verif_scale2.c
tools/testing/selftests/bpf/progs/test_verif_scale3.c
tools/testing/selftests/bpf/progs/test_xdp_adjust_tail_grow.c
tools/testing/selftests/bpf/progs/test_xdp_bpf2bpf.c
tools/testing/selftests/bpf/progs/test_xdp_dynptr.c
tools/testing/selftests/bpf/progs/test_xdp_noinline.c
tools/testing/selftests/bpf/progs/test_xdp_vlan.c
tools/testing/selftests/bpf/progs/type_cast.c
tools/testing/selftests/bpf/progs/udp_limit.c
tools/testing/selftests/bpf/progs/user_ringbuf_success.c
tools/testing/selftests/bpf/progs/xdp_features.c
tools/testing/selftests/bpf/progs/xdping_kern.c
tools/testing/selftests/bpf/progs/xdpwall.c

index 9ba14c37bbcc925da01da0c493b75860de31fef1..5ddcc46fd886800862e9b405e30d1ecbc559ad03 100644 (file)
@@ -33,7 +33,6 @@ int dump_ksym(struct bpf_iter__ksym *ctx)
        __u32 seq_num = ctx->meta->seq_num;
        unsigned long value;
        char type;
-       int ret;
 
        if (!iter)
                return 0;
index b77adfd55d7324203660fa7b76f2327ff8bf585f..ec7f91850dec961443a8cdd704eeffcde8b79aab 100644 (file)
@@ -42,7 +42,6 @@ int change_tcp_cc(struct bpf_iter__tcp *ctx)
        char cur_cc[TCP_CA_NAME_MAX];
        struct tcp_sock *tp;
        struct sock *sk;
-       int ret;
 
        if (!bpf_tcp_sk(ctx->sk_common))
                return 0;
index de1fc82d271059f0e9d1b0f13b5b76821df2b75a..1d194455b109cc31bb81415ec51d65c37e17e824 100644 (file)
@@ -138,8 +138,6 @@ static int callback_set_0f(int i, void *ctx)
 SEC("fentry/" SYS_PREFIX "sys_nanosleep")
 int prog_non_constant_callback(void *ctx)
 {
-       struct callback_ctx data = {};
-
        if (bpf_get_current_pid_tgid() >> 32 != pid)
                return 0;
 
index ce96b33e38d668567c49c2113880a107154cb19d..50f95ec611650f4c00a5d3b832b93e759bb5950e 100644 (file)
@@ -52,7 +52,6 @@ int leak_prog(void *ctx)
 {
        struct prog_test_ref_kfunc *p;
        struct map_value *v;
-       unsigned long sl;
 
        v = bpf_map_lookup_elem(&array_map, &(int){0});
        if (!v)
index 88638315c582c2f190c97ab46ebc3975531f6065..ac86a8a616057303b91e3e175f9a2593aab82d29 100644 (file)
@@ -66,7 +66,6 @@ static inline int is_allowed_peer_cg(struct __sk_buff *skb,
 SEC("cgroup_skb/ingress")
 int ingress_lookup(struct __sk_buff *skb)
 {
-       __u32 serv_port_key = 0;
        struct ipv6hdr ip6h;
        struct tcphdr tcph;
 
index b42291ed95869f5dee079f3c7b7be529b828de42..807fb0ac41e9c5d920484374a43d6c5018d579c6 100644 (file)
@@ -109,6 +109,7 @@ int BPF_PROG(cgrp_kfunc_acquire_unreleased, struct cgroup *cgrp, const char *pat
        acquired = bpf_cgroup_acquire(cgrp);
 
        /* Acquired cgroup is never released. */
+       __sink(acquired);
 
        return 0;
 }
index 6652d18465b242a4759b456b0f4bdfc9f2abfe92..8aeba1b75c838c3af7126dc376c08d8037a4d4b6 100644 (file)
@@ -84,7 +84,6 @@ int BPF_PROG(update_cookie_tracing, struct socket *sock,
             struct sockaddr *uaddr, int addr_len, int flags)
 {
        struct socket_cookie *p;
-       struct tcp_sock *tcp_sk;
 
        if (uaddr->sa_family != AF_INET6)
                return 0;
index 7615dc23d301ed55b24c27dd9ed827e2505e221f..4c7844e1dbfa5821e2f231f3b9929c84dfed2e9b 100644 (file)
@@ -24,7 +24,6 @@ void bpf_rcu_read_unlock(void) __ksym;
 SEC("?iter.s/cgroup")
 int cgroup_iter(struct bpf_iter__cgroup *ctx)
 {
-       struct seq_file *seq = ctx->meta->seq;
        struct cgroup *cgrp = ctx->cgroup;
        long *ptr;
 
index 2715fe27d4cf8b929829ac90612963c3d35e83ad..004f2acef2eb0b0d80184423a3e8ba399feee1f3 100644 (file)
@@ -77,7 +77,7 @@ int balancer_ingress(struct __sk_buff *ctx)
        void *data_end = (void *)(long)ctx->data_end;
        void *data = (void *)(long)ctx->data;
        void *ptr;
-       int ret = 0, nh_off, i = 0;
+       int nh_off, i = 0;
 
        nh_off = 14;
 
index c16f7563b84ec277a24a5359a036867cc5c81034..cfe83f0ef9e25d7a8c15be65e4d362fc65b25aa3 100644 (file)
@@ -23,6 +23,7 @@ int BPF_PROG(test_alloc_no_release, struct task_struct *task, u64 clone_flags)
        struct bpf_cpumask *cpumask;
 
        cpumask = create_cpumask();
+       __sink(cpumask);
 
        /* cpumask is never released. */
        return 0;
@@ -51,6 +52,7 @@ int BPF_PROG(test_acquire_wrong_cpumask, struct task_struct *task, u64 clone_fla
 
        /* Can't acquire a non-struct bpf_cpumask. */
        cpumask = bpf_cpumask_acquire((struct bpf_cpumask *)task->cpus_ptr);
+       __sink(cpumask);
 
        return 0;
 }
@@ -63,6 +65,7 @@ int BPF_PROG(test_mutate_cpumask, struct task_struct *task, u64 clone_flags)
 
        /* Can't set the CPU of a non-struct bpf_cpumask. */
        bpf_cpumask_set_cpu(0, (struct bpf_cpumask *)task->cpus_ptr);
+       __sink(cpumask);
 
        return 0;
 }
index 1d38bc65d4b0e658337c54383c2ed423c06fd2e7..97ed08c4ff03cf65a7e27185e761933024dd3747 100644 (file)
@@ -353,7 +353,6 @@ SEC("tp_btf/task_newtask")
 int BPF_PROG(test_insert_leave, struct task_struct *task, u64 clone_flags)
 {
        struct bpf_cpumask *cpumask;
-       struct __cpumask_map_value *v;
 
        cpumask = create_cpumask();
        if (!cpumask)
index 20ce920d891da5f42cbfcdd6def93d4a68fe8c6d..759eb5c245cd9a788cc0c307f532fdbfa42edf95 100644 (file)
@@ -271,7 +271,7 @@ SEC("?raw_tp")
 __failure __msg("value is outside of the allowed memory range")
 int data_slice_out_of_bounds_map_value(void *ctx)
 {
-       __u32 key = 0, map_val;
+       __u32 map_val;
        struct bpf_dynptr ptr;
        void *data;
 
@@ -388,7 +388,6 @@ int data_slice_missing_null_check2(void *ctx)
                /* this should fail */
                *data2 = 3;
 
-done:
        bpf_ringbuf_discard_dynptr(&ptr, 0);
        return 0;
 }
@@ -440,6 +439,7 @@ int invalid_write1(void *ctx)
 
        /* this should fail */
        data = bpf_dynptr_data(&ptr, 0, 1);
+       __sink(data);
 
        return 0;
 }
@@ -1374,6 +1374,7 @@ int invalid_slice_rdwr_rdonly(struct __sk_buff *skb)
         * changing packet data
         */
        hdr = bpf_dynptr_slice_rdwr(&ptr, 0, buffer, sizeof(buffer));
+       __sink(hdr);
 
        return 0;
 }
index c8358a7c792451c05667a009725856cbd056136f..b2fa6c47ecc0b0fe204027a5da9936e8c413caf5 100644 (file)
@@ -35,7 +35,7 @@ SEC("?tp/syscalls/sys_enter_nanosleep")
 int test_read_write(void *ctx)
 {
        char write_data[64] = "hello there, world!!";
-       char read_data[64] = {}, buf[64] = {};
+       char read_data[64] = {};
        struct bpf_dynptr ptr;
        int i;
 
@@ -170,7 +170,6 @@ int test_skb_readonly(struct __sk_buff *skb)
 {
        __u8 write_data[2] = {1, 2};
        struct bpf_dynptr ptr;
-       __u64 *data;
        int ret;
 
        if (bpf_dynptr_from_skb(skb, 0, &ptr)) {
@@ -191,10 +190,8 @@ int test_skb_readonly(struct __sk_buff *skb)
 SEC("?cgroup_skb/egress")
 int test_dynptr_skb_data(struct __sk_buff *skb)
 {
-       __u8 write_data[2] = {1, 2};
        struct bpf_dynptr ptr;
        __u64 *data;
-       int ret;
 
        if (bpf_dynptr_from_skb(skb, 0, &ptr)) {
                err = 1;
index 4547b059d487783205554d06ccfb5faff785f634..983b7c2333828a511810f3caffbd7a2bba0b5018 100644 (file)
@@ -120,8 +120,6 @@ int new_get_skb_ifindex(int val, struct __sk_buff *skb, int var)
        void *data = (void *)(long)skb->data;
        struct ipv6hdr ip6, *ip6p;
        int ifindex = skb->ifindex;
-       __u32 eth_proto;
-       __u32 nh_off;
 
        /* check that BPF extension can read packet via direct packet access */
        if (data + 14 + sizeof(ip6) > data_end)
index bb2a77c5b62b0a632e68731abbaa28d8718583de..370a0e1922e0ebfeb24e3c502d72d45c1cab6225 100644 (file)
@@ -23,7 +23,7 @@ struct {
 SEC("freplace/handle_kprobe")
 int new_handle_kprobe(struct pt_regs *ctx)
 {
-       struct hmap_elem zero = {}, *val;
+       struct hmap_elem *val;
        int key = 0;
 
        val = bpf_map_lookup_elem(&hash_map, &key);
index 84e5dc10243ca327de69d35aee1e5a2748d0a240..6b9b3c56f009a33bd733023dc953ab15ed14c655 100644 (file)
@@ -45,7 +45,6 @@ __failure __msg("unbounded memory access")
 int iter_err_unsafe_asm_loop(const void *ctx)
 {
        struct bpf_iter_num it;
-       int *v, i = 0;
 
        MY_PID_GUARD();
 
@@ -88,7 +87,7 @@ __success
 int iter_while_loop(const void *ctx)
 {
        struct bpf_iter_num it;
-       int *v, i;
+       int *v;
 
        MY_PID_GUARD();
 
@@ -106,7 +105,7 @@ __success
 int iter_while_loop_auto_cleanup(const void *ctx)
 {
        __attribute__((cleanup(bpf_iter_num_destroy))) struct bpf_iter_num it;
-       int *v, i;
+       int *v;
 
        MY_PID_GUARD();
 
@@ -124,7 +123,7 @@ __success
 int iter_for_loop(const void *ctx)
 {
        struct bpf_iter_num it;
-       int *v, i;
+       int *v;
 
        MY_PID_GUARD();
 
@@ -192,7 +191,7 @@ __success
 int iter_manual_unroll_loop(const void *ctx)
 {
        struct bpf_iter_num it;
-       int *v, i;
+       int *v;
 
        MY_PID_GUARD();
 
@@ -621,7 +620,7 @@ __success
 int iter_stack_array_loop(const void *ctx)
 {
        long arr1[16], arr2[16], sum = 0;
-       int *v, i;
+       int i;
 
        MY_PID_GUARD();
 
index b05571bc67d5b6d768bcb7cb4127873133fe4ce2..c4b49ceea967b82a8b8588c20bc18e1271596b10 100644 (file)
@@ -5,6 +5,7 @@
 #include <bpf/bpf_helpers.h>
 #include <bpf/bpf_tracing.h>
 #include <bpf/bpf_core_read.h>
+#include "bpf_misc.h"
 
 /* weak and shared between two files */
 const volatile int my_tid __weak;
@@ -51,6 +52,7 @@ __weak int set_output_weak(int x)
         * cause problems for BPF static linker
         */
        whatever = bpf_core_type_size(struct task_struct);
+       __sink(whatever);
 
        output_weak1 = x;
        return x;
@@ -71,6 +73,7 @@ int BPF_PROG(handler1, struct pt_regs *regs, long id)
 
        /* make sure we have CO-RE relocations in main program */
        whatever = bpf_core_type_size(struct task_struct);
+       __sink(whatever);
 
        set_output_val2(1000);
        set_output_ctx2(ctx); /* ctx definition is hidden in BPF_PROG macro */
index ee7e3848ee4fe2be14fbb01c4a515fc6e7733a4c..013ff0645f0ccb2eb229b79bb4f8cd4151db53d0 100644 (file)
@@ -5,6 +5,7 @@
 #include <bpf/bpf_helpers.h>
 #include <bpf/bpf_tracing.h>
 #include <bpf/bpf_core_read.h>
+#include "bpf_misc.h"
 
 /* weak and shared between both files */
 const volatile int my_tid __weak;
@@ -51,6 +52,7 @@ __weak int set_output_weak(int x)
         * cause problems for BPF static linker
         */
        whatever = 2 * bpf_core_type_size(struct task_struct);
+       __sink(whatever);
 
        output_weak2 = x;
        return 2 * x;
@@ -71,6 +73,7 @@ int BPF_PROG(handler2, struct pt_regs *regs, long id)
 
        /* make sure we have CO-RE relocations in main program */
        whatever = bpf_core_type_size(struct task_struct);
+       __sink(whatever);
 
        set_output_val1(2000);
        set_output_ctx1(ctx); /* ctx definition is hidden in BPF_PROG macro */
index 4fa4a9b01bde31704b0b4ad59e282400e7b56dc5..53ded51a3abbcfb4367923bb346ad9912dfb067b 100644 (file)
@@ -313,7 +313,6 @@ SEC("tc")
 int map_list_push_pop_multiple(void *ctx)
 {
        struct map_value *v;
-       int ret;
 
        v = bpf_map_lookup_elem(&array_map, &(int){0});
        if (!v)
@@ -326,7 +325,6 @@ int inner_map_list_push_pop_multiple(void *ctx)
 {
        struct map_value *v;
        void *map;
-       int ret;
 
        map = bpf_map_lookup_elem(&map_of_maps, &(int){0});
        if (!map)
@@ -352,7 +350,6 @@ SEC("tc")
 int map_list_in_list(void *ctx)
 {
        struct map_value *v;
-       int ret;
 
        v = bpf_map_lookup_elem(&array_map, &(int){0});
        if (!v)
@@ -365,7 +362,6 @@ int inner_map_list_in_list(void *ctx)
 {
        struct map_value *v;
        void *map;
-       int ret;
 
        map = bpf_map_lookup_elem(&map_of_maps, &(int){0});
        if (!map)
index 69cdc07cba13aacb66ddc9dd56e7fcafede8837f..41978b46f58eead49051f03e476bda6b1965d48c 100644 (file)
@@ -557,7 +557,6 @@ SEC("?tc")
 int incorrect_head_off2(void *ctx)
 {
        struct foo *f;
-       struct bar *b;
 
        f = bpf_obj_new(typeof(*f));
        if (!f)
index 19423ed862e35938749aa4575458f1c80a949f66..01c74bc870aeafee9d6169d9495004de14d3b068 100644 (file)
@@ -77,7 +77,6 @@ int BPF_PROG(inode_rename, struct inode *old_dir, struct dentry *old_dentry,
             struct inode *new_dir, struct dentry *new_dentry,
             unsigned int flags)
 {
-       __u32 pid = bpf_get_current_pid_tgid() >> 32;
        struct local_storage *storage;
        int err;
 
index 3903d30217b8f9bd76b5b0bb3a6ac32e25980ff4..dae5dab1bbf7d47d821748e68f3a769aa89cd2f0 100644 (file)
@@ -515,7 +515,6 @@ int test_ls_map_kptr_ref1(void *ctx)
 {
        struct task_struct *current;
        struct map_value *v;
-       int ret;
 
        current = bpf_get_current_task_btf();
        if (!current)
@@ -534,7 +533,6 @@ int test_ls_map_kptr_ref2(void *ctx)
 {
        struct task_struct *current;
        struct map_value *v;
-       int ret;
 
        current = bpf_get_current_task_btf();
        if (!current)
@@ -550,7 +548,6 @@ int test_ls_map_kptr_ref_del(void *ctx)
 {
        struct task_struct *current;
        struct map_value *v;
-       int ret;
 
        current = bpf_get_current_task_btf();
        if (!current)
index f718b2c212dc84aa1d42b9098b8feb8d78fd547c..f9ef8aee56f16207a402b41aa6e40596ea244993 100644 (file)
@@ -26,7 +26,6 @@ SEC("cgroup/skb")
 int bpf_nextcnt(struct __sk_buff *skb)
 {
        union percpu_net_cnt *percpu_cnt;
-       char fmt[] = "%d %llu %llu\n";
        union net_cnt *cnt;
        __u64 ts, dt;
        int ret;
index 1d8918dfbd3ff541b5abaf51b4bb9e6106e924f2..c0062645fc68bbb13f064390f2eccdf4faf233c0 100644 (file)
@@ -53,7 +53,6 @@ static int __strncmp(const void *m1, const void *m2, size_t len)
        do {                                                            \
                static const char _expectedval[EXPECTED_STRSIZE] =      \
                                                        _expected;      \
-               static const char _ptrtype[64] = #_type;                \
                __u64 _hflags = _flags | BTF_F_COMPACT;                 \
                static _type _ptrdata = __VA_ARGS__;                    \
                static struct btf_ptr _ptr = { };                       \
index 45204fe0c570bf896af0d2c8a90d846990feb03c..29c1639fc78a014c02a31ddfbbe3e6edf0b84245 100644 (file)
@@ -22,7 +22,6 @@ long dropped __attribute__((aligned(128))) = 0;
 SEC("fentry/" SYS_PREFIX "sys_getpgid")
 int bench_perfbuf(void *ctx)
 {
-       __u64 *sample;
        int i;
 
        for (i = 0; i < batch_cnt; i++) {
index f2e7a31c8d751e74a047565c9c892665a1791497..026d573ce1797ba715bf76f8d99a105643c57554 100644 (file)
@@ -345,7 +345,7 @@ int __on_event(struct bpf_raw_tracepoint_args *ctx)
 SEC("raw_tracepoint/kfree_skb")
 int on_event(struct bpf_raw_tracepoint_args* ctx)
 {
-       int i, ret = 0;
+       int ret = 0;
        ret |= __on_event(ctx);
        ret |= __on_event(ctx);
        ret |= __on_event(ctx);
index 340f97da1084fa9bd3ec45935744b2f01b8b7ad2..7651843f5a800f8535abc67270123fcce0532097 100644 (file)
@@ -16,17 +16,6 @@ struct node_data {
        struct bpf_list_node node;
 };
 
-static bool less(struct bpf_rb_node *a, const struct bpf_rb_node *b)
-{
-       struct node_data *node_a;
-       struct node_data *node_b;
-
-       node_a = container_of(a, struct node_data, node);
-       node_b = container_of(b, struct node_data, node);
-
-       return node_a->key < node_b->key;
-}
-
 #define private(name) SEC(".data." #name) __hidden __attribute__((aligned(8)))
 private(A) struct bpf_spin_lock glock;
 private(A) struct bpf_rb_root groot __contains(node_data, node);
index 1ced900f3fced6c33d7bd03bc53845c1fddf98a8..46d7d18a218fd3e1607d309ce2766ce57001c069 100644 (file)
@@ -105,7 +105,7 @@ long rbtree_api_remove_unadded_node(void *ctx)
 }
 
 SEC("?tc")
-__failure __msg("Unreleased reference id=2 alloc_insn=11")
+__failure __msg("Unreleased reference id=2 alloc_insn=10")
 long rbtree_api_remove_no_drop(void *ctx)
 {
        struct bpf_rb_node *res;
@@ -119,6 +119,7 @@ long rbtree_api_remove_no_drop(void *ctx)
        res = bpf_rbtree_remove(&groot, res);
 
        n = container_of(res, struct node_data, node);
+       __sink(n);
        bpf_spin_unlock(&glock);
 
        /* bpf_obj_drop(n) is missing here */
index 7250bb76d18ad682bec43c2c224bf21a34475c8d..6a8c88e58df24a2c3ef6888422a8ffac3c154ff6 100644 (file)
@@ -179,8 +179,6 @@ SEC("?fentry.s/" SYS_PREFIX "sys_getpgid")
 int miss_lock(void *ctx)
 {
        struct task_struct *task;
-       struct css_set *cgroups;
-       struct cgroup *dfl_cgrp;
 
        /* missing bpf_rcu_read_lock() */
        task = bpf_get_current_task_btf();
@@ -195,8 +193,6 @@ SEC("?fentry.s/" SYS_PREFIX "sys_getpgid")
 int miss_unlock(void *ctx)
 {
        struct task_struct *task;
-       struct css_set *cgroups;
-       struct cgroup *dfl_cgrp;
 
        /* missing bpf_rcu_read_unlock() */
        task = bpf_get_current_task_btf();
index a47bb0120719928c244dfefa1a8f1a42a02336df..76556e0b42b24c7e00fc6ca8f0ecdcafaa6a8480 100644 (file)
@@ -23,7 +23,6 @@ SEC("raw_tp/sys_enter")
 int BPF_PROG(read_bpf_task_storage_busy)
 {
        int *value;
-       int key;
 
        if (!CONFIG_PREEMPT)
                return 0;
index 3d1ae8b3402f9afde8109afa09ce31210448125d..59748c95471a281eec097c0c7d8d782b5c3033db 100644 (file)
@@ -17,8 +17,6 @@ SEC("cgroup/recvmsg4")
 int recvmsg4_prog(struct bpf_sock_addr *ctx)
 {
        struct bpf_sock *sk;
-       __u32 user_ip4;
-       __u16 user_port;
 
        sk = ctx->sk;
        if (!sk)
index 27dfb21b21b47d4490e79cc77a1c31cecdbdf545..d9a4016596d51a35b6dbf12372044fd366fc61a6 100644 (file)
@@ -20,8 +20,6 @@ SEC("cgroup/recvmsg6")
 int recvmsg6_prog(struct bpf_sock_addr *ctx)
 {
        struct bpf_sock *sk;
-       __u32 user_ip4;
-       __u16 user_port;
 
        sk = ctx->sk;
        if (!sk)
index ea75a44cb7fce5b0d0afb29f8d4e125cf6c3e8fc..351e79aef2fae1e19994cb4c8a5a6cfa394e7705 100644 (file)
@@ -21,8 +21,6 @@
 SEC("cgroup/sendmsg4")
 int sendmsg_v4_prog(struct bpf_sock_addr *ctx)
 {
-       int prio;
-
        if (ctx->type != SOCK_DGRAM)
                return 0;
 
index e2468a6d01a59a1f2f333618f5c40c50af59ed82..0660f29dca9555ffcfe3aaf91cadf59726480ebb 100644 (file)
@@ -1,6 +1,7 @@
 #include <linux/bpf.h>
 #include <bpf/bpf_helpers.h>
 #include <bpf/bpf_endian.h>
+#include "bpf_misc.h"
 
 struct {
        __uint(type, BPF_MAP_TYPE_SOCKMAP);
@@ -40,6 +41,9 @@ int bpf_prog2(struct __sk_buff *skb)
        __u8 *d = data;
        __u8 sk, map;
 
+       __sink(lport);
+       __sink(rport);
+
        if (data + 8 > data_end)
                return SK_DROP;
 
index e562be6356f308cfa36cfffe77475f273e31c0db..e02cfd3807469599eef5e1438ed8167e10b8f205 100644 (file)
@@ -391,7 +391,6 @@ static __always_inline void *read_map_var(struct strobemeta_cfg *cfg,
        struct strobe_map_raw map;
        void *location;
        uint64_t len;
-       int i;
 
        descr->tag_len = 0; /* presume no tag is set */
        descr->cnt = -1; /* presume no value is set */
index 7fab39a3bb12aa3d4927691bddd5f1547880c22e..99c8d1d8a187b9f14e0b617d5e545065067fce61 100644 (file)
@@ -2,6 +2,7 @@
 #include <linux/bpf.h>
 #include <bpf/bpf_helpers.h>
 #include "bpf_legacy.h"
+#include "bpf_misc.h"
 
 struct {
        __uint(type, BPF_MAP_TYPE_PROG_ARRAY);
@@ -20,6 +21,8 @@ int subprog_tail2(struct __sk_buff *skb)
        else
                bpf_tail_call_static(skb, &jmp_table, 1);
 
+       __sink(arr[sizeof(arr) - 1]);
+
        return skb->len;
 }
 
@@ -30,6 +33,8 @@ int subprog_tail(struct __sk_buff *skb)
 
        bpf_tail_call_static(skb, &jmp_table, 0);
 
+       __sink(arr[sizeof(arr) - 1]);
+
        return skb->len * 2;
 }
 
@@ -38,6 +43,8 @@ int classifier_0(struct __sk_buff *skb)
 {
        volatile char arr[128] = {};
 
+       __sink(arr[sizeof(arr) - 1]);
+
        return subprog_tail2(skb);
 }
 
@@ -46,6 +53,8 @@ int classifier_1(struct __sk_buff *skb)
 {
        volatile char arr[128] = {};
 
+       __sink(arr[sizeof(arr) - 1]);
+
        return skb->len * 3;
 }
 
@@ -54,6 +63,8 @@ int entry(struct __sk_buff *skb)
 {
        volatile char arr[128] = {};
 
+       __sink(arr[sizeof(arr) - 1]);
+
        return subprog_tail(skb);
 }
 
index 41ce83da78e8b5302889cb36bae60a399e9898d0..4a9f63bea66cdc9caaabac994025a0948f397572 100644 (file)
@@ -1,6 +1,7 @@
 // SPDX-License-Identifier: GPL-2.0
 #include <linux/bpf.h>
 #include <bpf/bpf_helpers.h>
+#include "bpf_misc.h"
 
 #define __unused __attribute__((unused))
 
@@ -36,6 +37,8 @@ int entry(struct __sk_buff *skb)
        /* Have data on stack which size is not a multiple of 8 */
        volatile char arr[1] = {};
 
+       __sink(arr[0]);
+
        return subprog_tail(skb);
 }
 
index f19d54eda4f1778e7c4f3548b68daeec020e4364..002c7f69e47f59ce71bc7265f8684dc339cc478f 100644 (file)
@@ -109,6 +109,7 @@ int BPF_PROG(task_kfunc_acquire_unreleased, struct task_struct *task, u64 clone_
        acquired = bpf_task_acquire(task);
 
        /* Acquired task is never released. */
+       __sink(acquired);
 
        return 0;
 }
index 9f359cfd29e7cdd983319eb76452483cb6e30fcd..aebc4bb14e7dd894043dbc7f58c56187fd673cfc 100644 (file)
@@ -171,8 +171,6 @@ static void lookup_compare_pid(const struct task_struct *p)
 SEC("tp_btf/task_newtask")
 int BPF_PROG(test_task_from_pid_arg, struct task_struct *task, u64 clone_flags)
 {
-       struct task_struct *acquired;
-
        if (!is_test_kfunc_task())
                return 0;
 
@@ -183,8 +181,6 @@ int BPF_PROG(test_task_from_pid_arg, struct task_struct *task, u64 clone_flags)
 SEC("tp_btf/task_newtask")
 int BPF_PROG(test_task_from_pid_current, struct task_struct *task, u64 clone_flags)
 {
-       struct task_struct *current, *acquired;
-
        if (!is_test_kfunc_task())
                return 0;
 
@@ -208,8 +204,6 @@ static int is_pid_lookup_valid(s32 pid)
 SEC("tp_btf/task_newtask")
 int BPF_PROG(test_task_from_pid_invalid, struct task_struct *task, u64 clone_flags)
 {
-       struct task_struct *acquired;
-
        if (!is_test_kfunc_task())
                return 0;
 
index 9fc603c9d673e901e0e9ff47fe70fcd8c21bcbb8..77ad8adf68daaae1e648d12e976683069d04da2f 100644 (file)
@@ -75,7 +75,6 @@ nf_ct_test(struct nf_conn *(*lookup_fn)(void *, struct bpf_sock_tuple *, u32,
        struct bpf_ct_opts___local opts_def = { .l4proto = IPPROTO_TCP, .netns_id = -1 };
        struct bpf_sock_tuple bpf_tuple;
        struct nf_conn *ct;
-       int err;
 
        __builtin_memset(&bpf_tuple, 0, sizeof(bpf_tuple.ipv4));
 
index f45a7095de7a7b47c269e7450b6b7e457c2d12b8..f41c81212ee9270120b4e72dba764afbba3bc64e 100644 (file)
@@ -455,7 +455,6 @@ static ret_t forward_to_next_hop(struct __sk_buff *skb, struct bpf_dynptr *dynpt
 
 static ret_t skip_next_hops(__u64 *offset, int n)
 {
-       __u32 res;
        switch (n) {
        case 1:
                *offset += sizeof(struct in_addr);
index ab1e647aeb317923fcc7c72e0a38fc29e6d2338c..b86fdda2a6ea5e52a80177b7420efa6f13265038 100644 (file)
@@ -42,7 +42,6 @@ int test_core_bitfields(void *ctx)
 {
        struct core_reloc_bitfields *in = (void *)&data.in;
        struct core_reloc_bitfields_output *out = (void *)&data.out;
-       uint64_t res;
 
        out->ub1 = BPF_CORE_READ_BITFIELD_PROBED(in, ub1);
        out->ub2 = BPF_CORE_READ_BITFIELD_PROBED(in, ub2);
index 23970a20b3249dec95318164865f3ca62d0d7f45..b85fc8c423ba7c382a5e747275ed12c81e9ddbcd 100644 (file)
@@ -18,6 +18,8 @@ int f1(struct __sk_buff *skb)
 {
        volatile char buf[MAX_STACK] = {};
 
+       __sink(buf[MAX_STACK - 1]);
+
        return f0(0, skb) + skb->len;
 }
 
@@ -34,6 +36,8 @@ int f3(int val, struct __sk_buff *skb, int var)
 {
        volatile char buf[MAX_STACK] = {};
 
+       __sink(buf[MAX_STACK - 1]);
+
        return skb->ifindex * val * var;
 }
 
index 3dce97fb52a4b7723132bcb75ac9813c91295f62..2beab9c3b68a7747c40e8f8cc76c5ecebc7d1bd3 100644 (file)
@@ -18,6 +18,8 @@ int f1(struct __sk_buff *skb)
 {
        volatile char buf[MAX_STACK] = {};
 
+       __sink(buf[MAX_STACK - 1]);
+
        return f0(0, skb) + skb->len;
 }
 
@@ -34,6 +36,8 @@ int f3(int val, struct __sk_buff *skb, int var)
 {
        volatile char buf[MAX_STACK] = {};
 
+       __sink(buf[MAX_STACK - 1]);
+
        return skb->ifindex * val * var;
 }
 
index 473a22794a628f6a02647d216fbf7b1e2de65cf0..8b438128f46b8aa74ddd4115bd132ef24ec9ac4c 100644 (file)
@@ -28,7 +28,7 @@ struct bigelement {
 SEC("raw_tracepoint/sys_enter")
 int bpf_hash_large_key_test(void *ctx)
 {
-       int zero = 0, err = 1, value = 42;
+       int zero = 0, value = 42;
        struct bigelement *key;
 
        key = bpf_map_lookup_elem(&key_map, &zero);
index a72a5bf3812a8f9877e3be7645c75be03d38e860..27109b877714f0b263c9fe560ac3baa270f777bf 100644 (file)
@@ -35,7 +35,6 @@ SEC("raw_tp/sys_enter")
 int handler2(const void *ctx)
 {
        int *active;
-       __u32 cpu;
 
        active = bpf_this_cpu_ptr(&bpf_prog_active);
        write_active(active);
index 64c2d9ced529ccd4aeb1890a067f60844377613a..42718cd8e6a49989a9e984c663f83515df8b1730 100644 (file)
@@ -56,7 +56,7 @@ int handle_legacy(void *ctx)
 SEC("tp/raw_syscalls/sys_enter")
 int handle_modern(void *ctx)
 {
-       int zero = 0, cur_pid;
+       int cur_pid;
 
        cur_pid = bpf_get_current_pid_tgid() >> 32;
        if (cur_pid != my_pid_var)
index acf073db9e8b96e037d0818971fe42e5ea7eba38..1c02511b73cdbb3ca972db57d69993979c3b04f8 100644 (file)
@@ -33,7 +33,7 @@ struct {
 SEC("cgroup/skb")
 int bpf_map_lock_test(struct __sk_buff *skb)
 {
-       struct hmap_elem zero = {}, *val;
+       struct hmap_elem *val;
        int rnd = bpf_get_prandom_u32();
        int key = 0, err = 1, i;
        struct array_elem *q;
index ded71b3ff6b4760df2dcbe970c5ac52b31a6dcaa..2850ae788a919515778c10183a0d296fa651efae 100644 (file)
@@ -4,6 +4,7 @@
 #include <stddef.h>
 #include <linux/bpf.h>
 #include <bpf/bpf_helpers.h>
+#include "bpf_misc.h"
 
 struct {
        __uint(type, BPF_MAP_TYPE_ARRAY);
@@ -19,6 +20,7 @@ int test_obj_id(void *ctx)
        __u64 *value;
 
        value = bpf_map_lookup_elem(&test_map_id, &key);
+       __sink(value);
 
        return 0;
 }
index 79bab9b50e9e5459064dc8ce72583d6c2ca7276b..d9b2ba7ac340e72ba2884977e1d7e3b97f1673ae 100644 (file)
@@ -87,7 +87,6 @@ int xdp_ingress_v6(struct xdp_md *xdp)
        __u8 tcp_hdr_opt_len = 0;
        struct tcphdr *tcp_hdr;
        __u64 tcp_offset = 0;
-       __u32 off;
        int err;
 
        tcp_offset = sizeof(struct ethhdr) + sizeof(struct ipv6hdr);
index d3b319722e30914c010db507d35879cb4baf627d..dc6e43bc6a62c1db06196a4d9bfd3e19d332c6e8 100644 (file)
@@ -30,7 +30,7 @@ __u32 server_id;
 static int parse_hdr_opt(struct bpf_dynptr *ptr, __u32 *off, __u8 *hdr_bytes_remaining,
                         __u32 *server_id)
 {
-       __u8 *tcp_opt, kind, hdr_len;
+       __u8 kind, hdr_len;
        __u8 buffer[sizeof(kind) + sizeof(hdr_len) + sizeof(*server_id)];
        __u8 *data;
 
index 5cd7c096f62d22e220217d95837b0a1f1effe29d..bce7173152c6c4e3945523c887fbcd493c5277df 100644 (file)
@@ -13,6 +13,7 @@
 #include <linux/pkt_cls.h>
 #include <bpf/bpf_helpers.h>
 #include <bpf/bpf_endian.h>
+#include "bpf_misc.h"
 
 /* llvm will optimize both subprograms into exactly the same BPF assembly
  *
@@ -51,6 +52,8 @@ int get_skb_len(struct __sk_buff *skb)
 {
        volatile char buf[MAX_STACK] = {};
 
+       __sink(buf[MAX_STACK - 1]);
+
        return skb->len;
 }
 
@@ -73,6 +76,8 @@ int get_skb_ifindex(int val, struct __sk_buff *skb, int var)
 {
        volatile char buf[MAX_STACK] = {};
 
+       __sink(buf[MAX_STACK - 1]);
+
        return skb->ifindex * val * var;
 }
 
index 5bdc0d38efc0581350064b78531684afccf965b6..501cefa976333feaf0ef1e904750372cafd0777a 100644 (file)
@@ -41,7 +41,6 @@ int test_ringbuf(void *ctx)
 {
        int cur_pid = bpf_get_current_pid_tgid() >> 32;
        struct sample *sample;
-       int zero = 0;
 
        if (cur_pid != pid)
                return 0;
index 2760bf60d05adcb6923b5a58102e8d5b77613e53..21bb7da90ea50de5b97df7b26d812c21e4f79241 100644 (file)
@@ -53,6 +53,7 @@ int test_ringbuf_mem_map_key(void *ctx)
        /* test using 'sample' (PTR_TO_MEM | MEM_ALLOC) as map key arg
         */
        lookup_val = (int *)bpf_map_lookup_elem(&hash_map, sample);
+       __sink(lookup_val);
 
        /* workaround - memcpy is necessary so that verifier doesn't
         * complain with:
index e416e0ce12b7ddb37478137eb4e1c9147d90a1e1..9626baa6779c893d84cd60a963faca148659fa7b 100644 (file)
@@ -59,7 +59,6 @@ int test_ringbuf(void *ctx)
        int cur_pid = bpf_get_current_pid_tgid() >> 32;
        struct sample *sample;
        void *rb;
-       int zero = 0;
 
        if (cur_pid != pid)
                return 0;
index 7d56ed47cd4d86e80ba8f497838ada6d24a8d8a6..5eb25c6ad75b1a9c61f22e978d817d3dc88b3a2f 100644 (file)
@@ -64,7 +64,7 @@ SEC("sk_reuseport")
 int _select_by_skb_data(struct sk_reuseport_md *reuse_md)
 {
        __u32 linum, index = 0, flags = 0, index_zero = 0;
-       __u32 *result_cnt, *linum_value;
+       __u32 *result_cnt;
        struct data_check data_check = {};
        struct cmd *cmd, cmd_copy;
        void *data, *data_end;
index 21b19b758c4ebda1b60a5100af7b9e29fba2947c..3079244c7f96ea9d7c58bc51eb9ee8bb91f9f158 100644 (file)
@@ -15,6 +15,7 @@
 #include <sys/socket.h>
 #include <bpf/bpf_helpers.h>
 #include <bpf/bpf_endian.h>
+#include "bpf_misc.h"
 
 #if defined(IPROUTE2_HAVE_LIBBPF)
 /* Use a new-style map definition. */
@@ -57,7 +58,6 @@ get_tuple(struct __sk_buff *skb, bool *ipv4, bool *tcp)
        void *data = (void *)(long)skb->data;
        struct bpf_sock_tuple *result;
        struct ethhdr *eth;
-       __u64 tuple_len;
        __u8 proto = 0;
        __u64 ihl_len;
 
@@ -94,6 +94,7 @@ get_tuple(struct __sk_buff *skb, bool *ipv4, bool *tcp)
                return NULL;
 
        *tcp = (proto == IPPROTO_TCP);
+       __sink(ihl_len);
        return result;
 }
 
@@ -173,7 +174,6 @@ int bpf_sk_assign_test(struct __sk_buff *skb)
        struct bpf_sock_tuple *tuple;
        bool ipv4 = false;
        bool tcp = false;
-       int tuple_len;
        int ret = 0;
 
        tuple = get_tuple(skb, &ipv4, &tcp);
index 6058dcb11b36cf26d80475c48153881afe5d8684..71f844b9b902b5b55ad1053fab28c25e68b28677 100644 (file)
@@ -391,7 +391,6 @@ SEC("sk_lookup")
 int ctx_narrow_access(struct bpf_sk_lookup *ctx)
 {
        struct bpf_sock *sk;
-       int err, family;
        __u32 val_u32;
        bool v4;
 
@@ -645,9 +644,7 @@ static __always_inline int select_server_a(struct bpf_sk_lookup *ctx)
 SEC("sk_lookup")
 int multi_prog_redir1(struct bpf_sk_lookup *ctx)
 {
-       int ret;
-
-       ret = select_server_a(ctx);
+       (void)select_server_a(ctx);
        bpf_map_update_elem(&run_map, &KEY_PROG1, &PROG_DONE, BPF_ANY);
        return SK_PASS;
 }
@@ -655,9 +652,7 @@ int multi_prog_redir1(struct bpf_sk_lookup *ctx)
 SEC("sk_lookup")
 int multi_prog_redir2(struct bpf_sk_lookup *ctx)
 {
-       int ret;
-
-       ret = select_server_a(ctx);
+       (void)select_server_a(ctx);
        bpf_map_update_elem(&run_map, &KEY_PROG2, &PROG_DONE, BPF_ANY);
        return SK_PASS;
 }
index 6ccf6d546074e867ce1ace666da4e6f2f950be49..e9efc3263022530f6bac55b0294d441d5f54ddf5 100644 (file)
@@ -110,7 +110,6 @@ int err_modify_sk_pointer(struct __sk_buff *skb)
 {
        struct bpf_sock_tuple tuple = {};
        struct bpf_sock *sk;
-       __u32 family;
 
        sk = bpf_sk_lookup_tcp(skb, &tuple, sizeof(tuple), BPF_F_CURRENT_NETNS, 0);
        if (sk) {
@@ -125,7 +124,6 @@ int err_modify_sk_or_null_pointer(struct __sk_buff *skb)
 {
        struct bpf_sock_tuple tuple = {};
        struct bpf_sock *sk;
-       __u32 family;
 
        sk = bpf_sk_lookup_tcp(skb, &tuple, sizeof(tuple), BPF_F_CURRENT_NETNS, 0);
        sk += 1;
index 9f4b8f9f1181120ec359b649b6d4c22841873380..bbad3c2d9aa57d7e996b4bf46b3d27ecf37f867f 100644 (file)
@@ -121,7 +121,7 @@ static void tpcpy(struct bpf_tcp_sock *dst,
 SEC("cgroup_skb/egress")
 int egress_read_sock_fields(struct __sk_buff *skb)
 {
-       struct bpf_spinlock_cnt cli_cnt_init = { .lock = 0, .cnt = 0xeB9F };
+       struct bpf_spinlock_cnt cli_cnt_init = { .lock = {}, .cnt = 0xeB9F };
        struct bpf_spinlock_cnt *pkt_out_cnt, *pkt_out_cnt10;
        struct bpf_tcp_sock *tp, *tp_ret;
        struct bpf_sock *sk, *sk_ret;
index 6c85b00f27b2e5e795bc3fa9dbb21c50d9999ece..baf9ebc6d903f0a8b97dcb01b8d0870aa33c28fd 100644 (file)
@@ -14,6 +14,7 @@
 #include <sys/socket.h>
 #include <bpf/bpf_helpers.h>
 #include <bpf/bpf_endian.h>
+#include "bpf_misc.h"
 
 /* Sockmap sample program connects a client and a backend together
  * using cgroups.
@@ -111,12 +112,15 @@ int bpf_prog2(struct __sk_buff *skb)
        int len, *f, ret, zero = 0;
        __u64 flags = 0;
 
+       __sink(rport);
        if (lport == 10000)
                ret = 10;
        else
                ret = 1;
 
        len = (__u32)skb->data_end - (__u32)skb->data;
+       __sink(len);
+
        f = bpf_map_lookup_elem(&sock_skb_opts, &zero);
        if (f && *f) {
                ret = 3;
@@ -180,7 +184,6 @@ int bpf_prog3(struct __sk_buff *skb)
        if (err)
                return SK_DROP;
        bpf_write_pass(skb, 13);
-tls_out:
        return ret;
 }
 
@@ -188,8 +191,7 @@ SEC("sockops")
 int bpf_sockmap(struct bpf_sock_ops *skops)
 {
        __u32 lport, rport;
-       int op, err = 0, index, key, ret;
-
+       int op, err, ret;
 
        op = (int) skops->op;
 
@@ -228,6 +230,8 @@ int bpf_sockmap(struct bpf_sock_ops *skops)
                break;
        }
 
+       __sink(err);
+
        return 0;
 }
 
@@ -321,6 +325,10 @@ int bpf_prog8(struct sk_msg_md *msg)
        } else {
                return SK_DROP;
        }
+
+       __sink(data_end);
+       __sink(data);
+
        return SK_PASS;
 }
 SEC("sk_msg4")
index 5bd10409285b46e79525b755db3b5731966331eb..b2440a0ff422a395eef70ba7edca75dd7c9deb1f 100644 (file)
@@ -3,6 +3,7 @@
 #include <linux/bpf.h>
 #include <linux/version.h>
 #include <bpf/bpf_helpers.h>
+#include "bpf_misc.h"
 
 struct hmap_elem {
        volatile int cnt;
@@ -89,6 +90,8 @@ int bpf_spin_lock_test(struct __sk_buff *skb)
        credit = q->credit;
        bpf_spin_unlock(&q->lock);
 
+       __sink(credit);
+
        /* spin_lock in cgroup local storage */
        cls = bpf_get_local_storage(&cls_map, 0);
        bpf_spin_lock(&cls->lock);
index 125beec31834b7f2e860897dfa8e74574e571638..74ec09f040b7a265c9573b96890bfeae6a92b7f8 100644 (file)
@@ -163,9 +163,9 @@ static int skb_get_type(struct __sk_buff *skb)
                ip6h = data + sizeof(struct ethhdr);
                if (ip6h + 1 > data_end)
                        return -1;
-               if (v6_equal(ip6h->saddr, (struct in6_addr)ip6_src))
+               if (v6_equal(ip6h->saddr, (struct in6_addr){{ip6_src}}))
                        ns = SRC_NS;
-               else if (v6_equal(ip6h->saddr, (struct in6_addr)ip6_dst))
+               else if (v6_equal(ip6h->saddr, (struct in6_addr){{ip6_dst}}))
                        ns = DST_NS;
                inet_proto = ip6h->nexthdr;
                trans = ip6h + 1;
index 3e32ea375ab4edf3e2b6a484e810a057210ff971..de15155f2609f9ac1155610edda10bb5d946168d 100644 (file)
@@ -94,7 +94,7 @@ int tc_dst(struct __sk_buff *skb)
                redirect = is_remote_ep_v4(skb, __bpf_constant_htonl(ip4_src));
                break;
        case __bpf_constant_htons(ETH_P_IPV6):
-               redirect = is_remote_ep_v6(skb, (struct in6_addr)ip6_src);
+               redirect = is_remote_ep_v6(skb, (struct in6_addr){{ip6_src}});
                break;
        }
 
@@ -119,7 +119,7 @@ int tc_src(struct __sk_buff *skb)
                redirect = is_remote_ep_v4(skb, __bpf_constant_htonl(ip4_dst));
                break;
        case __bpf_constant_htons(ETH_P_IPV6):
-               redirect = is_remote_ep_v6(skb, (struct in6_addr)ip6_dst);
+               redirect = is_remote_ep_v6(skb, (struct in6_addr){{ip6_dst}});
                break;
        }
 
index 3ded052807576ea4dd0dea1f180179a0d2b5eee3..cf7ed8cbb1fe7bd7951f4ee0e25eba6bbbcb78ba 100644 (file)
@@ -46,8 +46,6 @@ int bpf_testcb(struct bpf_sock_ops *skops)
        struct bpf_sock_ops *reuse = skops;
        struct tcphdr *thdr;
        int window_clamp = 9216;
-       int good_call_rv = 0;
-       int bad_call_rv = 0;
        int save_syn = 1;
        int rv = -1;
        int v = 0;
index 95b4aa0928baa578da680ae304a30d7c75d1e990..9ab2d55ab7c0e040a6a606f40be8ad64c135a588 100644 (file)
@@ -209,7 +209,6 @@ int erspan_get_tunnel(struct __sk_buff *skb)
 {
        struct bpf_tunnel_key key;
        struct erspan_metadata md;
-       __u32 index;
        int ret;
 
        ret = bpf_skb_get_tunnel_key(skb, &key, sizeof(key), 0);
@@ -289,7 +288,6 @@ int ip4ip6erspan_get_tunnel(struct __sk_buff *skb)
 {
        struct bpf_tunnel_key key;
        struct erspan_metadata md;
-       __u32 index;
        int ret;
 
        ret = bpf_skb_get_tunnel_key(skb, &key, sizeof(key),
@@ -405,8 +403,6 @@ int vxlan_get_tunnel_src(struct __sk_buff *skb)
        int ret;
        struct bpf_tunnel_key key;
        struct vxlan_metadata md;
-       __u32 orig_daddr;
-       __u32 index = 0;
 
        ret = bpf_skb_get_tunnel_key(skb, &key, sizeof(key),
                                     BPF_F_TUNINFO_FLAGS);
@@ -443,9 +439,7 @@ int veth_set_outer_dst(struct __sk_buff *skb)
        void *data_end = (void *)(long)skb->data_end;
        struct udphdr *udph;
        struct iphdr *iph;
-       __u32 index = 0;
        int ret = 0;
-       int shrink;
        __s64 csum;
 
        if ((void *)eth + sizeof(*eth) > data_end) {
index aa6de32b50d1d16a782d3df3166e5237340c3f4a..962f3462066a89ee071c4cc7d7b0b584818e015c 100644 (file)
@@ -18,8 +18,6 @@ int usdt_100_sum;
 SEC("usdt//proc/self/exe:test:usdt_100")
 int BPF_USDT(usdt_100, int x)
 {
-       long tmp;
-
        if (my_pid != (bpf_get_current_pid_tgid() >> 32))
                return 0;
 
index ac6135d9374c46e248225045f91ebe34de5ef89f..323a73fb2e8c2175c0c6a59800fc1e56862c9587 100644 (file)
@@ -11,7 +11,7 @@ int balancer_ingress(struct __sk_buff *ctx)
        void *data_end = (void *)(long)ctx->data_end;
        void *data = (void *)(long)ctx->data;
        void *ptr;
-       int ret = 0, nh_off, i = 0;
+       int nh_off, i = 0;
 
        nh_off = 14;
 
index f90ffcafd1e82b9bdea9a1fc80596e23f2197a0e..f5318f7570840851989cc86cefe37931aeb5eb6a 100644 (file)
@@ -11,7 +11,7 @@ int balancer_ingress(struct __sk_buff *ctx)
        void *data_end = (void *)(long)ctx->data_end;
        void *data = (void *)(long)ctx->data;
        void *ptr;
-       int ret = 0, nh_off, i = 0;
+       int nh_off, i = 0;
 
        nh_off = 14;
 
index ca33a9b711c4b1d95145f4aa8a3fa2df22ad54ca..2e06dbb1ad5c88bc929a35bed0ceb7be40c913ac 100644 (file)
@@ -11,7 +11,7 @@ int balancer_ingress(struct __sk_buff *ctx)
        void *data_end = (void *)(long)ctx->data_end;
        void *data = (void *)(long)ctx->data;
        void *ptr;
-       int ret = 0, nh_off, i = 0;
+       int nh_off, i = 0;
 
        nh_off = 32;
 
index 297c260fc364557b79bd9befddd733d847025a78..81bb38d72cedd58d533e92995f2efd6cdacde892 100644 (file)
@@ -5,8 +5,6 @@
 SEC("xdp")
 int _xdp_adjust_tail_grow(struct xdp_md *xdp)
 {
-       void *data_end = (void *)(long)xdp->data_end;
-       void *data = (void *)(long)xdp->data;
        int data_len = bpf_xdp_get_buff_len(xdp);
        int offset = 0;
        /* SKB_DATA_ALIGN(sizeof(struct skb_shared_info)) */
index 3379d303f41a99c664148ba5198942f8cabe37b5..ee48c49639714b56e4a1e64cea98fe8ba97f3a41 100644 (file)
@@ -45,8 +45,6 @@ SEC("fentry/FUNC")
 int BPF_PROG(trace_on_entry, struct xdp_buff *xdp)
 {
        struct meta meta;
-       void *data_end = (void *)(long)xdp->data_end;
-       void *data = (void *)(long)xdp->data;
 
        meta.ifindex = xdp->rxq->dev->ifindex;
        meta.pkt_len = bpf_xdp_get_buff_len((struct xdp_md *)xdp);
index 7521a805b5068be998089dde99e918aeb5765ffb..25ee4a22e48d94c648363490193102216a66dc78 100644 (file)
@@ -82,7 +82,6 @@ static __always_inline int handle_ipv4(struct xdp_md *xdp, struct bpf_dynptr *xd
        struct iptnl_info *tnl;
        struct ethhdr *new_eth;
        struct ethhdr *old_eth;
-       __u32 transport_hdr_sz;
        struct iphdr *iph;
        __u16 *next_iph;
        __u16 payload_len;
@@ -165,7 +164,6 @@ static __always_inline int handle_ipv6(struct xdp_md *xdp, struct bpf_dynptr *xd
        struct iptnl_info *tnl;
        struct ethhdr *new_eth;
        struct ethhdr *old_eth;
-       __u32 transport_hdr_sz;
        struct ipv6hdr *ip6h;
        __u16 payload_len;
        struct vip vip = {};
index ba48fcb98ab27de0f57f98ef779d6f63eb246da8..42c8f6ded0e4d2b0c5d3def1f5b9483ff0fada1c 100644 (file)
@@ -371,45 +371,6 @@ bool encap_v4(struct xdp_md *xdp, struct ctl_value *cval,
        return true;
 }
 
-static __attribute__ ((noinline))
-bool decap_v6(struct xdp_md *xdp, void **data, void **data_end, bool inner_v4)
-{
-       struct eth_hdr *new_eth;
-       struct eth_hdr *old_eth;
-
-       old_eth = *data;
-       new_eth = *data + sizeof(struct ipv6hdr);
-       memcpy(new_eth->eth_source, old_eth->eth_source, 6);
-       memcpy(new_eth->eth_dest, old_eth->eth_dest, 6);
-       if (inner_v4)
-               new_eth->eth_proto = 8;
-       else
-               new_eth->eth_proto = 56710;
-       if (bpf_xdp_adjust_head(xdp, (int)sizeof(struct ipv6hdr)))
-               return false;
-       *data = (void *)(long)xdp->data;
-       *data_end = (void *)(long)xdp->data_end;
-       return true;
-}
-
-static __attribute__ ((noinline))
-bool decap_v4(struct xdp_md *xdp, void **data, void **data_end)
-{
-       struct eth_hdr *new_eth;
-       struct eth_hdr *old_eth;
-
-       old_eth = *data;
-       new_eth = *data + sizeof(struct iphdr);
-       memcpy(new_eth->eth_source, old_eth->eth_source, 6);
-       memcpy(new_eth->eth_dest, old_eth->eth_dest, 6);
-       new_eth->eth_proto = 8;
-       if (bpf_xdp_adjust_head(xdp, (int)sizeof(struct iphdr)))
-               return false;
-       *data = (void *)(long)xdp->data;
-       *data_end = (void *)(long)xdp->data_end;
-       return true;
-}
-
 static __attribute__ ((noinline))
 int swap_mac_and_send(void *data, void *data_end)
 {
@@ -430,7 +391,6 @@ int send_icmp_reply(void *data, void *data_end)
        __u16 *next_iph_u16;
        __u32 tmp_addr = 0;
        struct iphdr *iph;
-       __u32 csum1 = 0;
        __u32 csum = 0;
        __u64 off = 0;
 
@@ -662,7 +622,6 @@ static int process_l3_headers_v4(struct packet_description *pckt,
                                 void *data_end)
 {
        struct iphdr *iph;
-       __u64 iph_len;
        int action;
 
        iph = data + off;
@@ -696,7 +655,6 @@ static int process_packet(void *data, __u64 off, void *data_end,
        struct packet_description pckt = { };
        struct vip_definition vip = { };
        struct lb_stats *data_stats;
-       struct eth_hdr *eth = data;
        void *lru_map = &lru_cache;
        struct vip_meta *vip_info;
        __u32 lru_stats_key = 513;
@@ -704,7 +662,6 @@ static int process_packet(void *data, __u64 off, void *data_end,
        __u32 stats_key = 512;
        struct ctl_value *cval;
        __u16 pkt_bytes;
-       __u64 iph_len;
        __u8 protocol;
        __u32 vip_num;
        int action;
index 4ddcb6dfe500fd7725dab4f6df43e0145f87779d..f3ec8086482d63559f3ee480ae1badc7c356d2c8 100644 (file)
@@ -209,19 +209,6 @@ int  xdp_prognum2(struct xdp_md *ctx)
        return XDP_PASS;
 }
 
-static __always_inline
-void shift_mac_4bytes_16bit(void *data)
-{
-       __u16 *p = data;
-
-       p[7] = p[5]; /* delete p[7] was vlan_hdr->h_vlan_TCI */
-       p[6] = p[4]; /* delete p[6] was ethhdr->h_proto */
-       p[5] = p[3];
-       p[4] = p[2];
-       p[3] = p[1];
-       p[2] = p[0];
-}
-
 static __always_inline
 void shift_mac_4bytes_32bit(void *data)
 {
index eb78e6f031299f61ca3729942de7abb21df3c86f..a9629ac230fdde51a49a6271d4600ed6c0462927 100644 (file)
@@ -63,7 +63,6 @@ SEC("?tp_btf/sys_enter")
 int BPF_PROG(untrusted_ptr, struct pt_regs *regs, long id)
 {
        struct task_struct *task, *task_dup;
-       long *ptr;
 
        task = bpf_get_current_task_btf();
        task_dup = bpf_rdonly_cast(task, bpf_core_type_id_kernel(struct task_struct));
index 165e3c2dd9a3d3604b64163c75ce471983a0ae80..4767451b59ac9114a018112a9c52c93e16653291 100644 (file)
@@ -17,7 +17,6 @@ SEC("cgroup/sock_create")
 int sock(struct bpf_sock *ctx)
 {
        int *sk_storage;
-       __u32 key;
 
        if (ctx->type != SOCK_DGRAM)
                return 1;
@@ -46,7 +45,6 @@ SEC("cgroup/sock_release")
 int sock_release(struct bpf_sock *ctx)
 {
        int *sk_storage;
-       __u32 key;
 
        if (ctx->type != SOCK_DGRAM)
                return 1;
index 0ade1110613b8eaccfa4f4a752bc85fd0e6cadc4..dd3bdf672633cd694800ccfada737502418b08b7 100644 (file)
@@ -162,8 +162,6 @@ SEC("fentry/" SYS_PREFIX "sys_prctl")
 int test_user_ringbuf_protocol(void *ctx)
 {
        long status = 0;
-       struct sample *sample = NULL;
-       struct bpf_dynptr ptr;
 
        if (!is_test_process())
                return 0;
@@ -183,10 +181,6 @@ int test_user_ringbuf_protocol(void *ctx)
 SEC("fentry/" SYS_PREFIX "sys_getpgid")
 int test_user_ringbuf(void *ctx)
 {
-       int status = 0;
-       struct sample *sample = NULL;
-       struct bpf_dynptr ptr;
-
        if (!is_test_process())
                return 0;
 
index 87c247d56f72361c5253a1d80314f344bd6fa9e2..67424084a38a1bd855648ddef18add65d780ad62 100644 (file)
@@ -70,7 +70,6 @@ xdp_process_echo_packet(struct xdp_md *xdp, bool dut)
        struct tlv_hdr *tlv;
        struct udphdr *uh;
        __be16 port;
-       __u8 *cmd;
 
        if (eh + 1 > (struct ethhdr *)data_end)
                return -EINVAL;
index 4ad73847b8a5de6227d48d5a5cb091ca2d5b88e7..54cf1765118b800c57788bfdb95d8553f1560ce6 100644 (file)
@@ -89,7 +89,6 @@ static __always_inline int icmp_check(struct xdp_md *ctx, int type)
 SEC("xdp")
 int xdping_client(struct xdp_md *ctx)
 {
-       void *data_end = (void *)(long)ctx->data_end;
        void *data = (void *)(long)ctx->data;
        struct pinginfo *pinginfo = NULL;
        struct ethhdr *eth = data;
@@ -153,7 +152,6 @@ int xdping_client(struct xdp_md *ctx)
 SEC("xdp")
 int xdping_server(struct xdp_md *ctx)
 {
-       void *data_end = (void *)(long)ctx->data_end;
        void *data = (void *)(long)ctx->data;
        struct ethhdr *eth = data;
        struct icmphdr *icmph;
index 7a891a0c3a39cfb74dbb45e6cd3076744e09a507..c2dd0c28237a2f2ddce4efd2c0ca755520c24a65 100644 (file)
@@ -321,7 +321,6 @@ int edgewall(struct xdp_md *ctx)
        void *data = (void *)(long)(ctx->data);
        struct fw_match_info match_info = {};
        struct pkt_info info = {};
-       __u8 parse_err = NO_ERR;
        void *transport_hdr;
        struct ethhdr *eth;
        bool filter_res;