On Thu, Apr 15, 2021 at 11:56 PM Jiri Olsa <jolsa@xxxxxxxxxx> wrote: > > On Thu, Apr 15, 2021 at 04:45:24PM -0700, Alexei Starovoitov wrote: > > On Wed, Apr 14, 2021 at 12:52 PM Jiri Olsa <jolsa@xxxxxxxxxx> wrote: > > > > > > hi, > > > while adding test for pinning the module while there's > > > trampoline attach to it, I noticed that we don't allow > > > link detach and following re-attach for trampolines. > > > Adding that for tracing and lsm programs. > > > > > > You need to have patch [1] from bpf tree for test module > > > attach test to pass. > > > > > > v5 changes: > > > - fixed missing hlist_del_init change > > > - fixed several ASSERT calls > > > - added extra patch for missing ';' > > > - added ASSERT macros to lsm test > > > - added acks > > > > It doesn't work: > > hi, > I got the same warning when running test without the > patch [1] I mentioned: > 861de02e5f3f bpf: Take module reference for trampoline in module > > I still don't see it in bpf-next/master Finally applied to bpf-next. Thank you and sorry for the delay.
