On Wed, Apr 20, 2022 at 9:44 AM Xu Kuohai <xukuohai@xxxxxxxxxx> wrote: > > On 4/16/2022 9:57 AM, Xu Kuohai wrote: > > On 4/16/2022 1:12 AM, Andrii Nakryiko wrote: > >> On Thu, Apr 14, 2022 at 9:10 AM Xu Kuohai <xukuohai@xxxxxxxxxx> wrote: > >>> > >>> Add bpf trampoline support for arm64. Most of the logic is the same as > >>> x86. > >>> > >>> fentry before bpf trampoline hooked: > >>> mov x9, x30 > >>> nop > >>> > >>> fentry after bpf trampoline hooked: > >>> mov x9, x30 > >>> bl <bpf_trampoline> > >>> > >>> Tested on qemu, result: > >>> #55 fentry_fexit:OK > >>> #56 fentry_test:OK > >>> #58 fexit_sleep:OK > >>> #59 fexit_stress:OK > >>> #60 fexit_test:OK > >>> #67 get_func_args_test:OK > >>> #68 get_func_ip_test:OK > >>> #101 modify_return:OK > >>> > >>> Signed-off-by: Xu Kuohai <xukuohai@xxxxxxxxxx> > >>> Acked-by: Song Liu <songliubraving@xxxxxx> > >>> --- > >> > >> Can you please also take a look at [0], which is an ongoing work to > >> add support for BPF cookie to BPF trampoline-based BPF programs. It's > >> very close to being done, so it would be good if you can implement > >> that at the same time. > > > > OK, I'll take a look and try to implemnt it. > > already implemented, but there are some conflicts between these two > series, will send v3 after trampoline cookie are merged. Awesome work, Thanks for doing this!