Re: [Devel] [PATCH vz10 1/4] selftests/bpf: Use local type for bpf_fou_encap in test_tunnel_kern

["Eva Kurchatova (Virtuozzo)" <eva.kurchatova@virtuozzo.com>]

On 8/21/26 18:09, Eva Kurchatova wrote:
> From: Gregory Bell <grbell@redhat.com>
>
> Replace the forward-declared struct bpf_fou_encap with the existing
> bpf_fou_encap___local type in the bpf_skb_set_fou_encap and
> bpf_skb_get_fou_encap declarations. This removes the need for
> the forward declaration and the explicit casts at each call.
>
> Fixes: d17f9b370df6 ("selftests/bpf: Fix compilation failure when CONFIG_NET_FOU!=y")
> Signed-off-by: Gregory Bell <grbell@redhat.com>
> Link: https://lore.kernel.org/r/20260417154122.2558890-3-grbell@redhat.com
> Signed-off-by: Alexei Starovoitov <ast@kernel.org>
> (cherry picked from commit afb0450be061907a0f5d36bd8b010ca30eda3d3b)
>
> https://virtuozzo.atlassian.net/browse/VSTOR-139677
> Feature: fix selftests
> Signed-off-by: Eva Kurchatova <eva.kurchatova@virtuozzo.com>
> ---
>   .../testing/selftests/bpf/progs/test_tunnel_kern.c  | 13 ++++++-------
>   1 file changed, 6 insertions(+), 7 deletions(-)
>
> diff --git a/tools/testing/selftests/bpf/progs/test_tunnel_kern.c b/tools/testing/selftests/bpf/progs/test_tunnel_kern.c
> index 32127f1cd687..30f1de458669 100644
> --- a/tools/testing/selftests/bpf/progs/test_tunnel_kern.c
> +++ b/tools/testing/selftests/bpf/progs/test_tunnel_kern.c
> @@ -6,6 +6,7 @@
>    * modify it under the terms of version 2 of the GNU General Public
>    * License as published by the Free Software Foundation.
>    */
> +#define BPF_NO_KFUNC_PROTOTYPES
>   #include "vmlinux.h"
>   #include <bpf/bpf_core_read.h>
>   #include <bpf/bpf_helpers.h>
> @@ -36,12 +37,10 @@ enum bpf_fou_encap_type___local {
>   	FOU_BPF_ENCAP_GUE___local,
>   };
>   
> -struct bpf_fou_encap;
> -
>   int bpf_skb_set_fou_encap(struct __sk_buff *skb_ctx,
> -			  struct bpf_fou_encap *encap, int type) __ksym;
> +			  struct bpf_fou_encap___local *encap, int type) __ksym;
>   int bpf_skb_get_fou_encap(struct __sk_buff *skb_ctx,
> -			  struct bpf_fou_encap *encap) __ksym;
> +			  struct bpf_fou_encap___local *encap) __ksym;
>   struct xfrm_state *
>   bpf_xdp_get_xfrm_state(struct xdp_md *ctx, struct bpf_xfrm_state_opts *opts,
>   		       u32 opts__sz) __ksym;
> @@ -781,7 +780,7 @@ int ipip_gue_set_tunnel(struct __sk_buff *skb)
>   	encap.sport = 0;
>   	encap.dport = bpf_htons(5555);
>   
> -	ret = bpf_skb_set_fou_encap(skb, (struct bpf_fou_encap *)&encap,
> +	ret = bpf_skb_set_fou_encap(skb, &encap,
>   				    bpf_core_enum_value(enum bpf_fou_encap_type___local,
>   							FOU_BPF_ENCAP_GUE___local));
>   	if (ret < 0) {
> @@ -820,7 +819,7 @@ int ipip_fou_set_tunnel(struct __sk_buff *skb)
>   	encap.sport = 0;
>   	encap.dport = bpf_htons(5555);
>   
> -	ret = bpf_skb_set_fou_encap(skb, (struct bpf_fou_encap *)&encap,
> +	ret = bpf_skb_set_fou_encap(skb, &encap,
>   				    FOU_BPF_ENCAP_FOU___local);
>   	if (ret < 0) {
>   		log_err(ret);
> @@ -843,7 +842,7 @@ int ipip_encap_get_tunnel(struct __sk_buff *skb)
>   		return TC_ACT_SHOT;
>   	}
>   
> -	ret = bpf_skb_get_fou_encap(skb, (struct bpf_fou_encap *)&encap);
> +	ret = bpf_skb_get_fou_encap(skb, &encap);
>   	if (ret < 0) {
>   		log_err(ret);
>   		return TC_ACT_SHOT;

Sent v2 patchset: 
https://lists.openvz.org/pipermail/devel/2026-September/084967.html

Added "ms/" prefix for upstream cherry-pick, split the upstreamable 
commit from commit to test_sock.c and test_tunnel.sh which are gone 
upstream.

_______________________________________________
Devel mailing list
Devel@openvz.org
https://lists.openvz.org/mailman/listinfo/devel