On Tue, Sep 19, 2023 at 10:31:56PM -0700, Song Liu wrote:
SNIP
> +int arch_bpf_trampoline_size(const struct btf_func_model *m, u32 flags,
> + struct bpf_tramp_links *tlinks, void *func_addr)
> +{
> + struct bpf_tramp_image im;
> + void *image;
> + int ret;
> +
> + /* Allocate a temporary buffer for __arch_prepare_bpf_trampoline().
> + * This will NOT cause fragmentation in direct map, as we do not
> + * call set_memory_*() on this buffer.
> + */
> + image = bpf_jit_alloc_exec(PAGE_SIZE);
> + if (!image)
> + return -ENOMEM;
> +
> + ret = __arch_prepare_bpf_trampoline(&im, image, image + PAGE_SIZE, m, flags,
> + tlinks, func_addr);
> + bpf_jit_free_exec(image);
> + if (ret < 0)
> + return ret;
> +
> + return ret;
this can be just 'return ret;'
jirka
> +}
> +
> static int emit_bpf_dispatcher(u8 **pprog, int a, int b, s64 *progs, u8 *image, u8 *buf)
> {
> u8 *jg_reloc, *prog = *pprog;
> diff --git a/include/linux/bpf.h b/include/linux/bpf.h
> index 548f3ef34ba1..5bbac549b0a0 100644
> --- a/include/linux/bpf.h
> +++ b/include/linux/bpf.h
> @@ -1087,6 +1087,8 @@ void *arch_alloc_bpf_trampoline(int size);
> void arch_free_bpf_trampoline(void *image, int size);
> void arch_protect_bpf_trampoline(void *image, int size);
> void arch_unprotect_bpf_trampoline(void *image, int size);
> +int arch_bpf_trampoline_size(const struct btf_func_model *m, u32 flags,
> + struct bpf_tramp_links *tlinks, void *func_addr);
>
> u64 notrace __bpf_prog_enter_sleepable_recur(struct bpf_prog *prog,
> struct bpf_tramp_run_ctx *run_ctx);
> diff --git a/kernel/bpf/trampoline.c b/kernel/bpf/trampoline.c
> index 5509bdf98067..285c5b7c1ea4 100644
> --- a/kernel/bpf/trampoline.c
> +++ b/kernel/bpf/trampoline.c
> @@ -1070,6 +1070,12 @@ void __weak arch_unprotect_bpf_trampoline(void *image, int size)
> set_memory_rw((long)image, 1);
> }
>
> +int __weak arch_bpf_trampoline_size(const struct btf_func_model *m, u32 flags,
> + struct bpf_tramp_links *tlinks, void *func_addr)
> +{
> + return -ENOTSUPP;
> +}
> +
> static int __init init_trampolines(void)
> {
> int i;
> --
> 2.34.1
>
>
