On Wed, Jan 3, 2024 at 11:06 AM Dmitrii Dolgov <9erthalion6@xxxxxxxxx> wrote:
>
[...]
> +
> +/*
> + * Dummy fentry bpf prog for testing fentry attachment chains
> + */
Comment style. This could fit in one line.
> +SEC("fentry/XXX")
> +int BPF_PROG(recursive_attach, int a)
> +{
> + return 0;
> +}
> diff --git a/tools/testing/selftests/bpf/progs/fentry_recursive_target.c b/tools/testing/selftests/bpf/progs/fentry_recursive_target.c
> new file mode 100644
> index 000000000000..6e0b5c716f8e
> --- /dev/null
> +++ b/tools/testing/selftests/bpf/progs/fentry_recursive_target.c
> @@ -0,0 +1,17 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/* Copyright (c) 2023 Red Hat, Inc. */
> +#include <linux/bpf.h>
> +#include <bpf/bpf_helpers.h>
> +#include <bpf/bpf_tracing.h>
> +
> +char _license[] SEC("license") = "GPL";
> +
> +/*
> + * Dummy fentry bpf prog for testing fentry attachment chains. It's going to be
> + * a start of the chain.
> + */
Comment style. I guess we don't need to respin the set just for this.
> +SEC("fentry/bpf_testmod_fentry_test1")
> +int BPF_PROG(test1, int a)
> +{
> + return 0;
> +}
> --
> 2.41.0
>
